diff options
author | Maarten Lankhorst <maarten.lankhorst@linux.intel.com> | 2018-10-30 15:47:31 +0100 |
---|---|---|
committer | Maarten Lankhorst <maarten.lankhorst@linux.intel.com> | 2019-01-28 12:55:40 +0100 |
commit | 4a94878797999d1d2f7d224c6cf2e2855cd3109a (patch) | |
tree | ab35184fd9b50df3f68de4861459cd62fbcd71f0 /tests/i915 | |
parent | e061af7bc37bfc77893dae8dab93d712d39d18df (diff) |
tests: Make a fb-tiling subtest to check for invalid tiling changes.
In 2013 we prevented changing the tiling of an active framebuffer:
commit 80075d492f8773209e26d11d6bb13ba624ef95a4
Author: Daniel Vetter <daniel.vetter@ffwll.ch>
Date: Wed Oct 9 21:23:52 2013 +0200
drm/i915: prevent tiling changes on framebuffer backing storage
After this we no longer have any use for the bad-tiling subtest, and
only have to make sure changing tiling on a fb is not allowed.
Remove the original bad-tiling subtest, and add a i915_fb_tiling
subtest for this case.
Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
[mlankhorst: Fix build system changes]
Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
Diffstat (limited to 'tests/i915')
-rw-r--r-- | tests/i915/fb_tiling.c | 46 |
1 files changed, 46 insertions, 0 deletions
diff --git a/tests/i915/fb_tiling.c b/tests/i915/fb_tiling.c new file mode 100644 index 00000000..7d5c3f1f --- /dev/null +++ b/tests/i915/fb_tiling.c @@ -0,0 +1,46 @@ +/* + * Copyright © 2018 Intel Corporation + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice (including the next + * paragraph) shall be included in all copies or substantial portions of the + * Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS + * IN THE SOFTWARE. + * + */ + +#include "igt.h" + +IGT_TEST_DESCRIPTION("Object tiling must be fixed after framebuffer creation."); + +igt_simple_main +{ + int drm_fd = drm_open_driver_master(DRIVER_INTEL); + struct igt_fb fb; + int ret; + + igt_create_fb(drm_fd, 512, 512, DRM_FORMAT_XRGB8888, + LOCAL_I915_FORMAT_MOD_X_TILED, &fb); + + ret = __gem_set_tiling(drm_fd, fb.gem_handle, I915_TILING_X, fb.strides[0]); + igt_assert_eq(ret, 0); + + ret = __gem_set_tiling(drm_fd, fb.gem_handle, I915_TILING_NONE, fb.strides[0]); + igt_assert_eq(ret, -EBUSY); + + igt_remove_fb(drm_fd, &fb); + close(drm_fd); +} |