Messages in this thread | | | Date | Mon, 24 Jun 2019 16:59:51 -0700 | From | Jacob Pan <> | Subject | Re: [PATCH v4 19/22] iommu/vt-d: Clean up for SVM device list |
| |
On Tue, 18 Jun 2019 17:42:37 +0100 Jonathan Cameron <jonathan.cameron@huawei.com> wrote:
> On Sun, 9 Jun 2019 06:44:19 -0700 > Jacob Pan <jacob.jun.pan@linux.intel.com> wrote: > > > Use combined macro for_each_svm_dev() to simplify SVM device > > iteration. > > > > Suggested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > > Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com> > > Reviewed-by: Eric Auger <eric.auger@redhat.com> > > --- > > drivers/iommu/intel-svm.c | 79 > > +++++++++++++++++++++++------------------------ 1 file changed, 39 > > insertions(+), 40 deletions(-) > > > > diff --git a/drivers/iommu/intel-svm.c b/drivers/iommu/intel-svm.c > > index 9cbcc1f..66d98e1 100644 > > --- a/drivers/iommu/intel-svm.c > > +++ b/drivers/iommu/intel-svm.c > > @@ -225,6 +225,9 @@ static const struct mmu_notifier_ops > > intel_mmuops = { > > static DEFINE_MUTEX(pasid_mutex); > > static LIST_HEAD(global_svm_list); > > +#define for_each_svm_dev() \ > > + list_for_each_entry(sdev, &svm->devs, list) \ > > + if (dev == sdev->dev) \ > > Could we make this macro less opaque and have it take the svm and dev > as arguments? > sounds good, it makes the code more readable.
| |