[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] vhost-user-test: add multiple queue test
From: |
Yuanhan Liu |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] vhost-user-test: add multiple queue test |
Date: |
Fri, 16 Oct 2015 15:33:32 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, Oct 16, 2015 at 10:20:08AM +0300, Michael S. Tsirkin wrote:
> On Fri, Oct 16, 2015 at 03:19:01PM +0800, Yuanhan Liu wrote:
> > Setting VHOST_USER_PROTOCOL_F_MQ protocol feature bit to claim that we
> > support MQ feature, and simply assume we support 2 queue pairs at most.
>
> Well - let's add a macro, or something?
Yeah, a macro is better.
--yliu
>
> >
> > Cc: Michael S. Tsirkin <address@hidden>
> > Cc: Jason Wang <address@hidden>
> > Signed-off-by: Yuanhan Liu <address@hidden>
> > ---
> > tests/vhost-user-test.c | 18 ++++++++++++++++--
> > 1 file changed, 16 insertions(+), 2 deletions(-)
> >
> > diff --git a/tests/vhost-user-test.c b/tests/vhost-user-test.c
> > index f181391..f0aa36f 100644
> > --- a/tests/vhost-user-test.c
> > +++ b/tests/vhost-user-test.c
> > @@ -33,7 +33,7 @@
> > #define QEMU_CMD_MEM " -m 512 -object
> > memory-backend-file,id=mem,size=512M,"\
> > "mem-path=%s,share=on -numa node,memdev=mem"
> > #define QEMU_CMD_CHR " -chardev socket,id=chr0,path=%s"
> > -#define QEMU_CMD_NETDEV " -netdev
> > vhost-user,id=net0,chardev=chr0,vhostforce"
> > +#define QEMU_CMD_NETDEV " -netdev
> > vhost-user,id=net0,chardev=chr0,vhostforce,queues=2"
> > #define QEMU_CMD_NET " -device virtio-net-pci,netdev=net0 "
> > #define QEMU_CMD_ROM " -option-rom ../pc-bios/pxe-virtio.rom"
> >
> > @@ -48,6 +48,8 @@
> >
> > #define VHOST_USER_F_PROTOCOL_FEATURES 30
> >
> > +#define VHOST_USER_PROTOCOL_F_MQ 0
> > +
> > typedef enum VhostUserRequest {
> > VHOST_USER_NONE = 0,
> > VHOST_USER_GET_FEATURES = 1,
> > @@ -66,6 +68,8 @@ typedef enum VhostUserRequest {
> > VHOST_USER_SET_VRING_ERR = 14,
> > VHOST_USER_GET_PROTOCOL_FEATURES = 15,
> > VHOST_USER_SET_PROTOCOL_FEATURES = 16,
> > + VHOST_USER_GET_QUEUE_NUM = 17,
> > + VHOST_USER_SET_VRING_ENABLE = 18,
> > VHOST_USER_MAX
> > } VhostUserRequest;
> >
> > @@ -232,7 +236,7 @@ static void chr_read(void *opaque, const uint8_t *buf,
> > int size)
> > /* send back features to qemu */
> > msg.flags |= VHOST_USER_REPLY_MASK;
> > msg.size = sizeof(m.u64);
> > - msg.u64 = 0;
> > + msg.u64 = (1ULL << VHOST_USER_PROTOCOL_F_MQ);
> > p = (uint8_t *) &msg;
> > qemu_chr_fe_write_all(chr, p, VHOST_USER_HDR_SIZE + msg.size);
> > break;
> > @@ -266,6 +270,16 @@ static void chr_read(void *opaque, const uint8_t *buf,
> > int size)
> > */
> > qemu_set_nonblock(fd);
> > break;
> > +
> > + case VHOST_USER_GET_QUEUE_NUM:
> > + /* send back the number of queues we support (let it be 2) to
> > qemu */
> > + msg.flags |= VHOST_USER_REPLY_MASK;
> > + msg.size = sizeof(m.u64);
> > + msg.u64 = 2;
> > + p = (uint8_t *) &msg;
> > + qemu_chr_fe_write_all(chr, p, VHOST_USER_HDR_SIZE + msg.size);
> > + break;
> > +
> > default:
> > break;
> > }
> > --
> > 1.9.0
- [Qemu-devel] [PATCH 1/3] Revert "vhost: rename VHOST_RESET_OWNER to VHOST_RESET_DEVICE", Yuanhan Liu, 2015/10/16
- [Qemu-devel] [PATCH 3/3] vhost-user-test: add multiple queue test, Yuanhan Liu, 2015/10/16
- [Qemu-devel] [PATCH 2/3] doc: vhost-user: request naming fix, Yuanhan Liu, 2015/10/16
- Re: [Qemu-devel] [PATCH 1/3] Revert "vhost: rename VHOST_RESET_OWNER to VHOST_RESET_DEVICE", Michael S. Tsirkin, 2015/10/16
- Re: [Qemu-devel] [PATCH 1/3] Revert "vhost: rename VHOST_RESET_OWNER to VHOST_RESET_DEVICE", Yuanhan Liu, 2015/10/21
- Re: [Qemu-devel] [PATCH 1/3] Revert "vhost: rename VHOST_RESET_OWNER to VHOST_RESET_DEVICE", Michael S. Tsirkin, 2015/10/21
- Re: [Qemu-devel] [PATCH 1/3] Revert "vhost: rename VHOST_RESET_OWNER to VHOST_RESET_DEVICE", Yuanhan Liu, 2015/10/21