* Ingo Molnar <mingo@elte.hu> wrote:another problem is the redefinition of apic_read(), causing: arch/x86/xen/enlighten.c: In function ‘xen_patch': arch/x86/xen/enlighten.c:1084: warning: label ‘patch_site' defined but not used arch/x86/xen/enlighten.c: At top level: arch/x86/xen/enlighten.c:1272: error: expected identifier before ‘(' token arch/x86/xen/enlighten.c:1273: error: expected ‘}' before ‘.' token with this config: http://redhat.com/~mingo/misc/config-Fri_Jul_11_21_51_18_CEST_2008.bad the continued spaghetti in all the APIC variants is quite ugly. This should all be handled via a single apic_ops template that should cover the paravirt and native variants as well. Ingo --
