kexec: remove unnecessary test in kimage_alloc_crash_control_pages()
authorMinfei Huang <mnfhuang@gmail.com>
Wed, 9 Sep 2015 22:38:58 +0000 (15:38 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 10 Sep 2015 20:29:01 +0000 (13:29 -0700)
Transforming PFN(Page Frame Number) to struct page is never failure, so we
can simplify the code logic to do the image->control_page assignment
directly in the loop, and remove the unnecessary conditional judgement.

Signed-off-by: Minfei Huang <mnfhuang@gmail.com>
Acked-by: Dave Young <dyoung@redhat.com>
Acked-by: Vivek Goyal <vgoyal@redhat.com>
Cc: Simon Horman <horms@verge.net.au>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
kernel/kexec_core.c

index 9aa25c034b2ef4b30c769171db9525b7a0e1d5cc..9ffc96b65d9a891b98ffd03c723420a67522f179 100644 (file)
@@ -439,11 +439,10 @@ static struct page *kimage_alloc_crash_control_pages(struct kimage *image,
                /* If I don't overlap any segments I have found my hole! */
                if (i == image->nr_segments) {
                        pages = pfn_to_page(hole_start >> PAGE_SHIFT);
+                       image->control_page = hole_end;
                        break;
                }
        }
-       if (pages)
-               image->control_page = hole_end;
 
        return pages;
 }