From: Colin Ian King <colin.king@xxxxxxxxxxxxx> Subject: lib/test_ubsan.c: make test_ubsan_misaligned_access() static test_ubsan_misaligned_access() is local to the source and does not need to be in global scope, so make it static. Cleans up sparse warning: lib/test_ubsan.c:91:6: warning: symbol 'test_ubsan_misaligned_access' was not declared. Should it be static? Link: http://lkml.kernel.org/r/20180313103048.28513-1-colin.king@xxxxxxxxxxxxx Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> Cc: Jinbum Park <jinb.park7@xxxxxxxxx> Cc: Andrey Ryabinin <aryabinin@xxxxxxxxxxxxx> Cc: Dmitry Vyukov <dvyukov@xxxxxxxxxx> Cc: Kees Cook <keescook@xxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- lib/test_ubsan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -puN lib/test_ubsan.c~lib-add-testing-module-for-ubsan-fix lib/test_ubsan.c --- a/lib/test_ubsan.c~lib-add-testing-module-for-ubsan-fix +++ a/lib/test_ubsan.c @@ -88,7 +88,7 @@ static void test_ubsan_null_ptr_deref(vo val = *ptr; } -void test_ubsan_misaligned_access(void) +static void test_ubsan_misaligned_access(void) { volatile char arr[5] __aligned(4) = {1, 2, 3, 4, 5}; volatile int *ptr, val = 6; _ -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html