From: Auger Eric <eric.auger@redhat.com> To: Jacob Pan <jacob.pan.linux@gmail.com>, iommu@lists.linux-foundation.org, LKML <linux-kernel@vger.kernel.org>, Jean-Philippe Brucker <jean-philippe@linaro.com>, Lu Baolu <baolu.lu@linux.intel.com>, Joerg Roedel <joro@8bytes.org>, David Woodhouse <dwmw2@infradead.org> Cc: Yi Liu <yi.l.liu@intel.com>, "Tian, Kevin" <kevin.tian@intel.com>, Raj Ashok <ashok.raj@intel.com>, Wu Hao <hao.wu@intel.com> Subject: Re: [PATCH v2 9/9] iommu/vt-d: Store guest PASID during bind Date: Tue, 1 Sep 2020 19:08:44 +0200 [thread overview] Message-ID: <60feda75-5862-c898-97b1-1f5eafdb8d8c@redhat.com> (raw) In-Reply-To: <1598070918-21321-10-git-send-email-jacob.jun.pan@linux.intel.com> Hi Jacob, On 8/22/20 6:35 AM, Jacob Pan wrote: > IOASID core maintains the guest-host mapping in the form of SPID and > IOASID. This patch assigns the guest PASID (if valid) as SPID while > binding guest page table with a host PASID. This mapping will be used > for lookup and notifications. > > Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com> > --- > drivers/iommu/intel/svm.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/iommu/intel/svm.c b/drivers/iommu/intel/svm.c > index d8a5efa75095..4c958b1aec4c 100644 > --- a/drivers/iommu/intel/svm.c > +++ b/drivers/iommu/intel/svm.c > @@ -406,6 +406,7 @@ int intel_svm_bind_gpasid(struct iommu_domain *domain, struct device *dev, > if (data->flags & IOMMU_SVA_GPASID_VAL) { > svm->gpasid = data->gpasid; > svm->flags |= SVM_FLAG_GUEST_PASID; > + ioasid_attach_spid(data->hpasid, data->gpasid); don't you want to handle the returned value? > } > svm->iommu = iommu; > /* > @@ -517,6 +518,7 @@ int intel_svm_unbind_gpasid(struct device *dev, int pasid) > ioasid_attach_data(pasid, NULL); > ioasid_notify(pasid, IOASID_UNBIND, > IOASID_NOTIFY_SET); > + ioasid_attach_spid(pasid, INVALID_IOASID); So this answers my previous question ;-) but won't it enter the if (!ioasid_data) path and fail to reset the spid? Eric > kfree(svm); > } > } >
WARNING: multiple messages have this Message-ID (diff)
From: Auger Eric <eric.auger@redhat.com> To: Jacob Pan <jacob.pan.linux@gmail.com>, iommu@lists.linux-foundation.org, LKML <linux-kernel@vger.kernel.org>, Jean-Philippe Brucker <jean-philippe@linaro.com>, Lu Baolu <baolu.lu@linux.intel.com>, Joerg Roedel <joro@8bytes.org>, David Woodhouse <dwmw2@infradead.org> Cc: "Tian, Kevin" <kevin.tian@intel.com>, Raj Ashok <ashok.raj@intel.com>, Wu Hao <hao.wu@intel.com> Subject: Re: [PATCH v2 9/9] iommu/vt-d: Store guest PASID during bind Date: Tue, 1 Sep 2020 19:08:44 +0200 [thread overview] Message-ID: <60feda75-5862-c898-97b1-1f5eafdb8d8c@redhat.com> (raw) In-Reply-To: <1598070918-21321-10-git-send-email-jacob.jun.pan@linux.intel.com> Hi Jacob, On 8/22/20 6:35 AM, Jacob Pan wrote: > IOASID core maintains the guest-host mapping in the form of SPID and > IOASID. This patch assigns the guest PASID (if valid) as SPID while > binding guest page table with a host PASID. This mapping will be used > for lookup and notifications. > > Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com> > --- > drivers/iommu/intel/svm.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/iommu/intel/svm.c b/drivers/iommu/intel/svm.c > index d8a5efa75095..4c958b1aec4c 100644 > --- a/drivers/iommu/intel/svm.c > +++ b/drivers/iommu/intel/svm.c > @@ -406,6 +406,7 @@ int intel_svm_bind_gpasid(struct iommu_domain *domain, struct device *dev, > if (data->flags & IOMMU_SVA_GPASID_VAL) { > svm->gpasid = data->gpasid; > svm->flags |= SVM_FLAG_GUEST_PASID; > + ioasid_attach_spid(data->hpasid, data->gpasid); don't you want to handle the returned value? > } > svm->iommu = iommu; > /* > @@ -517,6 +518,7 @@ int intel_svm_unbind_gpasid(struct device *dev, int pasid) > ioasid_attach_data(pasid, NULL); > ioasid_notify(pasid, IOASID_UNBIND, > IOASID_NOTIFY_SET); > + ioasid_attach_spid(pasid, INVALID_IOASID); So this answers my previous question ;-) but won't it enter the if (!ioasid_data) path and fail to reset the spid? Eric > kfree(svm); > } > } > _______________________________________________ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
next prev parent reply other threads:[~2020-09-01 17:09 UTC|newest] Thread overview: 111+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-08-22 4:35 [PATCH v2 0/9] IOASID extensions for guest SVA Jacob Pan 2020-08-22 4:35 ` Jacob Pan 2020-08-22 4:35 ` [PATCH v2 1/9] docs: Document IO Address Space ID (IOASID) APIs Jacob Pan 2020-08-22 4:35 ` Jacob Pan 2020-08-23 7:05 ` Lu Baolu 2020-08-23 7:05 ` Lu Baolu 2020-08-28 17:01 ` Jacob Pan 2020-08-28 17:01 ` Jacob Pan 2020-08-24 10:32 ` Jean-Philippe Brucker 2020-08-24 10:32 ` Jean-Philippe Brucker 2020-08-27 16:21 ` Auger Eric 2020-08-27 16:21 ` Auger Eric 2020-09-01 16:56 ` Jacob Pan 2020-09-01 16:56 ` Jacob Pan 2020-09-07 8:03 ` Auger Eric 2020-09-07 8:03 ` Auger Eric 2020-09-08 17:29 ` Jacob Pan 2020-08-28 22:24 ` Jacob Pan 2020-08-28 22:24 ` Jacob Pan 2020-08-22 4:35 ` [PATCH v2 2/9] iommu/ioasid: Rename ioasid_set_data() Jacob Pan 2020-08-22 4:35 ` Jacob Pan 2020-08-24 18:29 ` Jean-Philippe Brucker 2020-08-24 18:29 ` Jean-Philippe Brucker 2020-09-01 11:51 ` Auger Eric 2020-09-01 11:51 ` Auger Eric 2020-08-22 4:35 ` [PATCH v2 3/9] iommu/ioasid: Introduce ioasid_set APIs Jacob Pan 2020-08-22 4:35 ` Jacob Pan 2020-08-22 12:53 ` kernel test robot 2020-08-22 12:53 ` kernel test robot 2020-08-22 12:53 ` kernel test robot 2020-08-24 2:24 ` Lu Baolu 2020-08-24 2:24 ` Lu Baolu 2020-09-01 21:28 ` Jacob Pan 2020-09-01 21:28 ` Jacob Pan 2020-09-02 2:39 ` Lu Baolu 2020-09-02 2:39 ` Lu Baolu 2020-08-24 18:28 ` Jean-Philippe Brucker 2020-08-24 18:28 ` Jean-Philippe Brucker 2020-08-24 18:30 ` Randy Dunlap 2020-08-24 18:30 ` Randy Dunlap 2020-09-02 21:46 ` Jacob Pan 2020-09-02 21:46 ` Jacob Pan 2020-08-24 18:34 ` Randy Dunlap 2020-08-24 18:34 ` Randy Dunlap 2020-09-02 21:47 ` Jacob Pan 2020-09-02 21:47 ` Jacob Pan 2020-09-02 21:44 ` Jacob Pan 2020-09-02 21:44 ` Jacob Pan 2020-09-01 11:51 ` Auger Eric 2020-09-01 11:51 ` Auger Eric 2020-09-03 21:07 ` Jacob Pan 2020-09-03 21:07 ` Jacob Pan 2020-09-07 8:04 ` Auger Eric 2020-09-07 8:04 ` Auger Eric 2020-08-22 4:35 ` [PATCH v2 4/9] iommu/ioasid: Add reference couting functions Jacob Pan 2020-08-22 4:35 ` Jacob Pan 2020-08-24 2:26 ` Lu Baolu 2020-08-24 2:26 ` Lu Baolu 2020-08-25 10:20 ` Jean-Philippe Brucker 2020-08-25 10:20 ` Jean-Philippe Brucker 2020-08-25 10:19 ` Jean-Philippe Brucker 2020-08-25 10:19 ` Jean-Philippe Brucker 2020-09-08 20:30 ` Jacob Pan 2020-09-01 12:13 ` Auger Eric 2020-09-01 12:13 ` Auger Eric 2020-09-08 20:49 ` Jacob Pan 2020-09-24 18:29 ` Shameerali Kolothum Thodi 2020-09-24 18:29 ` Shameerali Kolothum Thodi 2020-08-22 4:35 ` [PATCH v2 5/9] iommu/ioasid: Introduce ioasid_set private ID Jacob Pan 2020-08-22 4:35 ` Jacob Pan 2020-08-22 8:36 ` kernel test robot 2020-08-22 8:36 ` kernel test robot 2020-08-22 8:36 ` kernel test robot 2020-08-22 9:03 ` kernel test robot 2020-08-22 9:03 ` kernel test robot 2020-08-22 9:03 ` kernel test robot 2020-08-25 10:22 ` Jean-Philippe Brucker 2020-08-25 10:22 ` Jean-Philippe Brucker 2020-09-08 22:19 ` Jacob Pan 2020-09-08 22:19 ` Jacob Pan 2020-09-01 15:38 ` Auger Eric 2020-09-01 15:38 ` Auger Eric 2020-09-08 22:40 ` Jacob Pan 2020-09-08 22:40 ` Jacob Pan 2020-09-10 9:18 ` Auger Eric 2020-09-10 9:18 ` Auger Eric 2020-08-22 4:35 ` [PATCH v2 6/9] iommu/ioasid: Introduce notification APIs Jacob Pan 2020-08-22 4:35 ` Jacob Pan 2020-08-25 10:26 ` Jean-Philippe Brucker 2020-08-25 10:26 ` Jean-Philippe Brucker 2020-09-09 20:37 ` Jacob Pan 2020-09-09 20:37 ` Jacob Pan 2020-09-01 16:49 ` Auger Eric 2020-09-01 16:49 ` Auger Eric 2020-09-09 22:58 ` Jacob Pan 2020-09-09 22:58 ` Jacob Pan 2020-09-10 8:59 ` Auger Eric 2020-09-10 8:59 ` Auger Eric 2020-08-22 4:35 ` [PATCH v2 7/9] iommu/vt-d: Listen to IOASID notifications Jacob Pan 2020-08-22 4:35 ` Jacob Pan 2020-09-01 17:03 ` Auger Eric 2020-09-01 17:03 ` Auger Eric 2020-09-10 4:54 ` Jacob Pan 2020-08-22 4:35 ` [PATCH v2 8/9] iommu/vt-d: Send IOASID bind/unbind notifications Jacob Pan 2020-08-22 4:35 ` Jacob Pan 2020-08-22 4:35 ` [PATCH v2 9/9] iommu/vt-d: Store guest PASID during bind Jacob Pan 2020-08-22 4:35 ` Jacob Pan 2020-09-01 17:08 ` Auger Eric [this message] 2020-09-01 17:08 ` Auger Eric 2020-09-10 17:12 ` Jacob Pan 2020-09-10 17:12 ` Jacob Pan
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=60feda75-5862-c898-97b1-1f5eafdb8d8c@redhat.com \ --to=eric.auger@redhat.com \ --cc=ashok.raj@intel.com \ --cc=baolu.lu@linux.intel.com \ --cc=dwmw2@infradead.org \ --cc=hao.wu@intel.com \ --cc=iommu@lists.linux-foundation.org \ --cc=jacob.pan.linux@gmail.com \ --cc=jean-philippe@linaro.com \ --cc=joro@8bytes.org \ --cc=kevin.tian@intel.com \ --cc=linux-kernel@vger.kernel.org \ --cc=yi.l.liu@intel.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.