Re: [PATCH v3 5/6] iommu/vt-d: Flush PASID-based iotlb for iova over first level

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v3 5/6] iommu/vt-d: Flush PASID-based iotlb for iova over first level
From: Lu Baolu <baolu.lu@xxxxxxxxxxxxxxx>
Date: Tue, 17 Dec 2019 09:39:12 +0800
Cc: baolu.lu@xxxxxxxxxxxxxxx, "Raj, Ashok" <ashok.raj@xxxxxxxxx>, "Kumar, Sanjay K" <sanjay.k.kumar@xxxxxxxxx>, "jacob.jun.pan@xxxxxxxxxxxxxxx" <jacob.jun.pan@xxxxxxxxxxxxxxx>, "Tian, Kevin" <kevin.tian@xxxxxxxxx>, "Sun, Yi Y" <yi.y.sun@xxxxxxxxx>, Peter Xu <peterx@xxxxxxxxxx>, "iommu@xxxxxxxxxxxxxxxxxxxxxxxxxx" <iommu@xxxxxxxxxxxxxxxxxxxxxxxxxx>, "kvm@xxxxxxxxxxxxxxx" <kvm@xxxxxxxxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>
In-reply-to: <b4a879b2-a5c7-b0bf-8cd4-7397aeebc381@linux.intel.com>
References: <20191211021219.8997-1-baolu.lu@linux.intel.com> <20191211021219.8997-6-baolu.lu@linux.intel.com> <A2975661238FB949B60364EF0F2C25743A130C08@SHSMSX104.ccr.corp.intel.com> <f1e5cfea-8b11-6d72-8e57-65daea51c050@linux.intel.com> <A2975661238FB949B60364EF0F2C25743A132C50@SHSMSX104.ccr.corp.intel.com> <6a5f6695-d1fd-e7d1-3ea3-f222a1ef0e54@linux.intel.com> <b4a879b2-a5c7-b0bf-8cd4-7397aeebc381@linux.intel.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.1

Hi,

On 12/17/19 9:37 AM, Lu Baolu wrote:

You are right. I will change it accordingly. The logic should look
like:

if (domain attached to physical device)
     flush_piotlb_with_RID2PASID()
else if (domain_attached_to_mdev_device)
     flush_piotlb_with_default_pasid()


Both! so no "else" here.

Best regards,
baolu

Does this work for you? Thanks for catching this!

Follow-Ups:
- RE: [PATCH v3 5/6] iommu/vt-d: Flush PASID-based iotlb for iova over first level
  - From: Liu, Yi L

References:
- [PATCH v3 0/6] Use 1st-level for IOVA translation
  - From: Lu Baolu
- [PATCH v3 5/6] iommu/vt-d: Flush PASID-based iotlb for iova over first level
  - From: Lu Baolu
- RE: [PATCH v3 5/6] iommu/vt-d: Flush PASID-based iotlb for iova over first level
  - From: Liu, Yi L
- Re: [PATCH v3 5/6] iommu/vt-d: Flush PASID-based iotlb for iova over first level
  - From: Lu Baolu
- RE: [PATCH v3 5/6] iommu/vt-d: Flush PASID-based iotlb for iova over first level
  - From: Liu, Yi L
- Re: [PATCH v3 5/6] iommu/vt-d: Flush PASID-based iotlb for iova over first level
  - From: Lu Baolu
- Re: [PATCH v3 5/6] iommu/vt-d: Flush PASID-based iotlb for iova over first level
  - From: Lu Baolu

Prev by Date: Re: [PATCH v3 5/6] iommu/vt-d: Flush PASID-based iotlb for iova over first level
Next by Date: Re: [PATCH 3/3] kvm/arm: Standardize kvm exit reason field
Previous by thread: Re: [PATCH v3 5/6] iommu/vt-d: Flush PASID-based iotlb for iova over first level
Next by thread: RE: [PATCH v3 5/6] iommu/vt-d: Flush PASID-based iotlb for iova over first level
Index(es):
- Date
- Thread

[Index of Archives] [KVM ARM] [KVM ia64] [KVM ppc] [Virtualization Tools] [Spice Development] [Libvirt] [Libvirt Users] [Linux USB Devel] [Linux Audio Users] [Yosemite Questions] [Linux Kernel] [Linux SCSI] [XFree86]