Re: mlx5 ConnectX diagnostic misc driver

Leon Romanovsky
2023-10-19T17:12:19Z

On Thu, Oct 19, 2023 at 05:36:04PM +0200, Greg Kroah-Hartman wrote:
> On Thu, Oct 19, 2023 at 08:24:51AM -0700, Jakub Kicinski wrote:
> > > The ConnectX HW family supported by the mlx5 drivers uses an architecture
> > > where a FW component executes "mailbox RPCs" issued by the driver to make
> > > changes to the device. This results in a complex debugging environment
> > > where the FW component has information and complex low level state that
> > > needs to be accessed to userspace for debugging purposes.
> >
> > You're being very dishonest towards Greg by not telling him that this
> > is a networking device, and the networking maintainers explicitly nacked
> > this backdoor. Nacked it, because you can't answer basic questions like
> > "what are the use cases" with more than "custom config and debug".
> >
> > Whether Greg wants to take this into the "misc" pile is entirely up
> > to him, but you gotta give him more context.
>
> Well, in this case, no way in hell will I be taking this. If this is a
> networking device,

It is not networking device. Only by coincidence, the PCI core of mlx5
driver is located in netdev, and it serves other subsystems as well
(VFIO, VDPA, RDMA, e.t.c.).

Just as a reminder, we added auxiliary bus to make sure that this device
is properly decoupled across various subsystems.

> it needs to go through the normal networking driver review process,
> thanks for the heads up.

That process is not relevant here, not everything in the world is netdev.

Thanks

>
> greg k-h