[lkml]   [2008]   [Aug]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [ANNOUNCE] mdb: Merkey's Linux Kernel Debugger 2.6.27-rc4 released
    On Thursday 21 August 2008 22:26, wrote:

    > I used the smp_wmb() functions. I noted a couple of things. a) some of
    > these macros just emit __asm__ __volatile__ into the code so why not just
    > say "volatile" to begin with

    It is not the same as volatile type. What it does is tell the compiler
    to clobber all registers or temporaries. This something pretty well
    defined and hard to get wrong compared to volatile type.

    > b) smp_wmb() in some cases worked and in
    > other cases jut optimized away the global reference.

    Linux barriers aren't going to force a load to be emitted, if it can be
    optimized away. If it optimized away a store, then I'd like to see a
    test case.

    > c) I can go back and
    > break the code again by inserting them and building broken assembler d) I
    > ave been doing hardware and software design since the early 1980;s, I
    > invented SMP affinity scheduling, and yes, I understand barriers and this
    > concept of instruction score-boarding and optimization very well -- its
    > not an excuse for a busted C compiler.

    The point is not whether it is possible to work with volatile types, but
    that we tend not to use them in Linux to deal with concurrency.

    Also, barriers seem to work fine for everybody else, so I think it is
    likely you either aren't using them correctly, or have other bugs in the

    > It did not break all the places in the code, but broke enough for SMP to
    > lock up and fail, It turned global variables into local variables. If
    > you want me to reproduce this I can but it will have to wait til this
    > evening
    > because I have some product releases to get out the door at Omega 8 today.
    > It's simple to reproduce. Take away the volatile declaration for the
    > rlock_t structure in mdb-ia32.c (rlock_t debug_lock) in all code
    > references and watch the thing lock up in SMP with multiple processors in
    > the debugger each stuck with their own local copy of debug_lock.

    You should disable preempt before getting the processor id. Can't see any
    other possible bugs, but you should be able to see from the disassembly
    pretty easily.

     \ /
      Last update: 2008-08-21 15:41    [W:0.022 / U:73.192 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site