On Sat, 2011-08-06 at 11:22 -0700, Jeremy Fitzhardinge wrote: > On 08/05/2011 06:58 PM, Konrad Rzeszutek Wilk wrote: > > On Fri, Aug 05, 2011 at 11:14:23PM +0200, Ingo Molnar wrote: > >> * Ingo Molnar <mingo@xxxxxxx> wrote: > >> > >>> * Randy Dunlap <rdunlap@xxxxxxxxxxxx> wrote: > >>> > >>>> On 08/04/11 15:40, Konrad Rzeszutek Wilk wrote: > >>>>> On Thu, Aug 04, 2011 at 06:32:59PM -0400, Konrad Rzeszutek Wilk wrote: > >>>>>>>>> These build failures are still triggering upstream: > >>>>>>>>> > >>>>>>>>> arch/x86/xen/trace.c:44:2: error: array index in initializer not of integer type > >>>>>>>>> arch/x86/xen/trace.c:44:2: error: (near initialization for ‘xen_hypercall_names’) > >>>>>>>>> arch/x86/xen/trace.c:45:1: error: ‘__HYPERVISOR_arch_4’ undeclared here (not in a function) > >>>>>>>>> arch/x86/xen/trace.c:45:2: error: array index in initializer not of integer type > >>>>>>>>> arch/x86/xen/trace.c:45:2: error: (near initialization for ‘xen_hypercall_names’) > >>>>>>>> Oh, that I haven't seen. Can you send me the .config for that please. > >>>>>>> You can't be trying very hard then. I see lots of these (but no, > >>>>>> Ah, I am getting it now. Thanks for reporting it. > >>>>> This should do the trick: > >>>> Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxx> > >>> That patch did the trick here too: > >>> > >>> Acked-by: Ingo Molnar <mingo@xxxxxxx> > >> Except that i'm still seeing the occasional build failure - see the > >> error log below. Config attached. > > Much appreciate for the report. I believe this fix (which I think hit > > linux-next yesterday?) should do that: > > > > commit 1e9ea2656b656edd3c8de98675bbc0340211b5bd > > Author: Jeremy Fitzhardinge <jeremy@xxxxxxxx> > > Date: Wed Aug 3 09:43:44 2011 -0700 > > > > xen/tracing: it looks like we wanted CONFIG_FTRACE > > > > Apparently we wanted CONFIG_FTRACE rather the CONFIG_FUNCTION_TRACER. > > > > Reported-by: Sander Eikelenboom <linux@xxxxxxxxxxxxxx> > > Tested-by: Sander Eikelenboom <linux@xxxxxxxxxxxxxx> > > Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@xxxxxxxxxx> > > Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> > > > > diff --git a/arch/x86/xen/Makefile b/arch/x86/xen/Makefile > > index 45e94ac..3326204 100644 > > --- a/arch/x86/xen/Makefile > > +++ b/arch/x86/xen/Makefile > > @@ -15,7 +15,7 @@ obj-y := enlighten.o setup.o multicalls.o mmu.o irq.o \ > > grant-table.o suspend.o platform-pci-unplug.o \ > > p2m.o > > > > -obj-$(CONFIG_FUNCTION_TRACER) += trace.o > > +obj-$(CONFIG_FTRACE) += trace.o > > > > I'm not sure this is correct either. Maybe it should be > CONFIG_TRACEPOINTS? Steven? Actually, I believe the correct answer is: CONFIG_EVENT_TRACING -- Steve -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
