From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 7DDDEA04A3 for ; Fri, 5 Jun 2020 20:26:43 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 75B191D510; Fri, 5 Jun 2020 20:26:43 +0200 (CEST) Received: from us-smtp-delivery-1.mimecast.com (us-smtp-2.mimecast.com [205.139.110.61]) by dpdk.org (Postfix) with ESMTP id 0737B1D510 for ; Fri, 5 Jun 2020 20:26:41 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1591381601; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=/jyQmI9aVcbmvTXCEhBYv2PW5dKHZ1V63Fvfnpr/9Lk=; b=DFUScjk87OcIkNrg9K0Pzsh2mvVCAsaY9IZKiVKs8b85WfCQ+Q2GGapsDxFX0wRzyj4vK1 DOhxVCtJLBPrat5Dao80XuGRCvIPlZpHIuYNtShzgEIZAnh3PS/93pnBZvVKogcyxW1/TT axCesBJFotPg+r6ehJhka6TPqk3Adqk= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-208-O0zyaRIxNV64QA2HIGvjvA-1; Fri, 05 Jun 2020 14:26:38 -0400 X-MC-Unique: O0zyaRIxNV64QA2HIGvjvA-1 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id AA081102C886; Fri, 5 Jun 2020 18:26:37 +0000 (UTC) Received: from rh.redhat.com (unknown [10.33.36.130]) by smtp.corp.redhat.com (Postfix) with ESMTP id 9B66268402; Fri, 5 Jun 2020 18:26:11 +0000 (UTC) From: Kevin Traynor To: David Marchand Cc: Gaetan Rivet , dpdk stable Date: Fri, 5 Jun 2020 19:24:19 +0100 Message-Id: <20200605182525.22483-22-ktraynor@redhat.com> In-Reply-To: <20200605182525.22483-1-ktraynor@redhat.com> References: <20200605182525.22483-1-ktraynor@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 8bit Subject: [dpdk-stable] patch 'remove references to private PCI probe function' has been queued to LTS release 18.11.9 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" 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 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 Reviewed-by: Gaetan Rivet --- 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@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)