[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] virtio-net: Remove RX_EXTRA feature
From: |
Rusty Russell |
Subject: |
Re: [Qemu-devel] [PATCH] virtio-net: Remove RX_EXTRA feature |
Date: |
Mon, 15 Jun 2009 14:48:13 +0930 |
User-agent: |
KMail/1.11.2 (Linux/2.6.28-11-generic; KDE/4.2.2; i686; ; ) |
On Sun, 14 Jun 2009 11:01:37 pm Alex Williamson wrote:
> On Sun, Jun 14, 2009 at 3:39 AM, Michael S. Tsirkin<address@hidden> wrote:
> > On Fri, Jun 12, 2009 at 01:17:04PM -0600, Alex Williamson wrote:
> >> No need to use a feature bit to identify the RX modes. The guest
> >> will get an error back if the backend doesn't support these controls.
> >>
> >> Signed-off-by: Alex Williamson <address@hidden>
> >
> > What's the motivation here - are we running out of feature bits already?
> > Using feature bits for everything makes the interface more consistent
> > and IMO easier to use.
>
> See the thread starting here:
>
> http://lists.gnu.org/archive/html/qemu-devel/2009-06/msg00519.html
>
> Both Rusty and Mark voted against the feature bit.
Not quite. I was unclear. I support the use of a feature bit. My question
is more meta.
Features do need justification. (Not much, but some). It's hard to see the
use case where this feature offers significant advantages over promisc.
Without
that, I'd wait until someone reports a need.
Thanks,
Rusty.