Re: [PATCH v2 2/2] elevator: acquire q->sysfs_lock in elevator_change()

2013-09-22 Thread Tejun Heo
On Fri, Aug 30, 2013 at 06:47:16PM -0400, Tomoki Sekiyama wrote: > Add locking of q->sysfs_lock into elevator_change() (an exported function) > to ensure it is held to protect q->elevator from elevator_init(), even if > elevator_change() is called from non-sysfs paths. > sysfs path

Re: [PATCH v2 2/2] elevator: acquire q-sysfs_lock in elevator_change()

2013-09-22 Thread Tejun Heo
On Fri, Aug 30, 2013 at 06:47:16PM -0400, Tomoki Sekiyama wrote: Add locking of q-sysfs_lock into elevator_change() (an exported function) to ensure it is held to protect q-elevator from elevator_init(), even if elevator_change() is called from non-sysfs paths. sysfs path (elv_iosched_store)

[PATCH v2 2/2] elevator: acquire q->sysfs_lock in elevator_change()

2013-08-30 Thread Tomoki Sekiyama
Add locking of q->sysfs_lock into elevator_change() (an exported function) to ensure it is held to protect q->elevator from elevator_init(), even if elevator_change() is called from non-sysfs paths. sysfs path (elv_iosched_store) uses __elevator_change(), non-locking version, as the lock is

[PATCH v2 2/2] elevator: acquire q-sysfs_lock in elevator_change()

2013-08-30 Thread Tomoki Sekiyama
Add locking of q-sysfs_lock into elevator_change() (an exported function) to ensure it is held to protect q-elevator from elevator_init(), even if elevator_change() is called from non-sysfs paths. sysfs path (elv_iosched_store) uses __elevator_change(), non-locking version, as the lock is already