tree: git://git.cmpxchg.org/linux-mmotm.git master head: 9477ec75947f2cf0fc47e8ab781a5e9171099be2 commit: b03c0167d85a990598c69922fa6a290cba5b5ec8 [144/319] mm/zpool: update zswap to use zpool reproduce: make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) >> mm/zsmalloc.c:253:6: sparse: symbol 'zs_zpool_destroy' was not declared. Should it be static? >> mm/zsmalloc.c:258:5: sparse: symbol 'zs_zpool_malloc' was not declared. Should it be static? >> mm/zsmalloc.c:263:6: sparse: symbol 'zs_zpool_free' was not declared. Should it be static? >> mm/zsmalloc.c:268:5: sparse: symbol 'zs_zpool_shrink' was not declared. Should it be static? >> mm/zsmalloc.c:274:6: sparse: symbol 'zs_zpool_map' was not declared. Should it be static? >> mm/zsmalloc.c:294:6: sparse: symbol 'zs_zpool_unmap' was not declared. Should it be static? >> mm/zsmalloc.c:299:5: sparse: symbol 'zs_zpool_total_size' was not declared. Should it be static? Please consider folding the attached diff :-) --- 0-DAY kernel build testing backend Open Source Technology Center http://lists.01.org/mailman/listinfo/kbuild Intel Corporation
From: Fengguang Wu <fengguang.wu@xxxxxxxxx> Subject: [PATCH mmotm] mm/zpool: zs_zpool_destroy() can be static TO: Dan Streetman <ddstreet@xxxxxxxx> CC: Johannes Weiner <hannes@xxxxxxxxxxx> CC: linux-mm@xxxxxxxxx CC: linux-kernel@xxxxxxxxxxxxxxx CC: Dan Streetman <ddstreet@xxxxxxxx> CC: Johannes Weiner <hannes@xxxxxxxxxxx> Signed-off-by: Fengguang Wu <fengguang.wu@xxxxxxxxx> --- zsmalloc.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c index feba644..20dc632 100644 --- a/mm/zsmalloc.c +++ b/mm/zsmalloc.c @@ -250,28 +250,28 @@ static void *zs_zpool_create(gfp_t gfp, struct zpool_ops *zpool_ops) return zs_create_pool(gfp); } -void zs_zpool_destroy(void *pool) +static void zs_zpool_destroy(void *pool) { zs_destroy_pool(pool); } -int zs_zpool_malloc(void *pool, size_t size, unsigned long *handle) +static int zs_zpool_malloc(void *pool, size_t size, unsigned long *handle) { *handle = zs_malloc(pool, size); return *handle ? 0 : -1; } -void zs_zpool_free(void *pool, unsigned long handle) +static void zs_zpool_free(void *pool, unsigned long handle) { zs_free(pool, handle); } -int zs_zpool_shrink(void *pool, unsigned int pages, +static int zs_zpool_shrink(void *pool, unsigned int pages, unsigned int *reclaimed) { return -EINVAL; } -void *zs_zpool_map(void *pool, unsigned long handle, +static void *zs_zpool_map(void *pool, unsigned long handle, enum zpool_mapmode mm) { enum zs_mapmode zs_mm; @@ -291,12 +291,12 @@ void *zs_zpool_map(void *pool, unsigned long handle, return zs_map_object(pool, handle, zs_mm); } -void zs_zpool_unmap(void *pool, unsigned long handle) +static void zs_zpool_unmap(void *pool, unsigned long handle) { zs_unmap_object(pool, handle); } -u64 zs_zpool_total_size(void *pool) +static u64 zs_zpool_total_size(void *pool) { return zs_get_total_size_bytes(pool); }