Commit d9674dda authored by Sasha Levin's avatar Sasha Levin Committed by Konrad Rzeszutek Wilk

mm: frontswap: make all branches of if statement in put page consistent

Currently it has a complex structure where different things are compared
at each branch. Simplify that and make both branches look similar.
Reviewed-by: default avatarMinchan Kim <minchan@kernel.org>
Signed-off-by: default avatarSasha Levin <levinsasha928@gmail.com>
Signed-off-by: default avatarKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
parent 69217b4c
......@@ -140,16 +140,16 @@ int __frontswap_store(struct page *page)
inc_frontswap_succ_stores();
if (!dup)
atomic_inc(&sis->frontswap_pages);
} else if (dup) {
} else {
/*
failed dup always results in automatic invalidate of
the (older) page from frontswap
*/
frontswap_clear(sis, offset);
atomic_dec(&sis->frontswap_pages);
inc_frontswap_failed_stores();
} else {
inc_frontswap_failed_stores();
if (dup) {
frontswap_clear(sis, offset);
atomic_dec(&sis->frontswap_pages);
}
}
if (frontswap_writethrough_enabled)
/* report failure so swap also writes to swap device */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment