diff options
author | 2020-12-10 22:01:03 +0800 | |
---|---|---|
committer | 2020-12-22 09:54:48 -0500 | |
commit | b77709237e72d6467fb27bfbad163f7221ecd648 (patch) | |
tree | d0395f65fad7405ac711e25a448095ac555ff455 /drivers | |
parent | dm ebs: avoid double unlikely() notation when using IS_ERR() (diff) | |
download | linux-b77709237e72d6467fb27bfbad163f7221ecd648.tar.gz linux-b77709237e72d6467fb27bfbad163f7221ecd648.tar.bz2 linux-b77709237e72d6467fb27bfbad163f7221ecd648.zip |
dm cache: simplify the return expression of load_mapping()
Simplify the return expression.
Signed-off-by: Zheng Yongjun <zhengyongjun3@huawei.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/md/dm-cache-target.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/md/dm-cache-target.c b/drivers/md/dm-cache-target.c index 4bc453f5bbaa..541c45027cc8 100644 --- a/drivers/md/dm-cache-target.c +++ b/drivers/md/dm-cache-target.c @@ -2840,7 +2840,6 @@ static void cache_postsuspend(struct dm_target *ti) static int load_mapping(void *context, dm_oblock_t oblock, dm_cblock_t cblock, bool dirty, uint32_t hint, bool hint_valid) { - int r; struct cache *cache = context; if (dirty) { @@ -2849,11 +2848,7 @@ static int load_mapping(void *context, dm_oblock_t oblock, dm_cblock_t cblock, } else clear_bit(from_cblock(cblock), cache->dirty_bitset); - r = policy_load_mapping(cache->policy, oblock, cblock, dirty, hint, hint_valid); - if (r) - return r; - - return 0; + return policy_load_mapping(cache->policy, oblock, cblock, dirty, hint, hint_valid); } /* |