Re: [PATCH 1/3] nvme: fix the dangerous reference of namespaces list

2018-02-12 Thread Sagi Grimberg
Looks good. Reviewed-by: Keith Busch Thanks Keith, appiled to nvme-4.17

Re: [PATCH 1/3] nvme: fix the dangerous reference of namespaces list

2018-02-12 Thread Sagi Grimberg
Looks good. Reviewed-by: Keith Busch Thanks Keith, appiled to nvme-4.17

Re: [PATCH 1/3] nvme: fix the dangerous reference of namespaces list

2018-02-12 Thread Keith Busch
Looks good. Reviewed-by: Keith Busch

Re: [PATCH 1/3] nvme: fix the dangerous reference of namespaces list

2018-02-12 Thread Keith Busch
Looks good. Reviewed-by: Keith Busch

Re: [PATCH 1/3] nvme: fix the dangerous reference of namespaces list

2018-02-12 Thread Sagi Grimberg
Looks good, Reviewed-by: Sagi Grimberg

Re: [PATCH 1/3] nvme: fix the dangerous reference of namespaces list

2018-02-12 Thread Sagi Grimberg
Looks good, Reviewed-by: Sagi Grimberg

[PATCH 1/3] nvme: fix the dangerous reference of namespaces list

2018-02-12 Thread Jianchao Wang
nvme_remove_namespaces and nvme_remove_invalid_namespaces reference the ctrl->namespaces list w/o holding namespaces_mutext. It is ok to invoke nvme_ns_remove there, but what if there is others. To be safer, reference the ctrl->namespaces list under namespaces_mutext. Signed-off-by: Jianchao

[PATCH 1/3] nvme: fix the dangerous reference of namespaces list

2018-02-12 Thread Jianchao Wang
nvme_remove_namespaces and nvme_remove_invalid_namespaces reference the ctrl->namespaces list w/o holding namespaces_mutext. It is ok to invoke nvme_ns_remove there, but what if there is others. To be safer, reference the ctrl->namespaces list under namespaces_mutext. Signed-off-by: Jianchao