iommu/iova: Avoid over-allocating when size-aligned
authorRobin Murphy <robin.murphy@arm.com>
Thu, 16 Jul 2015 18:40:12 +0000 (19:40 +0100)
committerDavid Woodhouse <David.Woodhouse@intel.com>
Tue, 28 Jul 2015 14:47:56 +0000 (15:47 +0100)
commit8f6429c7cb59f28433253575cc8e3262eed63592
treeb0ba92c894735701a01d289f92c42c221c6d3cf1
parent52721d9d3334c1cb1f76219a161084094ec634dc
iommu/iova: Avoid over-allocating when size-aligned

Currently, allocating a size-aligned IOVA region quietly adjusts the
actual allocation size in the process, returning a rounded-up
power-of-two-sized allocation. This results in mismatched behaviour in
the IOMMU driver if the original size was not a power of two, where the
original size is mapped, but the rounded-up IOVA size is unmapped.

Whilst some IOMMUs will happily unmap already-unmapped pages, others
consider this an error, so fix it by computing the necessary alignment
padding without altering the actual allocation size. Also clean up by
making pad_size unsigned, since its callers always pass unsigned values
and negative padding makes little sense here anyway.

Signed-off-by: Robin Murphy <robin.murphy@arm.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
drivers/iommu/intel-iommu.c
drivers/iommu/iova.c