Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757172AbYJNDJw (ORCPT ); Mon, 13 Oct 2008 23:09:52 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755574AbYJNDJl (ORCPT ); Mon, 13 Oct 2008 23:09:41 -0400 Received: from mga02.intel.com ([134.134.136.20]:41226 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755404AbYJNDJk (ORCPT ); Mon, 13 Oct 2008 23:09:40 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.33,406,1220252400"; d="scan'208";a="450870768" Date: Tue, 14 Oct 2008 10:14:35 +0800 From: Yu Zhao To: Matthew Wilcox Cc: eddie.dong@intel.com, "linux-pci@vger.kernel.org" , Jesse Barnes , Randy Dunlap , Grant Grundler , Alex Chiang , Roland Dreier , Greg KH , "linux-kernel@vger.kernel.org" , "kvm@vger.kernel.org" , "virtualization@lists.linux-foundation.org" Subject: Re: [PATCH 6/6 v3] PCI: document the change Message-ID: <20081014021435.GA1482@yzhao12-linux.sh.intel.com> References: <20081001160706.GI13822@parisc-linux.org> <08DF4D958216244799FC84F3514D70F00235CC69@pdsmsx415.ccr.corp.intel.com> <20081014010827.GX25780@parisc-linux.org> <08DF4D958216244799FC84F3514D70F00235CE27@pdsmsx415.ccr.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <08DF4D958216244799FC84F3514D70F00235CE27@pdsmsx415.ccr.corp.intel.com> User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2923 Lines: 66 On Tue, Oct 14, 2008 at 10:31:03AM +0800, Dong, Eddie wrote: > Matthew Wilcox wrote: > > On Tue, Oct 14, 2008 at 08:23:34AM +0800, Dong, Eddie > > wrote: > >> Matthew Wilcox wrote: > >>> Wouldn't it be more useful to have the iov/N directories > >>> be a symlink to the actual pci_dev used by the virtual > >>> function? > >> > >> The main concern here is that a VF may be disabed such > >> as when PF enter D3 state or undergo an reset and thus > >> be plug-off, but user won't re-configure the VF in case > >> the PF return back to working state. > > > > If we're relying on the user to reconfigure virtual > > functions on return to D0 from D3, that's a very fragile > > system. > > No. that is the concern we don't put those configuration under VF nodes because it will disappear. > Do I miss something? > > > > >>>> +For network device, there are: > >>>> + - /sys/bus/pci/devices/BB:DD.F/iov/N/mac > >>>> + - /sys/bus/pci/devices/BB:DD.F/iov/N/vlan > >>>> + (value update will notify PF driver) > >>> > >>> We already have tools to set the MAC and VLAN parameters > >>> for network devices. > >> > >> Do you mean Ethtool? If yes, it is impossible for SR-IOV > >> since the configuration has to be done in PF side, > >> rather than VF. > > > > I don't think ethtool has that ability; ip(8) can set mac > > addresses and vconfig(8) sets vlan parameters. > > > > The device driver already has to be aware of SR-IOV. If > > it's going to support the standard tools (and it damn > > well ought to), then it should call the PF driver to set > > these kinds of parameters. > > OK, as if it has the VF parameter, will look into details. Neither ip(8) nor vconfig(8) can set MAC and VLAN address for VF when the VF driver is not loaded. > BTW, the SR-IOV patch is not only for network, some other devices such as IDE will use same code base as well and we image it could have other parameter to set such as starting LBA of a IDE VF. As Eddie said, we have two problems here: 1) User has to set device specific parameters of a VF when he wants to use this VF with KVM (assign this device to KVM guest). In this case, VF driver is not loaded in the host environment. So operations which are implemented as driver callback (e.g. set_mac_address()) are not supported. 2) For security reason, some SR-IOV devices prohibit the VF driver configuring the VF via its own register space. Instead, the configurations must be done through the PF which the VF is associated with. This means PF driver has to receive parameters that are used to configure its VFs. These parameters obviously can be passed by traditional tools, if without modification for SR-IOV. -- 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/