[PATCH] drm/amdgpu: bump version for vram lost counter query

Christian König ckoenig.leichtzumerken at gmail.com
Fri Oct 13 08:22:03 UTC 2017


Am 13.10.2017 um 09:48 schrieb Michel Dänzer:
> On 12/10/17 10:54 PM, Alex Deucher wrote:
>> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
>> ---
>>   drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 3 ++-
>>   1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
>> index b9a3258..38cd8bd 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
>> @@ -72,9 +72,10 @@
>>    * - 3.20.0 - Add support for local BOs
>>    * - 3.21.0 - Add DRM_AMDGPU_FENCE_TO_HANDLE ioctl
>>    * - 3.22.0 - Add DRM_AMDGPU_SCHED ioctl
>> + * - 3.23.0 - Add query for vram lost counter
> I'd recommend spelling it "VRAM" here and in the shortlog (acronyms
> should be spelled in upper case).

Agree.

>
>
>>    */
>>   #define KMS_DRIVER_MAJOR	3
>> -#define KMS_DRIVER_MINOR	22
>> +#define KMS_DRIVER_MINOR	23
>>   #define KMS_DRIVER_PATCHLEVEL	0
>>   
>>   int amdgpu_vram_limit = 0;
>>
> Either way though,
>
> Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>
I'm thinking about if we shouldn't also expose the GPU reset counter?

Anyway patch is Reviewed-by: Christian König <christian.koenig at amd.com> 
as well for now.

Regards,
Christian.


More information about the amd-gfx mailing list