Re: 2.6.17-rc1-mm1: KEXEC became SMP-only

[Zachary Amsden <zach@vmware.com>]

Andrew Morton wrote:
>
>>  struct subarch_hooks subarch_hook_vector = {
>>       .machine_power_off = machine_power_off,
>>       .machine_halt = machine_halt,
>>       .machine_irq_setup = machine_irq_setup,
>>       .machine_subarch_setup = machine_subarch_probe
>>       ...
>>  };
>>
>>  And machine_subarch_probe can dynamically change this vector if it 
>>  confirms that the platform is appropriate?
>>     
>
> I don't recall anyone expressing any desire for the ability to set these
> things at runtime.  Unless there is such a requirement I'd suggest that the
> best way to address Eric's point is to simply rename the relevant functions
> from foo() to subarch_foo().
>   

Avoiding the runtime assignment isn't possible if you want a generic 
subarch that truly can run on multiple different platforms.

I prefer runtime assignment not for this reason, but simply because it 
also eliminates two artifacts:

1) You can add new subarch hooks without breaking every other 
sub-architecture
2) You don't need #ifdef SUBARCH_FUNC_FOO goo to do this (renaming 
voyager_halt -> default)

Zach