Received-SPF: pass (google.com: domain of linux-kernel+bounces-155490-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
From: Geethasowjanya Akula <gakula@marvell.com>
To: Jiri Pirko <jiri@resnulli.us>
CC: "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "kuba@kernel.org" <kuba@kernel.org>,
        "davem@davemloft.net"
	<davem@davemloft.net>,
        "pabeni@redhat.com" <pabeni@redhat.com>,
        "edumazet@google.com" <edumazet@google.com>,
        Sunil Kovvuri Goutham
	<sgoutham@marvell.com>,
        Subbaraya Sundeep Bhatta <sbhatta@marvell.com>,
        Hariprasad Kelam <hkelam@marvell.com>
Subject: RE: [EXTERNAL] Re: [net-next PATCH v2 0/9] Introduce RVU representors
Thread-Topic: [EXTERNAL] Re: [net-next PATCH v2 0/9] Introduce RVU
 representors
Thread-Index: AQHalJsFkqfhspWruUiIYa8/IeAzYbF10/0AgAAqKiA=
Date: Tue, 23 Apr 2024 15:39:15 +0000
Message-ID: 
 <BL1PR18MB434216126F143177DE9A1D9ACD112@BL1PR18MB4342.namprd18.prod.outlook.com>
References: <20240422095401.14245-1-gakula@marvell.com>
 <Ziexkkz8HCtIVRap@nanopsycho>
In-Reply-To: <Ziexkkz8HCtIVRap@nanopsycho>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: 
 =?us-ascii?Q?V1ceUGcqMj0/GfjzIt2N3xvnrm1RVtwEkaiviy5/aJrh4uO+ntKiqOY+pI0U?=
 =?us-ascii?Q?0t38PrAkXC4E/4JwkIaNvN4kK/eA23SssWr8ks+Y/Jpl+doe7RwmcdvOlQSk?=
 =?us-ascii?Q?5SCJsBEqq49hO6kA6xVfcPCaZHZDEJaKQSHzXbNDkzN0Qz1Mdkdq3PTqxmM7?=
 =?us-ascii?Q?eDLfuQzZmGKsPfBvnZMqE7f97y00/ih+f1mkfp20kbRIZ3PsHkbauZLmKb6G?=
 =?us-ascii?Q?E5bBAakqor+5KPG2wEBD9ODNMK/9xwfT145dwqEWtZvqFRVQ/m7Sw2Ba/oVk?=
 =?us-ascii?Q?YN6/ogcIgnGiyP3UlftAJF5uWEmutjlFMFQGvU3Aw3Kj6WVIssN7jZBboQvQ?=
 =?us-ascii?Q?otmWx5Cl6QZ6U55RzwOr5GTbetePgA7I2uNV74j/whS4X3gtP5cP4tzlOVXO?=
 =?us-ascii?Q?VjzimPQIK7vqGAzb610JZiLD3oqamWC7Un5CHgQQOJbvw+S3keTNzuEu2qiu?=
 =?us-ascii?Q?iK3i/OsHz2+486vXJeanZC2o1ApUr6xb6kel10cjSVIxD7KxtK83JBPNDwX5?=
 =?us-ascii?Q?lHAPSZWAiXzDiQXATIdNbCjzASGlg/JTXcV9AN3HcNLrg720x3Chmw26Nzey?=
 =?us-ascii?Q?I2M3LQvI82zIjMuTxG1DcN7PQXcPJa0BGxTDCNj7op3ptejozh1ljMzX+FM5?=
 =?us-ascii?Q?OnwCcyEi5eGGVVzmrPIFouvmW+7Nf2HcMocCSOd9j9qq02mZlvnksSpAhkPf?=
 =?us-ascii?Q?DzR48OCOAKFKbePLFiDGdnR7aZT8Dobl4ngk7QmZ+WGNkHa35cmNtw91/9yu?=
 =?us-ascii?Q?px8Pcbk7xI4/GOeUatoxX/uNPgQnWOVoyzitjWarAnbudN9PqRqViaL5rttH?=
 =?us-ascii?Q?JtTnXnngOft6cIHxNjfFJQ+7nOQHwlW9m5fI6RzzgC1Va38e5DFZJov98PHZ?=
 =?us-ascii?Q?bAWH0QW7grC0ohi886+R5se8OJrHR1VkxvqkUJb8jAwFkJ3YdsVzKEkJSctM?=
 =?us-ascii?Q?vpwHLqS7UygJLOS0uFtWHuJUhDysWbPxKKLMgXL26AT16dMWG2acVibtKT17?=
 =?us-ascii?Q?f5AHKJj6AL3quh7FJJQyJEDv4p5QAWz7Az6nes+XnYShgITPWbTOwsBAKdXP?=
 =?us-ascii?Q?yWt5fVHcSq4UxUPyEpD7Ad3T8fVtCoi23gX3Cux6B9Fr5mB1Gdx7pKzlnqTs?=
 =?us-ascii?Q?WpMHX4vDpYDzaV7OT7fuzQ0ysdyNmr8yTJe5dZPyzOvBz4T0WdQOKMAYjkr+?=
 =?us-ascii?Q?/Ys+/0ZcBLrYnd8mhWQFirQpecI6JPcZxX3Ea+2Vm0YinyKJ1WOL5eodwrh5?=
 =?us-ascii?Q?82qBFuprOg3Ltrdg2iCdcJ0cfVNeZWdU6wWKkhL0jrBoBaJ0rpRpqJcc41U+?=
 =?us-ascii?Q?JhnUkMFqL1zWII+yPB/hiRfO1SZhT1MYGp3DAwxSgga0Xb3qTLwOuYHisPcU?=
 =?us-ascii?Q?x32KvMhDyfJgaEthz3NikH4JdLLlc/mjpoxci9LynOiI/k2/5jwFLF3phZp2?=
 =?us-ascii?Q?YvztHP1t1UBNNOv5TtKi5V8X3fwHi9xHqcvicd4dpkKOFwne2VmZfoy7qlzV?=
 =?us-ascii?Q?Vg1kQtGywaIq4RA0VCMGKumeZJRyU0gWdPpgVMg11UIm8tEuzSrGaDDM04oz?=
 =?us-ascii?Q?N0wL3ZteU9DQ4/eQCtg=3D?=
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: BL1PR18MB4342.namprd18.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 8a1a030b-d4d9-46fd-81d8-08dc63ab8877
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Apr 2024 15:39:15.9533
 (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: I8fNhYczbJ1H0I1ZYhzmybPap3kQdg4W1WgXRcZckuAWrrkZ/qJ0M6W3QJeh+0kDTHG7PLPzTEPVPoW7OIDgQg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR18MB3442
X-Proofpoint-ORIG-GUID: 6sLNtIOOxsaF9Ut1FUKx8NhQx8laKhpA
X-Proofpoint-GUID: 6sLNtIOOxsaF9Ut1FUKx8NhQx8laKhpA
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-23_12,2024-04-23_02,2023-05-22_02


> -----Original Message-----
> From: Jiri Pirko <jiri@resnulli.us>
> Sent: Tuesday, April 23, 2024 6:33 PM
> To: Geethasowjanya Akula <gakula@marvell.com>
> Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org; kuba@kernel.org=
;
> davem@davemloft.net; pabeni@redhat.com; edumazet@google.com; Sunil
> Kovvuri Goutham <sgoutham@marvell.com>; Subbaraya Sundeep Bhatta
> <sbhatta@marvell.com>; Hariprasad Kelam <hkelam@marvell.com>
> Subject: [EXTERNAL] Re: [net-next PATCH v2 0/9] Introduce RVU representor=
s
>=20
> Prioritize security for external emails: Confirm sender and content safet=
y
> before clicking links or opening attachments
>=20
> ----------------------------------------------------------------------
> Mon, Apr 22, 2024 at 11:53:52AM CEST, gakula@marvell.com wrote:
> >This series adds representor support for each rvu devices.
> >When switchdev mode is enabled, representor netdev is registered for
> >each rvu device. In implementation of representor model, one NIX HW LF
> >with multiple SQ and RQ is reserved, where each RQ and SQ of the LF are
> >mapped to a representor. A loopback channel is reserved to support
> >packet path between representors and VFs.
> >CN10K silicon supports 2 types of MACs, RPM and SDP. This patch set
> >adds representor support for both RPM and SDP MAC interfaces.
> >
> >- Patch 1: Refactors and exports the shared service functions.
> >- Patch 2: Implements basic representor driver.
> >- Patch 3: Add devlink support to create representor netdevs that
> >  can be used to manage VFs.
> >- Patch 4: Implements basec netdev_ndo_ops.
> >- Patch 5: Installs tcam rules to route packets between representor and
> >	   VFs.
> >- Patch 6: Enables fetching VF stats via representor interface.
> >- Patch 7: Adds support to sync link state between representors and VFs.
> >- Patch 8: Enables configuring VF MTU via representor netdevs.
> >- Patch 9: Add representors for sdp MAC.
>=20
>=20
> Could you please add some command outputs to the cover letter? Like $
> devlink dev $ devlink port
>=20
#devlink dev eswitch set pci/0002:1c:00.0 mode switchdev
> outputs at least.
>=20
>=20
> >
> >-----------
> >v1-v2:
> > -Fixed build warnings.
> > -Address review comments provided by "Kalesh Anakkur Purayil".
> >
> >Geetha sowjanya (9):
> >  octeontx2-pf: Refactoring RVU driver
> >  octeontx2-pf: RVU representor driver
> >  octeontx2-pf: Create representor netdev
> >  octeontx2-pf: Add basic net_device_ops
> >  octeontx2-af: Add packet path between representor and VF
> >  octeontx2-pf: Get VF stats via representor
> >  octeontx2-pf: Add support to sync link state between representor and
> >    VFs
> >  octeontx2-pf: Configure VF mtu via representor
> >  octeontx2-pf: Add representors for sdp MAC
> >
> > .../net/ethernet/marvell/octeontx2/Kconfig    |   8 +
> > .../ethernet/marvell/octeontx2/af/Makefile    |   3 +-
> > .../ethernet/marvell/octeontx2/af/common.h    |   2 +
> > .../net/ethernet/marvell/octeontx2/af/mbox.h  |  73 +++
> > .../net/ethernet/marvell/octeontx2/af/npc.h   |   1 +
> > .../net/ethernet/marvell/octeontx2/af/rvu.h   |  30 +-
> > .../marvell/octeontx2/af/rvu_debugfs.c        |  27 -
> > .../marvell/octeontx2/af/rvu_devlink.c        |   6 +
> > .../ethernet/marvell/octeontx2/af/rvu_nix.c   |  75 ++-
> > .../marvell/octeontx2/af/rvu_npc_fs.c         |   4 +
> > .../ethernet/marvell/octeontx2/af/rvu_rep.c   | 457 ++++++++++++++
> > .../marvell/octeontx2/af/rvu_struct.h         |  26 +
> > .../marvell/octeontx2/af/rvu_switch.c         |  20 +-
> > .../ethernet/marvell/octeontx2/nic/Makefile   |   2 +
> > .../ethernet/marvell/octeontx2/nic/cn10k.c    |   4 +-
> > .../ethernet/marvell/octeontx2/nic/cn10k.h    |   2 +-
> > .../marvell/octeontx2/nic/otx2_common.c       |  53 +-
> > .../marvell/octeontx2/nic/otx2_common.h       |  83 ++-
> > .../marvell/octeontx2/nic/otx2_devlink.c      |  47 ++
> > .../ethernet/marvell/octeontx2/nic/otx2_pf.c  | 305 ++++++---
> > .../ethernet/marvell/octeontx2/nic/otx2_reg.h |   1 +
> > .../marvell/octeontx2/nic/otx2_txrx.c         |  35 +-
> > .../marvell/octeontx2/nic/otx2_txrx.h         |   3 +-
> > .../ethernet/marvell/octeontx2/nic/otx2_vf.c  |  18 +-
> > .../net/ethernet/marvell/octeontx2/nic/rep.c  | 596 ++++++++++++++++++
> > .../net/ethernet/marvell/octeontx2/nic/rep.h  |  51 ++
> > 26 files changed, 1707 insertions(+), 225 deletions(-) create mode
> > 100644 drivers/net/ethernet/marvell/octeontx2/af/rvu_rep.c
> > create mode 100644 drivers/net/ethernet/marvell/octeontx2/nic/rep.c
> > create mode 100644 drivers/net/ethernet/marvell/octeontx2/nic/rep.h
> >
> >--
> >2.25.1
> >
> >