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 A2D2CA0527 for ; Mon, 9 Nov 2020 19:42:57 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 9DB2A6883; Mon, 9 Nov 2020 19:42:56 +0100 (CET) Received: from mail-wr1-f66.google.com (mail-wr1-f66.google.com [209.85.221.66]) by dpdk.org (Postfix) with ESMTP id B9B516883 for ; Mon, 9 Nov 2020 19:42:54 +0100 (CET) Received: by mail-wr1-f66.google.com with SMTP id l1so5464120wrb.9 for ; Mon, 09 Nov 2020 10:42:54 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=mRUjAtToCOAD59NPzEvJYnB6rxUbNdv7lRVz1mkU4ms=; b=WsJQcVdH3GVaw3tFb/kkwe87TfkGDUsQPbcxPnM1ruYjrtF/VqLBSNlWl5WxUEXdQW BbQJBerFBqes/0nFcPBHMNtP4nHn93vHgjfnFcwftycW8jL7H25VDM4ie0M4zRw6ZBUe 68BbYmoufREtXNjF60wLBzPLcxBzki0CEBkEXqkNkdOiGwJtn3IhuUNDu60MnZNpqlv+ gHFFDzOnA6r8bng2QHrbMTfr2CrmQsCnWvVk15bQusa7wcvuzrubc8GBnek3rfpfAn1L yjvWehF782TI2yhf9Adn7nZKu+Amx8caj6uE+QAvpVnoxJpZltQZTzglX6IepxPyN0Px rN8A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=mRUjAtToCOAD59NPzEvJYnB6rxUbNdv7lRVz1mkU4ms=; b=Wozkx1gVF19ksLzSsj/7Xi41hgabCt5ePGFRJUBvwr6PAaBRscxiO7Hgb5r+Bkob2a Hld7YYb4XBX4cowqkFMovbv5R3jLVkGSk+zgGslqMWbYD+YC4quDxMHpH+cO/kYQpWke Z5QDETdnriHlVXmn6VgsHH/s45g02xsLd443w59nDs7UhzBTf6KHl1jXmGNgYKULvJ8e heWuqAsFgzqzqJLR+uSvg6IohL66O09MP4ahRagXie3FQSC6G0fZoKwd6udFFouOHj1h 9lnpEKe93s1UmUTRqyMcd2GRWf/3MnJYerpUDlHkR6BfWXCe66SCz49fOBTHXVmA3PaH yhgg== X-Gm-Message-State: AOAM530WqiLvBHKSKs456pQzhOywIOQSiAISjy8bpDZEBgJDYnYRRS+e +oEq04ZtKjhexLk5hT12kUoZFA+ChEGN7Q== X-Google-Smtp-Source: ABdhPJzP/hvezfmVNQI9sGh+TQbXL0yNRba5zrY4A/c0xyMK5R9RtTMrV99WBtg7hXjIImPvsKVecw== X-Received: by 2002:a5d:6744:: with SMTP id l4mr18779704wrw.378.1604947373496; Mon, 09 Nov 2020 10:42:53 -0800 (PST) Received: from localhost ([88.98.246.218]) by smtp.gmail.com with ESMTPSA id p4sm332595wmc.46.2020.11.09.10.42.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 09 Nov 2020 10:42:52 -0800 (PST) From: luca.boccassi@gmail.com To: Xueming Li Cc: Matan Azrad , dpdk stable Date: Mon, 9 Nov 2020 18:40:18 +0000 Message-Id: <20201109184111.3463090-30-luca.boccassi@gmail.com> X-Mailer: git-send-email 2.27.0 In-Reply-To: <20201109184111.3463090-1-luca.boccassi@gmail.com> References: <20201028104606.3504127-207-luca.boccassi@gmail.com> <20201109184111.3463090-1-luca.boccassi@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [dpdk-stable] patch 'net/mlx5: fix port shared data reference count' has been queued to stable release 19.11.6 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 stable release 19.11.6 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 11/11/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/bluca/dpdk-stable This queued commit can be viewed at: https://github.com/bluca/dpdk-stable/commit/54a97855544b7f8bbed432e1bf7566e884957077 Thanks. Luca Boccassi --- >From 54a97855544b7f8bbed432e1bf7566e884957077 Mon Sep 17 00:00:00 2001 From: Xueming Li Date: Wed, 21 Oct 2020 11:15:23 +0000 Subject: [PATCH] net/mlx5: fix port shared data reference count [ upstream commit 16dbba257c9ceccb30577d189dc6ca7e1042552f ] When probe a representor, tag cache hash table and modification cache hash table allocated memory upon each port, overwrote previous existing cache in shared context data. This patch moves reference check of shared data prior to hash table allocation to avoid such issue. Fixes: 6801116688fe ("net/mlx5: fix multiple flow table hash list") Fixes: 1ef4cdef2682 ("net/mlx5: fix flow tag hash list conversion") Acked-by: Matan Azrad Signed-off-by: Xueming Li --- drivers/net/mlx5/mlx5.c | 30 ++++++++---------------------- drivers/net/mlx5/mlx5.h | 2 -- drivers/net/mlx5/mlx5_flow_dv.c | 4 ++-- 3 files changed, 10 insertions(+), 26 deletions(-) diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c index 88ed5ffa5c..f2327903eb 100644 --- a/drivers/net/mlx5/mlx5.c +++ b/drivers/net/mlx5/mlx5.c @@ -951,13 +951,12 @@ mlx5_alloc_shared_dr(struct mlx5_priv *priv) { struct mlx5_ibv_shared *sh = priv->sh; char s[MLX5_HLIST_NAMESIZE]; - int err = 0; + int err; - if (!sh->flow_tbls) - err = mlx5_alloc_table_hash_list(priv); - else - DRV_LOG(DEBUG, "sh->flow_tbls[%p] already created, reuse", - (void *)sh->flow_tbls); + assert(sh && sh->refcnt); + if (sh->refcnt > 1) + return 0; + err = mlx5_alloc_table_hash_list(priv); if (err) return err; /* Create tags hash list table. */ @@ -971,12 +970,6 @@ mlx5_alloc_shared_dr(struct mlx5_priv *priv) #ifdef HAVE_MLX5DV_DR void *domain; - if (sh->dv_refcnt) { - /* Shared DV/DR structures is already initialized. */ - sh->dv_refcnt++; - priv->dr_shared = 1; - return 0; - } /* Reference counter is zero, we should initialize structures. */ domain = mlx5_glue->dr_create_domain(sh->ctx, MLX5DV_DR_DOMAIN_TYPE_NIC_RX); @@ -1010,8 +1003,6 @@ mlx5_alloc_shared_dr(struct mlx5_priv *priv) #endif sh->pop_vlan_action = mlx5_glue->dr_create_flow_action_pop_vlan(); #endif /* HAVE_MLX5DV_DR */ - sh->dv_refcnt++; - priv->dr_shared = 1; return 0; error: /* Rollback the created objects. */ @@ -1053,17 +1044,12 @@ error: static void mlx5_free_shared_dr(struct mlx5_priv *priv) { - struct mlx5_ibv_shared *sh; + struct mlx5_ibv_shared *sh = priv->sh; - if (!priv->dr_shared) + assert(sh && sh->dv_refcnt); + if (sh->refcnt > 1) return; - priv->dr_shared = 0; - sh = priv->sh; - assert(sh); #ifdef HAVE_MLX5DV_DR - assert(sh->dv_refcnt); - if (sh->dv_refcnt && --sh->dv_refcnt) - return; if (sh->rx_domain) { mlx5_glue->dr_destroy_domain(sh->rx_domain); sh->rx_domain = NULL; diff --git a/drivers/net/mlx5/mlx5.h b/drivers/net/mlx5/mlx5.h index fd66aaf318..2f76d162cf 100644 --- a/drivers/net/mlx5/mlx5.h +++ b/drivers/net/mlx5/mlx5.h @@ -669,7 +669,6 @@ struct mlx5_ibv_shared { uint32_t dv_meta_mask; /* flow META metadata supported mask. */ uint32_t dv_mark_mask; /* flow MARK metadata supported mask. */ uint32_t dv_regc0_mask; /* available bits of metatada reg_c[0]. */ - uint32_t dv_refcnt; /* DV/DR data reference counter. */ void *fdb_domain; /* FDB Direct Rules name space handle. */ void *rx_domain; /* RX Direct Rules name space handle. */ void *tx_domain; /* TX Direct Rules name space handle. */ @@ -731,7 +730,6 @@ struct mlx5_priv { unsigned int isolated:1; /* Whether isolated mode is enabled. */ unsigned int representor:1; /* Device is a port representor. */ unsigned int master:1; /* Device is a E-Switch master. */ - unsigned int dr_shared:1; /* DV/DR data is shared. */ unsigned int counter_fallback:1; /* Use counter fallback management. */ unsigned int mtr_en:1; /* Whether support meter. */ unsigned int mtr_reg_share:1; /* Whether support meter REG_C share. */ diff --git a/drivers/net/mlx5/mlx5_flow_dv.c b/drivers/net/mlx5/mlx5_flow_dv.c index 60d6af04e2..4ba1df7027 100644 --- a/drivers/net/mlx5/mlx5_flow_dv.c +++ b/drivers/net/mlx5/mlx5_flow_dv.c @@ -280,7 +280,7 @@ flow_dv_shared_lock(struct rte_eth_dev *dev) struct mlx5_priv *priv = dev->data->dev_private; struct mlx5_ibv_shared *sh = priv->sh; - if (sh->dv_refcnt > 1) { + if (sh->refcnt > 1) { int ret; ret = pthread_mutex_lock(&sh->dv_mutex); @@ -295,7 +295,7 @@ flow_dv_shared_unlock(struct rte_eth_dev *dev) struct mlx5_priv *priv = dev->data->dev_private; struct mlx5_ibv_shared *sh = priv->sh; - if (sh->dv_refcnt > 1) { + if (sh->refcnt > 1) { int ret; ret = pthread_mutex_unlock(&sh->dv_mutex); -- 2.27.0 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2020-11-09 18:40:12.417235841 +0000 +++ 0030-net-mlx5-fix-port-shared-data-reference-count.patch 2020-11-09 18:40:11.143311469 +0000 @@ -1 +1 @@ -From 16dbba257c9ceccb30577d189dc6ca7e1042552f Mon Sep 17 00:00:00 2001 +From 54a97855544b7f8bbed432e1bf7566e884957077 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 16dbba257c9ceccb30577d189dc6ca7e1042552f ] + @@ -15 +16,0 @@ -Cc: stable@dpdk.org @@ -20,3 +21,3 @@ - drivers/net/mlx5/linux/mlx5_os.c | 30 ++++++++---------------------- - drivers/net/mlx5/mlx5.h | 2 -- - drivers/net/mlx5/mlx5_flow_dv.c | 4 ++-- + drivers/net/mlx5/mlx5.c | 30 ++++++++---------------------- + drivers/net/mlx5/mlx5.h | 2 -- + drivers/net/mlx5/mlx5_flow_dv.c | 4 ++-- @@ -25,5 +26,5 @@ -diff --git a/drivers/net/mlx5/linux/mlx5_os.c b/drivers/net/mlx5/linux/mlx5_os.c -index 457008ef9e..40f9446d43 100644 ---- a/drivers/net/mlx5/linux/mlx5_os.c -+++ b/drivers/net/mlx5/linux/mlx5_os.c -@@ -226,13 +226,12 @@ mlx5_alloc_shared_dr(struct mlx5_priv *priv) +diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c +index 88ed5ffa5c..f2327903eb 100644 +--- a/drivers/net/mlx5/mlx5.c ++++ b/drivers/net/mlx5/mlx5.c +@@ -951,13 +951,12 @@ mlx5_alloc_shared_dr(struct mlx5_priv *priv) @@ -31 +32 @@ - struct mlx5_dev_ctx_shared *sh = priv->sh; + struct mlx5_ibv_shared *sh = priv->sh; @@ -39 +40 @@ -- DRV_LOG(DEBUG, "sh->flow_tbls[%p] already created, reuse\n", +- DRV_LOG(DEBUG, "sh->flow_tbls[%p] already created, reuse", @@ -41 +42 @@ -+ MLX5_ASSERT(sh && sh->refcnt); ++ assert(sh && sh->refcnt); @@ -48 +49 @@ -@@ -261,12 +260,6 @@ mlx5_alloc_shared_dr(struct mlx5_priv *priv) +@@ -971,12 +970,6 @@ mlx5_alloc_shared_dr(struct mlx5_priv *priv) @@ -61,2 +62,2 @@ -@@ -306,8 +299,6 @@ mlx5_alloc_shared_dr(struct mlx5_priv *priv) - } +@@ -1010,8 +1003,6 @@ mlx5_alloc_shared_dr(struct mlx5_priv *priv) + #endif @@ -70,3 +71,3 @@ -@@ -357,17 +348,12 @@ error: - void - mlx5_os_free_shared_dr(struct mlx5_priv *priv) +@@ -1053,17 +1044,12 @@ error: + static void + mlx5_free_shared_dr(struct mlx5_priv *priv) @@ -74,2 +75,2 @@ -- struct mlx5_dev_ctx_shared *sh; -+ struct mlx5_dev_ctx_shared *sh = priv->sh; +- struct mlx5_ibv_shared *sh; ++ struct mlx5_ibv_shared *sh = priv->sh; @@ -78 +79 @@ -+ MLX5_ASSERT(sh && sh->refcnt); ++ assert(sh && sh->dv_refcnt); @@ -83 +84 @@ -- MLX5_ASSERT(sh); +- assert(sh); @@ -85 +86 @@ -- MLX5_ASSERT(sh->dv_refcnt); +- assert(sh->dv_refcnt); @@ -92 +93 @@ -index fa69c66137..c9d5d71630 100644 +index fd66aaf318..2f76d162cf 100644 @@ -95 +96 @@ -@@ -635,7 +635,6 @@ struct mlx5_dev_ctx_shared { +@@ -669,7 +669,6 @@ struct mlx5_ibv_shared { @@ -103 +104 @@ -@@ -823,7 +822,6 @@ struct mlx5_priv { +@@ -731,7 +730,6 @@ struct mlx5_priv { @@ -108 +109 @@ - unsigned int txpp_en:1; /* Tx packet pacing enabled. */ + unsigned int counter_fallback:1; /* Use counter fallback management. */ @@ -112 +113 @@ -index 49d96362f8..15cd34e331 100644 +index 60d6af04e2..4ba1df7027 100644 @@ -115 +116 @@ -@@ -293,7 +293,7 @@ flow_dv_shared_lock(struct rte_eth_dev *dev) +@@ -280,7 +280,7 @@ flow_dv_shared_lock(struct rte_eth_dev *dev) @@ -117 +118 @@ - struct mlx5_dev_ctx_shared *sh = priv->sh; + struct mlx5_ibv_shared *sh = priv->sh; @@ -124 +125 @@ -@@ -308,7 +308,7 @@ flow_dv_shared_unlock(struct rte_eth_dev *dev) +@@ -295,7 +295,7 @@ flow_dv_shared_unlock(struct rte_eth_dev *dev) @@ -126 +127 @@ - struct mlx5_dev_ctx_shared *sh = priv->sh; + struct mlx5_ibv_shared *sh = priv->sh;