lkml.org 
[lkml]   [2010]   [Dec]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    Subject[RFC][PATCH 00/10] incorrect unlikely() and likely() cleanups
    This is my end of year branch annotation cleanups. I compiled my
    kernel with:

    CONFIG_PROFILE_ANNOTATED_BRANCHES

    And booted this on my main work station, as well as a couple of
    test boxes. Letting it run for a while I then looked at the results.

    cat /debug/tracing/trace_stat/branch_annotated

    Which gives a result looking something like this:

    correct incorrect % Function File Line
    ------- --------- - -------- ---- ----
    0 366631256 100 sched_info_switch sched_stats.h 269
    0 211488512 100 sched_info_queued sched_stats.h 222
    0 197083456 100 sched_info_dequeued sched_stats.h 177
    0 130545377 100 syscall_trace_enter ptrace.c 1375
    0 130545340 100 syscall_trace_leave ptrace.c 1403
    0 326960 100 pre_schedule_rt sched_rt.c 1476
    0 5214 100 signal_pending sched.h 2313
    0 1305 100 trylock_page pagemap.h 316

    It sorts the list with those that have the highest percentage of
    incorrect uses and the largest number of them.

    The sched_info_switch() is part of the sched_stats which falls into:

    if (unlikely(1))

    And gcc is smart enough to ignore it, but because it uses a static
    inline function that returns 1, the branch profiler misses that it
    is indeed a constant.

    I also ignored those that only had a few hits, like the signal_pending
    above.

    Anyway, there are 10 areas that I found that really do not need to have
    an annotated branch, or the annotation is simply the opposite of
    what it should be.

    If people are fine with these, I'm hoping to get acks and then push this
    out as a real clean up for 2.6.38.


    The following patches are in:

    git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-2.6-trace.git

    branch: unlikely/core


    Steven Rostedt (10):
    sched: Change rt_task(prev) in pre_schedule_rt to likely
    mm: Remove likely() from mapping_unevictable()
    workqueue: It is likely that WORKER_NOT_RUNNING is true
    sched: Change pick_next_task_rt from unlikely to likely
    mm: Remove likely() from grab_cache_page_write_begin()
    sched: Remove unlikely() from rt_policy() in sched.c
    x86: Remove unlikey()'s from sched_switch segment tests
    fs: Remove unlikely() from fput_light()
    fs: Remove unlikely() from fget_light()
    sched: Remove unlikely() from ttwu_post_activation

    ----
    arch/x86/kernel/process_64.c | 6 +++---
    fs/file_table.c | 2 +-
    include/linux/file.h | 2 +-
    include/linux/pagemap.h | 2 +-
    kernel/sched.c | 4 ++--
    kernel/sched_rt.c | 4 ++--
    kernel/workqueue.c | 4 ++--
    mm/filemap.c | 2 +-
    8 files changed, 13 insertions(+), 13 deletions(-)


    \
     
     \ /
      Last update: 2010-12-07 03:17    [W:0.024 / U:3.252 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site