All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Lucas De Marchi <lucas.demarchi@intel.com>
Cc: igt-dev@lists.freedesktop.org
Subject: Re: [igt-dev] [PATCH i-g-t v2] Follow kernel's resource streamer removal
Date: Thu, 19 Jul 2018 12:51:13 -0700	[thread overview]
Message-ID: <20180719195113.GK31791@intel.com> (raw)
In-Reply-To: <20180719191614.14525-1-lucas.demarchi@intel.com>

On Thu, Jul 19, 2018 at 12:16:14PM -0700, Lucas De Marchi wrote:
> Resource streamer is being removed from all GENs, so just test it
> returns EINVAL for all GENs when a batch is submitted with
> I915_EXEC_RESOURCE_STREAMER keeping one test per ring.
> 
> v2: let one test per ring rather than just one test

I believe this patch should already remove LOCAL_I915_EXEC_RESOURCE_STREAMER

So, probably, we would be able to soon kill that on the kernel side as well?!

> 
> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> ---
> 
> This requires the patch for the kernel removing RS from all gens.
> 
>  tests/gem_exec_params.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
> 
> diff --git a/tests/gem_exec_params.c b/tests/gem_exec_params.c
> index 04c21c05..f822297f 100644
> --- a/tests/gem_exec_params.c
> +++ b/tests/gem_exec_params.c
> @@ -375,25 +375,21 @@ igt_main
>  	}
>  
>  	igt_subtest("rs-invalid-on-bsd-ring") {
> -		igt_require(IS_HASWELL(devid) || intel_gen(devid) >= 8);
>  		execbuf.flags = I915_EXEC_BSD | LOCAL_I915_EXEC_RESOURCE_STREAMER;
>  		RUN_FAIL(EINVAL);
>  	}
>  
>  	igt_subtest("rs-invalid-on-blt-ring") {
> -		igt_require(IS_HASWELL(devid) || intel_gen(devid) >= 8);
>  		execbuf.flags = I915_EXEC_BLT | LOCAL_I915_EXEC_RESOURCE_STREAMER;
>  		RUN_FAIL(EINVAL);
>  	}
>  
>  	igt_subtest("rs-invalid-on-vebox-ring") {
> -		igt_require(IS_HASWELL(devid) || intel_gen(devid) >= 8);
>  		execbuf.flags = I915_EXEC_VEBOX | LOCAL_I915_EXEC_RESOURCE_STREAMER;
>  		RUN_FAIL(EINVAL);
>  	}
>  
> -	igt_subtest("rs-invalid-gen") {
> -		igt_require(!IS_HASWELL(devid) && intel_gen(devid) < 8);
> +	igt_subtest("rs-invalid-on-render-ring") {
>  		execbuf.flags = I915_EXEC_RENDER | LOCAL_I915_EXEC_RESOURCE_STREAMER;
>  		RUN_FAIL(EINVAL);
>  	}
> -- 
> 2.17.1
> 
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

  parent reply	other threads:[~2018-07-19 19:51 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-12 20:51 [igt-dev] [PATCH i-g-t] Remove resource streamer tests Lucas De Marchi
2018-07-12 21:15 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2018-07-13  0:07 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2018-07-13  8:53 ` [igt-dev] [PATCH i-g-t] " Daniel Vetter
2018-07-19 19:16   ` [igt-dev] [PATCH i-g-t v2] Follow kernel's resource streamer removal Lucas De Marchi
2018-07-19 19:26     ` Chris Wilson
2018-07-23 21:15       ` Lucas De Marchi
2018-07-23 21:17         ` Lucas De Marchi
2018-07-24  9:50         ` Chris Wilson
2018-07-31 23:53       ` [igt-dev] [PATCH i-g-t v3] " Lucas De Marchi
2018-08-01  8:53         ` Chris Wilson
2018-08-02  5:03           ` Rodrigo Vivi
2018-07-19 19:51     ` Rodrigo Vivi [this message]
2018-07-19 20:04       ` [igt-dev] [PATCH i-g-t v2] " Chris Wilson
2018-07-19 20:40         ` Rodrigo Vivi
2018-07-19 19:57 ` [igt-dev] ✓ Fi.CI.BAT: success for Remove resource streamer tests (rev2) Patchwork
2018-07-20  1:27 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2018-08-01  0:38 ` [igt-dev] ✓ Fi.CI.BAT: success for Remove resource streamer tests (rev3) Patchwork
2018-08-01  1:56 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180719195113.GK31791@intel.com \
    --to=rodrigo.vivi@intel.com \
    --cc=igt-dev@lists.freedesktop.org \
    --cc=lucas.demarchi@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.