Re: [PATCH] ehea: add kdump support

2008-02-20 Thread Jeff Garzik
Thomas Klein wrote: This patch adds kdump support to the ehea driver. As the firmware doesn't free resource handles automatically, the driver has to run an as simple as possible free resource function in case of a crash shutdown. The function iterates over two arrays freeing all resource handles

Re: [PATCH] ehea: add kdump support

2008-02-20 Thread Jeff Garzik
Thomas Klein wrote: This patch adds kdump support to the ehea driver. As the firmware doesn't free resource handles automatically, the driver has to run an as simple as possible free resource function in case of a crash shutdown. The function iterates over two arrays freeing all resource handles

[PATCH] ehea: add kdump support

2008-02-13 Thread Thomas Klein
This patch adds kdump support to the ehea driver. As the firmware doesn't free resource handles automatically, the driver has to run an as simple as possible free resource function in case of a crash shutdown. The function iterates over two arrays freeing all resource handles which are stored

[PATCH] ehea: add kdump support

2008-02-13 Thread Thomas Klein
This patch adds kdump support to the ehea driver. As the firmware doesn't free resource handles automatically, the driver has to run an as simple as possible free resource function in case of a crash shutdown. The function iterates over two arrays freeing all resource handles which are stored

Re: [PATCH] ehea: Add kdump support

2007-11-26 Thread Michael Neuling
In message <[EMAIL PROTECTED]> you wrote: > Michael Ellerman wrote on 26.11.2007 09:16:28: > > Solutions that might be better: > > > > a) if there are a finite number of handles and we can predict their > > values, just delete them all in the kdump kernel before the driver > > loads. > >

Re: [PATCH] ehea: Add kdump support

2007-11-26 Thread Linas Vepstas
Hi, On Mon, Nov 26, 2007 at 01:41:37PM -0200, Luke Browning wrote: > On Mon, 2007-11-26 at 19:16 +1100, Michael Ellerman wrote: > > > For kdump we have to assume that the kernel is fundamentally broken, If I may so humbly suggest: since ehea is a power6 thing only, we should refocus our

Re: [PATCH] ehea: Add kdump support

2007-11-26 Thread Luke Browning
On Mon, 2007-11-26 at 19:16 +1100, Michael Ellerman wrote: > > Hi Thomas, > > I'm sorry, but this patch is all wrong IMHO. > > For kdump we have to assume that the kernel is fundamentally broken, > we've panicked, so something bad has happened - every line of kernel > code that is run

Re: [PATCH] ehea: Add kdump support

2007-11-26 Thread Christoph Raisch
Michael Ellerman wrote on 26.11.2007 09:16:28: > Solutions that might be better: > > a) if there are a finite number of handles and we can predict their > values, just delete them all in the kdump kernel before the driver > loads. Guessing the values does not work, because of the handle

Re: [PATCH] ehea: Add kdump support

2007-11-26 Thread Michael Ellerman
On Fri, 2007-11-09 at 14:33 +0100, Thomas Klein wrote: > To support ehea driver reloading in a kdump kernel the driver has to perform > firmware handle deregistrations when the original kernel crashes. As there's > currently no notifier chain for machine crashes this patch enables kdump > support

Re: [PATCH] ehea: Add kdump support

2007-11-26 Thread Michael Ellerman
On Fri, 2007-11-09 at 14:33 +0100, Thomas Klein wrote: To support ehea driver reloading in a kdump kernel the driver has to perform firmware handle deregistrations when the original kernel crashes. As there's currently no notifier chain for machine crashes this patch enables kdump support in

Re: [PATCH] ehea: Add kdump support

2007-11-26 Thread Christoph Raisch
Michael Ellerman wrote on 26.11.2007 09:16:28: Solutions that might be better: a) if there are a finite number of handles and we can predict their values, just delete them all in the kdump kernel before the driver loads. Guessing the values does not work, because of the handle

Re: [PATCH] ehea: Add kdump support

2007-11-26 Thread Luke Browning
On Mon, 2007-11-26 at 19:16 +1100, Michael Ellerman wrote: Hi Thomas, I'm sorry, but this patch is all wrong IMHO. For kdump we have to assume that the kernel is fundamentally broken, we've panicked, so something bad has happened - every line of kernel code that is run decreases the

Re: [PATCH] ehea: Add kdump support

2007-11-26 Thread Linas Vepstas
Hi, On Mon, Nov 26, 2007 at 01:41:37PM -0200, Luke Browning wrote: On Mon, 2007-11-26 at 19:16 +1100, Michael Ellerman wrote: For kdump we have to assume that the kernel is fundamentally broken, If I may so humbly suggest: since ehea is a power6 thing only, we should refocus our energies

Re: [PATCH] ehea: Add kdump support

2007-11-26 Thread Michael Neuling
In message [EMAIL PROTECTED] you wrote: Michael Ellerman wrote on 26.11.2007 09:16:28: Solutions that might be better: a) if there are a finite number of handles and we can predict their values, just delete them all in the kdump kernel before the driver loads. Guessing the

Re: [PATCH] ehea: Add kdump support

2007-11-09 Thread Michael Neuling
> To support ehea driver reloading in a kdump kernel the driver has to > perform firmware handle deregistrations when the original kernel > crashes. As there's currently no notifier chain for machine crashes > this patch enables kdump support in the ehea driver by bending the >

[PATCH] ehea: Add kdump support

2007-11-09 Thread Thomas Klein
To support ehea driver reloading in a kdump kernel the driver has to perform firmware handle deregistrations when the original kernel crashes. As there's currently no notifier chain for machine crashes this patch enables kdump support in the ehea driver by bending the ppc_md.machine_crash_shutdown

[PATCH] ehea: Add kdump support

2007-11-09 Thread Thomas Klein
To support ehea driver reloading in a kdump kernel the driver has to perform firmware handle deregistrations when the original kernel crashes. As there's currently no notifier chain for machine crashes this patch enables kdump support in the ehea driver by bending the ppc_md.machine_crash_shutdown

Re: [PATCH] ehea: Add kdump support

2007-11-09 Thread Michael Neuling
To support ehea driver reloading in a kdump kernel the driver has to perform firmware handle deregistrations when the original kernel crashes. As there's currently no notifier chain for machine crashes this patch enables kdump support in the ehea driver by bending the