Received-SPF: pass (google.com: domain of linux-kernel+bounces-142626-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
From: Sunil Kovvuri Goutham <sgoutham@marvell.com>
To: Jakub Kicinski <kuba@kernel.org>, Hariprasad Kelam <hkelam@marvell.com>
CC: "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "davem@davemloft.net" <davem@davemloft.net>,
        Geethasowjanya Akula
	<gakula@marvell.com>,
        Jerin Jacob <jerinj@marvell.com>, Linu Cherian
	<lcherian@marvell.com>,
        Subbaraya Sundeep Bhatta <sbhatta@marvell.com>,
        Naveen Mamindlapalli <naveenm@marvell.com>,
        "edumazet@google.com"
	<edumazet@google.com>,
        "pabeni@redhat.com" <pabeni@redhat.com>
Subject: Re: [net-next PatchV2] octeontx2-af: map management port always to
 first PF
Thread-Topic: [net-next PatchV2] octeontx2-af: map management port always to
 first PF
Thread-Index: AQHajNIEkFklwUV4yUCKEkgnURAeSQ==
Date: Fri, 12 Apr 2024 12:07:40 +0000
Message-ID: 
 <BY3PR18MB47373F66158D5E98147EDFCDC6042@BY3PR18MB4737.namprd18.prod.outlook.com>
References: <20240410132538.20158-1-hkelam@marvell.com>
 <20240411195532.033e21fb@kernel.org>
In-Reply-To: <20240411195532.033e21fb@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?lTJ50dxUa3TCDgQS8TYQfm9scdfFltKuPP3k08Z6ilRdFhcxij2kZN3VXe4U?=
 =?us-ascii?Q?hjo4XZWC53VzPd6xpzLCKd5ht7ERhioyFcVmubz13gVxbM8XZT+jt/jmxQes?=
 =?us-ascii?Q?0N9xJ7MyGOqGyvhqqL+x2l3Wf3hdGegprMuCLN8xoY1SXYgIOEetFNkvJEuc?=
 =?us-ascii?Q?LGqF9lQqWOPvqhVHs371eEflrHPqJkGnKkOMLaUDixyJJ6sLsCgpMtvsx+wm?=
 =?us-ascii?Q?iuIqeaYR2G1GG3Lk4V/aC5GHXxX20C0eGAvAMBrJSvSoazgasfvz48sg+UOB?=
 =?us-ascii?Q?qC+QedK+mxa4nO/UghPZ8oqNutAFkfcJpAexGsNbNZgctfRnIxKRzR7Ri6sb?=
 =?us-ascii?Q?XUFEC5P2EbGMlSHZVM7bQl8x46VJqWxn9nF0Q1DF+hP93rP6563NvQoPM/d6?=
 =?us-ascii?Q?Jgt8sLYvix0UScVjOdsAGEH2nKD9OyM4QqVTV4E1ucLOEYNl1kJjlglPwFnB?=
 =?us-ascii?Q?OBCgocFzgAnxa3HpbBIIYWdHiBC6W9UPjlUOGJrheD7DTrK9/q93hq8marij?=
 =?us-ascii?Q?MozV/WhGeOAi29RdDTLUI200LzCsy9a96njch2Y0RLP8OwXmaYVYN118pk4z?=
 =?us-ascii?Q?jKpuNjhlNBNpbvzmoVm9Bjexy90GtpKtUmxe8ooIbQUM9sEs4Ifq0A1IbE5w?=
 =?us-ascii?Q?lobyVnlBgSVFXbaNSl5SbV9vHl2wDC7wwW4HyHVvyoLa/Xs5QGNyQUvI0qUT?=
 =?us-ascii?Q?sRkyZXP/TI8HTKP4RDUi/3Y3UGYjSYhfSWZN9kqAJKfd5c4zmVVEVDdQAyYC?=
 =?us-ascii?Q?ReM2DmghiPyS+gQP6Bbuag9jrGzMfbMY8TDkjS8+KoMVT46L2INyHg6Kip6o?=
 =?us-ascii?Q?iaV9QxbUMnb8L4Q8gqY0dY9RGFj75CFcj97X4kmoraXRhRIv8CaWfLJBs6q9?=
 =?us-ascii?Q?aqn+PS+sO3nfQ7wJlhpEK51+keYI5aMS9ow6L6E5FPMiHPCkjc6dq7wwQaCe?=
 =?us-ascii?Q?IQpPzFG7JR/9b9K+KUxcdypHUv4vtjeqq9Jkb/Esc9W2g+0GdhfSGosO3cMQ?=
 =?us-ascii?Q?uQ9MRznKYguqGJ9CwN4IcRR5tM6lAsfSG+sJIe026uztgYbomvzelQIZuFTi?=
 =?us-ascii?Q?bzPYkbHgJQv5se3p+UKJM4XL0jUYWvYO3qZQdPI1cBGdYLb7pHuyts907Px9?=
 =?us-ascii?Q?GR9S4WQuGjcjzmqe/2KLovB0NQxVf7gzxOiTAIT1l/m35AxJNREF/aKnC4hV?=
 =?us-ascii?Q?hiz4mxU7aRAfz7KAPcLjB8mMcm/mwhBsnkWZjjzW23Tb3znLymvbLR8IYQrP?=
 =?us-ascii?Q?dBl3Hz7Qm/xTZ+1zFGrtOadIcPStiVjw+H7GMb8FrdulhYoWeWOgkEdmAUe8?=
 =?us-ascii?Q?iQd9EBWz5WJDFQQTDRLE5ak5sjOALJ1qzEQPvjjbB5Ele1drtmYr0SOsGJfB?=
 =?us-ascii?Q?vBkMBFmzP/r59aJI9LIjZ+7Kko/335oQEw8t9LeE88QHSdxdItudbwPubcDj?=
 =?us-ascii?Q?QlyWmFDWLIcePpSGvkk2SO14G5FagvX3uLmOdEiSfrIRJCriceYWGdOctZIA?=
 =?us-ascii?Q?Rh76XDTIvQ+srdhsI/4uRm8b0awEQ6880lp/46XFTiEtzVfTFu/9c44rwKbF?=
 =?us-ascii?Q?F1lC+lIGwZ+fg8aQlMz2WBvKSfdNgzA/d0F+5gBN?=
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: BY3PR18MB4737.namprd18.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: fa53836e-f819-4c31-8c60-08dc5ae926b1
X-MS-Exchange-CrossTenant-originalarrivaltime: 12 Apr 2024 12:07:40.2771
 (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: C3vFkP8IMWlk/cuQ4Emr5FA9LnjMAySgSrdrRd4AQBDJJkiU68fqkwiv+lDoR6Yw+sEmTzNGIq6WWhoS/Yepfw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR18MB5057
X-Proofpoint-GUID: QeU6Ov45KQpKcVbfybfkp7AqcXrcKUat
X-Proofpoint-ORIG-GUID: QeU6Ov45KQpKcVbfybfkp7AqcXrcKUat
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2024-04-12_08,2024-04-09_01,2023-05-22_02


> -----Original Message-----
> From: Jakub Kicinski <kuba@kernel.org>
> Sent: Friday, April 12, 2024 8:26 AM
> To: Hariprasad Kelam <hkelam@marvell.com>
> Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org;
> davem@davemloft.net; Sunil Kovvuri Goutham <sgoutham@marvell.com>;
> Geethasowjanya Akula <gakula@marvell.com>; Jerin Jacob
> <jerinj@marvell.com>; Linu Cherian <lcherian@marvell.com>; Subbaraya
> Sundeep Bhatta <sbhatta@marvell.com>; Naveen Mamindlapalli
> <naveenm@marvell.com>; edumazet@google.com; pabeni@redhat.com
> Subject: Re: [net-next PatchV2] octeontx2-af: map management
> port always to first PF
>=20
> ----------------------------------------------------------------------
> On Wed, 10 Apr 2024 18:55:38 +0530 Hariprasad Kelam wrote:
> > The user can enable or disable any MAC block or a few ports of the
> > block. The management port's interface name varies depending on the
> > setup of the user if its not mapped to the first pf.
>=20
> There is no concept of management port in Linux networking.
> I may be missing the point, but I'm unable to review this in the context =
of the
> upstream Linux kernel.

Agree, that there is no concept of management port in Linux.
From Octeon hardware pov, there are multiple MACs and each MAC (internally =
called RPM) is capable of supporting multiple interfaces (called LMACs).
Let's say there are two RPMs on the board and RPM0 is configured to 2x50G a=
nd RPM1 is configured as 4x10G.
When kernel boots with this config, let's say the interface names are eth0,=
 eth1.. eth5.
If user is using 'eth3' for NFS, DHCP, SSH etc (ie for device management pu=
rposes) and then if user changes RPM0
config to 4x10G, then in the subsequent boot the same RPM1:LMAC0 could be n=
amed as 'eth5' now.
Customers have reported that their scripts are not working in these scenari=
os and they want some predictable naming.

What this patch does is that, RPM:LMAC0 which customer is using for managem=
ent port is always mapped to=20
same PCI device, so that interface naming remains unchanged irrespective of=
 different RPM configurations.

Thanks,
Sunil.