lkml.org 
[lkml]   [2011]   [May]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 4/4] watchdog: configure nmi watchdog period based on watchdog_thresh

    * Mandeep Singh Baines <msb@chromium.org> wrote:

    > Before the conversion of the NMI watchdog to perf event, the watchdog
    > timeout was 5 seconds. Now it is 60 seconds. For my particular application,
    > netbooks, 5 seconds was a better timeout. With a short timeout, we
    > catch faults earlier and are able to send back a panic. With a 60 second
    > timeout, the user is unlikely to wait and will instead hit the power
    > button, causing us to lose the panic info.

    That's an interesting observation. Have you been able to measure/observe this
    effect somehow, or do you presume that users find 60 seconds too long?

    This would be a concern for upstream as well i guess.

    > This change configures the NMI period based on the watchdog_thresh.

    Hm, our tolerance for the two thresholds is not just human but technical: hard
    lockup warnings should indeed be triggered after just a few seconds, soft
    lockups can have false positives under extreme conditions.

    So we generally want a higher threshold for soft lockups than for hard lockups.

    So how about we couple the thresholds with a factor: we make the soft threshold
    twice the amount of time the hard threshold is? Then we could change the
    upstream default as well i think: lets change the NMI timeout to 10 seconds
    (and thus have the soft threshold at 20 seconds). Is 20 seconds short enough
    for most users to not hit reset?

    We might want to change another aspect of the NMI watchdog: right now it tries
    to abort the offending task - which is really nasty if there was a spuriously
    long irqs-off section somewhere in the kernel. How about we just print a
    warning instead?

    Thanks,

    Ingo


    \
     
     \ /
      Last update: 2011-05-17 09:19    [W:4.078 / U:1.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site