Attachment 132739 Details for Bug 101718 – debug patch - no fbc reason

[patch] debug patch - no fbc reason

no-fbc-reason.patch (text/plain), 2.10 KB, created by krisman on 2017-07-17 18:41:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: krisman

Created: 2017-07-17 18:41:57 UTC

Size: 2.10 KB

patch

obsolete

>diff --git a/tests/kms_fbcon_fbt.c b/tests/kms_fbcon_fbt.c
>index d0090912694e..c33959e18976 100644
>--- a/tests/kms_fbcon_fbt.c
>+++ b/tests/kms_fbcon_fbt.c
>@@ -196,6 +196,14 @@ static void disable_features(void)
> 	igt_set_module_param_int(psr.param_name, 0);
> }
> 
>+static void fbc_print_status(int fd)
>+{
>+	char buf[128];
>+
>+	igt_debugfs_read(fd, "i915_fbc_status", buf);
>+	igt_info("FBC status:\n%s\n", buf);
>+}
>+
> static void subtest(struct feature *feature, bool suspend)
> {
> 	struct drm_info drm;
>@@ -210,17 +218,29 @@ static void subtest(struct feature *feature, bool suspend)
> 
> 	kmstest_unset_all_crtcs(drm.fd, drm.res);
> 	wait_user("Modes unset.");
>-	igt_assert(!feature->wait_until_enabled(drm.fd));
>+	if (!feature->wait_until_enabled(drm.fd)) {
>+		fbc_print_status(drm.fd);
>+		igt_assert_f(false, "FBC disabled\n");
>+
>+	}
> 
> 	set_mode_for_one_screen(&drm, &fb, feature->connector_possible_fn);
> 	wait_user("Screen set.");
>-	igt_assert(feature->wait_until_enabled(drm.fd));
>+	if (!feature->wait_until_enabled(drm.fd)) {
>+		fbc_print_status(drm.fd);
>+		igt_assert_f(false, "FBC disabled screenset\n");
>+
>+	}
> 
> 	if (suspend) {
> 		igt_system_suspend_autoresume(SUSPEND_STATE_MEM,
> 					      SUSPEND_TEST_NONE);
> 		sleep(5);
>-		igt_assert(feature->wait_until_enabled(drm.fd));
>+
>+		if (!feature->wait_until_enabled(drm.fd)) {
>+			fbc_print_status(drm.fd);
>+			igt_assert_f(false, "FBC disabled after suspend\n");
>+		}
> 	}
> 
> 	igt_remove_fb(drm.fd, &fb);
>@@ -230,14 +250,22 @@ static void subtest(struct feature *feature, bool suspend)
> 	sleep(3);
> 
> 	wait_user("Back to fbcon.");
>-	igt_assert(!feature->wait_until_enabled(drm.fd));
>+	if (!feature->wait_until_enabled(drm.fd)) {
>+		fbc_print_status(drm.fd);
>+		igt_assert_f(false, "FBC disabled after fbcon restore \n");
>+
>+	}
> 
> 	if (suspend) {
> 		igt_system_suspend_autoresume(SUSPEND_STATE_MEM,
> 					      SUSPEND_TEST_NONE);
> 		sleep(5);
>-		igt_assert(!feature->wait_until_enabled(drm.fd));
>+		if (!feature->wait_until_enabled(drm.fd)) {
>+			fbc_print_status(drm.fd);
>+			igt_assert_f(false, "FBC disabled after suspend with fbcon\n");
>+		}
> 	}
>+
> }
> 
> static void setup_environment(void)

Actions: View | Splinter Review

Attachments on bug 101718: 132494 | 132495 | 132496 | 132739 | 133076 | 133077 | 133081 | 133082 | 133381 | 133382 | 133383

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.