Re: [PATCH v14 06/13] iommu/smmuv3: Allow stage 1 invalidation with unmanaged ASIDs

2021-04-09 Thread Kunkun Jiang
On 2021/4/9 16:31, Auger Eric wrote: Hi Kunkun, On 4/9/21 6:48 AM, Kunkun Jiang wrote: Hi Eric, On 2021/4/8 20:30, Auger Eric wrote: Hi Kunkun, On 4/1/21 2:37 PM, Kunkun Jiang wrote: Hi Eric, On 2021/2/24 4:56, Eric Auger wrote: With nested stage support, soon we will need to invalidate

Re: [PATCH v14 06/13] iommu/smmuv3: Allow stage 1 invalidation with unmanaged ASIDs

2021-04-08 Thread Kunkun Jiang
Hi Eric, On 2021/4/8 20:30, Auger Eric wrote: Hi Kunkun, On 4/1/21 2:37 PM, Kunkun Jiang wrote: Hi Eric, On 2021/2/24 4:56, Eric Auger wrote: With nested stage support, soon we will need to invalidate S1 contexts and ranges tagged with an unmanaged asid, this latter being managed by the

Re: [PATCH v14 06/13] iommu/smmuv3: Allow stage 1 invalidation with unmanaged ASIDs

2021-04-01 Thread Kunkun Jiang
(smmu_domain-> domain.pgsize_bitmap). In nested mode, it is determined by host PAGE_SIZE. If the host kernel and guest kernel has different translation granule (e.g. host 16K, guest 4K), __arm_smmu_tlb_inv_range() will issue an incorrect tlbi command. Do you have any idea

[PATCH] kvm: Fixes lack of KVM_CAP_MANUAL_DIRTY_LOG_PROTECT2 enabled check

2021-01-12 Thread Kunkun Jiang
5353 ("kvm: introduce manual dirty log reprotect") Signed-off-by: Kunkun Jiang --- virt/kvm/kvm_main.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index fa9e3614d30e..8f5633d8a0e8 100644 --- a/virt/kvm/kvm_main.c +++ b/virt/kvm/kvm_main.

Re: [PATCH v11 04/13] vfio/pci: Add VFIO_REGION_TYPE_NESTED region type

2020-12-17 Thread Kunkun Jiang
macro *define VFIO_REGION_TYPE_NESTED    (2)* is in conflict with *#define VFIO_REGION_TYPE_CCW    (2)*. Thanks, Kunkun Jiang.

Re: [PATCH] iommu/io-pgtalbe-arm: Remove "iopte_type(pte, l)" extra parameter "l"

2020-12-07 Thread Kunkun Jiang
Hi Will, On 2020/12/7 18:24, Will Deacon wrote: On Mon, Dec 07, 2020 at 04:14:04PM +0800, Kunkun Jiang wrote: Knowing from the code, the macro "iopte_type(pte, l)" doesn't use the parameter "l" (level). So we'd better to remove it. Fixes: e1d3c0fd701df(iommu: add

[PATCH RESEND] iommu/io-pgtalbe-arm: Remove "iopte_type(pte, l)" extra parameter "l"

2020-12-07 Thread Kunkun Jiang
Knowing from the code, the macro "iopte_type(pte, l)" doesn't use the parameter "l" (level). So we'd better to remove it. Fixes: e1d3c0fd701df(iommu: add ARM LPAE page table allocator) Signed-off-by: Kunkun Jiang --- drivers/iommu/io-pgtable-arm.c | 10 +-

[PATCH] iommu/io-pgtalbe-arm: Remove "iopte_type(pte, l)" extra parameter "l"

2020-12-07 Thread Kunkun Jiang
Knowing from the code, the macro "iopte_type(pte, l)" doesn't use the parameter "l" (level). So we'd better to remove it. Fixes: e1d3c0fd701df(iommu: add ARM LPAE page table allocator) Signed-off-by: Kunkun Jiang --- drivers/iommu/io-pgtable-arm.c | 2 +- 1 f

Re: [PATCH v13 05/15] iommu/smmuv3: Get prepared for nested stage support

2020-12-03 Thread Kunkun Jiang
On 2020/12/3 21:01, Auger Eric wrote: Hi Kunkun, On 12/3/20 1:32 PM, Kunkun Jiang wrote: Hi Eric, On 2020/11/18 19:21, Eric Auger wrote: When nested stage translation is setup, both s1_cfg and s2_cfg are set. We introduce a new smmu domain abort field that will be set upon guest stage1