lkml.org 
[lkml]   [2010]   [May]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3/3] lockdep: No need to disable preemption in debug atomic ops
    Date
    No need to disable preemption in the debug_atomic_* ops, as
    we ensure interrupts are disabled already.

    So let's use the __this_cpu_ops() rather than this_cpu_ops() that
    enclose the ops in a preempt disabled section.

    Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
    Cc: Peter Zijlstra <peterz@infradead.org>
    ---
    kernel/lockdep_internals.h | 4 ++--
    1 files changed, 2 insertions(+), 2 deletions(-)

    diff --git a/kernel/lockdep_internals.h b/kernel/lockdep_internals.h
    index 8d929c7..4f560cf 100644
    --- a/kernel/lockdep_internals.h
    +++ b/kernel/lockdep_internals.h
    @@ -144,12 +144,12 @@ DECLARE_PER_CPU(struct lockdep_stats, lockdep_stats);

    #define debug_atomic_inc(ptr) { \
    WARN_ON_ONCE(!irqs_disabled()); \
    - this_cpu_inc(lockdep_stats.ptr); \
    + __this_cpu_inc(lockdep_stats.ptr); \
    }

    #define debug_atomic_dec(ptr) { \
    WARN_ON_ONCE(!irqs_disabled()); \
    - this_cpu_dec(lockdep_stats.ptr); \
    + __this_cpu_dec(lockdep_stats.ptr); \
    }

    #define debug_atomic_read(ptr) ({ \
    --
    1.6.2.3


    \
     
     \ /
      Last update: 2010-05-04 05:47    [W:0.022 / U:0.404 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site