From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 10FDBC0015E for ; Sat, 5 Aug 2023 16:33:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229891AbjHEQdN (ORCPT ); Sat, 5 Aug 2023 12:33:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41478 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229441AbjHEQdM (ORCPT ); Sat, 5 Aug 2023 12:33:12 -0400 Received: from 1wt.eu (ded1.1wt.eu [163.172.96.212]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 4E40D10F8 for ; Sat, 5 Aug 2023 09:33:11 -0700 (PDT) Received: (from willy@localhost) by pcw.home.local (8.15.2/8.15.2/Submit) id 375GWpDW023835; Sat, 5 Aug 2023 18:32:51 +0200 Date: Sat, 5 Aug 2023 18:32:51 +0200 From: Willy Tarreau To: Thomas =?iso-8859-1?Q?Wei=DFschuh?= , Yuan Tan Cc: Shuah Khan , linux-kselftest@vger.kernel.org, "linux-kernel@vger.kernel.org Zhangjin Wu" Subject: Re: [PATCH v3 14/14] selftests/nolibc: enable compiler warnings Message-ID: <20230805163251.GC15284@1wt.eu> References: <20230803-nolibc-warnings-v3-0-bcc1a096ae02@weissschuh.net> <20230803-nolibc-warnings-v3-14-bcc1a096ae02@weissschuh.net> <20230805162327.GB15284@1wt.eu> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20230805162327.GB15284@1wt.eu> User-Agent: Mutt/1.10.1 (2018-07-13) Precedence: bulk List-ID: X-Mailing-List: linux-kselftest@vger.kernel.org On Sat, Aug 05, 2023 at 06:23:27PM +0200, Willy Tarreau wrote: > On Thu, Aug 03, 2023 at 09:28:58AM +0200, Thomas Weißschuh wrote: > > It will help the developers to avoid cruft and detect some bugs. > > > > Signed-off-by: Thomas Weißschuh > > --- > > tools/testing/selftests/nolibc/Makefile | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/tools/testing/selftests/nolibc/Makefile b/tools/testing/selftests/nolibc/Makefile > > index b82d29b6c37f..e8d09cbee2ab 100644 > > --- a/tools/testing/selftests/nolibc/Makefile > > +++ b/tools/testing/selftests/nolibc/Makefile > > @@ -79,7 +79,7 @@ endif > > CFLAGS_s390 = -m64 > > CFLAGS_mips = -EL > > CFLAGS_STACKPROTECTOR ?= $(call cc-option,-mstack-protector-guard=global $(call cc-option,-fstack-protector-all)) > > -CFLAGS ?= -Os -fno-ident -fno-asynchronous-unwind-tables -std=c89 \ > > +CFLAGS ?= -Os -fno-ident -fno-asynchronous-unwind-tables -std=c89 -W -Wall -Wextra \ > > $(call cc-option,-fno-stack-protector) \ > > $(CFLAGS_$(ARCH)) $(CFLAGS_STACKPROTECTOR) > > LDFLAGS := > > I'm now getting this with gcc < 9: > > nolibc-test.c: In function 'test_pipe': > nolibc-test.c:811:10: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] > if (len != strlen(msg)) > ^ > The reason is that len is ssize_t and strlen() is size_t. I tried different > approaches here but the cleanest remains turning len to size_t (we don't > use its sign anyway), so I'll do that one as well. By the way since the original commit is still not upstream I'd rather reinject this change directly in it. Yuan, are you OK with this ? diff --git a/tools/testing/selftests/nolibc/nolibc-test.c b/tools/testing/selftests/nolibc/nolibc-test.c index 0145a44af990..95884168cea0 100644 --- a/tools/testing/selftests/nolibc/nolibc-test.c +++ b/tools/testing/selftests/nolibc/nolibc-test.c @@ -798,7 +798,7 @@ int test_pipe(void) const char *const msg = "hello, nolibc"; int pipefd[2]; char buf[32]; - ssize_t len; + size_t len; if (pipe(pipefd) == -1) Thanks, Willy