Re: [PATCH 0/3 RFC] macvtap: Implement getPhysfn to support sending a port profile message for a VF to its PF

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 8/2/11 4:15 AM, "Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote:

> 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
> 
Yes, I looked at it and saw that I had to move quite a few node_device data
structures and code in src/util/pci.c. Wasn't sure if there was a reason for
it to not already exist in src/util/pci.c. So decided to post this patch as
RFC for specific directions.

>From the patches that I posted, I still need the stuff that tries to derive
the net device from the pci device. I will re-evaluate moving the node
device stuff to get PF-VF relationship information into src/util/pci.c and
repost the patches.

Thanks for the comments.

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]