Re: [PATCH 6/9] drm/amdgpu: Set/clear CPU_ACCESS_REQUIRED flag on page fault and CS

2017-06-26 Thread Marek Olšák
On Mon, Jun 26, 2017 at 11:27 AM, Michel Dänzer wrote: > On 25/06/17 03:00 AM, Christian König wrote: >> Am 23.06.2017 um 19:39 schrieb John Brooks: >>> When the AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED flag is given by >>> userspace, >>> it should only be treated as a hint to initially place a BO so

Re: [PATCH 6/9] drm/amdgpu: Set/clear CPU_ACCESS_REQUIRED flag on page fault and CS

2017-06-26 Thread Michel Dänzer
On 25/06/17 03:00 AM, Christian König wrote: > Am 23.06.2017 um 19:39 schrieb John Brooks: >> When the AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED flag is given by >> userspace, >> it should only be treated as a hint to initially place a BO somewhere CPU >> accessible, rather than having a permanent effe

Re: [PATCH 6/9] drm/amdgpu: Set/clear CPU_ACCESS_REQUIRED flag on page fault and CS

2017-06-24 Thread John Brooks
On Sat, Jun 24, 2017 at 08:00:05PM +0200, Christian König wrote: > Am 23.06.2017 um 19:39 schrieb John Brooks: > >When the AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED flag is given by userspace, > >it should only be treated as a hint to initially place a BO somewhere CPU > >accessible, rather than having

Re: [PATCH 6/9] drm/amdgpu: Set/clear CPU_ACCESS_REQUIRED flag on page fault and CS

2017-06-24 Thread Christian König
Am 23.06.2017 um 19:39 schrieb John Brooks: When the AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED flag is given by userspace, it should only be treated as a hint to initially place a BO somewhere CPU accessible, rather than having a permanent effect on BO placement. And that is a clear NAK from my sid

[PATCH 6/9] drm/amdgpu: Set/clear CPU_ACCESS_REQUIRED flag on page fault and CS

2017-06-23 Thread John Brooks
When the AMDGPU_GEM_CREATE_CPU_ACCESS_REQUIRED flag is given by userspace, it should only be treated as a hint to initially place a BO somewhere CPU accessible, rather than having a permanent effect on BO placement. Instead of the flag being set in stone at BO creation, set the flag when a page fa