[lkml]   [2008]   [May]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [patch 2.6.26-rc4-git] PM: boot time suspend selftest

    * Andrew Morton <> wrote:

    > > + WARN_ON_ONCE(msec > (TEST_SUSPEND_SECONDS * 1000));
    > We should have a comment here explaining what we're warning about.
    > Why would it take more that five seconds?

    i asked for that because we had regressions in the past in the form of
    "it takes one minute to resume".

    > Better might be to just add a nice printk - I don't think we need the
    > stack trace here.

    please keep the warn-on so that it can be detected automatically. Adding
    yet another printk just complicates the automated answer to the "is this
    kernel that just booted up fine or not" question.

    In fact i'd love to have the analogue to /proc/lockdep_debug's
    "debug_locks: 0" output. I.e. the kernel should know it via one central
    flag whether any bugs that need human review have been detected so far.

    Say /proc/sys/kernel/kernel_is_buggy. This value could even be
    multi-level: a WARN_ON() increases it by +1, a kernel crash increases it
    by +1000. ( That way i could run overnight tests that will only stop on
    a kernel_is_buggy >= 1000 condition, while it could ignore simpler
    WARN_ON()s. )


     \ /
      Last update: 2008-05-30 13:03    [W:0.023 / U:1.472 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site