Re: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function

2019-08-24 Thread Christian König
Virtualization Team |AMD -Original Message- From: Koenig, Christian Sent: Friday, August 23, 2019 8:47 PM To: Liu, Monk ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function Am 23.08.19 um 10:57 schrieb Liu, Monk: vram_lost = amdg

RE: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function

2019-08-23 Thread Liu, Monk
@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function for mode1 and BACO, I think we can assume vram is lost because the UMC gets reset in that case. Some of the data may still look valid, but it's not necessarily reliable. For mode2, vram shou

RE: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function

2019-08-23 Thread Liu, Monk
Team |AMD -Original Message- From: Koenig, Christian Sent: Friday, August 23, 2019 8:47 PM To: Liu, Monk ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function Am 23.08.19 um 10:57 schrieb Liu, Monk: >>> vram_los

Re: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function

2019-08-23 Thread Deucher, Alexander
md-gfx on behalf of Koenig, Christian Sent: Friday, August 23, 2019 8:47 AM To: Liu, Monk ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function Am 23.08.19 um 10:57 schrieb Liu, Monk: >>> vram_lost = amdgpu_device_check

Re: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function

2019-08-23 Thread Koenig, Christian
t;> reset is definitely a vram lost reset >> >> If you increase the counter in general function that will be not >> accurate _________ >> Monk Liu|GPU Virtualization Team |AMD >> >> >> -Original Message- >> From: Chris

RE: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function

2019-08-23 Thread Liu, Monk
> Monk Liu|GPU Virtualization Team |AMD > > > -----Original Message- > From: Christian König > Sent: Friday, August 23, 2019 4:27 PM > To: Liu, Monk ; amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH] drm/amdgpu: introduce vram lost paramter for > reset funct

Re: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function

2019-08-23 Thread Koenig, Christian
From: Christian König > Sent: Friday, August 23, 2019 4:27 PM > To: Liu, Monk ; amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH] drm/amdgpu: introduce vram lost paramter for reset > function > > Am 23.08.19 um 05:34 schrieb Monk Liu: >> for SOC15/vega10 the BACO reset would introduce

RE: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function

2019-08-23 Thread Liu, Monk
From: Christian König Sent: Friday, August 23, 2019 4:27 PM To: Liu, Monk ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function Am 23.08.19 um 05:34 schrieb Monk Liu: > for SOC15/vega10 the BACO reset would introduce vram lost in the high

Re: [PATCH] drm/amdgpu: introduce vram lost paramter for reset function

2019-08-23 Thread Christian König
Am 23.08.19 um 05:34 schrieb Monk Liu: for SOC15/vega10 the BACO reset would introduce vram lost in the high end address range and current kmd's vram lost checking cannot catch it since it only check visible frame buffer TODO: to confirm if mode1/2 reset would introduce vram lost Looks good in