From patchwork Sat Oct 10 04:05:10 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ajit Khaparde X-Patchwork-Id: 80246 X-Patchwork-Delegate: ajit.khaparde@broadcom.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 2302BA04BC; Sat, 10 Oct 2020 06:07:22 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 10F451D63E; Sat, 10 Oct 2020 06:07:20 +0200 (CEST) Received: from mail-ot1-f99.google.com (mail-ot1-f99.google.com [209.85.210.99]) by dpdk.org (Postfix) with ESMTP id A66C31D63E for ; Sat, 10 Oct 2020 06:07:18 +0200 (CEST) Received: by mail-ot1-f99.google.com with SMTP id n61so10846884ota.10 for ; Fri, 09 Oct 2020 21:07:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=fhyxQgccbhTxm4aqRro0eF3BNVW9QJEWMufpAKagYHE=; b=fmuXcV10Qn6GSi3xIdrz4Owm03DxtFfboJ0LEkJVKbSizJvk1JNI7pCNSOrT8Qm9VF kviX7wj1M3EiuK2J7SAjLgwnhygc1S4ZarG2QFuu+D05UhOeUzW9UYJHoly8a9z8godG rhwN5KUbZRc4OGOk1rmkCQQkbjuqvdi9Eimk8= 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=fhyxQgccbhTxm4aqRro0eF3BNVW9QJEWMufpAKagYHE=; b=QHWMLB15gW0NGBHem32Zi5lkHbBL/qAd0DnH2eUiklW9V2jmDnOl3k0EPb3fL/oycO mhhyBzs4+iEcP+Bo9+ER+aWjthxaZ67CivrtGM3YemA/oXdL/7Fw485G3Q09dPh8gylx 7j1edOZugDEEighfMOmrbOzLNMhZx3bmhylEoa9EBeZpXxFvSr/81syaeTb0yQTqPMs0 QPsaTdfhw9VR2y7Fg84WfDex88ao4ApxvJppiN2B0C/eB63FX0EjSSAaAyVBex1+bliP P/TJSDjw+iKT7wr05sjlqYC6A8VVlbS6hmJaOx45dwK6nNaTSdH4tdyLW5p8QFckF+En ukNA== X-Gm-Message-State: AOAM530vyBTiBQWnBYsbpGINvSwtRAfDqbyzmspwlu/s6w8Ki+xzsKsi 8OAQFoBdkA4CwjSE0SPaFcH7nk5km2FvyCTvne5G3N+C1852YCRrJIt7IXMjifBrhIWNK/62SPA Etw9BbYGpnQwNoF7OGA7pdXmQYrPrMIAQJzQFD13SF20m7jN5jgiw8EloFO27bs426MC2bRK83A PLIg== X-Google-Smtp-Source: ABdhPJyiO1rSI2+HMlLO9OFOGsrAbksglcgZm/TSh1AWm4vHbJ7WHkbe5QPb+N1/v4xP8bOS6EO4YOFM+zDs X-Received: by 2002:a05:6830:2436:: with SMTP id k22mr11541241ots.185.1602302836865; Fri, 09 Oct 2020 21:07:16 -0700 (PDT) Received: from localhost.localdomain ([192.19.223.252]) by smtp-relay.gmail.com with ESMTPS id 67sm1555892oop.24.2020.10.09.21.07.16 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Fri, 09 Oct 2020 21:07:16 -0700 (PDT) X-Relaying-Domain: broadcom.com From: Ajit Khaparde To: dev@dpdk.org Cc: Kishore Padmanabha , stable@dpdk.org, Mike Baucom Date: Fri, 9 Oct 2020 21:05:10 -0700 Message-Id: <20201010040520.63049-2-ajit.khaparde@broadcom.com> X-Mailer: git-send-email 2.21.1 (Apple Git-122.3) In-Reply-To: <20201009111130.10422-1-somnath.kotur@broadcom.com> References: <20201009111130.10422-1-somnath.kotur@broadcom.com> MIME-Version: 1.0 Subject: [dpdk-dev] [PATCH v2 01/12] net/bnxt: fix the corruption of the session details X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" From: Kishore Padmanabha The session details that is shared among multiple ports need to be outside the bnxt structure. Fixes: 70e64b27af5b ("net/bnxt: support ULP session manager cleanup") Cc: stable@dpdk.org Signed-off-by: Kishore Padmanabha Reviewed-by: Mike Baucom Reviewed-by: Ajit Khaparde --- drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/net/bnxt/tf_ulp/bnxt_ulp.c b/drivers/net/bnxt/tf_ulp/bnxt_ulp.c index 289619411..a4d48c71a 100644 --- a/drivers/net/bnxt/tf_ulp/bnxt_ulp.c +++ b/drivers/net/bnxt/tf_ulp/bnxt_ulp.c @@ -159,7 +159,9 @@ ulp_ctx_session_open(struct bnxt *bp, } if (!session->session_opened) { session->session_opened = 1; - session->g_tfp = &bp->tfp; + session->g_tfp = rte_zmalloc("bnxt_ulp_session_tfp", + sizeof(struct tf), 0); + session->g_tfp->session = bp->tfp.session; } return rc; } @@ -176,6 +178,7 @@ ulp_ctx_session_close(struct bnxt *bp, if (session->session_opened) tf_close_session(&bp->tfp); session->session_opened = 0; + rte_free(session->g_tfp); session->g_tfp = NULL; }