>-              start += PAGE_ALIGN(k->start);
>-              end = (start + (k->num_pages << EFI_PAGE_SHIFT)) & PAGE_MASK;

But this is just two lines ... I can't see how the unaligned "end" is
escaping past this point!

I do see a bug that the second line should be:

    end = (k->start + (k->num_pages << EFI_PAGE_SHIFT)) & PAGE_MASK;

since the "num_pages" size starts at the start of the block, not
at the possibly rounded up "start".

I'll apply this, and the clean-up bits.  I'd already made a similar
change to uncached.c (which I had mailed out yesterday, but seems to
have disappeared into the ether).

-Tony
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to