| Message ID | 20211105162940.3319021-1-anders.roxell@linaro.org (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | selftests: mqueue|vDSO: fix convert pointer warnings | expand |
On 11/5/21 10:29 AM, Anders Roxell wrote: > When building selftests, mqueue and vDSO the following shows up: > > warning: passing 'int *' to parameter of type 'unsigned int *' converts between pointers to integer types with different sign [-Wpointer-sign] > > warning: passing 'const char *' to parameter of type 'const unsigned char *' converts between pointers to integer types where one is of the unique plain 'char' type and the other is not [-Wpointer-sign] > > The code looked OK so what normally are done are, adding the compiler > directive to hide the warnings '-Wno-pointer-sign'. > Would be nice to see the lines of code that are generating these warns. I don't want to see the warnings suppressed all together in the Makefile. I don't see any point in doing so and it will suppress real problems in the future. I won't be accepting this patch. thanks, -- Shuah
diff --git a/tools/testing/selftests/mqueue/Makefile b/tools/testing/selftests/mqueue/Makefile index 8a58055fc1f5..814907b6344f 100644 --- a/tools/testing/selftests/mqueue/Makefile +++ b/tools/testing/selftests/mqueue/Makefile @@ -1,5 +1,5 @@ # SPDX-License-Identifier: GPL-2.0 -CFLAGS += -O2 +CFLAGS += -O2 -Wno-pointer-sign LDLIBS = -lrt -lpthread -lpopt TEST_GEN_PROGS := mq_open_tests mq_perf_tests diff --git a/tools/testing/selftests/vDSO/Makefile b/tools/testing/selftests/vDSO/Makefile index d53a4d8008f9..2a60233ba2fb 100644 --- a/tools/testing/selftests/vDSO/Makefile +++ b/tools/testing/selftests/vDSO/Makefile @@ -12,7 +12,7 @@ TEST_GEN_PROGS += $(OUTPUT)/vdso_standalone_test_x86 endif TEST_GEN_PROGS += $(OUTPUT)/vdso_test_correctness -CFLAGS := -std=gnu99 +CFLAGS := -std=gnu99 -Wno-pointer-sign CFLAGS_vdso_standalone_test_x86 := -nostdlib -fno-asynchronous-unwind-tables -fno-stack-protector LDFLAGS_vdso_test_correctness := -ldl ifeq ($(CONFIG_X86_32),y)
When building selftests, mqueue and vDSO the following shows up: warning: passing 'int *' to parameter of type 'unsigned int *' converts between pointers to integer types with different sign [-Wpointer-sign] warning: passing 'const char *' to parameter of type 'const unsigned char *' converts between pointers to integer types where one is of the unique plain 'char' type and the other is not [-Wpointer-sign] The code looked OK so what normally are done are, adding the compiler directive to hide the warnings '-Wno-pointer-sign'. Signed-off-by: Anders Roxell <anders.roxell@linaro.org> --- tools/testing/selftests/mqueue/Makefile | 2 +- tools/testing/selftests/vDSO/Makefile | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)