On Mon, Aug 01, 2011 at 09:16:09PM -0400, Dave Allan wrote: > On Mon, Aug 01, 2011 at 04:56:58PM -0700, Roopa Prabhu wrote: > > This patch tries to fix getPhysFn in macvtap.c to get the physical > > function(PF) of the direct attach interface, if the interface is a SR-IOV VF. > > It derives the PF/VF relationship from sysfs. > > > > There is some code to do this in node device driver. But it is local > > to the node device driver . I did not see a clean way to use some of the > > node device stuff here. The node device driver looks at PCI capabilities to > > get the same information. > > We should not have two implementations of this functionality in the > code. Either the node device code should be made to use this version > or vice versa. We already have a file src/util/pci.c that contains a bunch of helper APIs for dealing with PCI devices. If we need some shared code between macvtap and the node device driver, then we ought to put it in the pci.c file Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list