nathan.zook@amd.com wrote:
> e820.map[0]);
> #ifdef E820_DEBUG
> for (i=0; i < e820.nr_map; i++) {
> - printk("e820: %08x @ %08x ", (int)e820.map[i].size,
> - (int)e820.map[i].addr);
> + printk("%lx %08lx %lx %08lx ",
> + (long)((e820.map[i].addr) >> 32),(long) (e820.map[i].addr),
> + (long)((e820.map[i].size) >> 32), (long) (e820.map[i].size));
No No No!
Use %10Lx to print out long long values, don't shift and print out 2 longs.
Yes, it's expensive and ugly and horrible, but it's debugging code that
will be ifdeffed out in the production kernel and it's more readable until
then.
____
david parsons \bi/ Ack. Pfft.
\/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/