Re: [PATCH 23/24] ibnbd: a bit of documentation

2018-02-05 Thread Sagi Grimberg
/sys/kernel was chosen ages ago and I completely forgot to move it to configfs. IBTRS is not a block device, so for some read-only entries (statistics or states) something else should be probably used, not configfs. Or it is fine to read state of the connection from configfs? For me sounds a b

Re: [PATCH 23/24] ibnbd: a bit of documentation

2018-02-05 Thread Roman Penyaev
On Fri, Feb 2, 2018 at 4:55 PM, Bart Van Assche wrote: > On Fri, 2018-02-02 at 15:09 +0100, Roman Pen wrote: >> +Entries under /sys/kernel/ibnbd_client/ >> +=== >> [ ... ] > > You will need Greg KH's permission to add new entries directly under > /sys/kernel. >

Re: [PATCH 23/24] ibnbd: a bit of documentation

2018-02-02 Thread Bart Van Assche
On Fri, 2018-02-02 at 15:09 +0100, Roman Pen wrote: > +Entries under /sys/kernel/ibnbd_client/ > +=== > [ ... ] You will need Greg KH's permission to add new entries directly under /sys/kernel. Since I think that it is unlikely that he will give that permission

[PATCH 23/24] ibnbd: a bit of documentation

2018-02-02 Thread Roman Pen
README with description of major sysfs entries. Signed-off-by: Roman Pen Signed-off-by: Danil Kipnis Cc: Jack Wang --- drivers/block/ibnbd/README | 272 + 1 file changed, 272 insertions(+) diff --git a/drivers/block/ibnbd/README b/drivers/block/ibnb