Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-21 Thread Christian König
Am 21.02.22 um 04:28 schrieb Qiang Yu: On Fri, Feb 18, 2022 at 6:24 PM Christian König wrote: Am 18.02.22 um 11:16 schrieb Qiang Yu: [SNIP] If amdgpu_vm_ready() use evicting flag, it's still not equivalent to check vm idle: true -> vm idle, false -> vm may be idle or busy. Yeah, but why

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-20 Thread Qiang Yu
On Fri, Feb 18, 2022 at 6:24 PM Christian König wrote: > > Am 18.02.22 um 11:16 schrieb Qiang Yu: > > [SNIP] > >>> If amdgpu_vm_ready() use evicting flag, it's still not equivalent to check > >>> vm idle: true -> vm idle, false -> vm may be idle or busy. > >> Yeah, but why should that be

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-18 Thread Christian König
Am 18.02.22 um 11:16 schrieb Qiang Yu: [SNIP] If amdgpu_vm_ready() use evicting flag, it's still not equivalent to check vm idle: true -> vm idle, false -> vm may be idle or busy. Yeah, but why should that be relevant? The amdgpu_vm_ready() return if we can do page table updates or not. If

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-18 Thread Qiang Yu
On Fri, Feb 18, 2022 at 5:27 PM Christian König wrote: > > Am 18.02.22 um 09:58 schrieb Qiang Yu: > > On Fri, Feb 18, 2022 at 3:46 PM Christian König > > wrote: > >> Am 18.02.22 um 04:08 schrieb Qiang Yu: > >>> On Thu, Feb 17, 2022 at 8:22 PM Christian König > >>> wrote: > Am 17.02.22 um

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-18 Thread Christian König
Am 18.02.22 um 09:58 schrieb Qiang Yu: On Fri, Feb 18, 2022 at 3:46 PM Christian König wrote: Am 18.02.22 um 04:08 schrieb Qiang Yu: On Thu, Feb 17, 2022 at 8:22 PM Christian König wrote: Am 17.02.22 um 11:58 schrieb Qiang Yu: On Thu, Feb 17, 2022 at 6:39 PM Christian König wrote: Am

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-18 Thread Qiang Yu
On Fri, Feb 18, 2022 at 3:46 PM Christian König wrote: > > Am 18.02.22 um 04:08 schrieb Qiang Yu: > > On Thu, Feb 17, 2022 at 8:22 PM Christian König > > wrote: > >> Am 17.02.22 um 11:58 schrieb Qiang Yu: > >>> On Thu, Feb 17, 2022 at 6:39 PM Christian König > >>> wrote: > > Am

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-17 Thread Christian König
Am 18.02.22 um 04:08 schrieb Qiang Yu: On Thu, Feb 17, 2022 at 8:22 PM Christian König wrote: Am 17.02.22 um 11:58 schrieb Qiang Yu: On Thu, Feb 17, 2022 at 6:39 PM Christian König wrote: Am 17.02.22 um 11:13 schrieb Qiang Yu: On Thu, Feb 17, 2022 at 5:46 PM Christian König wrote: Am

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-17 Thread Qiang Yu
On Thu, Feb 17, 2022 at 8:22 PM Christian König wrote: > > Am 17.02.22 um 11:58 schrieb Qiang Yu: > > On Thu, Feb 17, 2022 at 6:39 PM Christian König > > wrote: > >> > >> > >> Am 17.02.22 um 11:13 schrieb Qiang Yu: > >>> On Thu, Feb 17, 2022 at 5:46 PM Christian König > >>> wrote: > Am

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-17 Thread Christian König
Am 17.02.22 um 11:58 schrieb Qiang Yu: On Thu, Feb 17, 2022 at 6:39 PM Christian König wrote: Am 17.02.22 um 11:13 schrieb Qiang Yu: On Thu, Feb 17, 2022 at 5:46 PM Christian König wrote: Am 17.02.22 um 10:40 schrieb Qiang Yu: On Thu, Feb 17, 2022 at 5:15 PM Christian König wrote: Am

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-17 Thread Qiang Yu
On Thu, Feb 17, 2022 at 6:39 PM Christian König wrote: > > > > Am 17.02.22 um 11:13 schrieb Qiang Yu: > > On Thu, Feb 17, 2022 at 5:46 PM Christian König > > wrote: > >> Am 17.02.22 um 10:40 schrieb Qiang Yu: > >>> On Thu, Feb 17, 2022 at 5:15 PM Christian König > >>> wrote: > Am 17.02.22

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-17 Thread Christian König
Am 17.02.22 um 11:13 schrieb Qiang Yu: On Thu, Feb 17, 2022 at 5:46 PM Christian König wrote: Am 17.02.22 um 10:40 schrieb Qiang Yu: On Thu, Feb 17, 2022 at 5:15 PM Christian König wrote: Am 17.02.22 um 10:04 schrieb Qiang Yu: Workstation application ANSA/META get this error dmesg:

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-17 Thread Qiang Yu
On Thu, Feb 17, 2022 at 5:46 PM Christian König wrote: > > Am 17.02.22 um 10:40 schrieb Qiang Yu: > > On Thu, Feb 17, 2022 at 5:15 PM Christian König > > wrote: > >> Am 17.02.22 um 10:04 schrieb Qiang Yu: > >>> Workstation application ANSA/META get this error dmesg: > >>>

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-17 Thread Christian König
Am 17.02.22 um 10:40 schrieb Qiang Yu: On Thu, Feb 17, 2022 at 5:15 PM Christian König wrote: Am 17.02.22 um 10:04 schrieb Qiang Yu: Workstation application ANSA/META get this error dmesg: [drm:amdgpu_gem_va_ioctl [amdgpu]] *ERROR* Couldn't update BO_VA (-16) This is caused by: 1. create a

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-17 Thread Qiang Yu
On Thu, Feb 17, 2022 at 5:15 PM Christian König wrote: > > Am 17.02.22 um 10:04 schrieb Qiang Yu: > > Workstation application ANSA/META get this error dmesg: > > [drm:amdgpu_gem_va_ioctl [amdgpu]] *ERROR* Couldn't update BO_VA (-16) > > > > This is caused by: > > 1. create a 256MB buffer in

Re: [PATCH] drm/amdgpu: check vm bo eviction valuable at last

2022-02-17 Thread Christian König
Am 17.02.22 um 10:04 schrieb Qiang Yu: Workstation application ANSA/META get this error dmesg: [drm:amdgpu_gem_va_ioctl [amdgpu]] *ERROR* Couldn't update BO_VA (-16) This is caused by: 1. create a 256MB buffer in invisible VRAM 2. CPU map the buffer and access it causes vm_fault and try to move