Re: [PATCH v8 1/7] x86/crash: fix minor typo/bug in debug message

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 5/9/22 00:26, Baoquan He wrote:
On 05/09/22 at 10:46am, Sourabh Jain wrote:

On 06/05/22 00:15, Eric DeVolder wrote:
The pr_debug() intends to display the memsz member, but the
parameter is actually the bufsz member (which is already
displayed). Correct this to display memsz value.

Signed-off-by: Eric DeVolder<eric.devolder@xxxxxxxxxx>
Reviewed-by: David Hildenbrand<david@xxxxxxxxxx>
Acked-by: Baoquan He<bhe@xxxxxxxxxx>
---
   arch/x86/kernel/crash.c | 2 +-
   1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/kernel/crash.c b/arch/x86/kernel/crash.c
index e8326a8d1c5d..9730c88530fc 100644
--- a/arch/x86/kernel/crash.c
+++ b/arch/x86/kernel/crash.c
@@ -407,7 +407,7 @@ int crash_load_segments(struct kimage *image)
   	}
   	image->elf_load_addr = kbuf.mem;
   	pr_debug("Loaded ELF headers at 0x%lx bufsz=0x%lx memsz=0x%lx\n",
-		 image->elf_load_addr, kbuf.bufsz, kbuf.bufsz);
+		 image->elf_load_addr, kbuf.bufsz, kbuf.memsz);

I think we can push this patch separately.

Boris has taken this into tip/x86/kdump.

Excellent, I'll remove this patch going forward.
Eric

_______________________________________________
kexec mailing list
kexec@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/kexec



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux