Messages in this thread | | | Date | Fri, 17 Feb 2006 21:11:23 +0100 | From | Ingo Molnar <> | Subject | Re: [patch] timer-irq-driven soft-watchdog, cleanups |
| |
* Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> wrote:
> Sorry but I have enough more high priority issues to take care of and > I'm not going to spend any more time on soft lockups even if they are > really problems in IDE subsystem. If this is not fixed before 2.6.16 > I'm submitting patch to Linus making DETECT_SOFTLOCKUP depend on > "CONFIG_IDE=n"... at least users will be able to use their systems > instead of seeing lockups.
i have lots of IDE based systems (they dont use PIO though) and i'm not seeing these. I'll oppose such a patch if it's to hide genuine issues - the 10 seconds tolerance is already generous i think. I'll of course fix any false positives which are the fault of the softlockup-watchdog, but from your mails it appears to me that the IDE warnings are indeed genuine.
If the source of the delay is hard to fix you can temporarily work it around in the code by putting in the touch_softlockup_watchdog() lines - that will also document the places that cause long delays - which is a good thing.
It is entirely feasible to put a touch_softlockup_watchdog() call into every PIO OP - even a single-byte PIO related IN/OUT instruction takes a couple of microseconds, so a touch_softlockup_watchdog() wont even show up on the radar.
> DETECT_SOFTLOCKUP should be an aim in development not a method of > forcing driver maintainers to work on specific issues...
well, 10+ seconds delays on a running system are not really acceptable, and can cause other problems. The softlockup-watchdog is optional and can be easily turned off in the .config.
Ingo - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |