| |
|
 |
|
 |
|
|
|
| |
|
|
xen-devel
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.
cheers,
Gerd
--
Gerd Hoffmann <kraxel@xxxxxxx>
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
| |
|
Home