Re: [Intel-gfx] [PATCH] Revert drm/i915: Disallow pin ioctl completely for kms drivers

2014-11-27 Thread Jani Nikula
On Tue, 25 Nov 2014, Chris Wilson ch...@chris-wilson.co.uk wrote: This reverts commit c211a47c2c28562f8a3fff9e027be1a3ed9e154a. This causes an unwarranteed API break for existing and active userspace. Ville had an interesting observation regarding the commit being reverted:

Re: [Intel-gfx] [PATCH] Revert drm/i915: Disallow pin ioctl completely for kms drivers

2014-11-27 Thread Chris Wilson
On Thu, Nov 27, 2014 at 03:56:38PM +0200, Jani Nikula wrote: On Tue, 25 Nov 2014, Chris Wilson ch...@chris-wilson.co.uk wrote: This reverts commit c211a47c2c28562f8a3fff9e027be1a3ed9e154a. This causes an unwarranteed API break for existing and active userspace. Ville had an interesting

Re: [Intel-gfx] [PATCH] Revert drm/i915: Disallow pin ioctl completely for kms drivers

2014-11-26 Thread Daniel Vetter
On Tue, Nov 25, 2014 at 08:47:10PM +, Chris Wilson wrote: On Tue, Nov 25, 2014 at 02:34:02PM +0100, Daniel Vetter wrote: On Tue, Nov 25, 2014 at 12:06:33PM +, Chris Wilson wrote: On Tue, Nov 25, 2014 at 01:01:39PM +0100, Daniel Vetter wrote: On Tue, Nov 25, 2014 at 11:42:56AM

[Intel-gfx] [PATCH] Revert drm/i915: Disallow pin ioctl completely for kms drivers

2014-11-25 Thread Chris Wilson
This reverts commit c211a47c2c28562f8a3fff9e027be1a3ed9e154a. This causes an unwarranteed API break for existing and active userspace. Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk --- drivers/gpu/drm/i915/i915_gem.c | 90 + 1 file changed, 90

Re: [Intel-gfx] [PATCH] Revert drm/i915: Disallow pin ioctl completely for kms drivers

2014-11-25 Thread Daniel Vetter
On Tue, Nov 25, 2014 at 11:42:56AM +, Chris Wilson wrote: This reverts commit c211a47c2c28562f8a3fff9e027be1a3ed9e154a. This causes an unwarranteed API break for existing and active userspace. Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk Hm, SNA still seems to be able to cope

Re: [Intel-gfx] [PATCH] Revert drm/i915: Disallow pin ioctl completely for kms drivers

2014-11-25 Thread Chris Wilson
On Tue, Nov 25, 2014 at 01:01:39PM +0100, Daniel Vetter wrote: On Tue, Nov 25, 2014 at 11:42:56AM +, Chris Wilson wrote: This reverts commit c211a47c2c28562f8a3fff9e027be1a3ed9e154a. This causes an unwarranteed API break for existing and active userspace. Signed-off-by: Chris

Re: [Intel-gfx] [PATCH] Revert drm/i915: Disallow pin ioctl completely for kms drivers

2014-11-25 Thread Daniel Vetter
On Tue, Nov 25, 2014 at 12:06:33PM +, Chris Wilson wrote: On Tue, Nov 25, 2014 at 01:01:39PM +0100, Daniel Vetter wrote: On Tue, Nov 25, 2014 at 11:42:56AM +, Chris Wilson wrote: This reverts commit c211a47c2c28562f8a3fff9e027be1a3ed9e154a. This causes an unwarranteed API

Re: [Intel-gfx] [PATCH] Revert drm/i915: Disallow pin ioctl completely for kms drivers

2014-11-25 Thread Chris Wilson
On Tue, Nov 25, 2014 at 02:34:02PM +0100, Daniel Vetter wrote: On Tue, Nov 25, 2014 at 12:06:33PM +, Chris Wilson wrote: On Tue, Nov 25, 2014 at 01:01:39PM +0100, Daniel Vetter wrote: On Tue, Nov 25, 2014 at 11:42:56AM +, Chris Wilson wrote: This reverts commit