Re: [PATCH] romsignature/checksum cleanup

From: Zachary Amsden
Date: Wed Dec 27 2006 - 19:32:55 EST

Rusty Russell wrote:
On Mon, 2006-12-25 at 01:53 +0100, Rene Herman wrote:
Rene Herman wrote:

Use adding __init to romsignature() (it's only called from probe_roms() which is itself __init) as an excuse to submit a pedantic cleanup.
Hmm, by the way, if romsignature() needs this probe_kernel_address() thing, why doesn't romchecksum()?

I assume it's all in the same page, but CC'ing Zach is easier than
reading the code 8)

Some hypervisors don't emulate the traditional physical layout of the first 1M of memory, so those pages might never get physical mappings established during the boot process, causing access to them to fault. Presumably, if the first page is there with a good signature, the entire ROM is mapped. I think Jeremy added this for Xen, and it's harmless on native hardware.

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at