userptr support in drm drivers

2016-02-29 Thread Daniel Vetter
On Fri, Feb 19, 2016 at 03:58:24PM +0100, Christian König wrote: > >If a minority of users need to opt out from MMU_NOTIFIER, then > >DRM_RADEON_USERPTR and friends should default to y and be hidden behind > >EXPERT. That way you avoid asking yet more questions to everyone else. > >Would that work

userptr support in drm drivers

2016-02-19 Thread Christian König
> If a minority of users need to opt out from MMU_NOTIFIER, then > DRM_RADEON_USERPTR and friends should default to y and be hidden behind > EXPERT. That way you avoid asking yet more questions to everyone else. > Would that work for you? No, just the other way around. When MMU_NOTIFIER is selecte

userptr support in drm drivers

2016-02-19 Thread Jean Delvare
Hi Christian, On Thu, 18 Feb 2016 10:48:18 +0100, Christian König wrote: > Am 18.02.2016 um 09:59 schrieb Jean Delvare: > > Maybe I was not clear enough in my original post, but I am really > > advocating for FEWER kconfig options, not more. Plus I just explained > > why I think the radeon and am

userptr support in drm drivers

2016-02-18 Thread Christian König
Hi Jean, Am 18.02.2016 um 09:59 schrieb Jean Delvare: > Hi Daniel, > > On Tue, 16 Feb 2016 22:14:12 +0100, Daniel Vetter wrote: >> On Tue, Feb 16, 2016 at 09:33:51PM +0100, Christian König wrote: >>> At least for Radeon and Amdgpu the current situation is actually what we >>> want. >>> Howev

userptr support in drm drivers

2016-02-18 Thread Jean Delvare
Hi Daniel, On Tue, 16 Feb 2016 22:14:12 +0100, Daniel Vetter wrote: > On Tue, Feb 16, 2016 at 09:33:51PM +0100, Christian König wrote: > > At least for Radeon and Amdgpu the current situation is actually what we > > want. > > > > >However I still find it confusing > > >that full userptr support

userptr support in drm drivers

2016-02-18 Thread Jean Delvare
Hi Christian, On Tue, 16 Feb 2016 21:33:51 +0100, Christian König wrote: > At least for Radeon and Amdgpu the current situation is actually what we > want. > > > However I still find it confusing > > that full userptr support is under #if defined(CONFIG_MMU_NOTIFIER), > > and not under #if defi

userptr support in drm drivers

2016-02-16 Thread Daniel Vetter
On Tue, Feb 16, 2016 at 09:33:51PM +0100, Christian König wrote: > At least for Radeon and Amdgpu the current situation is actually what we > want. > > >However I still find it confusing > >that full userptr support is under #if defined(CONFIG_MMU_NOTIFIER), > >and not under #if defined(CONFIG_RA

userptr support in drm drivers

2016-02-16 Thread Christian König
At least for Radeon and Amdgpu the current situation is actually what we want. > However I still find it confusing > that full userptr support is under #if defined(CONFIG_MMU_NOTIFIER), > and not under #if defined(CONFIG_RADEON_USERPTR), resp. #if > defined(CONFIG_AMDGPU_USERPTR). It means that f

userptr support in drm drivers

2016-02-16 Thread Jean Delvare
Hi all, While checking the openSUSE kernel configuration, I noticed a couple oddities regarding userptr support in the i915, radeon and amdgpu drm drivers. I'll like to discuss the current situation and come up with an agreement on how this could be cleaned up. Firstly, i915. This driver has user