No it wasn't.
The main objection was that you SHOULD NOT USE PAGE_SIZE_MASK AT ALL!
You should have used a pagetable-specific macro, becuse PAGE_MASK is
simply fundamentally WRONG, and has absolutely nothing to do with the PFN
bits, whether sign-extended or not!
The fact is, the page frane number bits are *not* ~PAGE_MASK or anything
like that. They share only the low bits - not the high bits.
Linus
--