* [igt-dev] [PATCH i-g-t] lib/i915: Remove discontinued i915 module parameters
@ 2019-07-31 22:03 Ashutosh Dixit
2019-08-01 6:47 ` Chris Wilson
2019-08-01 14:17 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork
0 siblings, 2 replies; 4+ messages in thread
From: Ashutosh Dixit @ 2019-07-31 22:03 UTC (permalink / raw)
To: igt-dev
i915 module parameters "enable_guc_submission" and "enable_execlists"
have been discontinued since 2017 so remove them from IGT too.
Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
---
lib/i915/gem_submission.c | 10 +---------
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/lib/i915/gem_submission.c b/lib/i915/gem_submission.c
index a8bb45c6..7602d7f6 100644
--- a/lib/i915/gem_submission.c
+++ b/lib/i915/gem_submission.c
@@ -74,7 +74,6 @@ unsigned gem_submission_method(int fd)
{
const int gen = intel_gen(intel_get_drm_devid(fd));
unsigned flags = 0;
- int result;
int dir;
@@ -87,14 +86,7 @@ unsigned gem_submission_method(int fd)
goto out;
}
- if (igt_sysfs_get_boolean(dir, "enable_guc_submission")) {
- flags |= GEM_SUBMISSION_GUC | GEM_SUBMISSION_EXECLISTS;
- goto out;
- }
-
- if (igt_sysfs_scanf(dir, "enable_execlists", "%d", &result) != 1)
- result = gen >= 8;
- if (result) {
+ if (gen >= 8) {
flags |= GEM_SUBMISSION_EXECLISTS;
goto out;
}
--
2.22.0
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [igt-dev] [PATCH i-g-t] lib/i915: Remove discontinued i915 module parameters
2019-07-31 22:03 [igt-dev] [PATCH i-g-t] lib/i915: Remove discontinued i915 module parameters Ashutosh Dixit
@ 2019-08-01 6:47 ` Chris Wilson
2019-08-01 14:57 ` Chris Wilson
2019-08-01 14:17 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork
1 sibling, 1 reply; 4+ messages in thread
From: Chris Wilson @ 2019-08-01 6:47 UTC (permalink / raw)
To: Ashutosh Dixit, igt-dev
Quoting Ashutosh Dixit (2019-07-31 23:03:59)
> i915 module parameters "enable_guc_submission" and "enable_execlists"
> have been discontinued since 2017 so remove them from IGT too.
>
> Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 4+ messages in thread
* [igt-dev] ✗ Fi.CI.BAT: failure for lib/i915: Remove discontinued i915 module parameters
2019-07-31 22:03 [igt-dev] [PATCH i-g-t] lib/i915: Remove discontinued i915 module parameters Ashutosh Dixit
2019-08-01 6:47 ` Chris Wilson
@ 2019-08-01 14:17 ` Patchwork
1 sibling, 0 replies; 4+ messages in thread
From: Patchwork @ 2019-08-01 14:17 UTC (permalink / raw)
To: Ashutosh Dixit; +Cc: igt-dev
== Series Details ==
Series: lib/i915: Remove discontinued i915 module parameters
URL : https://patchwork.freedesktop.org/series/64519/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_6602 -> IGTPW_3314
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with IGTPW_3314 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_3314, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://patchwork.freedesktop.org/api/1.0/series/64519/revisions/1/mbox/
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_3314:
### IGT changes ###
#### Possible regressions ####
* igt@prime_vgem@basic-fence-read:
- fi-bsw-kefka: [PASS][1] -> [INCOMPLETE][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-bsw-kefka/igt@prime_vgem@basic-fence-read.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-bsw-kefka/igt@prime_vgem@basic-fence-read.html
Known issues
------------
Here are the changes found in IGTPW_3314 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_pm_rpm@module-reload:
- fi-cfl-8109u: [PASS][3] -> [INCOMPLETE][4] ([fdo#107802])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-cfl-8109u/igt@i915_pm_rpm@module-reload.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-cfl-8109u/igt@i915_pm_rpm@module-reload.html
* igt@i915_selftest@live_hangcheck:
- fi-icl-dsi: [PASS][5] -> [INCOMPLETE][6] ([fdo#107713] / [fdo#108569])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-icl-dsi/igt@i915_selftest@live_hangcheck.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-icl-dsi/igt@i915_selftest@live_hangcheck.html
* igt@kms_chamelium@common-hpd-after-suspend:
- fi-kbl-7567u: [PASS][7] -> [WARN][8] ([fdo#109380])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-kbl-7567u/igt@kms_chamelium@common-hpd-after-suspend.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-kbl-7567u/igt@kms_chamelium@common-hpd-after-suspend.html
* igt@kms_pipe_crc_basic@read-crc-pipe-c:
- fi-kbl-7567u: [PASS][9] -> [SKIP][10] ([fdo#109271]) +23 similar issues
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-kbl-7567u/igt@kms_pipe_crc_basic@read-crc-pipe-c.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-kbl-7567u/igt@kms_pipe_crc_basic@read-crc-pipe-c.html
* igt@prime_vgem@basic-fence-mmap:
- fi-gdg-551: [PASS][11] -> [INCOMPLETE][12] ([fdo#108316] / [fdo#111276])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-gdg-551/igt@prime_vgem@basic-fence-mmap.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-gdg-551/igt@prime_vgem@basic-fence-mmap.html
* igt@prime_vgem@basic-write:
- fi-icl-u3: [PASS][13] -> [DMESG-WARN][14] ([fdo#107724])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-icl-u3/igt@prime_vgem@basic-write.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-icl-u3/igt@prime_vgem@basic-write.html
#### Possible fixes ####
* igt@gem_mmap@basic:
- fi-icl-u3: [DMESG-WARN][15] ([fdo#107724]) -> [PASS][16] +1 similar issue
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-icl-u3/igt@gem_mmap@basic.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-icl-u3/igt@gem_mmap@basic.html
* igt@i915_selftest@live_execlists:
- fi-skl-gvtdvm: [DMESG-FAIL][17] ([fdo#111108]) -> [PASS][18]
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-skl-gvtdvm/igt@i915_selftest@live_execlists.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-skl-gvtdvm/igt@i915_selftest@live_execlists.html
* igt@i915_selftest@live_hangcheck:
- {fi-icl-u4}: [INCOMPLETE][19] ([fdo#107713] / [fdo#108569]) -> [PASS][20]
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-icl-u4/igt@i915_selftest@live_hangcheck.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-icl-u4/igt@i915_selftest@live_hangcheck.html
* igt@kms_busy@basic-flip-a:
- fi-kbl-7567u: [SKIP][21] ([fdo#109271] / [fdo#109278]) -> [PASS][22] +2 similar issues
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-kbl-7567u/igt@kms_busy@basic-flip-a.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-kbl-7567u/igt@kms_busy@basic-flip-a.html
* igt@kms_chamelium@dp-edid-read:
- fi-icl-u2: [FAIL][23] ([fdo#109483] / [fdo#109635 ]) -> [PASS][24]
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-icl-u2/igt@kms_chamelium@dp-edid-read.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-icl-u2/igt@kms_chamelium@dp-edid-read.html
* igt@prime_vgem@basic-fence-mmap:
- fi-elk-e7500: [INCOMPLETE][25] ([fdo#103989]) -> [PASS][26]
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-elk-e7500/igt@prime_vgem@basic-fence-mmap.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-elk-e7500/igt@prime_vgem@basic-fence-mmap.html
- fi-bwr-2160: [INCOMPLETE][27] ([fdo#111276]) -> [PASS][28]
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-bwr-2160/igt@prime_vgem@basic-fence-mmap.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-bwr-2160/igt@prime_vgem@basic-fence-mmap.html
* igt@prime_vgem@basic-fence-read:
- fi-pnv-d510: [INCOMPLETE][29] ([fdo#110740]) -> [PASS][30]
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-pnv-d510/igt@prime_vgem@basic-fence-read.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-pnv-d510/igt@prime_vgem@basic-fence-read.html
* igt@prime_vgem@basic-sync-default:
- fi-bxt-dsi: [FAIL][31] ([fdo#111277]) -> [PASS][32]
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-bxt-dsi/igt@prime_vgem@basic-sync-default.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-bxt-dsi/igt@prime_vgem@basic-sync-default.html
#### Warnings ####
* igt@kms_chamelium@vga-edid-read:
- fi-icl-u2: [FAIL][33] -> [SKIP][34] ([fdo#109309])
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6602/fi-icl-u2/igt@kms_chamelium@vga-edid-read.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/fi-icl-u2/igt@kms_chamelium@vga-edid-read.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#103989]: https://bugs.freedesktop.org/show_bug.cgi?id=103989
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
[fdo#107802]: https://bugs.freedesktop.org/show_bug.cgi?id=107802
[fdo#108316]: https://bugs.freedesktop.org/show_bug.cgi?id=108316
[fdo#108569]: https://bugs.freedesktop.org/show_bug.cgi?id=108569
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109309]: https://bugs.freedesktop.org/show_bug.cgi?id=109309
[fdo#109380]: https://bugs.freedesktop.org/show_bug.cgi?id=109380
[fdo#109483]: https://bugs.freedesktop.org/show_bug.cgi?id=109483
[fdo#109635 ]: https://bugs.freedesktop.org/show_bug.cgi?id=109635
[fdo#110740]: https://bugs.freedesktop.org/show_bug.cgi?id=110740
[fdo#111108]: https://bugs.freedesktop.org/show_bug.cgi?id=111108
[fdo#111276]: https://bugs.freedesktop.org/show_bug.cgi?id=111276
[fdo#111277]: https://bugs.freedesktop.org/show_bug.cgi?id=111277
Participating hosts (50 -> 44)
------------------------------
Missing (6): fi-kbl-soraka fi-byt-squawks fi-bsw-cyan fi-icl-y fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_5119 -> IGTPW_3314
CI-20190529: 20190529
CI_DRM_6602: dd7a77416867b70c404a75cb413f717d3ca1bfdc @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_3314: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/
IGT_5119: b646668b947851459e0aec0ecfc54e9ed5503bfc @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3314/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [igt-dev] [PATCH i-g-t] lib/i915: Remove discontinued i915 module parameters
2019-08-01 6:47 ` Chris Wilson
@ 2019-08-01 14:57 ` Chris Wilson
0 siblings, 0 replies; 4+ messages in thread
From: Chris Wilson @ 2019-08-01 14:57 UTC (permalink / raw)
To: Ashutosh Dixit, igt-dev
Quoting Chris Wilson (2019-08-01 07:47:57)
> Quoting Ashutosh Dixit (2019-07-31 23:03:59)
> > i915 module parameters "enable_guc_submission" and "enable_execlists"
> > have been discontinued since 2017 so remove them from IGT too.
> >
> > Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
And pushed, thanks for the patch.
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-08-01 14:57 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-31 22:03 [igt-dev] [PATCH i-g-t] lib/i915: Remove discontinued i915 module parameters Ashutosh Dixit
2019-08-01 6:47 ` Chris Wilson
2019-08-01 14:57 ` Chris Wilson
2019-08-01 14:17 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.