Received-SPF: pass (google.com: domain of linux-kernel+bounces-156547-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
From: Sai Krishna Gajula <saikrishnag@marvell.com>
To: Jakub Kicinski <kuba@kernel.org>
CC: "davem@davemloft.net" <davem@davemloft.net>,
        "edumazet@google.com"
	<edumazet@google.com>,
        "pabeni@redhat.com" <pabeni@redhat.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Sunil Kovvuri
 Goutham <sgoutham@marvell.com>,
        Geethasowjanya Akula <gakula@marvell.com>,
        Hariprasad Kelam <hkelam@marvell.com>,
        Subbaraya Sundeep Bhatta
	<sbhatta@marvell.com>
Subject: Re: [net-next PATCH v2] octeontx2-pf: Add ucast filter count
 configurability via devlink.
Thread-Topic: [net-next PATCH v2] octeontx2-pf: Add ucast filter count
 configurability via devlink.
Thread-Index: AQHaliHuy+VRqHz+okuwEwJ8Kj9CHA==
Date: Wed, 24 Apr 2024 08:32:23 +0000
Message-ID: 
 <BY3PR18MB47073B7DAB65F1DAEBD92C11A0102@BY3PR18MB4707.namprd18.prod.outlook.com>
References: <20240418190031.1115865-1-saikrishnag@marvell.com>
 <20240422141845.108b9c87@kernel.org>
In-Reply-To: <20240422141845.108b9c87@kernel.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: 
 =?us-ascii?Q?sizxkZ2E29rCE7Tx0c66vmFRQXz41WrShz4YS3Qk67Qvs2439nxIEXqIw0+O?=
 =?us-ascii?Q?mvT9HzX0xwZmD2yJwCEmT7bbYna+4M1U+uG0//hPfEVOhjHWwqu9iMvoVpMm?=
 =?us-ascii?Q?ZTNVpCbjWg6UdMEb2g20HlG5HRSHc0TErVK1zeo/b4g0dxeTZser3R0Ie8GP?=
 =?us-ascii?Q?R/NdczPaiXXTrIj9YoVaP5WMSZBE6lWWvI90i0Xare0tonQfqorKTnKWI9x7?=
 =?us-ascii?Q?aXMFuf67lShOgavLhspzrS1SFjb3BJ46+dLpjkPNGuYPvQiOyXeJ9GtJU/dS?=
 =?us-ascii?Q?mKri0h06yf61uJo6UV+C/IAdBYOnsMqzRm3pszflTyyf5mrDaJx2bQ0IIary?=
 =?us-ascii?Q?aPAzOT6bENzyw13GJka+T1ehNeIonWHmEe2wpCn/J907dZgC2hUvWrFZZd/A?=
 =?us-ascii?Q?kLu0tQ3FOy7Gutd5nO+H6ZvwzViZ3ngOYwlk5jQLfc/QbHVjcQ2gSf0kCiFW?=
 =?us-ascii?Q?ryvJzgchHUb8D4dxjCFOxYNwAGFbbYbfmirrB8ookw6qbFpN/gTDIIaLk7zI?=
 =?us-ascii?Q?VwFB0DRVrIQjmov417/MAF+FyQaN/Ceo/6iA7kTG7E9s0y0sjG92yvMBp8jG?=
 =?us-ascii?Q?/Oi8+6WVyXEOLM110h77ykBXxZue/RFXBqw1toJZvaY2JkLIr54othZP1O9o?=
 =?us-ascii?Q?Mf0ZeFHWl4sStL6hR05uwm+EdkKWSpzx02qbCmAcbHTlYJQNc9XLEnP6dhSW?=
 =?us-ascii?Q?lHyr/IPU5TtpLgf5iDsCnxs2E0g6EfxAn0JlwIVTa59WeUYgnUAz2O3Ki8Qr?=
 =?us-ascii?Q?ebBUzG0GeqdSPJ/gj0RsZ5k/jXxa6PpbAJ3buBbwK3/MY+/I6gupMnCgqrPG?=
 =?us-ascii?Q?RtZeqgV6dZ3zbY7Uw4HuVIIg1ByWKWfyON0h9nfyCUD4zX7pCmsXO1GbV1E+?=
 =?us-ascii?Q?QSUyuKae9lCOj3phracQoeH7w43q1E/xZkd7CNgykIWWi4xvx8R1bt9u1c6N?=
 =?us-ascii?Q?ePoFfT2pbOrXLHaZThWnxBqZFu5lsHUZCD2nbI2LNuJq3SMRpq9Hh98MBW/L?=
 =?us-ascii?Q?sEjX51vFy/7HG8rfI4We87AvSOv0TnDfo1EIbaf1dksUaL0BiWZO4D5oaxvQ?=
 =?us-ascii?Q?+Pha10ws/fn3OHuOVMwcKkDxjks3SoHSGBAjdATCmk6bNo99EsgAX7r1q7iF?=
 =?us-ascii?Q?lkBoaArZxyOAAXlqh8nMqtpWTLu20JBLcLPgWXejxxPJmh+t5lHKLx4nQGsr?=
 =?us-ascii?Q?TB2AXooKuKIo/j1OxHIkyfbCNae6RYhM2iNBUpYfhoJGz+3zx6eEatG26wTC?=
 =?us-ascii?Q?DNioJv+DqEejgz0XFu3CNuUGdw53QLEVXtso52Q673y+yLsQajjt0Gpn8nWl?=
 =?us-ascii?Q?c5WYknLGtq7ma+qqmEPh/aUw2MvOQlF/yKY2snJ0YdDn711uLDAllUykBrKI?=
 =?us-ascii?Q?JS6cmVmjCsvHpO8Bbk9L/xfIEn1Qh+xis2BsZPVs9qwfaw7auiO4n2IDY7Yv?=
 =?us-ascii?Q?XAaijqOP0iNEFlj9YXSM1IrlkFnMVrn6DJ/eLH8OzQjzJ0iLwpxeMULiInWV?=
 =?us-ascii?Q?ra+JNM82NzUJK0fAPh2O5eu2CCGTHFFL+kskrA87vkY7oalFMc7wtvqQgKYa?=
 =?us-ascii?Q?2Z13lqFnij3Xddw9cELDV/pd1aw9PqjY+7Vk4Zjt?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
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: 03b6df0a-de57-4c29-c27e-08dc643910db
X-MS-Exchange-CrossTenant-originalarrivaltime: 24 Apr 2024 08:32:23.8160
 (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: qVtM8/UBTO4zDdQKJdhKPNcmgsugou1CxC8DcfgMhqCrhd4O1C9Ge5WDAZ8cjltHKEnYuLJ0RoxIIk/iQA19wQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR18MB4883
X-Proofpoint-GUID: wIVCBcYVjbg5w70KFgQLsvBiYppAfX2j
X-Proofpoint-ORIG-GUID: wIVCBcYVjbg5w70KFgQLsvBiYppAfX2j
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.293,Aquarius:18.0.1011,Hydra:6.0.650,FMLib:17.11.176.26
 definitions=2024-04-24_06,2024-04-23_02,2023-05-22_02


> -----Original Message-----
> From: Jakub Kicinski <kuba@kernel.org>
> Sent: Tuesday, April 23, 2024 2:49 AM
> To: Sai Krishna Gajula <saikrishnag@marvell.com>
> Cc: davem@davemloft.net; edumazet@google.com; pabeni@redhat.com;
> netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Sunil Kovvuri Gouth=
am
> <sgoutham@marvell.com>; Geethasowjanya Akula <gakula@marvell.com>;
> Hariprasad Kelam <hkelam@marvell.com>; Subbaraya Sundeep Bhatta
> <sbhatta@marvell.com>
> Subject: Re: [net-next PATCH v2] octeontx2-pf: Add ucast filter
> count configurability via devlink.
>=20
> On Fri, 19 Apr 2024 00:30:31 +0530 Sai Krishna wrote:
> > +
> 	DEVLINK_PARAM_DRIVER(OTX2_DEVLINK_PARAM_ID_UCAST_FLT_C
> NT,
> > +			     "unicast_filter_count", DEVLINK_PARAM_TYPE_U8,
> > +			     BIT(DEVLINK_PARAM_CMODE_RUNTIME),
> > +			     otx2_dl_ucast_flt_cnt_get,
> otx2_dl_ucast_flt_cnt_set,
> > +			     otx2_dl_ucast_flt_cnt_validate),
>=20
> All devlink params must be documented
> (Documentation/networking/devlink)
> including the explanation of the tradeoffs involved in changing the setti=
ng.

Ack, will add the documentation details and submit patch V3

> --
> pw-bot: cr