[lkml]   [2011]   [Aug]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH v6 2/2] Output stall data in debugfs
    On Thu, Aug 11, 2011 at 11:48:26AM -0700, Andi Kleen wrote:
    > Alex Neronskiy <> writes:
    > > From: Alex Neronskiy <>
    > >
    > > Instead of using the log, use debugfs for output of both stall
    > > lengths and stack traces. Printing to the log can result in
    > > watchdog touches,
    > Why? Because of printk being slow or something else?

    No because the serial console driver does a touch_nmi_watchdog(). So if
    we are trying to output debug info _before_ the lockup detector goes off,
    we effectively shoot ourselves in the foot by reseting the lockup detector
    everytime we print something. Hence we are trying to capture the data and
    output using another interface.

    > The first could be probably workarounded, especially if you
    > already have "two buffers"
    > > distorting the very events being measured.
    > > Additionally, the information will not distract from lockups
    > > when users view the log.
    > >
    > > A two-buffer system is used to ensure that the trace information
    > > can always be recorded without contention.
    > This implies that kernel bug reports will often not contain the
    > back trace, right? Seems like a bad thing to me because it will
    > make bug reports worse.

    These are debug traces. The real lockup traces will still print to the
    console as they do today. Nothing will change from that perspective.

    What these patches do is give you insight into what part of your system is
    coming close but not causing lockups. If we have the hardlockup detector
    set to warn or panic after 5 seconds of no interrupts, then these patches
    can give you backtraces after 3 or 4 seconds (these traces might enable
    interrupts after 4 seconds so no lock up occurs, but maybe something worth
    noting). It's just way to gather hueristics on system behaviour
    regarding lockups.


    > -Andi
    > --
    > -- Speaking for myself only

     \ /
      Last update: 2011-08-11 21:07    [W:0.023 / U:0.412 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site