Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752682AbYKFRjZ (ORCPT ); Thu, 6 Nov 2008 12:39:25 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1750882AbYKFRjN (ORCPT ); Thu, 6 Nov 2008 12:39:13 -0500 Received: from g4t0016.houston.hp.com ([15.201.24.19]:27972 "EHLO g4t0016.houston.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750806AbYKFRjL convert rfc822-to-8bit (ORCPT ); Thu, 6 Nov 2008 12:39:11 -0500 From: "Fischer, Anna" To: Greg KH , H L CC: "randy.dunlap@oracle.com" , "grundler@parisc-linux.org" , "Chiang, Alexander" , "matthew@wil.cx" , "linux-pci@vger.kernel.org" , "rdreier@cisco.com" , "linux-kernel@vger.kernel.org" , "jbarnes@virtuousgeek.org" , "virtualization@lists.linux-foundation.org" , "kvm@vger.kernel.org" , "mingo@elte.hu" Date: Thu, 6 Nov 2008 17:38:16 +0000 Subject: RE: [PATCH 0/16 v6] PCI: Linux kernel SR-IOV support Thread-Topic: [PATCH 0/16 v6] PCI: Linux kernel SR-IOV support Thread-Index: AclAMThqNWsXL2vXQLeW5e+3VTRvgwAALQAg Message-ID: <0199E0D51A61344794750DC57738F58E5E26F996C4@GVW1118EXC.americas.hpqcorp.net> References: <20081106154351.GA30459@kroah.com> <894107.30288.qm@web45108.mail.sp1.yahoo.com> <20081106164919.GA4099@kroah.com> In-Reply-To: <20081106164919.GA4099@kroah.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: acceptlanguage: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3075 Lines: 43 > On Thu, Nov 06, 2008 at 08:41:53AM -0800, H L wrote: > > I have not modified any existing drivers, but instead I threw > together > > a bare-bones module enabling me to make a call to pci_iov_register() > > and then poke at an SR-IOV adapter's /sys entries for which no driver > > was loaded. > > > > It appears from my perusal thus far that drivers using these new > > SR-IOV patches will require modification; i.e. the driver associated > > with the Physical Function (PF) will be required to make the > > pci_iov_register() call along with the requisite notify() function. > > Essentially this suggests to me a model for the PF driver to perform > > any "global actions" or setup on behalf of VFs before enabling them > > after which VF drivers could be associated. > > Where would the VF drivers have to be associated? On the "pci_dev" > level or on a higher one? A VF appears to the Linux OS as a standard (full, additional) PCI device. The driver is associated in the same way as for a normal PCI device. Ideally, you would use SR-IOV devices on a virtualized system, for example, using Xen. A VF can then be assigned to a guest domain as a full PCI device. > Will all drivers that want to bind to a "VF" device need to be > rewritten? Currently, any vendor providing a SR-IOV device needs to provide a PF driver and a VF driver that runs on their hardware. A VF driver does not necessarily need to know much about SR-IOV but just run on the presented PCI device. You might want to have a communication channel between PF and VF driver though, for various reasons, if such a channel is not provided in hardware. > > I have so far only seen Yu Zhao's "7-patch" set. I've not yet looked > > at his subsequently tendered "15-patch" set so I don't know what has > > changed. The hardware/firmware implementation for any given SR-IOV > > compatible device, will determine the extent of differences required > > between a PF driver and a VF driver. > > Yeah, that's what I'm worried/curious about. Without seeing the code > for such a driver, how can we properly evaluate if this infrastructure > is the correct one and the proper way to do all of this? Yu's API allows a PF driver to register with the Linux PCI code and use it to activate VFs and allocate their resources. The PF driver needs to be modified to work with that API. While you can argue about how that API is supposed to look like, it is clear that such an API is required in some form. The PF driver needs to know when VFs are active as it might want to allocate further (device-specific) resources to VFs or initiate further (device-specific) configurations. While probably a lot of SR-IOV specific code has to be in the PF driver, there is also support required from the Linux PCI subsystem, which is to some extend provided by Yu's patches. Anna -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/