On Mon, Jan 21, 2013 at 01:38:23PM -0800, Sukadev Bhattiprolu wrote: > Jiri Olsa [jo...@redhat.com] wrote: > | On Fri, Jan 18, 2013 at 05:30:52PM -0800, Sukadev Bhattiprolu wrote:
SNIP > __u64 is 'unsigned long long' on x86 and PRIu64 is 'llu' which is fine. > > __u64 is 'unsigned long' on Power and PRIu64 is 'lu' which is again fine. > > But __u64 is 'unsigned long long' on x86_64, but PRIu64 is '%lu' bc __WORDSIZE > is 64. > > On x86_64, shouldn't __u64, be defined as 'unsigned long' rather than > 'unsigned long long' - ie include 'int-l64.h' rather than 'int-ll64.h' ? hum, not sure ;-) will try to find some time to look on that > > BTW, does 'perf' with my patch compile, (with warnings) for you on x86_64 > with 'WERROR=0 make' ? this one passes with warnings jirka -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/