Re: [PATCH v2] drm/amdgpu: set start timestamp of fence in the right place

2024-07-11 Thread Christian König
, Alexander Subject: Re: [PATCH v2] drm/amdgpu: set start timestamp of fence in the right place Am 10.07.24 um 09:54 schrieb Zhu, Jiadong: [AMD Official Use Only - AMD Internal Distribution Only] -Original Message- From: Christian König Sent: Wednesday, July 10, 2024 3:17 PM To: Zhu

RE: [PATCH v2] drm/amdgpu: set start timestamp of fence in the right place

2024-07-10 Thread Zhu, Jiadong
gt; >> To: Zhu, Jiadong ; > >> amd-gfx@lists.freedesktop.org; Deucher, Alexander > >> > >> Subject: Re: [PATCH v2] drm/amdgpu: set start timestamp of fence in > >> the right place > >> > >> Am 10.07.24 um 09:54 schrieb Zhu, Jiadong: &g

Re: [PATCH v2] drm/amdgpu: set start timestamp of fence in the right place

2024-07-10 Thread Christian König
Subject: Re: [PATCH v2] drm/amdgpu: set start timestamp of fence in the right place Am 10.07.24 um 02:31 schrieb jiadong@amd.com: From: Jiadong Zhu The job's embedded fence is dma_fence which shall not be conversed to amdgpu_fence. Good catch. The start timestamp shall be saved on jo

RE: [PATCH v2] drm/amdgpu: set start timestamp of fence in the right place

2024-07-10 Thread Zhu, Jiadong
3:17 PM > >> To: Zhu, Jiadong ; amd- > g...@lists.freedesktop.org > >> Subject: Re: [PATCH v2] drm/amdgpu: set start timestamp of fence in > >> the right place > >> > >> Am 10.07.24 um 02:31 schrieb jiadong@amd.com: > >>> From: Jiad

Re: [PATCH v2] drm/amdgpu: set start timestamp of fence in the right place

2024-07-10 Thread Christian König
Am 10.07.24 um 09:54 schrieb Zhu, Jiadong: [AMD Official Use Only - AMD Internal Distribution Only] -Original Message- From: Christian König Sent: Wednesday, July 10, 2024 3:17 PM To: Zhu, Jiadong ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH v2] drm/amdgpu: set start timestamp

RE: [PATCH v2] drm/amdgpu: set start timestamp of fence in the right place

2024-07-10 Thread Zhu, Jiadong
[AMD Official Use Only - AMD Internal Distribution Only] > -Original Message- > From: Christian König > Sent: Wednesday, July 10, 2024 3:17 PM > To: Zhu, Jiadong ; amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH v2] drm/amdgpu: set start timestamp of fence in the >

RE: [PATCH v2] drm/amdgpu: set start timestamp of fence in the right place

2024-07-10 Thread Zhu, Jiadong
[AMD Official Use Only - AMD Internal Distribution Only] > -Original Message- > From: Christian König > Sent: Wednesday, July 10, 2024 3:17 PM > To: Zhu, Jiadong ; amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH v2] drm/amdgpu: set start timestamp of fence in the >

Re: [PATCH v2] drm/amdgpu: set start timestamp of fence in the right place

2024-07-10 Thread Christian König
Am 10.07.24 um 02:31 schrieb jiadong@amd.com: From: Jiadong Zhu The job's embedded fence is dma_fence which shall not be conversed to amdgpu_fence. Good catch. The start timestamp shall be saved on job for hw_fence. But NAK to that approach. Why do we need the start time here in the

[PATCH v2] drm/amdgpu: set start timestamp of fence in the right place

2024-07-09 Thread jiadong.zhu
From: Jiadong Zhu The job's embedded fence is dma_fence which shall not be conversed to amdgpu_fence. The start timestamp shall be saved on job for hw_fence. v2: optimize get_fence_start_time. Signed-off-by: Jiadong Zhu --- drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c | 31 --