diff mbox

[i-g-t,2/2] aubdump: support new ioctl

Message ID 20170129233542.7246-2-lionel.g.landwerlin@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Lionel Landwerlin Jan. 29, 2017, 11:35 p.m. UTC
Otherwise we won't log anything with OpenGL applications using a
recent Mesa driver.

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
---
 tools/aubdump.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Petri Latvala Jan. 30, 2017, 12:29 p.m. UTC | #1
On Sun, Jan 29, 2017 at 11:35:42PM +0000, Lionel Landwerlin wrote:
> Otherwise we won't log anything with OpenGL applications using a
> recent Mesa driver.
> 
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
> ---
>  tools/aubdump.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tools/aubdump.c b/tools/aubdump.c
> index d53addc2..59ef3e7e 100644
> --- a/tools/aubdump.c
> +++ b/tools/aubdump.c
> @@ -632,7 +632,8 @@ ioctl(int fd, unsigned long request, ...)
>  			return libc_ioctl(fd, request, argp);
>  		}
>  
> -		case DRM_IOCTL_I915_GEM_EXECBUFFER2: {
> +		case DRM_IOCTL_I915_GEM_EXECBUFFER2:
> +		case DRM_IOCTL_I915_GEM_EXECBUFFER2_WR: {
>  			dump_execbuffer2(fd, argp);
>  			if (device_override)
>  				return 0;


libdrm version check in configure.ac needs to be bumped to a version
containing that ioctl token.


--
Petri Latvala
Chris Wilson Jan. 30, 2017, 12:41 p.m. UTC | #2
On Mon, Jan 30, 2017 at 02:29:07PM +0200, Petri Latvala wrote:
> On Sun, Jan 29, 2017 at 11:35:42PM +0000, Lionel Landwerlin wrote:
> > Otherwise we won't log anything with OpenGL applications using a
> > recent Mesa driver.
> > 
> > Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
> > ---
> >  tools/aubdump.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/tools/aubdump.c b/tools/aubdump.c
> > index d53addc2..59ef3e7e 100644
> > --- a/tools/aubdump.c
> > +++ b/tools/aubdump.c
> > @@ -632,7 +632,8 @@ ioctl(int fd, unsigned long request, ...)
> >  			return libc_ioctl(fd, request, argp);
> >  		}
> >  
> > -		case DRM_IOCTL_I915_GEM_EXECBUFFER2: {
> > +		case DRM_IOCTL_I915_GEM_EXECBUFFER2:
> > +		case DRM_IOCTL_I915_GEM_EXECBUFFER2_WR: {

Alternatively stip out the ioctl number and use that for the case.
-Chris
diff mbox

Patch

diff --git a/tools/aubdump.c b/tools/aubdump.c
index d53addc2..59ef3e7e 100644
--- a/tools/aubdump.c
+++ b/tools/aubdump.c
@@ -632,7 +632,8 @@  ioctl(int fd, unsigned long request, ...)
 			return libc_ioctl(fd, request, argp);
 		}
 
-		case DRM_IOCTL_I915_GEM_EXECBUFFER2: {
+		case DRM_IOCTL_I915_GEM_EXECBUFFER2:
+		case DRM_IOCTL_I915_GEM_EXECBUFFER2_WR: {
 			dump_execbuffer2(fd, argp);
 			if (device_override)
 				return 0;