Re: [PATCH] lockup_detector: Make DETECT_HUNT_TASK default depend onLOCKUP_DETECTOR

[Don Zickus]

On Mon, Aug 09, 2010 at 10:07:40AM +0200, Heiko Carstens wrote:
> 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.

I have learned recently that is applies to all virtual machines including
KVM, Xen and VMWare(?).  However, you only see this when you overload the
hypervisor with lots of guests.  Which is why you normally don't see this
on those types of guests.

But any time based detection debug features (softlockup, hardlockup,
hung_task) could potentially run into this.

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/