[dpdk-stable] patch 'bus/pci: use given name as generic name' has been queued to stable release 17.05.2
Yuanhan Liu
yliu at fridaylinux.org
Tue Aug 29 09:47:28 CEST 2017
On Mon, Aug 21, 2017 at 05:30:20PM +0800, Yuanhan Liu wrote:
> Hi,
>
> FYI, your patch has been queued to stable release 17.05.2
>
> Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
> It will be pushed if I get no objections before 08/24/17. So please
> shout if anyone has objections.
There is a build error:
lib/librte_eal/common/eal_common_pci.c:133:41: error: ‘struct rte_devargs’ has no member named ‘name’
dev->device.name = dev->device.devargs->name;
I usually will do a build test before sending this out. Not quite sure
why I missed that this time :/
So I will drop it in this release. Let me know if you think a backport
is needed.
Sorry for that!
--yliu
>
> ---
> >From abbdc42771b8999624afe375b4d5d8e036d7ab71 Mon Sep 17 00:00:00 2001
> From: Gaetan Rivet <gaetan.rivet at 6wind.com>
> Date: Sat, 15 Jul 2017 19:56:39 +0200
> Subject: [PATCH] bus/pci: use given name as generic name
>
> [ upstream commit 23eaa9059ec24e95e32361f333ed0686f82bea74 ]
>
> When an application requests the use of a PCI device, it can currently
> interchangeably use either the longform DomBDF format (0000:00:00.0) or
> the shorter BDF format (00:00.0).
>
> When a device is inserted via the hotplug API, it must first be scanned
> and then will be identified by its name using `find_device`. The name of
> the device must match the name given by the user to be found and then
> probed.
>
> A new function sets the expected name for a scanned PCI device. It was
> previously generated from parsing the PCI address. This canonical name
> is superseded when an rte_devargs exists describing the device. In such
> case, the device takes the given name found within the rte_devargs.
>
> As the rte_devargs is linked to the rte_pci_device during scanning, it
> can be avoided during the probe. Additionally, this fixes the issue of
> the rte_devargs lookup not being done within rte_pci_probe_one.
>
> Fixes: beec692c5157 ("eal: add name field to generic device")
>
> Signed-off-by: Gaetan Rivet <gaetan.rivet at 6wind.com>
> ---
> lib/librte_eal/bsdapp/eal/eal_pci.c | 4 ++--
> lib/librte_eal/common/eal_common_pci.c | 29 ++++++++++++++++++++++++-----
> lib/librte_eal/common/eal_private.h | 5 +++++
> lib/librte_eal/linuxapp/eal/eal_pci.c | 4 ++--
> 4 files changed, 33 insertions(+), 9 deletions(-)
>
> diff --git a/lib/librte_eal/bsdapp/eal/eal_pci.c b/lib/librte_eal/bsdapp/eal/eal_pci.c
> index e321461..97a88ec 100644
> --- a/lib/librte_eal/bsdapp/eal/eal_pci.c
> +++ b/lib/librte_eal/bsdapp/eal/eal_pci.c
> @@ -282,8 +282,7 @@ pci_scan_one(int dev_pci_fd, struct pci_conf *conf)
> /* FreeBSD has no NUMA support (yet) */
> dev->device.numa_node = 0;
>
> - rte_pci_device_name(&dev->addr, dev->name, sizeof(dev->name));
> - dev->device.name = dev->name;
> + pci_name_set(dev);
>
> /* FreeBSD has only one pass through driver */
> dev->kdrv = RTE_KDRV_NIC_UIO;
> @@ -334,6 +333,7 @@ pci_scan_one(int dev_pci_fd, struct pci_conf *conf)
> } else { /* already registered */
> dev2->kdrv = dev->kdrv;
> dev2->max_vfs = dev->max_vfs;
> + pci_name_set(dev2);
> memmove(dev2->mem_resource,
> dev->mem_resource,
> sizeof(dev->mem_resource));
> diff --git a/lib/librte_eal/common/eal_common_pci.c b/lib/librte_eal/common/eal_common_pci.c
> index b749991..8f8627e 100644
> --- a/lib/librte_eal/common/eal_common_pci.c
> +++ b/lib/librte_eal/common/eal_common_pci.c
> @@ -113,6 +113,29 @@ static struct rte_devargs *pci_devargs_lookup(struct rte_pci_device *dev)
> return NULL;
> }
>
> +void
> +pci_name_set(struct rte_pci_device *dev)
> +{
> + struct rte_devargs *devargs;
> +
> + /* Each device has its internal, canonical name set. */
> + rte_pci_device_name(&dev->addr,
> + dev->name, sizeof(dev->name));
> + devargs = pci_devargs_lookup(dev);
> + dev->device.devargs = devargs;
> + /* In blacklist mode, if the device is not blacklisted, no
> + * rte_devargs exists for it.
> + */
> + if (devargs != NULL)
> + /* If an rte_devargs exists, the generic rte_device uses the
> + * given name as its namea
> + */
> + dev->device.name = dev->device.devargs->name;
> + else
> + /* Otherwise, it uses the internal, canonical form. */
> + dev->device.name = dev->name;
> +}
> +
> /* map a particular resource from a file */
> void *
> pci_map_resource(void *requested_addr, int fd, off_t offset, size_t size,
> @@ -419,11 +442,7 @@ rte_pci_probe(void)
> FOREACH_DEVICE_ON_PCIBUS(dev) {
> probed++;
>
> - /* set devargs in PCI structure */
> - devargs = pci_devargs_lookup(dev);
> - if (devargs != NULL)
> - dev->device.devargs = devargs;
> -
> + devargs = dev->device.devargs;
> /* probe all or only whitelisted devices */
> if (probe_all)
> ret = pci_probe_all_drivers(dev);
> diff --git a/lib/librte_eal/common/eal_private.h b/lib/librte_eal/common/eal_private.h
> index 6cacce0..0a1c3a7 100644
> --- a/lib/librte_eal/common/eal_private.h
> +++ b/lib/librte_eal/common/eal_private.h
> @@ -113,6 +113,11 @@ struct rte_pci_driver;
> struct rte_pci_device;
>
> /**
> + * Find the name of a PCI device.
> + */
> +void pci_name_set(struct rte_pci_device *dev);
> +
> +/**
> * Add a PCI device to the PCI Bus (append to PCI Device list). This function
> * also updates the bus references of the PCI Device (and the generic device
> * object embedded within.
> diff --git a/lib/librte_eal/linuxapp/eal/eal_pci.c b/lib/librte_eal/linuxapp/eal/eal_pci.c
> index 595622b..816b19d 100644
> --- a/lib/librte_eal/linuxapp/eal/eal_pci.c
> +++ b/lib/librte_eal/linuxapp/eal/eal_pci.c
> @@ -324,8 +324,7 @@ pci_scan_one(const char *dirname, const struct rte_pci_addr *addr)
> dev->device.numa_node = tmp;
> }
>
> - rte_pci_device_name(addr, dev->name, sizeof(dev->name));
> - dev->device.name = dev->name;
> + pci_name_set(dev);
>
> /* parse resources */
> snprintf(filename, sizeof(filename), "%s/resource", dirname);
> @@ -373,6 +372,7 @@ pci_scan_one(const char *dirname, const struct rte_pci_addr *addr)
> } else { /* already registered */
> dev2->kdrv = dev->kdrv;
> dev2->max_vfs = dev->max_vfs;
> + pci_name_set(dev2);
> memmove(dev2->mem_resource, dev->mem_resource,
> sizeof(dev->mem_resource));
> free(dev);
> --
> 2.7.4
More information about the stable
mailing list