From patchwork Fri May 22 23:55:01 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mike Baucom X-Patchwork-Id: 70543 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 9F945A0350; Sat, 23 May 2020 01:55:26 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 5D6EF1D9E5; Sat, 23 May 2020 01:55:26 +0200 (CEST) Received: from mail-wm1-f66.google.com (mail-wm1-f66.google.com [209.85.128.66]) by dpdk.org (Postfix) with ESMTP id 43D8E1D9D6 for ; Sat, 23 May 2020 01:55:25 +0200 (CEST) Received: by mail-wm1-f66.google.com with SMTP id v19so4436252wmj.0 for ; Fri, 22 May 2020 16:55:25 -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; bh=9tpP9KEOj8Drjg6mgr600ilu08QBa6QA5NjKlvOZtkE=; b=hJI1jPL2QAA4l1bfj2WFGq+6lfJnCDgQWwBRPQaP29CLWMx2/AnT98Vrmh1G6jA5ty I03ZaPtjLBsGG+G6AV6+Hpj5u3Sn3gb/4Lx8qD9EtUt1DYvEkB5x69WS6Qs7pIM32N+X GS6+b8aJME0N/yshDXs0NYtAVI0TboE0DuZAQ= 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; bh=9tpP9KEOj8Drjg6mgr600ilu08QBa6QA5NjKlvOZtkE=; b=DGkJ2nPrFb9mkrEwET47Ve22ZQ0ciwf6A9ba0wDiWEEKOd7WSKKnGfedJNUA5JyVGG 67LpB4h4ZVCv9II+LvpTE/O5NpDDJtjebgkOCHggT2C/fDxYXLGOuN5XYpx10wwcR+Jc EKILahWxdwFaZAfsbVg3Yr7sF6S3z6ez9QrUkYKEztgrW+CKbRzxHZkCzKPJj165QQiP tNM6xYaiS3KzOwZggiL3GvDf04oRmZgBiPZ3hLMJC0Ww8mu+1wrZBtvpUJwnRsING/3d dVuoWGg0KQTzHhBnPjw8MfWsbs7i4bkLNZYHU9AFjCsRlovjdUZV8o9AimeFPgcCQSAX /Y4Q== X-Gm-Message-State: AOAM533I2V+cDc5rPPKOD+bt7D0LyQC/VuIPNhClQ+drLxzjMhWM8Aeb +t5a1KYT8dIU1/+oalvMynk2pCm3NHy+QZIAQdretkVcLUhBuzEC3BT8Y7a5W4xU9xjl3Wqt5QY HBZ3PyE25B+9Y2fOwLTjbLds3lndO1/aANeoitbxSxPY4JCj2+59EfKFXNujuvw== X-Google-Smtp-Source: ABdhPJysO/IxbNcqteHbHTTAil9WxHOwkfUYErXTjWJ29ieUb8r1q/T4fFE0orXqp1jrzrMJsdWMCQ== X-Received: by 2002:a05:600c:22d3:: with SMTP id 19mr16272152wmg.161.1590191724607; Fri, 22 May 2020 16:55:24 -0700 (PDT) Received: from lab-rtp-b05-01.rtp.broadcom.com ([192.19.231.250]) by smtp.gmail.com with ESMTPSA id d13sm10577597wmb.39.2020.05.22.16.55.23 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Fri, 22 May 2020 16:55:24 -0700 (PDT) From: Mike Baucom To: dev@dpdk.org Cc: michael.baucom@broadcom.com Date: Fri, 22 May 2020 19:55:01 -0400 Message-Id: <1590191701-8420-1-git-send-email-michael.baucom@broadcom.com> X-Mailer: git-send-email 1.9.1 Subject: [dpdk-dev] [PATCH] net/bnxt: allow the mark to use a cfa code of zero 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" The mark code was too restrictive by disallowing a cfa_code of zero. This code loosens the requirement and allows zero. Fixes: b87abb2e55cb ("net/bnxt: support marking packet") Signed-off-by: Mike Baucom Reviewed-by: Kishore Padmanabha --- drivers/net/bnxt/bnxt_rxr.c | 20 +++++++++----------- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git a/drivers/net/bnxt/bnxt_rxr.c b/drivers/net/bnxt/bnxt_rxr.c index ee1acb1..91ff729 100644 --- a/drivers/net/bnxt/bnxt_rxr.c +++ b/drivers/net/bnxt/bnxt_rxr.c @@ -465,17 +465,15 @@ static inline struct rte_mbuf *bnxt_tpa_end( break; } - if (cfa_code) { - rc = ulp_mark_db_mark_get(bp->ulp_ctx, gfid, - cfa_code, &mark_id); - if (!rc) { - /* Got the mark, write it to the mbuf and return */ - mbuf->hash.fdir.hi = mark_id; - mbuf->udata64 = (cfa_code & 0xffffffffull) << 32; - mbuf->hash.fdir.id = rxcmp1->cfa_code; - mbuf->ol_flags |= PKT_RX_FDIR | PKT_RX_FDIR_ID; - return; - } + rc = ulp_mark_db_mark_get(bp->ulp_ctx, gfid, + cfa_code, &mark_id); + if (!rc) { + /* Got the mark, write it to the mbuf and return */ + mbuf->hash.fdir.hi = mark_id; + mbuf->udata64 = (cfa_code & 0xffffffffull) << 32; + mbuf->hash.fdir.id = rxcmp1->cfa_code; + mbuf->ol_flags |= PKT_RX_FDIR | PKT_RX_FDIR_ID; + return; } skip_mark: