[RESEND PATCH i-g-t v2 0/1] tests/i915_module_load: Use new name of fault injection module parameter

[RESEND PATCH i-g-t v2 0/1] tests/i915_module_load: Use new name of fault injection module parameter

[Janusz Krzysztofik]

Name of the i915 module parameter providing fault injection function is
changing for consistency with a new convention of naming i915 driver
internal functions called from the driver PCI .probe entry point.  Adjust
the test to use the new name.

v2: * add R-b (thanks Chris),
    * add cover letter to correctly support joint testing with a
      corresponding kernel driver side patch.

Janusz Krzysztofik (1):
  tests/i915_module_load: Use new name of fault injection module
    parameter

 tests/i915/i915_module_load.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.21.0

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

[Intel-gfx] [RESEND PATCH i-g-t v2 0/1] tests/i915_module_load: Use new name of fault injection module parameter

[Janusz Krzysztofik]

Name of the i915 module parameter providing fault injection function is
changing for consistency with a new convention of naming i915 driver
internal functions called from the driver PCI .probe entry point.  Adjust
the test to use the new name.

v2: * add R-b (thanks Chris),
    * add cover letter to correctly support joint testing with a
      corresponding kernel driver side patch.

Janusz Krzysztofik (1):
  tests/i915_module_load: Use new name of fault injection module
    parameter

 tests/i915/i915_module_load.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.21.0

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

[RESEND PATCH i-g-t v2 1/1] tests/i915_module_load: Use new name of fault injection module parameter

[Janusz Krzysztofik]

Name of the i915 module parameter providing fault injection function is
changing for consistency with a new convention of naming i915 driver
internal functions called from the driver PCI .probe entry point.  Adjust
the test to use the new name.

Suggested-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
Cc: Michał Wajdeczko <michal.wajdeczko@intel.com>
Cc: Michał Winiarski <michal.winiarski@intel.com>
Cc: Piotr Piórkowski <piotr.piorkowski@intel.com>
Cc: Tomasz Lis <tomasz.lis@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 tests/i915/i915_module_load.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
index 7fe83520..a620ab41 100644
--- a/tests/i915/i915_module_load.c
+++ b/tests/i915/i915_module_load.c
@@ -356,7 +356,7 @@ igt_main
 
 		igt_i915_driver_unload();
 
-		while (inject_fault("i915", "inject_load_failure", ++i) == 0)
+		while (inject_fault("i915", "inject_probe_failure", ++i) == 0)
 			;
 
 		/* We expect to hit at least one fault! */
-- 
2.21.0

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

[Intel-gfx] [RESEND PATCH i-g-t v2 1/1] tests/i915_module_load: Use new name of fault injection module parameter

[Janusz Krzysztofik]

Name of the i915 module parameter providing fault injection function is
changing for consistency with a new convention of naming i915 driver
internal functions called from the driver PCI .probe entry point.  Adjust
the test to use the new name.

Suggested-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
Cc: Michał Wajdeczko <michal.wajdeczko@intel.com>
Cc: Michał Winiarski <michal.winiarski@intel.com>
Cc: Piotr Piórkowski <piotr.piorkowski@intel.com>
Cc: Tomasz Lis <tomasz.lis@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 tests/i915/i915_module_load.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
index 7fe83520..a620ab41 100644
--- a/tests/i915/i915_module_load.c
+++ b/tests/i915/i915_module_load.c
@@ -356,7 +356,7 @@ igt_main
 
 		igt_i915_driver_unload();
 
-		while (inject_fault("i915", "inject_load_failure", ++i) == 0)
+		while (inject_fault("i915", "inject_probe_failure", ++i) == 0)
 			;
 
 		/* We expect to hit at least one fault! */
-- 
2.21.0

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

[igt-dev] ✗ Fi.CI.BAT: failure for tests/i915_module_load: Use new name of fault injection module parameter (rev3)

[Patchwork]

== Series Details ==

Series: tests/i915_module_load: Use new name of fault injection module parameter (rev3)
URL   : https://patchwork.freedesktop.org/series/67443/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_7209 -> IGTPW_3627
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_3627 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_3627, 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_3627/index.html

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in IGTPW_3627:

### IGT changes ###

#### Possible regressions ####

  * igt@i915_module_load@reload-with-fault-injection:
    - fi-kbl-8809g:       [PASS][1] -> [FAIL][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-kbl-8809g/igt@i915_module_load@reload-with-fault-injection.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-kbl-8809g/igt@i915_module_load@reload-with-fault-injection.html
    - fi-byt-j1900:       [PASS][3] -> [FAIL][4]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-byt-j1900/igt@i915_module_load@reload-with-fault-injection.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-byt-j1900/igt@i915_module_load@reload-with-fault-injection.html
    - fi-cfl-guc:         [PASS][5] -> [FAIL][6]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-cfl-guc/igt@i915_module_load@reload-with-fault-injection.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-cfl-guc/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-guc:         [PASS][7] -> [FAIL][8]
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-skl-guc/igt@i915_module_load@reload-with-fault-injection.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-skl-guc/igt@i915_module_load@reload-with-fault-injection.html
    - fi-cfl-8700k:       [PASS][9] -> [FAIL][10]
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-cfl-8700k/igt@i915_module_load@reload-with-fault-injection.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-cfl-8700k/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-6700k2:      [PASS][11] -> [FAIL][12]
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-skl-6700k2/igt@i915_module_load@reload-with-fault-injection.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-skl-6700k2/igt@i915_module_load@reload-with-fault-injection.html
    - fi-elk-e7500:       [PASS][13] -> [FAIL][14]
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-elk-e7500/igt@i915_module_load@reload-with-fault-injection.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-elk-e7500/igt@i915_module_load@reload-with-fault-injection.html
    - fi-blb-e6850:       [PASS][15] -> [FAIL][16]
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-blb-e6850/igt@i915_module_load@reload-with-fault-injection.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-blb-e6850/igt@i915_module_load@reload-with-fault-injection.html
    - fi-apl-guc:         [PASS][17] -> [FAIL][18]
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-apl-guc/igt@i915_module_load@reload-with-fault-injection.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-apl-guc/igt@i915_module_load@reload-with-fault-injection.html
    - fi-bsw-kefka:       [PASS][19] -> [FAIL][20]
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-bsw-kefka/igt@i915_module_load@reload-with-fault-injection.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-bsw-kefka/igt@i915_module_load@reload-with-fault-injection.html
    - fi-bwr-2160:        [PASS][21] -> [FAIL][22]
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-bwr-2160/igt@i915_module_load@reload-with-fault-injection.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-bwr-2160/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-6770hq:      [PASS][23] -> [FAIL][24]
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-skl-6770hq/igt@i915_module_load@reload-with-fault-injection.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-skl-6770hq/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-6600u:       [PASS][25] -> [FAIL][26]
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-skl-6600u/igt@i915_module_load@reload-with-fault-injection.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-skl-6600u/igt@i915_module_load@reload-with-fault-injection.html
    - fi-pnv-d510:        NOTRUN -> [FAIL][27]
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-pnv-d510/igt@i915_module_load@reload-with-fault-injection.html
    - fi-kbl-guc:         [PASS][28] -> [FAIL][29]
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-kbl-guc/igt@i915_module_load@reload-with-fault-injection.html
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-kbl-guc/igt@i915_module_load@reload-with-fault-injection.html
    - fi-cfl-8109u:       [PASS][30] -> [FAIL][31]
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-cfl-8109u/igt@i915_module_load@reload-with-fault-injection.html
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-cfl-8109u/igt@i915_module_load@reload-with-fault-injection.html
    - fi-kbl-soraka:      NOTRUN -> [FAIL][32]
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-kbl-soraka/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-lmem:        [PASS][33] -> [FAIL][34]
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-skl-lmem/igt@i915_module_load@reload-with-fault-injection.html
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-skl-lmem/igt@i915_module_load@reload-with-fault-injection.html
    - fi-kbl-r:           [PASS][35] -> [FAIL][36]
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-kbl-r/igt@i915_module_load@reload-with-fault-injection.html
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-kbl-r/igt@i915_module_load@reload-with-fault-injection.html
    - fi-gdg-551:         NOTRUN -> [FAIL][37]
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-gdg-551/igt@i915_module_load@reload-with-fault-injection.html
    - fi-byt-n2820:       [PASS][38] -> [FAIL][39]
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-byt-n2820/igt@i915_module_load@reload-with-fault-injection.html
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-byt-n2820/igt@i915_module_load@reload-with-fault-injection.html
    - fi-cml-u2:          [PASS][40] -> [FAIL][41]
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-cml-u2/igt@i915_module_load@reload-with-fault-injection.html
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-cml-u2/igt@i915_module_load@reload-with-fault-injection.html
    - fi-bxt-dsi:         [PASS][42] -> [FAIL][43]
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-bxt-dsi/igt@i915_module_load@reload-with-fault-injection.html
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-bxt-dsi/igt@i915_module_load@reload-with-fault-injection.html
    - fi-hsw-4770:        [PASS][44] -> [FAIL][45]
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-hsw-4770/igt@i915_module_load@reload-with-fault-injection.html
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-hsw-4770/igt@i915_module_load@reload-with-fault-injection.html
    - fi-snb-2600:        [PASS][46] -> [FAIL][47]
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-snb-2600/igt@i915_module_load@reload-with-fault-injection.html
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-snb-2600/igt@i915_module_load@reload-with-fault-injection.html
    - fi-whl-u:           [PASS][48] -> [FAIL][49]
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-whl-u/igt@i915_module_load@reload-with-fault-injection.html
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-whl-u/igt@i915_module_load@reload-with-fault-injection.html
    - fi-icl-u3:          [PASS][50] -> [FAIL][51]
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-icl-u3/igt@i915_module_load@reload-with-fault-injection.html
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-icl-u3/igt@i915_module_load@reload-with-fault-injection.html
    - fi-ilk-650:         [PASS][52] -> [FAIL][53]
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-ilk-650/igt@i915_module_load@reload-with-fault-injection.html
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-ilk-650/igt@i915_module_load@reload-with-fault-injection.html
    - fi-bsw-n3050:       [PASS][54] -> [FAIL][55]
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-bsw-n3050/igt@i915_module_load@reload-with-fault-injection.html
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-bsw-n3050/igt@i915_module_load@reload-with-fault-injection.html
    - fi-ivb-3770:        [PASS][56] -> [FAIL][57]
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-ivb-3770/igt@i915_module_load@reload-with-fault-injection.html
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-ivb-3770/igt@i915_module_load@reload-with-fault-injection.html
    - fi-hsw-peppy:       [PASS][58] -> [FAIL][59]
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-hsw-peppy/igt@i915_module_load@reload-with-fault-injection.html
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-hsw-peppy/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-iommu:       [PASS][60] -> [FAIL][61]
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-skl-iommu/igt@i915_module_load@reload-with-fault-injection.html
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-skl-iommu/igt@i915_module_load@reload-with-fault-injection.html
    - fi-glk-dsi:         [PASS][62] -> [FAIL][63]
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-glk-dsi/igt@i915_module_load@reload-with-fault-injection.html
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-glk-dsi/igt@i915_module_load@reload-with-fault-injection.html
    - fi-snb-2520m:       [PASS][64] -> [FAIL][65]
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-snb-2520m/igt@i915_module_load@reload-with-fault-injection.html
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-snb-2520m/igt@i915_module_load@reload-with-fault-injection.html
    - fi-kbl-x1275:       [PASS][66] -> [FAIL][67]
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-kbl-x1275/igt@i915_module_load@reload-with-fault-injection.html
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-kbl-x1275/igt@i915_module_load@reload-with-fault-injection.html
    - fi-bdw-5557u:       [PASS][68] -> [FAIL][69]
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-bdw-5557u/igt@i915_module_load@reload-with-fault-injection.html
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-bdw-5557u/igt@i915_module_load@reload-with-fault-injection.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * igt@i915_module_load@reload-with-fault-injection:
    - {fi-icl-guc}:       NOTRUN -> [FAIL][70]
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-icl-guc/igt@i915_module_load@reload-with-fault-injection.html
    - {fi-tgl-u}:         [PASS][71] -> [FAIL][72]
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-tgl-u/igt@i915_module_load@reload-with-fault-injection.html
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-tgl-u/igt@i915_module_load@reload-with-fault-injection.html
    - {fi-icl-dsi}:       [PASS][73] -> [FAIL][74]
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-icl-dsi/igt@i915_module_load@reload-with-fault-injection.html
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-icl-dsi/igt@i915_module_load@reload-with-fault-injection.html
    - {fi-icl-u4}:        [PASS][75] -> [FAIL][76]
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-icl-u4/igt@i915_module_load@reload-with-fault-injection.html
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-icl-u4/igt@i915_module_load@reload-with-fault-injection.html

  
Known issues
------------

  Here are the changes found in IGTPW_3627 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_mmap_gtt@basic-write-read:
    - fi-icl-u3:          [PASS][77] -> [DMESG-WARN][78] ([fdo#107724]) +1 similar issue
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-icl-u3/igt@gem_mmap_gtt@basic-write-read.html
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-icl-u3/igt@gem_mmap_gtt@basic-write-read.html

  * igt@i915_selftest@live_gem_contexts:
    - fi-bsw-n3050:       [PASS][79] -> [INCOMPLETE][80] ([fdo# 111542])
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-bsw-n3050/igt@i915_selftest@live_gem_contexts.html
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-bsw-n3050/igt@i915_selftest@live_gem_contexts.html

  
#### Possible fixes ####

  * igt@gem_flink_basic@basic:
    - fi-icl-u3:          [DMESG-WARN][81] ([fdo#107724] / [fdo#112052 ]) -> [PASS][82]
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-icl-u3/igt@gem_flink_basic@basic.html
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-icl-u3/igt@gem_flink_basic@basic.html

  
#### Warnings ####

  * igt@i915_module_load@reload-with-fault-injection:
    - fi-icl-u2:          [DMESG-WARN][83] ([fdo#110595]) -> [DMESG-FAIL][84] ([fdo#110595])
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7209/fi-icl-u2/igt@i915_module_load@reload-with-fault-injection.html
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/fi-icl-u2/igt@i915_module_load@reload-with-fault-injection.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo# 111542]: https://bugs.freedesktop.org/show_bug.cgi?id= 111542
  [fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
  [fdo#110595]: https://bugs.freedesktop.org/show_bug.cgi?id=110595
  [fdo#112052 ]: https://bugs.freedesktop.org/show_bug.cgi?id=112052 


Participating hosts (47 -> 43)
------------------------------

  Additional (3): fi-kbl-soraka fi-gdg-551 fi-pnv-d510 
  Missing    (7): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-icl-y fi-byt-clapper fi-bdw-samus 


Build changes
-------------

  * CI: CI-20190529 -> None
  * IGT: IGT_5249 -> IGTPW_3627

  CI-20190529: 20190529
  CI_DRM_7209: 11dd694075dfa8fbf60b2aa67d547652c2541764 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_3627: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/index.html
  IGT_5249: aee019cec9d7f3002371f6ddf102cb67d9976b71 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3627/index.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [RESEND PATCH i-g-t v2 1/1] tests/i915_module_load: Use new name of fault injection module parameter

[Chris Wilson]

Quoting Janusz Krzysztofik (2019-10-29 10:17:51)
> Name of the i915 module parameter providing fault injection function is
> changing for consistency with a new convention of naming i915 driver
> internal functions called from the driver PCI .probe entry point.  Adjust
> the test to use the new name.
> 
> Suggested-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
> Cc: Michał Wajdeczko <michal.wajdeczko@intel.com>
> Cc: Michał Winiarski <michal.winiarski@intel.com>
> Cc: Piotr Piórkowski <piotr.piorkowski@intel.com>
> Cc: Tomasz Lis <tomasz.lis@intel.com>
> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  tests/i915/i915_module_load.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
> index 7fe83520..a620ab41 100644
> --- a/tests/i915/i915_module_load.c
> +++ b/tests/i915/i915_module_load.c
> @@ -356,7 +356,7 @@ igt_main
>  
>                 igt_i915_driver_unload();
>  
> -               while (inject_fault("i915", "inject_load_failure", ++i) == 0)
> +               while (inject_fault("i915", "inject_probe_failure", ++i) == 0)

Hmm, I could do with a transition period so that we don't lose reload
tests on linus/stable.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [RESEND PATCH i-g-t v2 1/1] tests/i915_module_load: Use new name of fault injection module parameter

[Chris Wilson]

Quoting Janusz Krzysztofik (2019-10-29 10:17:51)
> Name of the i915 module parameter providing fault injection function is
> changing for consistency with a new convention of naming i915 driver
> internal functions called from the driver PCI .probe entry point.  Adjust
> the test to use the new name.
> 
> Suggested-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
> Cc: Michał Wajdeczko <michal.wajdeczko@intel.com>
> Cc: Michał Winiarski <michal.winiarski@intel.com>
> Cc: Piotr Piórkowski <piotr.piorkowski@intel.com>
> Cc: Tomasz Lis <tomasz.lis@intel.com>
> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  tests/i915/i915_module_load.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
> index 7fe83520..a620ab41 100644
> --- a/tests/i915/i915_module_load.c
> +++ b/tests/i915/i915_module_load.c
> @@ -356,7 +356,7 @@ igt_main
>  
>                 igt_i915_driver_unload();
>  
> -               while (inject_fault("i915", "inject_load_failure", ++i) == 0)
> +               while (inject_fault("i915", "inject_probe_failure", ++i) == 0)

Hmm, I could do with a transition period so that we don't lose reload
tests on linus/stable.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [igt-dev] [RESEND PATCH i-g-t v2 1/1] tests/i915_module_load: Use new name of fault injection module parameter

[Chris Wilson]

Quoting Janusz Krzysztofik (2019-10-29 10:17:51)
> Name of the i915 module parameter providing fault injection function is
> changing for consistency with a new convention of naming i915 driver
> internal functions called from the driver PCI .probe entry point.  Adjust
> the test to use the new name.
> 
> Suggested-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
> Cc: Michał Wajdeczko <michal.wajdeczko@intel.com>
> Cc: Michał Winiarski <michal.winiarski@intel.com>
> Cc: Piotr Piórkowski <piotr.piorkowski@intel.com>
> Cc: Tomasz Lis <tomasz.lis@intel.com>
> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  tests/i915/i915_module_load.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
> index 7fe83520..a620ab41 100644
> --- a/tests/i915/i915_module_load.c
> +++ b/tests/i915/i915_module_load.c
> @@ -356,7 +356,7 @@ igt_main
>  
>                 igt_i915_driver_unload();
>  
> -               while (inject_fault("i915", "inject_load_failure", ++i) == 0)
> +               while (inject_fault("i915", "inject_probe_failure", ++i) == 0)

Hmm, I could do with a transition period so that we don't lose reload
tests on linus/stable.
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[PATCH i-g-t] tests/i915_module_load: Use new name of fault injection module parameter

[Chris Wilson]

From: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>

Name of the i915 module parameter providing fault injection function is
changing for consistency with a new convention of naming i915 driver
internal functions called from the driver PCI .probe entry point.  Adjust
the test to use the new name.

[ickle: keep the old parameter name around for stable testing]

Suggested-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
Cc: Michał Wajdeczko <michal.wajdeczko@intel.com>
Cc: Michał Winiarski <michal.winiarski@intel.com>
Cc: Piotr Piórkowski <piotr.piorkowski@intel.com>
Cc: Tomasz Lis <tomasz.lis@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 lib/igt_kmod.c                | 40 +++++++++++++++++++++++++++++++++++
 lib/igt_kmod.h                |  2 ++
 tests/i915/i915_module_load.c |  8 ++++++-
 3 files changed, 49 insertions(+), 1 deletion(-)

diff --git a/lib/igt_kmod.c b/lib/igt_kmod.c
index c3da46678..f43da9f42 100644
--- a/lib/igt_kmod.c
+++ b/lib/igt_kmod.c
@@ -153,6 +153,46 @@ static int modprobe(struct kmod_module *kmod, const char *options)
 					       NULL, NULL, NULL);
 }
 
+/**
+ * igt_kmod_has_param:
+ * @mod_name: The name of the module
+ * @param: The name of the parameter
+ *
+ * Returns: true if the module has the parameter, false otherwise.
+ */
+bool igt_kmod_has_param(const char *module_name, const char *param)
+{
+	struct kmod_module *kmod;
+	struct kmod_list *d, *pre;
+	bool result = false;
+
+	if (kmod_module_new_from_name(kmod_ctx(), module_name, &kmod))
+		return false;
+
+	pre = NULL;
+	if (!kmod_module_get_info(kmod, &pre))
+		goto out;
+
+	kmod_list_foreach(d, pre) {
+		const char *key, *val;
+
+		key = kmod_module_info_get_key(d);
+		if (strcmp(key, "parmtype"))
+			continue;
+
+		val = kmod_module_info_get_value(d);
+		if (val && strcmp(val, param) == 0) {
+			result = true;
+			break;
+		}
+	}
+	kmod_module_info_free_list(pre);
+
+out:
+	kmod_module_unref(kmod);
+	return result;
+}
+
 /**
  * igt_kmod_load:
  * @mod_name: The name of the module
diff --git a/lib/igt_kmod.h b/lib/igt_kmod.h
index 87d36d400..e47ff2df6 100644
--- a/lib/igt_kmod.h
+++ b/lib/igt_kmod.h
@@ -31,6 +31,8 @@
 bool igt_kmod_is_loaded(const char *mod_name);
 void igt_kmod_list_loaded(void);
 
+bool igt_kmod_has_param(const char *mod_name, const char *param);
+
 int igt_kmod_load(const char *mod_name, const char *opts);
 int igt_kmod_unload(const char *mod_name, unsigned int flags);
 
diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
index f42083f53..7d9a5cfd2 100644
--- a/tests/i915/i915_module_load.c
+++ b/tests/i915/i915_module_load.c
@@ -350,11 +350,17 @@ igt_main
 	}
 
 	igt_subtest("reload-with-fault-injection") {
+		const char *param;
 		int i = 0;
 
 		igt_i915_driver_unload();
 
-		while (inject_fault("i915", "inject_load_failure", ++i) == 0)
+		param = "inject_probe_failure";
+		if (!igt_kmod_has_param("i915", param))
+			param = "inject_load_failure";
+		igt_require(igt_kmod_has_param("i915", param));
+
+		while (inject_fault("i915", param, ++i) == 0)
 			;
 
 		/* We expect to hit at least one fault! */
-- 
2.24.0.rc1

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

[Intel-gfx] [PATCH i-g-t] tests/i915_module_load: Use new name of fault injection module parameter

[Chris Wilson]

From: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>

Name of the i915 module parameter providing fault injection function is
changing for consistency with a new convention of naming i915 driver
internal functions called from the driver PCI .probe entry point.  Adjust
the test to use the new name.

[ickle: keep the old parameter name around for stable testing]

Suggested-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
Cc: Michał Wajdeczko <michal.wajdeczko@intel.com>
Cc: Michał Winiarski <michal.winiarski@intel.com>
Cc: Piotr Piórkowski <piotr.piorkowski@intel.com>
Cc: Tomasz Lis <tomasz.lis@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 lib/igt_kmod.c                | 40 +++++++++++++++++++++++++++++++++++
 lib/igt_kmod.h                |  2 ++
 tests/i915/i915_module_load.c |  8 ++++++-
 3 files changed, 49 insertions(+), 1 deletion(-)

diff --git a/lib/igt_kmod.c b/lib/igt_kmod.c
index c3da46678..f43da9f42 100644
--- a/lib/igt_kmod.c
+++ b/lib/igt_kmod.c
@@ -153,6 +153,46 @@ static int modprobe(struct kmod_module *kmod, const char *options)
 					       NULL, NULL, NULL);
 }
 
+/**
+ * igt_kmod_has_param:
+ * @mod_name: The name of the module
+ * @param: The name of the parameter
+ *
+ * Returns: true if the module has the parameter, false otherwise.
+ */
+bool igt_kmod_has_param(const char *module_name, const char *param)
+{
+	struct kmod_module *kmod;
+	struct kmod_list *d, *pre;
+	bool result = false;
+
+	if (kmod_module_new_from_name(kmod_ctx(), module_name, &kmod))
+		return false;
+
+	pre = NULL;
+	if (!kmod_module_get_info(kmod, &pre))
+		goto out;
+
+	kmod_list_foreach(d, pre) {
+		const char *key, *val;
+
+		key = kmod_module_info_get_key(d);
+		if (strcmp(key, "parmtype"))
+			continue;
+
+		val = kmod_module_info_get_value(d);
+		if (val && strcmp(val, param) == 0) {
+			result = true;
+			break;
+		}
+	}
+	kmod_module_info_free_list(pre);
+
+out:
+	kmod_module_unref(kmod);
+	return result;
+}
+
 /**
  * igt_kmod_load:
  * @mod_name: The name of the module
diff --git a/lib/igt_kmod.h b/lib/igt_kmod.h
index 87d36d400..e47ff2df6 100644
--- a/lib/igt_kmod.h
+++ b/lib/igt_kmod.h
@@ -31,6 +31,8 @@
 bool igt_kmod_is_loaded(const char *mod_name);
 void igt_kmod_list_loaded(void);
 
+bool igt_kmod_has_param(const char *mod_name, const char *param);
+
 int igt_kmod_load(const char *mod_name, const char *opts);
 int igt_kmod_unload(const char *mod_name, unsigned int flags);
 
diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
index f42083f53..7d9a5cfd2 100644
--- a/tests/i915/i915_module_load.c
+++ b/tests/i915/i915_module_load.c
@@ -350,11 +350,17 @@ igt_main
 	}
 
 	igt_subtest("reload-with-fault-injection") {
+		const char *param;
 		int i = 0;
 
 		igt_i915_driver_unload();
 
-		while (inject_fault("i915", "inject_load_failure", ++i) == 0)
+		param = "inject_probe_failure";
+		if (!igt_kmod_has_param("i915", param))
+			param = "inject_load_failure";
+		igt_require(igt_kmod_has_param("i915", param));
+
+		while (inject_fault("i915", param, ++i) == 0)
 			;
 
 		/* We expect to hit at least one fault! */
-- 
2.24.0.rc1

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

[igt-dev] [PATCH i-g-t] tests/i915_module_load: Use new name of fault injection module parameter

[Chris Wilson]

From: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>

Name of the i915 module parameter providing fault injection function is
changing for consistency with a new convention of naming i915 driver
internal functions called from the driver PCI .probe entry point.  Adjust
the test to use the new name.

[ickle: keep the old parameter name around for stable testing]

Suggested-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
Cc: Michał Wajdeczko <michal.wajdeczko@intel.com>
Cc: Michał Winiarski <michal.winiarski@intel.com>
Cc: Piotr Piórkowski <piotr.piorkowski@intel.com>
Cc: Tomasz Lis <tomasz.lis@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 lib/igt_kmod.c                | 40 +++++++++++++++++++++++++++++++++++
 lib/igt_kmod.h                |  2 ++
 tests/i915/i915_module_load.c |  8 ++++++-
 3 files changed, 49 insertions(+), 1 deletion(-)

diff --git a/lib/igt_kmod.c b/lib/igt_kmod.c
index c3da46678..f43da9f42 100644
--- a/lib/igt_kmod.c
+++ b/lib/igt_kmod.c
@@ -153,6 +153,46 @@ static int modprobe(struct kmod_module *kmod, const char *options)
 					       NULL, NULL, NULL);
 }
 
+/**
+ * igt_kmod_has_param:
+ * @mod_name: The name of the module
+ * @param: The name of the parameter
+ *
+ * Returns: true if the module has the parameter, false otherwise.
+ */
+bool igt_kmod_has_param(const char *module_name, const char *param)
+{
+	struct kmod_module *kmod;
+	struct kmod_list *d, *pre;
+	bool result = false;
+
+	if (kmod_module_new_from_name(kmod_ctx(), module_name, &kmod))
+		return false;
+
+	pre = NULL;
+	if (!kmod_module_get_info(kmod, &pre))
+		goto out;
+
+	kmod_list_foreach(d, pre) {
+		const char *key, *val;
+
+		key = kmod_module_info_get_key(d);
+		if (strcmp(key, "parmtype"))
+			continue;
+
+		val = kmod_module_info_get_value(d);
+		if (val && strcmp(val, param) == 0) {
+			result = true;
+			break;
+		}
+	}
+	kmod_module_info_free_list(pre);
+
+out:
+	kmod_module_unref(kmod);
+	return result;
+}
+
 /**
  * igt_kmod_load:
  * @mod_name: The name of the module
diff --git a/lib/igt_kmod.h b/lib/igt_kmod.h
index 87d36d400..e47ff2df6 100644
--- a/lib/igt_kmod.h
+++ b/lib/igt_kmod.h
@@ -31,6 +31,8 @@
 bool igt_kmod_is_loaded(const char *mod_name);
 void igt_kmod_list_loaded(void);
 
+bool igt_kmod_has_param(const char *mod_name, const char *param);
+
 int igt_kmod_load(const char *mod_name, const char *opts);
 int igt_kmod_unload(const char *mod_name, unsigned int flags);
 
diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
index f42083f53..7d9a5cfd2 100644
--- a/tests/i915/i915_module_load.c
+++ b/tests/i915/i915_module_load.c
@@ -350,11 +350,17 @@ igt_main
 	}
 
 	igt_subtest("reload-with-fault-injection") {
+		const char *param;
 		int i = 0;
 
 		igt_i915_driver_unload();
 
-		while (inject_fault("i915", "inject_load_failure", ++i) == 0)
+		param = "inject_probe_failure";
+		if (!igt_kmod_has_param("i915", param))
+			param = "inject_load_failure";
+		igt_require(igt_kmod_has_param("i915", param));
+
+		while (inject_fault("i915", param, ++i) == 0)
 			;
 
 		/* We expect to hit at least one fault! */
-- 
2.24.0.rc1

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [PATCH i-g-t] tests/i915_module_load: Use new name of fault injection module parameter

[Janusz Krzysztofik]

Hi Chris,

On Tuesday, October 29, 2019 1:49:26 PM CET Chris Wilson wrote:
> From: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
> 
> Name of the i915 module parameter providing fault injection function is
> changing for consistency with a new convention of naming i915 driver
> internal functions called from the driver PCI .probe entry point.  Adjust
> the test to use the new name.
> 
> [ickle: keep the old parameter name around for stable testing]
> 
> Suggested-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
> Cc: Michał Wajdeczko <michal.wajdeczko@intel.com>
> Cc: Michał Winiarski <michal.winiarski@intel.com>
> Cc: Piotr Piórkowski <piotr.piorkowski@intel.com>
> Cc: Tomasz Lis <tomasz.lis@intel.com>
> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  lib/igt_kmod.c                | 40 +++++++++++++++++++++++++++++++++++
>  lib/igt_kmod.h                |  2 ++
>  tests/i915/i915_module_load.c |  8 ++++++-
>  3 files changed, 49 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/igt_kmod.c b/lib/igt_kmod.c
> index c3da46678..f43da9f42 100644
> --- a/lib/igt_kmod.c
> +++ b/lib/igt_kmod.c
> @@ -153,6 +153,46 @@ static int modprobe(struct kmod_module *kmod, const 
char *options)
>  					       NULL, NULL, 
NULL);
>  }
>  
> +/**
> + * igt_kmod_has_param:
> + * @mod_name: The name of the module
> + * @param: The name of the parameter
> + *
> + * Returns: true if the module has the parameter, false otherwise.
> + */
> +bool igt_kmod_has_param(const char *module_name, const char *param)
> +{
> +	struct kmod_module *kmod;
> +	struct kmod_list *d, *pre;
> +	bool result = false;
> +
> +	if (kmod_module_new_from_name(kmod_ctx(), module_name, &kmod))
> +		return false;
> +
> +	pre = NULL;
> +	if (!kmod_module_get_info(kmod, &pre))
> +		goto out;
> +
> +	kmod_list_foreach(d, pre) {
> +		const char *key, *val;
> +
> +		key = kmod_module_info_get_key(d);
> +		if (strcmp(key, "parmtype"))
> +			continue;
> +
> +		val = kmod_module_info_get_value(d);
> +		if (val && strcmp(val, param) == 0) {
> +			result = true;
> +			break;
> +		}
> +	}
> +	kmod_module_info_free_list(pre);
> +
> +out:
> +	kmod_module_unref(kmod);
> +	return result;
> +}
> +
>  /**
>   * igt_kmod_load:
>   * @mod_name: The name of the module
> diff --git a/lib/igt_kmod.h b/lib/igt_kmod.h
> index 87d36d400..e47ff2df6 100644
> --- a/lib/igt_kmod.h
> +++ b/lib/igt_kmod.h
> @@ -31,6 +31,8 @@
>  bool igt_kmod_is_loaded(const char *mod_name);
>  void igt_kmod_list_loaded(void);
>  
> +bool igt_kmod_has_param(const char *mod_name, const char *param);
> +
>  int igt_kmod_load(const char *mod_name, const char *opts);
>  int igt_kmod_unload(const char *mod_name, unsigned int flags);
>  
> diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
> index f42083f53..7d9a5cfd2 100644
> --- a/tests/i915/i915_module_load.c
> +++ b/tests/i915/i915_module_load.c
> @@ -350,11 +350,17 @@ igt_main
>  	}
>  
>  	igt_subtest("reload-with-fault-injection") {
> +		const char *param;
>  		int i = 0;
>  
>  		igt_i915_driver_unload();
>  
> -		while (inject_fault("i915", "inject_load_failure", +
+i) == 0)
> +		param = "inject_probe_failure";
> +		if (!igt_kmod_has_param("i915", param))
> +			param = "inject_load_failure";
> +		igt_require(igt_kmod_has_param("i915", param));
> +
> +		while (inject_fault("i915", param, ++i) == 0)
>  			;

My first thought was to just retry the loop with the old parameter name if the 
new one fails but I'm OK with your approach (maybe there will be other users 
of the new library helper).  In case R-b is needed, please tell me.

Thanks,
Janusz



>  
>  		/* We expect to hit at least one fault! */
> 




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

Re: [Intel-gfx] [PATCH i-g-t] tests/i915_module_load: Use new name of fault injection module parameter

[Janusz Krzysztofik]

Hi Chris,

On Tuesday, October 29, 2019 1:49:26 PM CET Chris Wilson wrote:
> From: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
> 
> Name of the i915 module parameter providing fault injection function is
> changing for consistency with a new convention of naming i915 driver
> internal functions called from the driver PCI .probe entry point.  Adjust
> the test to use the new name.
> 
> [ickle: keep the old parameter name around for stable testing]
> 
> Suggested-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@linux.intel.com>
> Cc: Michał Wajdeczko <michal.wajdeczko@intel.com>
> Cc: Michał Winiarski <michal.winiarski@intel.com>
> Cc: Piotr Piórkowski <piotr.piorkowski@intel.com>
> Cc: Tomasz Lis <tomasz.lis@intel.com>
> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
> ---
>  lib/igt_kmod.c                | 40 +++++++++++++++++++++++++++++++++++
>  lib/igt_kmod.h                |  2 ++
>  tests/i915/i915_module_load.c |  8 ++++++-
>  3 files changed, 49 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/igt_kmod.c b/lib/igt_kmod.c
> index c3da46678..f43da9f42 100644
> --- a/lib/igt_kmod.c
> +++ b/lib/igt_kmod.c
> @@ -153,6 +153,46 @@ static int modprobe(struct kmod_module *kmod, const 
char *options)
>  					       NULL, NULL, 
NULL);
>  }
>  
> +/**
> + * igt_kmod_has_param:
> + * @mod_name: The name of the module
> + * @param: The name of the parameter
> + *
> + * Returns: true if the module has the parameter, false otherwise.
> + */
> +bool igt_kmod_has_param(const char *module_name, const char *param)
> +{
> +	struct kmod_module *kmod;
> +	struct kmod_list *d, *pre;
> +	bool result = false;
> +
> +	if (kmod_module_new_from_name(kmod_ctx(), module_name, &kmod))
> +		return false;
> +
> +	pre = NULL;
> +	if (!kmod_module_get_info(kmod, &pre))
> +		goto out;
> +
> +	kmod_list_foreach(d, pre) {
> +		const char *key, *val;
> +
> +		key = kmod_module_info_get_key(d);
> +		if (strcmp(key, "parmtype"))
> +			continue;
> +
> +		val = kmod_module_info_get_value(d);
> +		if (val && strcmp(val, param) == 0) {
> +			result = true;
> +			break;
> +		}
> +	}
> +	kmod_module_info_free_list(pre);
> +
> +out:
> +	kmod_module_unref(kmod);
> +	return result;
> +}
> +
>  /**
>   * igt_kmod_load:
>   * @mod_name: The name of the module
> diff --git a/lib/igt_kmod.h b/lib/igt_kmod.h
> index 87d36d400..e47ff2df6 100644
> --- a/lib/igt_kmod.h
> +++ b/lib/igt_kmod.h
> @@ -31,6 +31,8 @@
>  bool igt_kmod_is_loaded(const char *mod_name);
>  void igt_kmod_list_loaded(void);
>  
> +bool igt_kmod_has_param(const char *mod_name, const char *param);
> +
>  int igt_kmod_load(const char *mod_name, const char *opts);
>  int igt_kmod_unload(const char *mod_name, unsigned int flags);
>  
> diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
> index f42083f53..7d9a5cfd2 100644
> --- a/tests/i915/i915_module_load.c
> +++ b/tests/i915/i915_module_load.c
> @@ -350,11 +350,17 @@ igt_main
>  	}
>  
>  	igt_subtest("reload-with-fault-injection") {
> +		const char *param;
>  		int i = 0;
>  
>  		igt_i915_driver_unload();
>  
> -		while (inject_fault("i915", "inject_load_failure", +
+i) == 0)
> +		param = "inject_probe_failure";
> +		if (!igt_kmod_has_param("i915", param))
> +			param = "inject_load_failure";
> +		igt_require(igt_kmod_has_param("i915", param));
> +
> +		while (inject_fault("i915", param, ++i) == 0)
>  			;

My first thought was to just retry the loop with the old parameter name if the 
new one fails but I'm OK with your approach (maybe there will be other users 
of the new library helper).  In case R-b is needed, please tell me.

Thanks,
Janusz



>  
>  		/* We expect to hit at least one fault! */
> 




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

Re: [PATCH i-g-t] tests/i915_module_load: Use new name of fault injection module parameter

[Chris Wilson]

Quoting Janusz Krzysztofik (2019-10-29 13:35:48)
> > diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
> > index f42083f53..7d9a5cfd2 100644
> > --- a/tests/i915/i915_module_load.c
> > +++ b/tests/i915/i915_module_load.c
> > @@ -350,11 +350,17 @@ igt_main
> >       }
> >  
> >       igt_subtest("reload-with-fault-injection") {
> > +             const char *param;
> >               int i = 0;
> >  
> >               igt_i915_driver_unload();
> >  
> > -             while (inject_fault("i915", "inject_load_failure", +
> +i) == 0)
> > +             param = "inject_probe_failure";
> > +             if (!igt_kmod_has_param("i915", param))
> > +                     param = "inject_load_failure";
> > +             igt_require(igt_kmod_has_param("i915", param));
> > +
> > +             while (inject_fault("i915", param, ++i) == 0)
> >                       ;
> 
> My first thought was to just retry the loop with the old parameter name if the 
> new one fails but I'm OK with your approach (maybe there will be other users 
> of the new library helper).  In case R-b is needed, please tell me.

I first did the repeated loop as well. I thought people might object to
that as being a little too hacky :)

There's a few more places were we might want to use param probing --
currently we try and ignore modparams as ABI as much as possible!
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [PATCH i-g-t] tests/i915_module_load: Use new name of fault injection module parameter

[Chris Wilson]

Quoting Janusz Krzysztofik (2019-10-29 13:35:48)
> > diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
> > index f42083f53..7d9a5cfd2 100644
> > --- a/tests/i915/i915_module_load.c
> > +++ b/tests/i915/i915_module_load.c
> > @@ -350,11 +350,17 @@ igt_main
> >       }
> >  
> >       igt_subtest("reload-with-fault-injection") {
> > +             const char *param;
> >               int i = 0;
> >  
> >               igt_i915_driver_unload();
> >  
> > -             while (inject_fault("i915", "inject_load_failure", +
> +i) == 0)
> > +             param = "inject_probe_failure";
> > +             if (!igt_kmod_has_param("i915", param))
> > +                     param = "inject_load_failure";
> > +             igt_require(igt_kmod_has_param("i915", param));
> > +
> > +             while (inject_fault("i915", param, ++i) == 0)
> >                       ;
> 
> My first thought was to just retry the loop with the old parameter name if the 
> new one fails but I'm OK with your approach (maybe there will be other users 
> of the new library helper).  In case R-b is needed, please tell me.

I first did the repeated loop as well. I thought people might object to
that as being a little too hacky :)

There's a few more places were we might want to use param probing --
currently we try and ignore modparams as ABI as much as possible!
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[igt-dev] ✗ Fi.CI.BAT: failure for tests/i915_module_load: Use new name of fault injection module parameter (rev4)

[Patchwork]

== Series Details ==

Series: tests/i915_module_load: Use new name of fault injection module parameter (rev4)
URL   : https://patchwork.freedesktop.org/series/67443/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_7210 -> IGTPW_3630
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_3630 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_3630, 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_3630/index.html

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in IGTPW_3630:

### IGT changes ###

#### Possible regressions ####

  * igt@i915_module_load@reload-with-fault-injection:
    - fi-cml-u2:          [PASS][1] -> [SKIP][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-cml-u2/igt@i915_module_load@reload-with-fault-injection.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-cml-u2/igt@i915_module_load@reload-with-fault-injection.html

  * igt@i915_selftest@live_mman:
    - fi-bsw-kefka:       [PASS][3] -> [INCOMPLETE][4]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-bsw-kefka/igt@i915_selftest@live_mman.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-bsw-kefka/igt@i915_selftest@live_mman.html

  * igt@runner@aborted:
    - fi-bsw-kefka:       NOTRUN -> [FAIL][5]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-bsw-kefka/igt@runner@aborted.html

  
#### Warnings ####

  * igt@i915_module_load@reload-with-fault-injection:
    - fi-icl-u2:          [DMESG-WARN][6] ([fdo#110595]) -> [SKIP][7]
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-icl-u2/igt@i915_module_load@reload-with-fault-injection.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-icl-u2/igt@i915_module_load@reload-with-fault-injection.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * igt@i915_module_load@reload-with-fault-injection:
    - {fi-icl-guc}:       [PASS][8] -> [SKIP][9]
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-icl-guc/igt@i915_module_load@reload-with-fault-injection.html
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-icl-guc/igt@i915_module_load@reload-with-fault-injection.html
    - {fi-tgl-u}:         [PASS][10] -> [SKIP][11]
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-tgl-u/igt@i915_module_load@reload-with-fault-injection.html
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-tgl-u/igt@i915_module_load@reload-with-fault-injection.html
    - {fi-icl-dsi}:       [PASS][12] -> [SKIP][13]
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-icl-dsi/igt@i915_module_load@reload-with-fault-injection.html
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-icl-dsi/igt@i915_module_load@reload-with-fault-injection.html
    - {fi-icl-u4}:        [PASS][14] -> [SKIP][15]
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-icl-u4/igt@i915_module_load@reload-with-fault-injection.html
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-icl-u4/igt@i915_module_load@reload-with-fault-injection.html

  
Known issues
------------

  Here are the changes found in IGTPW_3630 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@i915_module_load@reload-with-fault-injection:
    - fi-kbl-8809g:       [PASS][16] -> [SKIP][17] ([fdo#109271])
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-kbl-8809g/igt@i915_module_load@reload-with-fault-injection.html
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-kbl-8809g/igt@i915_module_load@reload-with-fault-injection.html
    - fi-byt-j1900:       [PASS][18] -> [SKIP][19] ([fdo#109271])
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-byt-j1900/igt@i915_module_load@reload-with-fault-injection.html
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-byt-j1900/igt@i915_module_load@reload-with-fault-injection.html
    - fi-cfl-guc:         [PASS][20] -> [SKIP][21] ([fdo#109271])
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-cfl-guc/igt@i915_module_load@reload-with-fault-injection.html
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-cfl-guc/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-guc:         [PASS][22] -> [SKIP][23] ([fdo#109271])
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-skl-guc/igt@i915_module_load@reload-with-fault-injection.html
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-skl-guc/igt@i915_module_load@reload-with-fault-injection.html
    - fi-cfl-8700k:       [PASS][24] -> [SKIP][25] ([fdo#109271])
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-cfl-8700k/igt@i915_module_load@reload-with-fault-injection.html
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-cfl-8700k/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-6700k2:      [PASS][26] -> [SKIP][27] ([fdo#109271])
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-skl-6700k2/igt@i915_module_load@reload-with-fault-injection.html
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-skl-6700k2/igt@i915_module_load@reload-with-fault-injection.html
    - fi-elk-e7500:       [PASS][28] -> [SKIP][29] ([fdo#109271])
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-elk-e7500/igt@i915_module_load@reload-with-fault-injection.html
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-elk-e7500/igt@i915_module_load@reload-with-fault-injection.html
    - fi-blb-e6850:       [PASS][30] -> [SKIP][31] ([fdo#109271])
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-blb-e6850/igt@i915_module_load@reload-with-fault-injection.html
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-blb-e6850/igt@i915_module_load@reload-with-fault-injection.html
    - fi-apl-guc:         [PASS][32] -> [SKIP][33] ([fdo#109271])
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-apl-guc/igt@i915_module_load@reload-with-fault-injection.html
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-apl-guc/igt@i915_module_load@reload-with-fault-injection.html
    - fi-bsw-kefka:       [PASS][34] -> [SKIP][35] ([fdo#109271])
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-bsw-kefka/igt@i915_module_load@reload-with-fault-injection.html
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-bsw-kefka/igt@i915_module_load@reload-with-fault-injection.html
    - fi-bwr-2160:        [PASS][36] -> [SKIP][37] ([fdo#109271])
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-bwr-2160/igt@i915_module_load@reload-with-fault-injection.html
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-bwr-2160/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-6770hq:      [PASS][38] -> [SKIP][39] ([fdo#109271])
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-skl-6770hq/igt@i915_module_load@reload-with-fault-injection.html
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-skl-6770hq/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-6600u:       [PASS][40] -> [SKIP][41] ([fdo#109271])
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-skl-6600u/igt@i915_module_load@reload-with-fault-injection.html
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-skl-6600u/igt@i915_module_load@reload-with-fault-injection.html
    - fi-pnv-d510:        [PASS][42] -> [SKIP][43] ([fdo#109271])
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-pnv-d510/igt@i915_module_load@reload-with-fault-injection.html
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-pnv-d510/igt@i915_module_load@reload-with-fault-injection.html
    - fi-kbl-guc:         [PASS][44] -> [SKIP][45] ([fdo#109271])
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-kbl-guc/igt@i915_module_load@reload-with-fault-injection.html
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-kbl-guc/igt@i915_module_load@reload-with-fault-injection.html
    - fi-cfl-8109u:       [PASS][46] -> [SKIP][47] ([fdo#109271])
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-cfl-8109u/igt@i915_module_load@reload-with-fault-injection.html
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-cfl-8109u/igt@i915_module_load@reload-with-fault-injection.html
    - fi-kbl-soraka:      [PASS][48] -> [SKIP][49] ([fdo#109271])
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-kbl-soraka/igt@i915_module_load@reload-with-fault-injection.html
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-kbl-soraka/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-lmem:        [PASS][50] -> [SKIP][51] ([fdo#109271])
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-skl-lmem/igt@i915_module_load@reload-with-fault-injection.html
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-skl-lmem/igt@i915_module_load@reload-with-fault-injection.html
    - fi-kbl-r:           [PASS][52] -> [SKIP][53] ([fdo#109271])
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-kbl-r/igt@i915_module_load@reload-with-fault-injection.html
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-kbl-r/igt@i915_module_load@reload-with-fault-injection.html
    - fi-gdg-551:         [PASS][54] -> [SKIP][55] ([fdo#109271])
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-gdg-551/igt@i915_module_load@reload-with-fault-injection.html
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-gdg-551/igt@i915_module_load@reload-with-fault-injection.html
    - fi-byt-n2820:       [PASS][56] -> [SKIP][57] ([fdo#109271])
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-byt-n2820/igt@i915_module_load@reload-with-fault-injection.html
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-byt-n2820/igt@i915_module_load@reload-with-fault-injection.html
    - fi-bxt-dsi:         [PASS][58] -> [SKIP][59] ([fdo#109271])
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-bxt-dsi/igt@i915_module_load@reload-with-fault-injection.html
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-bxt-dsi/igt@i915_module_load@reload-with-fault-injection.html
    - fi-hsw-4770:        [PASS][60] -> [SKIP][61] ([fdo#109271])
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-hsw-4770/igt@i915_module_load@reload-with-fault-injection.html
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-hsw-4770/igt@i915_module_load@reload-with-fault-injection.html
    - fi-snb-2600:        [PASS][62] -> [SKIP][63] ([fdo#109271])
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-snb-2600/igt@i915_module_load@reload-with-fault-injection.html
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-snb-2600/igt@i915_module_load@reload-with-fault-injection.html
    - fi-whl-u:           [PASS][64] -> [SKIP][65] ([fdo#109271])
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-whl-u/igt@i915_module_load@reload-with-fault-injection.html
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-whl-u/igt@i915_module_load@reload-with-fault-injection.html
    - fi-ilk-650:         [PASS][66] -> [SKIP][67] ([fdo#109271])
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-ilk-650/igt@i915_module_load@reload-with-fault-injection.html
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-ilk-650/igt@i915_module_load@reload-with-fault-injection.html
    - fi-bsw-n3050:       [PASS][68] -> [SKIP][69] ([fdo#109271])
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-bsw-n3050/igt@i915_module_load@reload-with-fault-injection.html
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-bsw-n3050/igt@i915_module_load@reload-with-fault-injection.html
    - fi-ivb-3770:        [PASS][70] -> [SKIP][71] ([fdo#109271])
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-ivb-3770/igt@i915_module_load@reload-with-fault-injection.html
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-ivb-3770/igt@i915_module_load@reload-with-fault-injection.html
    - fi-skl-iommu:       [PASS][72] -> [SKIP][73] ([fdo#109271])
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-skl-iommu/igt@i915_module_load@reload-with-fault-injection.html
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-skl-iommu/igt@i915_module_load@reload-with-fault-injection.html
    - fi-glk-dsi:         [PASS][74] -> [SKIP][75] ([fdo#109271])
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-glk-dsi/igt@i915_module_load@reload-with-fault-injection.html
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-glk-dsi/igt@i915_module_load@reload-with-fault-injection.html
    - fi-snb-2520m:       [PASS][76] -> [SKIP][77] ([fdo#109271])
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-snb-2520m/igt@i915_module_load@reload-with-fault-injection.html
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-snb-2520m/igt@i915_module_load@reload-with-fault-injection.html
    - fi-kbl-x1275:       [PASS][78] -> [SKIP][79] ([fdo#109271])
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-kbl-x1275/igt@i915_module_load@reload-with-fault-injection.html
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-kbl-x1275/igt@i915_module_load@reload-with-fault-injection.html
    - fi-bdw-5557u:       [PASS][80] -> [SKIP][81] ([fdo#109271])
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-bdw-5557u/igt@i915_module_load@reload-with-fault-injection.html
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-bdw-5557u/igt@i915_module_load@reload-with-fault-injection.html

  * igt@i915_selftest@live_gem_contexts:
    - fi-bsw-n3050:       [PASS][82] -> [INCOMPLETE][83] ([fdo# 111542])
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_7210/fi-bsw-n3050/igt@i915_selftest@live_gem_contexts.html
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/fi-bsw-n3050/igt@i915_selftest@live_gem_contexts.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo# 111542]: https://bugs.freedesktop.org/show_bug.cgi?id= 111542
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#110595]: https://bugs.freedesktop.org/show_bug.cgi?id=110595


Participating hosts (49 -> 43)
------------------------------

  Additional (2): fi-hsw-peppy fi-kbl-7500u 
  Missing    (8): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-icl-u3 fi-icl-y fi-byt-clapper fi-bdw-samus 


Build changes
-------------

  * CI: CI-20190529 -> None
  * IGT: IGT_5249 -> IGTPW_3630

  CI-20190529: 20190529
  CI_DRM_7210: 2b91a301d17b7c529c93aeed7ac6a7762ffe9d56 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_3630: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/index.html
  IGT_5249: aee019cec9d7f3002371f6ddf102cb67d9976b71 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3630/index.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev