linux-kernel - Re: [RFC PATCH v2 00/15] ACPI IORT ARM SMMU v3 support

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]

Date:	Wed, 22 Jun 2016 10:45:05 +0800
From:	Hanjun Guo <hanjun.guo@...aro.org>
To:	Lorenzo Pieralisi <lorenzo.pieralisi@....com>
Cc:	iommu@...ts.linux-foundation.org,
	Will Deacon <will.deacon@....com>,
	Marc Zyngier <marc.zyngier@....com>,
	Robin Murphy <robin.murphy@....com>,
	Joerg Roedel <joro@...tes.org>,
	"Rafael J. Wysocki" <rjw@...ysocki.net>,
	Tomasz Nowicki <tn@...ihalf.com>, Jon Masters <jcm@...hat.com>,
	Sinan Kaya <okaya@...eaurora.org>, linux-acpi@...r.kernel.org,
	linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org, linuxarm@...wei.com
Subject: Re: [RFC PATCH v2 00/15] ACPI IORT ARM SMMU v3 support

On 2016/6/21 22:27, Lorenzo Pieralisi wrote:
> Hi Hanjun,
>
> On Tue, Jun 21, 2016 at 06:37:17PM +0800, Hanjun Guo wrote:
>> Hi Lorenzo,
>>
>> On 2016/6/7 21:30, Lorenzo Pieralisi wrote:
>>> This RFC patch series is v2 of a previous posting:
>>>
>>> https://lkml.org/lkml/2016/4/14/702
>>>
>>> v1 -> v2:
>>> 	- Rebased on top of dependencies series [1][2][3](v4.7-rc1)
>>> 	- Removed IOMMU fwnode generalization
>>> 	- Implemented ARM SMMU v3 ACPI probing instead of ARM SMMU v2
>>> 	  owing to patch series dependencies [1]
>>> 	- Moved platform device creation logic to IORT code to
>>> 	  generalize its usage for ARM SMMU v1-v2-v3 components
>>> 	- Removed reliance on ACPI early device probing
>>> 	- Created IORT specific iommu_xlate() translation hook leaving
>>> 	  OF code unchanged according to v1 reviews
>>>
>>> The ACPI IORT table provides information that allows instantiating
>>> ARM SMMU devices and carrying out id mappings between components on
>>> ARM based systems (devices, IOMMUs, interrupt controllers).
>>>
>>> http://infocenter.arm.com/help/topic/com.arm.doc.den0049b/DEN0049B_IO_Remapping_Table.pdf
>>>
>>> Building on basic IORT support, available through [2]:
>>>
>>> this patchset enables ARM SMMU v3 support on ACPI systems.
>>
>> I'm trying to test your patches on D03 (SMMUv3 based) but ...
>
> What do you mean by "I am trying.." :), have you actually tested
> this series ?

I was rebasing this series on top of Tomasz's latest ITS patchset,
and found that patch 1~2,5 are not needed, and also have conflicts
(expected), so I think it's better to wait for the updated version
from you :)

>
>> [...]
>>> [1] R.Murphy "Generic DT bindings for PCI and ARM SMMU v3"
>>>    http://marc.info/?l=linux-arm-kernel&m=146497432413816&w=2
>>
>> ...This patch set is still in discussion and seems not work
>> for non-PCI devices.
>
> Can you be more specific please ? Yes, both series are work
> in progress.
>
>>> [2] T.Nowicki "Introduce ACPI world to ITS irqchip" v5
>>>    http://marc.info/?l=linux-acpi&m=146469369703684&w=2
>>
>> Tomasz sent out the v7 and included patches in your series.
>>
>> I think a updated version before the test makes sense, what
>> do you think? Let me know your thoughts.
>
> I am working with Robin so that the xlate() mechanism works
> properly and seamlessly for both DT and ACPI, given the
> dependencies I think it makes more sense to wait for Tomasz
> and Robin patches to get merged or at least stabilize before
> doing anything else, I am curently working on a v3.

OK.

Thanks
Hanjun