btrfs: properly track when rescan worker is running
authorJeff Mahoney <jeffm@suse.com>
Mon, 15 Aug 2016 16:10:33 +0000 (12:10 -0400)
committerChris Mason <clm@fb.com>
Thu, 25 Aug 2016 10:58:19 +0000 (03:58 -0700)
commitd2c609b834d62f1e91f1635a27dca29f7806d3d6
tree446eb20c30b212f1ad9def7dc95cb156fd3a2951
parenteecba891d38051ebf7f4af6394d188a5fd151a6a
btrfs: properly track when rescan worker is running

The qgroup_flags field is overloaded such that it reflects the on-disk
status of qgroups and the runtime state.  The BTRFS_QGROUP_STATUS_FLAG_RESCAN
flag is used to indicate that a rescan operation is in progress, but if
the file system is unmounted while a rescan is running, the rescan
operation is paused.  If the file system is then mounted read-only,
the flag will still be present but the rescan operation will not have
been resumed.  When we go to umount, btrfs_qgroup_wait_for_completion
will see the flag and interpret it to mean that the rescan worker is
still running and will wait for a completion that will never come.

This patch uses a separate flag to indicate when the worker is
running.  The locking and state surrounding the qgroup rescan worker
needs a lot of attention beyond this patch but this is enough to
avoid a hung umount.

Cc: <stable@vger.kernel.org> # v4.4+
Signed-off-by; Jeff Mahoney <jeffm@suse.com>
Reviewed-by: Qu Wenruo <quwenruo@cn.fujitsu.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Signed-off-by: Chris Mason <clm@fb.com>
fs/btrfs/ctree.h
fs/btrfs/disk-io.c
fs/btrfs/qgroup.c