Re: [PATCH v2 1/2] iommu/dma: Stop getting dma_32bit_pfn wrong
On Mon, Jan 16, 2017 at 01:24:54PM +, Robin Murphy wrote: > iommu_dma_init_domain() was originally written under the misconception > that dma_32bit_pfn represented some sort of size limit for IOVA domains. > Since the truth is almost the exact opposite of that, rework the logic > and comments to reflect its real purpose of optimising lookups when > allocating from a subset of the available 64-bit space. > > Signed-off-by: Robin Murphy> --- > > Sending this as a v2 since both patches have been seen before, and #1 is > ever so slightly tweaked. #2 applies on top of Eric's MSI series, since > that seems ready to go now - there is a trivial merge conflict otherwise > around the extra argument in the __alloc_iova() call. > > Robin. > > drivers/iommu/dma-iommu.c | 23 ++- > 1 file changed, 18 insertions(+), 5 deletions(-) Applied both, thanks. ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH v2 1/2] iommu/dma: Stop getting dma_32bit_pfn wrong
On Mon, Jan 16, 2017 at 01:24:54PM +, Robin Murphy wrote: > iommu_dma_init_domain() was originally written under the misconception > that dma_32bit_pfn represented some sort of size limit for IOVA domains. > Since the truth is almost the exact opposite of that, rework the logic > and comments to reflect its real purpose of optimising lookups when > allocating from a subset of the available 64-bit space. > > Signed-off-by: Robin Murphy> --- > > Sending this as a v2 since both patches have been seen before, and #1 is > ever so slightly tweaked. #2 applies on top of Eric's MSI series, since > that seems ready to go now - there is a trivial merge conflict otherwise > around the extra argument in the __alloc_iova() call. > > Robin. > > drivers/iommu/dma-iommu.c | 23 ++- > 1 file changed, 18 insertions(+), 5 deletions(-) Tested-by: Will Deacon Will ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu