Re: [PATCH rdma-next v4 01/12] RDMA/core: Introduce RDMA subsystem ibdev_* print functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 01, 2019 at 02:47:42PM -0300, Jason Gunthorpe wrote:
> On Mon, Apr 01, 2019 at 05:18:16PM +0000, Parav Pandit wrote:
> >
> >
> > > From: Gal Pressman <galpress@xxxxxxxxxx>
> > > Sent: Thursday, March 28, 2019 7:39 AM
> > > To: Jason Gunthorpe <jgg@xxxxxxxx>; Doug Ledford <dledford@xxxxxxxxxx>
> > > Cc: Yossi Leybovich <sleybo@xxxxxxxxxx>; Alexander Matushevsky
> > > <matua@xxxxxxxxxx>; Leah Shalev <shalevl@xxxxxxxxxx>; Dave Goodell
> > > <goodell@xxxxxxxxxx>; Brian Barrett <bbarrett@xxxxxxxxxx>; linux-
> > > rdma@xxxxxxxxxxxxxxx; Sean Hefty <sean.hefty@xxxxxxxxx>; Dennis
> > > Dalessandro <dennis.dalessandro@xxxxxxxxx>; Leon Romanovsky
> > > <leon@xxxxxxxxxx>; Christoph Hellwig <hch@xxxxxxxxxxxxx>; Parav Pandit
> > > <parav@xxxxxxxxxxxx>; Sagi Grimberg <sagi@xxxxxxxxxxx>; Steve Wise
> > > <larrystevenwise@xxxxxxxxx>; Shiraz Saleem <shiraz.saleem@xxxxxxxxx>;
> > > Gal Pressman <galpress@xxxxxxxxxx>; Jason Baron <jbaron@xxxxxxxxxx>
> > > Subject: [PATCH rdma-next v4 01/12] RDMA/core: Introduce RDMA
> > > subsystem ibdev_* print functions
> > >
> > > Similarly to dev/netdev/etc printk helpers, add standard printk helpers for
> > > the RDMA subsystem.
> > >
> > > Example output:
> > > efa 0000:00:06.0 efa_0: Hello World!
> > > efa_0: Hello World! (no parent device set) (NULL ib_device): Hello World!
> > > (ibdev is NULL)
> > >
>
> > We are working on mdev devices whose device name in kernel defined
> > as 36 letters.. (standard uuid based) which is too long.
>
> Well, then you have a problem in netdev, so whatever is solved in
> netdev we should solve it here. Otherwise this should continue to copy
> netdev.
>
> I continue to think naming devices with random GUIDs is a really dumb
> thing to do.

You are already doing it but in small scale by assigning some random
indexes to ib_device name, e.g. in efa_X and efa_X+1, X and X+1 are
GUIDs which mean nothing.

Thanks

>
> Jason

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux