Re: [PATCH] drm/amdgpu: fix no full coverage issue for gprs initialization

2021-04-27 Thread Zeng, Oak
--Original Message- > From: Christian König > Sent: Tuesday, April 27, 2021 23:17 > To: Zhang, Hawking ; Li, Dennis ; amd-gfx@lists.freedesktop.org; Deucher, Alexander ; Kuehling, Felix ; Koenig, Christian > Subject: Re: [PATCH] drm/amdgpu: fix no full coverage issu

Re: [PATCH] drm/amdgpu: fix no full coverage issue for gprs initialization

2021-04-27 Thread Christian König
-gfx@lists.freedesktop.org; Deucher, Alexander ; Kuehling, Felix ; Koenig, Christian Subject: Re: [PATCH] drm/amdgpu: fix no full coverage issue for gprs initialization This is only done during bootup, isn't it? Wouldn't it be better to use the normal IB pool instead of the direct o

RE: [PATCH] drm/amdgpu: fix no full coverage issue for gprs initialization

2021-04-27 Thread Zhang, Hawking
Subject: Re: [PATCH] drm/amdgpu: fix no full coverage issue for gprs initialization This is only done during bootup, isn't it? Wouldn't it be better to use the normal IB pool instead of the direct one? Or do we also need to do this during GPU reset? Regards, Christian. Am 27.04.21 um 16:

Re: [PATCH] drm/amdgpu: fix no full coverage issue for gprs initialization

2021-04-27 Thread Christian König
This is only done during bootup, isn't it? Wouldn't it be better to use the normal IB pool instead of the direct one? Or do we also need to do this during GPU reset? Regards, Christian. Am 27.04.21 um 16:55 schrieb Zhang, Hawking: [AMD Public Use] Please split the following into another pat

RE: [PATCH] drm/amdgpu: fix no full coverage issue for gprs initialization

2021-04-27 Thread Zhang, Hawking
-gfx@lists.freedesktop.org; Deucher, Alexander ; Kuehling, Felix ; Koenig, Christian Cc: Li, Dennis Subject: RE: [PATCH] drm/amdgpu: fix no full coverage issue for gprs initialization [AMD Public Use] Please split the following into another patch when you commit the one. Other than that, the

RE: [PATCH] drm/amdgpu: fix no full coverage issue for gprs initialization

2021-04-27 Thread Zhang, Hawking
[AMD Public Use] Please split the following into another patch when you commit the one. Other than that, the patch is Reviewed-by: Hawking Zhang Regards, Hawking @@ -479,8 +710,6 @@ void gfx_v9_4_2_init_golden_registers(struct amdgpu_device *adev, die_id);