qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCHv2 0/2] virtio-net: mq feature fixes


From: Michael S. Tsirkin
Subject: Re: [Qemu-devel] [PATCHv2 0/2] virtio-net: mq feature fixes
Date: Thu, 7 Feb 2013 16:00:41 +0200

On Thu, Feb 07, 2013 at 02:54:19PM +0100, Stefan Hajnoczi wrote:
> On Thu, Feb 07, 2013 at 11:26:03AM +0200, Michael S. Tsirkin wrote:
> > mq feature is not needed: we can look at the number of queues and set
> > the flag accordingly.
> > Removing this feature removes ambiguity (what does it mean to have
> > queues=2 with mq=off?).
> > 
> > Changes from v1:
> > - split the patch so each one can be applied separately
> > - add comments as suggested by mjt
> > 
> > Signed-off-by: Michael S. Tsirkin <address@hidden>
> > 
> > 
> > Michael S. Tsirkin (2):
> >   virtio-net: remove mq feature
> >   virtio-net: work around for windows driver bug
> > 
> >  hw/pc_piix.c    |  4 ----
> >  hw/virtio-net.c | 19 ++++++++++++++++---
> >  hw/virtio-net.h | 10 ++++++----
> >  3 files changed, 22 insertions(+), 11 deletions(-)
> > 
> > -- 
> > MST
> 
> Happy although there is overlap with Jesse's patch in fixing the config
> size issue.
> 
> Reviewed-by: Stefan Hajnoczi <address@hidden>

Right, that's why I split them up.  If the one by Jesse is applied the
second patch in the series can be dropped.

-- 
MST



reply via email to

[Prev in Thread] Current Thread [Next in Thread]