On 01/07/2007 07:07 PM, Jeremy Fitzhardinge wrote: > Rene Herman wrote: >> Doing the set_fs() and pagefault_{disable,enable} calls for every >> single byte during the checksum seems rather silly. > > Why? Because it makes for dumb code. But oh well, given that it all compiles to basically nothing I guess I'll stop objecting. Andrew: The attached removes the assumption that if the first page of an ISA ROM is mapped, it'll all be mapped. This'll also stop people reading this code from wondering if they're looking at a bug. This replaces "romsignature-checksum-cleanup.patch" in current -mm. Signed-off-by: Rene Herman Not-strongly-objected-to-by: Jeremy Fitzhardinge Rene.