[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/7] virtio-net: Filter cleanup/improvements
From: |
Alex Williamson |
Subject: |
Re: [Qemu-devel] [PATCH 0/7] virtio-net: Filter cleanup/improvements |
Date: |
Fri, 12 Jun 2009 13:19:19 -0600 |
On Fri, 2009-06-12 at 18:07 +0100, Mark McLoughlin wrote:
> On Wed, 2009-06-10 at 16:21 +0930, Rusty Russell wrote:
> > On Wed, 10 Jun 2009 04:55:10 am Mark McLoughlin wrote:
> > > Hi Alex,
> > >
> > > On Fri, 2009-06-05 at 14:46 -0600, Alex Williamson wrote:
> > > > This series cleans up a few things around packet filtering. I've
> > > > probably gone a little overboard on breaking up patches, if we want to
> > > > avoid bumping the save version_id so much, these could be mostly lumped
> > > > together. The main features here are more efficient handling of the
> > > > filtering between unicast and multicast, better overflow tracking,
> > > > adding
> > > > more RX modes, and increasing the size of the filter table and control
> > > > queue depth.
> > >
> > > The whole series looks good to me. Please make sure to send Rusty a
> > > patch with the VIRTIO_NET_F_CTRL_RX_EXTRA etc. header additions.
> >
> > Remain unconvinced that it's worth a feature bit. It'd pretty obscure
> > (hell,
> > even multicast is pretty obscure IRL).
>
> Yeah, we probably don't need the feature bit - the guest can just handle
> a VIRTIO_NET_ERR status if the host doesn't implement it.
>
> Alex - if you agree, could you post a qemu patch to kill the
> CTRL_RX_EXTRA feature? And also post a linux/virtio_net.h patch to add
> the new rx modes?
Two votes against it, I'll drop the feature bit. I'll post the qemu
patch in a minute and follow-up with the linux patch once that goes in.
Thanks,
Alex
- Re: [Qemu-devel] [PATCH 6/7] virtio-net: Add new RX filter controls, (continued)