lkml.org 
[lkml]   [2006]   [May]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH] mm: cleanup swap unused warning
    Date
    Are there any users of swp_entry_t when CONFIG_SWAP is not defined?

    This patch fixes a warning for !CONFIG_SWAP for me.

    ---
    if CONFIG_SWAP is not defined we get:

    mm/vmscan.c: In function ‘remove_mapping’:
    mm/vmscan.c:387: warning: unused variable ‘swap’

    Signed-off-by: Con Kolivas <kernel@kolivas.org>

    ---
    include/linux/swap.h | 15 +++++++++++----
    1 files changed, 11 insertions(+), 4 deletions(-)

    Index: linux-2.6.17-rc3-mm1/include/linux/swap.h
    ===================================================================
    --- linux-2.6.17-rc3-mm1.orig/include/linux/swap.h 2006-05-10 21:14:41.000000000 +1000
    +++ linux-2.6.17-rc3-mm1/include/linux/swap.h 2006-05-10 21:24:31.000000000 +1000
    @@ -67,13 +67,20 @@ union swap_header {
    } info;
    };

    - /* A swap entry has to fit into a "unsigned long", as
    - * the entry is hidden in the "index" field of the
    - * swapper address space.
    - */
    +/*
    + * A swap entry has to fit into a "unsigned long", as
    + * the entry is hidden in the "index" field of the
    + * swapper address space.
    + */
    +#ifdef CONFIG_SWAP
    typedef struct {
    unsigned long val;
    } swp_entry_t;
    +#else
    +typedef struct {
    + unsigned long val;
    +} swp_entry_t __attribute__((__unused__));
    +#endif

    /*
    * current->reclaim_state points to one of these when a task is running
    --
    -ck
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-05-10 13:36    [W:0.021 / U:31.420 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site