On Mon, Sep 20, 2010 at 07:06, <[email protected]> wrote: > Revision 9155 Author sonicz Date 2010-09-20 07:06:18 -0400 (Mon, 20 Sep > 2010) > > Log Message > > Fix bug[#6254] [#5805] access_ok: extend memory start to cover init memory > > Instead of change logic of access_ok to check memory region across the > init memory and generic allocable memory border, extend the start > address of generic allocable memory to cover the adjacent init memory > after init memory region is freed. > > Modified: trunk/arch/blackfin/mm/init.c (9154 => 9155) > > unsigned long addr; > /* next to check that the page we free is not a partial page */ > - for (addr = begin; addr + PAGE_SIZE <= end; addr += PAGE_SIZE) { > + for (addr = begin; addr + PAGE_SIZE < end; addr += PAGE_SIZE) {
why did you need to change this ? since you're adding a value to an address, the result is not a valid address itself, but one byte past it. so the compare is typically a "<=" instead of "<". -mike _______________________________________________ Linux-kernel-commits mailing list [email protected] https://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits
