Commit 32d0c98e authored by ye xingchen's avatar ye xingchen Committed by Andrew Morton
Browse files

kexec: remove the unneeded result variable 

Return the value kimage_add_entry() directly instead of storing it in
another redundant variable.

Link: https://lkml.kernel.org/r/20220929042936.22012-3-bhe@redhat.com


Signed-off-by: default avatarye xingchen <ye.xingchen@zte.com.cn>
Signed-off-by: default avatarBaoquan He <bhe@redhat.com>
Reported-by: default avatarZeal Robot <zealci@zte.com.cn>
Acked-by: default avatarBaoquan He <bhe@redhat.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Chen Lifu <chenlifu@huawei.com>
Cc: "Eric W . Biederman" <ebiederm@xmission.com>
Cc: Jianglei Nie <niejianglei2021@163.com>
Cc: Li Chen <lchen@ambarella.com>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Petr Mladek <pmladek@suse.com>
Cc: Russell King <linux@armlinux.org.uk>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent 12b9d301

kernel/kexec_core.c

+2 −8
Original line number Diff line number Diff line
@@ -561,23 +561,17 @@ static int kimage_add_entry(struct kimage *image, kimage_entry_t entry) 
static int kimage_set_destination(struct kimage *image,

				   unsigned long destination)

{

	int result;



	destination &= PAGE_MASK;

	result = kimage_add_entry(image, destination | IND_DESTINATION);



	return result;

	return kimage_add_entry(image, destination | IND_DESTINATION);

}





static int kimage_add_page(struct kimage *image, unsigned long page)

{

	int result;



	page &= PAGE_MASK;

	result = kimage_add_entry(image, page | IND_SOURCE);



	return result;

	return kimage_add_entry(image, page | IND_SOURCE);

}