Re: [PATCH V2] sparc64: mem boot option correction

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Bob Picco <bpicco@xxxxxxxxxx>
Date: Tue, 16 Sep 2014 09:29:54 -0400

> From: bob picco <bpicco@xxxxxxxxxx>
> 
> The "mem" boot option can result in many unexpected consequences. This patch
> attempts to prevent boot hangs which have been experienced on T4-4 and T5-8.
> Basically the boot loader allocates vmlinuz and initrd higher in available
> OBP physical memory. For example, on a 2Tb T5-8 it isn't possible to boot
> with mem=20G.
> 
> The patch utilizes memblock to avoid reserved regions and trim memory which
> is only free. Other improvements are possible for a multi-node machine.
 ...
> The patch was tested on T4-1, T5-8 and Jalap?no.
> 
> Cc: sparclinux@xxxxxxxxxxxxxxx
> Signed-off-by: Bob Picco <bob.picco@xxxxxxxxxx>
> ---
> V2: correct programming style syntax.

I applied this with one small stylistic change:

> +	for_each_free_mem_range(i, any_nid, &pa_start, &pa_end, &nid)

	for_each_free_mem_range(i, NUMA_NO_NODE, &pa_start, &pa_end, NULL)

> +	for_each_free_mem_range(i, any_nid, &pa_start, &pa_end, &nid) {

	for_each_free_mem_range(i, NUMA_NO_NODE, &pa_start, &pa_end, NULL) {

And I killed the now-unused local variables.

Thanks Bob.
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Development]     [DCCP]     [Linux ARM Development]     [Linux]     [Photo]     [Yosemite Help]     [Linux ARM Kernel]     [Linux SCSI]     [Linux x86_64]     [Linux Hams]

  Powered by Linux