[Intel-gfx] [PATCH] lib: don't clear i915_ring_stop flags on start
Mika Kuoppala
mika.kuoppala at linux.intel.com
Mon May 19 12:48:36 CEST 2014
as some tests, like drv_hangman, sets the flags with script
and then expects them to stick.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=78322
Signed-off-by: Mika Kuoppala <mika.kuoppala at intel.com>
---
lib/drmtest.c | 17 +++++++++--------
lib/igt_core.h | 2 +-
2 files changed, 10 insertions(+), 9 deletions(-)
diff --git a/lib/drmtest.c b/lib/drmtest.c
index bd72e49..de710eb 100644
--- a/lib/drmtest.c
+++ b/lib/drmtest.c
@@ -88,15 +88,16 @@ is_intel(int fd)
return IS_INTEL(devid);
}
-static void check_stop_rings(void)
+static void check_stop_rings(const bool at_exit)
{
enum stop_ring_flags flags;
flags = igt_get_stop_rings();
igt_warn_on_f(flags != 0,
- "i915_ring_stop flags on exit 0x%x, can't quiescent gpu cleanly\n",
- flags);
+ "i915_ring_stop flags 0x%x on test %s, %s\n",
+ flags, at_exit ? "exit" : "start",
+ at_exit ? "can't quiescent gpu cleanly" : "ignoring");
- if (flags)
+ if (at_exit && flags)
igt_set_stop_rings(STOP_RING_NONE);
}
@@ -120,8 +121,6 @@ void gem_quiescent_gpu(int fd)
struct drm_i915_gem_execbuffer2 execbuf;
struct drm_i915_gem_exec_object2 gem_exec[1];
- check_stop_rings();
-
handle = gem_create(fd, 4096);
gem_write(fd, handle, 0, batch, sizeof(batch));
@@ -266,7 +265,7 @@ static void quiescent_gpu_at_exit(int sig)
if (at_exit_drm_fd < 0)
return;
- check_stop_rings();
+ check_stop_rings(true);
gem_quiescent_gpu(at_exit_drm_fd);
close(at_exit_drm_fd);
at_exit_drm_fd = -1;
@@ -277,7 +276,7 @@ static void quiescent_gpu_at_exit_render(int sig)
if (at_exit_drm_render_fd < 0)
return;
- check_stop_rings();
+ check_stop_rings(true);
gem_quiescent_gpu(at_exit_drm_render_fd);
close(at_exit_drm_render_fd);
at_exit_drm_render_fd = -1;
@@ -301,6 +300,7 @@ int drm_open_any(void)
if (__sync_fetch_and_add(&open_count, 1))
return fd;
+ check_stop_rings(false);
gem_quiescent_gpu(fd);
at_exit_drm_fd = __drm_open_any();
igt_install_exit_handler(quiescent_gpu_at_exit);
@@ -329,6 +329,7 @@ int drm_open_any_render(void)
return fd;
at_exit_drm_render_fd = __drm_open_any();
+ check_stop_rings(false);
gem_quiescent_gpu(fd);
igt_install_exit_handler(quiescent_gpu_at_exit_render);
diff --git a/lib/igt_core.h b/lib/igt_core.h
index f7f7015..3220381 100644
--- a/lib/igt_core.h
+++ b/lib/igt_core.h
@@ -458,7 +458,7 @@ extern enum igt_log_level igt_log_level;
*/
#define igt_warn_on(condition) do {\
if (condition) \
- igt_warn("Warning on condition %s in fucntion %s, file %s:%i\n", \
+ igt_warn("Warning on condition %s in function %s, file %s:%i\n", \
#condition, __func__, __FILE__, __LINE__); \
} while (0)
--
1.7.9.5
More information about the Intel-gfx
mailing list