From: Alex Williamson <alex.williamson@hp.com>
To: Mark McLoughlin <markmc@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: [PATCH][RFC] qemu:virtio-net: Use TUNSETTXFILTER for MAC filtering
Date: Fri, 06 Feb 2009 11:09:45 -0700 [thread overview]
Message-ID: <1233943785.7026.1353.camel@lappy> (raw)
In-Reply-To: <1233906439.3673.20.camel@blaa>
On Fri, 2009-02-06 at 07:47 +0000, Mark McLoughlin wrote:
> On Thu, 2009-02-05 at 21:51 -0700, Alex Williamson wrote:
> > Here are a couple issues I'm still pondering:
> > - Is the fd_rx_filter() interface sufficiently generic
> > - Should vlan_set_hw_rx_filter() live in net.c or elsewhere
> > - Is it ok to call fd_rx_filter() against all the vlan clients. I
> > exit on the first one, which covers the simple config.
>
> All of these are related to the problem that we're trying to add
> optimizations for the simple config (i.e. one NIC associated with one
> tap interface) without any clean representation of the simple config in
> the API.
>
> It's not strictly needed in this case - the VLAN API for setting filters
> could aggregate the filters of all VLAN clients - but what you really
> want is to only have the filter apply if virtio-net is paired with a tap
> interface.
Yes, or more precisely, it should only be applied if there are exactly
two devices on the vlan, and the "other" device implements fd_rx_filter.
I'll enforce that in the code. Thanks,
Alex
next prev parent reply other threads:[~2009-02-06 18:09 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-06 4:51 [PATCH][RFC] qemu:virtio-net: Use TUNSETTXFILTER for MAC filtering Alex Williamson
2009-02-06 4:51 ` [Qemu-devel] " Alex Williamson
2009-02-06 7:47 ` [Qemu-devel] " Mark McLoughlin
2009-02-06 18:09 ` Alex Williamson [this message]
2009-02-06 13:59 ` Anthony Liguori
2009-02-06 13:59 ` [Qemu-devel] " Anthony Liguori
2009-02-06 18:06 ` Alex Williamson
2009-02-06 18:06 ` [Qemu-devel] " Alex Williamson
2009-02-06 15:12 ` [Qemu-devel] " Paul Brook
2009-02-06 15:12 ` Paul Brook
2009-02-06 17:59 ` Alex Williamson
2009-02-06 17:59 ` Alex Williamson
2009-02-06 22:16 ` Paul Brook
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1233943785.7026.1353.camel@lappy \
--to=alex.williamson@hp.com \
--cc=markmc@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.