[lkml]   [2009]   [Jan]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[PATCH 1/4] wimax: fix '#ifndef CONFIG_BUG' layout to avoid warning
    Reported by Randy Dunlap:

    > Also, this warning needs to be fixed:
    > linux-next-20090106/net/wimax/id-table.c:133: warning: ISO C90
    > forbids mixed declarations and code

    Move the return on #defined(CONFIG_BUG) below the variable
    declarations so it doesn't violate ISO C90.

    On wimax_id_table_release() we want to do a debug check if CONFIG_BUG
    is enabled. However, we also want the debug code to be always compiled
    to ensure there is no bitrot. It will be optimized out by the compiler
    when CONFIG_BUG is disabled.

    Added a note to the function header stating this.

    Signed-off-by: Inaky Perez-Gonzalez <>
    net/wimax/id-table.c | 8 +++++---
    1 files changed, 5 insertions(+), 3 deletions(-)

    diff --git a/net/wimax/id-table.c b/net/wimax/id-table.c
    index d3b8855..5e685f7 100644
    --- a/net/wimax/id-table.c
    +++ b/net/wimax/id-table.c
    @@ -123,15 +123,17 @@ void wimax_id_table_rm(struct wimax_dev *wimax_dev)
    * Release the gennetlink family id / mapping table
    - * On debug, verify that the table is empty upon removal.
    + * On debug, verify that the table is empty upon removal. We want the
    + * code always compiled, to ensure it doesn't bit rot. It will be
    + * compiled out if CONFIG_BUG is disabled.
    void wimax_id_table_release(void)
    + struct wimax_dev *wimax_dev;
    #ifndef CONFIG_BUG
    - struct wimax_dev *wimax_dev;
    list_for_each_entry(wimax_dev, &wimax_id_table, id_table_node) {
    printk(KERN_ERR "BUG: %s wimax_dev %p ifindex %d not cleared\n",

     \ /
      Last update: 2009-01-07 20:25    [W:0.020 / U:19.344 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site