lkml.org 
[lkml]   [2014]   [Sep]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Subject[PATCH 0/7] Silence even more W=2 warnings
    Date
    The following patches silence over 100,000 warnings in a W=2
    kernel build. This series does most of it by using the compilers
    diagnostic controls. The first patch in the series adds macros to
    invoke the pragmas for those controls. Macros are provided for GCC
    and clang. Although they are highly compatible in this area, macros
    are provided for compiler-specific controls, and there is one
    example that uses a clang-specific control (look for DIAG_CLANG_IGNORE).

    Some missing-field-initializers warnings were resolved using
    the diagnostic control macros simply because so many lines
    would have had to have been changed. At this stage Mark thought
    about avoiding possible merge issues. If the maintainer would
    rather resolve them by using designated initialization, just
    say so.

    The combined effect of this patch series and his other patches
    that did not use these diagnostic control macros was to reduce
    the number of W=2 warnings from 127,164 to 1,345!

    Signed-off-by: Mark Rustad <mark.d.rustad@intel.com>
    Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>

    Mark Rustad (7):
    compiler: Add diagnostic control macros
    x86: Silence initializer-overrides warnings
    atomic: Silence nested-externs warnings
    bitops: Silence nested-externs warnings
    signal: Silence nested-externs warnings
    mm: Silence nested-externs warnings
    sched: Silence nested-externs warnings

    arch/x86/ia32/syscall_ia32.c | 2 ++
    include/linux/atomic.h | 2 ++
    include/linux/bitops.h | 2 ++
    include/linux/compiler-clang.h | 26 ++++++++++++++++++++++++++
    include/linux/compiler-gcc4.h | 31 +++++++++++++++++++++++++++++++
    include/linux/compiler.h | 20 ++++++++++++++++++++
    include/linux/mm.h | 2 ++
    include/linux/sched.h | 2 ++
    include/linux/signal.h | 6 ++++++
    9 files changed, 93 insertions(+)

    --
    1.9.3



    \
     
     \ /
      Last update: 2014-09-19 18:01    [W:3.111 / U:0.612 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site