[PATCH v2 0/1] mm/page_io.c: remove an unnecessary goto in generic_swapfile_activate()

[PATCH v2 0/1] mm/page_io.c: remove an unnecessary goto in generic_swapfile_activate()

[Zhen Lei]

v1 --> v2:
1. rename subject
2. give up adding an newline to seperate two labels

Zhen Lei (1):
  mm/page_io.c: remove an unnecessary goto in
    generic_swapfile_activate()

 mm/page_io.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

-- 
2.26.0.106.g9fadedd

[PATCH v2 1/1] mm/page_io.c: remove an unnecessary goto in generic_swapfile_activate()

[Zhen Lei]

This "goto" doesn't reduce any code, but bother the readers.

Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
Reviewed-by: David Hildenbrand <david@redhat.com>
---
 mm/page_io.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/mm/page_io.c b/mm/page_io.c
index 76965be1d40e..19d429abb2cc 100644
--- a/mm/page_io.c
+++ b/mm/page_io.c
@@ -237,8 +237,7 @@ int generic_swapfile_activate(struct swap_info_struct *sis,
 	return ret;
 bad_bmap:
 	pr_err("swapon: swapfile has holes\n");
-	ret = -EINVAL;
-	goto out;
+	return -EINVAL;
 }
 
 /*
-- 
2.26.0.106.g9fadedd