From: Paul Brook <paul@codesourcery.com>
To: qemu-devel@nongnu.org
Cc: kvm@vger.kernel.org, Alex Williamson <alex.williamson@hp.com>
Subject: Re: [Qemu-devel] [PATCH][RFC] qemu:virtio-net: Use TUNSETTXFILTER for MAC filtering
Date: Fri, 6 Feb 2009 15:12:41 +0000 [thread overview]
Message-ID: <200902061512.42261.paul@codesourcery.com> (raw)
In-Reply-To: <20090206044853.3116.46699.stgit@kvm.aw>
On Friday 06 February 2009, Alex Williamson wrote:
> Now that virtio-net knows what packets the guest wants to see, we
> can start moving the filtering down the stack. This patch adds
> an interface to set the software filter in the tap device. It's
> fairly limited, but we can back it up with our own filtering if it
> overflows.
I think you've got the abstraction wrong here. Devices certainly shouldn't
care what else is connected to the vlan.
Remember that a qemu vlan is an arbitrary set of network devices. You need to
be able to support multiple devices all with their own filters.
There should be two interface points between the the vlan and the device:
1) The device can indicate that it only cares about incoming (vlan to device)
packets sent to a subset of MAC addresses.
2) The vlan can request that a device discards outgoing (device to vlan)
packets that don't match a subset of MAC addresses.
Typically a emulated NIC will implement (1) and a host interface will
implement (2).
The generic vlan code should be responsible for using the information provided
by (1) to set (2) appropriately. Remember that network devices can be
hotplugged.
On a related note, you need to add comments making it clear that this is a
performance optimisation, and must not be relied on. A device may still
receive packets that should have been filtered out. If a device needs
reliable filtering (most "real" hardware does), then this still needs to be
implemented per-device.
Paul
next prev parent reply other threads:[~2009-02-06 15:12 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-06 4:51 [Qemu-devel] [PATCH][RFC] qemu:virtio-net: Use TUNSETTXFILTER for MAC filtering Alex Williamson
2009-02-06 7:47 ` [Qemu-devel] " Mark McLoughlin
2009-02-06 18:09 ` Alex Williamson
2009-02-06 13:59 ` Anthony Liguori
2009-02-06 18:06 ` Alex Williamson
2009-02-06 15:12 ` Paul Brook [this message]
2009-02-06 17:59 ` [Qemu-devel] " 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=200902061512.42261.paul@codesourcery.com \
--to=paul@codesourcery.com \
--cc=alex.williamson@hp.com \
--cc=kvm@vger.kernel.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).