[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v10 7/7] vhost-user: add a new message to disabl
From: |
Yuanhan Liu |
Subject: |
Re: [Qemu-devel] [PATCH v10 7/7] vhost-user: add a new message to disable/enable a specific virt queue. |
Date: |
Wed, 23 Sep 2015 10:12:29 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, Sep 22, 2015 at 08:06:58PM -0600, Eric Blake wrote:
> On 09/22/2015 08:05 PM, Yuanhan Liu wrote:
>
> >>> + * VHOST_USER_SET_VRING_ENABLE
> >>> +
> >>> + Id: 18
> >>> + Equivalent ioctl: N/A
> >>> + Master payload: vring state description
> >>> +
> >>> + Signal slave to enable or disable corresponding vring.
> >>
> >> Does there need to be any QMP control to manually change a given queue,
> >> or is it all used under the hood with no need for management apps to
> >> care other than their initial request of max queues?
> >
> > TBH, I don't know. As far as I know, there is only one queue pair will
> > be enabled by default, and it's user's job to enable (or disable) more
> > queue pairs, say, by ethtool:
> >
> > # ethtool -L eth0 combined <queue pair number>
> >
> > Which ends up sending the VHOST_USER_SET_VRING_ENABLE to actually enable
> > (or disable) a specific queue pairs.
>
> So if I'm understanding, it is ethtool, not qemu, that is used to turn
> on additional queues, and therefore we don't need a QMP command in qemu
> to control things.
I guess so, and that's what Michael told me before.
> > Does that answer your question?
>
> I think so, but I'll let other reviewers more familiar with this area of
> code give a final say.
Michael?
--yliu
- [Qemu-devel] [PATCH v10 2/7] vhost-user: add protocol feature negotiation, (continued)
[Qemu-devel] [PATCH v10 6/7] vhost-user: add multiple queue support, Yuanhan Liu, 2015/09/18
Re: [Qemu-devel] [PATCH v10 6/7] vhost-user: add multiple queue support, Michael S. Tsirkin, 2015/09/22
Re: [Qemu-devel] [PATCH v10 6/7] vhost-user: add multiple queue support, Eric Blake, 2015/09/22
Re: [Qemu-devel] [PATCH v10 6/7] vhost-user: add multiple queue support, Eduardo Habkost, 2015/09/22