[PATCH 3/6] ARM: mm: Drop the lowmem watermark check from virt_addr_valid()

Russell King - ARM Linux linux at arm.linux.org.uk
Fri Nov 15 09:20:14 EST 2013


On Thu, Nov 14, 2013 at 02:37:43PM -0500, Santosh Shilimkar wrote:
> Slab allocator can allocate memory beyond the lowmem watermark
> which can lead to false failure of virt_addr_valid().

This is definitely going to cause problems.

> So drop the check. The issue was seen with percpu_alloc()
> in KVM code which was allocating memory beyond lowmem watermark.
> 
> Am not completly sure whether this is the right fix and if it could
> impact any other user of virt_addr_valid(). Without this fix as
> pointed out the KVM init was failing in my testing.

virt_addr_valid() gets used in some places to check whether the virtual
address is part of the lowmem mapping and explicitly not part of vmalloc()
or DMA coherent space:

drivers/mtd/nand/gpmi-nand/gpmi-nand.c
drivers/net/wireless/ath/ath6kl/sdio.c

It opens up the checks in include/linux/scatterlist.h to start accepting
non-streaming DMA capable buffers as well.

It also bypasses a check in the slab code to ensure that it's a potentially
valid pointer that was handed out by slab.

.. etc ..



More information about the linux-arm-kernel mailing list