Re: [PATCH 3/9] nvme: change namespaces_mutext to namespaces_rwsem

2018-02-11 Thread jianchao.wang
Hi Sagi Thanks for your kindly response. And sorry for my bad description. On 02/11/2018 07:17 PM, Sagi Grimberg wrote: >> namespaces_mutext is used to synchronize the operations on ctrl >> namespaces list. Most of the time, it is a read operation. It is >> better to change it from mutex to rwse

Re: [PATCH 3/9] nvme: change namespaces_mutext to namespaces_rwsem

2018-02-11 Thread Sagi Grimberg
namespaces_mutext is used to synchronize the operations on ctrl namespaces list. Most of the time, it is a read operation. It is better to change it from mutex to rwsem. On the other hand, the namespaces mutex could introduce circular dependency easily. On the other hand of what? Also, can you

[PATCH 3/9] nvme: change namespaces_mutext to namespaces_rwsem

2018-02-11 Thread Jianchao Wang
namespaces_mutext is used to synchronize the operations on ctrl namespaces list. Most of the time, it is a read operation. It is better to change it from mutex to rwsem. On the other hand, the namespaces mutex could introduce circular dependency easily. Signed-off-by: Jianchao Wang --- drivers/