Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Linux: Kernel

[patch 0/3] [x86] Fix crashkernel reservation on NUMA machines

 

 

Linux kernel RSS feed   Index | Next | Previous | View Threaded


bwalle at suse

Jun 8, 2008, 6:46 AM

Post #1 of 3 (273 views)
Permalink
[patch 0/3] [x86] Fix crashkernel reservation on NUMA machines

This patch series fixes the crashkernel reservation on NUMA machine. The
regression was discovered by Dave Anderson <anderson [at] redhat>.

The background is that on NUMA machines, reserve_bootmem_generic() is required
instead of reserve_bootmem(). To achieve that, it's necessary to make a few
API changes.

The patches are against latest linux-2.6 git. They should still go into 2.6.26
since it's only bug fixing. For 2.6.27, we should unify crashkernel reservation
for i386 and x86-64.

Tested on both i386 and x86-64. Compilation was tested with both kexec disabled
and enabled. The change is x86 only, so no need to test on other architectures.


Signed-off-by: Bernhard Walle <bwalle [at] suse>

--
Bernhard Walle, SUSE LINUX Products GmbH, Architecture Maintenance
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


vgoyal at redhat

Jun 9, 2008, 6:06 AM

Post #2 of 3 (240 views)
Permalink
Re: [patch 0/3] [x86] Fix crashkernel reservation on NUMA machines [In reply to]

On Sun, Jun 08, 2008 at 03:46:28PM +0200, Bernhard Walle wrote:
> This patch series fixes the crashkernel reservation on NUMA machine. The
> regression was discovered by Dave Anderson <anderson [at] redhat>.
>
> The background is that on NUMA machines, reserve_bootmem_generic() is required
> instead of reserve_bootmem(). To achieve that, it's necessary to make a few
> API changes.
>
> The patches are against latest linux-2.6 git. They should still go into 2.6.26
> since it's only bug fixing. For 2.6.27, we should unify crashkernel reservation
> for i386 and x86-64.
>
> Tested on both i386 and x86-64. Compilation was tested with both kexec disabled
> and enabled. The change is x86 only, so no need to test on other architectures.
>
>
> Signed-off-by: Bernhard Walle <bwalle [at] suse>

Yes we should be using reserve_bootmem_generic() for reserving memory
on NUMA machines. Recently myself and Dave A. ran into crash while
reserving memory using resreve_bootmem() on a NUMA machine.

Reason for crash? reserve_bootmem() always assumes node id to be zero, and
that was not the case.

Acked-by: Vivek Goyal <vgoyal [at] redhat>

Thanks
Vivek
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


mingo at elte

Jun 10, 2008, 5:44 AM

Post #3 of 3 (249 views)
Permalink
Re: [patch 0/3] [x86] Fix crashkernel reservation on NUMA machines [In reply to]

* Bernhard Walle <bwalle [at] suse> wrote:

> This patch series fixes the crashkernel reservation on NUMA machine.
> The regression was discovered by Dave Anderson <anderson [at] redhat>.
>
> The background is that on NUMA machines, reserve_bootmem_generic() is
> required instead of reserve_bootmem(). To achieve that, it's necessary
> to make a few API changes.
>
> The patches are against latest linux-2.6 git. They should still go
> into 2.6.26 since it's only bug fixing. For 2.6.27, we should unify
> crashkernel reservation for i386 and x86-64.
>
> Tested on both i386 and x86-64. Compilation was tested with both kexec
> disabled and enabled. The change is x86 only, so no need to test on
> other architectures.

applied to tip/x86/numa - thanks Bernhard.

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo [at] vger
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Linux kernel RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.