qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Mark McLoughlin <markmc@redhat.com>
To: Alex Williamson <alex.williamson@hp.com>
Cc: qemu-devel@nongnu.org, kvm@vger.kernel.org
Subject: [Qemu-devel] Re: [PATCH 0/7] qemu:virtio-net: Add MAC and VLAN filtering
Date: Mon, 19 Jan 2009 09:45:36 +0000	[thread overview]
Message-ID: <1232358336.5627.21.camel@blaa> (raw)
In-Reply-To: <20090116210954.16725.44321.stgit@kvm.aw>

On Fri, 2009-01-16 at 14:09 -0700, Alex Williamson wrote:
> This series adds the ability for the guest to set the virtio-net device
> MAC address, a new control virtqueue for setting configuration data from
> the guest, and new interfaces making use of the control virtqueue for
> setting RX mode options, MAC filter table entries, and VLAN filter bits.
> The end result is that the virtio-net device has more of the features of
> a realy hardware NIC and can be configured to drop packets the guest
> isn't interested in seeing.
> 
> This version incorporates the review comments from Mark McLoughlin,
> particularly including much better commit logs, factoring control
> commands into separate functions, and making a local ETH_ALEN define.
> Also new in this version is the addition of VLAN filtering.  Please
> comment and/or apply.  Thanks,
> 
> Alex
> 
> Note - This series depends on the previous patch sent to correct
> save and load of the virtio-net state.

I think it makes sense to wait for the linux side to be accepted first.
It's there the virtio guest ABI is defined, really. Apart from that, the
whole series looks good to me:

Acked-by: Mark McLoughlin <markmc@redhat.com>

Cheers,
Mark.

  parent reply	other threads:[~2009-01-19  9:45 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-16 21:09 [Qemu-devel] [PATCH 0/7] qemu:virtio-net: Add MAC and VLAN filtering Alex Williamson
2009-01-16 21:09 ` [Qemu-devel] [PATCH 1/7] qemu:virtio-net: Allow setting the MAC address via set_config Alex Williamson
2009-01-16 21:10 ` [Qemu-devel] [PATCH 2/7] qemu:virtio-net: Define ETH_ALEN for use when manipulating MAC addresses Alex Williamson
2009-01-16 21:10 ` [Qemu-devel] [PATCH 3/7] qemu:virtio-net: Add a virtqueue for control commands from the guest Alex Williamson
2009-01-16 21:10 ` [Qemu-devel] [PATCH 4/7] qemu:virtio-net: Add promiscuous and all-multicast mode bits Alex Williamson
2009-01-16 21:10 ` [Qemu-devel] [PATCH 5/7] qemu:virtio-net: Enable filtering based on MAC, promisc, broadcast and allmulti Alex Williamson
2009-01-20 21:31   ` Alex Williamson
2009-01-16 21:10 ` [Qemu-devel] [PATCH 6/7] qemu:virtio-net: Add additional MACs via a filter table Alex Williamson
2009-01-20 21:34   ` Alex Williamson
2009-01-16 21:10 ` [Qemu-devel] [PATCH 7/7] qemu:virtio-net: Add VLAN filtering Alex Williamson
2009-01-20 16:45   ` Alex Williamson
2009-01-20 21:38     ` Alex Williamson
2009-01-19  9:45 ` Mark McLoughlin [this message]
2009-01-20  2:27 ` [Qemu-devel] Re: [PATCH 0/7] qemu:virtio-net: Add MAC and " Anthony Liguori

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=1232358336.5627.21.camel@blaa \
    --to=markmc@redhat.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).