All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH i-g-t] tests/pm_sseu: Require Gen8+ early in setup
@ 2015-04-04  1:01 jeff.mcgee
  2015-04-07  8:57 ` Daniel Vetter
  0 siblings, 1 reply; 2+ messages in thread
From: jeff.mcgee @ 2015-04-04  1:01 UTC (permalink / raw)
  To: intel-gfx

From: Jeff McGee <jeff.mcgee@intel.com>

Pre-Gen8 devices should be skipped early instead of failing
when test resources are not found.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89822
Signed-off-by: Jeff McGee <jeff.mcgee@intel.com>
---
 tests/pm_sseu.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/pm_sseu.c b/tests/pm_sseu.c
index 34465db..320371f 100644
--- a/tests/pm_sseu.c
+++ b/tests/pm_sseu.c
@@ -262,6 +262,8 @@ gem_init(void)
 
 	gem.devid = intel_get_drm_devid(gem.drm_fd);
 	gem.gen = intel_gen(gem.devid);
+	igt_require_f(gem.gen >= 8,
+		      "SSEU power gating only relevant for Gen8+");
 	gem.has_ppgtt = gem_uses_aliasing_ppgtt(gem.drm_fd);
 
 	gem.bufmgr = drm_intel_bufmgr_gem_init(gem.drm_fd, 4096);
-- 
2.3.3

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH i-g-t] tests/pm_sseu: Require Gen8+ early in setup
  2015-04-04  1:01 [PATCH i-g-t] tests/pm_sseu: Require Gen8+ early in setup jeff.mcgee
@ 2015-04-07  8:57 ` Daniel Vetter
  0 siblings, 0 replies; 2+ messages in thread
From: Daniel Vetter @ 2015-04-07  8:57 UTC (permalink / raw)
  To: jeff.mcgee; +Cc: intel-gfx

On Fri, Apr 03, 2015 at 06:01:12PM -0700, jeff.mcgee@intel.com wrote:
> From: Jeff McGee <jeff.mcgee@intel.com>
> 
> Pre-Gen8 devices should be skipped early instead of failing
> when test resources are not found.
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89822
> Signed-off-by: Jeff McGee <jeff.mcgee@intel.com>

Applied, thanks.
-Daniel

> ---
>  tests/pm_sseu.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/pm_sseu.c b/tests/pm_sseu.c
> index 34465db..320371f 100644
> --- a/tests/pm_sseu.c
> +++ b/tests/pm_sseu.c
> @@ -262,6 +262,8 @@ gem_init(void)
>  
>  	gem.devid = intel_get_drm_devid(gem.drm_fd);
>  	gem.gen = intel_gen(gem.devid);
> +	igt_require_f(gem.gen >= 8,
> +		      "SSEU power gating only relevant for Gen8+");
>  	gem.has_ppgtt = gem_uses_aliasing_ppgtt(gem.drm_fd);
>  
>  	gem.bufmgr = drm_intel_bufmgr_gem_init(gem.drm_fd, 4096);
> -- 
> 2.3.3
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-04-07  8:55 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-04  1:01 [PATCH i-g-t] tests/pm_sseu: Require Gen8+ early in setup jeff.mcgee
2015-04-07  8:57 ` Daniel Vetter

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.