On Tue, Dec 10, 2013 at 05:24:29PM +0200, Michael S. Tsirkin wrote: > On Mon, Dec 09, 2013 at 04:16:32PM -0800, Stephen Hemminger wrote: > > The virtio_net driver never sends the multicast address list to > > the host. This is because send command takes a pointer to scatter list > > to send but only inserts that one entry into the outgoing scatter list. > > > > This bug has been there since: > > commit f565a7c259d71cc186753653d978c646d2354b36 > > Author: Alex Williamson <alex.williamson@xxxxxx> > > Date: Wed Feb 4 09:02:45 2009 +0000 > > > > virtio_net: Add a MAC filter table > > > > Signed-off-by: Stephen Hemminger <stephen@xxxxxxxxxxxxxxxxxx> > > Acked-by: Michael S. Tsirkin <mst@xxxxxxxxxx> > > Question is, can we detect a broken driver in the host somehow? > If not we'll need a feature bit for hosts to actually do > MC filtering. Hmm Vlad and Amos made some good points about this patch. Dave, pls don't apply yet until these are resolved. > > --- a/drivers/net/virtio_net.c 2013-12-09 16:12:03.897891975 -0800 > > +++ b/drivers/net/virtio_net.c 2013-12-09 16:12:36.353164803 -0800 > > @@ -893,7 +893,7 @@ static bool virtnet_send_command(struct > > sg_init_one(&hdr, &ctrl, sizeof(ctrl)); > > sgs[out_num++] = &hdr; > > > > - if (out) > > + for (; out; out = sg_next(out)) > > sgs[out_num++] = out; > > if (in) > > sgs[out_num + in_num++] = in; _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linuxfoundation.org/mailman/listinfo/virtualization
