From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3D268A0562 for ; Fri, 19 Mar 2021 01:06:03 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 01AE54003F; Fri, 19 Mar 2021 01:06:03 +0100 (CET) Received: from mail-qk1-f170.google.com (mail-qk1-f170.google.com [209.85.222.170]) by mails.dpdk.org (Postfix) with ESMTP id 7A65B4003F for ; Fri, 19 Mar 2021 01:06:01 +0100 (CET) Received: by mail-qk1-f170.google.com with SMTP id i9so1193622qka.2 for ; Thu, 18 Mar 2021 17:06:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=ZtOpzdRoPOiU8IJwcRpoa1/F2E82jlnXIZrtRHnBwUo=; b=DaCnHU+0FmlRwzpv5p3Kw1e5ZUnUdTY/Wh+AskKkmQOBat9jBQ7Qn1pIbf+hZFLJ21 kF1wQeh3LkiNZweo4QYXXJ8eYBAw1TWPVa3HEGxXDEl7hbO4GsciTAAuK6oIYd1urq5N dyLgphuU1KlZ7HIEFqOzAfgPd4L2Od79jgJLo= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=ZtOpzdRoPOiU8IJwcRpoa1/F2E82jlnXIZrtRHnBwUo=; b=NgGXpvf4wZICxG5Z+UwwT+2KuJFEzhlYF+P3lablWx/emRU7H3XadN5MR3J1a39mtz ikoGKHaxp2vPnymUhLT3suQtmvdkilH8VJm1B/e7XsWx5FKNbNwjRUP81ksYsbYcFFU8 JB5dGlGOtkiZs/8ezbyXfvdcpshHcSLl9YRSh2Oi57S404F7ZLVxclJYRAq2w7e67rfR D7Tl9pbnggs64nIKR8QQToHin+VWbxd6RGSVYJTfry89byWOj5QZOFcxv58QLu7yfHzA LP3UAODOUsfG9crmKDtG/fv0S2QfTB7syuBI1ypEaB0qk8qpF7M/rEdb3hDZsOYx/+p7 LIqA== X-Gm-Message-State: AOAM530S6HYCmszFgSMAPTIo+UXxTToMvLAuXn5PgtFVf/z1YCujUEMR AAO1e+BYJd97HWc5Blcq8esSuUIHsBvPUla9cLr2b/KdwDg9bE6Q X-Google-Smtp-Source: ABdhPJzxYmxnFDrsOpK2t5GbUHF//KtuEaH9opoLVUeDK1pq2RqSIdXOfnBkzH/rpxs4JfOqQzocmjH1FqWbr84Ex+M= X-Received: by 2002:a37:43cc:: with SMTP id q195mr7151650qka.455.1616112360704; Thu, 18 Mar 2021 17:06:00 -0700 (PDT) MIME-Version: 1.0 References: <20210318004730.24360-1-ajit.khaparde@broadcom.com> In-Reply-To: <20210318004730.24360-1-ajit.khaparde@broadcom.com> From: Ajit Khaparde Date: Thu, 18 Mar 2021 17:05:44 -0700 Message-ID: To: dpdk-dev Cc: dpdk stable Content-Type: multipart/signed; protocol="application/pkcs7-signature"; micalg=sha-256; boundary="00000000000033a84d05bdd87b2b" X-Content-Filtered-By: Mailman/MimeDel 2.1.29 Subject: Re: [dpdk-stable] [PATCH] net/bnxt: fix RSS context cleanup X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 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" --00000000000033a84d05bdd87b2b Content-Type: text/plain; charset="UTF-8" On Wed, Mar 17, 2021 at 5:47 PM Ajit Khaparde wrote: > > The PMD is allocating an extra RSS context with each port start. > But it is freeing only one RSS context during port stop. So at some point > we run out of RSS contexts when we do multiple port stop/start sequences. > bnxt_hwrm_vnic_ctx_alloc() is called by bnxt_setup_one_vnic(), but > bnxt_hwrm_vnic_ctx_free() is not called in the corresponding > bnxt_free_one_vnic(). > > Fix this by calling bnxt_hwrm_vnic_ctx_free() in bnxt_free_one_vnic(). > > Fixes: 7fe5668d2ea3 ("net/bnxt: support VLAN filter and strip") > Cc: stable@dpdk.org > > Signed-off-by: Ajit Khaparde Patch applied to dpdk-next-net-brcm. > --- > drivers/net/bnxt/bnxt_ethdev.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c > index 1997783a7d..ff92f999db 100644 > --- a/drivers/net/bnxt/bnxt_ethdev.c > +++ b/drivers/net/bnxt/bnxt_ethdev.c > @@ -2531,6 +2531,10 @@ static int bnxt_free_one_vnic(struct bnxt *bp, uint16_t vnic_id) > } > bnxt_del_dflt_mac_filter(bp, vnic); > > + rc = bnxt_hwrm_vnic_ctx_free(bp, vnic); > + if (rc) > + return rc; > + > rc = bnxt_hwrm_vnic_free(bp, vnic); > if (rc) > return rc; > -- > 2.21.1 (Apple Git-122.3) > --00000000000033a84d05bdd87b2b--