Re: [PATCH v4] drm/amd/amdgpu embed hw_fence into amdgpu_job

2021-08-11 Thread Jingwen Chen
ngwen, > > Your patch has caused amdgpu driver load failure on all ASICs. Please revert > it first and come up with a proper fix. > > Regards, > Guchun > > -Original Message----- > From: amd-gfx On Behalf Of Andrey > Grodzovsky > Sent: Wednesday, Augu

RE: [PATCH v4] drm/amd/amdgpu embed hw_fence into amdgpu_job

2021-08-11 Thread Chen, Guchun
Zhang ; Jack Zhang Subject: RE: [PATCH v4] drm/amd/amdgpu embed hw_fence into amdgpu_job [Public] Hi Jingwen, Your patch has caused amdgpu driver load failure on all ASICs. Please revert it first and come up with a proper fix. Regards, Guchun -Original Message- From: amd-gfx On Behalf Of

RE: [PATCH v4] drm/amd/amdgpu embed hw_fence into amdgpu_job

2021-08-11 Thread Chen, Guchun
-gfx@lists.freedesktop.org Cc: Liu, Monk ; Koenig, Christian ; Jack Zhang ; Jack Zhang Subject: Re: [PATCH v4] drm/amd/amdgpu embed hw_fence into amdgpu_job Reviewed-by: Andrey Grodzovsky Andrey On 2021-08-09 11:22 p.m., Jingwen Chen wrote: > From: Jack Zhang > > Why: Previously hw

Re: [PATCH v4] drm/amd/amdgpu embed hw_fence into amdgpu_job

2021-08-10 Thread Andrey Grodzovsky
Reviewed-by: Andrey Grodzovsky Andrey On 2021-08-09 11:22 p.m., Jingwen Chen wrote: From: Jack Zhang Why: Previously hw fence is alloced separately with job. It caused historical lifetime issues and corner cases. The ideal situation is to take fence to manage both job and fence's lifetime, a