From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752691AbeCOSkz (ORCPT ); Thu, 15 Mar 2018 14:40:55 -0400 Received: from mail-pf0-f194.google.com ([209.85.192.194]:38277 "EHLO mail-pf0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751508AbeCOSkv (ORCPT ); Thu, 15 Mar 2018 14:40:51 -0400 X-Google-Smtp-Source: AG47ELsMaOs85kY8h1923hZ8fuU1WGWjVUdb9OWoZq6M4+tfKbmq2dZGl4HPpFlZUSH403bCAcdKYw== Subject: [pci PATCH v7 0/5] Add support for unmanaged SR-IOV From: Alexander Duyck To: bhelgaas@google.com, alexander.h.duyck@intel.com, linux-pci@vger.kernel.org Cc: virtio-dev@lists.oasis-open.org, kvm@vger.kernel.org, netdev@vger.kernel.org, dan.daly@intel.com, linux-kernel@vger.kernel.org, linux-nvme@lists.infradead.org, keith.busch@intel.com, netanel@amazon.com, ddutile@redhat.com, mheyne@amazon.de, liang-min.wang@intel.com, mark.d.rustad@intel.com, dwmw2@infradead.org, hch@lst.de, dwmw@amazon.co.uk Date: Thu, 15 Mar 2018 11:40:48 -0700 Message-ID: <20180315183449.3102.64791.stgit@localhost.localdomain> User-Agent: StGit/0.17.1-dirty MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This series is meant to add support for SR-IOV on devices when the VFs are not managed by the kernel. Examples of recent patches attempting to do this include: virto - https://patchwork.kernel.org/patch/10241225/ pci-stub - https://patchwork.kernel.org/patch/10109935/ vfio - https://patchwork.kernel.org/patch/10103353/ uio - https://patchwork.kernel.org/patch/9974031/ Since this is quickly blowing up into a multi-driver problem it is probably best to implement this solution as generically as possible. This series is an attempt to do that. What we do with this patch set is provide a generic framework to enable SR-IOV in the case that the PF driver doesn't support managing the VFs itself. I based my patch set originally on the patch by Mark Rustad but there isn't much left after going through and cleaning out the bits that were no longer needed, and after incorporating the feedback from David Miller. At this point the only items to be fully reused was his patch description which is now present in patch 3 of the set. This solution is limited in scope to just adding support for devices that provide no functionality for SR-IOV other than allocating the VFs by calling pci_enable_sriov. Previous sets had included patches for VFIO, but for now I am dropping that as the scope of that work is larger then I think I can take on at this time. v2: Reduced scope back to just virtio_pci and vfio-pci Broke into 3 patch set from single patch Changed autoprobe behavior to always set when num_vfs is set non-zero v3: Updated Documentation to clarify when sriov_unmanaged_autoprobe is used Wrapped vfio_pci_sriov_configure to fix build errors w/o SR-IOV in kernel v4: Dropped vfio-pci patch Added ena and nvme to drivers now using pci_sriov_configure_unmanaged Dropped pci_disable_sriov call in virtio_pci to be consistent with ena v5: Dropped sriov_unmanaged_autoprobe and pci_sriov_conifgure_unmanaged Added new patch that enables pci_sriov_configure_simple Updated drivers to use pci_sriov_configure_simple v6: Defined pci_sriov_configure_simple as NULL when SR-IOV is not enabled Updated drivers to drop "#ifdef" checks for IOV Added pci-pf-stub as place for PF-only drivers to add support v7: Dropped pci_id table explanation from pci-pf-stub driver Updated pci_sriov_configure_simple to drop need for err value Fixed comment explaining why pci_sriov_configure_simple is NULL Cc: Mark Rustad Cc: Maximilian Heyne Cc: Liang-Min Wang Cc: David Woodhouse --- Alexander Duyck (5): pci: Add pci_sriov_configure_simple for PFs that don't manage VF resources virtio_pci: Add support for unmanaged SR-IOV on virtio_pci devices ena: Migrate over to unmanaged SR-IOV support nvme: Migrate over to unmanaged SR-IOV support pci-pf-stub: Add PF driver stub for PFs that function only to enable VFs drivers/net/ethernet/amazon/ena/ena_netdev.c | 28 ------------- drivers/nvme/host/pci.c | 20 ---------- drivers/pci/Kconfig | 12 ++++++ drivers/pci/Makefile | 2 + drivers/pci/iov.c | 31 +++++++++++++++ drivers/pci/pci-pf-stub.c | 54 ++++++++++++++++++++++++++ drivers/virtio/virtio_pci_common.c | 1 include/linux/pci.h | 3 + include/linux/pci_ids.h | 2 + 9 files changed, 107 insertions(+), 46 deletions(-) create mode 100644 drivers/pci/pci-pf-stub.c -- From mboxrd@z Thu Jan 1 00:00:00 1970 From: alexander.duyck@gmail.com (Alexander Duyck) Date: Thu, 15 Mar 2018 11:40:48 -0700 Subject: [pci PATCH v7 0/5] Add support for unmanaged SR-IOV Message-ID: <20180315183449.3102.64791.stgit@localhost.localdomain> This series is meant to add support for SR-IOV on devices when the VFs are not managed by the kernel. Examples of recent patches attempting to do this include: virto - https://patchwork.kernel.org/patch/10241225/ pci-stub - https://patchwork.kernel.org/patch/10109935/ vfio - https://patchwork.kernel.org/patch/10103353/ uio - https://patchwork.kernel.org/patch/9974031/ Since this is quickly blowing up into a multi-driver problem it is probably best to implement this solution as generically as possible. This series is an attempt to do that. What we do with this patch set is provide a generic framework to enable SR-IOV in the case that the PF driver doesn't support managing the VFs itself. I based my patch set originally on the patch by Mark Rustad but there isn't much left after going through and cleaning out the bits that were no longer needed, and after incorporating the feedback from David Miller. At this point the only items to be fully reused was his patch description which is now present in patch 3 of the set. This solution is limited in scope to just adding support for devices that provide no functionality for SR-IOV other than allocating the VFs by calling pci_enable_sriov. Previous sets had included patches for VFIO, but for now I am dropping that as the scope of that work is larger then I think I can take on at this time. v2: Reduced scope back to just virtio_pci and vfio-pci Broke into 3 patch set from single patch Changed autoprobe behavior to always set when num_vfs is set non-zero v3: Updated Documentation to clarify when sriov_unmanaged_autoprobe is used Wrapped vfio_pci_sriov_configure to fix build errors w/o SR-IOV in kernel v4: Dropped vfio-pci patch Added ena and nvme to drivers now using pci_sriov_configure_unmanaged Dropped pci_disable_sriov call in virtio_pci to be consistent with ena v5: Dropped sriov_unmanaged_autoprobe and pci_sriov_conifgure_unmanaged Added new patch that enables pci_sriov_configure_simple Updated drivers to use pci_sriov_configure_simple v6: Defined pci_sriov_configure_simple as NULL when SR-IOV is not enabled Updated drivers to drop "#ifdef" checks for IOV Added pci-pf-stub as place for PF-only drivers to add support v7: Dropped pci_id table explanation from pci-pf-stub driver Updated pci_sriov_configure_simple to drop need for err value Fixed comment explaining why pci_sriov_configure_simple is NULL Cc: Mark Rustad Cc: Maximilian Heyne Cc: Liang-Min Wang Cc: David Woodhouse --- Alexander Duyck (5): pci: Add pci_sriov_configure_simple for PFs that don't manage VF resources virtio_pci: Add support for unmanaged SR-IOV on virtio_pci devices ena: Migrate over to unmanaged SR-IOV support nvme: Migrate over to unmanaged SR-IOV support pci-pf-stub: Add PF driver stub for PFs that function only to enable VFs drivers/net/ethernet/amazon/ena/ena_netdev.c | 28 ------------- drivers/nvme/host/pci.c | 20 ---------- drivers/pci/Kconfig | 12 ++++++ drivers/pci/Makefile | 2 + drivers/pci/iov.c | 31 +++++++++++++++ drivers/pci/pci-pf-stub.c | 54 ++++++++++++++++++++++++++ drivers/virtio/virtio_pci_common.c | 1 include/linux/pci.h | 3 + include/linux/pci_ids.h | 2 + 9 files changed, 107 insertions(+), 46 deletions(-) create mode 100644 drivers/pci/pci-pf-stub.c -- From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: virtio-dev-return-3573-cohuck=redhat.com@lists.oasis-open.org Sender: List-Post: List-Help: List-Unsubscribe: List-Subscribe: Received: from lists.oasis-open.org (oasis-open.org [66.179.20.138]) by lists.oasis-open.org (Postfix) with ESMTP id B5CC1581819D for ; Thu, 15 Mar 2018 11:40:52 -0700 (PDT) From: Alexander Duyck Date: Thu, 15 Mar 2018 11:40:48 -0700 Message-ID: <20180315183449.3102.64791.stgit@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Subject: [virtio-dev] [pci PATCH v7 0/5] Add support for unmanaged SR-IOV To: bhelgaas@google.com, alexander.h.duyck@intel.com, linux-pci@vger.kernel.org Cc: virtio-dev@lists.oasis-open.org, kvm@vger.kernel.org, netdev@vger.kernel.org, dan.daly@intel.com, linux-kernel@vger.kernel.org, linux-nvme@lists.infradead.org, keith.busch@intel.com, netanel@amazon.com, ddutile@redhat.com, mheyne@amazon.de, liang-min.wang@intel.com, mark.d.rustad@intel.com, dwmw2@infradead.org, hch@lst.de, dwmw@amazon.co.uk List-ID: This series is meant to add support for SR-IOV on devices when the VFs are not managed by the kernel. Examples of recent patches attempting to do this include: virto - https://patchwork.kernel.org/patch/10241225/ pci-stub - https://patchwork.kernel.org/patch/10109935/ vfio - https://patchwork.kernel.org/patch/10103353/ uio - https://patchwork.kernel.org/patch/9974031/ Since this is quickly blowing up into a multi-driver problem it is probably best to implement this solution as generically as possible. This series is an attempt to do that. What we do with this patch set is provide a generic framework to enable SR-IOV in the case that the PF driver doesn't support managing the VFs itself. I based my patch set originally on the patch by Mark Rustad but there isn't much left after going through and cleaning out the bits that were no longer needed, and after incorporating the feedback from David Miller. At this point the only items to be fully reused was his patch description which is now present in patch 3 of the set. This solution is limited in scope to just adding support for devices that provide no functionality for SR-IOV other than allocating the VFs by calling pci_enable_sriov. Previous sets had included patches for VFIO, but for now I am dropping that as the scope of that work is larger then I think I can take on at this time. v2: Reduced scope back to just virtio_pci and vfio-pci Broke into 3 patch set from single patch Changed autoprobe behavior to always set when num_vfs is set non-zero v3: Updated Documentation to clarify when sriov_unmanaged_autoprobe is used Wrapped vfio_pci_sriov_configure to fix build errors w/o SR-IOV in kernel v4: Dropped vfio-pci patch Added ena and nvme to drivers now using pci_sriov_configure_unmanaged Dropped pci_disable_sriov call in virtio_pci to be consistent with ena v5: Dropped sriov_unmanaged_autoprobe and pci_sriov_conifgure_unmanaged Added new patch that enables pci_sriov_configure_simple Updated drivers to use pci_sriov_configure_simple v6: Defined pci_sriov_configure_simple as NULL when SR-IOV is not enabled Updated drivers to drop "#ifdef" checks for IOV Added pci-pf-stub as place for PF-only drivers to add support v7: Dropped pci_id table explanation from pci-pf-stub driver Updated pci_sriov_configure_simple to drop need for err value Fixed comment explaining why pci_sriov_configure_simple is NULL Cc: Mark Rustad Cc: Maximilian Heyne Cc: Liang-Min Wang Cc: David Woodhouse --- Alexander Duyck (5): pci: Add pci_sriov_configure_simple for PFs that don't manage VF resources virtio_pci: Add support for unmanaged SR-IOV on virtio_pci devices ena: Migrate over to unmanaged SR-IOV support nvme: Migrate over to unmanaged SR-IOV support pci-pf-stub: Add PF driver stub for PFs that function only to enable VFs drivers/net/ethernet/amazon/ena/ena_netdev.c | 28 ------------- drivers/nvme/host/pci.c | 20 ---------- drivers/pci/Kconfig | 12 ++++++ drivers/pci/Makefile | 2 + drivers/pci/iov.c | 31 +++++++++++++++ drivers/pci/pci-pf-stub.c | 54 ++++++++++++++++++++++++++ drivers/virtio/virtio_pci_common.c | 1 include/linux/pci.h | 3 + include/linux/pci_ids.h | 2 + 9 files changed, 107 insertions(+), 46 deletions(-) create mode 100644 drivers/pci/pci-pf-stub.c -- --------------------------------------------------------------------- To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org