Re: [PATCH] nvme: add namespace paths links

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

 



On 4/25/22 15:54, Karel Zak wrote:
On Tue, Apr 05, 2022 at 07:48:36AM +0200, Christoph Hellwig wrote:
On Tue, Apr 05, 2022 at 07:46:40AM +0200, Hannes Reinecke wrote:
On 4/5/22 07:42, Christoph Hellwig wrote:
On Fri, Apr 01, 2022 at 11:28:55AM +0200, Hannes Reinecke wrote:
It is really annoying to always have to loop through the entire
/sys/block directory just to find the namespace path links for
a single namespace in libnvme/nvme-cli.
So provide links to the namespace paths for efficient lookup.

I think having some form of links would be useful.  Quite a while ago
Thadeu looked into adding some form of relationship for lsblk and
friends.  Maybe it would be good to sync up and make sure whatever
links we are adding would be useful for all users?

Care to elaborate?
Using the block device name is the most straightforward way here; we don't
have a good enumeration to leverage as the paths are essentially just a
linked list.

We sure can have a subdirectory 'paths', and stick the links in there.
If that helps with lsblk ...

I personally don't care.  But I've added Thadeu and the util-linux list
to get everyone on board.
lsblk internally can accept arbitrary relations between devices
(and libsmartcols can display it). If you introduce something new or
extend the current holders/slaves, it will probably be accessible for
lsblk to use it.

Christoph, ping?

Shall I resend the patch or can you pick it up directly?

Cheers,

Hannes
--
Dr. Hannes Reinecke		           Kernel Storage Architect
hare@xxxxxxx			                  +49 911 74053 688
SUSE Software Solutions Germany GmbH, Maxfeldstr. 5, 90409 Nürnberg
HRB 36809 (AG Nürnberg), GF: Felix Imendörffer



[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux