[dpdk-stable] patch 'ethdev: fix port id allocation' has been queued to LTS release 17.11.1
Yuanhan Liu
yliu at fridaylinux.org
Sun Feb 11 03:46:53 CET 2018
On Thu, Feb 01, 2018 at 05:47:54PM +0800, Yuanhan Liu wrote:
> Hi,
>
> FYI, your patch has been queued to LTS release 17.11.1
>
> Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
> It will be pushed if I get no objections before 02/03/18. So please
> shout if anyone has objections.
It introduced an crash at port allocation.
And I have now reverted it. Please do a backport if necessary.
Thanks.
--yliu
>
> ---
> >From 67abe74f3c09e6f166902c9fc5f4d7f65f7b3007 Mon Sep 17 00:00:00 2001
> From: Matan Azrad <matan at mellanox.com>
> Date: Mon, 22 Jan 2018 16:38:18 +0000
> Subject: [PATCH] ethdev: fix port id allocation
>
> [ upstream commit 8ee892a2385c50427c03db5cef1789babceb5999 ]
>
> rte_eth_dev_find_free_port() found a free port by state checking.
> The state field are in local process memory, so other DPDK processes
> may get the same port ID because their local states may be different.
>
> Replace the state checking by the ethdev port name checking,
> so, if the name is an empty string the port ID will be detected as
> unused.
>
> Fixes: d948f596fee2 ("ethdev: fix port data mismatched in multiple process model")
>
> Suggested-by: Konstantin Ananyev <konstantin.ananyev at intel.com>
> Signed-off-by: Matan Azrad <matan at mellanox.com>
> Acked-by: Thomas Monjalon <thomas at monjalon.net>
> Acked-by: Konstantin Ananyev <konstantin.ananyev at intel.com>
> ---
> lib/librte_ether/rte_ethdev.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.c
> index 9dcb0d2..8ea85cd 100644
> --- a/lib/librte_ether/rte_ethdev.c
> +++ b/lib/librte_ether/rte_ethdev.c
> @@ -192,8 +192,12 @@ rte_eth_dev_find_free_port(void)
> unsigned i;
>
> for (i = 0; i < RTE_MAX_ETHPORTS; i++) {
> - if (rte_eth_devices[i].state == RTE_ETH_DEV_UNUSED)
> + /* Using shared name field to find a free port. */
> + if (rte_eth_dev_data[i].name[0] == '\0') {
> + RTE_ASSERT(rte_eth_devices[i].state ==
> + RTE_ETH_DEV_UNUSED);
> return i;
> + }
> }
> return RTE_MAX_ETHPORTS;
> }
> --
> 2.7.4
More information about the stable
mailing list