Thread (11 messages) 11 messages, 3 authors, 2007-12-25

Re: [PATCH 007 of 7] md: Get name for block device in sysfs

From: Kay Sievers <hidden>
Date: 2007-12-15 16:58:41
Also in: lkml

On Dec 14, 2007 7:26 AM, NeilBrown [off-list ref] wrote:
quoted hunk ↗ jump to hunk
Given an fd on a block device, returns a string like

        /block/sda/sda1

which can be used to find related information in /sys.

Ideally we should have an ioctl that works on char devices as well,
but that seems far from trivial, so it seems reasonable to have
this until the later can be implemented.

Cc: Jens Axboe <redacted>
Signed-off-by: Neil Brown <redacted>

### Diffstat output
 ./block/ioctl.c      |   13 +++++++++++++
 ./include/linux/fs.h |    2 ++
 2 files changed, 15 insertions(+)

diff .prev/block/ioctl.c ./block/ioctl.c
--- .prev/block/ioctl.c 2007-12-14 17:18:50.000000000 +1100
+++ ./block/ioctl.c     2007-12-14 16:15:41.000000000 +1100
@@ -227,8 +227,21 @@ int blkdev_ioctl(struct inode *inode, st
        struct block_device *bdev = inode->i_bdev;
        struct gendisk *disk = bdev->bd_disk;
        int ret, n;
+       char b[BDEVNAME_SIZE*2  + 10];

        switch(cmd) {
+       case BLKGETNAME:
+               strcpy(b, "/block/");
As pointed out to when you came up with the idea, we can't do this. A devpath
is a path to the device and will not necessarily start with "/block" for block
devices. It may start with "/devices" and can be much longer than
BDEVNAME_SIZE*2  + 10.

Please do not apply!

Kay
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help