[dpdk-stable] patch 'remove references to private PCI probe function' has been queued to LTS release 18.11.9

[Kevin Traynor]

Hi,

FYI, your patch has been queued to LTS release 18.11.9

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 06/10/20. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Queued patches are on a temporary branch at:
https://github.com/kevintraynor/dpdk-stable-queue

This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable-queue/commit/3538ace6654b31aecbac0581e1f93c3046c2abfe

Thanks.

Kevin.

---
>From 3538ace6654b31aecbac0581e1f93c3046c2abfe Mon Sep 17 00:00:00 2001
From: David Marchand <david.marchand at redhat.com>
Date: Wed, 6 May 2020 14:43:13 +0200
Subject: [PATCH] remove references to private PCI probe function

[ upstream commit 87db93e07aeeecc96a9e63b43a652511b95013d6 ]

rte_pci_probe() is private to the PCI bus.
Clean the remaining references in the documentation and comments.

Fixes: c752998b5e2e ("pci: introduce library and driver")

Signed-off-by: David Marchand <david.marchand at redhat.com>
Reviewed-by: Gaetan Rivet <grive at u256.net>
---
 doc/guides/sample_app_ug/l2_forward_real_virtual.rst |  9 ---------
 doc/guides/sample_app_ug/link_status_intr.rst        |  7 -------
 doc/guides/sample_app_ug/multi_process.rst           |  2 +-
 drivers/bus/pci/pci_common.c                         |  6 +++---
 drivers/bus/pci/private.h                            | 10 ----------
 drivers/net/virtio/virtio_user_ethdev.c              |  2 +-
 6 files changed, 5 insertions(+), 31 deletions(-)

diff --git a/doc/guides/sample_app_ug/l2_forward_real_virtual.rst b/doc/guides/sample_app_ug/l2_forward_real_virtual.rst
index 87653ec7bd..9729095872 100644
--- a/doc/guides/sample_app_ug/l2_forward_real_virtual.rst
+++ b/doc/guides/sample_app_ug/l2_forward_real_virtual.rst
@@ -195,7 +195,4 @@ in the *DPDK Programmer's Guide* - Rel 1.4 EAR and the *DPDK API Reference*.
 .. code-block:: c
 
-    if (rte_pci_probe() < 0)
-        rte_exit(EXIT_FAILURE, "Cannot probe PCI\n");
-
     /* reset l2fwd_dst_ports */
 
@@ -227,10 +224,4 @@ in the *DPDK Programmer's Guide* - Rel 1.4 EAR and the *DPDK API Reference*.
     }
 
-Observe that:
-
-*   rte_igb_pmd_init_all() simultaneously registers the driver as a PCI driver and as an Ethernet* Poll Mode Driver.
-
-*   rte_pci_probe() parses the devices on the PCI bus and initializes recognized devices.
-
 The next step is to configure the RX and TX queues.
 For each port, there is only one RX queue (only one lcore is able to poll a given port).
diff --git a/doc/guides/sample_app_ug/link_status_intr.rst b/doc/guides/sample_app_ug/link_status_intr.rst
index 695c088e88..45f2439a32 100644
--- a/doc/guides/sample_app_ug/link_status_intr.rst
+++ b/doc/guides/sample_app_ug/link_status_intr.rst
@@ -89,7 +89,4 @@ To fully understand this code, it is recommended to study the chapters that rela
 .. code-block:: c
 
-    if (rte_pci_probe() < 0)
-        rte_exit(EXIT_FAILURE, "Cannot probe PCI\n");
-
     /*
      * Each logical core is assigned a dedicated TX queue on each port.
@@ -116,8 +113,4 @@ To fully understand this code, it is recommended to study the chapters that rela
     }
 
-Observe that:
-
-*   rte_pci_probe()  parses the devices on the PCI bus and initializes recognized devices.
-
 The next step is to configure the RX and TX queues.
 For each port, there is only one RX queue (only one lcore is able to poll a given port).
diff --git a/doc/guides/sample_app_ug/multi_process.rst b/doc/guides/sample_app_ug/multi_process.rst
index 9c374da6f7..f2a79a6397 100644
--- a/doc/guides/sample_app_ug/multi_process.rst
+++ b/doc/guides/sample_app_ug/multi_process.rst
@@ -210,5 +210,5 @@ How the Application Works
 
 The initialization calls in both the primary and secondary instances are the same for the most part,
-calling the rte_eal_init(), 1 G and 10 G driver initialization and then rte_pci_probe() functions.
+calling the rte_eal_init(), 1 G and 10 G driver initialization and then probing devices.
 Thereafter, the initialization done depends on whether the process is configured as a primary or secondary instance.
 
diff --git a/drivers/bus/pci/pci_common.c b/drivers/bus/pci/pci_common.c
index 6276e5d695..79d4665720 100644
--- a/drivers/bus/pci/pci_common.c
+++ b/drivers/bus/pci/pci_common.c
@@ -274,6 +274,6 @@ pci_probe_all_drivers(struct rte_pci_device *dev)
  * for discovered devices.
  */
-int
-rte_pci_probe(void)
+static int
+pci_probe(void)
 {
 	struct rte_pci_device *dev = NULL;
@@ -532,5 +532,5 @@ struct rte_pci_bus rte_pci_bus = {
 	.bus = {
 		.scan = rte_pci_scan,
-		.probe = rte_pci_probe,
+		.probe = pci_probe,
 		.find_device = pci_find_device,
 		.plug = pci_plug,
diff --git a/drivers/bus/pci/private.h b/drivers/bus/pci/private.h
index 13c3324bb0..22cefc8b93 100644
--- a/drivers/bus/pci/private.h
+++ b/drivers/bus/pci/private.h
@@ -18,14 +18,4 @@ struct rte_pci_device;
 extern struct rte_pci_bus rte_pci_bus;
 
-/**
- * Probe the PCI bus
- *
- * @return
- *   - 0 on success.
- *   - !0 on error.
- */
-int
-rte_pci_probe(void);
-
 /**
  * Scan the content of the PCI bus, and the devices in the devices
diff --git a/drivers/net/virtio/virtio_user_ethdev.c b/drivers/net/virtio/virtio_user_ethdev.c
index 6c184075dd..7c275c7397 100644
--- a/drivers/net/virtio/virtio_user_ethdev.c
+++ b/drivers/net/virtio/virtio_user_ethdev.c
@@ -620,5 +620,5 @@ virtio_user_pmd_probe(struct rte_vdev_device *dev)
 	}
 
-	/* previously called by rte_pci_probe() for physical dev */
+	/* previously called by pci probing for physical dev */
 	if (eth_virtio_dev_init(eth_dev) < 0) {
 		PMD_INIT_LOG(ERR, "eth_virtio_dev_init fails");
-- 
2.21.3

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2020-06-05 19:20:52.112162302 +0100
+++ 0022-remove-references-to-private-PCI-probe-function.patch	2020-06-05 19:20:50.746042267 +0100
@@ -1 +1 @@
-From 87db93e07aeeecc96a9e63b43a652511b95013d6 Mon Sep 17 00:00:00 2001
+From 3538ace6654b31aecbac0581e1f93c3046c2abfe Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 87db93e07aeeecc96a9e63b43a652511b95013d6 ]
+
@@ -10 +11,0 @@
-Cc: stable at dpdk.org
@@ -15 +15,0 @@
- doc/guides/sample_app_ug/l2_forward_event.rst        |  8 --------
@@ -22 +22 @@
- 7 files changed, 5 insertions(+), 39 deletions(-)
+ 6 files changed, 5 insertions(+), 31 deletions(-)
@@ -24,22 +23,0 @@
-diff --git a/doc/guides/sample_app_ug/l2_forward_event.rst b/doc/guides/sample_app_ug/l2_forward_event.rst
-index 8bdf352c4e..d536eee819 100644
---- a/doc/guides/sample_app_ug/l2_forward_event.rst
-+++ b/doc/guides/sample_app_ug/l2_forward_event.rst
-@@ -205,7 +205,4 @@ chapters that related to the Poll Mode and Event mode Driver in the
- .. code-block:: c
- 
--    if (rte_pci_probe() < 0)
--        rte_panic("Cannot probe PCI\n");
--
-     /* reset l2fwd_dst_ports */
- 
-@@ -237,9 +234,4 @@ chapters that related to the Poll Mode and Event mode Driver in the
-     }
- 
--Observe that:
--
--*   rte_pci_probe() parses the devices on the PCI bus and initializes recognized
--    devices.
--
- The next step is to configure the RX and TX queues. For each port, there is only
- one RX queue (only one lcore is able to poll a given port). The number of TX
@@ -47 +25 @@
-index 39d6b0067a..671d0c7c19 100644
+index 87653ec7bd..9729095872 100644
@@ -70 +48 @@
-index 5283be8b7c..04c40f2854 100644
+index 695c088e88..45f2439a32 100644
@@ -102 +80 @@
-index 6585a4b476..648705582a 100644
+index 6276e5d695..79d4665720 100644
@@ -105 +83 @@
-@@ -293,6 +293,6 @@ pci_probe_all_drivers(struct rte_pci_device *dev)
+@@ -274,6 +274,6 @@ pci_probe_all_drivers(struct rte_pci_device *dev)
@@ -114 +92 @@
-@@ -680,5 +680,5 @@ struct rte_pci_bus rte_pci_bus = {
+@@ -532,5 +532,5 @@ struct rte_pci_bus rte_pci_bus = {
@@ -122 +100 @@
-index a205d4d9f0..af1c7ae5fe 100644
+index 13c3324bb0..22cefc8b93 100644
@@ -141 +119 @@
-index c54698ad1e..dfc5291d4c 100644
+index 6c184075dd..7c275c7397 100644
@@ -144 +122 @@
-@@ -734,5 +734,5 @@ virtio_user_pmd_probe(struct rte_vdev_device *dev)
+@@ -620,5 +620,5 @@ virtio_user_pmd_probe(struct rte_vdev_device *dev)