lib/core: Avoid unused result in backtrace printing
PATCH 1/3 lib/core: Avoid unused result in backtrace printing
From: Tvrtko Ursulin <[email protected]>
Add helpers for direct write to stderr to consolidate the code and avoid the unused result warning in build.
--- lib/igt_core.c | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-)
lib/igt_core.c +16 -7
@@ -1239,7 +1239,7 @@ static const char hex[] = "0123456789abcdef";static void xputch (int c) { -write (STDERR_FILENO , (const void *) &c,1 ); +igt_assert_eq (write (STDERR_FILENO , (const void *) &c,1 ),1 ); } static int
@@ -1384,13 +1384,23 @@ xprintf(const char *fmt, ...)va_end (ap); } +static void __write_stderr (const char *str,size_t len) +{ +igt_assert_eq (write (STDERR_FILENO , str, len), len); +} + +static void write_stderr (const char *str) +{ +__write_stderr (str,strlen (str)); +} +static void print_backtrace_sig_safe (void ) {unw_cursor_t cursor;unw_context_t uc;int stack_num =0 ; -write (STDERR_FILENO ,"Stack trace: \n" ,15 ); +write_stderr ("Stack trace: \n" );unw_getcontext (&uc);unw_init_local (&cursor, &uc);
@@ -1899,11 +1909,10 @@ static void fatal_sig_handler(int sig)continue ;if (handled_signals[i].name_len) { -igt_assert_eq (write (STDERR_FILENO ,"Received signal " ,16 ), -16 ); -igt_assert_eq (write (STDERR_FILENO , handled_signals[i].name, handled_signals[i].name_len), - handled_signals[i].name_len); -igt_assert_eq (write (STDERR_FILENO ,".\n" ,2 ),2 ); +write_stderr ("Received signal " ); +__write_stderr (handled_signals[i].name, + handled_signals[i].name_len); +write_stderr (".\n" ); }if (crash_signal (sig)) {
Signature · 6 lines
2.9.5 _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
PATCH 2/3 gem_syslatency: Avoid arithmetic on void * warning
From: Tvrtko Ursulin <[email protected]>
--- benchmarks/gem_syslatency.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
benchmarks/gem_syslatency.c +1 -1
@@ -206,7 +206,7 @@ static void *sys_thp_alloc(void *arg)assert (ptr !=MAP_FAILED );madvise (ptr, sz,MADV_HUGEPAGE );for (size_t page =0 ; page < sz; page +=PAGE_SIZE ) - *(volatile uint32_t *)(ptr + page) =0 ; + *(volatile uint32_t *)((unsigned char *)ptr + page) =0 ;munmap (ptr, sz);clock_gettime (CLOCK_MONOTONIC , &now);
Signature · 6 lines
2.9.5 _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [PATCH i-g-t 2/3] gem_syslatency: Avoid arithmetic on void * warning
Quoting Tvrtko Ursulin (2017-11-03 11:47:55)
Quoted 1 line
From: Tvrtko Ursulin <[email protected]>
Turn off the warning? If we want to adhere to the kernel coding standards, we may as well benefit from them as well. -Chris
_______________________________________________
Intel-gfx mailing list [email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
PATCH 3/3 intel_gvtg_test: Handle system(3) return value.
In reply to Tvrtko UrsulinFrom: Tvrtko Ursulin <[email protected]>
Avoid the build warning by checking the pkill either did not find any guests or managed to send a signal to all of them.
--- tools/intel_gvtg_test.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
tools/intel_gvtg_test.c +3 -1
@@ -144,7 +144,9 @@ static void create_guest(void)static void destroy_all_guest (void ) { -system ("pkill qemu" ); +int ret =system ("pkill qemu" ); + +igt_assert (ret ==0 || ret ==1 ); }static void remove_vgpu (void )
Signature · 6 lines
2.9.5 _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [PATCH i-g-t 3/3] intel_gvtg_test: Handle system(3) return value.
Quoting Tvrtko Ursulin (2017-11-03 11:47:56)
Quoted 4 lines
From: Tvrtko Ursulin <[email protected]>
Avoid the build warning by checking the pkill either did not find any guests or managed to send a signal to all of them.
This is tools, do we have the igt framework so that calling igt_assert when lead to a confusing mass of "not in a test" warnings? -Chris
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [PATCH i-g-t 3/3] intel_gvtg_test: Handle system(3) return value.
Quoted 8 lines · Chris Wilson
Quoting Tvrtko Ursulin (2017-11-03 11:47:56)
Quoted 4 lines
From: Tvrtko Ursulin <[email protected]>
Avoid the build warning by checking the pkill either did not find any guests or managed to send a signal to all of them.
This is tools, do we have the igt framework so that calling
igt_assertwhen lead to a confusing mass of "not in a test" warnings?
Who knows, but the tools was already using them. :)
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Quoting Tvrtko Ursulin (2017-11-03 11:47:54)
Quoted 15 lines
From: Tvrtko Ursulin <[email protected]>
Add helpers for direct write to stderr to consolidate the code and avoid the unused result warning in build.
Signed-off-by Tvrtko Ursulin <[email protected]>--- lib/igt_core.c | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-)lib/igt_core.c +0
-0
lib/igt_core.c static const char hex[] = "0123456789abcdef"; · line 1242
Quoted 5 lines
lib/igt_core.c +1
-1@@ -1239,7 +1239,7 @@ static const char hex[] = "0123456789abcdef";static void xputch (int c) { -write (STDERR_FILENO , (const void *) &c,1 ); +igt_assert_eq (write (STDERR_FILENO , (const void *) &c,1 ),1 );
Infinite recursion, you can't use an assert from inside the assert handler.
-Chris
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Quoted 11 lines · Chris Wilson
Quoting Tvrtko Ursulin (2017-11-03 11:47:54)
Quoted 10 lines
From: Tvrtko Ursulin <[email protected]>
Add helpers for direct write to stderr to consolidate the code and avoid the unused result warning in build.
Signed-off-by Tvrtko Ursulin <[email protected]>--- lib/igt_core.c | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-)
Quoted 13 lines
lib/igt_core.c static const char hex[] = "0123456789abcdef"; · line 1242
Quoted 10 lines
lib/igt_core.c +1
-1@@ -1239,7 +1239,7 @@ static const char hex[] = "0123456789abcdef";static void xputch (int c) { -write (STDERR_FILENO , (const void *) &c,1 ); +igt_assert_eq (write (STDERR_FILENO , (const void *) &c,1 ),1 );Infinite recursion, you can't use an assert from inside the assert handler.
I thought it is the signal handler. Tested sending some signals and it survived. If there is a flaw there could instead nerf them with void cast, don't know.
Tvrtko
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Quoting Tvrtko Ursulin (2017-11-03 12:04:15)
Quoted 27 lines · Chris Wilson
Quoted 11 lines
Quoting Tvrtko Ursulin (2017-11-03 11:47:54)
Quoted 10 lines
From: Tvrtko Ursulin <[email protected]>
Add helpers for direct write to stderr to consolidate the code and avoid the unused result warning in build.
Signed-off-by Tvrtko Ursulin <[email protected]>--- lib/igt_core.c | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-)Quoted 13 lines
lib/igt_core.c static const char hex[] = "0123456789abcdef"; · line 1242
Quoted 10 lines
lib/igt_core.c +1
-1@@ -1239,7 +1239,7 @@ static const char hex[] = "0123456789abcdef";static void xputch (int c) { -write (STDERR_FILENO , (const void *) &c,1 ); +igt_assert_eq (write (STDERR_FILENO , (const void *) &c,1 ),1 );Infinite recursion, you can't use an assert from inside the assert handler.
Quoted 3 lines
I thought it is the signal handler. Tested sending some signals and it survived. If there is a flaw there could instead nerf them with void cast, don't know.
I do recall these are meant to be fail safe, so using something like assert which may end up here again is no-no. igt_ignore_warn()? -Chris
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Quoted 28 lines · Chris Wilson
Quoting Tvrtko Ursulin (2017-11-03 12:04:15)
Quoted 27 lines · Chris Wilson
Quoted 24 lines
Quoting Tvrtko Ursulin (2017-11-03 11:47:54)
Quoted 15 lines
From: Tvrtko Ursulin <[email protected]>
Add helpers for direct write to stderr to consolidate the code and avoid the unused result warning in build.
Signed-off-by Tvrtko Ursulin <[email protected]>--- lib/igt_core.c | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-)lib/igt_core.c +0
-0lib/igt_core.c static const char hex[] = "0123456789abcdef"; · line 1242
Quoted 5 lines
lib/igt_core.c +1
-1@@ -1239,7 +1239,7 @@ static const char hex[] = "0123456789abcdef";static void xputch (int c) { -write (STDERR_FILENO , (const void *) &c,1 ); +igt_assert_eq (write (STDERR_FILENO , (const void *) &c,1 ),1 );Infinite recursion, you can't use an assert from inside the assert handler.
Quoted 6 lines
Quoted 3 lines
I thought it is the signal handler. Tested sending some signals and it survived. If there is a flaw there could instead nerf them with void cast, don't know.
I do recall these are meant to be fail safe, so using something like assert which may end up here again is no-no.
igt_ignore_warn()?
But then on the other hand igt_fail seems to have handling for exit handlers using igt_asserts. Which suggests what I've done might be fine after all.
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Quoting Tvrtko Ursulin (2017-11-03 12:56:14)
Quoted 34 lines · Chris Wilson
Quoted 32 lines
Quoting Tvrtko Ursulin (2017-11-03 12:04:15)
Quoted 30 lines · Chris Wilson
Quoted 11 lines
Quoting Tvrtko Ursulin (2017-11-03 11:47:54)
Quoted 10 lines
From: Tvrtko Ursulin <[email protected]>
Add helpers for direct write to stderr to consolidate the code and avoid the unused result warning in build.
Signed-off-by Tvrtko Ursulin <[email protected]>--- lib/igt_core.c | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-)Quoted 13 lines
lib/igt_core.c static const char hex[] = "0123456789abcdef"; · line 1242
Quoted 10 lines
lib/igt_core.c +1
-1@@ -1239,7 +1239,7 @@ static const char hex[] = "0123456789abcdef";static void xputch (int c) { -write (STDERR_FILENO , (const void *) &c,1 ); +igt_assert_eq (write (STDERR_FILENO , (const void *) &c,1 ),1 );Infinite recursion, you can't use an assert from inside the assert handler.
I thought it is the signal handler. Tested sending some signals and it survived. If there is a flaw there could instead nerf them with void cast, don't know.
Quoted 6 lines
Quoted 2 lines
I do recall these are meant to be fail safe, so using something like assert which may end up here again is no-no.
igt_ignore_warn()?But then on the other hand
igt_failseems to have handling for exit handlers usingigt_asserts. Which suggests what I've done might be fine after all.
Otoh, where does the assert go if we fail to write to stderr? I do not like the means by which we are meant to be writing the death throes to cause further output. These routines are the lowest level and meant to be impervious, callable from any context, calling back up into igt is a layering violation.
-Chris
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Quoted 42 lines · Chris Wilson
Quoting Tvrtko Ursulin (2017-11-03 12:56:14)
Quoted 34 lines · Chris Wilson
Quoted 32 lines
Quoting Tvrtko Ursulin (2017-11-03 12:04:15)
Quoted 30 lines · Chris Wilson
Quoted 11 lines
Quoting Tvrtko Ursulin (2017-11-03 11:47:54)
Quoted 10 lines
From: Tvrtko Ursulin <[email protected]>
Add helpers for direct write to stderr to consolidate the code and avoid the unused result warning in build.
Signed-off-by Tvrtko Ursulin <[email protected]>--- lib/igt_core.c | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-)Quoted 13 lines
lib/igt_core.c static const char hex[] = "0123456789abcdef"; · line 1242
Quoted 10 lines
lib/igt_core.c +1
-1@@ -1239,7 +1239,7 @@ static const char hex[] = "0123456789abcdef";static void xputch (int c) { -write (STDERR_FILENO , (const void *) &c,1 ); +igt_assert_eq (write (STDERR_FILENO , (const void *) &c,1 ),1 );Infinite recursion, you can't use an assert from inside the assert handler.
I thought it is the signal handler. Tested sending some signals and it survived. If there is a flaw there could instead nerf them with void cast, don't know.
Quoted 6 lines
Quoted 2 lines
I do recall these are meant to be fail safe, so using something like assert which may end up here again is no-no.
igt_ignore_warn()?But then on the other hand
igt_failseems to have handling for exit handlers usingigt_asserts. Which suggests what I've done might be fine after all.
Quoted 5 lines
Otoh, where does the assert go if we fail to write to stderr? I do not like the means by which we are meant to be writing the death throes to cause further output. These routines are the lowest level and meant to be impervious, callable from any context, calling back up into igt is a layering violation.
I can send a version which avoids that, but also note fatal_sig_handler already uses igt_assert. I'll end up changing more than I bargained for when I started. :)
Regards,
Tvrtko
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
PATCH v2 1/3 lib/core: Avoid unused result in backtrace printing
In reply to Chris WilsonFrom: Tvrtko Ursulin <[email protected]>
Add helpers for direct write to stderr to consolidate the code and avoid the unused result warning in build.
This also stops fatal_sig_handler from using igt_assert making the fatal exit path more conceptually correct.
v2: Use igt_ignore_warn since it is questionable whether igt_assert
should be used on the exit paths. (Chris Wilson)
--- lib/igt_core.c | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-)
lib/igt_core.c +16 -7
@@ -1239,7 +1239,7 @@ static const char hex[] = "0123456789abcdef";static void xputch (int c) { -write (STDERR_FILENO , (const void *) &c,1 ); +igt_ignore_warn (write (STDERR_FILENO , (const void *) &c,1 )); } static int
@@ -1384,13 +1384,23 @@ xprintf(const char *fmt, ...)va_end (ap); } +static void __write_stderr (const char *str,size_t len) +{ +igt_ignore_warn (write (STDERR_FILENO , str, len)); +} + +static void write_stderr (const char *str) +{ +__write_stderr (str,strlen (str)); +} +static void print_backtrace_sig_safe (void ) {unw_cursor_t cursor;unw_context_t uc;int stack_num =0 ; -write (STDERR_FILENO ,"Stack trace: \n" ,15 ); +write_stderr ("Stack trace: \n" );unw_getcontext (&uc);unw_init_local (&cursor, &uc);
@@ -1899,11 +1909,10 @@ static void fatal_sig_handler(int sig)continue ;if (handled_signals[i].name_len) { -igt_assert_eq (write (STDERR_FILENO ,"Received signal " ,16 ), -16 ); -igt_assert_eq (write (STDERR_FILENO , handled_signals[i].name, handled_signals[i].name_len), - handled_signals[i].name_len); -igt_assert_eq (write (STDERR_FILENO ,".\n" ,2 ),2 ); +write_stderr ("Received signal " ); +__write_stderr (handled_signals[i].name, + handled_signals[i].name_len); +write_stderr (".\n" ); }if (crash_signal (sig)) {
Signature · 6 lines
2.9.5 _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Patchwork
CI✗ Fi.CI.BAT: warning for series starting with [1/3] lib/core: Avoid unused result in backtrace printing
In reply to Tvrtko Ursulin== Series Details ==
== Series Details ==
Series: series starting with [1/3] lib/core: Avoid unused result in backtrace printing
URL : https://patchwork.freedesktop.org/series/33119/
State : warning
== Summary ==
IGT patchset tested on top of latest successful build c8d1ea24d3bfaf11b223bbe22407aeca196d0d89 tests/debugfs_test: Pretty print subdirectories
with latest DRM-Tip kernel build CI_DRM_33112f945d948d05 drm-tip: 2017y-11m-03d-09h-28m-20s UTC integration manifest
No testlist changes.
Test chamelium:
Subgroup dp-crc-fast:
incomplete -> PASS (fi-kbl-7500u) fdo#102514
Test gem_exec_flush:
Subgroup basic-wb-rw-default:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-wb-set-default:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_exec_nop:
Subgroup basic-parallel:
pass -> DMESG-WARN (fi-glk-dsi) fdo#103514 +4
Subgroup basic-series:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_exec_parallel:
Subgroup basic:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_exec_reloc:
Subgroup basic-cpu-read:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-gtt-read:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-cpu:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-gtt:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-read:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-cpu-noreloc:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-gtt-noreloc:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-cpu-gtt-noreloc:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-gtt-cpu-noreloc:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-cpu-read-noreloc:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-gtt-read-noreloc:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-cpu-noreloc:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-gtt-noreloc:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-read-noreloc:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-cpu-active:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-gtt-active:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-cpu-gtt-active:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-gtt-cpu-active:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-cpu-read-active:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-gtt-read-active:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-cpu-active:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-gtt-active:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-read-active:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-softpin:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_exec_store:
Subgroup basic-all:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-blt:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-bsd:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-default:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-render:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-vebox:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_exec_suspend:
Subgroup basic:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-s3:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-s4-devices:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_flink_basic:
Subgroup bad-flink:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup bad-open:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_linear_blits:
Subgroup basic:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_mmap:
Subgroup basic:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-small-bo:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_mmap_gtt:
Subgroup basic:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-copy:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-read:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-read-no-prefault:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-small-bo:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-small-bo-tiledx:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-small-bo-tiledy:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-small-copy:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-small-copy-xy:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-wc:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-gtt:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-gtt-no-prefault:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-no-prefault:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-read:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write-read-distinct:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_pread:
Subgroup basic:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_pwrite:
Subgroup basic:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_render_linear_blits:
Subgroup basic:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_render_tiled_blits:
Subgroup basic:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_ringfill:
Subgroup basic-default:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-default-interruptible:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-default-hang:
pass -> DMESG-WARN (fi-glk-dsi) fdo#103359
Test gem_sync:
Subgroup basic-all:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-each:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-many-each:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-store-all:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-store-each:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_tiled_blits:
Subgroup basic:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_tiled_fence_blits:
Subgroup basic:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_tiled_pread_basic:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_wait:
Subgroup basic-wait-all:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-await-all:
pass -> DMESG-WARN (fi-glk-dsi)
Test gem_workarounds:
Subgroup basic-read:
pass -> DMESG-WARN (fi-glk-dsi)
Test kms_addfb_basic:
Subgroup addfb25-bad-modifier:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup addfb25-framebuffer-vs-set-tiling:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup addfb25-modifier-no-flag:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup addfb25-x-tiled:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup addfb25-x-tiled-mismatch:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup addfb25-yf-tiled:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup addfb25-y-tiled:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup addfb25-y-tiled-small:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup bad-pitch-0:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup bad-pitch-1024:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup bad-pitch-128:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup bad-pitch-256:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup bad-pitch-32:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup bad-pitch-63:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup bad-pitch-65536:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup bad-pitch-999:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-x-tiled:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-y-tiled:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup framebuffer-vs-set-tiling:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup invalid-get-prop:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup invalid-get-prop-any:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup invalid-set-prop:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup invalid-set-prop-any:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup no-handle:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup size-max:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup small-bo:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup unused-offsets:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup unused-pitches:
pass -> DMESG-WARN (fi-glk-dsi)
Test kms_busy:
Subgroup basic-flip-a:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-flip-b:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-flip-c:
pass -> DMESG-WARN (fi-glk-dsi)
Test kms_cursor_legacy:
Subgroup basic-busy-flip-before-cursor-atomic:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-busy-flip-before-cursor-legacy:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-flip-after-cursor-atomic:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-flip-after-cursor-legacy:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-flip-after-cursor-varying-size:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-flip-before-cursor-atomic:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-flip-before-cursor-legacy:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-flip-before-cursor-varying-size:
pass -> DMESG-WARN (fi-glk-dsi)
Test kms_flip:
Subgroup basic-flip-vs-dpms:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-flip-vs-modeset:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-flip-vs-wf_vblank:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-plain-flip:
pass -> DMESG-WARN (fi-glk-dsi)
Test kms_frontbuffer_tracking:
Subgroup basic:
fail -> DMESG-FAIL (fi-glk-dsi) fdo#103167
Test kms_pipe_crc_basic:
Subgroup bad-nb-words-1:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup bad-nb-words-3:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup bad-pipe:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup bad-source:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup nonblocking-crc-pipe-a:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup nonblocking-crc-pipe-a-frame-sequence:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup nonblocking-crc-pipe-b:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup nonblocking-crc-pipe-b-frame-sequence:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup nonblocking-crc-pipe-c:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup nonblocking-crc-pipe-c-frame-sequence:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup read-crc-pipe-a:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup read-crc-pipe-a-frame-sequence:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup read-crc-pipe-b:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup read-crc-pipe-b-frame-sequence:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup read-crc-pipe-c:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup read-crc-pipe-c-frame-sequence:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup suspend-read-crc-pipe-b:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup suspend-read-crc-pipe-c:
pass -> DMESG-WARN (fi-glk-dsi)
Test kms_setmode:
Subgroup basic-clone-single-crtc:
pass -> DMESG-WARN (fi-glk-dsi)
Test pm_backlight:
Subgroup basic-brightness:
pass -> DMESG-WARN (fi-glk-dsi)
Test pm_rpm:
Subgroup basic-pci-d3-state:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-rte:
pass -> DMESG-WARN (fi-glk-dsi)
Test pm_rps:
Subgroup basic-api:
pass -> DMESG-WARN (fi-glk-dsi)
Test prime_busy:
Subgroup basic-after-default:
pass -> DMESG-WARN (fi-glk-dsi)
Test prime_self_import:
Subgroup basic-with_fd_dup:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-with_one_bo:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-with_one_bo_two_files:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-with_two_bos:
pass -> DMESG-WARN (fi-glk-dsi)
Test prime_vgem:
Subgroup basic-busy-default:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-fence-mmap:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-fence-read:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-fence-wait-default:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-gtt:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-read:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-sync-default:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-wait-default:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-write:
pass -> DMESG-WARN (fi-glk-dsi)
Test vgem_basic:
Subgroup setversion:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup create:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup debugfs:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup dmabuf-export:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup dmabuf-fence:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup dmabuf-fence-before:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup dmabuf-mmap:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup mmap:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup second-client:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup sysfs:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup unload:
pass -> DMESG-WARN (fi-glk-dsi)
Test drv_module_reload:
Subgroup basic-reload:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-no-display:
pass -> DMESG-WARN (fi-glk-dsi)
Subgroup basic-reload-inject:
pass -> DMESG-WARN (fi-glk-dsi)fdo#102514 https://bugs.freedesktop.org/show_bug.cgi?id=102514
fdo#103514 https://bugs.freedesktop.org/show_bug.cgi?id=103514
fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
fi-bdw-5557u total:289 pass:268 dwarn:0 dfail:0 fail:0 skip:21 time:444s fi-bdw-gvtdvm total:289 pass:265 dwarn:0 dfail:0 fail:0 skip:24 time:450s fi-blb-e6850 total:289 pass:223 dwarn:1 dfail:0 fail:0 skip:65 time:382s fi-bsw-n3050 total:289 pass:243 dwarn:0 dfail:0 fail:0 skip:46 time:556s fi-bwr-2160 total:289 pass:183 dwarn:0 dfail:0 fail:0 skip:106 time:278s fi-bxt-dsi total:289 pass:259 dwarn:0 dfail:0 fail:0 skip:30 time:512s fi-bxt-j4205 total:289 pass:260 dwarn:0 dfail:0 fail:0 skip:29 time:507s fi-byt-j1900 total:289 pass:253 dwarn:1 dfail:0 fail:0 skip:35 time:512s fi-byt-n2820 total:289 pass:249 dwarn:1 dfail:0 fail:0 skip:39 time:493s fi-cfl-s total:289 pass:254 dwarn:3 dfail:0 fail:0 skip:32 time:554s fi-elk-e7500 total:289 pass:229 dwarn:0 dfail:0 fail:0 skip:60 time:439s fi-gdg-551 total:289 pass:178 dwarn:1 dfail:0 fail:1 skip:109 time:264s fi-glk-1 total:289 pass:261 dwarn:0 dfail:0 fail:0 skip:28 time:583s fi-glk-dsi total:289 pass:82 dwarn:176 dfail:1 fail:0 skip:30 time:623s fi-hsw-4770 total:289 pass:262 dwarn:0 dfail:0 fail:0 skip:27 time:434s fi-hsw-4770r total:289 pass:262 dwarn:0 dfail:0 fail:0 skip:27 time:432s fi-ilk-650 total:289 pass:228 dwarn:0 dfail:0 fail:0 skip:61 time:427s fi-ivb-3770 total:289 pass:260 dwarn:0 dfail:0 fail:0 skip:29 time:461s fi-kbl-7500u total:289 pass:264 dwarn:1 dfail:0 fail:0 skip:24 time:493s fi-kbl-7560u total:289 pass:270 dwarn:0 dfail:0 fail:0 skip:19 time:577s fi-kbl-7567u total:289 pass:269 dwarn:0 dfail:0 fail:0 skip:20 time:481s fi-kbl-r total:289 pass:262 dwarn:0 dfail:0 fail:0 skip:27 time:589s fi-pnv-d510 total:289 pass:222 dwarn:1 dfail:0 fail:0 skip:66 time:576s fi-skl-6260u total:289 pass:269 dwarn:0 dfail:0 fail:0 skip:20 time:455s fi-skl-6600u total:289 pass:262 dwarn:0 dfail:0 fail:0 skip:27 time:597s fi-skl-6700hq total:289 pass:263 dwarn:0 dfail:0 fail:0 skip:26 time:652s fi-skl-6700k total:289 pass:265 dwarn:0 dfail:0 fail:0 skip:24 time:534s fi-skl-6770hq total:289 pass:269 dwarn:0 dfail:0 fail:0 skip:20 time:504s fi-skl-gvtdvm total:289 pass:266 dwarn:0 dfail:0 fail:0 skip:23 time:457s fi-snb-2520m total:289 pass:250 dwarn:0 dfail:0 fail:0 skip:39 time:586s fi-snb-2600 total:289 pass:249 dwarn:0 dfail:0 fail:0 skip:40 time:424s
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_471/
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Patchwork
CI✗ Fi.CI.BAT: warning for series starting with [v2,1/3] lib/core: Avoid unused result in backtrace printing (rev2)
In reply to Tvrtko Ursulin== Series Details ==
== Series Details ==
Series: series starting with [v2,1/3] lib/core: Avoid unused result in backtrace printing (rev2)
URL : https://patchwork.freedesktop.org/series/33119/
State : warning
== Summary ==
IGT patchset tested on top of latest successful build c8d1ea24d3bfaf11b223bbe22407aeca196d0d89 tests/debugfs_test: Pretty print subdirectories
with latest DRM-Tip kernel build CI_DRM_33112f945d948d05 drm-tip: 2017y-11m-03d-09h-28m-20s UTC integration manifest
No testlist changes.
Test chamelium:
Subgroup dp-crc-fast:
incomplete -> PASS (fi-kbl-7500u) fdo#102514
Test kms_pipe_crc_basic:
Subgroup read-crc-pipe-b:
pass -> SKIP (fi-hsw-4770r)fdo#102514 https://bugs.freedesktop.org/show_bug.cgi?id=102514
fi-bdw-5557u total:289 pass:268 dwarn:0 dfail:0 fail:0 skip:21 time:446s fi-blb-e6850 total:289 pass:223 dwarn:1 dfail:0 fail:0 skip:65 time:380s fi-bsw-n3050 total:289 pass:243 dwarn:0 dfail:0 fail:0 skip:46 time:543s fi-bwr-2160 total:289 pass:183 dwarn:0 dfail:0 fail:0 skip:106 time:277s fi-bxt-dsi total:289 pass:259 dwarn:0 dfail:0 fail:0 skip:30 time:510s fi-bxt-j4205 total:289 pass:260 dwarn:0 dfail:0 fail:0 skip:29 time:504s fi-byt-j1900 total:289 pass:253 dwarn:1 dfail:0 fail:0 skip:35 time:512s fi-byt-n2820 total:289 pass:249 dwarn:1 dfail:0 fail:0 skip:39 time:492s fi-cfl-s total:289 pass:254 dwarn:3 dfail:0 fail:0 skip:32 time:560s fi-elk-e7500 total:289 pass:229 dwarn:0 dfail:0 fail:0 skip:60 time:428s fi-gdg-551 total:289 pass:178 dwarn:1 dfail:0 fail:1 skip:109 time:261s fi-glk-1 total:289 pass:261 dwarn:0 dfail:0 fail:0 skip:28 time:589s fi-glk-dsi total:289 pass:258 dwarn:0 dfail:0 fail:1 skip:30 time:494s fi-hsw-4770 total:289 pass:262 dwarn:0 dfail:0 fail:0 skip:27 time:430s fi-hsw-4770r total:289 pass:261 dwarn:0 dfail:0 fail:0 skip:28 time:437s fi-ilk-650 total:289 pass:228 dwarn:0 dfail:0 fail:0 skip:61 time:432s fi-ivb-3520m total:289 pass:260 dwarn:0 dfail:0 fail:0 skip:29 time:496s fi-ivb-3770 total:289 pass:260 dwarn:0 dfail:0 fail:0 skip:29 time:467s fi-kbl-7500u total:289 pass:264 dwarn:1 dfail:0 fail:0 skip:24 time:492s fi-kbl-7560u total:289 pass:270 dwarn:0 dfail:0 fail:0 skip:19 time:571s fi-kbl-7567u total:289 pass:269 dwarn:0 dfail:0 fail:0 skip:20 time:480s fi-kbl-r total:289 pass:262 dwarn:0 dfail:0 fail:0 skip:27 time:588s fi-pnv-d510 total:289 pass:222 dwarn:1 dfail:0 fail:0 skip:66 time:574s fi-skl-6260u total:289 pass:269 dwarn:0 dfail:0 fail:0 skip:20 time:453s fi-skl-6600u total:289 pass:262 dwarn:0 dfail:0 fail:0 skip:27 time:594s fi-skl-6700hq total:289 pass:263 dwarn:0 dfail:0 fail:0 skip:26 time:652s fi-skl-6700k total:289 pass:265 dwarn:0 dfail:0 fail:0 skip:24 time:529s fi-skl-6770hq total:289 pass:269 dwarn:0 dfail:0 fail:0 skip:20 time:510s fi-skl-gvtdvm total:289 pass:266 dwarn:0 dfail:0 fail:0 skip:23 time:464s fi-snb-2520m total:289 pass:250 dwarn:0 dfail:0 fail:0 skip:39 time:578s fi-snb-2600 total:289 pass:249 dwarn:0 dfail:0 fail:0 skip:40 time:436s
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_473/
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx