Re: [PATCH 3/7] block: Remove sysfs lock from elevator_init_rq()

2019-08-23 Thread Bart Van Assche
On 8/22/19 5:15 PM, Damien Le Moal wrote: > Since elevator_init_rq() is called before the device queue is registered > in sysfs, there is no possible conflict with elevator_switch(). Remove > the unnecessary locking of q->sysfs_lock mutex. > > Signed-off-by: Damien Le Moal > --- > block/elevator

Re: [PATCH 3/7] block: Remove sysfs lock from elevator_init_rq()

2019-08-23 Thread Johannes Thumshirn
On 23/08/2019 02:15, Damien Le Moal wrote: > Since elevator_init_rq() is called before the device queue is registered elevator_init_mq() ~^ (and in Subject) Otherwise: Reviewed-by: Johannes Thumshirn -- Johannes ThumshirnSUSE Labs Filesystems jthumsh...@suse.de

[PATCH 3/7] block: Remove sysfs lock from elevator_init_rq()

2019-08-22 Thread Damien Le Moal
Since elevator_init_rq() is called before the device queue is registered in sysfs, there is no possible conflict with elevator_switch(). Remove the unnecessary locking of q->sysfs_lock mutex. Signed-off-by: Damien Le Moal --- block/elevator.c | 12 ++-- 1 file changed, 2 insertions(+), 1