From patchwork Fri Aug 30 14:00:06 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sriharsha Basavapatna X-Patchwork-Id: 143468 X-Patchwork-Delegate: ajit.khaparde@broadcom.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 51B12458A8; Fri, 30 Aug 2024 15:51:38 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id BA80A42EAC; Fri, 30 Aug 2024 15:51:12 +0200 (CEST) Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com [209.85.214.174]) by mails.dpdk.org (Postfix) with ESMTP id 63EED40291 for ; Fri, 30 Aug 2024 15:51:10 +0200 (CEST) Received: by mail-pl1-f174.google.com with SMTP id d9443c01a7336-201e64607a5so14531055ad.2 for ; Fri, 30 Aug 2024 06:51:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; t=1725025869; x=1725630669; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=2uQKaMu+75/OrhVAdYbc60KaEuLlpQZzWTNakjmiCPQ=; b=VA+tobbI2gTxy41S0FgaQCQzS4t25CWqlq8y0F5rKvFqCBXNhP9kem0fvILfet0fiR W5EAi8MJwVVU4EBBdRgy7y/FtGcCKaEmHy7jjfzxlhIkufmGW1G6YMEf8ceaHdv51TZe 0B4hW0Bsg75dOlydyefRjLAcqVWrlUGJasP+0= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1725025869; x=1725630669; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=2uQKaMu+75/OrhVAdYbc60KaEuLlpQZzWTNakjmiCPQ=; b=h0aDQO5vEpUwDANeUnSXOYW2fjUc//HTTgk29UGoekPyDPztnzv+33OrJqA2BA+Y4Y +6SndqoFsCnn7ZYsauYevSwsm/yljQ2e9RB8pGxJ+LapieLWOFAO2yrDDUuFiwGU17Wy 4avLxmSZl7ModrVpdrMXYynBhpxDdef0xaeTlUEhsYrgbuI2t1cW2Rm9B3jP/ZgVl5Z/ ZDL85tlZ6OawG+QTJGm38cXuvpmrpIkt7Ee7G0zMCeiH4wq52sZjiA00Dog+PxqF8EUM WiL50N2AXxuJ3/c3GiWztVtPOlgZFDioJocLNcUcPUoYIdGLYKL4+ix/6Lo+HFAyJqlw wSaA== X-Gm-Message-State: AOJu0Yw2uRZUkmmy6+PMP+jG9JN6D/KAZJX2nRgR6/nLU0Nmstr7zIRz 3tyjVtYV3TJyBMLbnnErFOVZpWIod8iFHY7gO8WpEZ7eHXLBAIy6dqIKUeSg7zFxJMZutwvmYhU 1dT/ljVsvYP62+7l0rHCScsx4uJmEtvgWQeFZ2iofvz72QFc+uXoRYiI1qY64NxUgzO5DacGdiW 1P7DajNGC9cn+A8wTHjrCgpbP13lZXHSKOnkDyr+y3ZQMJ X-Google-Smtp-Source: AGHT+IHWeVoDj554UMS90aoSrEbXqrDzltABKVeKy/rcoLh7VJQ8SRi1px2c0Y8O4Wmi4HS+dhiaHA== X-Received: by 2002:a17:902:dac2:b0:1ff:4d66:d7bb with SMTP id d9443c01a7336-2050c3730bfmr76751775ad.36.1725025868978; Fri, 30 Aug 2024 06:51:08 -0700 (PDT) Received: from dhcp-10-123-154-23.dhcp.broadcom.net ([192.19.234.250]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-205155658dfsm27067145ad.297.2024.08.30.06.51.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 30 Aug 2024 06:51:08 -0700 (PDT) From: Sriharsha Basavapatna To: dev@dpdk.org Cc: Farah Smith , Kishore Padmanabha , Shahaji Bhosle , Sriharsha Basavapatna Subject: [PATCH 04/47] net/bnxt: tf_core: Thor TF EM key size check Date: Fri, 30 Aug 2024 19:30:06 +0530 Message-Id: <20240830140049.1715230-5-sriharsha.basavapatna@broadcom.com> X-Mailer: git-send-email 2.39.0.189.g4dbebc36b0 In-Reply-To: <20240830140049.1715230-1-sriharsha.basavapatna@broadcom.com> References: <20240830140049.1715230-1-sriharsha.basavapatna@broadcom.com> MIME-Version: 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org From: Farah Smith The maximum EM key size is 640 bits for Thor. But the lookup record + the key size is 679 bits. This value must be rounded up to a 128 bit aligned number. So the size check should be 96 bytes rather than 80. This fix allows keys > 601 bits to be successfully inserted. Fixes: 539931eab3a5 ("net/bnxt: support EM with FKB") Signed-off-by: Farah Smith Reviewed-by: Farah Smith Reviewed-by: Kishore Padmanabha Reviewed-by: Shahaji Bhosle Signed-off-by: Sriharsha Basavapatna --- drivers/net/bnxt/tf_core/tf_msg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/bnxt/tf_core/tf_msg.c b/drivers/net/bnxt/tf_core/tf_msg.c index 08e9783d52..dd5ea1c80e 100644 --- a/drivers/net/bnxt/tf_core/tf_msg.c +++ b/drivers/net/bnxt/tf_core/tf_msg.c @@ -25,7 +25,7 @@ */ #define TF_MSG_SET_GLOBAL_CFG_DATA_SIZE 16 #define TF_MSG_EM_INSERT_KEY_SIZE 64 -#define TF_MSG_EM_INSERT_RECORD_SIZE 80 +#define TF_MSG_EM_INSERT_RECORD_SIZE 96 #define TF_MSG_TBL_TYPE_SET_DATA_SIZE 88 /* Compile check - Catch any msg changes that we depend on, like the