* [igt-dev] [CI 1/5] lib/debugfs: Function to read debugfs with the directory already open
@ 2018-09-05 21:15 Dhinakaran Pandiyan
2018-09-05 21:15 ` [igt-dev] [CI 2/5] tests/psr: Avoid opening of already open debugfs dir Dhinakaran Pandiyan
` (5 more replies)
0 siblings, 6 replies; 7+ messages in thread
From: Dhinakaran Pandiyan @ 2018-09-05 21:15 UTC (permalink / raw)
To: igt-dev; +Cc: Dhinakaran Pandiyan
tests/kms_frontbuffer_tracking and tests/kms_psr read debugfs nodes
several times after opening the directory once. There is already an
implementation of this in the kms_frontbuffer_tracking, moving that
functionality to the library will allow us to share the code with kms_psr
and kms_fbcon_fbt
Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
---
lib/igt_debugfs.c | 39 ++++++++++++++++++++++++--------
lib/igt_debugfs.h | 3 ++-
tests/kms_frontbuffer_tracking.c | 7 ++----
3 files changed, 34 insertions(+), 15 deletions(-)
diff --git a/lib/igt_debugfs.c b/lib/igt_debugfs.c
index 14753a9e..baedc225 100644
--- a/lib/igt_debugfs.c
+++ b/lib/igt_debugfs.c
@@ -262,26 +262,47 @@ int igt_debugfs_open(int device, const char *filename, int mode)
return ret;
}
+/**
+ * igt_debugfs_simple_read:
+ * @filename: file name
+ * @buf: buffer where the contents will be stored, allocated by the caller
+ * @size: size of the buffer
+ *
+ * This function is similar to __igt_debugfs_read, the difference is that it
+ * expects the debugfs directory to be open and it's descriptor passed as the
+ * first argument.
+ *
+ * Returns:
+ * -errorno on failure or bytes read on success
+ */
+int igt_debugfs_simple_read(int dir, const char *filename, char *buf, int size)
+{
+ int len;
+
+ len = igt_sysfs_read(dir, filename, buf, size - 1);
+ if (len < 0)
+ buf[0] = '\0';
+ else
+ buf[len] = '\0';
+
+ return len;
+}
+
/**
* __igt_debugfs_read:
* @filename: file name
* @buf: buffer where the contents will be stored, allocated by the caller
- * @buf_size: size of the buffer
+ * @size: size of the buffer
*
* This function opens the debugfs file, reads it, stores the content in the
* provided buffer, then closes the file. Users should make sure that the buffer
* provided is big enough to fit the whole file, plus one byte.
*/
-void __igt_debugfs_read(int fd, const char *filename, char *buf, int buf_size)
+void __igt_debugfs_read(int fd, const char *filename, char *buf, int size)
{
- int dir;
- int len;
+ int dir = igt_debugfs_dir(fd);
- dir = igt_debugfs_dir(fd);
- len = igt_sysfs_read(dir, filename, buf, buf_size - 1);
- if (len < 0)
- len = 0;
- buf[len] = '\0';
+ igt_debugfs_simple_read(dir, filename, buf, size);
close(dir);
}
diff --git a/lib/igt_debugfs.h b/lib/igt_debugfs.h
index db634a82..ff8612dc 100644
--- a/lib/igt_debugfs.h
+++ b/lib/igt_debugfs.h
@@ -37,7 +37,8 @@ char *igt_debugfs_path(int device, char *path, int pathlen);
int igt_debugfs_dir(int device);
int igt_debugfs_open(int fd, const char *filename, int mode);
-void __igt_debugfs_read(int fd, const char *filename, char *buf, int buf_size);
+void __igt_debugfs_read(int fd, const char *filename, char *buf, int size);
+int igt_debugfs_simple_read(int dir, const char *filename, char *buf, int size);
bool igt_debugfs_search(int fd, const char *filename, const char *substring);
/**
diff --git a/tests/kms_frontbuffer_tracking.c b/tests/kms_frontbuffer_tracking.c
index 921aaffc..eab84926 100644
--- a/tests/kms_frontbuffer_tracking.c
+++ b/tests/kms_frontbuffer_tracking.c
@@ -720,12 +720,9 @@ static void set_mode_for_params(struct modeset_params *params)
static void __debugfs_read(const char *param, char *buf, int len)
{
- len = igt_sysfs_read(drm.debugfs, param, buf, len - 1);
- if (len < 0) {
+ len = igt_debugfs_simple_read(drm.debugfs, param, buf, len);
+ if (len < 0)
igt_assert_eq(len, -ENODEV);
- len = 0;
- }
- buf[len] = '\0';
}
static int __debugfs_write(const char *param, char *buf, int len)
--
2.17.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [igt-dev] [CI 2/5] tests/psr: Avoid opening of already open debugfs dir
2018-09-05 21:15 [igt-dev] [CI 1/5] lib/debugfs: Function to read debugfs with the directory already open Dhinakaran Pandiyan
@ 2018-09-05 21:15 ` Dhinakaran Pandiyan
2018-09-05 21:15 ` [igt-dev] [CI 3/5] tests/fbcon_fbt: Avoid opening debugfs dir repeatedly Dhinakaran Pandiyan
` (4 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Dhinakaran Pandiyan @ 2018-09-05 21:15 UTC (permalink / raw)
To: igt-dev; +Cc: Dhinakaran Pandiyan
The tests calls igt_debugfs_dir() to open the debugfs dir and further
along calls igt_debugfs_read() each time i915_edp_psr_status needs to be
read. As igt_debugfs_read() opens the directory unnecessarily, switch to
using the newly added igt_debugfs_simple_read()
v2: Commit message typo (Jose)
Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
---
lib/igt_psr.c | 2 +-
tests/kms_frontbuffer_tracking.c | 3 ++-
tests/kms_psr.c | 10 ++++++----
3 files changed, 9 insertions(+), 6 deletions(-)
diff --git a/lib/igt_psr.c b/lib/igt_psr.c
index bc142632..6c5cf43d 100644
--- a/lib/igt_psr.c
+++ b/lib/igt_psr.c
@@ -30,7 +30,7 @@ bool psr_active(int fd, bool check_active)
bool active;
char buf[512];
- igt_debugfs_read(fd, "i915_edp_psr_status", buf);
+ igt_debugfs_simple_read(fd, "i915_edp_psr_status", buf, sizeof(buf));
active = strstr(buf, "HW Enabled & Active bit: yes\n") &&
(strstr(buf, "SRDENT") || strstr(buf, "SLEEP"));
return check_active ? active : !active;
diff --git a/tests/kms_frontbuffer_tracking.c b/tests/kms_frontbuffer_tracking.c
index eab84926..e9abceb6 100644
--- a/tests/kms_frontbuffer_tracking.c
+++ b/tests/kms_frontbuffer_tracking.c
@@ -1632,7 +1632,8 @@ static void do_status_assertions(int flags)
if (flags & ASSERT_PSR_ENABLED)
igt_assert_f(psr_wait_entry(drm.fd), "PSR still disabled\n");
else if (flags & ASSERT_PSR_DISABLED)
- igt_assert_f(psr_active(drm.fd, false), "PSR still enabled\n");
+ igt_assert_f(psr_active(drm.debugfs, false),
+ "PSR still enabled\n");
}
static void __do_assertions(const struct test_mode *t, int flags,
diff --git a/tests/kms_psr.c b/tests/kms_psr.c
index 218b3960..3e64cf04 100644
--- a/tests/kms_psr.c
+++ b/tests/kms_psr.c
@@ -192,7 +192,8 @@ static bool sink_support(data_t *data)
{
char buf[512];
- igt_debugfs_read(data->drm_fd, "i915_edp_psr_status", buf);
+ igt_debugfs_simple_read(data->debugfs_fd, "i915_edp_psr_status",
+ buf, sizeof(buf));
return data->with_psr_disabled ||
strstr(buf, "Sink_Support: yes\n");
@@ -203,7 +204,7 @@ static bool psr_wait_entry_if_enabled(data_t *data)
if (data->with_psr_disabled)
return true;
- return psr_wait_entry(data->drm_fd);
+ return psr_wait_entry(data->debugfs_fd);
}
static inline void manual(const char *expected)
@@ -215,7 +216,8 @@ static bool drrs_disabled(data_t *data)
{
char buf[512];
- igt_debugfs_read(data->drm_fd, "i915_drrs_status", buf);
+ igt_debugfs_simple_read(data->debugfs_fd, "i915_drrs_status",
+ buf, sizeof(buf));
return !strstr(buf, "DRRS Supported: Yes\n");
}
@@ -292,7 +294,7 @@ static void run_test(data_t *data)
expected = "screen GREEN";
break;
}
- igt_assert(psr_active(data->drm_fd, false));
+ igt_assert(psr_active(data->debugfs_fd, false));
manual(expected);
}
--
2.17.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [igt-dev] [CI 3/5] tests/fbcon_fbt: Avoid opening debugfs dir repeatedly
2018-09-05 21:15 [igt-dev] [CI 1/5] lib/debugfs: Function to read debugfs with the directory already open Dhinakaran Pandiyan
2018-09-05 21:15 ` [igt-dev] [CI 2/5] tests/psr: Avoid opening of already open debugfs dir Dhinakaran Pandiyan
@ 2018-09-05 21:15 ` Dhinakaran Pandiyan
2018-09-05 21:15 ` [igt-dev] [CI 4/5] tests/psr: Test PSR1 in kms_psr Dhinakaran Pandiyan
` (3 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Dhinakaran Pandiyan @ 2018-09-05 21:15 UTC (permalink / raw)
To: igt-dev; +Cc: Dhinakaran Pandiyan
Make use of igt_debugfs_simple_read() to open debugfs dir just once.
v2: Renamed function parameters s/fd/debugfs_fd (Jose)
Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
---
tests/kms_fbcon_fbt.c | 46 ++++++++++++++++++++++++-------------------
1 file changed, 26 insertions(+), 20 deletions(-)
diff --git a/tests/kms_fbcon_fbt.c b/tests/kms_fbcon_fbt.c
index 280ddbd7..d1f3a385 100644
--- a/tests/kms_fbcon_fbt.c
+++ b/tests/kms_fbcon_fbt.c
@@ -39,6 +39,7 @@ static bool do_wait_user = false;
struct drm_info {
int fd;
+ int debugfs_fd;
drmModeResPtr res;
drmModeConnectorPtr connectors[MAX_CONNECTORS];
};
@@ -58,6 +59,7 @@ static void setup_drm(struct drm_info *drm)
int i;
drm->fd = drm_open_driver_master(DRIVER_INTEL);
+ drm->debugfs_fd = igt_debugfs_dir(drm->fd);
drm->res = drmModeGetResources(drm->fd);
igt_assert(drm->res->count_connectors <= MAX_CONNECTORS);
@@ -82,12 +84,14 @@ static void teardown_drm(struct drm_info *drm)
igt_assert(close(drm->fd) == 0);
}
-static bool fbc_supported_on_chipset(int fd)
+static bool fbc_supported_on_chipset(int debugfs_fd)
{
char buf[128];
+ int ret;
- igt_debugfs_read(fd, "i915_fbc_status", buf);
- if (*buf == '\0') /* !HAS_FBC -> -ENODEV*/
+ ret = igt_debugfs_simple_read(debugfs_fd, "i915_fbc_status",
+ buf, sizeof(buf));
+ if (ret < 0)
return false;
return !strstr(buf, "FBC unsupported on this chipset\n");
@@ -98,19 +102,19 @@ static bool connector_can_fbc(drmModeConnectorPtr connector)
return true;
}
-static void fbc_print_status(int fd)
+static void fbc_print_status(int debugfs_fd)
{
static char buf[128];
- igt_debugfs_read(fd, "i915_fbc_status", buf);
+ igt_debugfs_simple_read(debugfs_fd, "i915_fbc_status", buf, sizeof(buf));
igt_debug("FBC status: %s\n", buf);
}
-static bool fbc_is_enabled(int fd)
+static bool fbc_is_enabled(int debugfs_fd)
{
char buf[128];
- igt_debugfs_read(fd, "i915_fbc_status", buf);
+ igt_debugfs_simple_read(debugfs_fd, "i915_fbc_status", buf, sizeof(buf));
return strstr(buf, "FBC enabled\n");
}
@@ -160,12 +164,14 @@ static void set_mode_for_one_screen(struct drm_info *drm, struct igt_fb *fb,
igt_assert_eq(rc, 0);
}
-static bool psr_supported_on_chipset(int fd)
+static bool psr_supported_on_chipset(int debugfs_fd)
{
char buf[256];
+ int ret;
- igt_debugfs_read(fd, "i915_edp_psr_status", buf);
- if (*buf == '\0') /* !HAS_PSR -> -ENODEV*/
+ ret = igt_debugfs_simple_read(debugfs_fd, "i915_edp_psr_status",
+ buf, sizeof(buf));
+ if (ret < 0)
return false;
return strstr(buf, "Sink_Support: yes\n");
@@ -176,19 +182,19 @@ static bool connector_can_psr(drmModeConnectorPtr connector)
return (connector->connector_type == DRM_MODE_CONNECTOR_eDP);
}
-static void psr_print_status(int fd)
+static void psr_print_status(int debugfs_fd)
{
static char buf[256];
- igt_debugfs_read(fd, "i915_edp_psr_status", buf);
+ igt_debugfs_simple_read(debugfs_fd, "i915_edp_psr_status", buf, sizeof(buf));
igt_debug("PSR status: %s\n", buf);
}
-static bool psr_is_enabled(int fd)
+static bool psr_is_enabled(int debugfs_fd)
{
char buf[256];
- igt_debugfs_read(fd, "i915_edp_psr_status", buf);
+ igt_debugfs_simple_read(debugfs_fd, "i915_edp_psr_status", buf, sizeof(buf));
return strstr(buf, "\nHW Enabled & Active bit: yes\n");
}
@@ -229,24 +235,24 @@ static void subtest(struct feature *feature, bool suspend)
setup_drm(&drm);
- igt_require(feature->supported_on_chipset(drm.fd));
+ igt_require(feature->supported_on_chipset(drm.debugfs_fd));
disable_features();
igt_set_module_param_int(feature->param_name, 1);
kmstest_unset_all_crtcs(drm.fd, drm.res);
wait_user("Modes unset.");
- igt_assert(!feature->wait_until_enabled(drm.fd));
+ igt_assert(!feature->wait_until_enabled(drm.debugfs_fd));
set_mode_for_one_screen(&drm, &fb, feature->connector_possible_fn);
wait_user("Screen set.");
- igt_assert(feature->wait_until_enabled(drm.fd));
+ igt_assert(feature->wait_until_enabled(drm.debugfs_fd));
if (suspend) {
igt_system_suspend_autoresume(SUSPEND_STATE_MEM,
SUSPEND_TEST_NONE);
sleep(5);
- igt_assert(feature->wait_until_enabled(drm.fd));
+ igt_assert(feature->wait_until_enabled(drm.debugfs_fd));
}
igt_remove_fb(drm.fd, &fb);
@@ -256,13 +262,13 @@ static void subtest(struct feature *feature, bool suspend)
sleep(3);
wait_user("Back to fbcon.");
- igt_assert(!feature->wait_until_enabled(drm.fd));
+ igt_assert(!feature->wait_until_enabled(drm.debugfs_fd));
if (suspend) {
igt_system_suspend_autoresume(SUSPEND_STATE_MEM,
SUSPEND_TEST_NONE);
sleep(5);
- igt_assert(!feature->wait_until_enabled(drm.fd));
+ igt_assert(!feature->wait_until_enabled(drm.debugfs_fd));
}
}
--
2.17.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [igt-dev] [CI 4/5] tests/psr: Test PSR1 in kms_psr
2018-09-05 21:15 [igt-dev] [CI 1/5] lib/debugfs: Function to read debugfs with the directory already open Dhinakaran Pandiyan
2018-09-05 21:15 ` [igt-dev] [CI 2/5] tests/psr: Avoid opening of already open debugfs dir Dhinakaran Pandiyan
2018-09-05 21:15 ` [igt-dev] [CI 3/5] tests/fbcon_fbt: Avoid opening debugfs dir repeatedly Dhinakaran Pandiyan
@ 2018-09-05 21:15 ` Dhinakaran Pandiyan
2018-09-05 21:15 ` [igt-dev] [CI 5/5] tests/fbcon_fbt: Enable PSR1 via debugfs Dhinakaran Pandiyan
` (2 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Dhinakaran Pandiyan @ 2018-09-05 21:15 UTC (permalink / raw)
To: igt-dev; +Cc: Dhinakaran Pandiyan
The tests we have work only with PSR1, but setting the module parameter
enable_psr enables PSR2 if the panel supports it. Make use of the newly
added debugfs toggle to enable only PSR1 when testing.
Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
---
tests/kms_psr.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/tests/kms_psr.c b/tests/kms_psr.c
index 3e64cf04..fcc04770 100644
--- a/tests/kms_psr.c
+++ b/tests/kms_psr.c
@@ -414,8 +414,9 @@ int main(int argc, char *argv[])
kmstest_set_vt_graphics_mode();
data.devid = intel_get_drm_devid(data.drm_fd);
- igt_set_module_param_int("enable_psr", data.with_psr_disabled ?
- 0 : 1);
+ if (!data.with_psr_disabled)
+ psr_enable(data.debugfs_fd);
+
igt_require_f(sink_support(&data),
"Sink does not support PSR\n");
@@ -490,6 +491,9 @@ int main(int argc, char *argv[])
}
igt_fixture {
+ if (!data.with_psr_disabled)
+ psr_disable(data.debugfs_fd);
+
close(data.debugfs_fd);
drm_intel_bufmgr_destroy(data.bufmgr);
display_fini(&data);
--
2.17.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [igt-dev] [CI 5/5] tests/fbcon_fbt: Enable PSR1 via debugfs
2018-09-05 21:15 [igt-dev] [CI 1/5] lib/debugfs: Function to read debugfs with the directory already open Dhinakaran Pandiyan
` (2 preceding siblings ...)
2018-09-05 21:15 ` [igt-dev] [CI 4/5] tests/psr: Test PSR1 in kms_psr Dhinakaran Pandiyan
@ 2018-09-05 21:15 ` Dhinakaran Pandiyan
2018-09-05 22:58 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [CI,1/5] lib/debugfs: Function to read debugfs with the directory already open Patchwork
2018-09-06 8:10 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
5 siblings, 0 replies; 7+ messages in thread
From: Dhinakaran Pandiyan @ 2018-09-05 21:15 UTC (permalink / raw)
To: igt-dev; +Cc: Dhinakaran Pandiyan
Test only PSR1 on PSR2 panels by making use of the debugfs toggle.
Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
---
tests/kms_fbcon_fbt.c | 33 ++++++++++++++++++++++-----------
1 file changed, 22 insertions(+), 11 deletions(-)
diff --git a/tests/kms_fbcon_fbt.c b/tests/kms_fbcon_fbt.c
index d1f3a385..4cfececa 100644
--- a/tests/kms_fbcon_fbt.c
+++ b/tests/kms_fbcon_fbt.c
@@ -25,6 +25,7 @@
*/
#include "igt.h"
+#include "igt_psr.h"
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
@@ -205,29 +206,39 @@ static bool psr_wait_until_enabled(int fd)
return r;
}
+static void disable_features(int fd)
+{
+ igt_set_module_param_int("enable_fbc", 0);
+ psr_disable(fd);
+}
+
+static inline void fbc_modparam_enable(int fd)
+{
+ igt_set_module_param_int("enable_fbc", 1);
+}
+
+static inline void psr_debugfs_enable(int fd)
+{
+ psr_enable(fd);
+}
+
struct feature {
bool (*supported_on_chipset)(int fd);
bool (*wait_until_enabled)(int fd);
bool (*connector_possible_fn)(drmModeConnectorPtr connector);
- const char *param_name;
+ void (*enable)(int fd);
} fbc = {
.supported_on_chipset = fbc_supported_on_chipset,
.wait_until_enabled = fbc_wait_until_enabled,
.connector_possible_fn = connector_can_fbc,
- .param_name = "enable_fbc",
+ .enable = fbc_modparam_enable,
}, psr = {
.supported_on_chipset = psr_supported_on_chipset,
.wait_until_enabled = psr_wait_until_enabled,
.connector_possible_fn = connector_can_psr,
- .param_name = "enable_psr",
+ .enable = psr_debugfs_enable,
};
-static void disable_features(void)
-{
- igt_set_module_param_int(fbc.param_name, 0);
- igt_set_module_param_int(psr.param_name, 0);
-}
-
static void subtest(struct feature *feature, bool suspend)
{
struct drm_info drm;
@@ -237,8 +248,8 @@ static void subtest(struct feature *feature, bool suspend)
igt_require(feature->supported_on_chipset(drm.debugfs_fd));
- disable_features();
- igt_set_module_param_int(feature->param_name, 1);
+ disable_features(drm.debugfs_fd);
+ feature->enable(drm.debugfs_fd);
kmstest_unset_all_crtcs(drm.fd, drm.res);
wait_user("Modes unset.");
--
2.17.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for series starting with [CI,1/5] lib/debugfs: Function to read debugfs with the directory already open
2018-09-05 21:15 [igt-dev] [CI 1/5] lib/debugfs: Function to read debugfs with the directory already open Dhinakaran Pandiyan
` (3 preceding siblings ...)
2018-09-05 21:15 ` [igt-dev] [CI 5/5] tests/fbcon_fbt: Enable PSR1 via debugfs Dhinakaran Pandiyan
@ 2018-09-05 22:58 ` Patchwork
2018-09-06 8:10 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
5 siblings, 0 replies; 7+ messages in thread
From: Patchwork @ 2018-09-05 22:58 UTC (permalink / raw)
To: Dhinakaran Pandiyan; +Cc: igt-dev
== Series Details ==
Series: series starting with [CI,1/5] lib/debugfs: Function to read debugfs with the directory already open
URL : https://patchwork.freedesktop.org/series/49236/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4772 -> IGTPW_1797 =
== Summary - SUCCESS ==
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/49236/revisions/1/mbox/
== Known issues ==
Here are the changes found in IGTPW_1797 that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@kms_frontbuffer_tracking@basic:
fi-byt-clapper: PASS -> FAIL (fdo#103167)
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
fi-snb-2520m: PASS -> INCOMPLETE (fdo#103713)
==== Possible fixes ====
igt@gem_exec_suspend@basic-s4-devices:
fi-kbl-7500u: DMESG-WARN (fdo#105128, fdo#107139) -> PASS
igt@kms_pipe_crc_basic@read-crc-pipe-b-frame-sequence:
fi-ilk-650: DMESG-WARN (fdo#106387) -> PASS
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b:
fi-byt-clapper: FAIL (fdo#107362, fdo#103191) -> PASS
fi-blb-e6850: INCOMPLETE (fdo#107718) -> PASS
igt@kms_psr@cursor_plane_move:
fi-cnl-psr: FAIL (fdo#107717) -> PASS
igt@kms_psr@primary_page_flip:
fi-cnl-psr: FAIL (fdo#107336) -> PASS
fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
fdo#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
fdo#103713 https://bugs.freedesktop.org/show_bug.cgi?id=103713
fdo#105128 https://bugs.freedesktop.org/show_bug.cgi?id=105128
fdo#106387 https://bugs.freedesktop.org/show_bug.cgi?id=106387
fdo#107139 https://bugs.freedesktop.org/show_bug.cgi?id=107139
fdo#107336 https://bugs.freedesktop.org/show_bug.cgi?id=107336
fdo#107362 https://bugs.freedesktop.org/show_bug.cgi?id=107362
fdo#107717 https://bugs.freedesktop.org/show_bug.cgi?id=107717
fdo#107718 https://bugs.freedesktop.org/show_bug.cgi?id=107718
== Participating hosts (54 -> 49) ==
Missing (5): fi-ctg-p8600 fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-hsw-4200u
== Build changes ==
* IGT: IGT_4629 -> IGTPW_1797
CI_DRM_4772: 1351ee8f3aacdb8f4a71cd17a7035556065c59a9 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_1797: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1797/
IGT_4629: c3b6d69aa3dd2d1a6c1f2e787670a0aef78f2ea5 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1797/issues.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 7+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for series starting with [CI,1/5] lib/debugfs: Function to read debugfs with the directory already open
2018-09-05 21:15 [igt-dev] [CI 1/5] lib/debugfs: Function to read debugfs with the directory already open Dhinakaran Pandiyan
` (4 preceding siblings ...)
2018-09-05 22:58 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [CI,1/5] lib/debugfs: Function to read debugfs with the directory already open Patchwork
@ 2018-09-06 8:10 ` Patchwork
5 siblings, 0 replies; 7+ messages in thread
From: Patchwork @ 2018-09-06 8:10 UTC (permalink / raw)
To: Dhinakaran Pandiyan; +Cc: igt-dev
== Series Details ==
Series: series starting with [CI,1/5] lib/debugfs: Function to read debugfs with the directory already open
URL : https://patchwork.freedesktop.org/series/49236/
State : success
== Summary ==
= CI Bug Log - changes from IGT_4629_full -> IGTPW_1797_full =
== Summary - WARNING ==
Minor unknown changes coming with IGTPW_1797_full need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_1797_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://patchwork.freedesktop.org/api/1.0/series/49236/revisions/1/mbox/
== Possible new issues ==
Here are the unknown changes that may have been introduced in IGTPW_1797_full:
=== IGT changes ===
==== Warnings ====
igt@pm_rc6_residency@rc6-accuracy:
shard-kbl: SKIP -> PASS
== Known issues ==
Here are the changes found in IGTPW_1797_full that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@drv_suspend@shrink:
shard-kbl: PASS -> INCOMPLETE (fdo#103665, fdo#106886)
igt@gem_softpin@noreloc-s3:
shard-kbl: PASS -> DMESG-WARN (fdo#103313, fdo#103558, fdo#105602)
igt@gem_workarounds@suspend-resume-fd:
shard-apl: PASS -> INCOMPLETE (fdo#103927)
igt@kms_flip@plain-flip-ts-check-interruptible:
shard-kbl: PASS -> DMESG-WARN (fdo#103558, fdo#105602) +2
igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-pri-indfb-draw-pwrite:
shard-glk: PASS -> FAIL (fdo#103167) +1
igt@kms_rotation_crc@primary-rotation-180:
shard-snb: PASS -> FAIL (fdo#103925)
igt@prime_busy@basic-after-default:
shard-snb: PASS -> INCOMPLETE (fdo#105411)
==== Possible fixes ====
igt@gem_ppgtt@blt-vs-render-ctx0:
shard-kbl: INCOMPLETE (fdo#106023, fdo#103665) -> PASS
igt@kms_flip@2x-flip-vs-expired-vblank-interruptible:
shard-glk: FAIL (fdo#105363) -> PASS
igt@kms_plane@plane-panning-bottom-right-suspend-pipe-a-planes:
shard-apl: FAIL (fdo#103375) -> PASS
igt@perf_pmu@rc6-runtime-pm:
shard-kbl: FAIL (fdo#105010) -> PASS
igt@prime_busy@before-bsd1:
shard-snb: INCOMPLETE (fdo#105411) -> SKIP
igt@testdisplay:
shard-glk: INCOMPLETE (fdo#107093, fdo#103359, k.org#198133) -> PASS
fdo#103167 https://bugs.freedesktop.org/show_bug.cgi?id=103167
fdo#103313 https://bugs.freedesktop.org/show_bug.cgi?id=103313
fdo#103359 https://bugs.freedesktop.org/show_bug.cgi?id=103359
fdo#103375 https://bugs.freedesktop.org/show_bug.cgi?id=103375
fdo#103558 https://bugs.freedesktop.org/show_bug.cgi?id=103558
fdo#103665 https://bugs.freedesktop.org/show_bug.cgi?id=103665
fdo#103925 https://bugs.freedesktop.org/show_bug.cgi?id=103925
fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927
fdo#105010 https://bugs.freedesktop.org/show_bug.cgi?id=105010
fdo#105363 https://bugs.freedesktop.org/show_bug.cgi?id=105363
fdo#105411 https://bugs.freedesktop.org/show_bug.cgi?id=105411
fdo#105602 https://bugs.freedesktop.org/show_bug.cgi?id=105602
fdo#106023 https://bugs.freedesktop.org/show_bug.cgi?id=106023
fdo#106886 https://bugs.freedesktop.org/show_bug.cgi?id=106886
fdo#107093 https://bugs.freedesktop.org/show_bug.cgi?id=107093
k.org#198133 https://bugzilla.kernel.org/show_bug.cgi?id=198133
== Participating hosts (5 -> 5) ==
No changes in participating hosts
== Build changes ==
* IGT: IGT_4629 -> IGTPW_1797
* Linux: CI_DRM_4770 -> CI_DRM_4772
CI_DRM_4770: 0c3535cf60140d017a5df73d84d06e8b1a5b5d3b @ git://anongit.freedesktop.org/gfx-ci/linux
CI_DRM_4772: 1351ee8f3aacdb8f4a71cd17a7035556065c59a9 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_1797: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1797/
IGT_4629: c3b6d69aa3dd2d1a6c1f2e787670a0aef78f2ea5 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_1797/shards.html
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-09-06 8:10 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-05 21:15 [igt-dev] [CI 1/5] lib/debugfs: Function to read debugfs with the directory already open Dhinakaran Pandiyan
2018-09-05 21:15 ` [igt-dev] [CI 2/5] tests/psr: Avoid opening of already open debugfs dir Dhinakaran Pandiyan
2018-09-05 21:15 ` [igt-dev] [CI 3/5] tests/fbcon_fbt: Avoid opening debugfs dir repeatedly Dhinakaran Pandiyan
2018-09-05 21:15 ` [igt-dev] [CI 4/5] tests/psr: Test PSR1 in kms_psr Dhinakaran Pandiyan
2018-09-05 21:15 ` [igt-dev] [CI 5/5] tests/fbcon_fbt: Enable PSR1 via debugfs Dhinakaran Pandiyan
2018-09-05 22:58 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [CI,1/5] lib/debugfs: Function to read debugfs with the directory already open Patchwork
2018-09-06 8:10 ` [igt-dev] ✓ Fi.CI.IGT: " 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.