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 44029A04B5 for ; Thu, 29 Oct 2020 11:36:53 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 0BAE1C960; Thu, 29 Oct 2020 11:36:52 +0100 (CET) Received: from mail-wr1-f47.google.com (mail-wr1-f47.google.com [209.85.221.47]) by dpdk.org (Postfix) with ESMTP id 582F6C960 for ; Thu, 29 Oct 2020 11:36:50 +0100 (CET) Received: by mail-wr1-f47.google.com with SMTP id x7so2188157wrl.3 for ; Thu, 29 Oct 2020 03:36:50 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to :references:content-transfer-encoding:user-agent:mime-version; bh=6TDaeECK40J8ithrzFe3o7HczwgQ9h6ti6zLSszVp6k=; b=BdVvnJ+PZViCJI+o7CNMOT3XFIMVVN6zzPHt7Iu9AXrC04lPkW2yjnGVaTC1pcv8zN VIE+gxFdxo1zLvmmyJ2LZAUuwyh1pX2T8u/dQz1ZzMV/MDi+AYcF6Tmto4vmpzaPp3AZ He969UgwXmNnDph8g8UofubRNjRdF9jD0zWUctNe/tIGcRmNqKgno3tEdpEOhAkPmSJB lv6n9jZmtu06KzFtcnyq8lSlM92SEsXzKY597Dy3AiSQaiET3wxPG6RNx890khRxvGaI aMigcEtGrFz9sV8wVpYvohUAdbAf87FcozLxd00hD8HFLWGI8hroDDb3vwc3zky1s3Nd U/TA== X-Gm-Message-State: AOAM532j3KVVQPOalncTNnldGcgYV9Q3teBUJ21ha6ynX+zywCFn50wZ LnP//2JBnYseP+WPnWZfOpmyYgcGyu6ex3hRqhI= X-Google-Smtp-Source: ABdhPJw4PVOA3Wv0NF58kWmevmkyNrmZC2FK8vqRuD2ZwwOROzGIatYqLjp0CM7fpS/49Y+7OnGQ7w== X-Received: by 2002:adf:edc5:: with SMTP id v5mr4665041wro.112.1603967809134; Thu, 29 Oct 2020 03:36:49 -0700 (PDT) Received: from localhost ([88.98.246.218]) by smtp.gmail.com with ESMTPSA id t6sm812771wrp.68.2020.10.29.03.36.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 29 Oct 2020 03:36:48 -0700 (PDT) Message-ID: <97365917b7997e22444eefdfb3e9f1e45f543c7a.camel@debian.org> From: Luca Boccassi To: Somnath Kotur , stable@dpdk.org Cc: Ajit Khaparde Date: Thu, 29 Oct 2020 10:36:47 +0000 In-Reply-To: <20201029084419.9783-1-somnath.kotur@broadcom.com> References: <20201029084419.9783-1-somnath.kotur@broadcom.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.30.5-1.1 MIME-Version: 1.0 Subject: Re: [dpdk-stable] [19.11 1/2] net/bnxt: fix checking VNIC in shutdown path 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 Thu, 2020-10-29 at 14:14 +0530, Somnath Kotur wrote: > [ backported from upstream commit 4b029f02de3a0ce9cdd9a3475b84ca2e42d7428= 1 ] >=20 > Add a couple of NULL pointer checks in bnxt_free_all_filters() > and bnxt_free_vnics() respectively to guard against certain error > injection/recovery scenarios where it was found that the application > was crashing with the bp->vnic_info pointer being NULL. >=20 > Fixes: 51fafb89a9a0 ("net/bnxt: get rid of ff pools and use VNIC info arr= ay") > Cc: stable@dpdk.org >=20 > Signed-off-by: Somnath Kotur > Reviewed-by: Ajit Khaparde > --- > drivers/net/bnxt/bnxt_filter.c | 15 +++++++++------ > drivers/net/bnxt/bnxt_vnic.c | 3 +++ > 2 files changed, 12 insertions(+), 6 deletions(-) Hi, Both of these patches appear to be 100% identical to the ones I pushed yesterday? Was there something unclear in the generated email that made you think the rebase was different that we can fix? https://github.com/bluca/dpdk-stable/commit/77abebfe906e80404ecdfa71c60da31= 8cd0fb563 https://github.com/bluca/dpdk-stable/commit/618bfe2417717a3b446180a10d1a229= 9d3f626e9 --=20 Kind regards, Luca Boccassi