On 28/02/18 20:34, Sinan Kaya wrote:
> On 2/12/2018 1:33 PM, Jean-Philippe Brucker wrote:
>> +int iommu_sva_unbind_group(struct iommu_group *group, int pasid)
>> +{
>> + struct group_device *device;
>> +
>> + mutex_lock(&group->mutex);
>> + list_for_each_entry(device, &group->devices, list)
>> + iommu_sva_unbind_device(device->dev, pasid);
>> + mutex_unlock(&group->mutex);
>> +
>> + return 0;
>> +}
>
> I think we should handle the errors returned by iommu_sva_unbind_device() here
> or at least print a warning if we want to still continue unbinding.
Agreed, though bind_group/unbind_group are probably going away in next
series
Thanks,
Jean
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
