summaryrefslogtreecommitdiffstats
path: root/COPYING
diff options
context:
space:
mode:
authorTomasz Nowicki <tomasz.nowicki@caviumnetworks.com>2017-09-20 10:52:02 +0200
committerJoerg Roedel <jroedel@suse.de>2017-10-12 14:18:02 +0200
commit538d5b333216c3daa7a5821307164f10af73ec8c (patch)
treed38cb857a6bb78913797588854ffa4b691f01bfb /COPYING
parent4d689b619445894f6b6fcbc496f6d302bd9e44a5 (diff)
downloadlinux-stable-538d5b333216c3daa7a5821307164f10af73ec8c.tar.gz
linux-stable-538d5b333216c3daa7a5821307164f10af73ec8c.tar.bz2
linux-stable-538d5b333216c3daa7a5821307164f10af73ec8c.zip
iommu/iova: Make rcache flush optional on IOVA allocation failure
Since IOVA allocation failure is not unusual case we need to flush CPUs' rcache in hope we will succeed in next round. However, it is useful to decide whether we need rcache flush step because of two reasons: - Not scalability. On large system with ~100 CPUs iterating and flushing rcache for each CPU becomes serious bottleneck so we may want to defer it. - free_cpu_cached_iovas() does not care about max PFN we are interested in. Thus we may flush our rcaches and still get no new IOVA like in the commonly used scenario: if (dma_limit > DMA_BIT_MASK(32) && dev_is_pci(dev)) iova = alloc_iova_fast(iovad, iova_len, DMA_BIT_MASK(32) >> shift); if (!iova) iova = alloc_iova_fast(iovad, iova_len, dma_limit >> shift); 1. First alloc_iova_fast() call is limited to DMA_BIT_MASK(32) to get PCI devices a SAC address 2. alloc_iova() fails due to full 32-bit space 3. rcaches contain PFNs out of 32-bit space so free_cpu_cached_iovas() throws entries away for nothing and alloc_iova() fails again 4. Next alloc_iova_fast() call cannot take advantage of rcache since we have just defeated caches. In this case we pick the slowest option to proceed. This patch reworks flushed_rcache local flag to be additional function argument instead and control rcache flush step. Also, it updates all users to do the flush as the last chance. Signed-off-by: Tomasz Nowicki <Tomasz.Nowicki@caviumnetworks.com> Reviewed-by: Robin Murphy <robin.murphy@arm.com> Tested-by: Nate Watterson <nwatters@codeaurora.org> Signed-off-by: Joerg Roedel <jroedel@suse.de>
Diffstat (limited to 'COPYING')
0 files changed, 0 insertions, 0 deletions