Re: [EXTERNAL] Re: [net-next PATCH v11 00/11] Introduce RVU representors

Sun, Sep 01, 2024 at 12:01:02PM CEST, gakula@marvell.com wrote:

quoted

-----Original Message-----
From: Jiri Pirko <jiri@resnulli.us>
Sent: Thursday, August 22, 2024 8:12 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 [off-list ref]; Subbaraya Sundeep Bhatta
[off-list ref]; Hariprasad Kelam [off-list ref]
Subject: [EXTERNAL] Re: [net-next PATCH v11 00/11] Introduce RVU
representors

Thu, Aug 22, 2024 at 03:20:20PM CEST, gakula@marvell.com wrote:

quoted

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.
- Patch 10: Add devlink port support.

What is the fastpath? Where do you offload any configuration that actually
ensures VF<->physical_port and VF<->VF traffic? There should be some
bridge/tc/route offload.

Packet between  VFs and VF -> physical ports are done based on tcam rules installed by  TC only.

Where is the code implementing that?

quoted

Or, what I fear, do you use some implicit mac-based steering? If yes, you

No, we don’t do any mac based traffic steerring.

quoted

should not. In switchdev mode, if user does not configure representors to
forward packets, there is no packet forwarding.