From: Baoquan He <bhe@xxxxxxxxxx> Pratyush has made a kernel patch [0] to add the physical address of direct mapping kcore program segments. So no need to calculate it specifically now. And the old code is not correct since it calls vaddr_to_paddr() which has been not ready at that time. [0] http://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/?h=next-20170206&id=c9d4e5d7b7fd6c74e134ca44df8a5386efbc561c Signed-off-by: Baoquan He <bhe at redhat.com> --- elf_info.c | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/elf_info.c b/elf_info.c index 100272f83c48..8e2437622141 100644 --- a/elf_info.c +++ b/elf_info.c @@ -857,22 +857,6 @@ static int exclude_segment(struct pt_load_segment **pt_loads, return 0; } -static int -process_dump_load(struct pt_load_segment *pls) -{ - unsigned long long paddr; - - paddr = vaddr_to_paddr(pls->virt_start); - pls->phys_start = paddr; - pls->phys_end = paddr + (pls->virt_end - pls->virt_start); - DEBUG_MSG("process_dump_load\n"); - DEBUG_MSG(" phys_start : %llx\n", pls->phys_start); - DEBUG_MSG(" phys_end : %llx\n", pls->phys_end); - DEBUG_MSG(" virt_start : %llx\n", pls->virt_start); - DEBUG_MSG(" virt_end : %llx\n", pls->virt_end); - - return TRUE; -} int get_kcore_dump_loads(void) { @@ -917,7 +901,6 @@ int get_kcore_dump_loads(void) } pls[j] = *p; - process_dump_load(&pls[j]); j++; } -- 2.9.3
