AuxXxilium/linux_dsm_epyc7002
1
0
Fork 0
mirror of https://github.com/AuxXxilium/linux_dsm_epyc7002.git synced 2024-11-24 05:30:52 +07:00
Code Issues Actions Packages Projects Releases Wiki Activity

Btrfs: remove unnecessary ->s_umount in cleaner_kthread()

Browse Source 
In order to avoid the R/O remount, we acquired ->s_umount lock during
we deleted the dead snapshots and subvolumes. But it is unnecessary,
because we have cleaner_mutex.

We use cleaner_mutex to protect the process of the dead snapshots/subvolumes
deletion. And when we remount the fs to be R/O, we also acquire this mutex to
do cleanup after we change the status of the fs. That is this lock can serialize
the above operations, the cleaner can be aware of the status of the fs, and if
the cleaner is deleting the dead snapshots/subvolumes, the remount task will
wait for it. So it is safe to remove ->s_umount in cleaner_kthread().

Cc: David Sterba <dsterba@suse.cz>
Signed-off-by: Miao Xie <miaox@cn.fujitsu.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
This commit is contained in:
Miao Xie 2013-05-14 10:20:40 +00:00 committed by Josef Bacik
parent 3c64a1aba7
commit d027824564
1 changed files with 27 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
fs/btrfs/disk-io.c
View File

@ -1673,24 +1673,40 @@ static void end_workqueue_fn(struct btrfs_work *work)
bio_endio(bio, error);
}
/*
* If we remount the fs to be R/O, the cleaner needn't do anything except
* sleeping. This function is used to check the status of the fs.
*/
static inline int need_cleaner_sleep(struct btrfs_root *root)
{
return root->fs_info->sb->s_flags & MS_RDONLY;
}
static int cleaner_kthread(void *arg)
{
struct btrfs_root *root = arg;
int again;
do {
int again = 0;
again = 0;
if (!(root->fs_info->sb->s_flags & MS_RDONLY) &&
down_read_trylock(&root->fs_info->sb->s_umount)) {
if (mutex_trylock(&root->fs_info->cleaner_mutex)) {
btrfs_run_delayed_iputs(root);
again = btrfs_clean_one_deleted_snapshot(root);
mutex_unlock(&root->fs_info->cleaner_mutex);
}
btrfs_run_defrag_inodes(root->fs_info);
up_read(&root->fs_info->sb->s_umount);
}
/* Make the cleaner go to sleep early. */
if (need_cleaner_sleep(root))
goto sleep;
if (!mutex_trylock(&root->fs_info->cleaner_mutex))
goto sleep;
btrfs_run_delayed_iputs(root);
again = btrfs_clean_one_deleted_snapshot(root);
mutex_unlock(&root->fs_info->cleaner_mutex);
/*
* The defragger has dealt with the R/O remount, needn't
* do anything special here.
*/
btrfs_run_defrag_inodes(root->fs_info);
sleep:
if (!try_to_freeze() && !again) {
set_current_state(TASK_INTERRUPTIBLE);
if (!kthread_should_stop())