[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] Re: [patch] Add kexec_ops & function pointers

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.


Gerd Hoffmann <kraxel@xxxxxxx>

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.