From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752404AbeCPEjh (ORCPT ); Fri, 16 Mar 2018 00:39:37 -0400 Received: from mga18.intel.com ([134.134.136.126]:43220 "EHLO mga18.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751343AbeCPEjf (ORCPT ); Fri, 16 Mar 2018 00:39:35 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.48,313,1517904000"; d="scan'208";a="24519130" From: Lu Baolu To: David Woodhouse , Joerg Roedel Cc: iommu@lists.linux-foundation.org, linux-kernel@vger.kernel.org, Lu Baolu , Jacob Pan , Kevin Tian , Sankaran Rajesh , Liu Yi L Subject: [PATCH 1/1] iommu/vt-d: Use real PASID for flush in caching mode Date: Fri, 16 Mar 2018 12:31:36 +0800 Message-Id: <1521174696-22495-1-git-send-email-baolu.lu@linux.intel.com> X-Mailer: git-send-email 2.7.4 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org If caching mode is supported, the hardware will cache none-present or erroneous translation entries. Hence, software should explicitly invalidate the PASID cache after a PASID table entry becomes present. We should issue such invalidation with the PASID value that we have changed. PASID 0 is not reserved for this case. Cc: Jacob Pan Cc: Kevin Tian Cc: Sankaran Rajesh Suggested-by: Ashok Raj Signed-off-by: Liu Yi L Signed-off-by: Lu Baolu --- drivers/iommu/intel-svm.c | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/drivers/iommu/intel-svm.c b/drivers/iommu/intel-svm.c index 99bc9bd..b7b88b5 100644 --- a/drivers/iommu/intel-svm.c +++ b/drivers/iommu/intel-svm.c @@ -422,17 +422,13 @@ int intel_svm_bind_mm(struct device *dev, int *pasid, int flags, struct svm_dev_ iommu->pasid_table[svm->pasid].val = pasid_entry_val; wmb(); - /* In caching mode, we still have to flush with PASID 0 when - * a PASID table entry becomes present. Not entirely clear - * *why* that would be the case — surely we could just issue - * a flush with the PASID value that we've changed? The PASID - * is the index into the table, after all. It's not like domain - * IDs in the case of the equivalent context-entry change in - * caching mode. And for that matter it's not entirely clear why - * a VMM would be in the business of caching the PASID table - * anyway. Surely that can be left entirely to the guest? */ + + /* + * Flush PASID cache when a PASID table entry becomes + * present. + */ if (cap_caching_mode(iommu->cap)) - intel_flush_pasid_dev(svm, sdev, 0); + intel_flush_pasid_dev(svm, sdev, svm->pasid); } list_add_rcu(&sdev->list, &svm->devs); -- 2.7.4 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lu Baolu Subject: [PATCH 1/1] iommu/vt-d: Use real PASID for flush in caching mode Date: Fri, 16 Mar 2018 12:31:36 +0800 Message-ID: <1521174696-22495-1-git-send-email-baolu.lu@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: David Woodhouse , Joerg Roedel Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org List-Id: iommu@lists.linux-foundation.org SWYgY2FjaGluZyBtb2RlIGlzIHN1cHBvcnRlZCwgdGhlIGhhcmR3YXJlIHdpbGwgY2FjaGUKbm9u ZS1wcmVzZW50IG9yIGVycm9uZW91cyB0cmFuc2xhdGlvbiBlbnRyaWVzLiBIZW5jZSwKc29mdHdh cmUgc2hvdWxkIGV4cGxpY2l0bHkgaW52YWxpZGF0ZSB0aGUgUEFTSUQgY2FjaGUKYWZ0ZXIgYSBQ QVNJRCB0YWJsZSBlbnRyeSBiZWNvbWVzIHByZXNlbnQuIFdlIHNob3VsZAppc3N1ZSBzdWNoIGlu dmFsaWRhdGlvbiB3aXRoIHRoZSBQQVNJRCB2YWx1ZSB0aGF0IHdlCmhhdmUgY2hhbmdlZC4gUEFT SUQgMCBpcyBub3QgcmVzZXJ2ZWQgZm9yIHRoaXMgY2FzZS4KCkNjOiBKYWNvYiBQYW4gPGphY29i Lmp1bi5wYW5AbGludXguaW50ZWwuY29tPgpDYzogS2V2aW4gVGlhbiA8a2V2aW4udGlhbkBpbnRl bC5jb20+CkNjOiBTYW5rYXJhbiBSYWplc2ggPHJhamVzaC5zYW5rYXJhbkBpbnRlbC5jb20+ClN1 Z2dlc3RlZC1ieTogQXNob2sgUmFqIDxhc2hvay5yYWpAaW50ZWwuY29tPgpTaWduZWQtb2ZmLWJ5 OiBMaXUgWWkgTCA8eWkubC5saXVAaW50ZWwuY29tPgpTaWduZWQtb2ZmLWJ5OiBMdSBCYW9sdSA8 YmFvbHUubHVAbGludXguaW50ZWwuY29tPgotLS0KIGRyaXZlcnMvaW9tbXUvaW50ZWwtc3ZtLmMg fCAxNiArKysrKystLS0tLS0tLS0tCiAxIGZpbGUgY2hhbmdlZCwgNiBpbnNlcnRpb25zKCspLCAx MCBkZWxldGlvbnMoLSkKCmRpZmYgLS1naXQgYS9kcml2ZXJzL2lvbW11L2ludGVsLXN2bS5jIGIv ZHJpdmVycy9pb21tdS9pbnRlbC1zdm0uYwppbmRleCA5OWJjOWJkLi5iN2I4OGI1IDEwMDY0NAot LS0gYS9kcml2ZXJzL2lvbW11L2ludGVsLXN2bS5jCisrKyBiL2RyaXZlcnMvaW9tbXUvaW50ZWwt c3ZtLmMKQEAgLTQyMiwxNyArNDIyLDEzIEBAIGludCBpbnRlbF9zdm1fYmluZF9tbShzdHJ1Y3Qg ZGV2aWNlICpkZXYsIGludCAqcGFzaWQsIGludCBmbGFncywgc3RydWN0IHN2bV9kZXZfCiAJCWlv bW11LT5wYXNpZF90YWJsZVtzdm0tPnBhc2lkXS52YWwgPSBwYXNpZF9lbnRyeV92YWw7CiAKIAkJ d21iKCk7Ci0JCS8qIEluIGNhY2hpbmcgbW9kZSwgd2Ugc3RpbGwgaGF2ZSB0byBmbHVzaCB3aXRo IFBBU0lEIDAgd2hlbgotCQkgKiBhIFBBU0lEIHRhYmxlIGVudHJ5IGJlY29tZXMgcHJlc2VudC4g Tm90IGVudGlyZWx5IGNsZWFyCi0JCSAqICp3aHkqIHRoYXQgd291bGQgYmUgdGhlIGNhc2Ug4oCU IHN1cmVseSB3ZSBjb3VsZCBqdXN0IGlzc3VlCi0JCSAqIGEgZmx1c2ggd2l0aCB0aGUgUEFTSUQg dmFsdWUgdGhhdCB3ZSd2ZSBjaGFuZ2VkPyBUaGUgUEFTSUQKLQkJICogaXMgdGhlIGluZGV4IGlu dG8gdGhlIHRhYmxlLCBhZnRlciBhbGwuIEl0J3Mgbm90IGxpa2UgZG9tYWluCi0JCSAqIElEcyBp biB0aGUgY2FzZSBvZiB0aGUgZXF1aXZhbGVudCBjb250ZXh0LWVudHJ5IGNoYW5nZSBpbgotCQkg KiBjYWNoaW5nIG1vZGUuIEFuZCBmb3IgdGhhdCBtYXR0ZXIgaXQncyBub3QgZW50aXJlbHkgY2xl YXIgd2h5Ci0JCSAqIGEgVk1NIHdvdWxkIGJlIGluIHRoZSBidXNpbmVzcyBvZiBjYWNoaW5nIHRo ZSBQQVNJRCB0YWJsZQotCQkgKiBhbnl3YXkuIFN1cmVseSB0aGF0IGNhbiBiZSBsZWZ0IGVudGly ZWx5IHRvIHRoZSBndWVzdD8gKi8KKworCQkvKgorCQkgKiBGbHVzaCBQQVNJRCBjYWNoZSB3aGVu IGEgUEFTSUQgdGFibGUgZW50cnkgYmVjb21lcworCQkgKiBwcmVzZW50LgorCQkgKi8KIAkJaWYg KGNhcF9jYWNoaW5nX21vZGUoaW9tbXUtPmNhcCkpCi0JCQlpbnRlbF9mbHVzaF9wYXNpZF9kZXYo c3ZtLCBzZGV2LCAwKTsKKwkJCWludGVsX2ZsdXNoX3Bhc2lkX2Rldihzdm0sIHNkZXYsIHN2bS0+ cGFzaWQpOwogCX0KIAlsaXN0X2FkZF9yY3UoJnNkZXYtPmxpc3QsICZzdm0tPmRldnMpOwogCi0t IAoyLjcuNAoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18K aW9tbXUgbWFpbGluZyBsaXN0CmlvbW11QGxpc3RzLmxpbnV4LWZvdW5kYXRpb24ub3JnCmh0dHBz Oi8vbGlzdHMubGludXhmb3VuZGF0aW9uLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2lvbW11