From owner-freebsd-tinderbox@FreeBSD.ORG Thu Jun 17 13:21:11 2004 Return-Path: Delivered-To: freebsd-tinderbox@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id A634F16A4CF; Thu, 17 Jun 2004 13:21:11 +0000 (GMT) Received: from elvis.mu.org (elvis.mu.org [192.203.228.196]) by mx1.FreeBSD.org (Postfix) with ESMTP id 8BC4643D55; Thu, 17 Jun 2004 13:21:11 +0000 (GMT) (envelope-from mux@freebsd.org) Received: by elvis.mu.org (Postfix, from userid 1920) id 369B85C7E7; Thu, 17 Jun 2004 06:21:11 -0700 (PDT) Date: Thu, 17 Jun 2004 15:21:11 +0200 From: Maxime Henrion To: "Liam J. Foy" Message-ID: <20040617132111.GO9228@elvis.mu.org> References: <20040617094725.45EE37306D@freebsd-current.sentex.ca> <20040617133207.29a6f935.liamfoy@sepulcrum.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20040617133207.29a6f935.liamfoy@sepulcrum.org> User-Agent: Mutt/1.4.2.1i cc: amd64@freebsd.org cc: mlaier@FreeBSD.org cc: hackers@freebsd.org cc: FreeBSD Tinderbox cc: current@freebsd.org Subject: Re: [current tinderbox] failure on amd64/amd64 X-BeenThere: freebsd-tinderbox@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Tinderbox reports, responses, and meta-comments List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 Jun 2004 13:21:12 -0000 Liam J. Foy wrote: > On Thu, 17 Jun 2004 05:47:25 -0400 (EDT) > FreeBSD Tinderbox wrote: > > > TB --- 2004-06-17 08:56:11 - tinderbox 2.3 running on freebsd-current.sentex.ca > > TB --- 2004-06-17 08:56:11 - starting CURRENT tinderbox run for amd64/amd64 > > TB --- 2004-06-17 08:56:11 - checking out the source tree > > TB --- 2004-06-17 08:56:11 - cd /home/tinderbox/sandbox/CURRENT/amd64/amd64 > > TB --- 2004-06-17 08:56:11 - /usr/bin/cvs -f -R -q -d/home/ncvs update -Pd -A src > > TB --- 2004-06-17 09:01:06 - building world (CFLAGS=-O2 -pipe) > > TB --- 2004-06-17 09:01:06 - cd /home/tinderbox/sandbox/CURRENT/amd64/amd64/src > > TB --- 2004-06-17 09:01:06 - /usr/bin/make -B buildworld > > >>> Rebuilding the temporary build tree > > >>> stage 1.1: legacy release compatibility shims > > >>> stage 1.2: bootstrap tools > > >>> stage 2.1: cleaning up the object tree > > >>> stage 2.2: rebuilding the object tree > > >>> stage 2.3: build tools > > >>> stage 3: cross tools > > >>> stage 4.1: building includes > > >>> stage 4.2: building libraries > > >>> stage 4.3: make dependencies > > >>> stage 4.4: building everything > > [...] > > cc -O2 -pipe -o nos-tun nos-tun.o > > gzip -cn /tinderbox/CURRENT/amd64/amd64/src/sbin/nos-tun/nos-tun.8 > nos-tun.8.gz > > ===> sbin/pfctl > > cc -O2 -pipe -Wall -Wmissing-prototypes -Wno-uninitialized -Wstrict-prototypes -I/tinderbox/CURRENT/amd64/amd64/src/sbin/pfctl/../../contrib/pf/pfctl -I/tinderbox/CURRENT/amd64/amd64/src/sbin/pfctl/../../sys/contrib/pf -DENABLE_ALTQ -Wsystem-headers -Werror -Wall -Wno-format-y2k -Wno-uninitialized -c /tinderbox/CURRENT/amd64/amd64/src/contrib/pf/pfctl/pfctl.c > > cc -O2 -pipe -Wall -Wmissing-prototypes -Wno-uninitialized -Wstrict-prototypes -I/tinderbox/CURRENT/amd64/amd64/src/sbin/pfctl/../../contrib/pf/pfctl -I/tinderbox/CURRENT/amd64/amd64/src/sbin/pfctl/../../sys/contrib/pf -DENABLE_ALTQ -Wsystem-headers -Werror -Wall -Wno-format-y2k -Wno-uninitialized -c parse.c > > cc -O2 -pipe -Wall -Wmissing-prototypes -Wno-uninitialized -Wstrict-prototypes -I/tinderbox/CURRENT/amd64/amd64/src/sbin/pfctl/../../contrib/pf/pfctl -I/tinderbox/CURRENT/amd64/amd64/src/sbin/pfctl/../../sys/contrib/pf -DENABLE_ALTQ -Wsystem-headers -Werror -Wall -Wno-format-y2k -Wno-uninitialized -c /tinderbox/CURRENT/amd64/amd64/src/contrib/pf/pfctl/pfctl_parser.c > > /tinderbox/CURRENT/amd64/amd64/src/contrib/pf/pfctl/pfctl_parser.c: In function `print_status': > > /tinderbox/CURRENT/amd64/amd64/src/contrib/pf/pfctl/pfctl_parser.c:559: warning: long long int format, long unsigned int arg (arg 3) > > *** Error code 1 > > > > Stop in /tinderbox/CURRENT/amd64/amd64/src/sbin/pfctl. > > *** Error code 1 > > > > Stop in /tinderbox/CURRENT/amd64/amd64/src/sbin. > > *** Error code 1 > > > > Stop in /tinderbox/CURRENT/amd64/amd64/src. > > *** Error code 1 > > > > Stop in /tinderbox/CURRENT/amd64/amd64/src. > > *** Error code 1 > > > > Stop in /tinderbox/CURRENT/amd64/amd64/src. > > TB --- 2004-06-17 09:47:25 - WARNING: /usr/bin/make returned exit code 1 > > TB --- 2004-06-17 09:47:25 - ERROR: failed to build world > > TB --- 2004-06-17 09:47:25 - tinderbox aborted > > > > _______________________________________________ > > freebsd-current@freebsd.org mailing list > > http://lists.freebsd.org/mailman/listinfo/freebsd-current > > To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org" > > > The following patch should solve this problem I believe: > > --- /usr/src/contrib/pf/pfctl/pfctl_parser.c Thu Jun 17 13:20:07 2004 > +++ /home/liamfoy/pfctl_parser.c Thu Jun 17 13:26:16 2004 > @@ -555,7 +555,7 @@ > printf(" %-25s %14u %14s\n", "current entries", > s->src_nodes, ""); > for (i = 0; i < SCNT_MAX; i++) { > - printf(" %-25s %14lld ", pf_scounters[i], > + printf(" %-25s %14lu ", pf_scounters[i], > s->scounters[i]); > if (runtime > 0) > printf("%14.1f/s\n", > > > > Correct me if I am wrong :), would someone care to commit it if its fine? It's actually not correct. The scounters[i] here is of type u_int64_t, so long is not the correct format here (longs are 32bit on many 32bit architectures such as FreeBSD/i386). The correct format to use here is %llu (long long's are at least 64bit), with a cast to (unsigned long long), but most of the time we prefer to use %ju and a cast to uintmax_t. C99 also provides macros for fixed-size type formats, and we could thus use the PRIu64 macro here, but we don't use them because they're too ugly. I won't commit a fix because this code is in contrib/ and is maintained by Max Laier (see /usr/src/MAINTAINERS). I'm CC'ing him this mail though. Cheers, Maxime