powerpc/powernv/ioda: Remove explicit max window size check
authorAlexey Kardashevskiy <aik@ozlabs.ru>
Tue, 7 Nov 2017 03:43:01 +0000 (14:43 +1100)
committerMichael Ellerman <mpe@ellerman.id.au>
Tue, 7 Nov 2017 12:28:25 +0000 (23:28 +1100)
DMA windows can only have a size of power of two on IODA2 hardware and
using memory_hotplug_max() to determine the upper limit won't work
correcly if it returns not power of two value.

This removes the check as the platform code does this check in
pnv_pci_ioda2_setup_default_config() anyway; the other client is VFIO
and that thing checks against locked_vm limit which prevents the userspace
from locking too much memory.

It is expected to impact DPDK on machines with non-power-of-two RAM size,
mostly. KVM guests are less likely to be affected as usually guests get
less than half of hosts RAM.

Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/platforms/powernv/pci-ioda.c

index 7e87867..7490555 100644 (file)
@@ -2797,7 +2797,7 @@ static long pnv_pci_ioda2_table_alloc_pages(int nid, __u64 bus_offset,
        if (!levels || (levels > POWERNV_IOMMU_MAX_LEVELS))
                return -EINVAL;
 
-       if ((window_size > memory_hotplug_max()) || !is_power_of_2(window_size))
+       if (!is_power_of_2(window_size))
                return -EINVAL;
 
        /* Adjust direct table size from window_size and levels */