Received-SPF: pass (google.com: domain of linux-kernel+bounces-13122-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <41efbbfe-0712-4f6a-8bd6-dbae00a83c82@intel.com>
Date: Fri, 29 Dec 2023 22:50:58 +0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v9 00/10] Add iommufd nesting (part 2/2)
Content-Language: en-US
To: "Tian, Kevin" <kevin.tian@intel.com>, "joro@8bytes.org" <joro@8bytes.org>,
	"alex.williamson@redhat.com" <alex.williamson@redhat.com>, "jgg@nvidia.com"
	<jgg@nvidia.com>, "robin.murphy@arm.com" <robin.murphy@arm.com>,
	"baolu.lu@linux.intel.com" <baolu.lu@linux.intel.com>
CC: "cohuck@redhat.com" <cohuck@redhat.com>, "eric.auger@redhat.com"
	<eric.auger@redhat.com>, "nicolinc@nvidia.com" <nicolinc@nvidia.com>,
	"kvm@vger.kernel.org" <kvm@vger.kernel.org>, "mjrosato@linux.ibm.com"
	<mjrosato@linux.ibm.com>, "chao.p.peng@linux.intel.com"
	<chao.p.peng@linux.intel.com>, "yi.y.sun@linux.intel.com"
	<yi.y.sun@linux.intel.com>, "peterx@redhat.com" <peterx@redhat.com>,
	"jasowang@redhat.com" <jasowang@redhat.com>,
	"shameerali.kolothum.thodi@huawei.com"
	<shameerali.kolothum.thodi@huawei.com>, "lulu@redhat.com" <lulu@redhat.com>,
	"suravee.suthikulpanit@amd.com" <suravee.suthikulpanit@amd.com>,
	"iommu@lists.linux.dev" <iommu@lists.linux.dev>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-kselftest@vger.kernel.org" <linux-kselftest@vger.kernel.org>, "Duan,
 Zhenzhong" <zhenzhong.duan@intel.com>, "joao.m.martins@oracle.com"
	<joao.m.martins@oracle.com>, "Zeng, Xin" <xin.zeng@intel.com>, "Zhao, Yan Y"
	<yan.y.zhao@intel.com>, "j.granados@samsung.com" <j.granados@samsung.com>
References: <20231228150629.13149-1-yi.l.liu@intel.com>
 <BN9PR11MB52767FFD4DD1A4CDDCA782628C9DA@BN9PR11MB5276.namprd11.prod.outlook.com>
From: Yi Liu <yi.l.liu@intel.com>
In-Reply-To: <BN9PR11MB52767FFD4DD1A4CDDCA782628C9DA@BN9PR11MB5276.namprd11.prod.outlook.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Q1RpVUVQNHF0amNpWXEycDlSeko1UnErWCtLajBFY2JjMVU5VVhGY0tPWXBu?=
 =?utf-8?B?TU9MQlllVTNUSmV2OW9kMXhJYW8xYXNqM0d6Vkp1cGdYRVVnWlpMaFJ3dzBT?=
 =?utf-8?B?VXVHQlNsc3JnVEV2cG9uTytUUUhPbjVDOXduTEgxNTdvRlVaRyt3S0ovdmtS?=
 =?utf-8?B?YWNJRWg5cG9oMnEzNUt1N1ZVczVoRjloMDF2OHV0TEZLdjZkYlJ1bWpsdU9F?=
 =?utf-8?B?VDg1d2h1dzd5MTQ3blBaMFZpdWYrUjVYendvZEkwb3VtY3BheDByNHVha1Ni?=
 =?utf-8?B?S0k1RVZLR0xFUUlsd1NZa2pzM0toK0ZwbWM0Uk5STkJIMkc2VFY4d2hsLzF0?=
 =?utf-8?B?UE52ekQvM2JJTFExcG9JVUtEcGFKbCtZSkZqS2lGOVlnUDcyTDExVzJ0OFRR?=
 =?utf-8?B?MlF5alZoamVjRHRCNTM1eWdWc0t6d2hZUVNsM1ozMEkrRENrK3RNZGFlb2Q5?=
 =?utf-8?B?cnNCNmhOVld0MkZxN3F5Q2ZicDRtdiszdUFFWm9qL2tadmsyenE0bjY1aE9F?=
 =?utf-8?B?dExpNlIrSjBrUXRrdlVSb2lRT0JReTVZaGhyaXdZNVIyN0l0bW90Z1NTMi9y?=
 =?utf-8?B?dnpGVEJLSXdVMW9oNHRieVhmcDgvSEMvODdtZ2thbC8wSGt6TGdwQ1ErUWU3?=
 =?utf-8?B?M2phcllUdnVLdHBDVEtwcFFXbUQ2RnhUZEx0d2dMUGk1dUNVNFNUU2ZQUzc4?=
 =?utf-8?B?UXhwTlVjNGlIWmFiWVJ2MXN0Mkw4eHNuN0kxUlZpMzducFhSb3RRQkRQTHU1?=
 =?utf-8?B?N1hCMWRDdE9QK1NtaWlreHRaU3hFZldvaHJacndoWDVSQ2lwZGlwM056NURa?=
 =?utf-8?B?Nk8wRVdjdzdVT3VOcDJmeE1Ham1LNThIVFNNaGV3RGFFdWJ0SFJFRlRiUVJS?=
 =?utf-8?B?a0h1Y3M1RnlMTGtUSWNoWFVTOWRpRFFheTdUdVNyeVJOTkVqbnhBYmV4cklJ?=
 =?utf-8?B?aFE5bFdGd0hDRkNuWG00dU9IUmNKOGVJTmVwaFBBOTVkeGkyczMraEdZTzZW?=
 =?utf-8?B?ZmVaV1cxSGt2bnRYTWlsQ1dGMXJQb2huZ1RwUThNQitsdEtSbjlkZUNiQmcz?=
 =?utf-8?B?VVJrOUpzelYxTmVucU4vYnI4MHRha0gzQUJuODMxVjkzenJLRFpjbEwxSVdD?=
 =?utf-8?B?dG5LU3Voekh3NHo0VE1BZk04RWw4Kysvb01QN3N1MUZjQ3VHVDN1Z01nOWh6?=
 =?utf-8?B?UGtBSXhsdEoveG1IT3gvSXJESjhkbzhHTTB1S0N3RDkvSDlzaGp5UkhhcE1P?=
 =?utf-8?B?VFRtenBJQ1M4d1VJaWlybjVwM3VQc1RIRjBKeHdPb1JHUGl2bmFRWjd6VjF3?=
 =?utf-8?B?clV2dEtSRm9sRFFLaVBPME1uRGtVVUZDaEFpMFZCOVRMWGc5STJ4cU0rTzdm?=
 =?utf-8?B?ZHliTld3WnZZOGtBMFd0akdpZ0Z6d3Z5elBIZW1KZlA5a0t4VkNzVU9XOUgv?=
 =?utf-8?B?czBpTVhMYjNvN1ppUGtGWElaRllIaHhjNkI3ZkhwdjFhbGVkMkpTaTFQK1g0?=
 =?utf-8?B?MUhjOTlHUEZsa0RkYVRDcDlPTHlKY09EbnZ0QnduS1dpUHc4VzdxVkJBbnVk?=
 =?utf-8?B?RjhURzNLOUZFK05TdGlzNkEzbk5SbFBVd1dYQ1FZS2FWVmdXd2RKV3Q4TjRq?=
 =?utf-8?B?Nk0wd0VtYXl1ZWxrK0hpMHhBU2VPSEZRa214aUM5QzdLQkxrMzRORmVLVkk3?=
 =?utf-8?B?QXVzaFJKNWFQYklJN1piK00wUlJHNzIwVWJOR0piNkN3QzUyWUV3SERpaW1l?=
 =?utf-8?B?MUtGR2Z1WDA1RFY5SHFJbUNGeVBZZll3Smo0cTFGRDBQY0hnVlAzYk9tbDRs?=
 =?utf-8?B?bUxQOHFJbWkwQTdHZ3VuWjluOUpWaStIUG1TbnM5Y1JQeWpKQ0hKWDd4T0pt?=
 =?utf-8?B?S2Q5YWZLSjRObWRlWk5HZjlIQ04vNDcwc0p4UVpTR3h2WWhCMWNIODdOZm90?=
 =?utf-8?B?d20vWWRTMURJYlRKNEJiZXlRb0JJclBvQk5BWnFSVUI2NGRzZ21VMUVaVDl2?=
 =?utf-8?B?TGFQbHQ3M0NQWEdSanAzNlU1N3JiY3c2YTJvWktxSUNaN3BkODduM3NwTzZq?=
 =?utf-8?B?QTl6Qm9FQXFmRnFMdWF3Qk5aNlcxM0FMTWxOMytJcDFOZk9rU3Z1QU12MTRD?=
 =?utf-8?Q?r9zkRXWOza6mcjr+CpUuu0EGX?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 370ce198-8139-49d4-4cf7-08dc087d2d48
X-MS-Exchange-CrossTenant-AuthSource: DS0PR11MB7529.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Dec 2023 14:48:10.7574
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: SuIrfkHWTvmlDEBeyWENZwbK/lecsP1dHS5r6lPrJ70SBZuy44DMX0b8QrjqPvFPuzEp/ojmY047Q3LeeDFABQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB7285
X-OriginatorOrg: intel.com

On 2023/12/29 11:00, Tian, Kevin wrote:
>> From: Liu, Yi L <yi.l.liu@intel.com>
>> Sent: Thursday, December 28, 2023 11:06 PM
>>
>> This series is based on the first part which was merged [1], this series is to
>> add the cache invalidation interface or the userspace to invalidate cache
>> after
>> modifying the stage-1 page table. This includes both the iommufd changes
>> and the
>> VT-d driver changes.
>>
>> Complete code can be found in [2], QEMU could can be found in [3].
>>
>> At last, this is a team work together with Nicolin Chen, Lu Baolu. Thanks
>> them for the help. ^_^. Look forward to your feedbacks.
>>
>> [1] https://lore.kernel.org/linux-iommu/20231026044216.64964-1-
>> yi.l.liu@intel.com/ - merged
>> [2] https://github.com/yiliu1765/iommufd/tree/iommufd_nesting
>> [3]
>> https://github.com/yiliu1765/qemu/tree/zhenzhong/wip/iommufd_nesting_
>> rfcv1
>>
> 
> this looks good to me now, except a few minor comments to patch7.
> 
> let's see whether Jason may have a chance to look in his holiday and if ok
> whether he wants to see a new version or just change it while committing.

anyhow, I've updated it in below branch.

https://github.com/yiliu1765/iommufd/tree/iommufd_nesting

-- 
Regards,
Yi Liu