From owner-freebsd-ppc@FreeBSD.ORG Tue May 3 01:32:11 2005 Return-Path: Delivered-To: freebsd-ppc@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 9A7BD16A4CE; Tue, 3 May 2005 01:32:11 +0000 (GMT) Received: from elvis.mu.org (elvis.mu.org [192.203.228.196]) by mx1.FreeBSD.org (Postfix) with ESMTP id 9060343D75; Tue, 3 May 2005 01:32:10 +0000 (GMT) (envelope-from mux@freebsd.org) Received: by elvis.mu.org (Postfix, from userid 1920) id 36CA75CAD6; Mon, 2 May 2005 18:32:09 -0700 (PDT) Date: Tue, 3 May 2005 03:32:09 +0200 From: Maxime Henrion To: Peter Grehan Message-ID: <20050503013209.GA56827@elvis.mu.org> References: <20050502201921.4F0387306E@freebsd-current.sentex.ca> <4276C18F.1080207@freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4276C18F.1080207@freebsd.org> User-Agent: Mutt/1.4.2.1i cc: powerpc@freebsd.org cc: FreeBSD Tinderbox cc: current@freebsd.org Subject: Re: [current tinderbox] failure on powerpc/powerpc X-BeenThere: freebsd-ppc@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Porting FreeBSD to the PowerPC List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 03 May 2005 01:32:11 -0000 Peter Grehan wrote: > >cc -O2 -pipe -DIPFILTER_LOG -DIPFILTER_COMPILED -DIPFILTER_LOOKUP > >-DIPFILTER_SCAN -DIPFILTER_SYNC -DIPFILTER_CKSUM -I. > >-I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../contrib/ipfilter -I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../contrib/ipfilter/tools -I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../sys -I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../sys/contrib/ipfilter -DSTATETOP -D__UIO_EXPOSE -Wsystem-headers -Werror -Wall -Wno-format-y2k -Wno-uninitialized -c /tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../contrib/ipfilter/radix.c > >cc -O2 -pipe -DIPFILTER_LOG -DIPFILTER_COMPILED -DIPFILTER_LOOKUP > >-DIPFILTER_SCAN -DIPFILTER_SYNC -DIPFILTER_CKSUM -I. > >-I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../contrib/ipfilter -I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../contrib/ipfilter/tools -I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../sys -I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../sys/contrib/ipfilter -DSTATETOP -D__UIO_EXPOSE -Wsystem-headers -Werror -Wall -Wno-format-y2k -Wno-uninitialized -c /tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../contrib/ipfilter/bpf_filter.c > >cc -O2 -pipe -DIPFILTER_LOG -DIPFILTER_COMPILED -DIPFILTER_LOOKUP > >-DIPFILTER_SCAN -DIPFILTER_SYNC -DIPFILTER_CKSUM -I. > >-I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../contrib/ipfilter -I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../contrib/ipfilter/tools -I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../sys -I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../sys/contrib/ipfilter -DSTATETOP -D__UIO_EXPOSE -Wsystem-headers -Werror -Wall -Wno-format-y2k -Wno-uninitialized -o ipftest ipftest.o fil.o ip_frag.o ip_state.o ip_nat.o ip_proxy.o ip_auth.o ip_htable.o ip_lookup.o ip_pool.o ip_scan.o ip_sync.o ip_rules.o ip_fil.o ip_log.o ippool_y.o ippool_l.o ipf_y.o ipf_l.o ipnat_y.o ipnat_l.o md5.o radix.o bpf_filter.o -L/tinderbox/CURRENT/powerpc/powerpc/obj/powerpc/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../libipf -lipf -lkvm > >gzip -cn > >/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipftest/../../../contrib/ipfilter/man/ipftest.1 > ipftest.1.gz > >===> sbin/ipf/ipmon (all) > >cc -O2 -pipe -DLOGFAC=LOG_LOCAL0 -I. > >-I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipmon/../../../contrib/ipfilter -I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipmon/../../../contrib/ipfilter/tools -I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipmon/../../../sys -I/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipmon/../../../sys/contrib/ipfilter -DSTATETOP -D__UIO_EXPOSE -Wsystem-headers -Werror -Wall -Wno-format-y2k -Wno-uninitialized -c /tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipmon/../../../contrib/ipfilter/tools/ipmon.c > >/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipmon/../../../contrib/ipfilter/tools/ipmon.c: In function `print_ipflog': > >/tinderbox/CURRENT/powerpc/powerpc/src/sbin/ipf/ipmon/../../../contrib/ipfilter/tools/ipmon.c:1030: warning: comparison is always false due to limited range of data type > > I've sent mail to Darren about this: it's a PPC-only (well, maybe ARM > as well) default-unsigned-char problem. For those that want to > buildworld appended is the patch I'm using. > > later, > > Peter. > > RCS file: /home/ncvs/src/contrib/ipfilter/tools/ipmon.c,v > retrieving revision 1.2 > diff -u -r1.2 ipmon.c > --- ipmon.c 25 Apr 2005 18:20:15 -0000 1.2 > +++ ipmon.c 1 May 2005 08:42:53 -0000 > @@ -1024,7 +1024,7 @@ > (void) sprintf(t, "%*.*s%u", len, len, ipf->fl_ifname, > ipf->fl_unit); > t += strlen(t); > #endif > -#ifdef __sgi > +#if (defined(__sgi) || defined(__powerpc__)) > if ((ipf->fl_group[0] == 255) && (ipf->fl_group[1] == '\0')) > #else > if ((ipf->fl_group[0] == -1) && (ipf->fl_group[1] == '\0')) While I suppose it might be possible to drop the #if completely, it would be better to use #if CHAR_MIN == 0 here. That would make it work for FreeBSD/arm too, as well as any platform where chars are unsigned by default on which this code may run. Cheers, Maxime