From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753444AbeC1Vad (ORCPT ); Wed, 28 Mar 2018 17:30:33 -0400 Received: from mga03.intel.com ([134.134.136.65]:4527 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753343AbeC1Vab (ORCPT ); Wed, 28 Mar 2018 17:30:31 -0400 X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.48,373,1517904000"; d="asc'?scan'208";a="215744699" From: "Rustad, Mark D" To: Alexander Duyck CC: "bhelgaas@google.com" , "Duyck, Alexander H" , "linux-pci@vger.kernel.org" , "virtio-dev@lists.oasis-open.org" , "kvm@vger.kernel.org" , Netdev , "Daly, Dan" , LKML , "linux-nvme@lists.infradead.org" , "Busch, Keith" , "netanel@amazon.com" , "ddutile@redhat.com" , "mheyne@amazon.de" , "Wang, Liang-min" , "dwmw2@infradead.org" , "hch@lst.de" , "dwmw@amazon.co.uk" Subject: Re: [virtio-dev] [pci PATCH v7 1/5] pci: Add pci_sriov_configure_simple for PFs that don't manage VF resources Thread-Topic: [virtio-dev] [pci PATCH v7 1/5] pci: Add pci_sriov_configure_simple for PFs that don't manage VF resources Thread-Index: AQHTvI08Vp52Z4tFxUi8VACDa+E/qaPmtDkA Date: Wed, 28 Mar 2018 21:30:29 +0000 Message-ID: References: <20180315183449.3102.64791.stgit@localhost.localdomain> <20180315184055.3102.2435.stgit@localhost.localdomain> In-Reply-To: <20180315184055.3102.2435.stgit@localhost.localdomain> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: yes X-MS-TNEF-Correlator: x-originating-ip: [10.233.80.200] Content-Type: multipart/signed; boundary="Apple-Mail=_BBED0712-6906-4470-94B1-31A72AA8A265"; protocol="application/pgp-signature"; micalg=pgp-sha256 MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --Apple-Mail=_BBED0712-6906-4470-94B1-31A72AA8A265 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=us-ascii On Mar 15, 2018, at 11:41 AM, Alexander Duyck wrote: > From: Alexander Duyck > > This patch adds a common configuration function called > pci_sriov_configure_simple that will allow for managing VFs on devices > where the PF is not capable of managing VF resources. > > Signed-off-by: Alexander Duyck > --- > > v5: New patch replacing pci_sriov_configure_unmanaged with > pci_sriov_configure_simple > Dropped bits related to autoprobe changes > v6: Defined pci_sriov_configure_simple as NULL if IOV is disabled > v7: Updated pci_sriov_configure_simple to drop need for err value > Fixed comment explaining why pci_sriov_configure_simple is NULL > > drivers/pci/iov.c | 31 +++++++++++++++++++++++++++++++ > include/linux/pci.h | 3 +++ > 2 files changed, 34 insertions(+) Tested with the device identified in patch #2. Tested-by: Mark Rustad -- Mark Rustad, Networking Division, Intel Corporation --Apple-Mail=_BBED0712-6906-4470-94B1-31A72AA8A265 Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="signature.asc" Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Message signed with OpenPGP -----BEGIN PGP SIGNATURE----- Comment: GPGTools - http://gpgtools.org iQIzBAEBCAAdFiEE6ug8b0Wg+ULmnksNPA7/547j7m4FAlq8CXQACgkQPA7/547j 7m6pew/8CP45+0ywjt3/o6XIMCG2lY8I4kEyzMy+qDRd6Enk2kQkcORMhSxNOrwF MXK7CSa01s3Y0AwjHuy+0d+/jDvxyUGuhw0Tb98DkL3T9OQDhgCuTPTC8/wu9XYS AoACL8alz4pouJM/P9VT24pGIkwbEWl0bE/LZ0fZvhOkVjLBp6f3E5aqgcMZlI4V uAEDdP/AHdessoOktzRWxmlEFJdBN/TDkobLM0I+ipI2VIJh+Wu8n2Cb9nT9ZrFz bgOVdwJMloky6CF23XHu/R+vrfmtQJf7/OKpI63it2ryX9g0c2ZKTw3RhqrBCR+c 470ZslFbxfQTMNZnI/+6pU5/mriHtEuN+X/Pyr0UVgjDezjtiId9bwNxbRUsZrvF MvzfHBixgYmtRa+cBguiV48qfkWrfXdmhteqYta786+Tr0zvKX833WAydO+/V6bx ZiBaZEa3LoSuOhGbzRjPxyMIy6nrkVYKvulm/N1G4iTzgs1Lj5heWiH6onW1y32o sLyEYpLVLHAEGVtHEUTxJU4xIbyW5hvY/dUqjOiXnvcIdP7yc+bo9s3kLiuEVn81 r8/VkR1W6+q1r5J/ED9iWEt7D7BkOBdH9THRIQIDq3O1vSzX6m+Pvhw2mjLcsY9U A5ABZsKAF+7gYtNlM4X5bl8tXhzNOJo4pP7zAHTOqffxeOIYD0A= =39ai -----END PGP SIGNATURE----- --Apple-Mail=_BBED0712-6906-4470-94B1-31A72AA8A265-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rustad, Mark D" Subject: Re: [pci PATCH v7 1/5] pci: Add pci_sriov_configure_simple for PFs that don't manage VF resources Date: Wed, 28 Mar 2018 21:30:29 +0000 Message-ID: References: <20180315183449.3102.64791.stgit@localhost.localdomain> <20180315184055.3102.2435.stgit@localhost.localdomain> Mime-Version: 1.0 Content-Type: multipart/signed; boundary="Apple-Mail=_BBED0712-6906-4470-94B1-31A72AA8A265"; protocol="application/pgp-signature"; micalg=pgp-sha256 Cc: "bhelgaas@google.com" , "Duyck, Alexander H" , "linux-pci@vger.kernel.org" , "virtio-dev@lists.oasis-open.org" , "kvm@vger.kernel.org" , Netdev , "Daly, Dan" , LKML , "linux-nvme@lists.infradead.org" , "Busch, Keith" , "netanel@amazon.com" , "ddutile@redhat.com" , "mheyne@amazon.de" , "Wang, Liang-min" , "dwmw2@infradead.org" , "hch@lst.de" , "dwmw@amazon.co.uk" To: Alexander Duyck Return-path: Sender: List-Post: List-Help: List-Unsubscribe: List-Subscribe: In-Reply-To: <20180315184055.3102.2435.stgit@localhost.localdomain> Content-Language: en-US List-Id: netdev.vger.kernel.org --Apple-Mail=_BBED0712-6906-4470-94B1-31A72AA8A265 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=us-ascii On Mar 15, 2018, at 11:41 AM, Alexander Duyck wrote: > From: Alexander Duyck > > This patch adds a common configuration function called > pci_sriov_configure_simple that will allow for managing VFs on devices > where the PF is not capable of managing VF resources. > > Signed-off-by: Alexander Duyck > --- > > v5: New patch replacing pci_sriov_configure_unmanaged with > pci_sriov_configure_simple > Dropped bits related to autoprobe changes > v6: Defined pci_sriov_configure_simple as NULL if IOV is disabled > v7: Updated pci_sriov_configure_simple to drop need for err value > Fixed comment explaining why pci_sriov_configure_simple is NULL > > drivers/pci/iov.c | 31 +++++++++++++++++++++++++++++++ > include/linux/pci.h | 3 +++ > 2 files changed, 34 insertions(+) Tested with the device identified in patch #2. Tested-by: Mark Rustad -- Mark Rustad, Networking Division, Intel Corporation --Apple-Mail=_BBED0712-6906-4470-94B1-31A72AA8A265 Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="signature.asc" Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Message signed with OpenPGP -----BEGIN PGP SIGNATURE----- Comment: GPGTools - http://gpgtools.org iQIzBAEBCAAdFiEE6ug8b0Wg+ULmnksNPA7/547j7m4FAlq8CXQACgkQPA7/547j 7m6pew/8CP45+0ywjt3/o6XIMCG2lY8I4kEyzMy+qDRd6Enk2kQkcORMhSxNOrwF MXK7CSa01s3Y0AwjHuy+0d+/jDvxyUGuhw0Tb98DkL3T9OQDhgCuTPTC8/wu9XYS AoACL8alz4pouJM/P9VT24pGIkwbEWl0bE/LZ0fZvhOkVjLBp6f3E5aqgcMZlI4V uAEDdP/AHdessoOktzRWxmlEFJdBN/TDkobLM0I+ipI2VIJh+Wu8n2Cb9nT9ZrFz bgOVdwJMloky6CF23XHu/R+vrfmtQJf7/OKpI63it2ryX9g0c2ZKTw3RhqrBCR+c 470ZslFbxfQTMNZnI/+6pU5/mriHtEuN+X/Pyr0UVgjDezjtiId9bwNxbRUsZrvF MvzfHBixgYmtRa+cBguiV48qfkWrfXdmhteqYta786+Tr0zvKX833WAydO+/V6bx ZiBaZEa3LoSuOhGbzRjPxyMIy6nrkVYKvulm/N1G4iTzgs1Lj5heWiH6onW1y32o sLyEYpLVLHAEGVtHEUTxJU4xIbyW5hvY/dUqjOiXnvcIdP7yc+bo9s3kLiuEVn81 r8/VkR1W6+q1r5J/ED9iWEt7D7BkOBdH9THRIQIDq3O1vSzX6m+Pvhw2mjLcsY9U A5ABZsKAF+7gYtNlM4X5bl8tXhzNOJo4pP7zAHTOqffxeOIYD0A= =39ai -----END PGP SIGNATURE----- --Apple-Mail=_BBED0712-6906-4470-94B1-31A72AA8A265-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: mark.d.rustad@intel.com (Rustad, Mark D) Date: Wed, 28 Mar 2018 21:30:29 +0000 Subject: [virtio-dev] [pci PATCH v7 1/5] pci: Add pci_sriov_configure_simple for PFs that don't manage VF resources In-Reply-To: <20180315184055.3102.2435.stgit@localhost.localdomain> References: <20180315183449.3102.64791.stgit@localhost.localdomain> <20180315184055.3102.2435.stgit@localhost.localdomain> Message-ID: On Mar 15, 2018,@11:41 AM, Alexander Duyck wrote: > From: Alexander Duyck > > This patch adds a common configuration function called > pci_sriov_configure_simple that will allow for managing VFs on devices > where the PF is not capable of managing VF resources. > > Signed-off-by: Alexander Duyck > --- > > v5: New patch replacing pci_sriov_configure_unmanaged with > pci_sriov_configure_simple > Dropped bits related to autoprobe changes > v6: Defined pci_sriov_configure_simple as NULL if IOV is disabled > v7: Updated pci_sriov_configure_simple to drop need for err value > Fixed comment explaining why pci_sriov_configure_simple is NULL > > drivers/pci/iov.c | 31 +++++++++++++++++++++++++++++++ > include/linux/pci.h | 3 +++ > 2 files changed, 34 insertions(+) Tested with the device identified in patch #2. Tested-by: Mark Rustad -- Mark Rustad, Networking Division, Intel Corporation -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 873 bytes Desc: Message signed with OpenPGP URL: