On Tue, Nov 04, 2014 at 02:00:43PM -0800, bradley.d.volkin@xxxxxxxxx wrote: > +static bool uses_cmd_parser(int fd, int gen) > +{ > + int parser_version = 0; > + drm_i915_getparam_t gp; > + int rc; > + > + gp.param = I915_PARAM_CMD_PARSER_VERSION; > + gp.value = &parser_version; > + rc = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp); > + if (rc || parser_version == 0) > + return false; > + > + if (!gem_uses_aliasing_ppgtt(fd)) > + return false; > + > + if (gen != 7) > + return false; Hm, these additional checks are a bit underwhelming, but I guess so be it. Since we have the cmd parser only on gen7 test coverage isn't really seriously reduced anyway. Applied, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx