Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1421190

Re: [PATCH 2/8] kexec_file: Generalize kexec_add_buffer.

Show all headers | View raw

Hello Dave,

Thanks for the quick review and for your comments.

I'll separate the change to add arch_walk_system_ram and the change to add 
kexec_locate_mem_hole into different patches, and add error handling for 
KEXEC_ON_CRASH.

Am Montag, 13 Juni 2016, 15:29:39 schrieb Dave Young:
> On 06/12/16 at 12:10am, Thiago Jung Bauermann wrote:
> > Allow architectures to specify different memory walking functions for
> > kexec_add_buffer. Intel uses iomem to track reserved memory ranges,
> > but PowerPC uses the memblock subsystem.
> 
> Can the crashk_res be inserted to iomem_resource so that only one
> weak function for system ram is needed?

Sorry, it's not clear to me what you mean by inserting crashk_res into 
iomem_resource, but I can add a bool for_crashkernel to arch_walk_system_ram 
so that it can decide which kind of memory to traverse, so the default 
implementation of kexec_file.c would be:

int __weak arch_walk_system_ram(bool for_crashkernel, unsigned long start,
				unsigned long end, bool top_down,
				void *data,
				int (*func)(u64, u64, void *))
{
	int ret;

	if (for_crashkernel)
		ret = walk_iomem_res_desc(crashk_res.desc,
					  IORESOURCE_SYSTEM_RAM |
					  IORESOURCE_BUSY,
					  start, end, data, func);
	else
		ret = walk_system_ram_res(start, end, data, func);

	if (ret != 1) {
		/* A suitable memory range could not be found for buffer */
		return -EADDRNOTAVAIL;
	}
}

and kexec_add_buffer / kexec_locate_mem_hole would call it with:

	if (image->type == KEXEC_TYPE_CRASH)
		ret = arch_walk_system_ram(true, crashk_res.start,
					   crashk_res.end, top_down, &buf,
					   locate_mem_hole_callback);
	else
		ret = arch_walk_system_ram(false, 0, -1, top_down, &buf,
					   locate_mem_hole_callback);

What do you think?

-- 
[]'s
Thiago Jung Bauermann
IBM Linux Technology Center

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/8] kexec_file_load implementation for PowerPC Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> - 2016-06-12 05:20 +0200
  [PATCH 4/8] powerpc: Generalize elf64_apply_relocate_add. Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> - 2016-06-12 05:20 +0200
  [PATCH 2/8] kexec_file: Generalize kexec_add_buffer. Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> - 2016-06-12 05:20 +0200
    Re: [PATCH 2/8] kexec_file: Generalize kexec_add_buffer. Dave Young <dyoung@redhat.com> - 2016-06-13 09:20 +0200
    Re: [PATCH 2/8] kexec_file: Generalize kexec_add_buffer. Dave Young <dyoung@redhat.com> - 2016-06-13 09:30 +0200
      Re: [PATCH 2/8] kexec_file: Generalize kexec_add_buffer. Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> - 2016-06-13 21:10 +0200
        Re: [PATCH 2/8] kexec_file: Generalize kexec_add_buffer. Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> - 2016-06-13 22:50 +0200
        Re: [PATCH 2/8] kexec_file: Generalize kexec_add_buffer. Dave Young <dyoung@redhat.com> - 2016-06-14 03:30 +0200
  [PATCH 1/8] kexec_file: Remove unused members from struct kexec_buf. Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> - 2016-06-12 05:20 +0200
    Re: [PATCH 1/8] kexec_file: Remove unused members from struct  kexec_buf. Dave Young <dyoung@redhat.com> - 2016-06-13 09:10 +0200
  [PATCH 7/8] powerpc: Add support for loading ELF kernels with kexec_file_load. Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> - 2016-06-12 05:20 +0200
  [PATCH 8/8] powerpc: Add purgatory for kexec_file_load implementation. Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> - 2016-06-12 05:20 +0200
  [PATCH 3/8] powerpc: Factor out relocation code from module_64.c to elf_util_64.c. Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> - 2016-06-12 05:20 +0200
  [PATCH 5/8] powerpc: Add functions to read ELF files of any endianness. Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> - 2016-06-12 05:20 +0200
  [PATCH 6/8] powerpc: Implement kexec_file_load. Thiago Jung Bauermann <bauerman@linux.vnet.ibm.com> - 2016-06-12 05:20 +0200
  Re: [PATCH 0/8] kexec_file_load implementation for PowerPC Dave Young <dyoung@redhat.com> - 2016-06-13 09:10 +0200

csiph-web