From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 4187E43380; Mon, 20 Nov 2023 17:41:14 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D16BD42DDE; Mon, 20 Nov 2023 17:41:13 +0100 (CET) Received: from mail-pf1-f182.google.com (mail-pf1-f182.google.com [209.85.210.182]) by mails.dpdk.org (Postfix) with ESMTP id E653B42DD2 for ; Mon, 20 Nov 2023 17:41:12 +0100 (CET) Received: by mail-pf1-f182.google.com with SMTP id d2e1a72fcca58-6cb9dd2ab56so947338b3a.3 for ; Mon, 20 Nov 2023 08:41:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1700498472; x=1701103272; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=UvkrG7MldHCKOqQlLt5r7z7mg7BaowttFQwSPQMkP08=; b=XovFE7C3qHPwENpYudTz4fD9Q1DN1fsyxMFd0XehlNSr4JmRFhSltz4IXYB+AcPb7Z hxx4D4XNjNepfsU9V3pDkhBpT7FKLfLj5nBHTS1zuB/Is+zjlObk7x2SHG8QAjQU2qSi lwMefAhJCUx+Cy6PqKMUypwd/waelGoJw9k60UbmIj7RM0wfVWWPjC0FNeHt5QbgQsOA H9ne4cA0+EI6dKIm+PzfmP9jyjUw2ezmKSjFwOpN9/3RjkDFDnLPCEDLiyxkQCaFdsKm vl/E6mL4qYruWwH3JaKDN7W/TbPCwIRLvcvG7Rx/C5mWtunhuDOKEQQr5MxLc9wETbW/ G66Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1700498472; x=1701103272; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=UvkrG7MldHCKOqQlLt5r7z7mg7BaowttFQwSPQMkP08=; b=ZLCmjFk8o00e1yOhkd7HpzGfAYKlDiRkP0vzxRYyzPJS/7+Vp43QL3+0MP8SKEY0jZ HKslND3QMKHVHsaW7BN9r3PIyqGyzB1vTwDsKxHbVyz7jh2lM324KnMS+ZQWrLyBqErG 2Y6bDknvVsFkSVlRR1fZ0M43IHpQ3bbqQFNEhaWhfGEeBO5iAFJFJu2iJ7lsxmmon6Hb t5IR7dMa/YHapdYsEIG94UUOypUsKWkHbdwAPIk89ezggeHQWS4RALvMK9uWWAPwQKjR +Zegni8RtxPRvEzWEby2jAn/PWxI/6gV4UjgL6hAsX3z4ueXLMv8kbB3fz6WRE7431O4 W/ng== X-Gm-Message-State: AOJu0YytuAu/32TVgt4TjPHXPqx5Pfu5vyFlbPkl+5RXc9JOiO0gE5cG uAyxlIG/mOYquoUjWfw8bfjLkg== X-Google-Smtp-Source: AGHT+IFG/dLNIYL4xRPETx8eDX93Kl06SC27kWLqgmr1AfCu6Vu+/kBKpSbkdkSdfJ6TAhT7ePcRgw== X-Received: by 2002:a05:6a00:1d10:b0:6c2:bc81:2787 with SMTP id a16-20020a056a001d1000b006c2bc812787mr7874674pfx.6.1700498471991; Mon, 20 Nov 2023 08:41:11 -0800 (PST) Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id a8-20020a62bd08000000b006c2d5a31ab9sm6310269pff.81.2023.11.20.08.41.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Nov 2023 08:41:11 -0800 (PST) Date: Mon, 20 Nov 2023 08:41:10 -0800 From: Stephen Hemminger To: Robin Jarry Cc: dev@dpdk.org, Sunil Kumar Kori Subject: Re: [PATCH v2 3/3] usertools/rss: add CNXK well-known key Message-ID: <20231120084110.616e2059@hermes.local> In-Reply-To: <20231120162256.74399-6-rjarry@redhat.com> References: <20231023080710.240402-3-rjarry@redhat.com> <20231120162256.74399-4-rjarry@redhat.com> <20231120162256.74399-6-rjarry@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit 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 On Mon, 20 Nov 2023 17:22:59 +0100 Robin Jarry wrote: > diff --git a/usertools/dpdk-rss-flows.py b/usertools/dpdk-rss-flows.py > index fd225a697f08..be9b3d760c03 100755 > --- a/usertools/dpdk-rss-flows.py > +++ b/usertools/dpdk-rss-flows.py > @@ -226,6 +226,23 @@ def reta_size(self, num_queues: int) -> int: > ), > reta_size=512, > ), > + "cnxk": DriverInfo( > + key=bytes( > + ( > + # fmt: off > + # roc_nix_rss_key_default_fill, see drivers/common/cnxk/roc_nix_rss.c > + # Marvell's cnxk NICs take 48 bytes keys > + 0xfe, 0xed, 0x0b, 0xad, 0xfe, 0xed, 0x0b, 0xad, > + 0xfe, 0xed, 0x0b, 0xad, 0xfe, 0xed, 0x0b, 0xad, > + 0xfe, 0xed, 0x0b, 0xad, 0xfe, 0xed, 0x0b, 0xad, > + 0xfe, 0xed, 0x0b, 0xad, 0xfe, 0xed, 0x0b, 0xad, > + 0xfe, 0xed, 0x0b, 0xad, 0xfe, 0xed, 0x0b, 0xad, > + 0xfe, 0xed, 0x0b, 0xad, 0xfe, 0xed, 0x0b, 0xad, > + # fmt: on > + ) > + ), > + reta_size=64, > + ), > } Maybe an enhancement in future to expose the default key better. Having it live in two places means adding new drivers has another thing to worry about (and not documented).