mm-check-the-return-value-of-lookup_page_ext-for-all-call-sites-check…

…patch-fixes

WARNING: Missing a blank line after declarations
torvalds#34: FILE: include/linux/page_idle.h:50:
+	struct page_ext *page_ext;
+	page_ext = lookup_page_ext(page);

WARNING: Missing a blank line after declarations
torvalds#45: FILE: include/linux/page_idle.h:60:
+	struct page_ext *page_ext;
+	page_ext = lookup_page_ext(page);

WARNING: Missing a blank line after declarations
torvalds#57: FILE: include/linux/page_idle.h:70:
+	struct page_ext *page_ext;
+	page_ext = lookup_page_ext(page);

WARNING: Missing a blank line after declarations
torvalds#68: FILE: include/linux/page_idle.h:80:
+	struct page_ext *page_ext;
+	page_ext = lookup_page_ext(page);

WARNING: Missing a blank line after declarations
torvalds#79: FILE: include/linux/page_idle.h:90:
+	struct page_ext *page_ext;
+	page_ext = lookup_page_ext(page);

WARNING: Missing a blank line after declarations
torvalds#90: FILE: include/linux/page_idle.h:100:
+	struct page_ext *page_ext;
+	page_ext = lookup_page_ext(page);

total: 1 errors, 6 warnings, 196 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

./patches/mm-check-the-return-value-of-lookup_page_ext-for-all-call-sites.patch has style problems, please review.

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.

Please run checkpatch prior to sending patches

Cc: Joonsoo Kim <[email protected]>
Cc: Yang Shi <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>

include/linux/page_idle.h

@@ -46,8 +46,8 @@ extern struct page_ext_operations page_idle_ops;
 
 static inline bool page_is_young(struct page *page)
 {
-	struct page_ext *page_ext;
-	page_ext = lookup_page_ext(page);
+	struct page_ext *page_ext = lookup_page_ext(page);
+
 	if (unlikely(!page_ext)
 		return false;
 
@@ -56,8 +56,8 @@ static inline bool page_is_young(struct page *page)
 
 static inline void set_page_young(struct page *page)
 {
-	struct page_ext *page_ext;
-	page_ext = lookup_page_ext(page);
+	struct page_ext *page_ext = lookup_page_ext(page);
+
 	if (unlikely(!page_ext)
 		return;
 
@@ -66,8 +66,8 @@ static inline void set_page_young(struct page *page)
 
 static inline bool test_and_clear_page_young(struct page *page)
 {
-	struct page_ext *page_ext;
-	page_ext = lookup_page_ext(page);
+	struct page_ext *page_ext = lookup_page_ext(page);
+
 	if (unlikely(!page_ext)
 		return false;
 
@@ -76,8 +76,8 @@ static inline bool test_and_clear_page_young(struct page *page)
 
 static inline bool page_is_idle(struct page *page)
 {
-	struct page_ext *page_ext;
-	page_ext = lookup_page_ext(page);
+	struct page_ext *page_ext = lookup_page_ext(page);
+
 	if (unlikely(!page_ext)
 		return false;
 
@@ -86,8 +86,8 @@ static inline bool page_is_idle(struct page *page)
 
 static inline void set_page_idle(struct page *page)
 {
-	struct page_ext *page_ext;
-	page_ext = lookup_page_ext(page);
+	struct page_ext *page_ext = lookup_page_ext(page);
+
 	if (unlikely(!page_ext)
 		return;
 
@@ -96,8 +96,8 @@ static inline void set_page_idle(struct page *page)
 
 static inline void clear_page_idle(struct page *page)
 {
-	struct page_ext *page_ext;
-	page_ext = lookup_page_ext(page);
+	struct page_ext *page_ext = lookup_page_ext(page);
+
 	if (unlikely(!page_ext)
 		return;