Received-SPF: pass (google.com: domain of linux-kernel+bounces-165898-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Message-ID: <703f15b0-d895-4518-9886-0827a6c4e769@amd.com>
Date: Thu, 2 May 2024 11:29:04 +1000
User-Agent: Mozilla Thunderbird Beta
Subject: Re: [RFC PATCH v2 0/6] powerpc: pSeries: vfio: iommu: Re-enable
 support for SPAPR TCE VFIO
Content-Language: en-US
To: Jason Gunthorpe <jgg@ziepe.ca>, Shivaprasad G Bhat <sbhat@linux.ibm.com>
Cc: mpe@ellerman.id.au, tpearson@raptorengineering.com,
 alex.williamson@redhat.com, linuxppc-dev@lists.ozlabs.org,
 npiggin@gmail.com, christophe.leroy@csgroup.eu, aneesh.kumar@kernel.org,
 naveen.n.rao@linux.ibm.com, gbatra@linux.vnet.ibm.com,
 brking@linux.vnet.ibm.com, aik@ozlabs.ru, ruscur@russell.cc,
 robh@kernel.org, linux-kernel@vger.kernel.org, joel@jms.id.au,
 kvm@vger.kernel.org, msuchanek@suse.de, oohall@gmail.com,
 mahesh@linux.ibm.com, jroedel@suse.de, vaibhav@linux.ibm.com,
 svaidy@linux.ibm.com
References: <171450753489.10851.3056035705169121613.stgit@linux.ibm.com>
 <20240501140942.GA1723318@ziepe.ca>
From: Alexey Kardashevskiy <aik@amd.com>
In-Reply-To: <20240501140942.GA1723318@ziepe.ca>
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?SlNWMWRkYnhHZC9xV3ROaUR1NVNvN0laZnBsWDBMdDRIKy9zQlhiY3VGa3JR?=
 =?utf-8?B?WVYyL205NlRQYUdyMlRubTg4Q1RCUnA4TWtrMU1DblA5RFV4WWUxWXY4VWpM?=
 =?utf-8?B?aXdTUHZ1V1JPQ09XUTlsb01YOEVNUWNJcFBhWGp5Z0VuVjhPeHdUdSt3WXlV?=
 =?utf-8?B?dkJMOVFKZDFEMDhKZWREbThWRzg4YlZPeXcwUjJ5T0wxYnJ0MTdDSG16aENB?=
 =?utf-8?B?U1dHRExzTHhJbndaMVdtaWhxZ2g5YkVZWXFyTzFxeU9vam1oNis0eEllWkhr?=
 =?utf-8?B?YWorNmtVZTdDZWp5VTZESFpiL3VkM2UzUU5JMTBQdTRDUW9RaXZkcEpKNEY2?=
 =?utf-8?B?aStTeE4wYTRDcmhITDBNcHluOEZ0YzRyUCt5TjBtSnhnTk9uTXJGL3Bqb0dy?=
 =?utf-8?B?RURqNnF3b2s4b2FmNGhYYWVsSWltMFdUWjZyVEhsSE9nRXlSbktkQ3BSQ1FN?=
 =?utf-8?B?NE1va3NFc1o0TklGUnBDaG4rby9SdmEyeHVucHQrWGhEQlJQUlBJSHQ2WGxx?=
 =?utf-8?B?d2hScFhpVk5NUVhvQnJoZDI2cmU5NTl3VGJlS2k1d1lPQkw1T21qMllIS1ll?=
 =?utf-8?B?VXkycU4wZlU0RnVUSmZ5WUw5N0JIOFZaVUduMXNiM3E4TC9jRGg0ejNvcXRj?=
 =?utf-8?B?RGNGeFMyQmlWSnpyeXQ3Y0ZraVFtMkNyVjl0ODhXYkg0NTluMmx2di93RzRS?=
 =?utf-8?B?bEVHb0tWTzhDT2pRVENRVFdWRWwxazNzNzM0QnhFOEx5b2h0YVRuU1hrOVVQ?=
 =?utf-8?B?anVUS0d4K0t0NDBEMllpSVVLNjlGWlgyM3NzSXprSGJ0ZDdDQy9PbGRLYk9J?=
 =?utf-8?B?RVFsV2E3aG53TjJaOE1mT0NwYVl2ZkZiOUE5RmdvTVJWRkI3Sk9hU082b1ZF?=
 =?utf-8?B?SXVUZjZPK3M4TzBYNHhUMnBtTjE2MU5LRVZjYnBIK3RHV0UyYmdQYXlCdHFT?=
 =?utf-8?B?YkJRQWM1WDRrSm1VSTRNS28ySnNCRTMrSUtrVXFndG9ya0NnalVuUFNlenli?=
 =?utf-8?B?ZFNCR2xxbzdnS2Z5ZTlSYmtmd2hiNlRJTEZ2d1dOVkpQSjlSN2l4ZDdmQzE0?=
 =?utf-8?B?T2pYTlFnTllPTVJrOFh1RlBBbmxoOHB6K3Y2bXRuN2lvaTNDTW1HaStnV00z?=
 =?utf-8?B?aGxkTXlObm5DcFhMbUxCMktsb0tmcFRJM2xOcG40cFM3T1lRSjNFdElyREJI?=
 =?utf-8?B?dmpDMk5UejNva2Z6RXRSL0pzUHJxZTRYb3RhTjlIdXZ6NWpvR3dMSkZSdHZY?=
 =?utf-8?B?ODFHdEQ5TzFWUHdQdW5Tb1lXaDU1YWtrRDhzOXlGekZxZ01EaXluVmw2VGkr?=
 =?utf-8?B?TDB4THhvb2dIbmtDaGxidjM5dE5yRVJaa1k3azZFTExCclBQQ2NIZzkydW9G?=
 =?utf-8?B?cDhrZnFKckQ3ck9JbFRRNXhCZW4xYk1rQkNCQ1VhbTRsMnQ4YW0zOGVsUFVi?=
 =?utf-8?B?YTRySkNGbVFiYmV5ODlQOGNpNVMybVNreVpUUk9QcTZxSzRoQ0RUZ0JjcVJC?=
 =?utf-8?B?aTdZTDRIc2UrT3dkcThKcklTcHZZOGJWVXowTWdXVWJRRFowWUpYazgzcWJQ?=
 =?utf-8?B?NVB2U1c0MDdpVzVCNFkvN0JZZDJsaWVrNW1YcEM1TlhaMHRVc2FEa0tYdVB6?=
 =?utf-8?B?Tmo2dGdsNVpkcnNvZEZLNlp2TEs1WWl2N2RsK1NGcGpDMjNQVGdSK2libkFO?=
 =?utf-8?B?V21ZTUhQT0lkUmp2UVZ5UExNdnFKa2JXRk13N09rSUdMSlV0RFAxOVVjOGdo?=
 =?utf-8?B?bDlhbVhsakJRdENQeTJVbktLMDZ0VEV6dGxGNmVxbi9yUTVIWTRiem4rMWdv?=
 =?utf-8?B?b3hDY0g3bU4rVTYzaWQzYTVSRHFTZXlMWE00NUowdkU0RlRNUGM2QzhlTmdQ?=
 =?utf-8?B?cUVhSmxZNzVPUFFDV3R2QUdoaDYyVkt3dlJkZldWUHQ0NnpMaXRCZ3BZelpo?=
 =?utf-8?B?WkdzTERIYWJhRElFOW9CaVhxNlVQL3NmbXpzMk1jcFdVbWZWUTJXVDV4UndR?=
 =?utf-8?B?bUZCNTJTOXU0c2RzSlRac0IvUVpPMmo2UEQ1SnFuSlBzclRDb1lmYy9RUVlR?=
 =?utf-8?B?ZDZPR3gyMTB4ZXZXeVRrV1h3MWJ5eExHMWhLcWtCZzYxNkNjNmNwVTR1bS92?=
 =?utf-8?Q?rEbs+KV2gAoio7Dq3NuUjUO6a?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c20c89aa-b082-4c82-bca3-08dc6a474723
X-MS-Exchange-CrossTenant-AuthSource: CH3PR12MB9194.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 May 2024 01:29:15.1648
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: GPb0AZVN/SwHPMcPivEWF0yvICGsJk6Sllq1QeLlWKZfvFZvP6hLXV2VmmBK3wybXoMRISNIfl2QD5EqEj86SA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR12MB4422


On 2/5/24 00:09, Jason Gunthorpe wrote:
> On Tue, Apr 30, 2024 at 03:05:34PM -0500, Shivaprasad G Bhat wrote:
>> RFC v1 was posted here [1]. As I was testing more and fixing the
>> issues, I realized its clean to have the table_group_ops implemented
>> the way it is done on PowerNV and stop 'borrowing' the DMA windows
>> for pSeries.
>>
>> This patch-set implements the iommu table_group_ops for pSeries for
>> VFIO SPAPR TCE sub-driver thereby enabling the VFIO support on POWER
>> pSeries machines.
> 
> Wait, did they previously not have any support?
 >
> Again, this TCE stuff needs to go away, not grow. I can grudgingly
> accept fixing it where it used to work, but not enabling more HW that
> never worked before! :(


This used to work when I tried last time 2+ years ago, not a new stuff. 
Thanks,


-- 
Alexey