Re: [PATCH v2] x86, nmi_watchdog: Remove ARCH_HAS_NMI_WATCHDOG andrely on CONFIG_HARDLOCKUP_DETECTOR

From: Don Zickus
Date: Wed Dec 22 2010 - 14:01:51 EST


On Wed, Dec 22, 2010 at 05:24:52PM +0100, Ingo Molnar wrote:
>
> * Don Zickus <dzickus@xxxxxxxxxx> wrote:
>
> > Tested with
> > ARCH=i386: allnoconfig, defconfig, allyesconfig, (various broken configs)
> > ARCH=x86_64: allnoconfig, defconfig, allyesconfig, (various broken configs)
> >
> > Hopefully, after this patch I won't get any more compile broken emails. :-)
> >
> > Signed-off-by: Don Zickus <dzickus@xxxxxxxxxx>
>
> There's still some assumptions in the oprofile code, which breaks the build with
> your patch applied:
>
> arch/x86/oprofile/op_model_p4.c:394:4: error: implicit declaration of function ârelease_perfctr_nmiâ
> arch/x86/oprofile/op_model_p4.c:403:4: error: implicit declaration of function ârelease_evntsel_nmiâ
> arch/x86/oprofile/op_model_p4.c:419:3: error: implicit declaration of function âreserve_perfctr_nmiâ
> arch/x86/oprofile/op_model_p4.c:428:3: error: implicit declaration of function âreserve_evntsel_nmiâ
>
> Config attached.

Thanks. I sent a patch to fix this. Basically they were using
linux/nmi.h instead of asm/nmi.h.

Cheers,
Don
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/