[lkml]   [2008]   [Jun]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] softlockup: show irqtrace
    On 6/25/08, Ingo Molnar <> wrote:
    > > (Compile tested on allyesconfig, allnoconfig, allnoconfig + softlockup
    > > debugging.)
    > hm, that must have taken quite some time to do on your desktop, right?

    Hm, no. I admit to having improved the process of compilation
    somewhat; I compiled only the affected files, or in case a CONFIG
    prevents a file from being built, the complete parent directory. I
    find that this works well in practice; the compiler usually warns
    about implicit declarations (which would otherwise show up as linker
    errors), so there is no need to link everything. (Sadly, this is not
    true when declarations are provided regardless of definition -- I
    think this is actually a case against putting declarations outside

    > FYI, there's no mandatory need to go through that buerocratic testing
    > hassle with patches submitted to -tip - especially with small patches
    > that look obvious and have a high chance of being right. We do non-stop
    > allyes/allno/allmod+randconfig 64-bit/32-bit build and boot testing for
    > every single patch added to the repository.
    > Building, booting and testing it in the environment where you expect it
    > to be used primarily should be more than enough.

    Well, it's still somewhat emberassing to submit patches that don't compile :-)

    This patch was also a typical error scenario; I introduce the call of
    a function which requires the inclusion of a new header, one which
    seemingly depends on a particular config option (lockdep). So I simply
    wanted to be sure myself -- and then I might as well put it in the
    e-mail. That also implicitly means that I did not boot test it at all.
    (Because, yeah, what it actually DOES should be fairly obvious.)

    Thanks, though! It's really nice to know the world's not coming to an
    end because of a broken patch ;-)


    "The animistic metaphor of the bug that maliciously sneaked in while
    the programmer was not looking is intellectually dishonest as it
    disguises that the error is the programmer's own creation."
    -- E. W. Dijkstra, EWD1036

     \ /
      Last update: 2008-06-25 19:13    [W:0.031 / U:8.764 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site