All of lore.kernel.org
 help / color / mirror / Atom feed
* Namespace removal deadlock in case of namespace detach command
@ 2016-04-18 12:00 Sunad Bhandary S
  2016-04-18 14:57 ` Keith Busch
  0 siblings, 1 reply; 3+ messages in thread
From: Sunad Bhandary S @ 2016-04-18 12:00 UTC (permalink / raw)


On receipt of a namespace attribute changed AER, we acquire the namespace mutex lock before proceeding to scan and validate the namespace list.
In case of namespace detach/delete command, nvme_ns_remove function deadlocks trying to acquire the already held lock.

Thanks and Regards,
Sunad

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Namespace removal deadlock in case of namespace detach command
  2016-04-18 12:00 Namespace removal deadlock in case of namespace detach command Sunad Bhandary S
@ 2016-04-18 14:57 ` Keith Busch
  2016-04-18 17:52   ` Ming Lin
  0 siblings, 1 reply; 3+ messages in thread
From: Keith Busch @ 2016-04-18 14:57 UTC (permalink / raw)


On Mon, Apr 18, 2016@12:00:51PM +0000, Sunad Bhandary S wrote:
> On receipt of a namespace attribute changed AER, we acquire the namespace mutex lock before proceeding to scan and validate the namespace list.
> In case of namespace detach/delete command, nvme_ns_remove function deadlocks trying to acquire the already held lock.

You're right, thanks for the catch. Do you have a fix, or should we
propose one?

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Namespace removal deadlock in case of namespace detach command
  2016-04-18 14:57 ` Keith Busch
@ 2016-04-18 17:52   ` Ming Lin
  0 siblings, 0 replies; 3+ messages in thread
From: Ming Lin @ 2016-04-18 17:52 UTC (permalink / raw)


On Mon, Apr 18, 2016@7:57 AM, Keith Busch <keith.busch@intel.com> wrote:
> On Mon, Apr 18, 2016@12:00:51PM +0000, Sunad Bhandary S wrote:
>> On receipt of a namespace attribute changed AER, we acquire the namespace mutex lock before proceeding to scan and validate the namespace list.
>> In case of namespace detach/delete command, nvme_ns_remove function deadlocks trying to acquire the already held lock.
>
> You're right, thanks for the catch. Do you have a fix, or should we
> propose one?

We have a fix in nvme over fabrics tree.
Let me post here soon.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-04-18 17:52 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-18 12:00 Namespace removal deadlock in case of namespace detach command Sunad Bhandary S
2016-04-18 14:57 ` Keith Busch
2016-04-18 17:52   ` Ming Lin

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.