[PATCH] kdump: Write a correct address of mem_section into vmcoreinfo
Ingo Molnar
mingo at kernel.org
Sat Jan 13 02:48:38 PST 2018
* Kirill A. Shutemov <kirill.shutemov at linux.intel.com> wrote:
> Depending on configuration mem_section can now be an array or a pointer
> to an array allocated dynamically. In most cases, we can continue to refer
> to it as 'mem_section' regardless of what it is.
>
> But there's one exception: '&mem_section' means "address of the array" if
> mem_section is an array, but if mem_section is a pointer, it would mean
> "address of the pointer".
>
> We've stepped onto this in kdump code. VMCOREINFO_SYMBOL(mem_section)
> writes down address of pointer into vmcoreinfo, not array as we wanted.
>
> Let's introduce VMCOREINFO_SYMBOL_ARRAY() that would handle the
> situation correctly for both cases.
>
> Signed-off-by: Kirill A. Shutemov <kirill.shutemov at linux.intel.com>
> Fixes: 83e3c48729d9 ("mm/sparsemem: Allocate mem_section at runtime for CONFIG_SPARSEMEM_EXTREME=y")
> Cc: stable at vger.kernel.org
> Acked-by: Baoquan He <bhe at redhat.com>
> Acked-by: Dave Young <dyoung at redhat.com>
You forgot the Reported-by - I added that to the commit.
Thanks,
Ingo
More information about the kexec
mailing list