Messages in this thread | | | Date | Mon, 9 May 2022 10:41:13 -0500 | Subject | Re: [PATCH v8 1/7] x86/crash: fix minor typo/bug in debug message | From | Eric DeVolder <> |
| |
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@oracle.com> >>> Reviewed-by: David Hildenbrand<david@redhat.com> >>> Acked-by: Baoquan He<bhe@redhat.com> >>> --- >>> 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
| |