On Mon, 2011-11-14 at 13:52 -0800, Rick Jones wrote: > From: Rick Jones <rick.jones2@xxxxxx> > > Add a new .bus_name to virtio_config_ops then modify virtio_net to > call through to it in an ethtool .get_drvinfo routine to report > bus_info in ethtool -i output which is consistent with other > emulated NICs and the output of lspci. [...] > diff --git a/drivers/lguest/lguest_device.c b/drivers/lguest/lguest_device.c > index 0dc30ff..3724d45 100644 > --- a/drivers/lguest/lguest_device.c > +++ b/drivers/lguest/lguest_device.c > @@ -381,6 +381,11 @@ error: > return PTR_ERR(vqs[i]); > } > > +static const char *lg_bus_name(struct virtio_device *vdev) > +{ > + return "Not Implemented"; > +} [...] > +static const char *kvm_bus_name(struct virtio_device *vdev) > +{ > + return "Not Implemented"; > +} [...] Please use the existing 'not implemented' value, which is the empty string. If you think ethtool should print some helpful message instead of an empty string, please submit a patch for ethtool. Ben. -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked. _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linuxfoundation.org/mailman/listinfo/virtualization