Re: [Intel-gfx] [PATCH] drm/i915: Revert "drm/i915: Reject the pin ioctl on gen6+"

2014-10-02 Thread Damien Lespiau
Hi Dave, On Fri, Oct 03, 2014 at 07:12:53AM +1000, Dave Airlie wrote: > I thought we had an agreement that no features that were specific to > the non-open source userspace drivers would be merged to the kernel, > due to security and maintenance concerns, i.e. exactly this concern, > we now have t

Re: [Intel-gfx] [PATCH] drm/i915: Revert "drm/i915: Reject the pin ioctl on gen6+"

2014-10-02 Thread Dave Airlie
On 2 October 2014 20:40, Bloomfield, Jon wrote: > On Fri, Sep 19, 2014 Daniel Vetter wrote: >> I still consider pinning stuff behind the kernels back too evil. So if people >> want that I'd like to see the use-case and why we can't do this differently. >> >> And I've never approved of the pin ioct

Re: [Intel-gfx] [PATCH] drm/i915: Revert "drm/i915: Reject the pin ioctl on gen6+"

2014-10-02 Thread Bloomfield, Jon
On Fri, Sep 19, 2014 Daniel Vetter wrote: > I still consider pinning stuff behind the kernels back too evil. So if people > want that I'd like to see the use-case and why we can't do this differently. > > And I've never approved of the pin ioctl but really loudly complained about > each occasion I

Re: [Intel-gfx] [PATCH] drm/i915: Revert "drm/i915: Reject the pin ioctl on gen6+"

2014-09-19 Thread Daniel Vetter
On Fri, Sep 19, 2014 at 03:25:55PM +0100, Damien Lespiau wrote: > Hi Daniel, > > On Mon, Jul 07, 2014 at 11:04:55PM +0200, Daniel Vetter wrote: > > On Thu, Jul 03, 2014 at 08:12:35AM +0100, Damien Lespiau wrote: > > > This reverts commit 02f6bcccf7c324115747aae2f0addd6af5d321cd. > > > > > > The O

Re: [Intel-gfx] [PATCH] drm/i915: Revert "drm/i915: Reject the pin ioctl on gen6+"

2014-09-19 Thread Damien Lespiau
Hi Daniel, On Mon, Jul 07, 2014 at 11:04:55PM +0200, Daniel Vetter wrote: > On Thu, Jul 03, 2014 at 08:12:35AM +0100, Damien Lespiau wrote: > > This reverts commit 02f6bcccf7c324115747aae2f0addd6af5d321cd. > > > > The OA buffer can contain global data (in particular, not linked to a > > context o

Re: [Intel-gfx] [PATCH] drm/i915: Revert "drm/i915: Reject the pin ioctl on gen6+"

2014-07-07 Thread Jesse Barnes
On Mon, 7 Jul 2014 23:04:55 +0200 Daniel Vetter wrote: > On Thu, Jul 03, 2014 at 08:12:35AM +0100, Damien Lespiau wrote: > > This reverts commit 02f6bcccf7c324115747aae2f0addd6af5d321cd. > > > > The OA buffer can contain global data (in particular, not linked to a > > context or a single batch e

Re: [Intel-gfx] [PATCH] drm/i915: Revert "drm/i915: Reject the pin ioctl on gen6+"

2014-07-07 Thread Daniel Vetter
On Thu, Jul 03, 2014 at 08:12:35AM +0100, Damien Lespiau wrote: > This reverts commit 02f6bcccf7c324115747aae2f0addd6af5d321cd. > > The OA buffer can contain global data (in particular, not linked to a > context or a single batch execution) about GPU events (eg. hw context > switches, rc6 transiti

Re: [Intel-gfx] [PATCH] drm/i915: Revert "drm/i915: Reject the pin ioctl on gen6+"

2014-07-03 Thread Chris Wilson
On Thu, Jul 03, 2014 at 08:12:35AM +0100, Damien Lespiau wrote: > This reverts commit 02f6bcccf7c324115747aae2f0addd6af5d321cd. > > The OA buffer can contain global data (in particular, not linked to a > context or a single batch execution) about GPU events (eg. hw context > switches, rc6 transiti

[Intel-gfx] [PATCH] drm/i915: Revert "drm/i915: Reject the pin ioctl on gen6+"

2014-07-03 Thread Damien Lespiau
This reverts commit 02f6bcccf7c324115747aae2f0addd6af5d321cd. The OA buffer can contain global data (in particular, not linked to a context or a single batch execution) about GPU events (eg. hw context switches, rc6 transitions, frequency changes, ...) and needs to be mapped to GGTT. The pin ioctl