Re: [PATCH 03/25] lmb: Print new doubled array location info

From: Yinghai Lu
Date: Tue Jun 29 2010 - 16:06:52 EST


On 06/29/2010 10:44 AM, Bjorn Helgaas wrote:
> On Tuesday, June 22, 2010 11:26:32 am Yinghai Lu wrote:
>> + if (lmb_debug)
>> + pr_info("lmb: %s array is doubled to %ld at %llx - %llx",
>> + lmb_type_name(type), type->max * 2, (u64)addr, (u64)addr + new_size);
>
> Please print this memory range the same way we print resources, e.g.,
> "%#010llx-%#010llx", with "addr" and "addr + new_size - 1".

ok, I will put # for 0x.

but i like to have

xxx - yyy : to include end
[xxx, yyy - 1]

just like current e820 print out.
and it would be more readable without too many ffff

Thanks

Yinghai
--
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/