* [igt-dev] [PATCH i-g-t] meson: Turn on -Werror
@ 2021-01-30 6:44 Ashutosh Dixit
2021-01-30 7:23 ` [igt-dev] ✗ GitLab.Pipeline: warning for meson: Turn on -Werror (rev2) Patchwork
` (9 more replies)
0 siblings, 10 replies; 13+ messages in thread
From: Ashutosh Dixit @ 2021-01-30 6:44 UTC (permalink / raw)
To: igt-dev
We can choose which compile warnings to enable, but once they are enabled
treat all warnings as error. This enforces stricter checks against compile
warnings creeping in.
v2: Fix redefinition warning errors from i915/gem_userptr_blits
Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
---
meson.build | 28 +++++++++++++++-------------
tests/i915/gem_userptr_blits.c | 2 --
2 files changed, 15 insertions(+), 15 deletions(-)
diff --git a/meson.build b/meson.build
index f4ee14f14a..3f3693f99d 100644
--- a/meson.build
+++ b/meson.build
@@ -51,19 +51,21 @@ cc_args = [
'-Wno-unused-parameter',
'-Wno-unused-result',
- '-Werror=address',
- '-Werror=array-bounds',
- '-Werror=implicit',
- '-Werror=init-self',
- '-Werror=int-to-pointer-cast',
- '-Werror=main',
- '-Werror=missing-braces',
- '-Werror=nonnull',
- '-Werror=pointer-to-int-cast',
- '-Werror=return-type',
- '-Werror=sequence-point',
- '-Werror=trigraphs',
- '-Werror=write-strings',
+ '-Waddress',
+ '-Warray-bounds',
+ '-Wimplicit',
+ '-Winit-self',
+ '-Wint-to-pointer-cast',
+ '-Wmain',
+ '-Wmissing-braces',
+ '-Wnonnull',
+ '-Wpointer-to-int-cast',
+ '-Wreturn-type',
+ '-Wsequence-point',
+ '-Wtrigraphs',
+ '-Wwrite-strings',
+
+ '-Werror',
# Disable the memory allocating builtins as they may cause unexpected behavior
# with our framework. They *may* get optimized out in favor of a register or
# stack variable, making them effectively local. Local variables do not play
diff --git a/tests/i915/gem_userptr_blits.c b/tests/i915/gem_userptr_blits.c
index 2f3954c5d6..31bca3b78f 100644
--- a/tests/i915/gem_userptr_blits.c
+++ b/tests/i915/gem_userptr_blits.c
@@ -55,8 +55,6 @@
#include <pthread.h>
#include <time.h>
-#include <linux/memfd.h>
-
#include "drm.h"
#include "i915_drm.h"
--
2.29.2
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 13+ messages in thread* [igt-dev] ✗ GitLab.Pipeline: warning for meson: Turn on -Werror (rev2) 2021-01-30 6:44 [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Ashutosh Dixit @ 2021-01-30 7:23 ` Patchwork 2021-01-30 7:45 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork ` (8 subsequent siblings) 9 siblings, 0 replies; 13+ messages in thread From: Patchwork @ 2021-01-30 7:23 UTC (permalink / raw) To: Ashutosh Dixit; +Cc: igt-dev == Series Details == Series: meson: Turn on -Werror (rev2) URL : https://patchwork.freedesktop.org/series/86465/ State : warning == Summary == Did not get list of undocumented tests for this run, something is wrong! Other than that, pipeline status: FAILED. see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/263939 for the overview. build:tests-fedora-clang has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/6995301): [7/975] Linking static target lib/libigt-i915_gem_submission_c.a. [8/975] Linking static target lib/libigt-i915_gem_ring_c.a. [9/975] Linking static target lib/libigt-i915_gem_vm_c.a. [10/975] Linking static target lib/libigt-i915_gem_mman_c.a. [11/975] Linking static target lib/libigt-igt_collection_c.a. [12/975] Linking static target lib/libigt-igt_device_scan_c.a. [13/975] Compiling C object 'lib/76b5a35@@igt-igt_aux_c@sta/igt_aux.c.o'. FAILED: lib/76b5a35@@igt-igt_aux_c@sta/igt_aux.c.o clang -Ilib/76b5a35@@igt-igt_aux_c@sta -Ilib -I../lib -I../include/drm-uapi -I../lib/stubs/syscalls -I. -I../ -I/usr/include/cairo -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/libdrm -I/usr/include/valgrind -Xclang -fcolor-diagnostics -pipe -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=gnu11 -O2 -g -D_GNU_SOURCE -include config.h -D_FORTIFY_SOURCE=2 -Wbad-function-cast -Wdeclaration-after-statement -Wformat=2 -Wmissing-declarations -Wmissing-format-attribute -Wmissing-noreturn -Wmissing-prototypes -Wnested-externs -Wold-style-definition -Wpointer-arith -Wredundant-decls -Wshadow -Wstrict-prototypes -Wuninitialized -Wunused -Wno-missing-field-initializers -Wno-pointer-arith -Wno-address-of-packed-member -Wno-sign-compare -Wno-type-limits -Wno-unused-parameter -Wno-unused-result -Waddress -Warray-bounds -Wimplicit -Winit-self -Wint-to-pointer-cast -Wmain -Wmissing-braces -Wnonnu ll -Wpointer-to-int-cast -Wreturn-type -Wsequence-point -Wtrigraphs -Wwrite-strings -Werror -fno-builtin-malloc -fno-builtin-calloc -fcommon -fPIC -pthread '-DIGT_DATADIR="/usr/local/share/igt-gpu-tools"' '-DIGT_SRCDIR="/builds/gfx-ci/igt-ci-tags/tests"' '-DIGT_LOG_DOMAIN="igt_aux"' -MD -MQ 'lib/76b5a35@@igt-igt_aux_c@sta/igt_aux.c.o' -MF 'lib/76b5a35@@igt-igt_aux_c@sta/igt_aux.c.o.d' -o 'lib/76b5a35@@igt-igt_aux_c@sta/igt_aux.c.o' -c ../lib/igt_aux.c ../lib/igt_aux.c:500:1: error: function 'sig_abort' could be declared with attribute 'noreturn' [-Werror,-Wmissing-noreturn] { ^ 1 error generated. ninja: build stopped: subcommand failed. section_end:1611990903:step_script section_start:1611990904:cleanup_file_variables Cleaning up file based variables section_end:1611990909:cleanup_file_variables ERROR: Job failed: exit code 1 == Logs == For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/263939 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for meson: Turn on -Werror (rev2) 2021-01-30 6:44 [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Ashutosh Dixit 2021-01-30 7:23 ` [igt-dev] ✗ GitLab.Pipeline: warning for meson: Turn on -Werror (rev2) Patchwork @ 2021-01-30 7:45 ` Patchwork 2021-01-30 9:19 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork ` (7 subsequent siblings) 9 siblings, 0 replies; 13+ messages in thread From: Patchwork @ 2021-01-30 7:45 UTC (permalink / raw) To: Ashutosh Dixit; +Cc: igt-dev [-- Attachment #1.1: Type: text/plain, Size: 3055 bytes --] == Series Details == Series: meson: Turn on -Werror (rev2) URL : https://patchwork.freedesktop.org/series/86465/ State : success == Summary == CI Bug Log - changes from CI_DRM_9705 -> IGTPW_5448 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/index.html Known issues ------------ Here are the changes found in IGTPW_5448 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@debugfs_test@read_all_entries: - fi-tgl-y: [PASS][1] -> [DMESG-WARN][2] ([i915#402]) +2 similar issues [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/fi-tgl-y/igt@debugfs_test@read_all_entries.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/fi-tgl-y/igt@debugfs_test@read_all_entries.html * igt@i915_selftest@live@sanitycheck: - fi-kbl-7500u: [PASS][3] -> [DMESG-WARN][4] ([i915#2605]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/fi-kbl-7500u/igt@i915_selftest@live@sanitycheck.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/fi-kbl-7500u/igt@i915_selftest@live@sanitycheck.html #### Possible fixes #### * igt@gem_linear_blits@basic: - fi-tgl-y: [DMESG-WARN][5] ([i915#402]) -> [PASS][6] +1 similar issue [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/fi-tgl-y/igt@gem_linear_blits@basic.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/fi-tgl-y/igt@gem_linear_blits@basic.html * igt@i915_selftest@live@gem_contexts: - fi-tgl-y: [SKIP][7] ([i915#2966]) -> [PASS][8] +13 similar issues [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/fi-tgl-y/igt@i915_selftest@live@gem_contexts.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/fi-tgl-y/igt@i915_selftest@live@gem_contexts.html * igt@i915_selftest@live@gtt: - fi-tgl-y: [FAIL][9] -> [PASS][10] [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/fi-tgl-y/igt@i915_selftest@live@gtt.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/fi-tgl-y/igt@i915_selftest@live@gtt.html [i915#2605]: https://gitlab.freedesktop.org/drm/intel/issues/2605 [i915#2966]: https://gitlab.freedesktop.org/drm/intel/issues/2966 [i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402 Participating hosts (44 -> 38) ------------------------------ Missing (6): fi-jsl-1 fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-dg1-1 fi-bdw-samus Build changes ------------- * CI: CI-20190529 -> None * IGT: IGT_5979 -> IGTPW_5448 CI-20190529: 20190529 CI_DRM_9705: 7b1b769c42017c6e6d603dbea0182e74090451f1 @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_5448: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/index.html IGT_5979: fdc23507d022b68443121ec2c1a951af27c87240 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/index.html [-- Attachment #1.2: Type: text/html, Size: 3839 bytes --] [-- Attachment #2: Type: text/plain, Size: 154 bytes --] _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for meson: Turn on -Werror (rev2) 2021-01-30 6:44 [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Ashutosh Dixit 2021-01-30 7:23 ` [igt-dev] ✗ GitLab.Pipeline: warning for meson: Turn on -Werror (rev2) Patchwork 2021-01-30 7:45 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork @ 2021-01-30 9:19 ` Patchwork 2021-01-30 12:11 ` [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Chris Wilson ` (6 subsequent siblings) 9 siblings, 0 replies; 13+ messages in thread From: Patchwork @ 2021-01-30 9:19 UTC (permalink / raw) To: Ashutosh Dixit; +Cc: igt-dev [-- Attachment #1.1: Type: text/plain, Size: 30247 bytes --] == Series Details == Series: meson: Turn on -Werror (rev2) URL : https://patchwork.freedesktop.org/series/86465/ State : success == Summary == CI Bug Log - changes from CI_DRM_9705_full -> IGTPW_5448_full ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/index.html Known issues ------------ Here are the changes found in IGTPW_5448_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_isolation@preservation-s3@vcs1: - shard-kbl: [PASS][1] -> [DMESG-WARN][2] ([i915#180]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl2/igt@gem_ctx_isolation@preservation-s3@vcs1.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl4/igt@gem_ctx_isolation@preservation-s3@vcs1.html * igt@gem_ctx_persistence@close-replace-race: - shard-glk: [PASS][3] -> [TIMEOUT][4] ([i915#2918]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk4/igt@gem_ctx_persistence@close-replace-race.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-glk4/igt@gem_ctx_persistence@close-replace-race.html * igt@gem_ctx_persistence@process: - shard-snb: NOTRUN -> [SKIP][5] ([fdo#109271] / [i915#1099]) +1 similar issue [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-snb2/igt@gem_ctx_persistence@process.html * igt@gem_ctx_persistence@processes: - shard-hsw: NOTRUN -> [SKIP][6] ([fdo#109271] / [i915#1099]) +2 similar issues [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-hsw2/igt@gem_ctx_persistence@processes.html * igt@gem_ctx_sseu@invalid-sseu: - shard-tglb: NOTRUN -> [SKIP][7] ([i915#280]) [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb7/igt@gem_ctx_sseu@invalid-sseu.html * igt@gem_exec_balancer@full-pulse: - shard-snb: NOTRUN -> [SKIP][8] ([fdo#109271]) +60 similar issues [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-snb5/igt@gem_exec_balancer@full-pulse.html * igt@gem_exec_fair@basic-none@vecs0: - shard-kbl: [PASS][9] -> [FAIL][10] ([i915#2842]) +1 similar issue [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl2/igt@gem_exec_fair@basic-none@vecs0.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl6/igt@gem_exec_fair@basic-none@vecs0.html * igt@gem_exec_fair@basic-pace-share@rcs0: - shard-tglb: [PASS][11] -> [FAIL][12] ([i915#2842]) +1 similar issue [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-tglb2/igt@gem_exec_fair@basic-pace-share@rcs0.html [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb2/igt@gem_exec_fair@basic-pace-share@rcs0.html * igt@gem_exec_fair@basic-pace@vcs1: - shard-kbl: [PASS][13] -> [SKIP][14] ([fdo#109271]) [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl4/igt@gem_exec_fair@basic-pace@vcs1.html [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl4/igt@gem_exec_fair@basic-pace@vcs1.html * igt@gem_exec_fair@basic-throttle@rcs0: - shard-glk: [PASS][15] -> [FAIL][16] ([i915#2842]) +3 similar issues [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk4/igt@gem_exec_fair@basic-throttle@rcs0.html [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-glk2/igt@gem_exec_fair@basic-throttle@rcs0.html * igt@gem_exec_params@rsvd2-dirt: - shard-tglb: NOTRUN -> [SKIP][17] ([fdo#109283]) [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb5/igt@gem_exec_params@rsvd2-dirt.html - shard-iclb: NOTRUN -> [SKIP][18] ([fdo#109283]) [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb1/igt@gem_exec_params@rsvd2-dirt.html * igt@gem_exec_reloc@basic-many-active@rcs0: - shard-apl: [PASS][19] -> [FAIL][20] ([i915#2389]) [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-apl4/igt@gem_exec_reloc@basic-many-active@rcs0.html [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl1/igt@gem_exec_reloc@basic-many-active@rcs0.html * igt@gem_exec_schedule@u-fairslice-all: - shard-iclb: [PASS][21] -> [DMESG-WARN][22] ([i915#2803]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-iclb7/igt@gem_exec_schedule@u-fairslice-all.html [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb3/igt@gem_exec_schedule@u-fairslice-all.html * igt@gem_huc_copy@huc-copy: - shard-apl: NOTRUN -> [SKIP][23] ([fdo#109271] / [i915#2190]) [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl1/igt@gem_huc_copy@huc-copy.html * igt@gem_pread@exhaustion: - shard-apl: NOTRUN -> [WARN][24] ([i915#2658]) [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl4/igt@gem_pread@exhaustion.html * igt@gem_render_copy@y-tiled-ccs-to-y-tiled-mc-ccs: - shard-iclb: NOTRUN -> [SKIP][25] ([i915#768]) +1 similar issue [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb6/igt@gem_render_copy@y-tiled-ccs-to-y-tiled-mc-ccs.html * igt@gem_softpin@evict-snoop: - shard-tglb: NOTRUN -> [SKIP][26] ([fdo#109312]) [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb3/igt@gem_softpin@evict-snoop.html * igt@gem_userptr_blits@coherency-sync: - shard-tglb: NOTRUN -> [SKIP][27] ([fdo#110542]) [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb8/igt@gem_userptr_blits@coherency-sync.html * igt@gem_userptr_blits@input-checking: - shard-kbl: NOTRUN -> [DMESG-WARN][28] ([i915#3002]) [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl6/igt@gem_userptr_blits@input-checking.html * igt@gem_userptr_blits@process-exit-mmap-busy@uc: - shard-tglb: NOTRUN -> [SKIP][29] ([i915#1699]) +3 similar issues [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb3/igt@gem_userptr_blits@process-exit-mmap-busy@uc.html * igt@gem_userptr_blits@process-exit-mmap@wb: - shard-apl: NOTRUN -> [SKIP][30] ([fdo#109271] / [i915#1699]) +3 similar issues [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl2/igt@gem_userptr_blits@process-exit-mmap@wb.html * igt@gen3_mixed_blits: - shard-iclb: NOTRUN -> [SKIP][31] ([fdo#109289]) +3 similar issues [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb4/igt@gen3_mixed_blits.html - shard-tglb: NOTRUN -> [SKIP][32] ([fdo#109289]) +2 similar issues [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb7/igt@gen3_mixed_blits.html * igt@gen9_exec_parse@cmd-crossing-page: - shard-tglb: NOTRUN -> [SKIP][33] ([fdo#112306]) [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb7/igt@gen9_exec_parse@cmd-crossing-page.html * igt@i915_pm_rc6_residency@media-rc6-accuracy: - shard-tglb: NOTRUN -> [SKIP][34] ([fdo#109289] / [fdo#111719]) [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb2/igt@i915_pm_rc6_residency@media-rc6-accuracy.html * igt@i915_pm_rc6_residency@rc6-fence: - shard-iclb: NOTRUN -> [WARN][35] ([i915#2684]) [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb2/igt@i915_pm_rc6_residency@rc6-fence.html * igt@i915_pm_rc6_residency@rc6-idle: - shard-tglb: NOTRUN -> [WARN][36] ([i915#2681] / [i915#2684]) [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb7/igt@i915_pm_rc6_residency@rc6-idle.html * igt@i915_suspend@fence-restore-untiled: - shard-kbl: [PASS][37] -> [INCOMPLETE][38] ([i915#155] / [i915#180]) [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl7/igt@i915_suspend@fence-restore-untiled.html [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl4/igt@i915_suspend@fence-restore-untiled.html * igt@kms_async_flips@test-time-stamp: - shard-tglb: [PASS][39] -> [FAIL][40] ([i915#2597]) [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-tglb5/igt@kms_async_flips@test-time-stamp.html [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb5/igt@kms_async_flips@test-time-stamp.html * igt@kms_big_fb@linear-8bpp-rotate-90: - shard-tglb: NOTRUN -> [SKIP][41] ([fdo#111614]) +4 similar issues [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb3/igt@kms_big_fb@linear-8bpp-rotate-90.html * igt@kms_big_fb@x-tiled-16bpp-rotate-270: - shard-iclb: NOTRUN -> [SKIP][42] ([fdo#110725] / [fdo#111614]) +3 similar issues [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb4/igt@kms_big_fb@x-tiled-16bpp-rotate-270.html * igt@kms_big_fb@yf-tiled-64bpp-rotate-180: - shard-tglb: NOTRUN -> [SKIP][43] ([fdo#111615]) +2 similar issues [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb3/igt@kms_big_fb@yf-tiled-64bpp-rotate-180.html - shard-iclb: NOTRUN -> [SKIP][44] ([fdo#110723]) [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb4/igt@kms_big_fb@yf-tiled-64bpp-rotate-180.html * igt@kms_big_joiner@basic: - shard-tglb: NOTRUN -> [SKIP][45] ([i915#2705]) [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb3/igt@kms_big_joiner@basic.html - shard-iclb: NOTRUN -> [SKIP][46] ([i915#2705]) +1 similar issue [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb4/igt@kms_big_joiner@basic.html * igt@kms_chamelium@dp-frame-dump: - shard-iclb: NOTRUN -> [SKIP][47] ([fdo#109284] / [fdo#111827]) +14 similar issues [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb4/igt@kms_chamelium@dp-frame-dump.html * igt@kms_chamelium@vga-hpd: - shard-apl: NOTRUN -> [SKIP][48] ([fdo#109271] / [fdo#111827]) +12 similar issues [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl7/igt@kms_chamelium@vga-hpd.html * igt@kms_color@pipe-a-degamma: - shard-tglb: NOTRUN -> [FAIL][49] ([i915#1149]) [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb7/igt@kms_color@pipe-a-degamma.html * igt@kms_color@pipe-d-ctm-0-75: - shard-tglb: NOTRUN -> [FAIL][50] ([i915#1149] / [i915#315]) [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb1/igt@kms_color@pipe-d-ctm-0-75.html * igt@kms_color_chamelium@pipe-b-ctm-0-5: - shard-glk: NOTRUN -> [SKIP][51] ([fdo#109271] / [fdo#111827]) +8 similar issues [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-glk9/igt@kms_color_chamelium@pipe-b-ctm-0-5.html * igt@kms_color_chamelium@pipe-d-ctm-red-to-blue: - shard-tglb: NOTRUN -> [SKIP][52] ([fdo#109284] / [fdo#111827]) +11 similar issues [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb3/igt@kms_color_chamelium@pipe-d-ctm-red-to-blue.html * igt@kms_color_chamelium@pipe-d-gamma: - shard-hsw: NOTRUN -> [SKIP][53] ([fdo#109271] / [fdo#111827]) +4 similar issues [53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-hsw6/igt@kms_color_chamelium@pipe-d-gamma.html - shard-iclb: NOTRUN -> [SKIP][54] ([fdo#109278] / [fdo#109284] / [fdo#111827]) +1 similar issue [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb2/igt@kms_color_chamelium@pipe-d-gamma.html * igt@kms_color_chamelium@pipe-invalid-gamma-lut-sizes: - shard-kbl: NOTRUN -> [SKIP][55] ([fdo#109271] / [fdo#111827]) +8 similar issues [55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl6/igt@kms_color_chamelium@pipe-invalid-gamma-lut-sizes.html - shard-snb: NOTRUN -> [SKIP][56] ([fdo#109271] / [fdo#111827]) +1 similar issue [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-snb2/igt@kms_color_chamelium@pipe-invalid-gamma-lut-sizes.html * igt@kms_content_protection@legacy: - shard-iclb: NOTRUN -> [SKIP][57] ([fdo#109300] / [fdo#111066]) [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb2/igt@kms_content_protection@legacy.html - shard-kbl: NOTRUN -> [TIMEOUT][58] ([i915#1319]) +1 similar issue [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl1/igt@kms_content_protection@legacy.html - shard-tglb: NOTRUN -> [SKIP][59] ([fdo#111828]) [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb6/igt@kms_content_protection@legacy.html - shard-apl: NOTRUN -> [TIMEOUT][60] ([i915#1319]) [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl7/igt@kms_content_protection@legacy.html * igt@kms_cursor_crc@pipe-a-cursor-suspend: - shard-apl: NOTRUN -> [DMESG-WARN][61] ([i915#180]) [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl1/igt@kms_cursor_crc@pipe-a-cursor-suspend.html * igt@kms_cursor_crc@pipe-c-cursor-512x512-random: - shard-iclb: NOTRUN -> [SKIP][62] ([fdo#109278] / [fdo#109279]) [62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb6/igt@kms_cursor_crc@pipe-c-cursor-512x512-random.html * igt@kms_cursor_crc@pipe-d-cursor-256x256-onscreen: - shard-hsw: NOTRUN -> [SKIP][63] ([fdo#109271] / [i915#533]) +6 similar issues [63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-hsw1/igt@kms_cursor_crc@pipe-d-cursor-256x256-onscreen.html * igt@kms_cursor_crc@pipe-d-cursor-512x512-sliding: - shard-tglb: NOTRUN -> [SKIP][64] ([fdo#109279]) +2 similar issues [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb7/igt@kms_cursor_crc@pipe-d-cursor-512x512-sliding.html * igt@kms_cursor_edge_walk@pipe-d-256x256-left-edge: - shard-iclb: NOTRUN -> [SKIP][65] ([fdo#109278]) +19 similar issues [65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb8/igt@kms_cursor_edge_walk@pipe-d-256x256-left-edge.html * igt@kms_cursor_legacy@cursor-vs-flip-varying-size: - shard-hsw: [PASS][66] -> [FAIL][67] ([i915#2370]) [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-hsw6/igt@kms_cursor_legacy@cursor-vs-flip-varying-size.html [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-hsw8/igt@kms_cursor_legacy@cursor-vs-flip-varying-size.html * igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions-varying-size: - shard-iclb: NOTRUN -> [SKIP][68] ([fdo#109274] / [fdo#109278]) +3 similar issues [68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb5/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions-varying-size.html * igt@kms_cursor_legacy@pipe-d-torture-bo: - shard-kbl: NOTRUN -> [SKIP][69] ([fdo#109271] / [i915#533]) +1 similar issue [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl6/igt@kms_cursor_legacy@pipe-d-torture-bo.html * igt@kms_dp_tiled_display@basic-test-pattern: - shard-iclb: NOTRUN -> [SKIP][70] ([i915#426]) [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb2/igt@kms_dp_tiled_display@basic-test-pattern.html - shard-tglb: NOTRUN -> [SKIP][71] ([i915#426]) [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb3/igt@kms_dp_tiled_display@basic-test-pattern.html * igt@kms_fbcon_fbt@fbc-suspend: - shard-kbl: [PASS][72] -> [INCOMPLETE][73] ([i915#155] / [i915#180] / [i915#636]) [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl2/igt@kms_fbcon_fbt@fbc-suspend.html [73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl4/igt@kms_fbcon_fbt@fbc-suspend.html * igt@kms_flip@2x-flip-vs-dpms: - shard-iclb: NOTRUN -> [SKIP][74] ([fdo#109274]) +3 similar issues [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb6/igt@kms_flip@2x-flip-vs-dpms.html * igt@kms_flip@flip-vs-expired-vblank@b-hdmi-a1: - shard-glk: [PASS][75] -> [FAIL][76] ([i915#79]) [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk7/igt@kms_flip@flip-vs-expired-vblank@b-hdmi-a1.html [76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-glk6/igt@kms_flip@flip-vs-expired-vblank@b-hdmi-a1.html * igt@kms_flip@flip-vs-suspend-interruptible@a-dp1: - shard-apl: [PASS][77] -> [DMESG-WARN][78] ([i915#180]) [77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-apl1/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html [78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl7/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html * igt@kms_flip@flip-vs-suspend@c-dp1: - shard-kbl: NOTRUN -> [DMESG-WARN][79] ([i915#180]) +1 similar issue [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl4/igt@kms_flip@flip-vs-suspend@c-dp1.html * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-mmap-cpu: - shard-snb: NOTRUN -> [FAIL][80] ([i915#2546]) [80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-snb6/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-mmap-cpu.html * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-render: - shard-glk: [PASS][81] -> [FAIL][82] ([i915#49]) [81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk8/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-render.html [82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-glk6/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-render.html * igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-indfb-draw-mmap-cpu: - shard-glk: NOTRUN -> [SKIP][83] ([fdo#109271]) +63 similar issues [83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-glk2/igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-indfb-draw-mmap-cpu.html * igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-cur-indfb-draw-mmap-wc: - shard-kbl: NOTRUN -> [SKIP][84] ([fdo#109271]) +77 similar issues [84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl7/igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-cur-indfb-draw-mmap-wc.html * igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-cur-indfb-draw-pwrite: - shard-hsw: NOTRUN -> [SKIP][85] ([fdo#109271]) +40 similar issues [85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-hsw8/igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-cur-indfb-draw-pwrite.html * igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-spr-indfb-draw-pwrite: - shard-iclb: NOTRUN -> [SKIP][86] ([fdo#109280]) +40 similar issues [86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-spr-indfb-draw-pwrite.html * igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-pwrite: - shard-tglb: NOTRUN -> [SKIP][87] ([fdo#111825]) +35 similar issues [87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb5/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-pwrite.html * igt@kms_pipe_crc_basic@disable-crc-after-crtc-pipe-d: - shard-apl: NOTRUN -> [SKIP][88] ([fdo#109271] / [i915#533]) [88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl7/igt@kms_pipe_crc_basic@disable-crc-after-crtc-pipe-d.html * igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes: - shard-apl: [PASS][89] -> [DMESG-WARN][90] ([i915#180] / [i915#533]) [89]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-apl1/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html [90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl7/igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes.html * igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb: - shard-kbl: NOTRUN -> [FAIL][91] ([fdo#108145] / [i915#265]) [91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl7/igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb.html - shard-apl: NOTRUN -> [FAIL][92] ([fdo#108145] / [i915#265]) [92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl2/igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb.html - shard-glk: NOTRUN -> [FAIL][93] ([fdo#108145] / [i915#265]) [93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-glk3/igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb.html * igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-4: - shard-iclb: NOTRUN -> [SKIP][94] ([i915#658]) +1 similar issue [94]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb8/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-4.html * igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5: - shard-glk: NOTRUN -> [SKIP][95] ([fdo#109271] / [i915#658]) +1 similar issue [95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-glk9/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5.html - shard-tglb: NOTRUN -> [SKIP][96] ([i915#2920]) +2 similar issues [96]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb6/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5.html - shard-kbl: NOTRUN -> [SKIP][97] ([fdo#109271] / [i915#658]) [97]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl1/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5.html - shard-iclb: NOTRUN -> [SKIP][98] ([i915#2920]) [98]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb2/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5.html * igt@kms_psr2_sf@plane-move-sf-dmg-area-2: - shard-apl: NOTRUN -> [SKIP][99] ([fdo#109271] / [i915#658]) [99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl2/igt@kms_psr2_sf@plane-move-sf-dmg-area-2.html * igt@kms_psr@psr2_primary_page_flip: - shard-iclb: NOTRUN -> [SKIP][100] ([fdo#109441]) +3 similar issues [100]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb3/igt@kms_psr@psr2_primary_page_flip.html * igt@kms_psr@psr2_suspend: - shard-iclb: [PASS][101] -> [SKIP][102] ([fdo#109441]) +1 similar issue [101]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-iclb2/igt@kms_psr@psr2_suspend.html [102]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb5/igt@kms_psr@psr2_suspend.html * igt@kms_psr@sprite_blt: - shard-hsw: NOTRUN -> [SKIP][103] ([fdo#109271] / [i915#1072]) +1 similar issue [103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-hsw2/igt@kms_psr@sprite_blt.html * igt@kms_sysfs_edid_timing: - shard-kbl: NOTRUN -> [FAIL][104] ([IGT#2]) [104]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl1/igt@kms_sysfs_edid_timing.html * igt@kms_vblank@pipe-d-wait-forked-hang: - shard-apl: NOTRUN -> [SKIP][105] ([fdo#109271]) +95 similar issues [105]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl3/igt@kms_vblank@pipe-d-wait-forked-hang.html * igt@kms_writeback@writeback-invalid-parameters: - shard-tglb: NOTRUN -> [SKIP][106] ([i915#2437]) [106]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb7/igt@kms_writeback@writeback-invalid-parameters.html - shard-iclb: NOTRUN -> [SKIP][107] ([i915#2437]) [107]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb6/igt@kms_writeback@writeback-invalid-parameters.html * igt@nouveau_crc@pipe-a-source-rg: - shard-iclb: NOTRUN -> [SKIP][108] ([i915#2530]) +2 similar issues [108]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb6/igt@nouveau_crc@pipe-a-source-rg.html * igt@nouveau_crc@pipe-b-source-outp-complete: - shard-tglb: NOTRUN -> [SKIP][109] ([i915#2530]) [109]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb5/igt@nouveau_crc@pipe-b-source-outp-complete.html * igt@prime_nv_pcopy@test3_1: - shard-tglb: NOTRUN -> [SKIP][110] ([fdo#109291]) +3 similar issues [110]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb2/igt@prime_nv_pcopy@test3_1.html * igt@prime_nv_pcopy@test3_4: - shard-iclb: NOTRUN -> [SKIP][111] ([fdo#109291]) +4 similar issues [111]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb5/igt@prime_nv_pcopy@test3_4.html * igt@prime_vgem@coherency-gtt: - shard-tglb: NOTRUN -> [SKIP][112] ([fdo#111656]) [112]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb3/igt@prime_vgem@coherency-gtt.html #### Possible fixes #### * igt@gem_exec_fair@basic-pace-solo@rcs0: - shard-kbl: [FAIL][113] ([i915#2842]) -> [PASS][114] [113]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl6/igt@gem_exec_fair@basic-pace-solo@rcs0.html [114]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl7/igt@gem_exec_fair@basic-pace-solo@rcs0.html * igt@gem_exec_fair@basic-pace@vcs1: - shard-tglb: [FAIL][115] ([i915#2842]) -> [PASS][116] [115]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-tglb1/igt@gem_exec_fair@basic-pace@vcs1.html [116]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb8/igt@gem_exec_fair@basic-pace@vcs1.html * igt@gem_exec_schedule@u-fairslice@rcs0: - shard-iclb: [DMESG-WARN][117] ([i915#2803]) -> [PASS][118] [117]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-iclb1/igt@gem_exec_schedule@u-fairslice@rcs0.html [118]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb4/igt@gem_exec_schedule@u-fairslice@rcs0.html * igt@gem_exec_schedule@u-fairslice@vcs0: - shard-glk: [DMESG-WARN][119] ([i915#1610] / [i915#2803]) -> [PASS][120] [119]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk8/igt@gem_exec_schedule@u-fairslice@vcs0.html [120]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-glk2/igt@gem_exec_schedule@u-fairslice@vcs0.html * igt@gem_exec_whisper@basic-queues-forked: - shard-iclb: [INCOMPLETE][121] ([i915#1895] / [i915#2405]) -> [PASS][122] [121]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-iclb4/igt@gem_exec_whisper@basic-queues-forked.html [122]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-iclb2/igt@gem_exec_whisper@basic-queues-forked.html - shard-glk: [DMESG-WARN][123] ([i915#118] / [i915#95]) -> [PASS][124] [123]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk4/igt@gem_exec_whisper@basic-queues-forked.html [124]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-glk9/igt@gem_exec_whisper@basic-queues-forked.html * igt@gem_huc_copy@huc-copy: - shard-tglb: [SKIP][125] ([i915#2190]) -> [PASS][126] [125]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-tglb6/igt@gem_huc_copy@huc-copy.html [126]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb1/igt@gem_huc_copy@huc-copy.html * igt@gem_request_retire@retire-vma-not-inactive: - shard-tglb: [DMESG-WARN][127] -> [PASS][128] [127]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-tglb5/igt@gem_request_retire@retire-vma-not-inactive.html [128]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb2/igt@gem_request_retire@retire-vma-not-inactive.html * igt@gem_vm_create@destroy-race: - shard-tglb: [TIMEOUT][129] ([i915#2795]) -> [PASS][130] [129]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-tglb7/igt@gem_vm_create@destroy-race.html [130]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb8/igt@gem_vm_create@destroy-race.html * igt@i915_selftest@live@hangcheck: - shard-hsw: [INCOMPLETE][131] ([i915#2782]) -> [PASS][132] [131]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-hsw7/igt@i915_selftest@live@hangcheck.html [132]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-hsw6/igt@i915_selftest@live@hangcheck.html * igt@kms_async_flips@alternate-sync-async-flip: - shard-glk: [FAIL][133] ([i915#2521]) -> [PASS][134] [133]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk1/igt@kms_async_flips@alternate-sync-async-flip.html [134]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-glk8/igt@kms_async_flips@alternate-sync-async-flip.html * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a: - shard-kbl: [DMESG-WARN][135] ([i915#180]) -> [PASS][136] +1 similar issue [135]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl4/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html [136]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-kbl6/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b: - shard-apl: [DMESG-WARN][137] ([i915#180]) -> [PASS][138] [137]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-apl7/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b.html [138]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-apl1/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b.html * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-d: - shard-tglb: [INCOMPLETE][139] ([i915#1436] / [i915#1982]) -> [PASS][140] [139]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-tglb1/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-d.html [140]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/shard-tglb7/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-d.html * igt@kms_psr@psr2_primary_mmap_cpu: - shard-iclb: [SKIP][141] ([fdo#109441]) -> [PASS][142] +2 similar issues [141]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-iclb8/igt@kms_psr@psr2_primary_mmap_cpu.html [142] == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5448/index.html [-- Attachment #1.2: Type: text/html, Size: 33747 bytes --] [-- Attachment #2: Type: text/plain, Size: 154 bytes --] _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] [PATCH i-g-t] meson: Turn on -Werror 2021-01-30 6:44 [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Ashutosh Dixit ` (2 preceding siblings ...) 2021-01-30 9:19 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork @ 2021-01-30 12:11 ` Chris Wilson 2021-01-30 12:22 ` [igt-dev] ✗ Fi.CI.BUILD: failure for meson: Turn on -Werror (rev3) Patchwork ` (5 subsequent siblings) 9 siblings, 0 replies; 13+ messages in thread From: Chris Wilson @ 2021-01-30 12:11 UTC (permalink / raw) To: igt-dev From: Ashutosh Dixit <ashutosh.dixit@intel.com> We can choose which compile warnings to enable, but once they are enabled treat all warnings as error. This enforces stricter checks against compile warnings creeping in. v2: Fix redefinition warning errors from i915/gem_userptr_blits v3: Fix the even more pedantic clang compilation Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com> --- benchmarks/kms_vblank.c | 8 +++---- lib/igt_aux.c | 8 +++---- lib/igt_core.c | 7 +++--- lib/igt_core.h | 33 ++++++++++++++-------------- lib/igt_gt.c | 3 +-- lib/igt_primes.c | 10 --------- lib/igt_sysrq.h | 2 +- lib/tests/igt_abort.c | 12 +++++----- lib/tests/igt_assert.c | 2 +- lib/tests/igt_conflicting_args.c | 2 +- lib/tests/igt_describe.c | 3 ++- lib/tests/igt_dynamic_subtests.c | 14 ++++++------ lib/tests/igt_fork.c | 12 +++++----- lib/tests/igt_invalid_subtest_name.c | 4 ++-- lib/tests/igt_nesting.c | 28 +++++++++++------------ lib/tests/igt_segfault.c | 2 +- lib/tests/igt_simulation.c | 2 +- lib/tests/igt_thread.c | 14 +++++++----- meson.build | 28 ++++++++++++----------- tests/dumb_buffer.c | 2 +- tests/i915/gem_gtt_speed.c | 2 +- tests/i915/gem_madvise.c | 2 +- tests/i915/gem_userptr_blits.c | 5 ++--- tests/i915/i915_pm_rpm.c | 2 +- tests/i915/perf_pmu.c | 2 +- tests/kms_atomic.c | 10 +++++---- tests/kms_vblank.c | 18 +++++++-------- tests/prime_nv_pcopy.c | 2 +- tests/prime_nv_test.c | 2 +- tests/testdisplay.c | 2 +- tests/vc4_purgeable_bo.c | 2 +- tools/skl_ddb_allocation.c | 9 -------- 32 files changed, 121 insertions(+), 133 deletions(-) diff --git a/benchmarks/kms_vblank.c b/benchmarks/kms_vblank.c index 838afa0dd..f32cc9e32 100644 --- a/benchmarks/kms_vblank.c +++ b/benchmarks/kms_vblank.c @@ -55,7 +55,7 @@ static int crtc0_active(int fd) union drm_wait_vblank vbl; memset(&vbl, 0, sizeof(vbl)); - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; return drmIoctl(fd, DRM_IOCTL_WAIT_VBLANK, &vbl) == 0; } @@ -74,14 +74,14 @@ static void vblank_query(int fd, int busy) drmIoctl(fd, DRM_IOCTL_WAIT_VBLANK, &vbl); } - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.sequence = 0; drmIoctl(fd, DRM_IOCTL_WAIT_VBLANK, &vbl); seq = vbl.reply.sequence; clock_gettime(CLOCK_MONOTONIC, &start); do { - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.sequence = 0; drmIoctl(fd, DRM_IOCTL_WAIT_VBLANK, &vbl); count++; @@ -108,7 +108,7 @@ static void vblank_event(int fd, int busy) drmIoctl(fd, DRM_IOCTL_WAIT_VBLANK, &vbl); } - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.sequence = 0; drmIoctl(fd, DRM_IOCTL_WAIT_VBLANK, &vbl); seq = vbl.reply.sequence; diff --git a/lib/igt_aux.c b/lib/igt_aux.c index fba349334..1217f5e88 100644 --- a/lib/igt_aux.c +++ b/lib/igt_aux.c @@ -276,7 +276,7 @@ bool __igt_sigiter_continue(struct __igt_sigiter *iter, bool enable) static struct igt_helper_process signal_helper; long long int sig_stat; -static void __attribute__((noreturn)) signal_helper_process(pid_t pid) +__noreturn static void signal_helper_process(pid_t pid) { /* Interrupt the parent process at 500Hz, just to be annoying */ while (1) { @@ -391,7 +391,7 @@ void igt_resume_signal_helper(void) } static struct igt_helper_process shrink_helper; -static void __attribute__((noreturn)) shrink_helper_process(int fd, pid_t pid) +__noreturn static void shrink_helper_process(int fd, pid_t pid) { while (1) { igt_drop_caches_set(fd, DROP_SHRINK_ALL); @@ -449,7 +449,7 @@ static void show_kernel_stack(pid_t pid) } static struct igt_helper_process hang_detector; -static void __attribute__((noreturn)) +__noreturn static void hang_detector_process(int fd, pid_t pid, dev_t rdev) { struct udev_monitor *mon = @@ -496,7 +496,7 @@ hang_detector_process(int fd, pid_t pid, dev_t rdev) exit(0); } -static void sig_abort(int sig) +__noreturn static void sig_abort(int sig) { errno = 0; /* inside a signal, last errno reporting is confusing */ igt_assert(!"GPU hung"); diff --git a/lib/igt_core.c b/lib/igt_core.c index 739e3efab..f9dfaa0dd 100644 --- a/lib/igt_core.c +++ b/lib/igt_core.c @@ -1376,8 +1376,7 @@ bool __igt_enter_dynamic_container(void) return true; } -static void exit_subtest(const char *) __attribute__((noreturn)); -static void exit_subtest(const char *result) +__noreturn static void exit_subtest(const char *result) { struct timespec now; const char *subtest_text = in_dynamic_subtest ? "Dynamic subtest" : "Subtest"; @@ -1610,7 +1609,7 @@ void igt_fail(int exitcode) * Since out test runner (piglit) does support fatal test exit codes, we * implement the default behaviour by waiting endlessly. */ -void __attribute__((noreturn)) igt_fatal_error(void) +void igt_fatal_error(void) { if (igt_check_boolean_env_var("IGT_REBOOT_ON_FATAL_ERROR", false)) { igt_warn("FATAL ERROR - REBOOTING\n"); @@ -2825,7 +2824,7 @@ out: } static const char *timeout_op; -static void __attribute__((noreturn)) igt_alarm_handler(int signal) +__noreturn static void igt_alarm_handler(int signal) { if (timeout_op) igt_info("Timed out: %s\n", timeout_op); diff --git a/lib/igt_core.h b/lib/igt_core.h index 206673875..6075d1539 100644 --- a/lib/igt_core.h +++ b/lib/igt_core.h @@ -79,6 +79,9 @@ #define igt_assume(e) BUILD_BUG_ON_INVALID(e) #endif +#define __noreturn __attribute__((noreturn)) +#define __maybe_unused __attribute__((unused)) + extern const char* __igt_test_description __attribute__((weak)); extern bool __igt_plain_output; extern char *igt_frame_dump_path; @@ -133,7 +136,7 @@ struct _GKeyFile *igt_load_igtrc(void); bool __igt_fixture(void); void __igt_fixture_complete(void); -void __igt_fixture_end(void) __attribute__((noreturn)); +__noreturn void __igt_fixture_end(void); /** * igt_fixture: * @@ -489,31 +492,29 @@ void igt_simple_init_parse_opts(int *argc, char **argv, __attribute__((constructor)) \ static void igt_tokencat(__igt_constructor_l, __LINE__)(void) -__attribute__((format(printf, 1, 2))) -void igt_skip(const char *f, ...) __attribute__((noreturn)); -__attribute__((format(printf, 5, 6))) +__noreturn __attribute__((format(printf, 1, 2))) +void igt_skip(const char *f, ...); +__noreturn __attribute__((format(printf, 5, 6))) void __igt_skip_check(const char *file, const int line, const char *func, const char *check, - const char *format, ...) __attribute__((noreturn)); + const char *format, ...); #define igt_skip_check(E, F...) \ __igt_skip_check(__FILE__, __LINE__, __func__, E, F) void igt_success(void); bool igt_can_fail(void); -void igt_fail(int exitcode) __attribute__((noreturn)); -__attribute__((format(printf, 6, 7))) +__noreturn void igt_fail(int exitcode); +__noreturn __attribute__((format(printf, 6, 7))) void __igt_fail_assert(const char *domain, const char *file, const int line, const char *func, const char *assertion, - const char *format, ...) - __attribute__((noreturn)); -__attribute__((format(printf, 6, 7))) -void __igt_abort(const char *domain, const char *file, const int line, - const char *func, const char *expression, - const char *f, ...) - __attribute__((noreturn)); -void igt_exit(void) __attribute__((noreturn)); -void igt_fatal_error(void) __attribute__((noreturn)); + const char *format, ...); +__noreturn __attribute__((format(printf, 6, 7))) +void __igt_abort(const char *domain, const char *file, + const int line, const char *func, const char *expression, + const char *f, ...); +__noreturn void igt_exit(void); +__noreturn void igt_fatal_error(void); /** * igt_ignore_warn: diff --git a/lib/igt_gt.c b/lib/igt_gt.c index 7dfcac038..f601d7260 100644 --- a/lib/igt_gt.c +++ b/lib/igt_gt.c @@ -380,8 +380,7 @@ void igt_force_gpu_reset(int drm_fd) /* GPU abusers */ static struct igt_helper_process hang_helper; -static void __attribute__((noreturn)) -hang_helper_process(pid_t pid, int fd) +__noreturn static void hang_helper_process(pid_t pid, int fd) { while (1) { if (kill(pid, 0)) /* Parent has died, so must we. */ diff --git a/lib/igt_primes.c b/lib/igt_primes.c index 3578a3c69..3941bb21a 100644 --- a/lib/igt_primes.c +++ b/lib/igt_primes.c @@ -64,21 +64,11 @@ static inline unsigned long __bit__(unsigned long nr) return 1UL << (nr % BITS_PER_LONG); } -static inline void set_bit(unsigned long nr, unsigned long *addr) -{ - addr[nr / BITS_PER_LONG] |= __bit__(nr); -} - static inline void clear_bit(unsigned long nr, unsigned long *addr) { addr[nr / BITS_PER_LONG] &= ~__bit__(nr); } -static inline bool test_bit(unsigned long nr, const unsigned long *addr) -{ - return addr[nr / BITS_PER_LONG] & __bit__(nr); -} - static unsigned long __find_next_bit(const unsigned long *addr, unsigned long nbits, unsigned long start, diff --git a/lib/igt_sysrq.h b/lib/igt_sysrq.h index 422473d2a..c941029d5 100644 --- a/lib/igt_sysrq.h +++ b/lib/igt_sysrq.h @@ -25,6 +25,6 @@ #ifndef __IGT_SYSRQ_H__ #define __IGT_SYSRQ_H__ -void igt_sysrq_reboot(void) __attribute__((noreturn)); +__noreturn void igt_sysrq_reboot(void); #endif /* __IGT_SYSRQ_H__ */ diff --git a/lib/tests/igt_abort.c b/lib/tests/igt_abort.c index 53b7d4fd5..356e41679 100644 --- a/lib/tests/igt_abort.c +++ b/lib/tests/igt_abort.c @@ -31,7 +31,7 @@ char test[] = "test"; char *fake_argv[] = { test }; int fake_argc = ARRAY_SIZE(fake_argv); -static void fake_simple_test(void) +__noreturn static void fake_simple_test(void) { igt_simple_init(fake_argc, fake_argv); @@ -40,7 +40,7 @@ static void fake_simple_test(void) exit(0); /* unreachable */ } -static void fake_fixture_test(void) +__noreturn static void fake_fixture_test(void) { igt_subtest_init(fake_argc, fake_argv); @@ -51,7 +51,7 @@ static void fake_fixture_test(void) exit(0); /* unreachable */ } -static void fake_outside_fixture_test(void) +__noreturn static void fake_outside_fixture_test(void) { igt_subtest_init(fake_argc, fake_argv); @@ -60,7 +60,7 @@ static void fake_outside_fixture_test(void) exit(0); /* unreachable */ } -static void fake_subtest_test(void) +__noreturn static void fake_subtest_test(void) { igt_subtest_init(fake_argc, fake_argv); @@ -76,7 +76,7 @@ static void fake_subtest_test(void) exit(0); /* unreachable */ } -static void fake_dynamic_test(void) +__noreturn static void fake_dynamic_test(void) { igt_subtest_init(fake_argc, fake_argv); @@ -97,7 +97,7 @@ static void fake_dynamic_test(void) exit(0); /* unreachable */ } -static void fake_outside_dynamic_test(void) +__noreturn static void fake_outside_dynamic_test(void) { igt_subtest_init(fake_argc, fake_argv); diff --git a/lib/tests/igt_assert.c b/lib/tests/igt_assert.c index c94ac6989..3523a4649 100644 --- a/lib/tests/igt_assert.c +++ b/lib/tests/igt_assert.c @@ -53,7 +53,7 @@ void (*test_to_run)(void) = NULL; exec_total++; \ } -static void fake_test(void) +__noreturn static void fake_test(void) { char test[] = "test"; char *argv_run[] = { test }; diff --git a/lib/tests/igt_conflicting_args.c b/lib/tests/igt_conflicting_args.c index b644fd4bc..b5fe2d781 100644 --- a/lib/tests/igt_conflicting_args.c +++ b/lib/tests/igt_conflicting_args.c @@ -43,7 +43,7 @@ static int opt_handler(int option, int option_index, void *input) return 0; } -static void fake_test(void) +__noreturn static void fake_test(void) { char test_name[] = "test"; char *argv[] = { test_name }; diff --git a/lib/tests/igt_describe.c b/lib/tests/igt_describe.c index ff45a6c4b..0cdb94af2 100644 --- a/lib/tests/igt_describe.c +++ b/lib/tests/igt_describe.c @@ -34,7 +34,8 @@ char *fake_argv[] = {prog, fake_arg}; int fake_argc = ARRAY_SIZE(fake_argv); IGT_TEST_DESCRIPTION("the top level description"); -static void fake_main(void) + +__noreturn static void fake_main(void) { igt_subtest_init(fake_argc, fake_argv); diff --git a/lib/tests/igt_dynamic_subtests.c b/lib/tests/igt_dynamic_subtests.c index bd1e16758..b18cfae6c 100644 --- a/lib/tests/igt_dynamic_subtests.c +++ b/lib/tests/igt_dynamic_subtests.c @@ -29,7 +29,7 @@ #include "igt_tests_common.h" -static void dynamic_subtest_in_normal_subtest(void) +__noreturn static void dynamic_subtest_in_normal_subtest(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -46,7 +46,7 @@ static void dynamic_subtest_in_normal_subtest(void) igt_exit(); } -static void invalid_dynamic_subtest_name(void) +__noreturn static void invalid_dynamic_subtest_name(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -63,7 +63,7 @@ static void invalid_dynamic_subtest_name(void) igt_exit(); } -static void dynamic_subtest_in_toplevel(void) +__noreturn static void dynamic_subtest_in_toplevel(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -78,7 +78,7 @@ static void dynamic_subtest_in_toplevel(void) igt_exit(); } -static void subtest_itself_failing(void) +__noreturn static void subtest_itself_failing(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -93,7 +93,7 @@ static void subtest_itself_failing(void) igt_exit(); } -static void subtest_itself_skipping(void) +__noreturn static void subtest_itself_skipping(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -108,7 +108,7 @@ static void subtest_itself_skipping(void) igt_exit(); } -static void dynamic_subtest_failure_leads_to_fail(void) +__noreturn static void dynamic_subtest_failure_leads_to_fail(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -125,7 +125,7 @@ static void dynamic_subtest_failure_leads_to_fail(void) igt_exit(); } -static void no_dynamic_subtests_entered_leads_to_skip(void) +__noreturn static void no_dynamic_subtests_entered_leads_to_skip(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; diff --git a/lib/tests/igt_fork.c b/lib/tests/igt_fork.c index 84492b2d1..d19d09452 100644 --- a/lib/tests/igt_fork.c +++ b/lib/tests/igt_fork.c @@ -38,7 +38,7 @@ char test[] = "test"; char *fake_argv[] = { test }; int fake_argc = ARRAY_SIZE(fake_argv); -static void igt_fork_vs_skip(void) +__noreturn static void igt_fork_vs_skip(void) { igt_simple_init(fake_argc, fake_argv); @@ -51,7 +51,7 @@ static void igt_fork_vs_skip(void) igt_exit(); } -static void igt_fork_vs_assert(void) +__noreturn static void igt_fork_vs_assert(void) { igt_simple_init(fake_argc, fake_argv); @@ -64,7 +64,7 @@ static void igt_fork_vs_assert(void) igt_exit(); } -static void igt_fork_leak(void) +__noreturn static void igt_fork_leak(void) { igt_simple_init(fake_argc, fake_argv); @@ -75,7 +75,7 @@ static void igt_fork_leak(void) igt_exit(); } -static void plain_fork_leak(void) +__noreturn static void plain_fork_leak(void) { int pid; @@ -93,7 +93,7 @@ static void plain_fork_leak(void) igt_exit(); } -static void igt_fork_timeout_leak(void) +__noreturn static void igt_fork_timeout_leak(void) { igt_simple_init(fake_argc, fake_argv); @@ -106,7 +106,7 @@ static void igt_fork_timeout_leak(void) igt_exit(); } -static void subtest_leak(void) +__noreturn static void subtest_leak(void) { pid_t *children = mmap(0, 4096, PROT_WRITE, MAP_SHARED | MAP_ANON, -1, 0); diff --git a/lib/tests/igt_invalid_subtest_name.c b/lib/tests/igt_invalid_subtest_name.c index 32c4bcbc9..b790482c0 100644 --- a/lib/tests/igt_invalid_subtest_name.c +++ b/lib/tests/igt_invalid_subtest_name.c @@ -29,7 +29,7 @@ #include "igt_tests_common.h" -static void invalid_subtest_name(void) +__noreturn static void invalid_subtest_name(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -44,7 +44,7 @@ static void invalid_subtest_name(void) igt_exit(); } -static void nonexisting_subtest(void) +__noreturn static void nonexisting_subtest(void) { char prog[] = "igt_no_exit"; char arg1[] = "--run-subtest"; diff --git a/lib/tests/igt_nesting.c b/lib/tests/igt_nesting.c index de674b6b8..6b209af82 100644 --- a/lib/tests/igt_nesting.c +++ b/lib/tests/igt_nesting.c @@ -30,7 +30,7 @@ char test[] = "test"; char *fake_argv[] = { test }; int fake_argc = ARRAY_SIZE(fake_argv); -static void all_valid_simple_test(void) +__noreturn static void all_valid_simple_test(void) { igt_simple_init(fake_argc, fake_argv); @@ -40,7 +40,7 @@ static void all_valid_simple_test(void) igt_exit(); } -static void all_valid(void) +__noreturn static void all_valid(void) { igt_subtest_init(fake_argc, fake_argv); @@ -87,7 +87,7 @@ static void all_valid(void) igt_exit(); } -static void invalid_subtest_in_simple_test(void) +__noreturn static void invalid_subtest_in_simple_test(void) { igt_simple_init(fake_argc, fake_argv); @@ -97,7 +97,7 @@ static void invalid_subtest_in_simple_test(void) igt_exit(); } -static void invalid_subtest_group_in_simple_test(void) +__noreturn static void invalid_subtest_group_in_simple_test(void) { igt_simple_init(fake_argc, fake_argv); @@ -107,7 +107,7 @@ static void invalid_subtest_group_in_simple_test(void) igt_exit(); } -static void invalid_subtest_with_dynamic_in_simple_test(void) +__noreturn static void invalid_subtest_with_dynamic_in_simple_test(void) { igt_simple_init(fake_argc, fake_argv); @@ -117,7 +117,7 @@ static void invalid_subtest_with_dynamic_in_simple_test(void) igt_exit(); } -static void invalid_dynamic_in_simple_test(void) +__noreturn static void invalid_dynamic_in_simple_test(void) { igt_simple_init(fake_argc, fake_argv); @@ -127,7 +127,7 @@ static void invalid_dynamic_in_simple_test(void) igt_exit(); } -static void invalid_fixture_in_fixture(void) +__noreturn static void invalid_fixture_in_fixture(void) { igt_subtest_init(fake_argc, fake_argv); @@ -139,7 +139,7 @@ static void invalid_fixture_in_fixture(void) igt_exit(); } -static void invalid_subtest_in_subtest(void) +__noreturn static void invalid_subtest_in_subtest(void) { igt_subtest_init(fake_argc, fake_argv); @@ -151,7 +151,7 @@ static void invalid_subtest_in_subtest(void) igt_exit(); } -static void invalid_top_level_dynamic(void) +__noreturn static void invalid_top_level_dynamic(void) { igt_subtest_init(fake_argc, fake_argv); @@ -161,7 +161,7 @@ static void invalid_top_level_dynamic(void) igt_exit(); } -static void invalid_dynamic_in_regular_subtest(void) +__noreturn static void invalid_dynamic_in_regular_subtest(void) { igt_subtest_init(fake_argc, fake_argv); @@ -173,7 +173,7 @@ static void invalid_dynamic_in_regular_subtest(void) igt_exit(); } -static void invalid_fixture_in_subtest(void) +__noreturn static void invalid_fixture_in_subtest(void) { igt_subtest_init(fake_argc, fake_argv); @@ -185,7 +185,7 @@ static void invalid_fixture_in_subtest(void) igt_exit(); } -static void invalid_top_level_skip(void) +__noreturn static void invalid_top_level_skip(void) { igt_subtest_init(fake_argc, fake_argv); @@ -194,7 +194,7 @@ static void invalid_top_level_skip(void) igt_exit(); } -static void invalid_top_level_assert(void) +__noreturn static void invalid_top_level_assert(void) { igt_subtest_init(fake_argc, fake_argv); @@ -203,7 +203,7 @@ static void invalid_top_level_assert(void) igt_exit(); } -static void invalid_dynamic_in_dynamic(void) +__noreturn static void invalid_dynamic_in_dynamic(void) { igt_subtest_init(fake_argc, fake_argv); diff --git a/lib/tests/igt_segfault.c b/lib/tests/igt_segfault.c index 38c040a6f..56efa9125 100644 --- a/lib/tests/igt_segfault.c +++ b/lib/tests/igt_segfault.c @@ -54,7 +54,7 @@ static void crashme(void) raise(SIGSEGV); } -static void fake_test(void) +__noreturn static void fake_test(void) { char prog[] = "test"; char *fake_argv[] = { prog }; diff --git a/lib/tests/igt_simulation.c b/lib/tests/igt_simulation.c index a0ea70008..9a2cf2aae 100644 --- a/lib/tests/igt_simulation.c +++ b/lib/tests/igt_simulation.c @@ -40,7 +40,7 @@ bool list_subtests; bool in_fixture; bool in_subtest; -static void fake_test(void) +__noreturn static void fake_test(void) { char test[] = "test"; char list[] = "--list-subtests"; diff --git a/lib/tests/igt_thread.c b/lib/tests/igt_thread.c index e9d37584c..eb92326dc 100644 --- a/lib/tests/igt_thread.c +++ b/lib/tests/igt_thread.c @@ -49,7 +49,8 @@ static void *require_thread(void *data) return NULL; } -static void one_subtest_fail(void) { +__noreturn static void one_subtest_fail(void) +{ igt_subtest_init(fake_argc, fake_argv); igt_subtest("subtest-a") { @@ -67,7 +68,8 @@ static void one_subtest_fail(void) { igt_exit(); } -static void one_dynamic_fail(void) { +__noreturn static void one_dynamic_fail(void) +{ igt_subtest_init(fake_argc, fake_argv); igt_subtest_with_dynamic("dynamic-container") { @@ -87,7 +89,7 @@ static void one_dynamic_fail(void) { igt_exit(); } -static void simple_success(void) { +__noreturn static void simple_success(void) { pthread_t thread; igt_simple_init(fake_argc, fake_argv); @@ -98,7 +100,8 @@ static void simple_success(void) { igt_exit(); } -static void simple_failure(void) { +__noreturn static void simple_failure(void) +{ pthread_t thread; igt_simple_init(fake_argc, fake_argv); @@ -109,7 +112,8 @@ static void simple_failure(void) { igt_exit(); } -static void require_non_main_thread(void) { +__noreturn static void require_non_main_thread(void) +{ pthread_t thread; igt_simple_init(fake_argc, fake_argv); diff --git a/meson.build b/meson.build index f4ee14f14..3f3693f99 100644 --- a/meson.build +++ b/meson.build @@ -51,19 +51,21 @@ cc_args = [ '-Wno-unused-parameter', '-Wno-unused-result', - '-Werror=address', - '-Werror=array-bounds', - '-Werror=implicit', - '-Werror=init-self', - '-Werror=int-to-pointer-cast', - '-Werror=main', - '-Werror=missing-braces', - '-Werror=nonnull', - '-Werror=pointer-to-int-cast', - '-Werror=return-type', - '-Werror=sequence-point', - '-Werror=trigraphs', - '-Werror=write-strings', + '-Waddress', + '-Warray-bounds', + '-Wimplicit', + '-Winit-self', + '-Wint-to-pointer-cast', + '-Wmain', + '-Wmissing-braces', + '-Wnonnull', + '-Wpointer-to-int-cast', + '-Wreturn-type', + '-Wsequence-point', + '-Wtrigraphs', + '-Wwrite-strings', + + '-Werror', # Disable the memory allocating builtins as they may cause unexpected behavior # with our framework. They *may* get optimized out in favor of a register or # stack variable, making them effectively local. Local variables do not play diff --git a/tests/dumb_buffer.c b/tests/dumb_buffer.c index 44f60b6f1..cb8d2f518 100644 --- a/tests/dumb_buffer.c +++ b/tests/dumb_buffer.c @@ -278,7 +278,7 @@ static void *thread_clear(void *data) static jmp_buf sigjmp; -static void sigprobe(int sig) +__noreturn static void sigprobe(int sig) { longjmp(sigjmp, sig); } diff --git a/tests/i915/gem_gtt_speed.c b/tests/i915/gem_gtt_speed.c index 06fe70f84..644af3d38 100644 --- a/tests/i915/gem_gtt_speed.c +++ b/tests/i915/gem_gtt_speed.c @@ -82,7 +82,7 @@ static inline unsigned x86_64_features(void) { return 0; } -static void streaming_load(void *src, int len) +__noreturn static void streaming_load(void *src, int len) { igt_assert(!"reached"); } diff --git a/tests/i915/gem_madvise.c b/tests/i915/gem_madvise.c index 623c8b091..2cd0b5d7c 100644 --- a/tests/i915/gem_madvise.c +++ b/tests/i915/gem_madvise.c @@ -51,7 +51,7 @@ IGT_TEST_DESCRIPTION("Checks that the kernel reports EFAULT when trying to use" static jmp_buf jmp; -static void __attribute__((noreturn)) sigtrap(int sig) +__noreturn static void sigtrap(int sig) { siglongjmp(jmp, sig); } diff --git a/tests/i915/gem_userptr_blits.c b/tests/i915/gem_userptr_blits.c index 2f3954c5d..1bc2d3600 100644 --- a/tests/i915/gem_userptr_blits.c +++ b/tests/i915/gem_userptr_blits.c @@ -55,8 +55,6 @@ #include <pthread.h> #include <time.h> -#include <linux/memfd.h> - #include "drm.h" #include "i915_drm.h" @@ -1550,7 +1548,8 @@ static void test_readonly(int i915) } static jmp_buf sigjmp; -static void sigjmp_handler(int sig) + +__noreturn static void sigjmp_handler(int sig) { siglongjmp(sigjmp, sig); } diff --git a/tests/i915/i915_pm_rpm.c b/tests/i915/i915_pm_rpm.c index 6321dd403..d9ec0388c 100644 --- a/tests/i915/i915_pm_rpm.c +++ b/tests/i915/i915_pm_rpm.c @@ -1483,7 +1483,7 @@ static void pci_d3_state_subtest(void) } } -static void __attribute__((noreturn)) stay_subtest(void) +__noreturn static void stay_subtest(void) { disable_all_screens_and_wait(&ms_data); diff --git a/tests/i915/perf_pmu.c b/tests/i915/perf_pmu.c index 7b175ea8d..50b5c82bc 100644 --- a/tests/i915/perf_pmu.c +++ b/tests/i915/perf_pmu.c @@ -1092,7 +1092,7 @@ event_wait(int gem_fd, const struct intel_execution_engine2 *e) for (;;) { union drm_wait_vblank vbl = { }; - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 1; igt_assert_eq(wait_vblank(gem_fd, &vbl), 0); diff --git a/tests/kms_atomic.c b/tests/kms_atomic.c index fa6190c9b..2965b350b 100644 --- a/tests/kms_atomic.c +++ b/tests/kms_atomic.c @@ -448,11 +448,11 @@ plane_immutable_zpos(igt_display_t *display, igt_pipe_t *pipe, for (int i = 0; i < n_planes - 1; i++) { igt_plane_t *plane_lower, *plane_upper; - if (plane_ptr[i] != NULL) - plane_lower = plane_ptr[i]; - else + if (plane_ptr[i] == NULL) continue; + plane_lower = plane_ptr[i]; + plane_upper = NULL; while (i < (n_planes - 1)) { if (plane_ptr[i + 1] != NULL) { plane_upper = plane_ptr[i + 1]; @@ -462,9 +462,11 @@ plane_immutable_zpos(igt_display_t *display, igt_pipe_t *pipe, continue; } } + if (!plane_upper) + continue; if ((plane_upper->type == DRM_PLANE_TYPE_CURSOR) || - (plane_lower->type == DRM_PLANE_TYPE_CURSOR)) + (plane_lower->type == DRM_PLANE_TYPE_CURSOR)) continue; igt_plane_set_position(plane_lower, 0, 0); diff --git a/tests/kms_vblank.c b/tests/kms_vblank.c index b00ebec53..31b3f0a69 100644 --- a/tests/kms_vblank.c +++ b/tests/kms_vblank.c @@ -230,14 +230,14 @@ static void accuracy(data_t *data, int fd, int nchildren) int n; memset(&vbl, 0, sizeof(vbl)); - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 1; igt_assert_eq(wait_vblank(fd, &vbl), 0); target = vbl.reply.sequence + total; for (n = 0; n < total; n++) { - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 1; igt_assert_eq(wait_vblank(fd, &vbl), 0); @@ -247,7 +247,7 @@ static void accuracy(data_t *data, int fd, int nchildren) vbl.request.sequence = target; igt_assert_eq(wait_vblank(fd, &vbl), 0); } - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 0; igt_assert_eq(wait_vblank(fd, &vbl), 0); @@ -268,7 +268,7 @@ static void vblank_query(data_t *data, int fd, int nchildren) unsigned long sq, count = 0; memset(&vbl, 0, sizeof(vbl)); - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 0; igt_assert_eq(wait_vblank(fd, &vbl), 0); @@ -277,7 +277,7 @@ static void vblank_query(data_t *data, int fd, int nchildren) clock_gettime(CLOCK_MONOTONIC, &start); do { - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 0; igt_assert_eq(wait_vblank(fd, &vbl), 0); @@ -297,7 +297,7 @@ static void vblank_wait(data_t *data, int fd, int nchildren) unsigned long sq, count = 0; memset(&vbl, 0, sizeof(vbl)); - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 0; igt_assert_eq(wait_vblank(fd, &vbl), 0); @@ -306,7 +306,7 @@ static void vblank_wait(data_t *data, int fd, int nchildren) clock_gettime(CLOCK_MONOTONIC, &start); do { - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 1; igt_assert_eq(wait_vblank(fd, &vbl), 0); @@ -367,7 +367,7 @@ static void vblank_ts_cont(data_t *data, int fd, int nchildren) if (data->flags & (MODESET | DPMS)) { /* Attempting to do a vblank while disabled should return -EINVAL */ memset(&vbl, 0, sizeof(vbl)); - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= kmstest_get_vbl_flag(data->pipe); igt_assert_eq(wait_vblank(fd, &vbl), -EINVAL); } @@ -476,7 +476,7 @@ static void invalid_subtest(data_t *data, int fd) /* First check all is well with a simple query */ memset(&vbl, 0, sizeof(vbl)); - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; igt_assert_eq(wait_vblank(fd, &vbl), 0); valid_flags = (_DRM_VBLANK_TYPES_MASK | diff --git a/tests/prime_nv_pcopy.c b/tests/prime_nv_pcopy.c index bbcf76807..e465e5fc0 100644 --- a/tests/prime_nv_pcopy.c +++ b/tests/prime_nv_pcopy.c @@ -139,7 +139,7 @@ BEGIN_NV04(struct nouveau_pushbuf *push, int subc, int mthd, int size) PUSH_DATA (push, 0x00000000 | (size << 18) | (subc << 13) | mthd); } -static inline void +__maybe_unused static inline void BEGIN_NI04(struct nouveau_pushbuf *push, int subc, int mthd, int size) { PUSH_DATA (push, 0x40000000 | (size << 18) | (subc << 13) | mthd); diff --git a/tests/prime_nv_test.c b/tests/prime_nv_test.c index b4714dcb5..81d142f42 100644 --- a/tests/prime_nv_test.c +++ b/tests/prime_nv_test.c @@ -190,7 +190,7 @@ static void test_nv_write_i915_gtt_mmap_read(void) /* test drm_intel_bo_map doesn't work properly, this tries to map the backing shmem fd, which doesn't exist for these objects */ -static void test_i915_import_cpu_mmap(void) +__noreturn static void test_i915_import_cpu_mmap(void) { drm_intel_bo *test_intel_bo; int prime_fd; diff --git a/tests/testdisplay.c b/tests/testdisplay.c index ef033dc60..e2560eaf5 100644 --- a/tests/testdisplay.c +++ b/tests/testdisplay.c @@ -542,7 +542,7 @@ int update_display(bool probe) #define dump_resource(res) if (res) dump_##res() -static void __attribute__((noreturn)) cleanup_and_exit(int ret) +__noreturn static void cleanup_and_exit(int ret) { close(drm_fd); exit(ret); diff --git a/tests/vc4_purgeable_bo.c b/tests/vc4_purgeable_bo.c index ee89e153f..d9f7547f5 100644 --- a/tests/vc4_purgeable_bo.c +++ b/tests/vc4_purgeable_bo.c @@ -44,7 +44,7 @@ struct igt_vc4_bo { static jmp_buf jmp; -static void __attribute__((noreturn)) sigtrap(int sig) +__noreturn static void sigtrap(int sig) { longjmp(jmp, sig); } diff --git a/tools/skl_ddb_allocation.c b/tools/skl_ddb_allocation.c index c7bfb2791..ff4c0aaff 100644 --- a/tools/skl_ddb_allocation.c +++ b/tools/skl_ddb_allocation.c @@ -140,15 +140,6 @@ static inline uint16_t skl_ddb_entry_size(const struct skl_ddb_entry *entry) return entry->end - entry->start; } -static inline bool skl_ddb_entry_equal(const struct skl_ddb_entry *e1, - const struct skl_ddb_entry *e2) -{ - if (e1->start == e2->start && e1->end == e2->end) - return true; - - return false; -} - struct skl_ddb_allocation { struct skl_ddb_entry plane[I915_MAX_PIPES][I915_MAX_PLANES]; struct skl_ddb_entry cursor[I915_MAX_PIPES]; -- 2.30.0 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [igt-dev] ✗ Fi.CI.BUILD: failure for meson: Turn on -Werror (rev3) 2021-01-30 6:44 [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Ashutosh Dixit ` (3 preceding siblings ...) 2021-01-30 12:11 ` [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Chris Wilson @ 2021-01-30 12:22 ` Patchwork 2021-01-30 12:34 ` [igt-dev] ✗ GitLab.Pipeline: warning " Patchwork ` (4 subsequent siblings) 9 siblings, 0 replies; 13+ messages in thread From: Patchwork @ 2021-01-30 12:22 UTC (permalink / raw) To: Chris Wilson; +Cc: igt-dev == Series Details == Series: meson: Turn on -Werror (rev3) URL : https://patchwork.freedesktop.org/series/86465/ State : failure == Summary == IGT patchset build failed on latest successful build fdc23507d022b68443121ec2c1a951af27c87240 i915/sysfs_clients: Check that client ids are cyclic 273/301 testcase check: gem_eio OK 0.13 s 274/301 testcase check: gem_exec_balancer OK 0.13 s 275/301 testcase check: gem_mmap_offset OK 0.15 s 276/301 testcase check: i915_pm_rc6_residency OK 0.15 s 277/301 testcase check: perf_pmu OK 0.14 s 278/301 testcase check: perf OK 0.12 s 279/301 testcase check: testdisplay OK 0.13 s 280/301 testcase check: kms_color OK 0.13 s 281/301 testcase check: kms_color_chamelium OK 0.14 s 282/301 testcase check: sw_sync OK 0.13 s 283/301 testcase check: amdgpu/amd_abm OK 0.13 s 284/301 testcase check: amdgpu/amd_basic OK 0.13 s 285/301 testcase check: amdgpu/amd_bypass OK 0.10 s 286/301 testcase check: amdgpu/amd_color OK 0.15 s 287/301 testcase check: amdgpu/amd_cs_nop OK 0.11 s 288/301 testcase check: amdgpu/amd_prime OK 0.09 s 289/301 runner OK 5.34 s 290/301 runner_json OK 0.10 s 291/301 assembler: test/mov OK 0.08 s 292/301 assembler: test/frc OK 0.04 s 293/301 assembler: test/regtype OK 0.08 s 294/301 assembler: test/rndd OK 0.08 s 295/301 assembler: test/rndu OK 0.01 s 296/301 assembler: test/rnde OK 0.04 s 297/301 assembler: test/rnde-intsrc OK 0.02 s 298/301 assembler: test/rndz OK 0.05 s 299/301 assembler: test/lzd OK 0.04 s 300/301 assembler: test/not OK 0.04 s 301/301 assembler: test/immediate OK 0.05 s OK: 300 FAIL: 1 SKIP: 0 TIMEOUT: 0 The output from the failed tests: 6/301 lib: igt_describe FAIL 0.31 s (killed by signal 6 SIGABRT) --- command --- /home/cidrm/igt-gpu-tools/build/lib/tests/igt_describe --- stderr --- igt_describe: ../lib/tests/igt_describe.c:181: main: Assertion `0 == strcmp(DESCRIBE_ALL_OUTPUT, out)' failed. ------- Full log written to /home/cidrm/igt-gpu-tools/build/meson-logs/testlog.txt FAILED: meson-test /usr/bin/python3 -u /usr/bin/meson test --no-rebuild --print-errorlogs ninja: build stopped: subcommand failed. _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] ✗ GitLab.Pipeline: warning for meson: Turn on -Werror (rev3) 2021-01-30 6:44 [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Ashutosh Dixit ` (4 preceding siblings ...) 2021-01-30 12:22 ` [igt-dev] ✗ Fi.CI.BUILD: failure for meson: Turn on -Werror (rev3) Patchwork @ 2021-01-30 12:34 ` Patchwork 2021-01-30 12:46 ` =?unknown-8bit?b?4pyX?= " Chris Wilson 2021-01-30 12:42 ` [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Chris Wilson ` (3 subsequent siblings) 9 siblings, 1 reply; 13+ messages in thread From: Patchwork @ 2021-01-30 12:34 UTC (permalink / raw) To: Chris Wilson; +Cc: igt-dev == Series Details == Series: meson: Turn on -Werror (rev3) URL : https://patchwork.freedesktop.org/series/86465/ State : warning == Summary == Did not get list of undocumented tests for this run, something is wrong! Other than that, pipeline status: FAILED. see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/263967 for the overview. build:tests-fedora-clang has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/6996076): ../assembler/gen4asm.h:63:21: note: passing argument to parameter 'msg' here void yyerror (char *msg); ^ assembler/1ca89e8@@intel-gen4asm@exe/gram.c:5776:16: error: passing 'const char [17]' to parameter of type 'char *' discards qualifiers [-Werror,-Wincompatible-pointer-types-discards-qualifiers] yyerror (YY_("memory exhausted")); ^~~~~~~~~~~~~~~~~~ assembler/1ca89e8@@intel-gen4asm@exe/gram.c:914:22: note: expanded from macro 'YY_' # define YY_(Msgid) Msgid ^~~~~ ../assembler/gen4asm.h:63:21: note: passing argument to parameter 'msg' here void yyerror (char *msg); ^ 3 errors generated. ninja: build stopped: subcommand failed. section_end:1612009430:step_script section_start:1612009430:cleanup_file_variables Cleaning up file based variables section_end:1612009432:cleanup_file_variables ERROR: Job failed: exit code 1 == Logs == For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/263967 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: =?unknown-8bit?b?4pyX?= GitLab.Pipeline: warning for meson: Turn on -Werror (rev3) 2021-01-30 12:34 ` [igt-dev] ✗ GitLab.Pipeline: warning " Patchwork @ 2021-01-30 12:46 ` Chris Wilson 0 siblings, 0 replies; 13+ messages in thread From: Chris Wilson @ 2021-01-30 12:46 UTC (permalink / raw) To: Patchwork, igt-dev Quoting Patchwork (2021-01-30 12:34:58) > == Series Details == > > Series: meson: Turn on -Werror (rev3) > URL : https://patchwork.freedesktop.org/series/86465/ > State : warning > > == Summary == > > Did not get list of undocumented tests for this run, something is wrong! > > Other than that, pipeline status: FAILED. > > see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/263967 for the overview. > > build:tests-fedora-clang has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/6996076): > ../assembler/gen4asm.h:63:21: note: passing argument to parameter 'msg' here > void yyerror (char *msg); > ^ > assembler/1ca89e8@@intel-gen4asm@exe/gram.c:5776:16: error: passing 'const char [17]' to parameter of type 'char *' discards qualifiers [-Werror,-Wincompatible-pointer-types-discards-qualifiers] > yyerror (YY_("memory exhausted")); > ^~~~~~~~~~~~~~~~~~ > assembler/1ca89e8@@intel-gen4asm@exe/gram.c:914:22: note: expanded from macro 'YY_' > # define YY_(Msgid) Msgid > ^~~~~ > ../assembler/gen4asm.h:63:21: note: passing argument to parameter 'msg' here > void yyerror (char *msg); > ^ > 3 errors generated. > ninja: build stopped: subcommand failed. > section_end:1612009430:step_script > section_start:1612009430:cleanup_file_variables > Cleaning up file based variables > section_end:1612009432:cleanup_file_variables > ERROR: Job failed: exit code 1 Yeah, I just removed the assembler. I'm not fighting to make autogenerated code clean. -Chris _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] [PATCH i-g-t] meson: Turn on -Werror 2021-01-30 6:44 [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Ashutosh Dixit ` (5 preceding siblings ...) 2021-01-30 12:34 ` [igt-dev] ✗ GitLab.Pipeline: warning " Patchwork @ 2021-01-30 12:42 ` Chris Wilson 2021-01-30 13:05 ` [igt-dev] ✗ GitLab.Pipeline: warning for meson: Turn on -Werror (rev4) Patchwork ` (2 subsequent siblings) 9 siblings, 0 replies; 13+ messages in thread From: Chris Wilson @ 2021-01-30 12:42 UTC (permalink / raw) To: igt-dev From: Ashutosh Dixit <ashutosh.dixit@intel.com> We can choose which compile warnings to enable, but once they are enabled treat all warnings as error. This enforces stricter checks against compile warnings creeping in. v2: Fix redefinition warning errors from i915/gem_userptr_blits v3: Fix the even more pedantic clang compilation v4: Do not alter whitespace in lib/tests/igt_describe! Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com> --- benchmarks/kms_vblank.c | 8 +++---- lib/igt_aux.c | 8 +++---- lib/igt_core.c | 7 +++--- lib/igt_core.h | 33 ++++++++++++++-------------- lib/igt_gt.c | 3 +-- lib/igt_primes.c | 10 --------- lib/igt_sysrq.h | 2 +- lib/tests/igt_abort.c | 12 +++++----- lib/tests/igt_assert.c | 2 +- lib/tests/igt_conflicting_args.c | 2 +- lib/tests/igt_describe.c | 2 +- lib/tests/igt_dynamic_subtests.c | 14 ++++++------ lib/tests/igt_fork.c | 12 +++++----- lib/tests/igt_invalid_subtest_name.c | 4 ++-- lib/tests/igt_nesting.c | 28 +++++++++++------------ lib/tests/igt_segfault.c | 2 +- lib/tests/igt_simulation.c | 2 +- lib/tests/igt_thread.c | 14 +++++++----- meson.build | 28 ++++++++++++----------- tests/dumb_buffer.c | 2 +- tests/i915/gem_gtt_speed.c | 2 +- tests/i915/gem_madvise.c | 2 +- tests/i915/gem_userptr_blits.c | 5 ++--- tests/i915/i915_pm_rpm.c | 2 +- tests/i915/perf_pmu.c | 2 +- tests/kms_atomic.c | 10 +++++---- tests/kms_vblank.c | 18 +++++++-------- tests/prime_nv_pcopy.c | 2 +- tests/prime_nv_test.c | 2 +- tests/testdisplay.c | 2 +- tests/vc4_purgeable_bo.c | 2 +- tools/skl_ddb_allocation.c | 9 -------- 32 files changed, 120 insertions(+), 133 deletions(-) diff --git a/benchmarks/kms_vblank.c b/benchmarks/kms_vblank.c index 838afa0dd..f32cc9e32 100644 --- a/benchmarks/kms_vblank.c +++ b/benchmarks/kms_vblank.c @@ -55,7 +55,7 @@ static int crtc0_active(int fd) union drm_wait_vblank vbl; memset(&vbl, 0, sizeof(vbl)); - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; return drmIoctl(fd, DRM_IOCTL_WAIT_VBLANK, &vbl) == 0; } @@ -74,14 +74,14 @@ static void vblank_query(int fd, int busy) drmIoctl(fd, DRM_IOCTL_WAIT_VBLANK, &vbl); } - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.sequence = 0; drmIoctl(fd, DRM_IOCTL_WAIT_VBLANK, &vbl); seq = vbl.reply.sequence; clock_gettime(CLOCK_MONOTONIC, &start); do { - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.sequence = 0; drmIoctl(fd, DRM_IOCTL_WAIT_VBLANK, &vbl); count++; @@ -108,7 +108,7 @@ static void vblank_event(int fd, int busy) drmIoctl(fd, DRM_IOCTL_WAIT_VBLANK, &vbl); } - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.sequence = 0; drmIoctl(fd, DRM_IOCTL_WAIT_VBLANK, &vbl); seq = vbl.reply.sequence; diff --git a/lib/igt_aux.c b/lib/igt_aux.c index fba349334..1217f5e88 100644 --- a/lib/igt_aux.c +++ b/lib/igt_aux.c @@ -276,7 +276,7 @@ bool __igt_sigiter_continue(struct __igt_sigiter *iter, bool enable) static struct igt_helper_process signal_helper; long long int sig_stat; -static void __attribute__((noreturn)) signal_helper_process(pid_t pid) +__noreturn static void signal_helper_process(pid_t pid) { /* Interrupt the parent process at 500Hz, just to be annoying */ while (1) { @@ -391,7 +391,7 @@ void igt_resume_signal_helper(void) } static struct igt_helper_process shrink_helper; -static void __attribute__((noreturn)) shrink_helper_process(int fd, pid_t pid) +__noreturn static void shrink_helper_process(int fd, pid_t pid) { while (1) { igt_drop_caches_set(fd, DROP_SHRINK_ALL); @@ -449,7 +449,7 @@ static void show_kernel_stack(pid_t pid) } static struct igt_helper_process hang_detector; -static void __attribute__((noreturn)) +__noreturn static void hang_detector_process(int fd, pid_t pid, dev_t rdev) { struct udev_monitor *mon = @@ -496,7 +496,7 @@ hang_detector_process(int fd, pid_t pid, dev_t rdev) exit(0); } -static void sig_abort(int sig) +__noreturn static void sig_abort(int sig) { errno = 0; /* inside a signal, last errno reporting is confusing */ igt_assert(!"GPU hung"); diff --git a/lib/igt_core.c b/lib/igt_core.c index 739e3efab..f9dfaa0dd 100644 --- a/lib/igt_core.c +++ b/lib/igt_core.c @@ -1376,8 +1376,7 @@ bool __igt_enter_dynamic_container(void) return true; } -static void exit_subtest(const char *) __attribute__((noreturn)); -static void exit_subtest(const char *result) +__noreturn static void exit_subtest(const char *result) { struct timespec now; const char *subtest_text = in_dynamic_subtest ? "Dynamic subtest" : "Subtest"; @@ -1610,7 +1609,7 @@ void igt_fail(int exitcode) * Since out test runner (piglit) does support fatal test exit codes, we * implement the default behaviour by waiting endlessly. */ -void __attribute__((noreturn)) igt_fatal_error(void) +void igt_fatal_error(void) { if (igt_check_boolean_env_var("IGT_REBOOT_ON_FATAL_ERROR", false)) { igt_warn("FATAL ERROR - REBOOTING\n"); @@ -2825,7 +2824,7 @@ out: } static const char *timeout_op; -static void __attribute__((noreturn)) igt_alarm_handler(int signal) +__noreturn static void igt_alarm_handler(int signal) { if (timeout_op) igt_info("Timed out: %s\n", timeout_op); diff --git a/lib/igt_core.h b/lib/igt_core.h index 206673875..6075d1539 100644 --- a/lib/igt_core.h +++ b/lib/igt_core.h @@ -79,6 +79,9 @@ #define igt_assume(e) BUILD_BUG_ON_INVALID(e) #endif +#define __noreturn __attribute__((noreturn)) +#define __maybe_unused __attribute__((unused)) + extern const char* __igt_test_description __attribute__((weak)); extern bool __igt_plain_output; extern char *igt_frame_dump_path; @@ -133,7 +136,7 @@ struct _GKeyFile *igt_load_igtrc(void); bool __igt_fixture(void); void __igt_fixture_complete(void); -void __igt_fixture_end(void) __attribute__((noreturn)); +__noreturn void __igt_fixture_end(void); /** * igt_fixture: * @@ -489,31 +492,29 @@ void igt_simple_init_parse_opts(int *argc, char **argv, __attribute__((constructor)) \ static void igt_tokencat(__igt_constructor_l, __LINE__)(void) -__attribute__((format(printf, 1, 2))) -void igt_skip(const char *f, ...) __attribute__((noreturn)); -__attribute__((format(printf, 5, 6))) +__noreturn __attribute__((format(printf, 1, 2))) +void igt_skip(const char *f, ...); +__noreturn __attribute__((format(printf, 5, 6))) void __igt_skip_check(const char *file, const int line, const char *func, const char *check, - const char *format, ...) __attribute__((noreturn)); + const char *format, ...); #define igt_skip_check(E, F...) \ __igt_skip_check(__FILE__, __LINE__, __func__, E, F) void igt_success(void); bool igt_can_fail(void); -void igt_fail(int exitcode) __attribute__((noreturn)); -__attribute__((format(printf, 6, 7))) +__noreturn void igt_fail(int exitcode); +__noreturn __attribute__((format(printf, 6, 7))) void __igt_fail_assert(const char *domain, const char *file, const int line, const char *func, const char *assertion, - const char *format, ...) - __attribute__((noreturn)); -__attribute__((format(printf, 6, 7))) -void __igt_abort(const char *domain, const char *file, const int line, - const char *func, const char *expression, - const char *f, ...) - __attribute__((noreturn)); -void igt_exit(void) __attribute__((noreturn)); -void igt_fatal_error(void) __attribute__((noreturn)); + const char *format, ...); +__noreturn __attribute__((format(printf, 6, 7))) +void __igt_abort(const char *domain, const char *file, + const int line, const char *func, const char *expression, + const char *f, ...); +__noreturn void igt_exit(void); +__noreturn void igt_fatal_error(void); /** * igt_ignore_warn: diff --git a/lib/igt_gt.c b/lib/igt_gt.c index 7dfcac038..f601d7260 100644 --- a/lib/igt_gt.c +++ b/lib/igt_gt.c @@ -380,8 +380,7 @@ void igt_force_gpu_reset(int drm_fd) /* GPU abusers */ static struct igt_helper_process hang_helper; -static void __attribute__((noreturn)) -hang_helper_process(pid_t pid, int fd) +__noreturn static void hang_helper_process(pid_t pid, int fd) { while (1) { if (kill(pid, 0)) /* Parent has died, so must we. */ diff --git a/lib/igt_primes.c b/lib/igt_primes.c index 3578a3c69..3941bb21a 100644 --- a/lib/igt_primes.c +++ b/lib/igt_primes.c @@ -64,21 +64,11 @@ static inline unsigned long __bit__(unsigned long nr) return 1UL << (nr % BITS_PER_LONG); } -static inline void set_bit(unsigned long nr, unsigned long *addr) -{ - addr[nr / BITS_PER_LONG] |= __bit__(nr); -} - static inline void clear_bit(unsigned long nr, unsigned long *addr) { addr[nr / BITS_PER_LONG] &= ~__bit__(nr); } -static inline bool test_bit(unsigned long nr, const unsigned long *addr) -{ - return addr[nr / BITS_PER_LONG] & __bit__(nr); -} - static unsigned long __find_next_bit(const unsigned long *addr, unsigned long nbits, unsigned long start, diff --git a/lib/igt_sysrq.h b/lib/igt_sysrq.h index 422473d2a..c941029d5 100644 --- a/lib/igt_sysrq.h +++ b/lib/igt_sysrq.h @@ -25,6 +25,6 @@ #ifndef __IGT_SYSRQ_H__ #define __IGT_SYSRQ_H__ -void igt_sysrq_reboot(void) __attribute__((noreturn)); +__noreturn void igt_sysrq_reboot(void); #endif /* __IGT_SYSRQ_H__ */ diff --git a/lib/tests/igt_abort.c b/lib/tests/igt_abort.c index 53b7d4fd5..356e41679 100644 --- a/lib/tests/igt_abort.c +++ b/lib/tests/igt_abort.c @@ -31,7 +31,7 @@ char test[] = "test"; char *fake_argv[] = { test }; int fake_argc = ARRAY_SIZE(fake_argv); -static void fake_simple_test(void) +__noreturn static void fake_simple_test(void) { igt_simple_init(fake_argc, fake_argv); @@ -40,7 +40,7 @@ static void fake_simple_test(void) exit(0); /* unreachable */ } -static void fake_fixture_test(void) +__noreturn static void fake_fixture_test(void) { igt_subtest_init(fake_argc, fake_argv); @@ -51,7 +51,7 @@ static void fake_fixture_test(void) exit(0); /* unreachable */ } -static void fake_outside_fixture_test(void) +__noreturn static void fake_outside_fixture_test(void) { igt_subtest_init(fake_argc, fake_argv); @@ -60,7 +60,7 @@ static void fake_outside_fixture_test(void) exit(0); /* unreachable */ } -static void fake_subtest_test(void) +__noreturn static void fake_subtest_test(void) { igt_subtest_init(fake_argc, fake_argv); @@ -76,7 +76,7 @@ static void fake_subtest_test(void) exit(0); /* unreachable */ } -static void fake_dynamic_test(void) +__noreturn static void fake_dynamic_test(void) { igt_subtest_init(fake_argc, fake_argv); @@ -97,7 +97,7 @@ static void fake_dynamic_test(void) exit(0); /* unreachable */ } -static void fake_outside_dynamic_test(void) +__noreturn static void fake_outside_dynamic_test(void) { igt_subtest_init(fake_argc, fake_argv); diff --git a/lib/tests/igt_assert.c b/lib/tests/igt_assert.c index c94ac6989..3523a4649 100644 --- a/lib/tests/igt_assert.c +++ b/lib/tests/igt_assert.c @@ -53,7 +53,7 @@ void (*test_to_run)(void) = NULL; exec_total++; \ } -static void fake_test(void) +__noreturn static void fake_test(void) { char test[] = "test"; char *argv_run[] = { test }; diff --git a/lib/tests/igt_conflicting_args.c b/lib/tests/igt_conflicting_args.c index b644fd4bc..b5fe2d781 100644 --- a/lib/tests/igt_conflicting_args.c +++ b/lib/tests/igt_conflicting_args.c @@ -43,7 +43,7 @@ static int opt_handler(int option, int option_index, void *input) return 0; } -static void fake_test(void) +__noreturn static void fake_test(void) { char test_name[] = "test"; char *argv[] = { test_name }; diff --git a/lib/tests/igt_describe.c b/lib/tests/igt_describe.c index ff45a6c4b..d71e051e7 100644 --- a/lib/tests/igt_describe.c +++ b/lib/tests/igt_describe.c @@ -34,7 +34,7 @@ char *fake_argv[] = {prog, fake_arg}; int fake_argc = ARRAY_SIZE(fake_argv); IGT_TEST_DESCRIPTION("the top level description"); -static void fake_main(void) +__noreturn static void fake_main(void) { igt_subtest_init(fake_argc, fake_argv); diff --git a/lib/tests/igt_dynamic_subtests.c b/lib/tests/igt_dynamic_subtests.c index bd1e16758..b18cfae6c 100644 --- a/lib/tests/igt_dynamic_subtests.c +++ b/lib/tests/igt_dynamic_subtests.c @@ -29,7 +29,7 @@ #include "igt_tests_common.h" -static void dynamic_subtest_in_normal_subtest(void) +__noreturn static void dynamic_subtest_in_normal_subtest(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -46,7 +46,7 @@ static void dynamic_subtest_in_normal_subtest(void) igt_exit(); } -static void invalid_dynamic_subtest_name(void) +__noreturn static void invalid_dynamic_subtest_name(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -63,7 +63,7 @@ static void invalid_dynamic_subtest_name(void) igt_exit(); } -static void dynamic_subtest_in_toplevel(void) +__noreturn static void dynamic_subtest_in_toplevel(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -78,7 +78,7 @@ static void dynamic_subtest_in_toplevel(void) igt_exit(); } -static void subtest_itself_failing(void) +__noreturn static void subtest_itself_failing(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -93,7 +93,7 @@ static void subtest_itself_failing(void) igt_exit(); } -static void subtest_itself_skipping(void) +__noreturn static void subtest_itself_skipping(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -108,7 +108,7 @@ static void subtest_itself_skipping(void) igt_exit(); } -static void dynamic_subtest_failure_leads_to_fail(void) +__noreturn static void dynamic_subtest_failure_leads_to_fail(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -125,7 +125,7 @@ static void dynamic_subtest_failure_leads_to_fail(void) igt_exit(); } -static void no_dynamic_subtests_entered_leads_to_skip(void) +__noreturn static void no_dynamic_subtests_entered_leads_to_skip(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; diff --git a/lib/tests/igt_fork.c b/lib/tests/igt_fork.c index 84492b2d1..d19d09452 100644 --- a/lib/tests/igt_fork.c +++ b/lib/tests/igt_fork.c @@ -38,7 +38,7 @@ char test[] = "test"; char *fake_argv[] = { test }; int fake_argc = ARRAY_SIZE(fake_argv); -static void igt_fork_vs_skip(void) +__noreturn static void igt_fork_vs_skip(void) { igt_simple_init(fake_argc, fake_argv); @@ -51,7 +51,7 @@ static void igt_fork_vs_skip(void) igt_exit(); } -static void igt_fork_vs_assert(void) +__noreturn static void igt_fork_vs_assert(void) { igt_simple_init(fake_argc, fake_argv); @@ -64,7 +64,7 @@ static void igt_fork_vs_assert(void) igt_exit(); } -static void igt_fork_leak(void) +__noreturn static void igt_fork_leak(void) { igt_simple_init(fake_argc, fake_argv); @@ -75,7 +75,7 @@ static void igt_fork_leak(void) igt_exit(); } -static void plain_fork_leak(void) +__noreturn static void plain_fork_leak(void) { int pid; @@ -93,7 +93,7 @@ static void plain_fork_leak(void) igt_exit(); } -static void igt_fork_timeout_leak(void) +__noreturn static void igt_fork_timeout_leak(void) { igt_simple_init(fake_argc, fake_argv); @@ -106,7 +106,7 @@ static void igt_fork_timeout_leak(void) igt_exit(); } -static void subtest_leak(void) +__noreturn static void subtest_leak(void) { pid_t *children = mmap(0, 4096, PROT_WRITE, MAP_SHARED | MAP_ANON, -1, 0); diff --git a/lib/tests/igt_invalid_subtest_name.c b/lib/tests/igt_invalid_subtest_name.c index 32c4bcbc9..b790482c0 100644 --- a/lib/tests/igt_invalid_subtest_name.c +++ b/lib/tests/igt_invalid_subtest_name.c @@ -29,7 +29,7 @@ #include "igt_tests_common.h" -static void invalid_subtest_name(void) +__noreturn static void invalid_subtest_name(void) { char prog[] = "igt_no_exit"; char *fake_argv[] = {prog}; @@ -44,7 +44,7 @@ static void invalid_subtest_name(void) igt_exit(); } -static void nonexisting_subtest(void) +__noreturn static void nonexisting_subtest(void) { char prog[] = "igt_no_exit"; char arg1[] = "--run-subtest"; diff --git a/lib/tests/igt_nesting.c b/lib/tests/igt_nesting.c index de674b6b8..6b209af82 100644 --- a/lib/tests/igt_nesting.c +++ b/lib/tests/igt_nesting.c @@ -30,7 +30,7 @@ char test[] = "test"; char *fake_argv[] = { test }; int fake_argc = ARRAY_SIZE(fake_argv); -static void all_valid_simple_test(void) +__noreturn static void all_valid_simple_test(void) { igt_simple_init(fake_argc, fake_argv); @@ -40,7 +40,7 @@ static void all_valid_simple_test(void) igt_exit(); } -static void all_valid(void) +__noreturn static void all_valid(void) { igt_subtest_init(fake_argc, fake_argv); @@ -87,7 +87,7 @@ static void all_valid(void) igt_exit(); } -static void invalid_subtest_in_simple_test(void) +__noreturn static void invalid_subtest_in_simple_test(void) { igt_simple_init(fake_argc, fake_argv); @@ -97,7 +97,7 @@ static void invalid_subtest_in_simple_test(void) igt_exit(); } -static void invalid_subtest_group_in_simple_test(void) +__noreturn static void invalid_subtest_group_in_simple_test(void) { igt_simple_init(fake_argc, fake_argv); @@ -107,7 +107,7 @@ static void invalid_subtest_group_in_simple_test(void) igt_exit(); } -static void invalid_subtest_with_dynamic_in_simple_test(void) +__noreturn static void invalid_subtest_with_dynamic_in_simple_test(void) { igt_simple_init(fake_argc, fake_argv); @@ -117,7 +117,7 @@ static void invalid_subtest_with_dynamic_in_simple_test(void) igt_exit(); } -static void invalid_dynamic_in_simple_test(void) +__noreturn static void invalid_dynamic_in_simple_test(void) { igt_simple_init(fake_argc, fake_argv); @@ -127,7 +127,7 @@ static void invalid_dynamic_in_simple_test(void) igt_exit(); } -static void invalid_fixture_in_fixture(void) +__noreturn static void invalid_fixture_in_fixture(void) { igt_subtest_init(fake_argc, fake_argv); @@ -139,7 +139,7 @@ static void invalid_fixture_in_fixture(void) igt_exit(); } -static void invalid_subtest_in_subtest(void) +__noreturn static void invalid_subtest_in_subtest(void) { igt_subtest_init(fake_argc, fake_argv); @@ -151,7 +151,7 @@ static void invalid_subtest_in_subtest(void) igt_exit(); } -static void invalid_top_level_dynamic(void) +__noreturn static void invalid_top_level_dynamic(void) { igt_subtest_init(fake_argc, fake_argv); @@ -161,7 +161,7 @@ static void invalid_top_level_dynamic(void) igt_exit(); } -static void invalid_dynamic_in_regular_subtest(void) +__noreturn static void invalid_dynamic_in_regular_subtest(void) { igt_subtest_init(fake_argc, fake_argv); @@ -173,7 +173,7 @@ static void invalid_dynamic_in_regular_subtest(void) igt_exit(); } -static void invalid_fixture_in_subtest(void) +__noreturn static void invalid_fixture_in_subtest(void) { igt_subtest_init(fake_argc, fake_argv); @@ -185,7 +185,7 @@ static void invalid_fixture_in_subtest(void) igt_exit(); } -static void invalid_top_level_skip(void) +__noreturn static void invalid_top_level_skip(void) { igt_subtest_init(fake_argc, fake_argv); @@ -194,7 +194,7 @@ static void invalid_top_level_skip(void) igt_exit(); } -static void invalid_top_level_assert(void) +__noreturn static void invalid_top_level_assert(void) { igt_subtest_init(fake_argc, fake_argv); @@ -203,7 +203,7 @@ static void invalid_top_level_assert(void) igt_exit(); } -static void invalid_dynamic_in_dynamic(void) +__noreturn static void invalid_dynamic_in_dynamic(void) { igt_subtest_init(fake_argc, fake_argv); diff --git a/lib/tests/igt_segfault.c b/lib/tests/igt_segfault.c index 38c040a6f..56efa9125 100644 --- a/lib/tests/igt_segfault.c +++ b/lib/tests/igt_segfault.c @@ -54,7 +54,7 @@ static void crashme(void) raise(SIGSEGV); } -static void fake_test(void) +__noreturn static void fake_test(void) { char prog[] = "test"; char *fake_argv[] = { prog }; diff --git a/lib/tests/igt_simulation.c b/lib/tests/igt_simulation.c index a0ea70008..9a2cf2aae 100644 --- a/lib/tests/igt_simulation.c +++ b/lib/tests/igt_simulation.c @@ -40,7 +40,7 @@ bool list_subtests; bool in_fixture; bool in_subtest; -static void fake_test(void) +__noreturn static void fake_test(void) { char test[] = "test"; char list[] = "--list-subtests"; diff --git a/lib/tests/igt_thread.c b/lib/tests/igt_thread.c index e9d37584c..eb92326dc 100644 --- a/lib/tests/igt_thread.c +++ b/lib/tests/igt_thread.c @@ -49,7 +49,8 @@ static void *require_thread(void *data) return NULL; } -static void one_subtest_fail(void) { +__noreturn static void one_subtest_fail(void) +{ igt_subtest_init(fake_argc, fake_argv); igt_subtest("subtest-a") { @@ -67,7 +68,8 @@ static void one_subtest_fail(void) { igt_exit(); } -static void one_dynamic_fail(void) { +__noreturn static void one_dynamic_fail(void) +{ igt_subtest_init(fake_argc, fake_argv); igt_subtest_with_dynamic("dynamic-container") { @@ -87,7 +89,7 @@ static void one_dynamic_fail(void) { igt_exit(); } -static void simple_success(void) { +__noreturn static void simple_success(void) { pthread_t thread; igt_simple_init(fake_argc, fake_argv); @@ -98,7 +100,8 @@ static void simple_success(void) { igt_exit(); } -static void simple_failure(void) { +__noreturn static void simple_failure(void) +{ pthread_t thread; igt_simple_init(fake_argc, fake_argv); @@ -109,7 +112,8 @@ static void simple_failure(void) { igt_exit(); } -static void require_non_main_thread(void) { +__noreturn static void require_non_main_thread(void) +{ pthread_t thread; igt_simple_init(fake_argc, fake_argv); diff --git a/meson.build b/meson.build index f4ee14f14..3f3693f99 100644 --- a/meson.build +++ b/meson.build @@ -51,19 +51,21 @@ cc_args = [ '-Wno-unused-parameter', '-Wno-unused-result', - '-Werror=address', - '-Werror=array-bounds', - '-Werror=implicit', - '-Werror=init-self', - '-Werror=int-to-pointer-cast', - '-Werror=main', - '-Werror=missing-braces', - '-Werror=nonnull', - '-Werror=pointer-to-int-cast', - '-Werror=return-type', - '-Werror=sequence-point', - '-Werror=trigraphs', - '-Werror=write-strings', + '-Waddress', + '-Warray-bounds', + '-Wimplicit', + '-Winit-self', + '-Wint-to-pointer-cast', + '-Wmain', + '-Wmissing-braces', + '-Wnonnull', + '-Wpointer-to-int-cast', + '-Wreturn-type', + '-Wsequence-point', + '-Wtrigraphs', + '-Wwrite-strings', + + '-Werror', # Disable the memory allocating builtins as they may cause unexpected behavior # with our framework. They *may* get optimized out in favor of a register or # stack variable, making them effectively local. Local variables do not play diff --git a/tests/dumb_buffer.c b/tests/dumb_buffer.c index 44f60b6f1..cb8d2f518 100644 --- a/tests/dumb_buffer.c +++ b/tests/dumb_buffer.c @@ -278,7 +278,7 @@ static void *thread_clear(void *data) static jmp_buf sigjmp; -static void sigprobe(int sig) +__noreturn static void sigprobe(int sig) { longjmp(sigjmp, sig); } diff --git a/tests/i915/gem_gtt_speed.c b/tests/i915/gem_gtt_speed.c index 06fe70f84..644af3d38 100644 --- a/tests/i915/gem_gtt_speed.c +++ b/tests/i915/gem_gtt_speed.c @@ -82,7 +82,7 @@ static inline unsigned x86_64_features(void) { return 0; } -static void streaming_load(void *src, int len) +__noreturn static void streaming_load(void *src, int len) { igt_assert(!"reached"); } diff --git a/tests/i915/gem_madvise.c b/tests/i915/gem_madvise.c index 623c8b091..2cd0b5d7c 100644 --- a/tests/i915/gem_madvise.c +++ b/tests/i915/gem_madvise.c @@ -51,7 +51,7 @@ IGT_TEST_DESCRIPTION("Checks that the kernel reports EFAULT when trying to use" static jmp_buf jmp; -static void __attribute__((noreturn)) sigtrap(int sig) +__noreturn static void sigtrap(int sig) { siglongjmp(jmp, sig); } diff --git a/tests/i915/gem_userptr_blits.c b/tests/i915/gem_userptr_blits.c index 2f3954c5d..1bc2d3600 100644 --- a/tests/i915/gem_userptr_blits.c +++ b/tests/i915/gem_userptr_blits.c @@ -55,8 +55,6 @@ #include <pthread.h> #include <time.h> -#include <linux/memfd.h> - #include "drm.h" #include "i915_drm.h" @@ -1550,7 +1548,8 @@ static void test_readonly(int i915) } static jmp_buf sigjmp; -static void sigjmp_handler(int sig) + +__noreturn static void sigjmp_handler(int sig) { siglongjmp(sigjmp, sig); } diff --git a/tests/i915/i915_pm_rpm.c b/tests/i915/i915_pm_rpm.c index 6321dd403..d9ec0388c 100644 --- a/tests/i915/i915_pm_rpm.c +++ b/tests/i915/i915_pm_rpm.c @@ -1483,7 +1483,7 @@ static void pci_d3_state_subtest(void) } } -static void __attribute__((noreturn)) stay_subtest(void) +__noreturn static void stay_subtest(void) { disable_all_screens_and_wait(&ms_data); diff --git a/tests/i915/perf_pmu.c b/tests/i915/perf_pmu.c index 7b175ea8d..50b5c82bc 100644 --- a/tests/i915/perf_pmu.c +++ b/tests/i915/perf_pmu.c @@ -1092,7 +1092,7 @@ event_wait(int gem_fd, const struct intel_execution_engine2 *e) for (;;) { union drm_wait_vblank vbl = { }; - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 1; igt_assert_eq(wait_vblank(gem_fd, &vbl), 0); diff --git a/tests/kms_atomic.c b/tests/kms_atomic.c index fa6190c9b..2965b350b 100644 --- a/tests/kms_atomic.c +++ b/tests/kms_atomic.c @@ -448,11 +448,11 @@ plane_immutable_zpos(igt_display_t *display, igt_pipe_t *pipe, for (int i = 0; i < n_planes - 1; i++) { igt_plane_t *plane_lower, *plane_upper; - if (plane_ptr[i] != NULL) - plane_lower = plane_ptr[i]; - else + if (plane_ptr[i] == NULL) continue; + plane_lower = plane_ptr[i]; + plane_upper = NULL; while (i < (n_planes - 1)) { if (plane_ptr[i + 1] != NULL) { plane_upper = plane_ptr[i + 1]; @@ -462,9 +462,11 @@ plane_immutable_zpos(igt_display_t *display, igt_pipe_t *pipe, continue; } } + if (!plane_upper) + continue; if ((plane_upper->type == DRM_PLANE_TYPE_CURSOR) || - (plane_lower->type == DRM_PLANE_TYPE_CURSOR)) + (plane_lower->type == DRM_PLANE_TYPE_CURSOR)) continue; igt_plane_set_position(plane_lower, 0, 0); diff --git a/tests/kms_vblank.c b/tests/kms_vblank.c index b00ebec53..31b3f0a69 100644 --- a/tests/kms_vblank.c +++ b/tests/kms_vblank.c @@ -230,14 +230,14 @@ static void accuracy(data_t *data, int fd, int nchildren) int n; memset(&vbl, 0, sizeof(vbl)); - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 1; igt_assert_eq(wait_vblank(fd, &vbl), 0); target = vbl.reply.sequence + total; for (n = 0; n < total; n++) { - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 1; igt_assert_eq(wait_vblank(fd, &vbl), 0); @@ -247,7 +247,7 @@ static void accuracy(data_t *data, int fd, int nchildren) vbl.request.sequence = target; igt_assert_eq(wait_vblank(fd, &vbl), 0); } - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 0; igt_assert_eq(wait_vblank(fd, &vbl), 0); @@ -268,7 +268,7 @@ static void vblank_query(data_t *data, int fd, int nchildren) unsigned long sq, count = 0; memset(&vbl, 0, sizeof(vbl)); - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 0; igt_assert_eq(wait_vblank(fd, &vbl), 0); @@ -277,7 +277,7 @@ static void vblank_query(data_t *data, int fd, int nchildren) clock_gettime(CLOCK_MONOTONIC, &start); do { - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 0; igt_assert_eq(wait_vblank(fd, &vbl), 0); @@ -297,7 +297,7 @@ static void vblank_wait(data_t *data, int fd, int nchildren) unsigned long sq, count = 0; memset(&vbl, 0, sizeof(vbl)); - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 0; igt_assert_eq(wait_vblank(fd, &vbl), 0); @@ -306,7 +306,7 @@ static void vblank_wait(data_t *data, int fd, int nchildren) clock_gettime(CLOCK_MONOTONIC, &start); do { - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= pipe_id_flag; vbl.request.sequence = 1; igt_assert_eq(wait_vblank(fd, &vbl), 0); @@ -367,7 +367,7 @@ static void vblank_ts_cont(data_t *data, int fd, int nchildren) if (data->flags & (MODESET | DPMS)) { /* Attempting to do a vblank while disabled should return -EINVAL */ memset(&vbl, 0, sizeof(vbl)); - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; vbl.request.type |= kmstest_get_vbl_flag(data->pipe); igt_assert_eq(wait_vblank(fd, &vbl), -EINVAL); } @@ -476,7 +476,7 @@ static void invalid_subtest(data_t *data, int fd) /* First check all is well with a simple query */ memset(&vbl, 0, sizeof(vbl)); - vbl.request.type = DRM_VBLANK_RELATIVE; + vbl.request.type = _DRM_VBLANK_RELATIVE; igt_assert_eq(wait_vblank(fd, &vbl), 0); valid_flags = (_DRM_VBLANK_TYPES_MASK | diff --git a/tests/prime_nv_pcopy.c b/tests/prime_nv_pcopy.c index bbcf76807..e465e5fc0 100644 --- a/tests/prime_nv_pcopy.c +++ b/tests/prime_nv_pcopy.c @@ -139,7 +139,7 @@ BEGIN_NV04(struct nouveau_pushbuf *push, int subc, int mthd, int size) PUSH_DATA (push, 0x00000000 | (size << 18) | (subc << 13) | mthd); } -static inline void +__maybe_unused static inline void BEGIN_NI04(struct nouveau_pushbuf *push, int subc, int mthd, int size) { PUSH_DATA (push, 0x40000000 | (size << 18) | (subc << 13) | mthd); diff --git a/tests/prime_nv_test.c b/tests/prime_nv_test.c index b4714dcb5..81d142f42 100644 --- a/tests/prime_nv_test.c +++ b/tests/prime_nv_test.c @@ -190,7 +190,7 @@ static void test_nv_write_i915_gtt_mmap_read(void) /* test drm_intel_bo_map doesn't work properly, this tries to map the backing shmem fd, which doesn't exist for these objects */ -static void test_i915_import_cpu_mmap(void) +__noreturn static void test_i915_import_cpu_mmap(void) { drm_intel_bo *test_intel_bo; int prime_fd; diff --git a/tests/testdisplay.c b/tests/testdisplay.c index ef033dc60..e2560eaf5 100644 --- a/tests/testdisplay.c +++ b/tests/testdisplay.c @@ -542,7 +542,7 @@ int update_display(bool probe) #define dump_resource(res) if (res) dump_##res() -static void __attribute__((noreturn)) cleanup_and_exit(int ret) +__noreturn static void cleanup_and_exit(int ret) { close(drm_fd); exit(ret); diff --git a/tests/vc4_purgeable_bo.c b/tests/vc4_purgeable_bo.c index ee89e153f..d9f7547f5 100644 --- a/tests/vc4_purgeable_bo.c +++ b/tests/vc4_purgeable_bo.c @@ -44,7 +44,7 @@ struct igt_vc4_bo { static jmp_buf jmp; -static void __attribute__((noreturn)) sigtrap(int sig) +__noreturn static void sigtrap(int sig) { longjmp(jmp, sig); } diff --git a/tools/skl_ddb_allocation.c b/tools/skl_ddb_allocation.c index c7bfb2791..ff4c0aaff 100644 --- a/tools/skl_ddb_allocation.c +++ b/tools/skl_ddb_allocation.c @@ -140,15 +140,6 @@ static inline uint16_t skl_ddb_entry_size(const struct skl_ddb_entry *entry) return entry->end - entry->start; } -static inline bool skl_ddb_entry_equal(const struct skl_ddb_entry *e1, - const struct skl_ddb_entry *e2) -{ - if (e1->start == e2->start && e1->end == e2->end) - return true; - - return false; -} - struct skl_ddb_allocation { struct skl_ddb_entry plane[I915_MAX_PIPES][I915_MAX_PLANES]; struct skl_ddb_entry cursor[I915_MAX_PIPES]; -- 2.30.0 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [igt-dev] ✗ GitLab.Pipeline: warning for meson: Turn on -Werror (rev4) 2021-01-30 6:44 [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Ashutosh Dixit ` (6 preceding siblings ...) 2021-01-30 12:42 ` [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Chris Wilson @ 2021-01-30 13:05 ` Patchwork 2021-01-30 13:22 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork 2021-01-30 14:26 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork 9 siblings, 0 replies; 13+ messages in thread From: Patchwork @ 2021-01-30 13:05 UTC (permalink / raw) To: Chris Wilson; +Cc: igt-dev == Series Details == Series: meson: Turn on -Werror (rev4) URL : https://patchwork.freedesktop.org/series/86465/ State : warning == Summary == Did not get list of undocumented tests for this run, something is wrong! Other than that, pipeline status: FAILED. see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/263968 for the overview. build:tests-fedora-clang has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/6996101): ../assembler/gen4asm.h:63:21: note: passing argument to parameter 'msg' here void yyerror (char *msg); ^ assembler/1ca89e8@@intel-gen4asm@exe/gram.c:5776:16: error: passing 'const char [17]' to parameter of type 'char *' discards qualifiers [-Werror,-Wincompatible-pointer-types-discards-qualifiers] yyerror (YY_("memory exhausted")); ^~~~~~~~~~~~~~~~~~ assembler/1ca89e8@@intel-gen4asm@exe/gram.c:914:22: note: expanded from macro 'YY_' # define YY_(Msgid) Msgid ^~~~~ ../assembler/gen4asm.h:63:21: note: passing argument to parameter 'msg' here void yyerror (char *msg); ^ 3 errors generated. ninja: build stopped: subcommand failed. section_end:1612011154:step_script section_start:1612011154:cleanup_file_variables Cleaning up file based variables section_end:1612011155:cleanup_file_variables ERROR: Job failed: exit code 1 == Logs == For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/263968 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for meson: Turn on -Werror (rev4) 2021-01-30 6:44 [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Ashutosh Dixit ` (7 preceding siblings ...) 2021-01-30 13:05 ` [igt-dev] ✗ GitLab.Pipeline: warning for meson: Turn on -Werror (rev4) Patchwork @ 2021-01-30 13:22 ` Patchwork 2021-01-30 14:26 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork 9 siblings, 0 replies; 13+ messages in thread From: Patchwork @ 2021-01-30 13:22 UTC (permalink / raw) To: Chris Wilson; +Cc: igt-dev [-- Attachment #1.1: Type: text/plain, Size: 3163 bytes --] == Series Details == Series: meson: Turn on -Werror (rev4) URL : https://patchwork.freedesktop.org/series/86465/ State : success == Summary == CI Bug Log - changes from CI_DRM_9705 -> IGTPW_5450 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/index.html Known issues ------------ Here are the changes found in IGTPW_5450 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@kms_chamelium@common-hpd-after-suspend: - fi-skl-6700k2: [PASS][1] -> [INCOMPLETE][2] ([i915#146] / [i915#2405]) [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/fi-skl-6700k2/igt@kms_chamelium@common-hpd-after-suspend.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/fi-skl-6700k2/igt@kms_chamelium@common-hpd-after-suspend.html * igt@prime_self_import@basic-with_two_bos: - fi-tgl-y: [PASS][3] -> [DMESG-WARN][4] ([i915#402]) [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/fi-tgl-y/igt@prime_self_import@basic-with_two_bos.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/fi-tgl-y/igt@prime_self_import@basic-with_two_bos.html #### Possible fixes #### * igt@gem_linear_blits@basic: - fi-tgl-y: [DMESG-WARN][5] ([i915#402]) -> [PASS][6] +1 similar issue [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/fi-tgl-y/igt@gem_linear_blits@basic.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/fi-tgl-y/igt@gem_linear_blits@basic.html * igt@i915_selftest@live@gem_contexts: - fi-tgl-y: [SKIP][7] ([i915#2966]) -> [PASS][8] +13 similar issues [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/fi-tgl-y/igt@i915_selftest@live@gem_contexts.html [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/fi-tgl-y/igt@i915_selftest@live@gem_contexts.html * igt@i915_selftest@live@gtt: - fi-tgl-y: [FAIL][9] -> [PASS][10] [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/fi-tgl-y/igt@i915_selftest@live@gtt.html [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/fi-tgl-y/igt@i915_selftest@live@gtt.html [i915#146]: https://gitlab.freedesktop.org/drm/intel/issues/146 [i915#2405]: https://gitlab.freedesktop.org/drm/intel/issues/2405 [i915#2966]: https://gitlab.freedesktop.org/drm/intel/issues/2966 [i915#402]: https://gitlab.freedesktop.org/drm/intel/issues/402 Participating hosts (44 -> 38) ------------------------------ Missing (6): fi-jsl-1 fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-dg1-1 fi-bdw-samus Build changes ------------- * CI: CI-20190529 -> None * IGT: IGT_5979 -> IGTPW_5450 CI-20190529: 20190529 CI_DRM_9705: 7b1b769c42017c6e6d603dbea0182e74090451f1 @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_5450: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/index.html IGT_5979: fdc23507d022b68443121ec2c1a951af27c87240 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/index.html [-- Attachment #1.2: Type: text/html, Size: 3945 bytes --] [-- Attachment #2: Type: text/plain, Size: 154 bytes --] _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for meson: Turn on -Werror (rev4) 2021-01-30 6:44 [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Ashutosh Dixit ` (8 preceding siblings ...) 2021-01-30 13:22 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork @ 2021-01-30 14:26 ` Patchwork 9 siblings, 0 replies; 13+ messages in thread From: Patchwork @ 2021-01-30 14:26 UTC (permalink / raw) To: Chris Wilson; +Cc: igt-dev [-- Attachment #1.1: Type: text/plain, Size: 30247 bytes --] == Series Details == Series: meson: Turn on -Werror (rev4) URL : https://patchwork.freedesktop.org/series/86465/ State : success == Summary == CI Bug Log - changes from CI_DRM_9705_full -> IGTPW_5450_full ==================================================== Summary ------- **WARNING** Minor unknown changes coming with IGTPW_5450_full need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in IGTPW_5450_full, please notify your bug team to allow them to document this new failure mode, which will reduce false positives in CI. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/index.html Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_5450_full: ### IGT changes ### #### Warnings #### * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a: - shard-kbl: [DMESG-WARN][1] ([i915#180]) -> [INCOMPLETE][2] [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl4/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl4/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html #### Suppressed #### The following results come from untrusted machines, tests, or statuses. They do not affect the overall result. * {igt@sysfs_clients@recycle}: - shard-glk: [PASS][3] -> [FAIL][4] [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk8/igt@sysfs_clients@recycle.html [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk4/igt@sysfs_clients@recycle.html - shard-tglb: [PASS][5] -> [FAIL][6] [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-tglb8/igt@sysfs_clients@recycle.html [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb3/igt@sysfs_clients@recycle.html Known issues ------------ Here are the changes found in IGTPW_5450_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_persistence@process: - shard-snb: NOTRUN -> [SKIP][7] ([fdo#109271] / [i915#1099]) +1 similar issue [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-snb5/igt@gem_ctx_persistence@process.html * igt@gem_ctx_persistence@processes: - shard-hsw: NOTRUN -> [SKIP][8] ([fdo#109271] / [i915#1099]) +2 similar issues [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-hsw6/igt@gem_ctx_persistence@processes.html * igt@gem_exec_balancer@full-pulse: - shard-snb: NOTRUN -> [SKIP][9] ([fdo#109271]) +66 similar issues [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-snb4/igt@gem_exec_balancer@full-pulse.html * igt@gem_exec_fair@basic-deadline: - shard-kbl: [PASS][10] -> [FAIL][11] ([i915#2846]) [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl6/igt@gem_exec_fair@basic-deadline.html [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl4/igt@gem_exec_fair@basic-deadline.html * igt@gem_exec_fair@basic-flow@rcs0: - shard-kbl: [PASS][12] -> [SKIP][13] ([fdo#109271]) [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl4/igt@gem_exec_fair@basic-flow@rcs0.html [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl1/igt@gem_exec_fair@basic-flow@rcs0.html * igt@gem_exec_fair@basic-none@vecs0: - shard-kbl: [PASS][14] -> [FAIL][15] ([i915#2842]) [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl2/igt@gem_exec_fair@basic-none@vecs0.html [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl7/igt@gem_exec_fair@basic-none@vecs0.html * igt@gem_exec_fair@basic-pace-share@rcs0: - shard-tglb: [PASS][16] -> [FAIL][17] ([i915#2842]) +1 similar issue [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-tglb2/igt@gem_exec_fair@basic-pace-share@rcs0.html [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb5/igt@gem_exec_fair@basic-pace-share@rcs0.html * igt@gem_exec_fair@basic-throttle@rcs0: - shard-glk: [PASS][18] -> [FAIL][19] ([i915#2842]) +3 similar issues [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk4/igt@gem_exec_fair@basic-throttle@rcs0.html [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk8/igt@gem_exec_fair@basic-throttle@rcs0.html * igt@gem_exec_params@rsvd2-dirt: - shard-tglb: NOTRUN -> [SKIP][20] ([fdo#109283]) [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb8/igt@gem_exec_params@rsvd2-dirt.html - shard-iclb: NOTRUN -> [SKIP][21] ([fdo#109283]) [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb2/igt@gem_exec_params@rsvd2-dirt.html * igt@gem_exec_schedule@u-fairslice@rcs0: - shard-tglb: [PASS][22] -> [DMESG-WARN][23] ([i915#2803]) [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-tglb8/igt@gem_exec_schedule@u-fairslice@rcs0.html [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb2/igt@gem_exec_schedule@u-fairslice@rcs0.html * igt@gem_exec_schedule@u-fairslice@vcs1: - shard-kbl: [PASS][24] -> [DMESG-WARN][25] ([i915#1610] / [i915#2803]) [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl2/igt@gem_exec_schedule@u-fairslice@vcs1.html [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl4/igt@gem_exec_schedule@u-fairslice@vcs1.html * igt@gem_fenced_exec_thrash@2-spare-fences: - shard-snb: [PASS][26] -> [INCOMPLETE][27] ([i915#2055]) [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-snb6/igt@gem_fenced_exec_thrash@2-spare-fences.html [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-snb6/igt@gem_fenced_exec_thrash@2-spare-fences.html * igt@gem_pread@exhaustion: - shard-apl: NOTRUN -> [WARN][28] ([i915#2658]) [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-apl2/igt@gem_pread@exhaustion.html * igt@gem_render_copy@y-tiled-ccs-to-y-tiled-mc-ccs: - shard-iclb: NOTRUN -> [SKIP][29] ([i915#768]) +1 similar issue [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb6/igt@gem_render_copy@y-tiled-ccs-to-y-tiled-mc-ccs.html * igt@gem_softpin@evict-snoop: - shard-tglb: NOTRUN -> [SKIP][30] ([fdo#109312]) [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb8/igt@gem_softpin@evict-snoop.html * igt@gem_sync@basic-all: - shard-glk: [PASS][31] -> [DMESG-WARN][32] ([i915#118] / [i915#95]) +1 similar issue [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk2/igt@gem_sync@basic-all.html [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk7/igt@gem_sync@basic-all.html * igt@gem_userptr_blits@coherency-sync: - shard-tglb: NOTRUN -> [SKIP][33] ([fdo#110542]) [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb5/igt@gem_userptr_blits@coherency-sync.html * igt@gem_userptr_blits@input-checking: - shard-kbl: NOTRUN -> [DMESG-WARN][34] ([i915#3002]) [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl2/igt@gem_userptr_blits@input-checking.html * igt@gem_userptr_blits@process-exit-mmap-busy@uc: - shard-tglb: NOTRUN -> [SKIP][35] ([i915#1699]) +3 similar issues [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb1/igt@gem_userptr_blits@process-exit-mmap-busy@uc.html * igt@gem_userptr_blits@process-exit-mmap@wb: - shard-apl: NOTRUN -> [SKIP][36] ([fdo#109271] / [i915#1699]) +3 similar issues [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-apl8/igt@gem_userptr_blits@process-exit-mmap@wb.html * igt@gem_workarounds@suspend-resume-fd: - shard-kbl: [PASS][37] -> [DMESG-WARN][38] ([i915#180]) +1 similar issue [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl6/igt@gem_workarounds@suspend-resume-fd.html [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl4/igt@gem_workarounds@suspend-resume-fd.html * igt@gen3_mixed_blits: - shard-iclb: NOTRUN -> [SKIP][39] ([fdo#109289]) +3 similar issues [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb3/igt@gen3_mixed_blits.html - shard-tglb: NOTRUN -> [SKIP][40] ([fdo#109289]) +2 similar issues [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb3/igt@gen3_mixed_blits.html * igt@gen9_exec_parse@allowed-all: - shard-iclb: NOTRUN -> [SKIP][41] ([fdo#112306]) [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb2/igt@gen9_exec_parse@allowed-all.html * igt@gen9_exec_parse@cmd-crossing-page: - shard-tglb: NOTRUN -> [SKIP][42] ([fdo#112306]) [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb1/igt@gen9_exec_parse@cmd-crossing-page.html * igt@i915_pm_rc6_residency@media-rc6-accuracy: - shard-tglb: NOTRUN -> [SKIP][43] ([fdo#109289] / [fdo#111719]) [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb7/igt@i915_pm_rc6_residency@media-rc6-accuracy.html * igt@i915_pm_rc6_residency@rc6-fence: - shard-iclb: NOTRUN -> [WARN][44] ([i915#1804] / [i915#2684]) [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb6/igt@i915_pm_rc6_residency@rc6-fence.html * igt@i915_pm_rc6_residency@rc6-idle: - shard-tglb: NOTRUN -> [WARN][45] ([i915#2681] / [i915#2684]) [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb6/igt@i915_pm_rc6_residency@rc6-idle.html * igt@i915_pm_rpm@gem-execbuf-stress-pc8: - shard-iclb: NOTRUN -> [SKIP][46] ([fdo#109293] / [fdo#109506]) [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb5/igt@i915_pm_rpm@gem-execbuf-stress-pc8.html - shard-tglb: NOTRUN -> [SKIP][47] ([fdo#109506] / [i915#2411]) [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb7/igt@i915_pm_rpm@gem-execbuf-stress-pc8.html * igt@i915_selftest@live@blt: - shard-snb: [PASS][48] -> [DMESG-FAIL][49] ([i915#1409]) [48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-snb4/igt@i915_selftest@live@blt.html [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-snb4/igt@i915_selftest@live@blt.html * igt@kms_big_fb@x-tiled-16bpp-rotate-270: - shard-iclb: NOTRUN -> [SKIP][50] ([fdo#110725] / [fdo#111614]) +3 similar issues [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb7/igt@kms_big_fb@x-tiled-16bpp-rotate-270.html - shard-tglb: NOTRUN -> [SKIP][51] ([fdo#111614]) +3 similar issues [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb7/igt@kms_big_fb@x-tiled-16bpp-rotate-270.html * igt@kms_big_fb@yf-tiled-64bpp-rotate-180: - shard-tglb: NOTRUN -> [SKIP][52] ([fdo#111615]) +2 similar issues [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb3/igt@kms_big_fb@yf-tiled-64bpp-rotate-180.html - shard-iclb: NOTRUN -> [SKIP][53] ([fdo#110723]) [53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb2/igt@kms_big_fb@yf-tiled-64bpp-rotate-180.html * igt@kms_big_joiner@basic: - shard-tglb: NOTRUN -> [SKIP][54] ([i915#2705]) [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb6/igt@kms_big_joiner@basic.html - shard-iclb: NOTRUN -> [SKIP][55] ([i915#2705]) +1 similar issue [55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb8/igt@kms_big_joiner@basic.html * igt@kms_chamelium@dp-frame-dump: - shard-iclb: NOTRUN -> [SKIP][56] ([fdo#109284] / [fdo#111827]) +15 similar issues [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb3/igt@kms_chamelium@dp-frame-dump.html * igt@kms_chamelium@vga-hpd: - shard-apl: NOTRUN -> [SKIP][57] ([fdo#109271] / [fdo#111827]) +13 similar issues [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-apl3/igt@kms_chamelium@vga-hpd.html * igt@kms_color@pipe-a-degamma: - shard-tglb: NOTRUN -> [FAIL][58] ([i915#1149]) [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb1/igt@kms_color@pipe-a-degamma.html * igt@kms_color@pipe-d-ctm-0-75: - shard-tglb: NOTRUN -> [FAIL][59] ([i915#1149] / [i915#315]) [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb1/igt@kms_color@pipe-d-ctm-0-75.html * igt@kms_color@pipe-d-ctm-negative: - shard-iclb: NOTRUN -> [SKIP][60] ([fdo#109278] / [i915#1149]) +1 similar issue [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb5/igt@kms_color@pipe-d-ctm-negative.html * igt@kms_color_chamelium@pipe-a-ctm-negative: - shard-kbl: NOTRUN -> [SKIP][61] ([fdo#109271] / [fdo#111827]) +5 similar issues [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl6/igt@kms_color_chamelium@pipe-a-ctm-negative.html * igt@kms_color_chamelium@pipe-b-ctm-0-5: - shard-glk: NOTRUN -> [SKIP][62] ([fdo#109271] / [fdo#111827]) +8 similar issues [62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk7/igt@kms_color_chamelium@pipe-b-ctm-0-5.html * igt@kms_color_chamelium@pipe-d-ctm-red-to-blue: - shard-tglb: NOTRUN -> [SKIP][63] ([fdo#109284] / [fdo#111827]) +11 similar issues [63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb5/igt@kms_color_chamelium@pipe-d-ctm-red-to-blue.html * igt@kms_color_chamelium@pipe-d-gamma: - shard-hsw: NOTRUN -> [SKIP][64] ([fdo#109271] / [fdo#111827]) +4 similar issues [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-hsw1/igt@kms_color_chamelium@pipe-d-gamma.html - shard-iclb: NOTRUN -> [SKIP][65] ([fdo#109278] / [fdo#109284] / [fdo#111827]) +1 similar issue [65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb1/igt@kms_color_chamelium@pipe-d-gamma.html * igt@kms_color_chamelium@pipe-invalid-gamma-lut-sizes: - shard-snb: NOTRUN -> [SKIP][66] ([fdo#109271] / [fdo#111827]) +2 similar issues [66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-snb5/igt@kms_color_chamelium@pipe-invalid-gamma-lut-sizes.html * igt@kms_content_protection@legacy: - shard-iclb: NOTRUN -> [SKIP][67] ([fdo#109300] / [fdo#111066]) +1 similar issue [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb5/igt@kms_content_protection@legacy.html - shard-kbl: NOTRUN -> [TIMEOUT][68] ([i915#1319]) +1 similar issue [68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl6/igt@kms_content_protection@legacy.html - shard-tglb: NOTRUN -> [SKIP][69] ([fdo#111828]) [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb2/igt@kms_content_protection@legacy.html - shard-apl: NOTRUN -> [TIMEOUT][70] ([i915#1319]) [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-apl2/igt@kms_content_protection@legacy.html * igt@kms_cursor_crc@pipe-c-cursor-512x512-random: - shard-iclb: NOTRUN -> [SKIP][71] ([fdo#109278] / [fdo#109279]) [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb4/igt@kms_cursor_crc@pipe-c-cursor-512x512-random.html * igt@kms_cursor_crc@pipe-d-cursor-256x256-onscreen: - shard-hsw: NOTRUN -> [SKIP][72] ([fdo#109271] / [i915#533]) +7 similar issues [72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-hsw6/igt@kms_cursor_crc@pipe-d-cursor-256x256-onscreen.html * igt@kms_cursor_crc@pipe-d-cursor-512x512-sliding: - shard-tglb: NOTRUN -> [SKIP][73] ([fdo#109279]) +2 similar issues [73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb7/igt@kms_cursor_crc@pipe-d-cursor-512x512-sliding.html * igt@kms_cursor_edge_walk@pipe-d-256x256-left-edge: - shard-iclb: NOTRUN -> [SKIP][74] ([fdo#109278]) +19 similar issues [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb4/igt@kms_cursor_edge_walk@pipe-d-256x256-left-edge.html * igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions-varying-size: - shard-iclb: NOTRUN -> [SKIP][75] ([fdo#109274] / [fdo#109278]) +4 similar issues [75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb6/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions-varying-size.html * igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size: - shard-apl: [PASS][76] -> [FAIL][77] ([i915#2346] / [i915#533]) [76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-apl7/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html [77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-apl4/igt@kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size.html * igt@kms_cursor_legacy@pipe-d-torture-bo: - shard-kbl: NOTRUN -> [SKIP][78] ([fdo#109271] / [i915#533]) +1 similar issue [78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl7/igt@kms_cursor_legacy@pipe-d-torture-bo.html * igt@kms_dp_tiled_display@basic-test-pattern: - shard-iclb: NOTRUN -> [SKIP][79] ([i915#426]) [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb7/igt@kms_dp_tiled_display@basic-test-pattern.html - shard-tglb: NOTRUN -> [SKIP][80] ([i915#426]) [80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb8/igt@kms_dp_tiled_display@basic-test-pattern.html * igt@kms_flip@2x-flip-vs-dpms: - shard-iclb: NOTRUN -> [SKIP][81] ([fdo#109274]) +3 similar issues [81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb5/igt@kms_flip@2x-flip-vs-dpms.html * igt@kms_flip@flip-vs-expired-vblank@a-edp1: - shard-tglb: NOTRUN -> [FAIL][82] ([i915#2598]) [82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb8/igt@kms_flip@flip-vs-expired-vblank@a-edp1.html * igt@kms_flip@flip-vs-suspend-interruptible@a-dp1: - shard-apl: [PASS][83] -> [DMESG-WARN][84] ([i915#180]) +1 similar issue [83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-apl1/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html [84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-apl8/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html * igt@kms_flip@flip-vs-suspend@c-dp1: - shard-kbl: NOTRUN -> [DMESG-WARN][85] ([i915#180]) +3 similar issues [85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl4/igt@kms_flip@flip-vs-suspend@c-dp1.html * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-onoff: - shard-apl: [PASS][86] -> [FAIL][87] ([i915#49]) [86]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-apl2/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-onoff.html [87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-apl3/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-onoff.html - shard-kbl: [PASS][88] -> [FAIL][89] ([i915#49]) [88]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl1/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-onoff.html [89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl4/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-onoff.html - shard-glk: [PASS][90] -> [FAIL][91] ([i915#49]) +1 similar issue [90]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk2/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-onoff.html [91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk2/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-cur-indfb-onoff.html * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-mmap-cpu: - shard-snb: NOTRUN -> [FAIL][92] ([i915#2546]) [92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-snb5/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-shrfb-draw-mmap-cpu.html * igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-indfb-draw-mmap-cpu: - shard-glk: NOTRUN -> [SKIP][93] ([fdo#109271]) +73 similar issues [93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk2/igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-indfb-draw-mmap-cpu.html * igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-cur-indfb-draw-pwrite: - shard-hsw: NOTRUN -> [SKIP][94] ([fdo#109271]) +42 similar issues [94]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-hsw8/igt@kms_frontbuffer_tracking@fbcpsr-2p-primscrn-cur-indfb-draw-pwrite.html * igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-pri-indfb-draw-render: - shard-kbl: NOTRUN -> [SKIP][95] ([fdo#109271]) +70 similar issues [95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl1/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-pri-indfb-draw-render.html * igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-spr-indfb-draw-pwrite: - shard-iclb: NOTRUN -> [SKIP][96] ([fdo#109280]) +43 similar issues [96]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb6/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-spr-indfb-draw-pwrite.html * igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-pwrite: - shard-tglb: NOTRUN -> [SKIP][97] ([fdo#111825]) +35 similar issues [97]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb5/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-pwrite.html * igt@kms_pipe_crc_basic@disable-crc-after-crtc-pipe-d: - shard-apl: NOTRUN -> [SKIP][98] ([fdo#109271] / [i915#533]) [98]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-apl1/igt@kms_pipe_crc_basic@disable-crc-after-crtc-pipe-d.html * igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb: - shard-kbl: NOTRUN -> [FAIL][99] ([fdo#108145] / [i915#265]) [99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl7/igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb.html - shard-apl: NOTRUN -> [FAIL][100] ([fdo#108145] / [i915#265]) [100]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-apl2/igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb.html - shard-glk: NOTRUN -> [FAIL][101] ([fdo#108145] / [i915#265]) [101]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk9/igt@kms_plane_alpha_blend@pipe-c-alpha-opaque-fb.html * igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-4: - shard-iclb: NOTRUN -> [SKIP][102] ([i915#658]) +2 similar issues [102]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb1/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-4.html * igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5: - shard-glk: NOTRUN -> [SKIP][103] ([fdo#109271] / [i915#658]) +1 similar issue [103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk4/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5.html - shard-tglb: NOTRUN -> [SKIP][104] ([i915#2920]) +2 similar issues [104]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb3/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-5.html * igt@kms_psr2_sf@plane-move-sf-dmg-area-2: - shard-apl: NOTRUN -> [SKIP][105] ([fdo#109271] / [i915#658]) +1 similar issue [105]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-apl8/igt@kms_psr2_sf@plane-move-sf-dmg-area-2.html * igt@kms_psr@psr2_cursor_render: - shard-iclb: NOTRUN -> [SKIP][106] ([fdo#109441]) +3 similar issues [106]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb1/igt@kms_psr@psr2_cursor_render.html * igt@kms_psr@psr2_suspend: - shard-iclb: [PASS][107] -> [SKIP][108] ([fdo#109441]) +1 similar issue [107]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-iclb2/igt@kms_psr@psr2_suspend.html [108]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb8/igt@kms_psr@psr2_suspend.html * igt@kms_psr@sprite_blt: - shard-hsw: NOTRUN -> [SKIP][109] ([fdo#109271] / [i915#1072]) +2 similar issues [109]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-hsw8/igt@kms_psr@sprite_blt.html * igt@kms_sysfs_edid_timing: - shard-kbl: NOTRUN -> [FAIL][110] ([IGT#2]) [110]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl6/igt@kms_sysfs_edid_timing.html * igt@kms_vblank@pipe-d-wait-forked-hang: - shard-apl: NOTRUN -> [SKIP][111] ([fdo#109271]) +102 similar issues [111]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-apl7/igt@kms_vblank@pipe-d-wait-forked-hang.html * igt@kms_writeback@writeback-invalid-parameters: - shard-tglb: NOTRUN -> [SKIP][112] ([i915#2437]) [112]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb3/igt@kms_writeback@writeback-invalid-parameters.html - shard-iclb: NOTRUN -> [SKIP][113] ([i915#2437]) [113]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb3/igt@kms_writeback@writeback-invalid-parameters.html * igt@nouveau_crc@pipe-a-source-rg: - shard-iclb: NOTRUN -> [SKIP][114] ([i915#2530]) +2 similar issues [114]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb7/igt@nouveau_crc@pipe-a-source-rg.html * igt@nouveau_crc@pipe-b-source-outp-complete: - shard-tglb: NOTRUN -> [SKIP][115] ([i915#2530]) [115]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb1/igt@nouveau_crc@pipe-b-source-outp-complete.html * igt@perf@polling-parameterized: - shard-glk: [PASS][116] -> [FAIL][117] ([i915#1542]) [116]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk3/igt@perf@polling-parameterized.html [117]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk1/igt@perf@polling-parameterized.html * igt@prime_nv_pcopy@test3_1: - shard-tglb: NOTRUN -> [SKIP][118] ([fdo#109291]) +3 similar issues [118]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb3/igt@prime_nv_pcopy@test3_1.html * igt@prime_nv_pcopy@test3_4: - shard-iclb: NOTRUN -> [SKIP][119] ([fdo#109291]) +4 similar issues [119]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb7/igt@prime_nv_pcopy@test3_4.html * igt@prime_vgem@coherency-blt: - shard-glk: [PASS][120] -> [INCOMPLETE][121] ([i915#2944]) [120]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk1/igt@prime_vgem@coherency-blt.html [121]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk9/igt@prime_vgem@coherency-blt.html * igt@prime_vgem@coherency-gtt: - shard-tglb: NOTRUN -> [SKIP][122] ([fdo#111656]) [122]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb3/igt@prime_vgem@coherency-gtt.html #### Possible fixes #### * igt@gem_exec_balancer@hang: - shard-iclb: [INCOMPLETE][123] -> [PASS][124] [123]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-iclb1/igt@gem_exec_balancer@hang.html [124]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb8/igt@gem_exec_balancer@hang.html * igt@gem_exec_fair@basic-none@vcs0: - shard-kbl: [FAIL][125] ([i915#2842]) -> [PASS][126] [125]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-kbl2/igt@gem_exec_fair@basic-none@vcs0.html [126]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-kbl7/igt@gem_exec_fair@basic-none@vcs0.html * igt@gem_exec_reloc@basic-many-active@rcs0: - shard-glk: [FAIL][127] ([i915#2389]) -> [PASS][128] [127]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk2/igt@gem_exec_reloc@basic-many-active@rcs0.html [128]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk6/igt@gem_exec_reloc@basic-many-active@rcs0.html * igt@gem_exec_schedule@u-fairslice@rcs0: - shard-iclb: [DMESG-WARN][129] ([i915#2803]) -> [PASS][130] [129]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-iclb1/igt@gem_exec_schedule@u-fairslice@rcs0.html [130]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb7/igt@gem_exec_schedule@u-fairslice@rcs0.html * igt@gem_exec_schedule@u-fairslice@vcs0: - shard-glk: [DMESG-WARN][131] ([i915#1610] / [i915#2803]) -> [PASS][132] [131]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk8/igt@gem_exec_schedule@u-fairslice@vcs0.html [132]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk6/igt@gem_exec_schedule@u-fairslice@vcs0.html * igt@gem_exec_whisper@basic-queues-forked: - shard-iclb: [INCOMPLETE][133] ([i915#1895] / [i915#2405]) -> [PASS][134] [133]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-iclb4/igt@gem_exec_whisper@basic-queues-forked.html [134]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-iclb1/igt@gem_exec_whisper@basic-queues-forked.html - shard-glk: [DMESG-WARN][135] ([i915#118] / [i915#95]) -> [PASS][136] [135]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-glk4/igt@gem_exec_whisper@basic-queues-forked.html [136]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-glk7/igt@gem_exec_whisper@basic-queues-forked.html * igt@gem_request_retire@retire-vma-not-inactive: - shard-tglb: [DMESG-WARN][137] -> [PASS][138] [137]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-tglb5/igt@gem_request_retire@retire-vma-not-inactive.html [138]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/shard-tglb5/igt@gem_request_retire@retire-vma-not-inactive.html * igt@gem_vm_create@destroy-race: - shard-tglb: [TIMEOUT][139] ([i915#2795]) -> [PASS][140] [139]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_9705/shard-tglb7/igt@gem_vm_create@destroy-race.html [140]: https://intel-g == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5450/index.html [-- Attachment #1.2: Type: text/html, Size: 33738 bytes --] [-- Attachment #2: Type: text/plain, Size: 154 bytes --] _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] [PATCH i-g-t] meson: Turn on -Werror @ 2021-01-30 4:17 Ashutosh Dixit 0 siblings, 0 replies; 13+ messages in thread From: Ashutosh Dixit @ 2021-01-30 4:17 UTC (permalink / raw) To: igt-dev; +Cc: petri.latvala We can choose which compile warnings to enable, but once they are enabled treat all warnings as error. This enforces stricter checks against compile warnings creeping in. Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com> --- meson.build | 28 +++++++++++++++------------- 1 file changed, 15 insertions(+), 13 deletions(-) diff --git a/meson.build b/meson.build index f4ee14f14a..3f3693f99d 100644 --- a/meson.build +++ b/meson.build @@ -51,19 +51,21 @@ cc_args = [ '-Wno-unused-parameter', '-Wno-unused-result', - '-Werror=address', - '-Werror=array-bounds', - '-Werror=implicit', - '-Werror=init-self', - '-Werror=int-to-pointer-cast', - '-Werror=main', - '-Werror=missing-braces', - '-Werror=nonnull', - '-Werror=pointer-to-int-cast', - '-Werror=return-type', - '-Werror=sequence-point', - '-Werror=trigraphs', - '-Werror=write-strings', + '-Waddress', + '-Warray-bounds', + '-Wimplicit', + '-Winit-self', + '-Wint-to-pointer-cast', + '-Wmain', + '-Wmissing-braces', + '-Wnonnull', + '-Wpointer-to-int-cast', + '-Wreturn-type', + '-Wsequence-point', + '-Wtrigraphs', + '-Wwrite-strings', + + '-Werror', # Disable the memory allocating builtins as they may cause unexpected behavior # with our framework. They *may* get optimized out in favor of a register or # stack variable, making them effectively local. Local variables do not play -- 2.29.2 _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply related [flat|nested] 13+ messages in thread
end of thread, other threads:[~2021-01-30 14:26 UTC | newest] Thread overview: 13+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2021-01-30 6:44 [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Ashutosh Dixit 2021-01-30 7:23 ` [igt-dev] ✗ GitLab.Pipeline: warning for meson: Turn on -Werror (rev2) Patchwork 2021-01-30 7:45 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork 2021-01-30 9:19 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork 2021-01-30 12:11 ` [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Chris Wilson 2021-01-30 12:22 ` [igt-dev] ✗ Fi.CI.BUILD: failure for meson: Turn on -Werror (rev3) Patchwork 2021-01-30 12:34 ` [igt-dev] ✗ GitLab.Pipeline: warning " Patchwork 2021-01-30 12:46 ` =?unknown-8bit?b?4pyX?= " Chris Wilson 2021-01-30 12:42 ` [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Chris Wilson 2021-01-30 13:05 ` [igt-dev] ✗ GitLab.Pipeline: warning for meson: Turn on -Werror (rev4) Patchwork 2021-01-30 13:22 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork 2021-01-30 14:26 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork -- strict thread matches above, loose matches on Subject: below -- 2021-01-30 4:17 [igt-dev] [PATCH i-g-t] meson: Turn on -Werror Ashutosh Dixit
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox