Messages in this thread | | | Date | Fri, 22 Nov 2013 17:10:02 +0900 | From | Masami Hiramatsu <> | Subject | Re: Re: [PATCH 2/2] tracing/probes: Fix basic print type functions |
| |
(2013/11/22 15:05), Namhyung Kim wrote: > Hi Oleg, > > On Wed, 20 Nov 2013 16:22:45 +0100, Oleg Nesterov wrote: >> On 11/18, Namhyung Kim wrote: >>> >>> -DEFINE_BASIC_PRINT_TYPE_FUNC(u8, "%x", unsigned int) >>> -DEFINE_BASIC_PRINT_TYPE_FUNC(u16, "%x", unsigned int) >>> -DEFINE_BASIC_PRINT_TYPE_FUNC(u32, "%lx", unsigned long) >>> +DEFINE_BASIC_PRINT_TYPE_FUNC(u8 , "%x", unsigned char) >>> +DEFINE_BASIC_PRINT_TYPE_FUNC(u16, "%x", unsigned short) >>> +DEFINE_BASIC_PRINT_TYPE_FUNC(u32, "%x", unsigned int) >>> +DEFINE_BASIC_PRINT_TYPE_FUNC(s8, "%d", signed char) >>> +DEFINE_BASIC_PRINT_TYPE_FUNC(s16, "%d", short) >>> +DEFINE_BASIC_PRINT_TYPE_FUNC(s32, "%d", int) >>> +#if BITS_PER_LONG == 32 >>> DEFINE_BASIC_PRINT_TYPE_FUNC(u64, "%llx", unsigned long long) >>> -DEFINE_BASIC_PRINT_TYPE_FUNC(s8, "%d", int) >>> -DEFINE_BASIC_PRINT_TYPE_FUNC(s16, "%d", int) >>> -DEFINE_BASIC_PRINT_TYPE_FUNC(s32, "%ld", long) >>> DEFINE_BASIC_PRINT_TYPE_FUNC(s64, "%lld", long long) >>> +#else /* BITS_PER_LONG == 64 */ >>> +DEFINE_BASIC_PRINT_TYPE_FUNC(u64, "%lx", unsigned long) >>> +DEFINE_BASIC_PRINT_TYPE_FUNC(s64, "%ld", long) >>> +#endif >> >> I must have missed something... Not only I do not understand why >> do we need "#if BITS_PER_LONG", I can't understand why >> DEFINE_BASIC_PRINT_TYPE_FUNC() needs "cast" argument. >> >> IOW, how about the patch below instead? > > Looks good to me. > > Masami, did you have any issue regarding the casts?
Hm, unless compiler doesn't complain about it,I'm OK for that. And as far as I tested on x86/x86-64/arm(32), this has no problem. :)
Acked-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Thank you!
-- Masami HIRAMATSU IT Management Research Dept. Linux Technology Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: masami.hiramatsu.pt@hitachi.com
| |