Re: [PATCH] lockup_detector: Make DETECT_HUNT_TASK default dependon LOCKUP_DETECTOR

From: Heiko Carstens
Date: Mon Aug 09 2010 - 04:06:04 EST


On Sun, Aug 08, 2010 at 09:58:42PM +0200, Frederic Weisbecker wrote:
> > Maybe a better change would be to make it more generally available - right now
> > it's:
> >
> > config LOCKUP_DETECTOR
> > bool "Detect Hard and Soft Lockups"
> > depends on DEBUG_KERNEL && !S390
> >
> > which means that it cannot be enabled when DEBUG_KERNEL is off.
> >
> > So i think we should:
> >
> > - Remove the s390 hack and add an ARCH_HAS_LOCKUP_DETECTOR flag
>
>
>
> If we do this, we'll need to add this config on every archs but s390.
> We should better have ARCH_WANT_NO_LOCKUP_DETECTOR. I know that
> "negative" meaning configs suck, but otherwise we would lose this
> support on many archs.
>
> Why s390 doesn't want the softlockup detector to begin with?

If I remember correctly then we disabled that back then because we got
false positives. The reason for those were that the softlockup detector
did not take steal time into account.
E.g. if a guest cpu runs for 10 seconds, but the hypervisor would steal
9 seconds in order to run other guest cpus this specific cpu would still
think it ran for 10 seconds and therefore would generate invalid warnings.
--
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/