oe-kbuild-all.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
* [akpm-mm:mm-nonmm-unstable 56/78] kernel/crash_core.c:331:23: error: invalid use of undefined type 'struct crash_mem'
@ 2023-08-11  9:10 kernel test robot
  2023-08-11 17:16 ` Eric DeVolder
  0 siblings, 1 reply; 2+ messages in thread
From: kernel test robot @ 2023-08-11  9:10 UTC (permalink / raw)
  To: Eric DeVolder
  Cc: oe-kbuild-all, Andrew Morton, Linux Memory Management List, Sourabh Jain

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-nonmm-unstable
head:   0f6b2c125ce19074c1086a70119b02dd30b089a4
commit: 3f1eba00ee9d93b66f1859d40537befe44ceb1cd [56/78] crash: move a few code bits to setup support of crash hotplug
config: i386-randconfig-r015-20230811 (https://download.01.org/0day-ci/archive/20230811/202308111729.ApUbiGtj-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce: (https://download.01.org/0day-ci/archive/20230811/202308111729.ApUbiGtj-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202308111729.ApUbiGtj-lkp@intel.com/

All errors (new ones prefixed by >>):

   kernel/crash_core.c:318:40: warning: 'struct crash_mem' declared inside parameter list will not be visible outside of this definition or declaration
     318 | int crash_prepare_elf64_headers(struct crash_mem *mem, int need_kernel_map,
         |                                        ^~~~~~~~~
   kernel/crash_core.c:318:5: warning: no previous prototype for 'crash_prepare_elf64_headers' [-Wmissing-prototypes]
     318 | int crash_prepare_elf64_headers(struct crash_mem *mem, int need_kernel_map,
         |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   kernel/crash_core.c: In function 'crash_prepare_elf64_headers':
>> kernel/crash_core.c:331:23: error: invalid use of undefined type 'struct crash_mem'
     331 |         nr_phdr += mem->nr_ranges;
         |                       ^~
   In file included from include/vdso/const.h:5,
                    from include/linux/const.h:4,
                    from include/linux/align.h:5,
                    from include/linux/kernel.h:15,
                    from include/linux/cpumask.h:10,
                    from include/linux/mm_types_task.h:14,
                    from include/linux/mm_types.h:5,
                    from include/linux/buildid.h:5,
                    from kernel/crash_core.c:7:
>> kernel/crash_core.c:343:32: error: 'ELF_CORE_HEADER_ALIGN' undeclared (first use in this function)
     343 |         elf_sz = ALIGN(elf_sz, ELF_CORE_HEADER_ALIGN);
         |                                ^~~~~~~~~~~~~~~~~~~~~
   include/uapi/linux/const.h:32:50: note: in definition of macro '__ALIGN_KERNEL_MASK'
      32 | #define __ALIGN_KERNEL_MASK(x, mask)    (((x) + (mask)) & ~(mask))
         |                                                  ^~~~
   include/linux/align.h:8:33: note: in expansion of macro '__ALIGN_KERNEL'
       8 | #define ALIGN(x, a)             __ALIGN_KERNEL((x), (a))
         |                                 ^~~~~~~~~~~~~~
   kernel/crash_core.c:343:18: note: in expansion of macro 'ALIGN'
     343 |         elf_sz = ALIGN(elf_sz, ELF_CORE_HEADER_ALIGN);
         |                  ^~~~~
   kernel/crash_core.c:343:32: note: each undeclared identifier is reported only once for each function it appears in
     343 |         elf_sz = ALIGN(elf_sz, ELF_CORE_HEADER_ALIGN);
         |                                ^~~~~~~~~~~~~~~~~~~~~
   include/uapi/linux/const.h:32:50: note: in definition of macro '__ALIGN_KERNEL_MASK'
      32 | #define __ALIGN_KERNEL_MASK(x, mask)    (((x) + (mask)) & ~(mask))
         |                                                  ^~~~
   include/linux/align.h:8:33: note: in expansion of macro '__ALIGN_KERNEL'
       8 | #define ALIGN(x, a)             __ALIGN_KERNEL((x), (a))
         |                                 ^~~~~~~~~~~~~~
   kernel/crash_core.c:343:18: note: in expansion of macro 'ALIGN'
     343 |         elf_sz = ALIGN(elf_sz, ELF_CORE_HEADER_ALIGN);
         |                  ^~~~~
   kernel/crash_core.c:393:28: error: invalid use of undefined type 'struct crash_mem'
     393 |         for (i = 0; i < mem->nr_ranges; i++) {
         |                            ^~
   kernel/crash_core.c:394:29: error: invalid use of undefined type 'struct crash_mem'
     394 |                 mstart = mem->ranges[i].start;
         |                             ^~
   kernel/crash_core.c:395:27: error: invalid use of undefined type 'struct crash_mem'
     395 |                 mend = mem->ranges[i].end;
         |                           ^~
   kernel/crash_core.c: At top level:
   kernel/crash_core.c:417:36: warning: 'struct crash_mem' declared inside parameter list will not be visible outside of this definition or declaration
     417 | int crash_exclude_mem_range(struct crash_mem *mem,
         |                                    ^~~~~~~~~
   kernel/crash_core.c:417:5: warning: no previous prototype for 'crash_exclude_mem_range' [-Wmissing-prototypes]
     417 | int crash_exclude_mem_range(struct crash_mem *mem,
         |     ^~~~~~~~~~~~~~~~~~~~~~~
   kernel/crash_core.c: In function 'crash_exclude_mem_range':
   kernel/crash_core.c:424:28: error: invalid use of undefined type 'struct crash_mem'
     424 |         for (i = 0; i < mem->nr_ranges; i++) {
         |                            ^~
   kernel/crash_core.c:425:28: error: invalid use of undefined type 'struct crash_mem'
     425 |                 start = mem->ranges[i].start;
         |                            ^~
   kernel/crash_core.c:426:26: error: invalid use of undefined type 'struct crash_mem'
     426 |                 end = mem->ranges[i].end;
         |                          ^~
   kernel/crash_core.c:441:28: error: invalid use of undefined type 'struct crash_mem'
     441 |                         mem->ranges[i].start = 0;
         |                            ^~
   kernel/crash_core.c:442:28: error: invalid use of undefined type 'struct crash_mem'
     442 |                         mem->ranges[i].end = 0;
         |                            ^~
   kernel/crash_core.c:443:36: error: invalid use of undefined type 'struct crash_mem'
     443 |                         if (i < mem->nr_ranges - 1) {
         |                                    ^~
   kernel/crash_core.c:445:52: error: invalid use of undefined type 'struct crash_mem'
     445 |                                 for (j = i; j < mem->nr_ranges - 1; j++) {
         |                                                    ^~
   kernel/crash_core.c:446:44: error: invalid use of undefined type 'struct crash_mem'
     446 |                                         mem->ranges[j].start =
         |                                            ^~
   kernel/crash_core.c:447:52: error: invalid use of undefined type 'struct crash_mem'
     447 |                                                 mem->ranges[j+1].start;
         |                                                    ^~
   kernel/crash_core.c:448:44: error: invalid use of undefined type 'struct crash_mem'
     448 |                                         mem->ranges[j].end =
         |                                            ^~
   kernel/crash_core.c:449:60: error: invalid use of undefined type 'struct crash_mem'
     449 |                                                         mem->ranges[j+1].end;
         |                                                            ^~
   kernel/crash_core.c:458:36: error: invalid use of undefined type 'struct crash_mem'
     458 |                                 mem->nr_ranges--;
         |                                    ^~
   kernel/crash_core.c:461:28: error: invalid use of undefined type 'struct crash_mem'
     461 |                         mem->nr_ranges--;
         |                            ^~
   kernel/crash_core.c:467:28: error: invalid use of undefined type 'struct crash_mem'
     467 |                         mem->ranges[i].end = p_start - 1;
         |                            ^~
   kernel/crash_core.c:471:28: error: invalid use of undefined type 'struct crash_mem'
     471 |                         mem->ranges[i].end = p_start - 1;
         |                            ^~
   kernel/crash_core.c:473:28: error: invalid use of undefined type 'struct crash_mem'
     473 |                         mem->ranges[i].start = p_end + 1;
         |                            ^~
   kernel/crash_core.c:482:21: error: invalid use of undefined type 'struct crash_mem'
     482 |         if (i == mem->max_nr_ranges - 1)
         |                     ^~
   kernel/crash_core.c:487:20: error: invalid use of undefined type 'struct crash_mem'
     487 |         if (j < mem->nr_ranges) {
         |                    ^~
   kernel/crash_core.c:489:29: error: invalid use of undefined type 'struct crash_mem'
     489 |                 for (i = mem->nr_ranges - 1; i >= j; i--)
         |                             ^~
   kernel/crash_core.c:490:28: error: invalid use of undefined type 'struct crash_mem'
     490 |                         mem->ranges[i + 1] = mem->ranges[i];
         |                            ^~


vim +331 kernel/crash_core.c

   317	
   318	int crash_prepare_elf64_headers(struct crash_mem *mem, int need_kernel_map,
   319				  void **addr, unsigned long *sz)
   320	{
   321		Elf64_Ehdr *ehdr;
   322		Elf64_Phdr *phdr;
   323		unsigned long nr_cpus = num_possible_cpus(), nr_phdr, elf_sz;
   324		unsigned char *buf;
   325		unsigned int cpu, i;
   326		unsigned long long notes_addr;
   327		unsigned long mstart, mend;
   328	
   329		/* extra phdr for vmcoreinfo ELF note */
   330		nr_phdr = nr_cpus + 1;
 > 331		nr_phdr += mem->nr_ranges;
   332	
   333		/*
   334		 * kexec-tools creates an extra PT_LOAD phdr for kernel text mapping
   335		 * area (for example, ffffffff80000000 - ffffffffa0000000 on x86_64).
   336		 * I think this is required by tools like gdb. So same physical
   337		 * memory will be mapped in two ELF headers. One will contain kernel
   338		 * text virtual addresses and other will have __va(physical) addresses.
   339		 */
   340	
   341		nr_phdr++;
   342		elf_sz = sizeof(Elf64_Ehdr) + nr_phdr * sizeof(Elf64_Phdr);
 > 343		elf_sz = ALIGN(elf_sz, ELF_CORE_HEADER_ALIGN);
   344	
   345		buf = vzalloc(elf_sz);
   346		if (!buf)
   347			return -ENOMEM;
   348	
   349		ehdr = (Elf64_Ehdr *)buf;
   350		phdr = (Elf64_Phdr *)(ehdr + 1);
   351		memcpy(ehdr->e_ident, ELFMAG, SELFMAG);
   352		ehdr->e_ident[EI_CLASS] = ELFCLASS64;
   353		ehdr->e_ident[EI_DATA] = ELFDATA2LSB;
   354		ehdr->e_ident[EI_VERSION] = EV_CURRENT;
   355		ehdr->e_ident[EI_OSABI] = ELF_OSABI;
   356		memset(ehdr->e_ident + EI_PAD, 0, EI_NIDENT - EI_PAD);
   357		ehdr->e_type = ET_CORE;
   358		ehdr->e_machine = ELF_ARCH;
   359		ehdr->e_version = EV_CURRENT;
   360		ehdr->e_phoff = sizeof(Elf64_Ehdr);
   361		ehdr->e_ehsize = sizeof(Elf64_Ehdr);
   362		ehdr->e_phentsize = sizeof(Elf64_Phdr);
   363	
   364		/* Prepare one phdr of type PT_NOTE for each present CPU */
   365		for_each_present_cpu(cpu) {
   366			phdr->p_type = PT_NOTE;
   367			notes_addr = per_cpu_ptr_to_phys(per_cpu_ptr(crash_notes, cpu));
   368			phdr->p_offset = phdr->p_paddr = notes_addr;
   369			phdr->p_filesz = phdr->p_memsz = sizeof(note_buf_t);
   370			(ehdr->e_phnum)++;
   371			phdr++;
   372		}
   373	
   374		/* Prepare one PT_NOTE header for vmcoreinfo */
   375		phdr->p_type = PT_NOTE;
   376		phdr->p_offset = phdr->p_paddr = paddr_vmcoreinfo_note();
   377		phdr->p_filesz = phdr->p_memsz = VMCOREINFO_NOTE_SIZE;
   378		(ehdr->e_phnum)++;
   379		phdr++;
   380	
   381		/* Prepare PT_LOAD type program header for kernel text region */
   382		if (need_kernel_map) {
   383			phdr->p_type = PT_LOAD;
   384			phdr->p_flags = PF_R|PF_W|PF_X;
   385			phdr->p_vaddr = (unsigned long) _text;
   386			phdr->p_filesz = phdr->p_memsz = _end - _text;
   387			phdr->p_offset = phdr->p_paddr = __pa_symbol(_text);
   388			ehdr->e_phnum++;
   389			phdr++;
   390		}
   391	
   392		/* Go through all the ranges in mem->ranges[] and prepare phdr */
   393		for (i = 0; i < mem->nr_ranges; i++) {
   394			mstart = mem->ranges[i].start;
   395			mend = mem->ranges[i].end;
   396	
   397			phdr->p_type = PT_LOAD;
   398			phdr->p_flags = PF_R|PF_W|PF_X;
   399			phdr->p_offset  = mstart;
   400	
   401			phdr->p_paddr = mstart;
   402			phdr->p_vaddr = (unsigned long) __va(mstart);
   403			phdr->p_filesz = phdr->p_memsz = mend - mstart + 1;
   404			phdr->p_align = 0;
   405			ehdr->e_phnum++;
   406			pr_debug("Crash PT_LOAD ELF header. phdr=%p vaddr=0x%llx, paddr=0x%llx, sz=0x%llx e_phnum=%d p_offset=0x%llx\n",
   407				phdr, phdr->p_vaddr, phdr->p_paddr, phdr->p_filesz,
   408				ehdr->e_phnum, phdr->p_offset);
   409			phdr++;
   410		}
   411	
   412		*addr = buf;
   413		*sz = elf_sz;
   414		return 0;
   415	}
   416	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [akpm-mm:mm-nonmm-unstable 56/78] kernel/crash_core.c:331:23: error: invalid use of undefined type 'struct crash_mem'
  2023-08-11  9:10 [akpm-mm:mm-nonmm-unstable 56/78] kernel/crash_core.c:331:23: error: invalid use of undefined type 'struct crash_mem' kernel test robot
@ 2023-08-11 17:16 ` Eric DeVolder
  0 siblings, 0 replies; 2+ messages in thread
From: Eric DeVolder @ 2023-08-11 17:16 UTC (permalink / raw)
  To: kernel test robot
  Cc: oe-kbuild-all, Andrew Morton, Linux Memory Management List, Sourabh Jain

The build problem has been reproduced and solved in v27. Specifically the builds that fail are 
because only PROC_KCORE has been set, which only enables CRASH_CORE by itself. I had to move a few 
items to obtain the needed visibility.

"[PATCH v27 0/8] crash: Kernel handling of CPU and memory hot un/plug"
https://lore.kernel.org/lkml/20230811170642.6696-1-eric.devolder@oracle.com/

In addition, I've spot tested/built a few other architectures and do not encounter build errors.

Thanks!
eric

On 8/11/23 04:10, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-nonmm-unstable
> head:   0f6b2c125ce19074c1086a70119b02dd30b089a4
> commit: 3f1eba00ee9d93b66f1859d40537befe44ceb1cd [56/78] crash: move a few code bits to setup support of crash hotplug
> config: i386-randconfig-r015-20230811 (https://download.01.org/0day-ci/archive/20230811/202308111729.ApUbiGtj-lkp@intel.com/config)
> compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
> reproduce: (https://download.01.org/0day-ci/archive/20230811/202308111729.ApUbiGtj-lkp@intel.com/reproduce)
> 
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@intel.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202308111729.ApUbiGtj-lkp@intel.com/
> 
> All errors (new ones prefixed by >>):
> 
>     kernel/crash_core.c:318:40: warning: 'struct crash_mem' declared inside parameter list will not be visible outside of this definition or declaration
>       318 | int crash_prepare_elf64_headers(struct crash_mem *mem, int need_kernel_map,
>           |                                        ^~~~~~~~~
>     kernel/crash_core.c:318:5: warning: no previous prototype for 'crash_prepare_elf64_headers' [-Wmissing-prototypes]
>       318 | int crash_prepare_elf64_headers(struct crash_mem *mem, int need_kernel_map,
>           |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>     kernel/crash_core.c: In function 'crash_prepare_elf64_headers':
>>> kernel/crash_core.c:331:23: error: invalid use of undefined type 'struct crash_mem'
>       331 |         nr_phdr += mem->nr_ranges;
>           |                       ^~
>     In file included from include/vdso/const.h:5,
>                      from include/linux/const.h:4,
>                      from include/linux/align.h:5,
>                      from include/linux/kernel.h:15,
>                      from include/linux/cpumask.h:10,
>                      from include/linux/mm_types_task.h:14,
>                      from include/linux/mm_types.h:5,
>                      from include/linux/buildid.h:5,
>                      from kernel/crash_core.c:7:
>>> kernel/crash_core.c:343:32: error: 'ELF_CORE_HEADER_ALIGN' undeclared (first use in this function)
>       343 |         elf_sz = ALIGN(elf_sz, ELF_CORE_HEADER_ALIGN);
>           |                                ^~~~~~~~~~~~~~~~~~~~~
>     include/uapi/linux/const.h:32:50: note: in definition of macro '__ALIGN_KERNEL_MASK'
>        32 | #define __ALIGN_KERNEL_MASK(x, mask)    (((x) + (mask)) & ~(mask))
>           |                                                  ^~~~
>     include/linux/align.h:8:33: note: in expansion of macro '__ALIGN_KERNEL'
>         8 | #define ALIGN(x, a)             __ALIGN_KERNEL((x), (a))
>           |                                 ^~~~~~~~~~~~~~
>     kernel/crash_core.c:343:18: note: in expansion of macro 'ALIGN'
>       343 |         elf_sz = ALIGN(elf_sz, ELF_CORE_HEADER_ALIGN);
>           |                  ^~~~~
>     kernel/crash_core.c:343:32: note: each undeclared identifier is reported only once for each function it appears in
>       343 |         elf_sz = ALIGN(elf_sz, ELF_CORE_HEADER_ALIGN);
>           |                                ^~~~~~~~~~~~~~~~~~~~~
>     include/uapi/linux/const.h:32:50: note: in definition of macro '__ALIGN_KERNEL_MASK'
>        32 | #define __ALIGN_KERNEL_MASK(x, mask)    (((x) + (mask)) & ~(mask))
>           |                                                  ^~~~
>     include/linux/align.h:8:33: note: in expansion of macro '__ALIGN_KERNEL'
>         8 | #define ALIGN(x, a)             __ALIGN_KERNEL((x), (a))
>           |                                 ^~~~~~~~~~~~~~
>     kernel/crash_core.c:343:18: note: in expansion of macro 'ALIGN'
>       343 |         elf_sz = ALIGN(elf_sz, ELF_CORE_HEADER_ALIGN);
>           |                  ^~~~~
>     kernel/crash_core.c:393:28: error: invalid use of undefined type 'struct crash_mem'
>       393 |         for (i = 0; i < mem->nr_ranges; i++) {
>           |                            ^~
>     kernel/crash_core.c:394:29: error: invalid use of undefined type 'struct crash_mem'
>       394 |                 mstart = mem->ranges[i].start;
>           |                             ^~
>     kernel/crash_core.c:395:27: error: invalid use of undefined type 'struct crash_mem'
>       395 |                 mend = mem->ranges[i].end;
>           |                           ^~
>     kernel/crash_core.c: At top level:
>     kernel/crash_core.c:417:36: warning: 'struct crash_mem' declared inside parameter list will not be visible outside of this definition or declaration
>       417 | int crash_exclude_mem_range(struct crash_mem *mem,
>           |                                    ^~~~~~~~~
>     kernel/crash_core.c:417:5: warning: no previous prototype for 'crash_exclude_mem_range' [-Wmissing-prototypes]
>       417 | int crash_exclude_mem_range(struct crash_mem *mem,
>           |     ^~~~~~~~~~~~~~~~~~~~~~~
>     kernel/crash_core.c: In function 'crash_exclude_mem_range':
>     kernel/crash_core.c:424:28: error: invalid use of undefined type 'struct crash_mem'
>       424 |         for (i = 0; i < mem->nr_ranges; i++) {
>           |                            ^~
>     kernel/crash_core.c:425:28: error: invalid use of undefined type 'struct crash_mem'
>       425 |                 start = mem->ranges[i].start;
>           |                            ^~
>     kernel/crash_core.c:426:26: error: invalid use of undefined type 'struct crash_mem'
>       426 |                 end = mem->ranges[i].end;
>           |                          ^~
>     kernel/crash_core.c:441:28: error: invalid use of undefined type 'struct crash_mem'
>       441 |                         mem->ranges[i].start = 0;
>           |                            ^~
>     kernel/crash_core.c:442:28: error: invalid use of undefined type 'struct crash_mem'
>       442 |                         mem->ranges[i].end = 0;
>           |                            ^~
>     kernel/crash_core.c:443:36: error: invalid use of undefined type 'struct crash_mem'
>       443 |                         if (i < mem->nr_ranges - 1) {
>           |                                    ^~
>     kernel/crash_core.c:445:52: error: invalid use of undefined type 'struct crash_mem'
>       445 |                                 for (j = i; j < mem->nr_ranges - 1; j++) {
>           |                                                    ^~
>     kernel/crash_core.c:446:44: error: invalid use of undefined type 'struct crash_mem'
>       446 |                                         mem->ranges[j].start =
>           |                                            ^~
>     kernel/crash_core.c:447:52: error: invalid use of undefined type 'struct crash_mem'
>       447 |                                                 mem->ranges[j+1].start;
>           |                                                    ^~
>     kernel/crash_core.c:448:44: error: invalid use of undefined type 'struct crash_mem'
>       448 |                                         mem->ranges[j].end =
>           |                                            ^~
>     kernel/crash_core.c:449:60: error: invalid use of undefined type 'struct crash_mem'
>       449 |                                                         mem->ranges[j+1].end;
>           |                                                            ^~
>     kernel/crash_core.c:458:36: error: invalid use of undefined type 'struct crash_mem'
>       458 |                                 mem->nr_ranges--;
>           |                                    ^~
>     kernel/crash_core.c:461:28: error: invalid use of undefined type 'struct crash_mem'
>       461 |                         mem->nr_ranges--;
>           |                            ^~
>     kernel/crash_core.c:467:28: error: invalid use of undefined type 'struct crash_mem'
>       467 |                         mem->ranges[i].end = p_start - 1;
>           |                            ^~
>     kernel/crash_core.c:471:28: error: invalid use of undefined type 'struct crash_mem'
>       471 |                         mem->ranges[i].end = p_start - 1;
>           |                            ^~
>     kernel/crash_core.c:473:28: error: invalid use of undefined type 'struct crash_mem'
>       473 |                         mem->ranges[i].start = p_end + 1;
>           |                            ^~
>     kernel/crash_core.c:482:21: error: invalid use of undefined type 'struct crash_mem'
>       482 |         if (i == mem->max_nr_ranges - 1)
>           |                     ^~
>     kernel/crash_core.c:487:20: error: invalid use of undefined type 'struct crash_mem'
>       487 |         if (j < mem->nr_ranges) {
>           |                    ^~
>     kernel/crash_core.c:489:29: error: invalid use of undefined type 'struct crash_mem'
>       489 |                 for (i = mem->nr_ranges - 1; i >= j; i--)
>           |                             ^~
>     kernel/crash_core.c:490:28: error: invalid use of undefined type 'struct crash_mem'
>       490 |                         mem->ranges[i + 1] = mem->ranges[i];
>           |                            ^~
> 
> 
> vim +331 kernel/crash_core.c
> 
>     317	
>     318	int crash_prepare_elf64_headers(struct crash_mem *mem, int need_kernel_map,
>     319				  void **addr, unsigned long *sz)
>     320	{
>     321		Elf64_Ehdr *ehdr;
>     322		Elf64_Phdr *phdr;
>     323		unsigned long nr_cpus = num_possible_cpus(), nr_phdr, elf_sz;
>     324		unsigned char *buf;
>     325		unsigned int cpu, i;
>     326		unsigned long long notes_addr;
>     327		unsigned long mstart, mend;
>     328	
>     329		/* extra phdr for vmcoreinfo ELF note */
>     330		nr_phdr = nr_cpus + 1;
>   > 331		nr_phdr += mem->nr_ranges;
>     332	
>     333		/*
>     334		 * kexec-tools creates an extra PT_LOAD phdr for kernel text mapping
>     335		 * area (for example, ffffffff80000000 - ffffffffa0000000 on x86_64).
>     336		 * I think this is required by tools like gdb. So same physical
>     337		 * memory will be mapped in two ELF headers. One will contain kernel
>     338		 * text virtual addresses and other will have __va(physical) addresses.
>     339		 */
>     340	
>     341		nr_phdr++;
>     342		elf_sz = sizeof(Elf64_Ehdr) + nr_phdr * sizeof(Elf64_Phdr);
>   > 343		elf_sz = ALIGN(elf_sz, ELF_CORE_HEADER_ALIGN);
>     344	
>     345		buf = vzalloc(elf_sz);
>     346		if (!buf)
>     347			return -ENOMEM;
>     348	
>     349		ehdr = (Elf64_Ehdr *)buf;
>     350		phdr = (Elf64_Phdr *)(ehdr + 1);
>     351		memcpy(ehdr->e_ident, ELFMAG, SELFMAG);
>     352		ehdr->e_ident[EI_CLASS] = ELFCLASS64;
>     353		ehdr->e_ident[EI_DATA] = ELFDATA2LSB;
>     354		ehdr->e_ident[EI_VERSION] = EV_CURRENT;
>     355		ehdr->e_ident[EI_OSABI] = ELF_OSABI;
>     356		memset(ehdr->e_ident + EI_PAD, 0, EI_NIDENT - EI_PAD);
>     357		ehdr->e_type = ET_CORE;
>     358		ehdr->e_machine = ELF_ARCH;
>     359		ehdr->e_version = EV_CURRENT;
>     360		ehdr->e_phoff = sizeof(Elf64_Ehdr);
>     361		ehdr->e_ehsize = sizeof(Elf64_Ehdr);
>     362		ehdr->e_phentsize = sizeof(Elf64_Phdr);
>     363	
>     364		/* Prepare one phdr of type PT_NOTE for each present CPU */
>     365		for_each_present_cpu(cpu) {
>     366			phdr->p_type = PT_NOTE;
>     367			notes_addr = per_cpu_ptr_to_phys(per_cpu_ptr(crash_notes, cpu));
>     368			phdr->p_offset = phdr->p_paddr = notes_addr;
>     369			phdr->p_filesz = phdr->p_memsz = sizeof(note_buf_t);
>     370			(ehdr->e_phnum)++;
>     371			phdr++;
>     372		}
>     373	
>     374		/* Prepare one PT_NOTE header for vmcoreinfo */
>     375		phdr->p_type = PT_NOTE;
>     376		phdr->p_offset = phdr->p_paddr = paddr_vmcoreinfo_note();
>     377		phdr->p_filesz = phdr->p_memsz = VMCOREINFO_NOTE_SIZE;
>     378		(ehdr->e_phnum)++;
>     379		phdr++;
>     380	
>     381		/* Prepare PT_LOAD type program header for kernel text region */
>     382		if (need_kernel_map) {
>     383			phdr->p_type = PT_LOAD;
>     384			phdr->p_flags = PF_R|PF_W|PF_X;
>     385			phdr->p_vaddr = (unsigned long) _text;
>     386			phdr->p_filesz = phdr->p_memsz = _end - _text;
>     387			phdr->p_offset = phdr->p_paddr = __pa_symbol(_text);
>     388			ehdr->e_phnum++;
>     389			phdr++;
>     390		}
>     391	
>     392		/* Go through all the ranges in mem->ranges[] and prepare phdr */
>     393		for (i = 0; i < mem->nr_ranges; i++) {
>     394			mstart = mem->ranges[i].start;
>     395			mend = mem->ranges[i].end;
>     396	
>     397			phdr->p_type = PT_LOAD;
>     398			phdr->p_flags = PF_R|PF_W|PF_X;
>     399			phdr->p_offset  = mstart;
>     400	
>     401			phdr->p_paddr = mstart;
>     402			phdr->p_vaddr = (unsigned long) __va(mstart);
>     403			phdr->p_filesz = phdr->p_memsz = mend - mstart + 1;
>     404			phdr->p_align = 0;
>     405			ehdr->e_phnum++;
>     406			pr_debug("Crash PT_LOAD ELF header. phdr=%p vaddr=0x%llx, paddr=0x%llx, sz=0x%llx e_phnum=%d p_offset=0x%llx\n",
>     407				phdr, phdr->p_vaddr, phdr->p_paddr, phdr->p_filesz,
>     408				ehdr->e_phnum, phdr->p_offset);
>     409			phdr++;
>     410		}
>     411	
>     412		*addr = buf;
>     413		*sz = elf_sz;
>     414		return 0;
>     415	}
>     416	
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-08-11 17:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-08-11  9:10 [akpm-mm:mm-nonmm-unstable 56/78] kernel/crash_core.c:331:23: error: invalid use of undefined type 'struct crash_mem' kernel test robot
2023-08-11 17:16 ` Eric DeVolder

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).