Messages in this thread | | | Date | Thu, 10 Jul 2008 13:22:46 -0700 | From | Suresh Siddha <> | Subject | Re: [patch 00/26] x64, x2apic/intr-remap: Interrupt-remapping and x2apic support |
| |
On Thu, Jul 10, 2008 at 12:53:20PM -0700, Ingo Molnar wrote: > quite some stuff! > > For review and testing purposes i've created a new topic branch for > this: tip/x86/x2apic and have picked up your patches into it. > > I've pushed it out, but it's not merged into tip/master yet (obviously, > you sent this just a few minutes ago :) > > It integrates fine with tip/master. If you do this: > > git-checkout tip/master > git-merge tip/x86/x2apic > > you'll get a clean merge. > > Btw., i threw it at the -tip test-cluster and got back a quick build > bugreport: > > 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 > arch/x86/kernel/paravirt.c:376:2: error: invalid preprocessing directive > #ifndedarch/x86/kernel/paravirt.c:384:2: error: #endif without #if > > with this config: > > http://redhat.com/~mingo/misc/config-Thu_Jul_10_21_43_28_CEST_2008.bad
Yes. It is conflicting with Jermey's recent changes. I will post the fixes soon, meanwhile everything should be ok with !CONFIG_PARAVIRT
thanks, suresh
| |