Re: Re: [patch] Add kexec_ops & function pointers

[Gerd Hoffmann <kraxel@suse.de>]

Ian Campbell wrote:
> On Tue, 2006-12-12 at 16:12 +0900, Magnus Damm wrote:
>> Basically, machine_kexec() should be converted to kexec_ops.kexec() in
>> two places. The second place is unfortunately located in
>> linux/kernel/sys.c which does not exist in sparse yet. So that needs
>> to be resolved somehow.
> 
> Hiding the indirections through the function table in the header via
> defines or inline functions would make things a lot cleaner in my
> opinion and as a bonus avoid this addition to the sparse tree.

No.  As machine_kexec() continues to exist (and is the default for
kexec_ops.kexec) you can't just turn that into a macro.  You have to
either fix the two callers (as done by the patch) or rename the function
to something different in arch/*/kernel/machine_kexec.c in order to be
able to reuse the name for the macro.

cheers,
  Gerd

-- 
Gerd Hoffmann <kraxel@suse.de>