diff options
| author | Gabriel Krisman Bertazi <krisman@collabora.co.uk> | 2017-04-26 14:57:49 -0300 |
|---|---|---|
| committer | Petri Latvala <petri.latvala@intel.com> | 2017-05-04 11:16:30 +0300 |
| commit | cbfdc33f772edf13dc55e98477512937af6bec53 (patch) | |
| tree | 53b4ee9b7202505c57fb4312ea1f6fcb1531277b /tests/kms_frontbuffer_tracking.c | |
| parent | 019fa3cd4a213cc939b660e35c41424aefc8ba71 (diff) | |
kms_frontbuffer_tracking: Don't poke compressing status for old cpus
Commit 2804afc606f8 ("kms_frontbuffer_tracking: fix compression
checking") removes the check whether the kernel supports reporting the
compression status before asserting on it. This breaks the test for
no good reason on old CPUs (SNB and earlier) where the kernel can't
report the compression status. Instead, we can check if the cpu
doesn't support reporting and adopt the same behavior as if
--no-fbc-compression-check was used.
Changes since v1:
- Move verification to setup_fbc (Paulo)
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100677
Signed-off-by: Gabriel Krisman Bertazi <krisman@collabora.co.uk>
Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Diffstat (limited to 'tests/kms_frontbuffer_tracking.c')
| -rw-r--r-- | tests/kms_frontbuffer_tracking.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/tests/kms_frontbuffer_tracking.c b/tests/kms_frontbuffer_tracking.c index 7cea4de4..c24e4a81 100644 --- a/tests/kms_frontbuffer_tracking.c +++ b/tests/kms_frontbuffer_tracking.c @@ -1516,6 +1516,7 @@ static bool fbc_supported_on_chipset(void) static void setup_fbc(void) { drmModeConnectorPtr c = get_connector(prim_mode_params.connector_id); + int devid = intel_get_drm_devid(drm.fd); if (!fbc_supported_on_chipset()) { igt_info("Can't test FBC: not supported on this chipset\n"); @@ -1532,6 +1533,11 @@ static void setup_fbc(void) "pipe A\n"); return; } + + /* Early Generations are not able to report compression status. */ + if (!AT_LEAST_GEN(devid, 7)) + opt.fbc_check_compression = false; + fbc.can_test = true; fbc_setup_last_action(); |