* [igt-dev] ✓ Fi.CI.BAT: success for Add a new IGT test to validate DC3CO state. (rev7)
[not found] <1572343926-22949-1-git-send-email-jeevan.b@intel.com>
@ 2019-10-29 11:03 ` Patchwork
[not found] ` <1572343926-22949-2-git-send-email-jeevan.b@intel.com>
` (2 subsequent siblings)
3 siblings, 0 replies; 4+ messages in thread
From: Patchwork @ 2019-10-29 11:03 UTC (permalink / raw
To: Jeevan B; +Cc: igt-dev
== Series Details ==
Series: Add a new IGT test to validate DC3CO state. (rev7)
URL : https://patchwork.freedesktop.org/series/66648/
State : success
== Summary ==
CI Bug Log - changes from IGT_5249 -> IGTPW_3626
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/index.html
Known issues
------------
Here are the changes found in IGTPW_3626 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_selftest@live_gem_contexts:
- fi-bsw-kefka: [PASS][1] -> [INCOMPLETE][2] ([fdo# 111542])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/fi-bsw-kefka/igt@i915_selftest@live_gem_contexts.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/fi-bsw-kefka/igt@i915_selftest@live_gem_contexts.html
* igt@kms_addfb_basic@bad-pitch-999:
- fi-icl-u3: [PASS][3] -> [DMESG-WARN][4] ([fdo#107724])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/fi-icl-u3/igt@kms_addfb_basic@bad-pitch-999.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/fi-icl-u3/igt@kms_addfb_basic@bad-pitch-999.html
#### Possible fixes ####
* igt@gem_basic@create-fd-close:
- {fi-icl-dsi}: [DMESG-WARN][5] ([fdo#106107]) -> [PASS][6]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/fi-icl-dsi/igt@gem_basic@create-fd-close.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/fi-icl-dsi/igt@gem_basic@create-fd-close.html
* igt@gem_ctx_create@basic-files:
- fi-apl-guc: [INCOMPLETE][7] ([fdo#103927]) -> [PASS][8]
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/fi-apl-guc/igt@gem_ctx_create@basic-files.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/fi-apl-guc/igt@gem_ctx_create@basic-files.html
* igt@gem_ctx_switch@rcs0:
- fi-cml-u2: [INCOMPLETE][9] ([fdo#110566]) -> [PASS][10]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/fi-cml-u2/igt@gem_ctx_switch@rcs0.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/fi-cml-u2/igt@gem_ctx_switch@rcs0.html
* igt@gem_mmap_gtt@basic-write-cpu-read-gtt:
- fi-icl-u3: [DMESG-WARN][11] ([fdo#107724]) -> [PASS][12] +1 similar issue
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/fi-icl-u3/igt@gem_mmap_gtt@basic-write-cpu-read-gtt.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/fi-icl-u3/igt@gem_mmap_gtt@basic-write-cpu-read-gtt.html
* {igt@i915_selftest@live_gt_heartbeat}:
- fi-bxt-dsi: [DMESG-FAIL][13] ([fdo#112096]) -> [PASS][14]
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/fi-bxt-dsi/igt@i915_selftest@live_gt_heartbeat.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/fi-bxt-dsi/igt@i915_selftest@live_gt_heartbeat.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#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#106107]: https://bugs.freedesktop.org/show_bug.cgi?id=106107
[fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
[fdo#110566]: https://bugs.freedesktop.org/show_bug.cgi?id=110566
[fdo#111764]: https://bugs.freedesktop.org/show_bug.cgi?id=111764
[fdo#112057]: https://bugs.freedesktop.org/show_bug.cgi?id=112057
[fdo#112096]: https://bugs.freedesktop.org/show_bug.cgi?id=112096
Participating hosts (50 -> 43)
------------------------------
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_3626
CI-20190529: 20190529
CI_DRM_7208: 0e6cec76a950de7c4284f588846616027080ec3d @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_3626: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/index.html
IGT_5249: aee019cec9d7f3002371f6ddf102cb67d9976b71 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Testlist changes ==
+igt@i915_pm_dc@dc3co-vpb-simulation
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/index.html
_______________________________________________
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 1/2] DC3CO PSR2 helpers
[not found] ` <1572343926-22949-2-git-send-email-jeevan.b@intel.com>
@ 2019-10-29 12:30 ` Arkadiusz Hiler
0 siblings, 0 replies; 4+ messages in thread
From: Arkadiusz Hiler @ 2019-10-29 12:30 UTC (permalink / raw
To: Jeevan B; +Cc: igt-dev
On Tue, Oct 29, 2019 at 03:42:05PM +0530, Jeevan B wrote:
> From: Anshuman Gupta <anshuman.gupta@intel.com>
>
> Add DC3CO IGT validation prerequisites stuff
> so we can enable DC3CO IGT test.
>
> v2: Removed psr2_idle_wait_entry and get_psr2_status function.
> v3: Changed macro definition and removed extra line.
> v4: Simplified edp_psr2_enabled function as per Arek's comments.
>
> Signed-off-by: Anshuman Gupta <anshuman.gupta@intel.com>
> Signed-off-by: Jeevan B <jeevan.b@intel.com>
> Reviewed-by: Animesh Manna <animesh.manna@intel.com>
> ---
> tests/i915/i915_pm_dc.c | 34 +++++++++++++++++++++++++++-------
> 1 file changed, 27 insertions(+), 7 deletions(-)
>
> diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c
> index ce3319b..44c6538 100644
> --- a/tests/i915/i915_pm_dc.c
> +++ b/tests/i915/i915_pm_dc.c
> @@ -34,8 +34,9 @@
> #include "limits.h"
>
> /* DC State Flags */
> -#define CHECK_DC5 1
> -#define CHECK_DC6 2
> +#define CHECK_DC5 (1 << 0)
> +#define CHECK_DC6 (1 << 1)
> +#define CHECK_DC3CO (1 << 2)
>
> typedef struct {
> int drm_fd;
> @@ -88,6 +89,16 @@ static bool edp_psr_sink_support(data_t *data)
> return strstr(buf, "Sink support: yes");
> }
>
> +static bool edp_psr2_enabled(data_t *data)
> +{
> + char buf[512];
> +
> + igt_debugfs_simple_read(data->debugfs_fd, "i915_edp_psr_status",
> + buf, sizeof(buf));
> +
> + return strstr(buf, "PSR mode: PSR2 enabled") != NULL;
> +}
> +
> static void cleanup_dc_psr(data_t *data)
> {
> igt_plane_t *primary;
> @@ -141,12 +152,18 @@ static uint32_t read_dc_counter(uint32_t drm_fd, int dc_flag)
> str = strstr(buf, "DC3 -> DC5 count");
> else if (dc_flag & CHECK_DC6)
> str = strstr(buf, "DC5 -> DC6 count");
> + else if (dc_flag & CHECK_DC3CO)
> + str = strstr(buf, "DC3CO count");
>
> - /* Check DC5/DC6 counter is available for the platform.
> + /* Check DC counter is available for the platform.
> * Skip the test if counter is not available.
> */
> - igt_skip_on_f(!str, "DC%d counter is not available\n",
> - dc_flag & CHECK_DC5 ? 5 : 6);
> + if (dc_flag & CHECK_DC3CO)
> + igt_skip_on_f(!str, "DC%d counter is not available\n",
> + dc_flag & CHECK_DC5 ? 5 : 6);
Why are you printing DC5/6 for DC3CO check here? Seems like a copy and
paste without the required edit.
You can just move those up so you don't have to check the flags twice,
something like:
diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c
index 46db1356..43af86d4 100644
--- a/tests/i915/i915_pm_dc.c
+++ b/tests/i915/i915_pm_dc.c
@@ -203,22 +203,17 @@ static uint32_t read_dc_counter(uint32_t drm_fd, int dc_flag)
igt_debugfs_read(drm_fd, "i915_dmc_info", buf);
- if (dc_flag & CHECK_DC5)
+ if (dc_flag & CHECK_DC5) {
str = strstr(buf, "DC3 -> DC5 count");
- else if (dc_flag & CHECK_DC6)
+ igt_skip_on_f(!str, "DC5 counter is not available\n");
+ } else if (dc_flag & CHECK_DC6) {
str = strstr(buf, "DC5 -> DC6 count");
- else if (dc_flag & CHECK_DC3CO)
+ igt_skip_on_f(!str, "DC6 counter is not available\n");
+ } else if (dc_flag & CHECK_DC3CO) {
str = strstr(buf, "DC3CO count");
+ igt_skip_on_f(!str, "DC3CO counter is not available\n");
+ }
- /* Check DC counter is available for the platform.
- * Skip the test if counter is not available.
- */
- if (dc_flag & CHECK_DC3CO)
- igt_skip_on_f(!str, "DC%d counter is not available\n",
- dc_flag & CHECK_DC5 ? 5 : 6);
- else
- igt_skip_on_f(!str, "DC%d counter is not available\n",
- dc_flag & CHECK_DC5 ? 5 : 6);
return get_dc_counter(str);
}
> + else
> + igt_skip_on_f(!str, "DC%d counter is not available\n",
> + dc_flag & CHECK_DC5 ? 5 : 6);
> return get_dc_counter(str);
> }
>
> @@ -158,9 +175,12 @@ static bool dc_state_wait_entry(int drm_fd, int dc_flag, int prev_dc_count)
>
> static void check_dc_counter(int drm_fd, int dc_flag, uint32_t prev_dc_count)
> {
> + char tmp[64];
> +
> + snprintf(tmp, sizeof(tmp), "%s", dc_flag & CHECK_DC3CO ? "DC3CO" :
> + (dc_flag & CHECK_DC5 ? "DC5" : "DC6"));
> igt_assert_f(dc_state_wait_entry(drm_fd, dc_flag, prev_dc_count),
> - "DC%d state is not achieved\n",
> - dc_flag & CHECK_DC5 ? 5 : 6);
> + "%s state is not achieved\n", tmp);
> }
>
> static void test_dc_state_psr(data_t *data, int dc_flag)
> --
> 2.7.4
>
_______________________________________________
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 2/2] Add a new IGT test to validate DC3CO state
[not found] ` <1572343926-22949-3-git-send-email-jeevan.b@intel.com>
@ 2019-10-29 12:37 ` Arkadiusz Hiler
0 siblings, 0 replies; 4+ messages in thread
From: Arkadiusz Hiler @ 2019-10-29 12:37 UTC (permalink / raw
To: Jeevan B; +Cc: igt-dev
On Tue, Oct 29, 2019 at 03:42:06PM +0530, Jeevan B wrote:
> Add a subtest for DC3CO video playback case
> to generate selective frame update and validate
> that system stays in DC3CO state during execution.
>
> v2: Changed PSR2 idle check to sleep check and addressed cosmetic changes.
> v3: Renamed a function and restructured code according to Anshuman’s comments.
> v4: Cosmetic changes.
> v5: Removed DC5 check, Platform check and a function parameter.
> Renamed a function name as per Arek and Imre's Comments.
> v6: Added a new function require_dc_counter as per Arek's Comments.
> The test is now running based on time instead of the number of frames
> increased frame delay from 1 to 1.5 as per Imre's Comments.
> v7: Removed psr2_active_sleep_check and corrected the switch indentation.
>
> Signed-off-by: Jeevan B <jeevan.b@intel.com>
> Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
> ---
> tests/i915/i915_pm_dc.c | 152 +++++++++++++++++++++++++++++++++++++++++++++++-
> 1 file changed, 151 insertions(+), 1 deletion(-)
>
> diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c
> index 44c6538..46db135 100644
> --- a/tests/i915/i915_pm_dc.c
> +++ b/tests/i915/i915_pm_dc.c
> @@ -32,6 +32,7 @@
> #include "igt_psr.h"
> #include "igt_sysfs.h"
> #include "limits.h"
> +#include "time.h"
>
> /* DC State Flags */
> #define CHECK_DC5 (1 << 0)
> @@ -39,12 +40,16 @@
> #define CHECK_DC3CO (1 << 2)
>
> typedef struct {
> + double r, g, b;
> +} color_t;
> +
> +typedef struct {
> int drm_fd;
> int msr_fd;
> int debugfs_fd;
> uint32_t devid;
> igt_display_t display;
> - struct igt_fb fb_white;
> + struct igt_fb fb_white, fb_rgb, fb_rgr;
> enum psr_mode op_psr_mode;
> drmModeModeInfo *mode;
> igt_output_t *output;
> @@ -110,6 +115,42 @@ static void cleanup_dc_psr(data_t *data)
> igt_remove_fb(data->drm_fd, &data->fb_white);
> }
>
> +static void cleanup_dc3co_fbs(data_t *data)
> +{
> + igt_plane_t *primary;
> +
> + primary = igt_output_get_plane_type(data->output,
> + DRM_PLANE_TYPE_PRIMARY);
> + igt_plane_set_fb(primary, NULL);
> + /* Clear Frame Buffers */
> + igt_display_commit(&data->display);
> + igt_remove_fb(data->drm_fd, &data->fb_rgb);
> + igt_remove_fb(data->drm_fd, &data->fb_rgr);
> +}
> +
> +static void paint_rectangles(data_t *data,
> + drmModeModeInfo *mode,
> + color_t *colors,
> + igt_fb_t *fb)
> +{
> + cairo_t *cr = igt_get_cairo_ctx(data->drm_fd, fb);
> + int i, l = mode->hdisplay / 3;
> + int rows_remaining = mode->hdisplay % 3;
> +
> + /* Paint 3 solid rectangles. */
> + for (i = 0 ; i < 3; i++) {
> + igt_paint_color(cr, i * l, 0, l, mode->vdisplay,
> + colors[i].r, colors[i].g, colors[i].b);
> + }
> +
> + if (rows_remaining > 0)
> + igt_paint_color(cr, i * l, 0, rows_remaining, mode->vdisplay,
> + colors[i - 1].r, colors[i - 1].g,
> + colors[i - 1].b);
> +
> + igt_put_cairo_ctx(data->drm_fd, fb, cr);
> +}
> +
> static void setup_primary(data_t *data)
> {
> igt_plane_t *primary;
> @@ -127,6 +168,20 @@ static void setup_primary(data_t *data)
> igt_display_commit(&data->display);
> }
>
> +static void create_color_fb(data_t *data, igt_fb_t *fb, color_t *fb_color)
> +{
> + int fb_id;
> +
> + fb_id = igt_create_fb(data->drm_fd,
> + data->mode->hdisplay,
> + data->mode->vdisplay,
> + DRM_FORMAT_XRGB8888,
> + LOCAL_DRM_FORMAT_MOD_NONE,
> + fb);
> + igt_assert(fb_id);
> + paint_rectangles(data, data->mode, fb_color, fb);
> +}
> +
> static uint32_t get_dc_counter(char *dc_data)
> {
> char *e;
> @@ -183,6 +238,94 @@ static void check_dc_counter(int drm_fd, int dc_flag, uint32_t prev_dc_count)
> "%s state is not achieved\n", tmp);
> }
>
> +static void setup_videoplayback(data_t *data)
> +{
> + color_t red_green_blue[] = {
> + { 1.0, 0.0, 0.0 },
> + { 0.0, 1.0, 0.0 },
> + { 0.0, 0.0, 1.0 },
> + };
> + color_t red_green_red[] = {
> + { 1.0, 0.0, 0.0 },
> + { 0.0, 1.0, 0.0 },
> + { 1.0, 0.0, 0.0 },
> + };
> +
> + create_color_fb(data, &data->fb_rgb, red_green_blue);
> + create_color_fb(data, &data->fb_rgr, red_green_red);
> +}
> +
> +static void check_dc3co_with_videoplayback_like_load(data_t *data)
> +{
> + igt_plane_t *primary;
> + uint32_t dc3co_prev_cnt;
> + int delay;
> + time_t secs = 6;
> + time_t startTime = time(NULL);
> +
> + primary = igt_output_get_plane_type(data->output,
> + DRM_PLANE_TYPE_PRIMARY);
> + igt_plane_set_fb(primary, NULL);
> + dc3co_prev_cnt = read_dc_counter(data->drm_fd, CHECK_DC3CO);
> + /* Calculate delay to generate idle frame in usec*/
> + delay = 1.5 * ((1000 * 1000) / data->mode->vrefresh);
> +
> + while (time(NULL) - startTime < secs) {
> + igt_plane_set_fb(primary, &data->fb_rgb);
> + igt_display_commit(&data->display);
> + usleep(delay);
> +
> + igt_plane_set_fb(primary, &data->fb_rgr);
> + igt_display_commit(&data->display);
> + usleep(delay);
> + }
> +
> + check_dc_counter(data->drm_fd, CHECK_DC3CO, dc3co_prev_cnt);
> +}
> +
> +static void require_dc_counter(int drm_fd, int dc_flag)
Nice! Thanks for introducing this. Now we should use that for the tests
that are using other counters so they have the requirements up-front and
don't skip on reading.
As a general rule of thumb:
* skip only from functions that have *skip* or *require* in the name
* let *read* and other functions to fail
Something like:
diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c
index 43af86d4..64d81440 100644
--- a/tests/i915/i915_pm_dc.c
+++ b/tests/i915/i915_pm_dc.c
@@ -205,13 +205,13 @@ static uint32_t read_dc_counter(uint32_t drm_fd, int dc_flag)
if (dc_flag & CHECK_DC5) {
str = strstr(buf, "DC3 -> DC5 count");
- igt_skip_on_f(!str, "DC5 counter is not available\n");
+ igt_assert_f(str, "DC5 counter is not available\n");
} else if (dc_flag & CHECK_DC6) {
str = strstr(buf, "DC5 -> DC6 count");
- igt_skip_on_f(!str, "DC6 counter is not available\n");
+ igt_assert_f(str, "DC6 counter is not available\n");
} else if (dc_flag & CHECK_DC3CO) {
str = strstr(buf, "DC3CO count");
- igt_skip_on_f(!str, "DC3CO counter is not available\n");
+ igt_assert_f(str, "DC3CO counter is not available\n");
}
return get_dc_counter(str);
@@ -305,7 +305,6 @@ static void require_dc_counter(int drm_fd, int dc_flag)
static void setup_dc3co(data_t *data)
{
- require_dc_counter(data->drm_fd, CHECK_DC3CO);
data->op_psr_mode = PSR_MODE_2;
psr_enable(data->debugfs_fd, data->op_psr_mode);
igt_require_f(edp_psr2_enabled(data),
@@ -314,6 +313,7 @@ static void setup_dc3co(data_t *data)
static void test_dc3co_vpb_simulation(data_t *data)
{
+ require_dc_counter(data->drm_fd, CHECK_DC3CO);
setup_output(data);
setup_dc3co(data);
setup_videoplayback(data);
@@ -325,6 +325,7 @@ static void test_dc_state_psr(data_t *data, int dc_flag)
{
uint32_t dc_counter_before_psr;
+ require_dc_counter(data->drm_fd, dc_flag);
dc_counter_before_psr = read_dc_counter(data->drm_fd, dc_flag);
setup_output(data);
setup_primary(data);
@@ -386,6 +387,7 @@ static void test_dc_state_dpms(data_t *data, int dc_flag)
{
uint32_t dc_counter;
+ require_dc_counter(data->drm_fd, dc_flag);
setup_dc_dpms(data);
dc_counter = read_dc_counter(data->drm_fd, dc_flag);
dpms_off(data);
> +{
> + char buf[4096];
> +
> + igt_debugfs_simple_read(drm_fd, "i915_dmc_info",
> + buf, sizeof(buf));
> +
> + switch (dc_flag) {
> + case CHECK_DC3CO:
> + igt_skip_on_f(strstr(buf, "DC3CO count"),
> + "DC3CO counter is not available\n");
> + break;
> + case CHECK_DC5:
> + igt_skip_on_f(strstr(buf, "DC3 -> DC5 count"),
> + "DC5 counter is not available\n");
> + break;
> + case CHECK_DC6:
> + igt_skip_on_f(strstr(buf, "DC5 -> DC6 count"),
> + "DC6 counter is not available\n");
> + break;
> + default:
> + igt_assert_f(0, "Unknown DC counter %d\n", dc_flag);
> + }
> +}
> +
> +static void setup_dc3co(data_t *data)
> +{
> + require_dc_counter(data->drm_fd, CHECK_DC3CO);
> + data->op_psr_mode = PSR_MODE_2;
> + psr_enable(data->debugfs_fd, data->op_psr_mode);
> + igt_require_f(edp_psr2_enabled(data),
> + "PSR2 is not enabled\n");
> +}
> +
> +static void test_dc3co_vpb_simulation(data_t *data)
> +{
> + setup_output(data);
> + setup_dc3co(data);
> + setup_videoplayback(data);
> + check_dc3co_with_videoplayback_like_load(data);
> + cleanup_dc3co_fbs(data);
> +}
> +
> static void test_dc_state_psr(data_t *data, int dc_flag)
> {
> uint32_t dc_counter_before_psr;
> @@ -284,6 +427,13 @@ int main(int argc, char *argv[])
> "Can't open /dev/cpu/0/msr.\n");
> }
>
> + igt_describe("This test simulate video playback "
> + "in order to validate DC3CO state "
> + "while PSR2 is active and in SLEEP state");
"This test" is something that the reader already knows. Using this kind of
reference makes sense when you need to address the whole thing few
sentences down the line. Here it just unecessarily inflates the word
count.
I suggest you to rephrase it like that:
"Make sure that we enter DC3CO when PSR2 is active and we are in SLEEP
state."
There are few more tests here that could use similar treatement so feel
free to do that in some follow up series later.
> + igt_subtest("dc3co-vpb-simulation") {
> + test_dc3co_vpb_simulation(&data);
> + }
> +
> igt_describe("This test validates display engine entry to DC5 state "
> "while PSR is active");
> igt_subtest("dc5-psr") {
> --
> 2.7.4
>
_______________________________________________
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
* [igt-dev] ✓ Fi.CI.IGT: success for Add a new IGT test to validate DC3CO state. (rev7)
[not found] <1572343926-22949-1-git-send-email-jeevan.b@intel.com>
` (2 preceding siblings ...)
[not found] ` <1572343926-22949-3-git-send-email-jeevan.b@intel.com>
@ 2019-10-29 22:19 ` Patchwork
3 siblings, 0 replies; 4+ messages in thread
From: Patchwork @ 2019-10-29 22:19 UTC (permalink / raw
To: Jeevan B; +Cc: igt-dev
== Series Details ==
Series: Add a new IGT test to validate DC3CO state. (rev7)
URL : https://patchwork.freedesktop.org/series/66648/
State : success
== Summary ==
CI Bug Log - changes from IGT_5249_full -> IGTPW_3626_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_3626_full:
### IGT changes ###
#### Possible regressions ####
* {igt@i915_pm_dc@dc3co-vpb-simulation} (NEW):
- shard-iclb: NOTRUN -> [SKIP][1]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb2/igt@i915_pm_dc@dc3co-vpb-simulation.html
#### Suppressed ####
The following results come from untrusted machines, tests, or statuses.
They do not affect the overall result.
* igt@kms_draw_crc@draw-method-xrgb8888-blt-xtiled:
- {shard-tglb}: [PASS][2] -> [INCOMPLETE][3]
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-tglb4/igt@kms_draw_crc@draw-method-xrgb8888-blt-xtiled.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-tglb7/igt@kms_draw_crc@draw-method-xrgb8888-blt-xtiled.html
* igt@kms_plane@plane-position-covered-pipe-a-planes:
- {shard-tglb}: NOTRUN -> [INCOMPLETE][4]
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-tglb5/igt@kms_plane@plane-position-covered-pipe-a-planes.html
New tests
---------
New tests have been introduced between IGT_5249_full and IGTPW_3626_full:
### New IGT tests (1) ###
* igt@i915_pm_dc@dc3co-vpb-simulation:
- Statuses : 6 skip(s)
- Exec time: [0.0, 0.03] s
Known issues
------------
Here are the changes found in IGTPW_3626_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_balancer@smoke:
- shard-iclb: [PASS][5] -> [SKIP][6] ([fdo#110854])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb1/igt@gem_exec_balancer@smoke.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb7/igt@gem_exec_balancer@smoke.html
* igt@gem_exec_schedule@preempt-queue-bsd1:
- shard-iclb: [PASS][7] -> [SKIP][8] ([fdo#109276]) +12 similar issues
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb2/igt@gem_exec_schedule@preempt-queue-bsd1.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb7/igt@gem_exec_schedule@preempt-queue-bsd1.html
* igt@gem_exec_schedule@preemptive-hang-bsd:
- shard-iclb: [PASS][9] -> [SKIP][10] ([fdo#112146]) +4 similar issues
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb5/igt@gem_exec_schedule@preemptive-hang-bsd.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb1/igt@gem_exec_schedule@preemptive-hang-bsd.html
* igt@gem_userptr_blits@sync-unmap-after-close:
- shard-hsw: [PASS][11] -> [DMESG-WARN][12] ([fdo#111870]) +3 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-hsw6/igt@gem_userptr_blits@sync-unmap-after-close.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-hsw2/igt@gem_userptr_blits@sync-unmap-after-close.html
* igt@gem_userptr_blits@sync-unmap-cycles:
- shard-snb: [PASS][13] -> [DMESG-WARN][14] ([fdo#111870]) +1 similar issue
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-snb5/igt@gem_userptr_blits@sync-unmap-cycles.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-snb5/igt@gem_userptr_blits@sync-unmap-cycles.html
* igt@i915_hangman@error-state-capture-vcs1:
- shard-iclb: [PASS][15] -> [SKIP][16] ([fdo#112080]) +6 similar issues
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb2/igt@i915_hangman@error-state-capture-vcs1.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb6/igt@i915_hangman@error-state-capture-vcs1.html
* igt@kms_cursor_crc@pipe-c-cursor-128x42-onscreen:
- shard-kbl: [PASS][17] -> [FAIL][18] ([fdo#103232])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-kbl4/igt@kms_cursor_crc@pipe-c-cursor-128x42-onscreen.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-kbl7/igt@kms_cursor_crc@pipe-c-cursor-128x42-onscreen.html
- shard-apl: [PASS][19] -> [FAIL][20] ([fdo#103232])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-apl8/igt@kms_cursor_crc@pipe-c-cursor-128x42-onscreen.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-apl7/igt@kms_cursor_crc@pipe-c-cursor-128x42-onscreen.html
* igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic:
- shard-hsw: [PASS][21] -> [FAIL][22] ([fdo#105767])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-hsw4/igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-hsw6/igt@kms_cursor_legacy@2x-long-cursor-vs-flip-atomic.html
* igt@kms_flip@flip-vs-suspend-interruptible:
- shard-apl: [PASS][23] -> [DMESG-WARN][24] ([fdo#108566]) +1 similar issue
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-apl7/igt@kms_flip@flip-vs-suspend-interruptible.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-apl6/igt@kms_flip@flip-vs-suspend-interruptible.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-indfb-draw-pwrite:
- shard-iclb: [PASS][25] -> [FAIL][26] ([fdo#103167]) +7 similar issues
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb2/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-indfb-draw-pwrite.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb8/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-indfb-draw-pwrite.html
* igt@kms_psr@psr2_primary_blt:
- shard-iclb: [PASS][27] -> [SKIP][28] ([fdo#109441])
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb2/igt@kms_psr@psr2_primary_blt.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb8/igt@kms_psr@psr2_primary_blt.html
* igt@kms_vblank@pipe-a-ts-continuation-suspend:
- shard-kbl: [PASS][29] -> [DMESG-WARN][30] ([fdo#108566]) +4 similar issues
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-kbl2/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-kbl1/igt@kms_vblank@pipe-a-ts-continuation-suspend.html
#### Possible fixes ####
* igt@gem_ctx_isolation@rcs0-s3:
- shard-kbl: [DMESG-WARN][31] ([fdo#108566]) -> [PASS][32] +4 similar issues
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-kbl4/igt@gem_ctx_isolation@rcs0-s3.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-kbl3/igt@gem_ctx_isolation@rcs0-s3.html
- shard-apl: [DMESG-WARN][33] ([fdo#108566]) -> [PASS][34] +2 similar issues
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-apl1/igt@gem_ctx_isolation@rcs0-s3.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-apl1/igt@gem_ctx_isolation@rcs0-s3.html
* igt@gem_ctx_isolation@vcs1-none:
- shard-iclb: [SKIP][35] ([fdo#109276] / [fdo#112080]) -> [PASS][36] +2 similar issues
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb3/igt@gem_ctx_isolation@vcs1-none.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb1/igt@gem_ctx_isolation@vcs1-none.html
* igt@gem_ctx_switch@queue-heavy:
- {shard-tglb}: [INCOMPLETE][37] ([fdo#111747]) -> [PASS][38] +1 similar issue
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-tglb3/igt@gem_ctx_switch@queue-heavy.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-tglb6/igt@gem_ctx_switch@queue-heavy.html
* igt@gem_ctx_switch@vcs1-heavy:
- shard-iclb: [SKIP][39] ([fdo#112080]) -> [PASS][40] +10 similar issues
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb6/igt@gem_ctx_switch@vcs1-heavy.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb4/igt@gem_ctx_switch@vcs1-heavy.html
* igt@gem_exec_schedule@fifo-bsd:
- shard-iclb: [SKIP][41] ([fdo#112146]) -> [PASS][42] +3 similar issues
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb4/igt@gem_exec_schedule@fifo-bsd.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb8/igt@gem_exec_schedule@fifo-bsd.html
* igt@gem_exec_schedule@wide-bsd2:
- shard-kbl: [INCOMPLETE][43] ([fdo#103665]) -> [PASS][44]
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-kbl4/igt@gem_exec_schedule@wide-bsd2.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-kbl2/igt@gem_exec_schedule@wide-bsd2.html
* igt@gem_userptr_blits@dmabuf-sync:
- shard-snb: [DMESG-WARN][45] ([fdo#111870]) -> [PASS][46] +3 similar issues
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-snb4/igt@gem_userptr_blits@dmabuf-sync.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-snb2/igt@gem_userptr_blits@dmabuf-sync.html
* igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy:
- shard-hsw: [DMESG-WARN][47] ([fdo#111870]) -> [PASS][48]
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-hsw1/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-hsw6/igt@gem_userptr_blits@map-fixed-invalidate-overlap-busy.html
* {igt@i915_pm_dc@dc6-psr}:
- shard-iclb: [FAIL][49] ([fdo#110548]) -> [PASS][50]
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb6/igt@i915_pm_dc@dc6-psr.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb8/igt@i915_pm_dc@dc6-psr.html
* igt@i915_pm_rpm@system-suspend:
- {shard-tglb}: [INCOMPLETE][51] ([fdo#111747] / [fdo#111850]) -> [PASS][52]
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-tglb1/igt@i915_pm_rpm@system-suspend.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-tglb2/igt@i915_pm_rpm@system-suspend.html
* igt@kms_flip@flip-vs-suspend-interruptible:
- shard-iclb: [DMESG-WARN][53] ([fdo#111764]) -> [PASS][54]
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb3/igt@kms_flip@flip-vs-suspend-interruptible.html
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb1/igt@kms_flip@flip-vs-suspend-interruptible.html
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-indfb-pgflip-blt:
- shard-iclb: [FAIL][55] ([fdo#103167]) -> [PASS][56] +4 similar issues
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb4/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-indfb-pgflip-blt.html
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb5/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-indfb-pgflip-blt.html
* igt@kms_frontbuffer_tracking@fbc-rgb565-draw-pwrite:
- {shard-tglb}: [FAIL][57] ([fdo#103167]) -> [PASS][58] +1 similar issue
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-tglb4/igt@kms_frontbuffer_tracking@fbc-rgb565-draw-pwrite.html
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-tglb1/igt@kms_frontbuffer_tracking@fbc-rgb565-draw-pwrite.html
* igt@kms_frontbuffer_tracking@psr-suspend:
- {shard-tglb}: [INCOMPLETE][59] ([fdo#111832] / [fdo#111850]) -> [PASS][60]
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-tglb1/igt@kms_frontbuffer_tracking@psr-suspend.html
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-tglb8/igt@kms_frontbuffer_tracking@psr-suspend.html
* igt@kms_plane_cursor@pipe-a-viewport-size-256:
- {shard-tglb}: [INCOMPLETE][61] -> [PASS][62]
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-tglb7/igt@kms_plane_cursor@pipe-a-viewport-size-256.html
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-tglb2/igt@kms_plane_cursor@pipe-a-viewport-size-256.html
* igt@kms_psr@psr2_cursor_mmap_cpu:
- shard-iclb: [SKIP][63] ([fdo#109441]) -> [PASS][64] +3 similar issues
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb6/igt@kms_psr@psr2_cursor_mmap_cpu.html
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb2/igt@kms_psr@psr2_cursor_mmap_cpu.html
* igt@prime_vgem@fence-wait-bsd2:
- shard-iclb: [SKIP][65] ([fdo#109276]) -> [PASS][66] +18 similar issues
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb7/igt@prime_vgem@fence-wait-bsd2.html
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb2/igt@prime_vgem@fence-wait-bsd2.html
#### Warnings ####
* igt@gem_mocs_settings@mocs-reset-bsd2:
- shard-iclb: [FAIL][67] ([fdo#111330]) -> [SKIP][68] ([fdo#109276])
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_5249/shard-iclb4/igt@gem_mocs_settings@mocs-reset-bsd2.html
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/shard-iclb5/igt@gem_mocs_settings@mocs-reset-bsd2.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232
[fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665
[fdo#105767]: https://bugs.freedesktop.org/show_bug.cgi?id=105767
[fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
[fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#110548]: https://bugs.freedesktop.org/show_bug.cgi?id=110548
[fdo#110854]: https://bugs.freedesktop.org/show_bug.cgi?id=110854
[fdo#111329]: https://bugs.freedesktop.org/show_bug.cgi?id=111329
[fdo#111330]: https://bugs.freedesktop.org/show_bug.cgi?id=111330
[fdo#111606]: https://bugs.freedesktop.org/show_bug.cgi?id=111606
[fdo#111646]: https://bugs.freedesktop.org/show_bug.cgi?id=111646
[fdo#111671]: https://bugs.freedesktop.org/show_bug.cgi?id=111671
[fdo#111677]: https://bugs.freedesktop.org/show_bug.cgi?id=111677
[fdo#111703]: https://bugs.freedesktop.org/show_bug.cgi?id=111703
[fdo#111747]: https://bugs.freedesktop.org/show_bug.cgi?id=111747
[fdo#111764]: https://bugs.freedesktop.org/show_bug.cgi?id=111764
[fdo#111832]: https://bugs.freedesktop.org/show_bug.cgi?id=111832
[fdo#111850]: https://bugs.freedesktop.org/show_bug.cgi?id=111850
[fdo#111870]: https://bugs.freedesktop.org/show_bug.cgi?id=111870
[fdo#111884]: https://bugs.freedesktop.org/show_bug.cgi?id=111884
[fdo#112037]: https://bugs.freedesktop.org/show_bug.cgi?id=112037
[fdo#112080]: https://bugs.freedesktop.org/show_bug.cgi?id=112080
[fdo#112146]: https://bugs.freedesktop.org/show_bug.cgi?id=112146
Participating hosts (8 -> 8)
------------------------------
No changes in participating hosts
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_5249 -> IGTPW_3626
CI-20190529: 20190529
CI_DRM_7208: 0e6cec76a950de7c4284f588846616027080ec3d @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_3626: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3626/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_3626/index.html
_______________________________________________
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-10-29 22:19 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1572343926-22949-1-git-send-email-jeevan.b@intel.com>
2019-10-29 11:03 ` [igt-dev] ✓ Fi.CI.BAT: success for Add a new IGT test to validate DC3CO state. (rev7) Patchwork
[not found] ` <1572343926-22949-2-git-send-email-jeevan.b@intel.com>
2019-10-29 12:30 ` [igt-dev] [PATCH i-g-t 1/2] DC3CO PSR2 helpers Arkadiusz Hiler
[not found] ` <1572343926-22949-3-git-send-email-jeevan.b@intel.com>
2019-10-29 12:37 ` [igt-dev] [PATCH i-g-t 2/2] Add a new IGT test to validate DC3CO state Arkadiusz Hiler
2019-10-29 22:19 ` [igt-dev] ✓ Fi.CI.IGT: success for Add a new IGT test to validate DC3CO state. (rev7) 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.