> Am 20.01.2020 um 03:25 schrieb Pingfan Liu <pi...@redhat.com>:
>
> After kernel commit ba72b4c8cf60 ("mm/sparsemem: support sub-section
> hotplug"), when hot-removed, section_mem_map is still encoded with section
> start pfn, not NULL. This break the current makedumpfile.
>
> Whatever section_mem_map coding info after hot-removed, it is reliable
> just to work on SECTION_MARKED_PRESENT bit. Fixing makedumpfile by this
> way.
>
> Signed-off-by: Pingfan Liu <pi...@redhat.com>
> To: kexec@lists.infradead.org
> Cc: Kazuhito Hagio <k-ha...@ab.jp.nec.com>
> Cc: Baoquan He <b...@redhat.com>
> Cc: David Hildenbrand <da...@redhat.com>
> Cc: Andrew Morton <a...@linux-foundation.org>
> Cc: Dan Williams <dan.j.willi...@intel.com>
> Cc: Oscar Salvador <osalva...@suse.de>
> Cc: Michal Hocko <mho...@kernel.org>
> Cc: Qian Cai <c...@lca.pw>
> ---
> makedumpfile.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/makedumpfile.c b/makedumpfile.c
> index e290fbd..ab40a58 100644
> --- a/makedumpfile.c
> +++ b/makedumpfile.c
> @@ -3406,8 +3406,6 @@ section_mem_map_addr(unsigned long addr, unsigned long
> *map_mask)
> map = ULONG(mem_section + OFFSET(mem_section.section_mem_map));
> mask = SECTION_MAP_MASK;
> *map_mask = map & ~mask;
> - if (map == 0x0)
> - *map_mask |= SECTION_MARKED_PRESENT;
Why was that added in the first place? This looks like dome compat handling to
me. Are we sure we can remove it?
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
