Re: [PATCH v4 09/10] iommu: Make iommu_queue_iopf() more generic

Baolu Lu
2023-08-26T07:35:31Z

On 8/25/23 4:17 PM, Tian, Kevin wrote:
>> From: Lu Baolu <baolu.lu@linux.intel.com>
>> Sent: Friday, August 25, 2023 10:30 AM
>>
>> + if (fault->prm.flags & IOMMU_FAULT_PAGE_REQUEST_PASID_VALID)
>> + domain = iommu_get_domain_for_dev_pasid(dev, fault-
>>> prm.pasid, 0);
>> + else
>> + domain = iommu_get_domain_for_dev(dev);
>> +
>> + if (!domain || !domain->iopf_handler) {
>> + dev_warn_ratelimited(dev,
>> + "iopf from pasid %d received without handler
>> installed\n",
>
> "without domain attached or handler installed"

Okay.

>
>>
>> +int iommu_sva_handle_iopf(struct iopf_group *group)
>> +{
>> + struct iommu_fault_param *fault_param = group->dev->iommu-
>>> fault_param;
>> +
>> + INIT_WORK(&group->work, iopf_handler);
>> + if (!queue_work(fault_param->queue->wq, &group->work))
>> + return -EBUSY;
>> +
>> + return 0;
>> +}
>
> this function is generic so the name should not tie to 'sva'.

Currently only sva uses it. It's fine to make it generic later when any
new use comes. Does it work to you?

Best regards,
baolu