[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20090118.220528.02874092.davem@davemloft.net>
Date: Sun, 18 Jan 2009 22:05:28 -0800 (PST)
From: David Miller <davem@...emloft.net>
To: alex.williamson@...com
Cc: netdev@...r.kernel.org, rusty@...tcorp.com.au, markmc@...hat.com,
kvm@...r.kernel.org
Subject: Re: [PATCH 0/5] virtio_net: Add MAC and VLAN filtering
From: Alex Williamson <alex.williamson@...com>
Date: Fri, 16 Jan 2009 14:13:12 -0700
> This series enables setting the virtio-net device MAC address, adds
> infrastructure for the new control virtqueue, and makes use of it
> to support set_rx_mode, unicast and multicast address lists, and
> supporting a hypervisor based VLAN filter. The goal is to make the
> virtio-net device support more of the features of a physical NIC and
> allow the hypervisor to discard packets we're not interested in.
>
> This version incorporates the review comments from Mark McLoughlin,
> specifically, much improved comments and commit logs, verifying the
> functionality of host not providing a MAC address, moving communication
> structs into virtio_net.h, adding warnings when things don't work, and
> making the strings grep'able. I've left the class/cmd split in the
> control header rather than consolidating it into a single value, I'm
> hoping I've made sufficient arguments for that. Also, I left the
> error return rather than a BUG_ON in send_command because I can't
> conditionally enable set_rx_mode as it's in a const struct. Instead,
> I've changed the caller to avoid the issue. Please comment and/or
> apply. Thanks,
I've been watching these patches passively, and it seems that
there have been some comments that Rusty hasn't submitted
certain virtio_net patches to me as well.
Please sort this all out and let me know what to queue up for
2.6.30
Thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists