From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 15E773E51DF for ; Thu, 23 Apr 2026 09:44:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.133.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776937457; cv=none; b=a1OZmk3DmCfVe+pKAKPev70YCf0dsfxsz/34ML/EYPI3B+cfuwwRUiEuGKaMd5VHZaYKHdzpk6HlHGMnBIHyTFv7t4vVzU06NuhVLuc4ML17qa0wzA462do1cqUuYJFmVaHVTDkpUw04RC2RYpRc/Fez5KaIb/rp9irQl0EOQiI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776937457; c=relaxed/simple; bh=Z/92kxlTMapvm6iTYEtSUelcWsiZw6+p+av2N6I37V8=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=V1RSEImDxQ3d5h1dsaHC4LRwE3vjm6+FX7GC8NCazy7A7BDUBfpDfHt+LpT3Ht5V8/GyUgUPA2FmEvijlwxOAMvKXCanUxcs7s9/Ed8e15yp1dNLy95JvcrjW22N6szl2hZTn1Wp8ytY/X/pLxqBp5VVRi2+Uj4KRr6xadl74cA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=PwYT88+y; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b=Rx7VUmfM; arc=none smtp.client-ip=170.10.133.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="PwYT88+y"; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b="Rx7VUmfM" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1776937455; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=3Czf+kEL9BcsX5OJ1yNM5tadNHCEbw6TrzapGuBP20Y=; b=PwYT88+y0yj1hD6HNPAYSUWX8STw+Fnn78s+trZ4lzFeRvDAxzHcK6+Km0kR/3R2iJJxnZ qgC/c3GdZ6mIr6qmR91syWH8tE7Df2AeQRs8/WWSb4is6H2iNfveHSagocbotmj3LRqH3N 8/HvEx1Y6BjjrbMfGNdfO/amNJw4bZs= Received: from mail-wm1-f70.google.com (mail-wm1-f70.google.com [209.85.128.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-8-kFEHneagOLqqox7-_Tsndg-1; Thu, 23 Apr 2026 05:44:13 -0400 X-MC-Unique: kFEHneagOLqqox7-_Tsndg-1 X-Mimecast-MFC-AGG-ID: kFEHneagOLqqox7-_Tsndg_1776937453 Received: by mail-wm1-f70.google.com with SMTP id 5b1f17b1804b1-4837b6f6b93so71713755e9.3 for ; Thu, 23 Apr 2026 02:44:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=google; t=1776937452; x=1777542252; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=3Czf+kEL9BcsX5OJ1yNM5tadNHCEbw6TrzapGuBP20Y=; b=Rx7VUmfMi3VBEGYuUrSO9Hz8ad3UdVfNhMlcMEfznuZsuenkg05IHDWYgIoU6ge2pl dvH/c/2S5GBRk3bvNeJ/5HXjPn7Qbw8nKI9xLtBtqbhfB8sZVtDpAXZ+hYqCaRnwM17M xD7r4zFaNBgj4Oq9bVLxX0UxqOYPnpOBZS5UZT2zrVSkiZPTd1RS3KaYp+tClIHPVOjo Ndpl4eV2jE+pn8IQkeZBCtVT+KyO5Fe4XxLUOQcm7DGiV8Fe3m+dA70ahTFopJnRhdc7 3UzhTABhx/R4U9jHUs0yHXBkrsFcvxG9I8fu62V2cf39X7z8y9ZLPKHzt+LLyXMqQetN JiEw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1776937452; x=1777542252; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=3Czf+kEL9BcsX5OJ1yNM5tadNHCEbw6TrzapGuBP20Y=; b=kMhnn/LZLi6yCakkDaNq+nw50DlKGhI8QfnUR5ad1+ySiDEi16hd1V/G7P7x2N7yIA F0HtBdBFNAZQUEEG+I6nR17cQLDKCgtuhzpAM5KXJToUjiF1y1UFhVdZQyTdVtR0DIIy 8GZD/52PeiGTxMg9ol5yX9RCxS7BwQqmnUrVJqkGaN3BHVUttbzQZkopQqoliNiVpV2F Xw/6N2wN8nkJORYk/wfC3H9xU6OKSZaBL5u+zrnpQ9myvGwJduHXoPjwE7C9dP2wwyYs bgBItS/rKu1zLBgP+zdehhVk+FcrraPj/IbfMgxHzrQCwWE0zb0lrztRW3v+QJS+UzKn +J/w== X-Forwarded-Encrypted: i=1; AFNElJ9qq62bcYVw/IPJHNVbVjEmuU+YJcPgO/C1GmsTfnDo+wvnQ/Qchno/JL6BQZ4GItJg2eYr11svFvGsKJM=@vger.kernel.org X-Gm-Message-State: AOJu0Yz0TrvV4pe+CAfVSTJTEtahiqLTI//J0HZRPlbDoIBobOtzgJqQ 7HRwXLzyfutGjra1TCfTC5xu2RzxEf5frSX+d4fRk4chMpcOje6fxDY4hIKBXnzPamLSxnSSH4c oY/8PMC8XImgA2c9CW+Sb1ciji437CKjFQyVgZyUnwITHw87RTZF2IuGQgldDPw0v/g== X-Gm-Gg: AeBDiev0RADjFSpZQtsxA5XaAZP1eo5RCO0H8x5u2EvdWkJJHDXS23Epq+f2sPTY6Lx ZJGok8rwEx1OJaF3mbjRXJAiT9R7durzVf+csP3vRGACB32gT2HtQ4JuXhKounIi2lom6RzbDEq fDkue51j6Tj3V5tIrqlm60A27UoUa9hOZU2LxA3/bmBzyLv7rzDOITPqtiJsbHIzbT+zORh4mm+ GhvB5JK14AC3CoGQA20Bop7PkGYloE7WEHNKpyelQbghFADQls9zBYmmAMIkK7cCfw0KHOpserF ukSauKtYbWBA4n/Bp3B/hQG5AiZtyyJ90K7J0HmUOWRWf9b/+7NIL5LohwusP6lG/b4BGPWP4zI o1+8uehA98dfC1TNGYfeoTDomJq4qZZJoSJ8stnwkqEUd02HmkQbhreR2zhZfTjb0kvs= X-Received: by 2002:a05:600c:2d91:b0:48a:5301:bb5c with SMTP id 5b1f17b1804b1-48a5301bc69mr110956245e9.16.1776937452417; Thu, 23 Apr 2026 02:44:12 -0700 (PDT) X-Received: by 2002:a05:600c:2d91:b0:48a:5301:bb5c with SMTP id 5b1f17b1804b1-48a5301bc69mr110956115e9.16.1776937451954; Thu, 23 Apr 2026 02:44:11 -0700 (PDT) Received: from [192.168.88.32] ([150.228.93.216]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-488fb7a051dsm157378285e9.18.2026.04.23.02.44.10 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 23 Apr 2026 02:44:11 -0700 (PDT) Message-ID: Date: Thu, 23 Apr 2026 11:44:10 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 net 0/11] octeontx2-af: npc: cn20k: MCAM fixes To: Ratheesh Kannoth , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Cc: sgoutham@marvell.com, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, andrew+netdev@lunn.ch, dan.carpenter@linaro.org References: <20260420023442.3295891-1-rkannoth@marvell.com> Content-Language: en-US From: Paolo Abeni In-Reply-To: <20260420023442.3295891-1-rkannoth@marvell.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 4/20/26 4:34 AM, Ratheesh Kannoth wrote: > This series tightens Marvell OcteonTX2 AF NPC support for CN20K > silicon around MCAM key typing, optional debugfs setup, defrag > allocation rollback, x2 versus x4 KEX profiles and default-rule > allocation, logical MCAM clear and configuration, default-rule index > bookkeeping and explicit teardown, and NIXLF reserved-slot lookup when > default rules are missing. > > Patches 1 through 3 focus on AF error handling: propagate > npc_mcam_idx_2_key_type() failures through cn20k MCAM enable, config, > copy, and read paths; treat cn20k NPC debugfs files as optional so > probe does not fail when debugfs is unavailable; and fix defrag MCAM > allocation rollback so allocation errno is not overwritten by subbank > index resolution. > > Patches 4 and 5 align default-rule and flow-install behaviour with the > loaded mkex profile: prefer x4 default entries when the profile is x4, > and reject x4 flow keys when the profile is strictly x2. > > Patches 6 through 8 refine cn20k MCAM programming: clear entries by > logical index and resolved key width, fix bank and CFG sequencing in > npc_cn20k_config_mcam_entry(), and read action metadata from the > correct bank in npc_cn20k_read_mcam_entry(). > > Patches 9 through 11 complete default-rule lifecycle handling: > initialize all default-rule index outputs up front, tear down default > MCAM rules explicitly (coordinated with npc_mcam_free_all_entries()), > and reject USHRT_MAX sentinel indices in npc_get_nixlf_mcam_index() > for cn20k. > > Ratheesh Kannoth (11): > octeontx2-af: npc: cn20k: Propagate MCAM key-type errors on cn20k > octeontx2-af: npc: cn20k: Drop debugfs_create_file() error checks in > init > octeontx2-af: npc: cn20k: Propagate errors in defrag MCAM alloc > rollback > octeontx2-af: npc: cn20k: Make default entries as x4. > octeontx2-af: npc: cn20k: Reject request for x4 entries in x2 > profile. > octeontx2-af: npc: cn20k: Clear MCAM entries by index and key width > octeontx2-af: npc: cn20k: Fix bank value. > octeontx2-af: npc: cn20k: Fix MCAM actions read > octeontx2-af: npc: cn20k: Initialize default-rule index outputs up > front > octeontx2-af: npc: cn20k: Tear down default MCAM rules explicitly on > free > octeontx2-af: npc: cn20k: Reject missing default-rule MCAM indices > > Signed-off-by: Ratheesh Kannoth > > -- > v1 -> v2: Addressed simon comments. Added more patch fixes to this series. > https://lore.kernel.org/netdev/20260418162013.GG280379@horms.kernel.org/ I strongly suggest avoid extending a series to address issues found by the AI review, unless such issues are actual regressions/problems introduced by the new code. Sashiko tends to find a lot of collateral problem and the series could easily expand beyond any hope of merging. Instead you should reply on the ML, commenting on sashiko's review, explain why or why not the outlined problems are valid and/or should be handled separately. Thanks, Paolo