summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJosé Roberto de Souza <jose.souza@intel.com>2019-03-27 12:40:20 -0700
committerJosé Roberto de Souza <jose.souza@intel.com>2019-04-15 12:52:17 -0700
commite03b53c67389a0f6896f468f038522ec32c191bc (patch)
tree83bcc9031f8963cbb48359f8c63da0e83e05177e
parent14f4879bdfe58b4dbc80afd7af99daeeca6f6506 (diff)
tests/fbcon_fbt: Do not try to open the same driver twice as master
If a subtest fail before the teardown_drm() call it will keep the driver open as master causing the test to fail when trying to open the driver again as master, so lets share the drm_info struct and check if driver is already open. Starting subtest: psr (kms_fbcon_fbt:5270) CRITICAL: Test assertion failure function subtest, file ../tests/kms_fbcon_fbt.c:259: (kms_fbcon_fbt:5270) CRITICAL: Failed assertion: feature->wait_until_update(drm.debugfs_fd) Stack trace: #0 ../lib/igt_core.c:1474 __igt_fail_assert() #1 ../tests/kms_fbcon_fbt.c:261 subtest() #2 ../tests/kms_fbcon_fbt.c:316 __real_main309() #3 ../tests/kms_fbcon_fbt.c:309 main() #4 ../csu/libc-start.c:325 __libc_start_main() #5 [_start+0x29] #6 [<unknown>+0x0] Subtest psr failed. **** DEBUG **** (kms_fbcon_fbt:5270) drmtest-DEBUG: Test requirement passed: !(fd<0) (kms_fbcon_fbt:5270) igt_debugfs-DEBUG: Opening debugfs directory '/sys/kernel/debug/dri/1' (kms_fbcon_fbt:5270) DEBUG: Test requirement passed: drm->res (kms_fbcon_fbt:5270) igt_kms-DEBUG: VT: graphics mode set (mode was 0x0) (kms_fbcon_fbt:5270) DEBUG: Test requirement passed: feature->supported_on_chipset(drm.debugfs_fd) (kms_fbcon_fbt:5270) CRITICAL: Test assertion failure function subtest, file ../tests/kms_fbcon_fbt.c:259: (kms_fbcon_fbt:5270) CRITICAL: Failed assertion: feature->wait_until_update(drm.debugfs_fd) (kms_fbcon_fbt:5270) igt_core-INFO: Stack trace: (kms_fbcon_fbt:5270) igt_core-INFO: #0 ../lib/igt_core.c:1474 __igt_fail_assert() (kms_fbcon_fbt:5270) igt_core-INFO: #1 ../tests/kms_fbcon_fbt.c:261 subtest() (kms_fbcon_fbt:5270) igt_core-INFO: #2 ../tests/kms_fbcon_fbt.c:316 __real_main309() (kms_fbcon_fbt:5270) igt_core-INFO: #3 ../tests/kms_fbcon_fbt.c:309 main() (kms_fbcon_fbt:5270) igt_core-INFO: #4 ../csu/libc-start.c:325 __libc_start_main() (kms_fbcon_fbt:5270) igt_core-INFO: #5 [_start+0x29] (kms_fbcon_fbt:5270) igt_core-INFO: #6 [<unknown>+0x0] **** END **** Subtest psr: FAIL (0.845s) Starting subtest: fbc-suspend Test requirement not met in function igt_device_set_master, file ../lib/igt_device.c:55: Test requirement: __igt_device_set_master(fd) == 0 Can't become DRM master, please check if no other DRM client is running. Subtest fbc-suspend: SKIP (0.001s) Starting subtest: psr-suspend Test requirement not met in function igt_device_set_master, file ../lib/igt_device.c:55: Test requirement: __igt_device_set_master(fd) == 0 Can't become DRM master, please check if no other DRM client is running. Subtest psr-suspend: SKIP (0.000s) Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Signed-off-by: José Roberto de Souza <jose.souza@intel.com>
-rw-r--r--tests/kms_fbcon_fbt.c49
1 files changed, 28 insertions, 21 deletions
diff --git a/tests/kms_fbcon_fbt.c b/tests/kms_fbcon_fbt.c
index 23b5f88a..4a149104 100644
--- a/tests/kms_fbcon_fbt.c
+++ b/tests/kms_fbcon_fbt.c
@@ -60,6 +60,9 @@ static void setup_drm(struct drm_info *drm)
{
int i;
+ if (drm->fd >= 0)
+ return;
+
drm->fd = drm_open_driver_master(DRIVER_INTEL);
drm->debugfs_fd = igt_debugfs_dir(drm->fd);
@@ -85,6 +88,7 @@ static void teardown_drm(struct drm_info *drm)
drmModeFreeResources(drm->res);
igt_assert(close(drm->fd) == 0);
+ drm->fd = -1;
}
static bool fbc_supported_on_chipset(int debugfs_fd)
@@ -252,47 +256,46 @@ struct feature {
.enable = psr_debugfs_enable,
};
-static void subtest(struct feature *feature, bool suspend)
+static void subtest(struct drm_info *drm, struct feature *feature, bool suspend)
{
- struct drm_info drm;
struct igt_fb fb;
- setup_drm(&drm);
+ setup_drm(drm);
- igt_require(feature->supported_on_chipset(drm.debugfs_fd));
+ igt_require(feature->supported_on_chipset(drm->debugfs_fd));
- disable_features(drm.debugfs_fd);
- feature->enable(drm.debugfs_fd);
+ disable_features(drm->debugfs_fd);
+ feature->enable(drm->debugfs_fd);
- kmstest_unset_all_crtcs(drm.fd, drm.res);
+ kmstest_unset_all_crtcs(drm->fd, drm->res);
wait_user("Modes unset.");
- igt_assert(!feature->wait_until_enabled(drm.debugfs_fd));
+ igt_assert(!feature->wait_until_enabled(drm->debugfs_fd));
- set_mode_for_one_screen(&drm, &fb, feature->connector_possible_fn);
+ set_mode_for_one_screen(drm, &fb, feature->connector_possible_fn);
wait_user("Screen set.");
- igt_assert(feature->wait_until_enabled(drm.debugfs_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.debugfs_fd));
+ igt_assert(feature->wait_until_enabled(drm->debugfs_fd));
}
- igt_remove_fb(drm.fd, &fb);
- teardown_drm(&drm);
+ igt_remove_fb(drm->fd, &fb);
+ teardown_drm(drm);
/* Wait for fbcon to restore itself. */
sleep(3);
wait_user("Back to fbcon.");
- igt_assert(feature->wait_until_update(drm.debugfs_fd));
+ igt_assert(feature->wait_until_update(drm->debugfs_fd));
if (suspend) {
igt_system_suspend_autoresume(SUSPEND_STATE_MEM,
SUSPEND_TEST_NONE);
sleep(5);
- igt_assert(feature->wait_until_update(drm.debugfs_fd));
+ igt_assert(feature->wait_until_update(drm->debugfs_fd));
}
}
@@ -312,24 +315,28 @@ static void setup_environment(void)
fbcon_blink_enable(true);
}
-static void teardown_environment(void)
+static void teardown_environment(struct drm_info *drm)
{
+ if (drm->fd >= 0)
+ teardown_drm(drm);
}
igt_main
{
+ struct drm_info drm = { .fd = -1 };
+
igt_fixture
setup_environment();
igt_subtest("fbc")
- subtest(&fbc, false);
+ subtest(&drm, &fbc, false);
igt_subtest("psr")
- subtest(&psr, false);
+ subtest(&drm, &psr, false);
igt_subtest("fbc-suspend")
- subtest(&fbc, true);
+ subtest(&drm, &fbc, true);
igt_subtest("psr-suspend")
- subtest(&psr, true);
+ subtest(&drm, &psr, true);
igt_fixture
- teardown_environment();
+ teardown_environment(&drm);
}