dm snapshot: only take lock for statustype info not table
authorMikulas Patocka <mpatocka@redhat.com>
Thu, 10 Dec 2009 23:51:53 +0000 (23:51 +0000)
committerAlasdair G Kergon <agk@redhat.com>
Thu, 10 Dec 2009 23:51:53 +0000 (23:51 +0000)
commit94e76572b5dd37b1f0f4b3742ee8a565daead932
tree2e7737f21d99a881b7a2d0977adad1d2bd9bfb87
parentd2bb7df8cac647b92f51fb84ae735771e7adbfa7
dm snapshot: only take lock for statustype info not table

Take snapshot lock only for STATUSTYPE_INFO, not STATUSTYPE_TABLE.

Commit 4c6fff445d7aa753957856278d4d93bcad6e2c14
(dm-snapshot-lock-snapshot-while-supplying-status.patch)
introduced this use of the lock, but userspace applications using
libdevmapper have been found to request STATUSTYPE_TABLE while the device
is suspended and the lock is already held, leading to deadlock.  Since
the lock is not necessary in this case, don't try to take it.

Cc: stable@kernel.org
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
drivers/md/dm-snap.c