linux-kernel - Re: [PATCH v5 0/6] Optimise 64-bit IOVA allocations

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20170927151133.bsdalf67br3p3hj2@8bytes.org>
Date:   Wed, 27 Sep 2017 17:11:33 +0200
From:   Joerg Roedel <joro@...tes.org>
To:     Robin Murphy <robin.murphy@....com>
Cc:     iommu@...ts.linux-foundation.org, thunder.leizhen@...wei.com,
        nwatters@...eaurora.org, tomasz.nowicki@...iumnetworks.com,
        dwoods@...lanox.com, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v5 0/6] Optimise 64-bit IOVA allocations

On Thu, Sep 21, 2017 at 04:52:41PM +0100, Robin Murphy wrote:
> v4: https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1493704.html
> 
> Right, this is hopefully the last version - I've put things back in a
> sensible order with the new additions at the end, so if they prove
> contentious the first 4 previously-tested patches can still get their
> time in -next. Patch #3 is updated to fix the bug brought to light by
> Tomasz, patch #6 includes the cleanup afterthought from v4, and I've
> nobbled one or two rogue whitespace changes I'd missed before.
> 
> And the diffstat is still even more negative than before, hooray! For
> the whole series, the total code size reduction of alloc_iova() comes
> to just over 26% (AArch64 GCC 6.3.1).
> 
> Robin.
> 
> 
> Robin Murphy (3):
>   iommu/iova: Extend rbtree node caching
>   iommu/iova: Add rbtree anchor node
>   iommu/iova: Simplify cached node logic
> 
> Zhen Lei (3):
>   iommu/iova: Optimise rbtree searching
>   iommu/iova: Optimise the padding calculation
>   iommu/iova: Make dma_32bit_pfn implicit

Applied, thanks Robin.