[Intel-gfx] [PATCH] gem_exec_parse: require PPGTT as well
Volkin, Bradley D
bradley.d.volkin at intel.com
Thu Jun 26 17:57:49 CEST 2014
On Thu, Jun 26, 2014 at 08:48:46AM -0700, Jesse Barnes wrote:
> The command parser may be present, but not active, so check for PPGTT
> before allowing this test to run.
>
> Signed-off-by: Jesse Barnes <jbarnes at virtuousgeek.org>
> ---
> tests/gem_exec_parse.c | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/tests/gem_exec_parse.c b/tests/gem_exec_parse.c
> index e78a192..f7376e3 100644
> --- a/tests/gem_exec_parse.c
> +++ b/tests/gem_exec_parse.c
> @@ -198,7 +198,7 @@ int fd;
> igt_main
> {
> igt_fixture {
> - int parser_version = 0;
> + int parser_version = 0, has_ppgtt = 0;
> drm_i915_getparam_t gp;
> int rc;
>
> @@ -209,6 +209,11 @@ igt_main
> rc = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
> igt_require(!rc && parser_version > 0);
>
> + gp.param = I915_PARAM_HAS_ALIASING_PPGTT;
> + gp.value = &has_ppgtt;
> + rc = drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
> + igt_require(!rc && has_ppgtt > 0);
> +
You could also shorten it to
igt_require(gem_uses_aliasing_ppgtt(fd));
if you like. And sorry, I should have added that check in the
first place.
Brad
> handle = gem_create(fd, 4096);
>
> /* ATM cmd parser only exists on gen7. */
> --
> 1.9.1
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
More information about the Intel-gfx
mailing list