Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Sai Krishna Gajula <saikrishnag@marvell.com>
To:     Simon Horman <simon.horman@corigine.com>
CC:     "davem@davemloft.net" <davem@davemloft.net>,
        "edumazet@google.com" <edumazet@google.com>,
        "kuba@kernel.org" <kuba@kernel.org>,
        "pabeni@redhat.com" <pabeni@redhat.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "leon@kernel.org" <leon@kernel.org>,
        Sunil Kovvuri Goutham <sgoutham@marvell.com>,
        Geethasowjanya Akula <gakula@marvell.com>,
        Linu Cherian <lcherian@marvell.com>,
        Jerin Jacob Kollanukkaran <jerinj@marvell.com>,
        Hariprasad Kelam <hkelam@marvell.com>,
        Subbaraya Sundeep Bhatta <sbhatta@marvell.com>,
        Ratheesh Kannoth <rkannoth@marvell.com>
Subject: Re: [net PATCH v3 03/10] octeontx2-af: Fix depth of cam and mem
 table.
Thread-Topic: [net PATCH v3 03/10] octeontx2-af: Fix depth of cam and mem
 table.
Thread-Index: AQHZc1tWiUU5NOOIDUGaP9GCY4w/bg==
Date:   Thu, 20 Apr 2023 07:40:11 +0000
Message-ID: <BY3PR18MB4707E69DD88E93E3D1E5FA13A0639@BY3PR18MB4707.namprd18.prod.outlook.com>
References: <20230419062018.286136-1-saikrishnag@marvell.com>
 <20230419062018.286136-4-saikrishnag@marvell.com>
 <ZD+6SOCYdXNG02s5@corigine.com>
In-Reply-To: <ZD+6SOCYdXNG02s5@corigine.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?dt3MncO59BbPhEMx3vEUzWdkuxUxOr+GZht6O7dwaW6d4rOhy8nO2r+PUcTo?=
 =?us-ascii?Q?nKM9M/e13mUPErg+XNzGCWxnUd+citjefbqlDwIAM+WU76BVjEjTLP/H3YHt?=
 =?us-ascii?Q?FDdcd//B0hYYtn1tKR5fLnyg0ZR4yh4lkZ6Qysqqj7agPHNDJ74XqgN4OHfO?=
 =?us-ascii?Q?Lg6HeJ7Jqc81S0QhLqgWhIwJqMf+r0SwAHttiKcj7zxsAEaGF691JHC4cxw+?=
 =?us-ascii?Q?mn/JJIDlVtlIJDE6uGKvRHGNAeOjRearHJj1jLB+3oi0c3T9n1qpq+xd6rk6?=
 =?us-ascii?Q?nnDgRg3bSoHY83MuS1LENa/Pk/8EHrcj9nQrOyFkc/00qVGkvWgnJ0Iaqkkg?=
 =?us-ascii?Q?gWvAvTA7fbQh45/4M9E/l4fvtiw28Es3kQmIc6eCmR9+LiZh3gqEQ8s8qA/H?=
 =?us-ascii?Q?K8GtHvFcZ2jXrcphUrk4ZGZKJ5kPci9w6Eltzfk+nZ1lklR0IflwA3blzWRl?=
 =?us-ascii?Q?1EQOAG3vEkzU7CaV4reBL1LiiuaQTEiXOPoBEoxwXYNKHeL21XG6nrkxMKZR?=
 =?us-ascii?Q?Ick0QOYNrLFr3iWOHKWYe6KxL2d/j+2P6OSYNydmesfPBwOLFBFpZIiDjc+z?=
 =?us-ascii?Q?vwQjpJfAednPvrqqApR83mqtfJIv/aV2ympPknBWMV7rlaSBXumiD0n1gUOK?=
 =?us-ascii?Q?/x59OPBLLM2IJHvq/fZn20x81p3BJ3wBMXGpcYXyCCUVhc7oQA4QHaqn41JN?=
 =?us-ascii?Q?aQVOutOad/Y8Z8cnfbeSGi+I6XPVYrkLRjFd1c482xXlx/PgNdAJm6xZ8you?=
 =?us-ascii?Q?5YEoW9Yjem8PJDZPfEDyvbWXs4RhPFZiO8AabomF3bJwzsM0ucEbQmAuEDG+?=
 =?us-ascii?Q?CV60Ou3NQ08GD+0ymQyks0VHNNMtuyhN/8pq25O3+y0/gCQQqQa4T2UB5NV3?=
 =?us-ascii?Q?V+ndWKmbj8Ajdr3REeGsN2ZCadTpPlrW/mQKKFSMv63bppUZ031O+Zy62Sn5?=
 =?us-ascii?Q?oIVpCdHn+Mp9vZEHgUo56F6+SOF+KthktMtql5wbBJy6RWvBoVz1aGnL9W5z?=
 =?us-ascii?Q?PGdTP9DnmpODLZmdq8ztDxn84m4JDBgFIKo8TZ+xYebye8/ZEbzQKClaLQrJ?=
 =?us-ascii?Q?pmwYsz2Peav6S2W2Ps6nTssMnWNsBnCK0GWC8/DKZ9sEaQBHe3UmKPRQ1cJr?=
 =?us-ascii?Q?xVFb9wCKiJ+NN133EpW7NKyJNUkG6TNP7stQANQbovnaB1kF47mcG1BK38Oa?=
 =?us-ascii?Q?y2f+vKfvCmCUyHmo6bs290rfW7bqur6lp9E1JUse6yQMHX3ntpMx9Ovdg6lh?=
 =?us-ascii?Q?CcDsepzBxfnu3yRTYLeHn12YO0wZa9uTCNuUh2tlYuEbAFfWYQu8VDMILpiV?=
 =?us-ascii?Q?9q6jqWhovG89Dtf+Ntqc/KMnOQucgKHhOd2lm4Q9RfOlYwuyvVrwP5NqdoGn?=
 =?us-ascii?Q?JPgD145it/DviiQ6uVUoQuPi7y9ofuVZtZQw1A6JQwnbnfnaUs9sSfLTlF+j?=
 =?us-ascii?Q?OmjWtfHDN6WoMyIJ4n4IFt3thvm4RpI2Dj3FsRmpW/zx118ALY4jUfsQpqYc?=
 =?us-ascii?Q?bkMhqLVFGeGunJFAAnyNLYXgRxpZiErkBd1K16FAoNykY4gN0wiWvC9pSXMq?=
 =?us-ascii?Q?Gyzw/DF8C6FyicWJ63ontk8PAnzBotvsz1hVAPj9?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: marvell.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BY3PR18MB4707.namprd18.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: bd6cfbea-bf22-450c-c150-08db417278db
X-MS-Exchange-CrossTenant-originalarrivaltime: 20 Apr 2023 07:40:11.2688
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: Hx6Q6i+5PS/ZMUL6iUnwG7+Il1wqTsD+1Se5ev9H54hu7/0TvczQz4iSffpbCCHFESt/wUCq4ZaUrVLjJoxm8A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR18MB5107
X-Proofpoint-ORIG-GUID: KU4orGbHR3ypSeqch3QqE92fO8dC7KP5
X-Proofpoint-GUID: KU4orGbHR3ypSeqch3QqE92fO8dC7KP5
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.254,Aquarius:18.0.942,Hydra:6.0.573,FMLib:17.11.170.22
 definitions=2023-04-20_04,2023-04-18_01,2023-02-09_01
X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Simon Horman <simon.horman@corigine.com>
> Sent: Wednesday, April 19, 2023 3:24 PM
> To: Sai Krishna Gajula <saikrishnag@marvell.com>
> Cc: davem@davemloft.net; edumazet@google.com; kuba@kernel.org;
> pabeni@redhat.com; netdev@vger.kernel.org; linux-
> kernel@vger.kernel.org; leon@kernel.org; Sunil Kovvuri Goutham
> <sgoutham@marvell.com>; Geethasowjanya Akula <gakula@marvell.com>;
> Linu Cherian <lcherian@marvell.com>; Jerin Jacob Kollanukkaran
> <jerinj@marvell.com>; Hariprasad Kelam <hkelam@marvell.com>;
> Subbaraya Sundeep Bhatta <sbhatta@marvell.com>; Ratheesh Kannoth
> <rkannoth@marvell.com>
> Subject: Re: [net PATCH v3 03/10] octeontx2-af: Fix depth of cam and
> mem table.
>=20
> On Wed, Apr 19, 2023 at 11:50:11AM +0530, Sai Krishna wrote:
> > From: Ratheesh Kannoth <rkannoth@marvell.com>
> >
> > Depth of CAM and MEM tables were wrongly configured. Fixed the same in
> > NPC module.
> >
> > Fixes: b747923afff8 ("octeontx2-af: Exact match support")
> > Signed-off-by: Ratheesh Kannoth <rkannoth@marvell.com>
> > Signed-off-by: Sunil Kovvuri Goutham <sgoutham@marvell.com>
> > Signed-off-by: Sai Krishna <saikrishnag@marvell.com>
>=20
> If there is an issue that this resolves, especially something user-visibl=
e, it
> might be useful to explain what it is - e.g. before and after. Likewise f=
or a
> number of other patches in the series.
>=20
> But that notwithstanding, this looks good, with the caveat that I can't v=
erify
> that the change matches the HW.
>=20

We will expand the commit messages with the explanation about the issues an=
d fixes that resolves the issues in v4 patches.

Thanks,
Sai
> Reviewed-by: Simon Horman <simon.horman@corigine.com>