cvs commit: src/sys/net if_ethersubr.c src/sys/sys mbuf.h src/sys/kern uipc_mbuf.c src/sys/conf NOTES options

[Max Laier]

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).

>   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



-- 
/"\  Best regards,                      | mlaier at freebsd.org
\ /  Max Laier                          | ICQ #67774661
 X   http://pf4freebsd.love2party.net/  | mlaier at EFnet
/ \  ASCII Ribbon Campaign              | Against HTML Mail and News