diff options
| author | Jaegeuk Kim <jaegeuk@kernel.org> | 2017-07-06 14:46:01 -0700 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2017-07-13 16:58:17 -0700 |
| commit | e4a2b45da36b6e2c31bdee76550f0eccd1f9a21f (patch) | |
| tree | 838049793d88edc47b7fcdbd2f3187ef624451d4 | |
| parent | 4f7f22b7c85a4b25247df14955c64761b401ebc5 (diff) | |
f2fs: relax migratepage for atomic written page
In order to avoid lock contention for atomic written pages, we'd better give
EBUSY in f2fs_migrate_page when mode is asynchronous. We expect it will be
released soon as transaction commits.
Reviewed-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
| -rw-r--r-- | fs/f2fs/data.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c index 4a1730cfe86c..dc204f178b13 100644 --- a/fs/f2fs/data.c +++ b/fs/f2fs/data.c @@ -2202,8 +2202,12 @@ int f2fs_migrate_page(struct address_space *mapping, BUG_ON(PageWriteback(page)); /* migrating an atomic written page is safe with the inmem_lock hold */ - if (atomic_written && !mutex_trylock(&fi->inmem_lock)) - return -EAGAIN; + if (atomic_written) { + if (mode != MIGRATE_SYNC) + return -EBUSY; + if (!mutex_trylock(&fi->inmem_lock)) + return -EAGAIN; + } /* * A reference is expected if PagePrivate set when move mapping, |