Bernhard Walle wrote:
> * Bernhard Walle <bwalle@xxxxxxx> [2007-03-01 13:57]:
>> * Horms <horms@xxxxxxxxxxxx> [2007-03-01 05:22]:
>>> On Wed, Feb 28, 2007 at 01:45:17PM -0600, John Keller wrote:
>>>> Add a missing #define for the platform_kernel_launch_event.
>>>> Without this fix, a call to platform_kernel_launch_event()
>>>> becomes a noop on generic kernels. SN systems require this
>>>> fix to successfully kdump/kexec from certain hardware errors.
>>>>
>>>> Signed-off-by: John Keller <jpk@xxxxxxx>
>>> I made a similar change when porting to xen, but I hadn't thought
>>> to see if mainline linux needs it to.
>>>
>>> Acked-by: Simon Horman <horms@xxxxxxxxxxxx>
>> I think there's an additional change needed. Without that, it leads to
>> a NULL pointer dereference.
Yep, it is needed. The missing line was in Jack's original patch
but was lost somehow in 2.6.20.
Acked-by: Jay Lan <jlan@xxxxxxx>
>
> Maybe I should also get my editor right ... ;):
>
> ---
> include/asm-ia64/machvec.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> Index: b/include/asm-ia64/machvec.h
> ===================================================================
> --- a/include/asm-ia64/machvec.h
> +++ b/include/asm-ia64/machvec.h
> @@ -168,6 +168,7 @@ extern void machvec_tlb_migrate_finish (
> # define platform_setup_msi_irq ia64_mv.setup_msi_irq
> # define platform_teardown_msi_irq ia64_mv.teardown_msi_irq
> # define platform_pci_fixup_bus ia64_mv.pci_fixup_bus
> +# define platform_kernel_launch_event ia64_mv.kernel_launch_event
> # endif
>
> /* __attribute__((__aligned__(16))) is required to make size of the
> @@ -269,6 +270,7 @@ struct ia64_machine_vector {
> platform_setup_msi_irq, \
> platform_teardown_msi_irq, \
> platform_pci_fixup_bus, \
> + platform_kernel_launch_event \
> }
>
> extern struct ia64_machine_vector ia64_mv;
> -
> To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|