From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f193.google.com (mail-wr0-f193.google.com [209.85.128.193]) by dpdk.org (Postfix) with ESMTP id 0A8F0F64 for ; Mon, 30 Apr 2018 16:54:19 +0200 (CEST) Received: by mail-wr0-f193.google.com with SMTP id v60-v6so8315362wrc.7 for ; Mon, 30 Apr 2018 07:54:19 -0700 (PDT) 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; bh=nl5J3kAdk99hl/Nbd3RWeOr7CAP/BgZ4Q4qPgXDwd74=; b=WBt0i32sL70xd30ItHi+omUrKmfFVP/eUmEhv/mIb+WaB3bUJfl/v+xBDVCctwUfo0 Y8PXra+Iz714zaYtRMhc2vdjgYSrwQn1KF9efwgj2L8GAqqjqKW98He6oD4C7HirvoZ3 kjYWERM2q5HGTsUXw8UimlvPgLuo33Kx1EXXhvsOzug8jlmh5MDVmtYnrE2AiNGNn1nx rxicvQvzEhB6aPqKOp4y703+gSnsQ3pez0m1ziblwt+EeH9cKCSQ7M+wCiCGbzUg7VuJ YyfJRMemUuUxRkHoF6Plkqpa1H4JoHIdM8ni8loh5aSV+cU2cgN9iST9iPW72Gbn4yuG 2rdg== 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; bh=nl5J3kAdk99hl/Nbd3RWeOr7CAP/BgZ4Q4qPgXDwd74=; b=gWNZr7aM1CWo/5giTdJJ6+2WpuF3eo44dPqpmGgz19iIKEAzvbWl1mXlnDdMW48hB7 L8YSw32OhnWe3MiI5tRP2KRDOSnln8tqA/E1UALp5243u8Yef97ns13At+rv+d3hBB9U 7Dy0HB16rb/gymo89bqd28BUG1tB24K1iW0wgeIOHasFgOJiSQ/llq3VDSVStClEcjex YtuiRRJjhIKSzAUuONpzlk+ft8w7TbunTgMQEBKKjw/7Egj0/7z44kUsd2U0FVrLhNB8 HBkvEW5CsQMvscCoPfPYsgigDDW5bGxvSwfXXcD2RQdwjFEDXwCsKUABnDGPCV51BwVe 4rTA== X-Gm-Message-State: ALQs6tB9t8Vz1X89NIPP/XiZrRaI7x519FJAOkER3vAmMcqixpDaBhZG u994E9kK8ut8B9t4363X3cCFiy825D4= X-Google-Smtp-Source: AB8JxZoFFFdawN0GHZSD8JsanMKJd/7Ao93JseSKoGQHWwju+0VvPhzGTMkepyRU3Pcq3/7p/jJFTQ== X-Received: by 2002:adf:a6f8:: with SMTP id t111-v6mr9019152wrc.161.1525100058779; Mon, 30 Apr 2018 07:54:18 -0700 (PDT) Received: from localhost ([2a00:23c5:be9a:5200:ce4c:82c0:d567:ecbb]) by smtp.gmail.com with ESMTPSA id 80sm14282941wml.44.2018.04.30.07.54.17 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 30 Apr 2018 07:54:17 -0700 (PDT) From: luca.boccassi@gmail.com To: Ajit Khaparde Cc: dpdk stable Date: Mon, 30 Apr 2018 15:53:28 +0100 Message-Id: <20180430145402.23057-9-luca.boccassi@gmail.com> X-Mailer: git-send-email 2.14.2 In-Reply-To: <20180430145402.23057-1-luca.boccassi@gmail.com> References: <20180430144223.18657-85-luca.boccassi@gmail.com> <20180430145402.23057-1-luca.boccassi@gmail.com> Subject: [dpdk-stable] patch 'net/bnxt: fix flow destroy' has been queued to stable release 18.02.2 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: , X-List-Received-Date: Mon, 30 Apr 2018 14:54:19 -0000 Hi, FYI, your patch has been queued to stable release 18.02.2 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 05/02/18. So please shout if anyone has objections. Thanks. Luca Boccassi --- >>From 70f16a7042b198ef2d31c7e005122684dc4e337d Mon Sep 17 00:00:00 2001 From: Ajit Khaparde Date: Mon, 16 Apr 2018 18:11:19 -0700 Subject: [PATCH] net/bnxt: fix flow destroy [ upstream commit 608e6fd6ab4f03b3f629779db47e8effe19ff4bc ] bnxt_hwrm_clear_l2_filter needs to be called only if the filter type is L2 and not otherwise. Also check for the return value of bnxt_hwrm_clear_l2_filter(). Fixes: 5ef3b79fdfe6 ("net/bnxt: support flow filter ops") Signed-off-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_filter.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/bnxt/bnxt_filter.c b/drivers/net/bnxt/bnxt_filter.c index b7718cdca..64a23f628 100644 --- a/drivers/net/bnxt/bnxt_filter.c +++ b/drivers/net/bnxt/bnxt_filter.c @@ -1182,8 +1182,8 @@ bnxt_flow_destroy(struct rte_eth_dev *dev, ret = bnxt_hwrm_clear_em_filter(bp, filter); if (filter->filter_type == HWRM_CFA_NTUPLE_FILTER) ret = bnxt_hwrm_clear_ntuple_filter(bp, filter); - - bnxt_hwrm_clear_l2_filter(bp, filter); + else + ret = bnxt_hwrm_clear_l2_filter(bp, filter); if (!ret) { STAILQ_REMOVE(&vnic->flow_list, flow, rte_flow, next); rte_free(flow); -- 2.14.2