Re: [PATCH] drm/amdgpu: fix VM sync with always valid BOs v2

2017-09-11 Thread Christian König
道: Reviewed-by: Roger He Thanks Roger(Hongbo.He) -Original Message- From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Monday, September 11, 2017 6:53 PM To: Zhou, David(ChunMing) ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amd

Re: [PATCH] drm/amdgpu: fix VM sync with always valid BOs v2

2017-09-11 Thread Zhou, David(ChunMing)
x [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Monday, September 11, 2017 6:53 PM To: Zhou, David(ChunMing) <mailto:david1.z...@amd.com>; amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> Subject: Re: [PATCH] drm/amdgpu: fix VM sy

Re: [PATCH] drm/amdgpu: fix VM sync with always valid BOs v2

2017-09-11 Thread Christian König
Subject: Re: [PATCH] drm/amdgpu: fix VM sync with always valid BOs v2 Am 11.09.2017 um 11:10 schrieb zhoucm1: > > > On 2017年09月11日 17:04, Christian König wrote: >>> NAK, as Roger pointed, the performance of Vulkan Dota2 drops from >>> 72fps to 24fps. >>> >>&

RE: [PATCH] drm/amdgpu: fix VM sync with always valid BOs v2

2017-09-11 Thread Zhou, David(ChunMing)
Behalf Of Christian K?nig Sent: Monday, September 11, 2017 6:53 PM To: Zhou, David(ChunMing) ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu: fix VM sync with always valid BOs v2 Am 11.09.2017 um 11:10 schrieb zhoucm1: > > > On 2017年09月11日 17:04, Christian König wrote: &g

RE: [PATCH] drm/amdgpu: fix VM sync with always valid BOs v2

2017-09-11 Thread He, Roger
: fix VM sync with always valid BOs v2 Am 11.09.2017 um 11:10 schrieb zhoucm1: > > > On 2017年09月11日 17:04, Christian König wrote: >>> NAK, as Roger pointed, the performance of Vulkan Dota2 drops from >>> 72fps to 24fps. >>> >>> This patches will k

Re: [PATCH] drm/amdgpu: fix VM sync with always valid BOs v2

2017-09-11 Thread Christian König
Am 11.09.2017 um 11:10 schrieb zhoucm1: On 2017年09月11日 17:04, Christian König wrote: NAK, as Roger pointed, the performance of Vulkan Dota2 drops from 72fps to 24fps. This patches will kill Per-vm-BO advantages, even drop to 1/3 of previous non-per-vm-bo. This is irrelevant and actually a

Re: [PATCH] drm/amdgpu: fix VM sync with always valid BOs v2

2017-09-11 Thread zhoucm1
On 2017年09月11日 17:04, Christian König wrote: NAK, as Roger pointed, the performance of Vulkan Dota2 drops from 72fps to 24fps. This patches will kill Per-vm-BO advantages, even drop to 1/3 of previous non-per-vm-bo. This is irrelevant and actually a foreseen consequence of per VM BOs. If

Re: [PATCH] drm/amdgpu: fix VM sync with always valid BOs v2

2017-09-11 Thread Christian König
NAK, as Roger pointed, the performance of Vulkan Dota2 drops from 72fps to 24fps. This patches will kill Per-vm-BO advantages, even drop to 1/3 of previous non-per-vm-bo. This is irrelevant and actually a foreseen consequence of per VM BOs. If the performance drops is in-acceptable then the

Re: [PATCH] drm/amdgpu: fix VM sync with always valid BOs v2

2017-09-11 Thread zhoucm1
NAK, as Roger pointed, the performance of Vulkan Dota2 drops from 72fps to 24fps. This patches will kill Per-vm-BO advantages, even drop to 1/3 of previous non-per-vm-bo. all moved and relocated fence have already synced, we just need to catch the va mapping but which is CS itself required,