From patchwork Tue Jul 28 06:34:23 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ajit Khaparde X-Patchwork-Id: 74905 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 AEF56A0521; Tue, 28 Jul 2020 08:36:03 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id A0D7A1C126; Tue, 28 Jul 2020 08:35:19 +0200 (CEST) Received: from mail-wm1-f98.google.com (mail-wm1-f98.google.com [209.85.128.98]) by dpdk.org (Postfix) with ESMTP id 88A3E1C116 for ; Tue, 28 Jul 2020 08:35:02 +0200 (CEST) Received: by mail-wm1-f98.google.com with SMTP id g8so5025431wmk.3 for ; Mon, 27 Jul 2020 23:35:02 -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=DJfY20B7yZgnbZ+96QabsBfwe8qhnyJRmHQjde5cfqU=; b=Hqq0Fdph1PYPiibYRUeHpbBI/SgR7EQV48YoLieGmWIavFNI0Iq1UjDbNacgQQQz67 ikLg7qmD8qwuAVFiTu0sKXhoNCmQmzz+ZqHF+dYa0KK8Z9q3kPUYVmghmDMVq4Cxfdfb sI5WFMR94rcG1DoPd27BfPIvrNcfac5xwEiOA= 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=DJfY20B7yZgnbZ+96QabsBfwe8qhnyJRmHQjde5cfqU=; b=O7xabmKWESoiAVL+Jqqa+f1/pORysFkrSYN5measubCm8KB6ZyGOP3Ik4Y/wtddLFg f7WvZj7AwD+YebZ8QSNm3DPisQvNGlRzHUOFFD7LIu9SrnR/tY1QIIiKqzQFKoNjFaLP 75ZPbW9u8YT8dvFQHKW8QNvp7hpJzdjHHAcsaCF4m88K5bLzPuTw9c9NiTinM848ZjOd IaLwNOq3lVcvbYsTHhgHo5AJIc8deIMAegEzWC0whNdDYs+Di5Xi0M8CCqS6D58+j8eh XKfntL+jrYHjCS/wkKgIqyblv2Ow48xR07ONcM8BQ4ZKGHDymqT3jzCZIubtE+6RzKHd OVAA== X-Gm-Message-State: AOAM532SbDzeeHbrblTEjm8SwAne/YMu7H96gvMbuisajl3Qb3Jp+9uj cTqpYdgPYLSEu1AEbftk0tOsnpsD0bpkUpc8FCqwSQPwIy4jMa6lR064U+NBmB934LQ4B6aK2yS c9fudFMXkUoUtuZnBYN58a81xv2PtmJrb4eUEcbQboXFdmUopq7NMXQ62XGRDeVXvrFCqwnjMUS nk0g== X-Google-Smtp-Source: ABdhPJwDB3PTeu9QjZ2SKCCUrq7vuZdXA7EnpSFYtMiMtUnX6MHtyTB560xcNQref/yIoYiStTRmJYlgHng6 X-Received: by 2002:a1c:9914:: with SMTP id b20mr2451914wme.15.1595918102131; Mon, 27 Jul 2020 23:35:02 -0700 (PDT) Received: from localhost.localdomain ([192.19.223.252]) by smtp-relay.gmail.com with ESMTPS id r16sm30828wmh.29.2020.07.27.23.35.00 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Mon, 27 Jul 2020 23:35:02 -0700 (PDT) X-Relaying-Domain: broadcom.com From: Ajit Khaparde To: dev@dpdk.org Cc: ferruh.yigit@intel.com, Venkat Duvvuru , Sriharsha Basavapatna , Somnath Kotur Date: Mon, 27 Jul 2020 23:34:23 -0700 Message-Id: <20200728063439.23114-7-ajit.khaparde@broadcom.com> X-Mailer: git-send-email 2.21.1 (Apple Git-122.3) In-Reply-To: <20200728063439.23114-1-ajit.khaparde@broadcom.com> References: <20200728063439.23114-1-ajit.khaparde@broadcom.com> MIME-Version: 1.0 Subject: [dpdk-dev] [PATCH v4 06/22] net/bnxt: fix mark id update to mbuf 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: Venkat Duvvuru When a packet is looped back from VF to VFR, it is marked to identify the VFR interface. However, this mark_id shouldn't be percolated up to the OVS as it is internal to pmd. This patch fixes it by skipping mark injection into mbuf if the packet is received on VFR interface. Fixes: 6dc83230b43b ("net/bnxt: support port representor data path") Signed-off-by: Venkat Duvvuru Signed-off-by: Sriharsha Basavapatna Reviewed-by: Somnath Kotur --- drivers/net/bnxt/bnxt_rxr.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/net/bnxt/bnxt_rxr.c b/drivers/net/bnxt/bnxt_rxr.c index baf73cb25..43b1256dc 100644 --- a/drivers/net/bnxt/bnxt_rxr.c +++ b/drivers/net/bnxt/bnxt_rxr.c @@ -485,6 +485,9 @@ bnxt_ulp_set_mark_in_mbuf(struct bnxt *bp, struct rx_pkt_cmpl_hi *rxcmp1, rc = ulp_mark_db_mark_get(bp->ulp_ctx, gfid, cfa_code, vfr_flag, &mark_id); if (!rc) { + /* VF to VFR Rx path. So, skip mark_id injection in mbuf */ + if (vfr_flag && *vfr_flag) + return mark_id; /* Got the mark, write it to the mbuf and return */ mbuf->hash.fdir.hi = mark_id; mbuf->udata64 = (cfa_code & 0xffffffffull) << 32;