Re: [PATCH v4 03/19] ARM: LPAE: use long long format when printingphysical addresses and ptes

From: Russell King - ARM Linux
Date: Sat Feb 12 2011 - 05:00:04 EST


On Mon, Jan 24, 2011 at 05:55:45PM +0000, Catalin Marinas wrote:
> diff --git a/arch/arm/kernel/setup.c b/arch/arm/kernel/setup.c
> index 5ea4fb7..3d23f0f 100644
> --- a/arch/arm/kernel/setup.c
> +++ b/arch/arm/kernel/setup.c
> @@ -449,7 +449,7 @@ static int __init arm_add_memory(unsigned long start, unsigned long size)
>
> if (meminfo.nr_banks >= NR_BANKS) {
> printk(KERN_CRIT "NR_BANKS too low, "
> - "ignoring memory at %#lx\n", start);
> + "ignoring memory at %#08llx\n", (long long)start);

This is not equivalent. %#lx produces '0x0'. %#08llx produces '0x000000'
not '0x00000000' - the '0x' is included in the field width. So you want
'%#010llx' or '0x%08llx' - there's no real advantage to either. Or just
convert '%#lx' to '%#llx'.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/