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

From: Jakub Kicinski
Date: Thu Aug 08 2024 - 10:41:06 EST


On Thu, 8 Aug 2024 11:30:29 +0000 Sunil Kovvuri Goutham wrote:
> >I can't bring myself to apply this.
> >Jiri do you have an opinion?
> >The device is a NPU/SmartNIC/DPU/IPU, it should be very flexible.
> >Yet, instead of just implementing the representors like everyone else you do your
> >own thing and create separate bus devices.
>
> Can you please elaborate what you mean by "create separate bus devices"

You have a separate PF (on top of the existing "AF" PF) which doesn't
have any datapath port of its own.

> Just to clarify we are not creating separate bus devices for representors perse.
> On our HW, there are multiple SRIOV PCI devices.

As in SR-IOV-capable PFs?

> We are using one of those PCI devices (pci/0002:1c:00.0), to attach required hardware resources to it, for doing packet IO
> Between representors and the representees. Once the HW resources (Rx, Tx queues) are attached to it
> we are registering multiple netdevs from this ie one rx/tx queue for one representor.
> Beyond that there are no other bus devices being created/used.

What port is used for communication to the host and the uplink?
We're on v10 and I keep asking for documentation showing the system
topology.