erofs: relinquish volume with mutex held
authorJingbo Xu <jefflexu@linux.alibaba.com>
Thu, 9 Feb 2023 06:39:12 +0000 (14:39 +0800)
committerGao Xiang <hsiangkao@linux.alibaba.com>
Wed, 15 Feb 2023 00:11:27 +0000 (08:11 +0800)
Relinquish fscache volume with mutex held.  Otherwise if a new domain is
registered when the old domain with the same name gets removed from the
list but not relinquished yet, fscache may complain the collision.

Fixes: 8b7adf1dff3d ("erofs: introduce fscache-based domain")
Signed-off-by: Jingbo Xu <jefflexu@linux.alibaba.com>
Reviewed-by: Jia Zhu <zhujia.zj@bytedance.com>
Link: https://lore.kernel.org/r/20230209063913.46341-4-jefflexu@linux.alibaba.com
Signed-off-by: Gao Xiang <hsiangkao@linux.alibaba.com>
fs/erofs/fscache.c

index 0ee08e51c4a65ca8d8c79361c8c2ea9e732031f5..9383afe09bd053ab0b25e012f8e99853d7b45e56 100644 (file)
@@ -326,8 +326,8 @@ static void erofs_fscache_domain_put(struct erofs_domain *domain)
                        kern_unmount(erofs_pseudo_mnt);
                        erofs_pseudo_mnt = NULL;
                }
-               mutex_unlock(&erofs_domain_list_lock);
                fscache_relinquish_volume(domain->volume, NULL, false);
+               mutex_unlock(&erofs_domain_list_lock);
                kfree(domain->domain_id);
                kfree(domain);
                return;