diff options
| author |   Guoqing Jiang <guoqing.jiang@cloud.ionos.com> | 2020-11-26 11:47:17 +0100 |
|---|---|---|
| committer |   Jens Axboe <axboe@kernel.dk> | 2020-12-04 09:41:10 -0700 |
| commit | 91f4acb2801ce4985483b0fa174bbe995d105417 (patch) | |
| tree | 5bd92aad5ca2b768122ce7ff686124fee46b1da2 /drivers/block/rnbd/rnbd-clt-sysfs.c | |
| parent | block/rnbd-clt: Make path parameter optional for map_device (diff) | |
| download | wireguard-linux-91f4acb2801ce4985483b0fa174bbe995d105417.tar.xz wireguard-linux-91f4acb2801ce4985483b0fa174bbe995d105417.zip | |
block/rnbd-clt: support mapping two devices with the same name from different servers
Previously, we can't map same device name from different sessions
due to the limitation of sysfs naming mechanism.
root@clt2:~# ls -l /sys/class/rnbd-client/ctl/devices/
total 0
lrwxrwxrwx 1 root 0 Sep 2 16:31 !dev!nullb1 -> ../../../block/rnbd0
We only use the device name in above, which caused device with
the same name can't be mapped from another server. To address
the issue, the sessname is appended to the node to differentiate
where the device comes from.
Also, we need to check if the pathname is existed in a specific
session instead of search it in global sess_list.
Signed-off-by: Guoqing Jiang <guoqing.jiang@cloud.ionos.com>
Signed-off-by: Gioh Kim <gi-oh.kim@cloud.ionos.com>
Signed-off-by: Jack Wang <jinpu.wang@cloud.ionos.com>
Reviewed-by: Md Haris Iqbal <haris.iqbal@cloud.ionos.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/block/rnbd/rnbd-clt-sysfs.c')
| -rw-r--r-- | drivers/block/rnbd/rnbd-clt-sysfs.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/block/rnbd/rnbd-clt-sysfs.c b/drivers/block/rnbd/rnbd-clt-sysfs.c index e7b41ec7cd6a..5d3c3c80dab4 100644 --- a/drivers/block/rnbd/rnbd-clt-sysfs.c +++ b/drivers/block/rnbd/rnbd-clt-sysfs.c @@ -480,6 +480,10 @@ static int rnbd_clt_get_path_name(struct rnbd_clt_dev *dev, char *buf, if (ret >= len) return -ENAMETOOLONG; + ret = snprintf(buf, len, "%s@%s", buf, dev->sess->sessname); + if (ret >= len) + return -ENAMETOOLONG; + return 0; } |