[1/2] drm/amdgpu: Add printing for RW extracted from VM_L2_PROTECTION_FAULT_STATUS

Submitted by Zhao, Yong on Aug. 13, 2019, 6:08 p.m.

Details

Message ID 20190813180744.10901-1-Yong.Zhao@amd.com
State New
Headers show
Series "Series without cover letter" ( rev: 1 ) in AMD X.Org drivers

Not browsing as part of any series.

Commit Message

Zhao, Yong Aug. 13, 2019, 6:08 p.m.
RW is also useful in most cases.

Change-Id: Icf4bd65ea168e5965a6a8ebe32ce9327a2de2851
Signed-off-by: Yong Zhao <Yong.Zhao@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 3 +++
 1 file changed, 3 insertions(+)

Patch hide | download patch | download mbox

diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
index 296e2d982578..ec5e858926ad 100644
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
@@ -387,6 +387,9 @@  static int gmc_v9_0_process_interrupt(struct amdgpu_device *adev,
 			dev_err(adev->dev, "\t MAPPING_ERROR: 0x%lx\n",
 				REG_GET_FIELD(status,
 				VM_L2_PROTECTION_FAULT_STATUS, MAPPING_ERROR));
+			dev_err(adev->dev, "\t RW: 0x%lx\n",
+				REG_GET_FIELD(status,
+				VM_L2_PROTECTION_FAULT_STATUS, RW));
 
 		}
 	}

Comments

Am 13.08.19 um 20:08 schrieb Zhao, Yong:
> RW is also useful in most cases.
>
> Change-Id: Icf4bd65ea168e5965a6a8ebe32ce9327a2de2851
> Signed-off-by: Yong Zhao <Yong.Zhao@amd.com>

Reviewed-by: Christian König <christian.koenig@amd.com> for the series.

> ---
>   drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> index 296e2d982578..ec5e858926ad 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> @@ -387,6 +387,9 @@ static int gmc_v9_0_process_interrupt(struct amdgpu_device *adev,
>   			dev_err(adev->dev, "\t MAPPING_ERROR: 0x%lx\n",
>   				REG_GET_FIELD(status,
>   				VM_L2_PROTECTION_FAULT_STATUS, MAPPING_ERROR));
> +			dev_err(adev->dev, "\t RW: 0x%lx\n",
> +				REG_GET_FIELD(status,
> +				VM_L2_PROTECTION_FAULT_STATUS, RW));
>   
>   		}
>   	}
Pushed! Thanks.

Yong

On 2019-08-14 3:27 a.m., Christian König wrote:
> Am 13.08.19 um 20:08 schrieb Zhao, Yong:

>> RW is also useful in most cases.

>>

>> Change-Id: Icf4bd65ea168e5965a6a8ebe32ce9327a2de2851

>> Signed-off-by: Yong Zhao <Yong.Zhao@amd.com>

>

> Reviewed-by: Christian König <christian.koenig@amd.com> for the series.

>

>> ---

>>   drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 3 +++

>>   1 file changed, 3 insertions(+)

>>

>> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c 

>> b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c

>> index 296e2d982578..ec5e858926ad 100644

>> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c

>> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c

>> @@ -387,6 +387,9 @@ static int gmc_v9_0_process_interrupt(struct 

>> amdgpu_device *adev,

>>               dev_err(adev->dev, "\t MAPPING_ERROR: 0x%lx\n",

>>                   REG_GET_FIELD(status,

>>                   VM_L2_PROTECTION_FAULT_STATUS, MAPPING_ERROR));

>> +            dev_err(adev->dev, "\t RW: 0x%lx\n",

>> +                REG_GET_FIELD(status,

>> +                VM_L2_PROTECTION_FAULT_STATUS, RW));

>>             }

>>       }

>