From: "José Roberto de Souza" <jose.souza@intel.com>
To: igt-dev@lists.freedesktop.org
Subject: [igt-dev] [PATCH i-g-t v6 4/7] tests/kms_fbcon_fbt: Reduce execution time
Date: Tue, 24 Mar 2020 13:29:27 -0700 [thread overview]
Message-ID: <20200324202930.30231-4-jose.souza@intel.com> (raw)
In-Reply-To: <20200324202930.30231-1-jose.souza@intel.com>
Every time fbc_wait_until_enabled() is called from
fbc_wait_until_update() it was waiting the whole 5 seconds of timeout
to return false as expected.
To save most 99% of this adding here fbc_wait_until_disabled().
v5:
- Fixed fbc_wait_until_update() test
Signed-off-by: José Roberto de Souza <jose.souza@intel.com>
---
tests/kms_fbcon_fbt.c | 18 ++++++++++++++----
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/tests/kms_fbcon_fbt.c b/tests/kms_fbcon_fbt.c
index ed4cccbe..b8000a24 100644
--- a/tests/kms_fbcon_fbt.c
+++ b/tests/kms_fbcon_fbt.c
@@ -113,18 +113,28 @@ static void fbc_print_status(int debugfs_fd)
igt_debug("FBC status: %s\n", buf);
}
-static bool fbc_is_enabled(int debugfs_fd)
+static bool fbc_check_status(int debugfs_fd, bool enabled)
{
char buf[128];
igt_debugfs_simple_read(debugfs_fd, "i915_fbc_status", buf,
sizeof(buf));
- return strstr(buf, "FBC enabled\n");
+ if (enabled)
+ return strstr(buf, "FBC enabled\n");
+ else
+ return strstr(buf, "FBC disabled");
}
static bool fbc_wait_until_enabled(int debugfs_fd)
{
- bool r = igt_wait(fbc_is_enabled(debugfs_fd), 5000, 1);
+ bool r = igt_wait(fbc_check_status(debugfs_fd, true), 5000, 1);
+ fbc_print_status(debugfs_fd);
+ return r;
+}
+
+static bool fbc_wait_until_disabled(int debugfs_fd)
+{
+ bool r = igt_wait(fbc_check_status(debugfs_fd, false), 5000, 1);
fbc_print_status(debugfs_fd);
return r;
}
@@ -141,7 +151,7 @@ static bool fbc_wait_until_update(int debugfs)
* If one day fbcon starts to use a tiled framebuffer we would need to
* check the 'Compressing' status as in each blink it would be disabled.
*/
- return !fbc_wait_until_enabled(debugfs);
+ return fbc_wait_until_disabled(debugfs);
}
typedef bool (*connector_possible_fn)(drmModeConnectorPtr connector);
--
2.26.0
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
next prev parent reply other threads:[~2020-03-24 20:28 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-24 20:29 [igt-dev] [PATCH i-g-t v6 1/7] tests/kms_frontbuffer_tracking: Add tiling to test_mode José Roberto de Souza
2020-03-24 20:29 ` [igt-dev] [PATCH i-g-t v6 2/7] tests/kms_frontbuffer_tracking: Improve tiling test coverage José Roberto de Souza
2020-03-24 20:29 ` [igt-dev] [PATCH i-g-t v6 3/7] tests/kms_frontbuffer_tracking: Enable positive test on linear tiling José Roberto de Souza
2020-03-24 20:29 ` José Roberto de Souza [this message]
2020-03-24 20:29 ` [igt-dev] [PATCH i-g-t v6 5/7] tests/kms_fbcon_fbt: Handle FBC enabled on fbcon in GEN9+ José Roberto de Souza
2020-03-24 20:29 ` [igt-dev] [PATCH i-g-t v6 6/7] DO_NOT_MERGE: Revert "tests/kms_frontbuffer_tracking: Enable positive test on linear tiling" José Roberto de Souza
2020-03-24 20:29 ` [igt-dev] [PATCH i-g-t v6 7/7] DO_NOT_MERGE: Revert "tests/kms_fbcon_fbt: Handle FBC enabled on fbcon in GEN9+" José Roberto de Souza
2020-03-24 22:03 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,v6,1/7] tests/kms_frontbuffer_tracking: Add tiling to test_mode Patchwork
2020-03-25 0:23 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200324202930.30231-4-jose.souza@intel.com \
--to=jose.souza@intel.com \
--cc=igt-dev@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox