Ralf Baechle wrote: > On Mon, Nov 22, 2004 at 08:00:04AM +0100, Thiemo Seufer wrote: > > > > Why bother, the unaligned exception handler should take care of this. > > > > It really does so for unaligned accesses from kernel space? > > Yes. In fact it's crucially important for this very case. Ok, I'll update the patch accordingly when I'm back to better connectivity than I have now. [snip] > > has 4 bytes and is loaded with lw. Using a macro which abstracts for > > 32/64bit compilation hides this needlessly, and can even lead to the > > erraneous impression the code would be useful for 64bit, too. > > I'm more following the religion of using such abstractions everywhere > because code tends to be copied around mindlessly ... I would agree if there was a roughly similiar 64bit version of the code. But due to the differences between 32bit and 64bit kernel there will never be one, so it's IMHO best to make them as distinct as reasonable in this case. Thiemo