RE: [PATCH rdma-next] RDMA/hfi1: Use PCI-ID as an identification in debugfs

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

 



>-----Original Message-----
>From: linux-rdma-owner@xxxxxxxxxxxxxxx [mailto:linux-rdma-
>owner@xxxxxxxxxxxxxxx] On Behalf Of Jason Gunthorpe
>Sent: Wednesday, December 12, 2018 10:35 PM
>To: Dalessandro, Dennis <dennis.dalessandro@xxxxxxxxx>
>Cc: Leon Romanovsky <leon@xxxxxxxxxx>; Yuval Shaia
><yuval.shaia@xxxxxxxxxx>; Doug Ledford <dledford@xxxxxxxxxx>;
>Marciniszyn, Mike <mike.marciniszyn@xxxxxxxxx>; RDMA mailing list <linux-
>rdma@xxxxxxxxxxxxxxx>
>Subject: Re: [PATCH rdma-next] RDMA/hfi1: Use PCI-ID as an identification in
>debugfs
>
>On Wed, Dec 12, 2018 at 03:05:59PM -0500, Dennis Dalessandro wrote:
>
>> > Because it is not related to verbs at all, but general core
>> > infrastructure change needed for all devices in RDMA subsystem,
>> > which are using struct ib_device to hold data.
>>
>> Core, verbs, whatever. Point is that debugfs is still something different
>> and doesn't have to be subject to the same naming scheme.
>
>You can use whatever names you like in debugfs, so long as they are
>guaranteed to be globally unique.

I am not really sure what this means. 

Globally unique in what way?  That there is no other debugfs entry with the same name?  That there is no other device with that name? That there is no other filename that is the same?

How do I guarantee global uniqueness?

>Since rename, using the IB device name is no longer globally unique so
>it cannot be used as the name in debugfs. You have to fix this..

Why and how does the name of the IB device name have any impact on a filename I have defined in debugfs?

If create new driver for card x and create a debugfs file my_file, and then someone renames an IB device to my_file, did I do something wrong?

If that happens, what will break?

Thanks,

Mike


>
>Jason



[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