Re: [PATCH v8] drm/amdgpu: handle IH ring1 overflow

2021-12-02 Thread philip yang
On 2021-12-02 7:43 a.m., Mike Lothian wrote: On Thu, 25 Nov 2021 at 20:42, Felix Kuehling wrote: OK. Dealing with processed timestamp rather than decoded timestamp fixes the race condition where drain would have returned before the last faul

Re: [PATCH v8] drm/amdgpu: handle IH ring1 overflow

2021-12-02 Thread Mike Lothian
On Thu, 25 Nov 2021 at 20:42, Felix Kuehling wrote: > > OK. Dealing with processed timestamp rather than decoded timestamp fixes > the race condition where drain would have returned before the last fault > was processed. But we're still assuming that each interrupt has a unique > timestamp. We hav

Re: [PATCH v8] drm/amdgpu: handle IH ring1 overflow

2021-11-25 Thread Felix Kuehling
Am 2021-11-25 um 3:30 p.m. schrieb Philip Yang: > IH ring1 is used to process GPU retry fault, overflow is enabled to > drain retry fault because we want receive other interrupts while > handling retry fault to recover range. There is no overflow flag set > when wptr pass rptr. Use timestamp of rpt

[PATCH v8] drm/amdgpu: handle IH ring1 overflow

2021-11-25 Thread Philip Yang
IH ring1 is used to process GPU retry fault, overflow is enabled to drain retry fault because we want receive other interrupts while handling retry fault to recover range. There is no overflow flag set when wptr pass rptr. Use timestamp of rptr and wptr to handle overflow and drain retry fault. If