Re: [Intel-gfx] [PATCH v3 4/5] drm: Add decoding of i915 ioctls

2015-07-10 Thread Dmitry V. Levin
On Fri, Jul 10, 2015 at 02:36:38PM +0200, Patrik Jakobsson wrote: > On Wed, Jul 08, 2015 at 03:11:36AM +0300, Dmitry V. Levin wrote: > > On Mon, Jul 06, 2015 at 04:40:24PM +0200, Gabriel Laskar wrote: [...] > > > Anyway, SYS_FUNC(ioctl) is a bit complicated, and the handling of the > > > fallbacks

Re: [Intel-gfx] [PATCH v3 4/5] drm: Add decoding of i915 ioctls

2015-07-10 Thread Patrik Jakobsson
On Wed, Jul 08, 2015 at 03:11:36AM +0300, Dmitry V. Levin wrote: > On Mon, Jul 06, 2015 at 04:40:24PM +0200, Gabriel Laskar wrote: > > On Mon, 6 Jul 2015 12:35:52 +0200, Patrik Jakobsson wrote: > > > On Fri, Jul 03, 2015 at 03:36:09AM +0300, Dmitry V. Levin wrote: > > > > On Wed, Jul 01, 2015 at 02

Re: [Intel-gfx] [PATCH v3 4/5] drm: Add decoding of i915 ioctls

2015-07-07 Thread Dmitry V. Levin
On Mon, Jul 06, 2015 at 04:40:24PM +0200, Gabriel Laskar wrote: > On Mon, 6 Jul 2015 12:35:52 +0200, Patrik Jakobsson wrote: > > On Fri, Jul 03, 2015 at 03:36:09AM +0300, Dmitry V. Levin wrote: > > > On Wed, Jul 01, 2015 at 02:52:47PM +0200, Patrik Jakobsson wrote: > > > [...] > > > > --- a/drm.c >

Re: [Intel-gfx] [PATCH v3 4/5] drm: Add decoding of i915 ioctls

2015-07-06 Thread Gabriel Laskar
On Mon, 6 Jul 2015 12:35:52 +0200 Patrik Jakobsson wrote: > On Fri, Jul 03, 2015 at 03:36:09AM +0300, Dmitry V. Levin wrote: > > On Wed, Jul 01, 2015 at 02:52:47PM +0200, Patrik Jakobsson wrote: > > [...] > > > --- a/drm.c > > > +++ b/drm.c > > > @@ -35,6 +35,9 @@ > > > > > > #define DRM_MAX_N

Re: [Intel-gfx] [PATCH v3 4/5] drm: Add decoding of i915 ioctls

2015-07-06 Thread Patrik Jakobsson
On Fri, Jul 03, 2015 at 03:36:09AM +0300, Dmitry V. Levin wrote: > On Wed, Jul 01, 2015 at 02:52:47PM +0200, Patrik Jakobsson wrote: > [...] > > --- a/drm.c > > +++ b/drm.c > > @@ -35,6 +35,9 @@ > > > > #define DRM_MAX_NAME_LEN 128 > > > > +extern int drm_i915_decode_number(struct tcb *tcp, un

Re: [Intel-gfx] [PATCH v3 4/5] drm: Add decoding of i915 ioctls

2015-07-02 Thread Dmitry V. Levin
On Wed, Jul 01, 2015 at 02:52:47PM +0200, Patrik Jakobsson wrote: [...] > --- a/drm.c > +++ b/drm.c > @@ -35,6 +35,9 @@ > > #define DRM_MAX_NAME_LEN 128 > > +extern int drm_i915_decode_number(struct tcb *tcp, unsigned int arg); Please rename "arg" to "code", and ... > +extern int drm_i915_io

[Intel-gfx] [PATCH v3 4/5] drm: Add decoding of i915 ioctls

2015-07-01 Thread Patrik Jakobsson
There are more ioctls to add but the ones in this patch are most commonly used. * Makefile.am: Add compilation of drm_i915.c * drm.c: Dispatch i915 ioctls * drm_i915.c: Decode DRM_IOCTL_I915_GETPARAM * drm_i915.c: Decode DRM_IOCTL_I915_SETPARAM * drm_i915.c: Decode DRM_IOCTL_I915_GEM_EXECBUFFER2 *