diff options
author | José Roberto de Souza <jose.souza@intel.com> | 2020-01-23 11:00:29 -0800 |
---|---|---|
committer | José Roberto de Souza <jose.souza@intel.com> | 2020-02-26 12:13:36 -0800 |
commit | 386b71eb9de6610107bf3cbc523b9a68243c6213 (patch) | |
tree | 98d5d70e7eb516541e4679576b1708017df31c97 | |
parent | 4f875016eb1ebc211b8aadb280ae16c7e6cdc8ba (diff) |
tests/kms_psr2_su: Print the SU blocks mismatches
This will help us debug failures in this test.
Also removing the previous debug message that was wrong, as it will
always be set to MAX_SCREEN_CHANGES when SU blocks do not match.
Signed-off-by: José Roberto de Souza <jose.souza@intel.com>
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
-rw-r--r-- | tests/kms_psr2_su.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/tests/kms_psr2_su.c b/tests/kms_psr2_su.c index f2244b29..9f40c735 100644 --- a/tests/kms_psr2_su.c +++ b/tests/kms_psr2_su.c @@ -183,9 +183,13 @@ static bool update_screen_and_test(data_t *data) igt_assert_f(data->op, "Operation not handled\n"); } - if (psr2_wait_su(data->debugfs_fd, &su_blocks)) + if (psr2_wait_su(data->debugfs_fd, &su_blocks)) { ret = su_blocks == EXPECTED_NUM_SU_BLOCKS; + if (!ret) + igt_debug("Not matching SU blocks read: %u\n", su_blocks); + } + return ret; } @@ -206,7 +210,6 @@ static void run(data_t *data) result = update_screen_and_test(data); } - igt_debug("Screen changes: %u\n", data->screen_changes); igt_assert_f(result, "No matching selective update blocks read from debugfs\n"); } |