All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Vetter <daniel@ffwll.ch>
To: Jesse Barnes <jbarnes@virtuousgeek.org>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 15/18] tests/pm_rpm: mark RTE and D3 tests as basic
Date: Fri, 14 Aug 2015 18:06:18 +0200	[thread overview]
Message-ID: <20150814160618.GD20434@phenom.ffwll.local> (raw)
In-Reply-To: <55CE0DDA.6070006@virtuousgeek.org>

On Fri, Aug 14, 2015 at 08:48:42AM -0700, Jesse Barnes wrote:
> On 08/14/2015 05:50 AM, Daniel Vetter wrote:
> > On Thu, Aug 13, 2015 at 01:31:38PM -0700, Jesse Barnes wrote:
> >> These always need to pass for basic PM functionality.
> >>
> >> Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
> >> ---
> >>  tests/pm_rpm.c | 4 ++--
> >>  1 file changed, 2 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/tests/pm_rpm.c b/tests/pm_rpm.c
> >> index d509fa8..7ae5806 100644
> >> --- a/tests/pm_rpm.c
> >> +++ b/tests/pm_rpm.c
> >> @@ -1832,11 +1832,11 @@ int main(int argc, char *argv[])
> >>  			stay_subtest();
> >>  
> >>  	/* Essential things */
> >> -	igt_subtest("rte")
> >> +	igt_subtest("basic-rte")
> > 
> > I thought our BAT criteria would be basic|rte?
> > 
> > rte is runtime environment check and more along the lines of "did QA set
> > up their machine correctly", so imo good to keep separate.
> 
> I was hoping to keep the BAT in the 'basic' namespace only, so rte tests
> would be included as well, but we'd need to prefix them with basic- too.
> Sounds like Paulo is ok with that, so I'll keep it here.  Makes test
> running that much simpler. :)

Yeah makes sense, ok with me if we go with just "basic".
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2015-08-14 16:06 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-13 20:31 [PATCH 01/18] tests/gem_storedw_loop: add new store_dword test to unify per-ring ones Jesse Barnes
2015-08-13 20:31 ` [PATCH 02/18] tests/drv_module_reload: rename drv_module_reload to include in BATs Jesse Barnes
2015-08-14 12:20   ` Daniel Vetter
2015-08-13 20:31 ` [PATCH 03/18] tests/drv_module_reload_basic: use linear_blits after module_reload for sanity check Jesse Barnes
2015-08-14 12:22   ` Daniel Vetter
2015-08-14 15:22     ` Jesse Barnes
2015-08-13 20:31 ` [PATCH 04/18] tests/drm_import_export: mark flink and prime tests as basic Jesse Barnes
2015-08-14 12:26   ` Daniel Vetter
2015-08-14 15:28     ` Jesse Barnes
2015-08-13 20:31 ` [PATCH 05/18] tests/drv_getparams: mark EU and subslice fetch " Jesse Barnes
2015-08-14 12:27   ` Daniel Vetter
2015-08-14 15:23     ` Jesse Barnes
2015-08-13 20:31 ` [PATCH 06/18] tests/drv_suspend: mark sysfs tests " Jesse Barnes
2015-08-14 12:29   ` Daniel Vetter
2015-08-14 15:29     ` Jesse Barnes
2015-08-14 16:01       ` Daniel Vetter
2015-08-14 16:10         ` Jesse Barnes
2015-08-13 20:31 ` [PATCH 07/18] tests/gem_ctx_exec: mark lrc lite restore " Jesse Barnes
2015-08-14 12:32   ` Daniel Vetter
2015-08-14 15:31     ` Jesse Barnes
2015-08-14 16:03       ` Daniel Vetter
2015-08-13 20:31 ` [PATCH 08/18] tests/gem_mmap: mark basic object creation tests " Jesse Barnes
2015-08-14 12:33   ` Daniel Vetter
2015-08-14 12:37     ` Chris Wilson
2015-08-14 12:54       ` Chris Wilson
2015-08-14 15:31     ` Jesse Barnes
2015-08-14 16:05       ` Daniel Vetter
2015-08-13 20:31 ` [PATCH 09/18] tests/gem_mmap_gtt: mark basic access and copy " Jesse Barnes
2015-08-14 12:35   ` Daniel Vetter
2015-08-13 20:31 ` [PATCH 10/18] tests/gem_pread/pwrite: mark normal " Jesse Barnes
2015-08-14 12:36   ` Daniel Vetter
2015-08-13 20:31 ` [PATCH 11/18] tests/gem_tiled_pread/pwrite: " Jesse Barnes
2015-08-14 12:41   ` Daniel Vetter
2015-08-14 15:39     ` Jesse Barnes
2015-08-13 20:31 ` [PATCH 12/18] tests/kms_addfb: mark simple fb creation " Jesse Barnes
2015-08-14 12:42   ` Daniel Vetter
2015-08-13 20:31 ` [PATCH 13/18] tests/kms_vblank: mark accuracy test " Jesse Barnes
2015-08-14 12:44   ` Daniel Vetter
2015-08-14 12:47     ` Daniel Vetter
2015-08-14 15:47     ` Jesse Barnes
2015-08-13 20:31 ` [PATCH 14/18] tests/pm_backlight: mark simple " Jesse Barnes
2015-08-14 12:48   ` Daniel Vetter
2015-08-13 20:31 ` [PATCH 15/18] tests/pm_rpm: mark RTE and D3 tests " Jesse Barnes
2015-08-14 12:50   ` Daniel Vetter
2015-08-14 13:09     ` Paulo Zanoni
2015-08-14 15:48     ` Jesse Barnes
2015-08-14 16:06       ` Daniel Vetter [this message]
2015-08-13 20:31 ` [PATCH 16/18] tests/kms_flip: add basic tests for flip, flip vs dpms, and flip modeset Jesse Barnes
2015-08-14 12:56   ` Daniel Vetter
2015-08-14 16:07     ` Jesse Barnes
2015-08-13 20:31 ` [PATCH 17/18] tests/kms_setmode: mark simple clone test as basic Jesse Barnes
2015-08-14 12:57   ` Daniel Vetter
2015-08-13 20:31 ` [PATCH 18/18] gitignore: ignore more files Jesse Barnes
2015-08-14  8:09   ` Daniel Vetter
2015-08-14 15:20     ` Jesse Barnes
2015-08-14 16:07       ` Daniel Vetter
2015-08-14 16:11         ` Jesse Barnes
2015-08-17  7:48           ` Jani Nikula
2015-08-14 12:19 ` [PATCH 01/18] tests/gem_storedw_loop: add new store_dword test to unify per-ring ones Daniel Vetter
2015-08-14 15:21   ` Jesse Barnes
2015-08-14 16:09     ` Daniel Vetter

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=20150814160618.GD20434@phenom.ffwll.local \
    --to=daniel@ffwll.ch \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=jbarnes@virtuousgeek.org \
    /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.