linux-kernel - Re: [RESEND PATCH 0/4] Add support for SMMU ECMDQ

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <20230724103111.0000131d@Huawei.com>
Date:   Mon, 24 Jul 2023 10:31:11 +0100
From:   Jonathan Cameron <Jonathan.Cameron@...wei.com>
To:     Tanmay Jagdale <tanmay@...vell.com>
CC:     <will@...nel.org>, <robin.murphy@....com>, <joro@...tes.org>,
        <baolu.lu@...ux.intel.com>, <thunder.leizhen@...wei.com>,
        <linux-arm-kernel@...ts.infradead.org>, <iommu@...ts.linux.dev>,
        <linux-kernel@...r.kernel.org>, <gcherian@...vell.com>,
        <sgoutham@...vell.com>, <lcherian@...vell.com>,
        <bbhushan2@...vell.com>
Subject: Re: [RESEND PATCH 0/4] Add support for SMMU ECMDQ

On Fri, 21 Jul 2023 02:35:09 -0400
Tanmay Jagdale <tanmay@...vell.com> wrote:

> Resending the patches by Zhen Lei <thunder.leizhen@...wei.com> that add
> support for SMMU's ECMDQ feature.
> 
> I have tested these patches on Marvell SoC that has the ECMDQ feature.

Hi Tammay,

If sending someone else's series you are 'handling' the patches, so should
add your own SoB after the original author's one.  Also, given you've tested them, a
Tested-by tag would make sense.

Any perf numbers you can share would also help push this forwards by showing
why people care.  I believe Leizhen's numbers were emulation based, so
not as good as real hardware for making the justification for the added
complexity!

Jonathan


> 
> Zhen Lei (4):
>   iommu/arm-smmu-v3: Add support for ECMDQ register mode
>   iommu/arm-smmu-v3: Ensure that a set of associated commands are
>     inserted in the same ECMDQ
>   iommu/arm-smmu-v3: Add arm_smmu_ecmdq_issue_cmdlist() for non-shared
>     ECMDQ
>   iommu/arm-smmu-v3: Add support for less than one ECMDQ per core
> 
>  drivers/iommu/arm/arm-smmu-v3/arm-smmu-v3.c | 419 +++++++++++++++++++-
>  drivers/iommu/arm/arm-smmu-v3/arm-smmu-v3.h |  37 ++
>  2 files changed, 448 insertions(+), 8 deletions(-)
>