From owner-cvs-all@FreeBSD.ORG Wed Apr 30 18:39:42 2008 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 82A891065675 for ; Wed, 30 Apr 2008 18:39:42 +0000 (UTC) (envelope-from julian@elischer.org) Received: from outC.internet-mail-service.net (outc.internet-mail-service.net [216.240.47.226]) by mx1.freebsd.org (Postfix) with ESMTP id 52FA28FC24 for ; Wed, 30 Apr 2008 18:39:42 +0000 (UTC) (envelope-from julian@elischer.org) Received: from mx0.idiom.com (HELO idiom.com) (216.240.32.160) by out.internet-mail-service.net (qpsmtpd/0.40) with ESMTP; Wed, 30 Apr 2008 16:56:50 -0700 Received: from julian-mac.elischer.org (localhost [127.0.0.1]) by idiom.com (Postfix) with ESMTP id 074662D6018; Wed, 30 Apr 2008 11:39:40 -0700 (PDT) Message-ID: <4818BCEF.1040308@elischer.org> Date: Wed, 30 Apr 2008 11:39:43 -0700 From: Julian Elischer User-Agent: Thunderbird 2.0.0.12 (Macintosh/20080213) MIME-Version: 1.0 To: Max Laier References: <200804292123.m3TLNLwT044155@repoman.freebsd.org> <200804301939.06987.max@love2party.net> In-Reply-To: <200804301939.06987.max@love2party.net> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: cvs-src@freebsd.org, src-committers@freebsd.org, Julian Elischer , cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/net if_ethersubr.c src/sys/sys mbuf.h src/sys/kern uipc_mbuf.c src/sys/conf NOTES options X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 30 Apr 2008 18:39:42 -0000 Max Laier wrote: > On Tuesday 29 April 2008 23:23:21 Julian Elischer wrote: >> julian 2008-04-29 21:23:21 UTC >> >> FreeBSD src repository >> >> Modified files: >> sys/net if_ethersubr.c >> sys/sys mbuf.h >> sys/kern uipc_mbuf.c >> sys/conf NOTES options >> Log: >> Add an option (compiled out by default) >> to profile outoing packets for a number of mbuf chain >> related parameters >> e.g. number of mbufs, wasted space. >> probably will do with further work later. > > This breaks the build: > http://tinderbox.freebsd.org/tinderbox-head-HEAD-amd64-amd64.brief > > 1) Use %u to print unsigned values > 2) printing [u]int64_t's has been broken since the beginning. The reason > is the unfortunate choice to have int64_t be a "long int" alias on > platforms with a 64bit long (while they could as easily be "long long > int" as on the other platforms where long is 32bit wide - this also means > that "long long" is > intmax_t which is an alias for int64_t). Hence you > either have to use the (ugly) PRIu64 macro, or %ju and cast to uintmax_t. > This is a no-op (as long as we don't have uint128_t or the like). I'm happy to change the types to any way you suggest.. how about just changing them to long long? > >> Reviewed by: various >> >> Revision Changes Path >> 1.1482 +5 -0 src/sys/conf/NOTES >> 1.627 +1 -0 src/sys/conf/options >> 1.178 +152 -0 src/sys/kern/uipc_mbuf.c >> 1.245 +2 -0 src/sys/net/if_ethersubr.c >> 1.225 +8 -0 src/sys/sys/mbuf.h > > >