Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <d3d885bc-accd-f5a3-d760-f095c7bb1c0f@amd.com>
Date:   Tue, 26 Jul 2022 21:00:03 +0700
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0)
 Gecko/20100101 Thunderbird/91.11.0
Subject: Re: [PATCH] iommu/amd: Disable guest vapic logging during early kdump
 init
Content-Language: en-US
To:     Joerg Roedel <joro@8bytes.org>
Cc:     Jerry Snitselaar <jsnitsel@redhat.com>,
        Will Deacon <will@kernel.org>, iommu@lists.linux.dev,
        linux-kernel@vger.kernel.org
References: <20220721003439.403435-1-jsnitsel@redhat.com>
 <Ytq2GRX7UJjvXk4w@8bytes.org> <a07eded4-c1d7-edf5-9c0e-dfa0c3373e4c@amd.com>
 <Yt+T5ZcvhGaK1e/h@8bytes.org>
From:   Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
In-Reply-To: <Yt+T5ZcvhGaK1e/h@8bytes.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?azFmajMycXVpcllPTVJjbkNoWWdSWGxBYzNlMHQ5RmR6MFl6S2VsWTNWdDB4?=
 =?utf-8?B?Um1rV2c5Z01HYjA5RXgvckdLNHpjL0hWNklqcGpESVZBcjdBL3NKSC9tbzZ2?=
 =?utf-8?B?MXZ5MWt5RUcvL3pHOHpyaXNLeEI1K0UrK0puc09NSEM5bE9TZEUvNWpTYWF4?=
 =?utf-8?B?dVd4alVNbkFtWDBMdGNRQTdKZGRpTjJuR1FNQS9NK0NtU2FtUFhTZUQ3T3dL?=
 =?utf-8?B?RFpvRVZ0dktyeEhLR0F4QXN2VXMxWmdvMHhmMno2c1N3ZGlPcXQ3QVVtTkx0?=
 =?utf-8?B?YU92UXpvdjZVWHFTYlNzTDVYTXY5VkhuMDFOdnltMFhmKzFIRjVWTWdhL0xa?=
 =?utf-8?B?OWlOWjk4dVlIRjlodmtwak5lUzdQMGpvbDlRYTFtTFhiRTJtRThJclZ2dGFC?=
 =?utf-8?B?RS8yV3VXZDBVc1J6L09RdkNYQjVIQjRJMWhWN1FySTBBMDNpMDl2d2E1ajVx?=
 =?utf-8?B?aDVicVRJRXNsVXhzMkxRR2orQWUwVnJCUHFtcVJSbDVvTTJnV3I2K1ZHWjJs?=
 =?utf-8?B?TnVQOURDNEFLZHI4c2pWNXZiYjRqZzk0aEpmNnJuMUhVOG9jTE1tYTFGV3ho?=
 =?utf-8?B?Vjd3RWsvUTl6anRReFR3ei9mUjc2UWhoN0pJR1NDcDhhVjlTczRsNFZGZmVE?=
 =?utf-8?B?OWFGOXcxaWhrMlN0ZWdmcVJWbTdxOExFTE9FUXVDbnZtT2VuNG5SL2VXTUNI?=
 =?utf-8?B?QTRxUEZEWXVRVUR6WUZaOUFTR1JCTEFoVSthazFuSGRNT1lDT2wwcGdMdFFF?=
 =?utf-8?B?eVE0dWx0a1ZSam5qK3Rkd1paSElmVis4MWQ0NXdRQXdlWDh5TEhSTXp6b3dj?=
 =?utf-8?B?SWZDK3NFczFldkJ0eVBLVG5wUkNhZ1JmRGJVUFQ3VG1BZms2bjVqOU9oQWVZ?=
 =?utf-8?B?WlZaUW5pd3RlVVNLeVNJMERmWnkvQWdsdGNBNFV5OTNrTlV5ckwwOUhkS0tO?=
 =?utf-8?B?cU80SVR4UHI2aG1sUURUS3hXR1h6aHppNjFMRGUrQU85YWNOYmwzUllhTVhl?=
 =?utf-8?B?MlpDZGEzQnVCWjhVcFNsTVdZUGFUNkpBVm42NzFpaGhGaG5vRkhGRzJ1dHdj?=
 =?utf-8?B?eXBkYlVYNitRUnUyME5RM0FBTU5kbG5ES2U4TjVJOTdtSVhYdjF2RytaNys4?=
 =?utf-8?B?eHJ1YXQyUHpMMXBIZ0J1eGh6MDZBRmYveXhmK2VYdzdVa1hvL0VwMzYyc2V1?=
 =?utf-8?B?U0tTKzJ3TSsvMVZVNVVxQlkvWlovK2g4dVFpdUF2TVp1ZjhuNEtFS2hRQmFE?=
 =?utf-8?B?SzA2b3hLOENiU2ozOFVsaWJ6NmpWYWZsRDMrQmZIN0oxZWhESUZXUzhHSlNC?=
 =?utf-8?B?QXFyTzFGRE50RU9iVEpFVVpub1B2MDNUR1NMU0pHaU9QZmR3MWp0M2I5ekZm?=
 =?utf-8?B?N2lYYzAybW91a2RzRjRYM2ZCcHV0ZUsvVnBhZWdvWFZsOXhKbVBIN0h3SUJ5?=
 =?utf-8?B?OGY5SVZiVS94cEVWNEJaZzRNMDJiRjV5c0RHeW9qYzNvbXJUTE5kbDk1R0xz?=
 =?utf-8?B?L1VFcFEyRWlUOXBzSVNONGMxZ0lLYk41TmhvcDBLTWVUMUw3QjlJUjVpN2VE?=
 =?utf-8?B?R2FRTWZOcERvbE4vM2dDMEFaS0MyZFhobE4yTlFTanB1dlFEYjNDVzh4Y0lI?=
 =?utf-8?B?c3d0U0NiWi8xN2VoRklMZU9oekdSU3dIUkQ3M3VtVzg4ZVhFMUxTdGZSbEFJ?=
 =?utf-8?B?N3hML2h6YjNHL3MwQjAwN241WmdtUDM3Zzg2SnMvZ1pQcENyZjA0VWs1dGdO?=
 =?utf-8?B?a09yQTVjcnl0ekcvaGhvbFUzWXNRM3ZnOGRYQzFrZEh6NTNWYjFpdS9ZbHhP?=
 =?utf-8?B?ZEltNjl0V0FKSDZMQUhQblJMYUJjRjhTbmhSTDlJRVordHNkd2pZY2ZGWTQ2?=
 =?utf-8?B?WXlmS0RpZ3VDZWpUWDMwMUZ5Zmc4SmIyRndRT3dGVUx2Lzk2a3RkRFdsdGlB?=
 =?utf-8?B?UHhFd3JUWmxad2dxdHBJSVBUL1ZtVXF2TUdxckVPOFgyVE84VGRzU0F0Y1Ev?=
 =?utf-8?B?YlpSK2JnblBsOEJqa005SVV2cnFOMnRPRUhuQmptNE1GZytEWlkyZjNjQlhU?=
 =?utf-8?B?MTZnd3hNSHEvbmZLNXQyZDc4VXhJU216Z3lVYnFRTVEwTzlTUjFmL2ZRQ1A5?=
 =?utf-8?B?bTlUSkEzbTN0dlBRT05ZcllyZjl6SHRmMVZpSnFXZVhkUDJ2NWY2bkZ6ZDFj?=
 =?utf-8?Q?zUEptl8GO6DjBgjg5leWMewK88fhBM0fQ+QNC24dl9zi?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4c1630a1-1d8f-4adc-4bea-08da6f0f277d
X-MS-Exchange-CrossTenant-AuthSource: DM8PR12MB5445.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jul 2022 14:00:10.7811
 (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: lUUIquUZEN7Ju0X2sw2VXRNZXfyvCjZ+fjr60FZ4tRyHCn5ksFNC0bHxQS8waaD0F1AhJYNOQcONr+fOyXTV6w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR12MB1382
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 7/26/22 2:12 PM, Joerg Roedel wrote:
> Hi Suravee,
> 
> On Mon, Jul 25, 2022 at 02:05:59PM +0700, Suravee Suthikulpanit wrote:
>> Sorry for late reply. I have been actually working on the new GAM and GALOG enabling code,
>> which should also address this issue as well. I'll send out the patch soon.
> 
> Okay, how about basing your changes on Jerry's fix? A backportable fix
> for a real issue is always better than a bigger rework (which can still
> happen on-top).
> 
> Regards,
> 
> 	Joerg

Ahh... I have just sent out the new patch

https://lore.kernel.org/lkml/20220726134348.6438-2-suravee.suthikulpanit@amd.com/

Please lemme know in case you want me to rebase this on top of Jerry's.

Suravee