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 E3581A04B5 for ; Sun, 1 Nov 2020 04:18:56 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id D56235AB7; Sun, 1 Nov 2020 04:18:55 +0100 (CET) Received: from mail-ot1-f67.google.com (mail-ot1-f67.google.com [209.85.210.67]) by dpdk.org (Postfix) with ESMTP id 0EF965AB7 for ; Sun, 1 Nov 2020 04:18:55 +0100 (CET) Received: by mail-ot1-f67.google.com with SMTP id n11so9409091ota.2 for ; Sat, 31 Oct 2020 20:18:54 -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=J/XQZmQMXE+IwILzXnVJhEmcBQXZ5zkHxgbKmCncM7g=; b=WgeQoJpLemdrUHC0EImTBs7LMnZXsJOSVk6DPgnCxdIorfxyGg03exVi+Linj0aJfV Ww96sNXuhLIagc2KD11u5p6teG/jq0rs60Hao+zzXj9qNCBn9Z4Y+5DX+BKYeG7trDmJ 1PqrCk4Gjd2UBIY/SZ/7R7eI24muHgqXLlry0= 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=J/XQZmQMXE+IwILzXnVJhEmcBQXZ5zkHxgbKmCncM7g=; b=nXZNAUhbcGBUkbIEJpQ+LRm7e2t/YRK7ODVJceweiXIrxz8NCVxdX437PeX7XFznij 5Foi55yErh/VQ1fyF6i6g4NSAqrtdyMErnBAc6zEpYXwxWG2KUHUQ9GJuCe6HI52H7ke Wr++VpzdoyODVwRcGKSnNlvAdgSGu99ejZSzq+rCvcudrkjDbT7xi/sP8/G5uZEz6kJL NRpoIkuUaOHZ4vEjVT6ZpmQgHjpdt3iGvUCDuIlykwrFhvRm+3Tu+w0uhtP0KFMCaHPE vcLHw1xEMeJThGEOMMlDvHbmnK3OVkFPnonbm5VW+BR/sdBldUgKzgCEvcB4Ma7Pl78d FYGg== X-Gm-Message-State: AOAM530NkS2Fk0ql/tZP4G+ju+0u1zKAu/33MYzEOH7dLcoik2rSIRA/ gp4zBp9tMEV/aCLrgBY4vRV6pvm5xXCpP9qNVWiNnQ== X-Google-Smtp-Source: ABdhPJwUFgKcIAciDrGZMV1bLOpSyTc01buvGBwzwOwXEpSMAyrhiqgYUVjsKn0L7TmVkwRxnD9tqgZKfBMRJXDdLvI= X-Received: by 2002:a9d:4d83:: with SMTP id u3mr2714498otk.283.1604200733239; Sat, 31 Oct 2020 20:18:53 -0700 (PDT) MIME-Version: 1.0 References: <20201030070304.22318-1-somnath.kotur@broadcom.com> In-Reply-To: <20201030070304.22318-1-somnath.kotur@broadcom.com> From: Ajit Khaparde Date: Sat, 31 Oct 2020 20:18:36 -0700 Message-ID: To: Somnath Kotur Cc: dpdk-dev , Ferruh Yigit , dpdk stable , Kalesh Anakkur Purayil Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH 1/2] net/bnxt: log failure for switch domain free 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" On Fri, Oct 30, 2020 at 12:11 AM Somnath Kotur wrote: > > Check and log an error message if switch domain free API fails > > Coverity issue: 362757 > > Fixes: 322bd6e70272 ("net/bnxt: add port representor infrastructure") > Cc: stable@dpdk.org > > Signed-off-by: Somnath Kotur > Reviewed-by: Kalesh Anakkur Purayil Patch applied to dpdk-next-net-brcm. > --- > drivers/net/bnxt/bnxt_ethdev.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c > index b2f72ea..31e94f2 100644 > --- a/drivers/net/bnxt/bnxt_ethdev.c > +++ b/drivers/net/bnxt/bnxt_ethdev.c > @@ -1343,8 +1343,14 @@ static int bnxt_dev_set_link_down_op(struct rte_eth_dev *eth_dev) > > static void bnxt_free_switch_domain(struct bnxt *bp) > { > - if (bp->switch_domain_id) > - rte_eth_switch_domain_free(bp->switch_domain_id); > + int rc = 0; > + > + if (bp->switch_domain_id) { > + rc = rte_eth_switch_domain_free(bp->switch_domain_id); > + if (rc) > + PMD_DRV_LOG(ERR, "free switch domain:%d fail: %d\n", > + bp->switch_domain_id, rc); > + } > } > > /* Unload the driver, release resources */ > -- > 2.7.4 >