Received-SPF: pass (google.com: domain of linux-kernel+bounces-96729-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <83c46d51-7d4c-4a2f-b34e-8b6700a5fca7@amd.com>
Date: Fri, 8 Mar 2024 14:46:39 +0530
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2 2/2] drm/amdgpu: add vm fault information to
 devcoredump
Content-Language: en-US
To: =?UTF-8?Q?Christian_K=C3=B6nig?= <christian.koenig@amd.com>,
 Sunil Khatri <sunil.khatri@amd.com>, Alex Deucher
 <alexander.deucher@amd.com>, Shashank Sharma <shashank.sharma@amd.com>
Cc: amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
 linux-kernel@vger.kernel.org, Mukul Joshi <mukul.joshi@amd.com>,
 Arunpravin Paneer Selvam <Arunpravin.PaneerSelvam@amd.com>
References: <20240307205054.3904657-1-sunil.khatri@amd.com>
 <20240307205054.3904657-3-sunil.khatri@amd.com>
 <ab7c8dde-c914-4e07-a95a-126976917416@amd.com>
From: "Khatri, Sunil" <sukhatri@amd.com>
In-Reply-To: <ab7c8dde-c914-4e07-a95a-126976917416@amd.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?RWtmSXZvUTUzUGJhUDcyK081aGdDOTdMWVJ0d1FqZitodWt0MHVUeVp5MVZu?=
 =?utf-8?B?R0pidEh6TzNjY3VJUTd3UmliZ0U0TjJ2cXc4R0Y3NXA2RTZSbHZ4VkVjbXlW?=
 =?utf-8?B?NFk1RUZqQmY2UGR4YjJ2L01MeEUvSzRwUUFmSVZDUE5sdVI3M3FlMGZrR0Ri?=
 =?utf-8?B?ckFXMnBWekhrcmRFMW9ZSDRTZFpaZ1RFbXZjQ3FQVkxIVmV2VnZXSERWRXJ0?=
 =?utf-8?B?M1pHVlZkWis1L1BSM0c5bkloM3NXSDMvQTdWS1IxZ01rcEx6TDliQ2FZRVFy?=
 =?utf-8?B?QmliZW9hZHoxUHVOQlhEeXZWcElodXBicU9XR2RpUFY0TjdRUUpFcHJ5ZWdK?=
 =?utf-8?B?R3BFT1JoYm9OUXJ1TlkzcUJGMnNyUkJHUGt3UmNpUjY3V2RUTjZHNWZBSlFX?=
 =?utf-8?B?SEw5N05oc1ZOVnhrK21LdFRaR2tZdlBKZjhjUGZFUTdwSWJxNVRGT21ZR2tt?=
 =?utf-8?B?QzRzejZZVlRpTzM0ZEc3VEV2UGVsZUVCaHB6TXA1VEVGRUF3MWdFaFF6amFB?=
 =?utf-8?B?WnQybEl5dEsvcks3c2tldzlMbi9iOGR3UXNHY1VuZG5Sd1pYT3F0eXBvQTI1?=
 =?utf-8?B?T2dNelVMaWtyZ2NVNnczald1cG9mWnRyS0ZRSzkvdXJ3UnQxSWxZdzFWQUJK?=
 =?utf-8?B?TTZkNS9DdGxYWnQ0M0JCT09aZ1dWZnFRV1M2a0dSdVh5MU03N3hZZGZqcHVB?=
 =?utf-8?B?QzNIbzFNc2FWZGY1Zk43RUJYTHdtNmp4b1ZxcmlsUGFmbGk2cEswclFWdTVn?=
 =?utf-8?B?ZlZSR005NnNwZisxZkZOR05teDJoODh3UHRpY052eGY2TStSNVpWTXFtZ1Qy?=
 =?utf-8?B?SkYyOGw1Zlp2RG1ZSkxqVUVlOU9FSmViZTgwSWcvdTBtRlFZVkQ5bFZrR3F3?=
 =?utf-8?B?YmtVZUJ6V3ByZmxMaUo5amhpWDNGdjlWd0NYV1NsaHMzQzY0akw5NDBQWWxw?=
 =?utf-8?B?eFlZVUszSi9Uci9OaFZaVFE0UVc5dk9ScDhhbUlabzN2K00rN08wOC9FUUVn?=
 =?utf-8?B?djhLMzRnSU0zaWN4ZDgxTkx3TmhYaGFKYTNIYlRtckt4TmhoemFPUG1xQ1Ay?=
 =?utf-8?B?czgxUFVEb1A1NFg3Q2tBZlpaa2xQb1J3WmZidWY1TWpLd0JmRXVCRmVtTmlB?=
 =?utf-8?B?b2xMM2Z1N24xRS9jSVNROFc4WmpJK2RJbnRlODRySVlPQ1JySFIwV0NJZ1Jv?=
 =?utf-8?B?WEp3SUYvUnFvMmMzSDJlT3JJTzRvb1VSVXl5eWJPUElhcUhqUXpmZDVOUm5J?=
 =?utf-8?B?dUFPaFlVa3Z6QzU1WE9wcXBXSzluMGtqMWMvUnBuR3lyU2wxL3N3cGpVbDJ4?=
 =?utf-8?B?NW9NZlgyOXpZMGZGWG9BMERWb3Y2Nk9zbVpvN3hpRDg1dnQyNitRaHN0R1Zi?=
 =?utf-8?B?U2lKQ0p1dGRFNGJjN0R1V0ZFd0Y0RVpQTi95REp6R2RxL0ZzSHJFYkhEWHpM?=
 =?utf-8?B?UmpCNjMzL1kvNDk4S0N1a3IvN0FCQjMxRjVLYWlpdGtHbzhJcmk4d1I4eWdo?=
 =?utf-8?B?Zmh3c1VudDJzYlA0OXRVUTFubFEyZVpiL294L2puSGlLS1VCbUcrN0dnQXVK?=
 =?utf-8?B?bVYyK1hhRGQyeGdlU2tYZi9HV3k1ZysrOVlJYk1sRThreU9qNzRJWDNQeUQ0?=
 =?utf-8?B?UzdLclRvUEdFWkpPV2lSbTZoV1ZNSTdFQzBBMUowd2tndzcrNlh1SUxvRVlN?=
 =?utf-8?B?bXJLY3cwMWlMVEt0dUdreW5yOEl4WklXUFdMK2EzWDdrdlJ2S3Z6a1dDZ1kz?=
 =?utf-8?B?R3QwT2FyYzBqRExNK2lPS291cHVJNGQzZXJOQTQxUHA1eSttK0tkQitkeS9F?=
 =?utf-8?B?bnBqcVd2NnZMRmxmNVpiKzVvQUF0K0MyWFNNdXFJZFFWYzNLcHZZR29tcXJK?=
 =?utf-8?B?UEFNbGVzR3JhNWJQMVBhMzZtN1duWFdzNFlrdUgrZ204QmlsTjkwbnllcldH?=
 =?utf-8?B?L3hCdzNHdEZ4Z3FBcGxyU1hJb2xpTkdtQXV5SVVHNERvWnZkaTRGanBiTXRF?=
 =?utf-8?B?aHoyV0lOdFEzbkFUN3FRQ1ZWdTNGUW9CMEpsSjNBdTZ1QUZIUXpMZFd6NjJK?=
 =?utf-8?B?ZWZBSEtlNXFCVHZBMkFKVStIVnA4ZUptQ2tCckVjVTVNT05NZW1waE1GbUVW?=
 =?utf-8?Q?USF0Toh3JIAhx5f3pSrjcPUxQ?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 16fb616a-23e7-47bc-5abd-08dc3f507af6
X-MS-Exchange-CrossTenant-AuthSource: PH7PR12MB5596.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Mar 2024 09:16:47.6008
 (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: f4VnuwpMCQJgr/BiLQuKpKqzFUavL+f3LuanL42kgUsThDQn1VZWtBgE/vWLP4g1REj7RuNZUlN6urfQcoTstA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: LV3PR12MB9401


On 3/8/2024 2:39 PM, Christian König wrote:
> Am 07.03.24 um 21:50 schrieb Sunil Khatri:
>> Add page fault information to the devcoredump.
>>
>> Output of devcoredump:
>> **** AMDGPU Device Coredump ****
>> version: 1
>> kernel: 6.7.0-amd-staging-drm-next
>> module: amdgpu
>> time: 29.725011811
>> process_name: soft_recovery_p PID: 1720
>>
>> Ring timed out details
>> IP Type: 0 Ring Name: gfx_0.0.0
>>
>> [gfxhub] Page fault observed
>> Faulty page starting at address: 0x0000000000000000
>> Protection fault status register: 0x301031
>>
>> VRAM is lost due to GPU reset!
>>
>> Signed-off-by: Sunil Khatri <sunil.khatri@amd.com>
>> ---
>>   drivers/gpu/drm/amd/amdgpu/amdgpu_reset.c | 14 +++++++++++++-
>>   1 file changed, 13 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_reset.c 
>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_reset.c
>> index 147100c27c2d..8794a3c21176 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_reset.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_reset.c
>> @@ -203,8 +203,20 @@ amdgpu_devcoredump_read(char *buffer, loff_t 
>> offset, size_t count,
>>                  coredump->ring->name);
>>       }
>>   +    if (coredump->adev) {
>> +        struct amdgpu_vm_fault_info *fault_info =
>> +            &coredump->adev->vm_manager.fault_info;
>> +
>> +        drm_printf(&p, "\n[%s] Page fault observed\n",
>> +               fault_info->vmhub ? "mmhub" : "gfxhub");
>> +        drm_printf(&p, "Faulty page starting at address: 0x%016llx\n",
>> +               fault_info->addr);
>> +        drm_printf(&p, "Protection fault status register: 0x%x\n",
>> +               fault_info->status);
>> +    }
>> +
>>       if (coredump->reset_vram_lost)
>> -        drm_printf(&p, "VRAM is lost due to GPU reset!\n");
>> +        drm_printf(&p, "\nVRAM is lost due to GPU reset!\n");
>
> Why this additional new line?
The intent is the devcoredump have different sections clearly demarcated 
with an new line else "VRAM is lost due to GPU reset!" seems part of the 
page fault information.
[gfxhub] Page fault observed
Faulty page starting at address: 0x0000000000000000
Protection fault status register: 0x301031

VRAM is lost due to GPU reset!

Regards
Sunil

>
> Apart from that looks really good to me.
>
> Regards,
> Christian.
>
>>       if (coredump->adev->reset_info.num_regs) {
>>           drm_printf(&p, "AMDGPU register dumps:\nOffset:     
>> Value:\n");
>