RE: [PATCH v3 00/11] iommu/vt-d: Optimize the use of locks

Posted by Tian, Kevin 3 years, 10 months ago

> From: Lu Baolu <baolu.lu@linux.intel.com>
> Sent: Wednesday, June 29, 2022 3:47 PM
> 
> v3:
>  - Split reduction of lock ranges from changing irqsave.
>    https://lore.kernel.org/linux-
> iommu/BN9PR11MB52760A3D7C6BF1AF9C9D34658CAA9@BN9PR11MB5276.
> namprd11.prod.outlook.com/
>  - Fully initialize the dev_info before adding it to the list.
>    https://lore.kernel.org/linux-
> iommu/BN9PR11MB52764D7CD86448C5E4EB46668CAA9@BN9PR11MB5276.
> namprd11.prod.outlook.com/
>  - Various code and comments refinement.
> 

This doesn't say why original patch2 was removed:

	"iommu/vt-d: Remove for_each_device_domain()"

It took me a while to realize that it's already covered by your another
patch fixing RID2PASID. 😊

Re: [PATCH v3 00/11] iommu/vt-d: Optimize the use of locks

Posted by Baolu Lu 3 years, 10 months ago

On 2022/7/1 15:53, Tian, Kevin wrote:
>> From: Lu Baolu <baolu.lu@linux.intel.com>
>> Sent: Wednesday, June 29, 2022 3:47 PM
>>
>> v3:
>>   - Split reduction of lock ranges from changing irqsave.
>>     https://lore.kernel.org/linux-
>> iommu/BN9PR11MB52760A3D7C6BF1AF9C9D34658CAA9@BN9PR11MB5276.
>> namprd11.prod.outlook.com/
>>   - Fully initialize the dev_info before adding it to the list.
>>     https://lore.kernel.org/linux-
>> iommu/BN9PR11MB52764D7CD86448C5E4EB46668CAA9@BN9PR11MB5276.
>> namprd11.prod.outlook.com/
>>   - Various code and comments refinement.
>>
> 
> This doesn't say why original patch2 was removed:
> 
> 	"iommu/vt-d: Remove for_each_device_domain()"
> 
> It took me a while to realize that it's already covered by your another
> patch fixing RID2PASID. 😊

My fault! I forgot to mention it in the change log. Sorry about it.

Best regards,
baolu