Re: [PATCH i-g-t v4 5/6] tests/kms_ccs: Test case where CCS is on a different BO

Ben Widawsky
2017-10-03T16:38:20Z

On 17-09-27 15:34:18, Gabriel Krisman Bertazi wrote:
>Signed-off-by: Gabriel Krisman Bertazi <krisman@collabora.co.uk>

Did someone recommend this test? While we have some hardware limitations on
current generations that make it difficult to use multiple BOs, it's certainly
not impossible, and future HW might make this limitation go away entirely.

I'd be fine if you wanted to use an invalid handle instead of a valid, but
different handle.

>---
> tests/kms_ccs.c | 17 ++++++++++++++---
> 1 file changed, 14 insertions(+), 3 deletions(-)
>
>diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
>index 35dfcca6be14..2d7105eb1323 100644
>--- a/tests/kms_ccs.c
>+++ b/tests/kms_ccs.c
>@@ -34,10 +34,11 @@ enum test_flags {
> TEST_BAD_PIXEL_FORMAT = 1 << 3,
> TEST_BAD_ROTATION_90 = 1 << 4,
> TEST_NO_AUX_BUFFER = 1 << 5,
>+ TEST_BAD_CCS_HANDLE = 1 << 6,
> };
>
> #define TEST_FAIL_ON_ADDFB2 \
>- (TEST_BAD_PIXEL_FORMAT | TEST_NO_AUX_BUFFER)
>+ (TEST_BAD_PIXEL_FORMAT | TEST_NO_AUX_BUFFER | TEST_BAD_CCS_HANDLE)
>
> enum test_fb_flags {
> FB_COMPRESSED = 1 << 0,
>@@ -275,6 +276,7 @@ static void generate_fb(data_t *data, struct igt_fb *fb,
> unsigned int size[2];
> uint64_t modifier;
> int ret;
>+ uint32_t ccs_handle;
>
> /* Use either compressed or Y-tiled to test. However, given the lack of
> * available bandwidth, we use linear for the primary plane when
>@@ -325,10 +327,15 @@ static void generate_fb(data_t *data, struct igt_fb *fb,
> size[1] = ccs_pitches * ALIGN(ccs_height, 32);
>
> f.handles[0] = gem_create(data->drm_fd, size[0] + size[1]);
>+ if (data->flags & TEST_BAD_CCS_HANDLE) {
>+ /* Put the CCS buffer on a different BO. */
>+ ccs_handle = gem_create(data->drm_fd, size[0] + size[1]);
>+ } else
>+ ccs_handle = f.handles[0];
>
> if (!(data->flags & TEST_NO_AUX_BUFFER)) {
> f.modifier[1] = modifier;
>- f.handles[1] = f.handles[0];
>+ f.handles[1] = ccs_handle;
> f.pitches[1] = ccs_pitches;
> f.offsets[1] = ccs_offsets;
>
>@@ -458,7 +465,8 @@ static void test_output(data_t *data)
>
> if (data->flags & TEST_BAD_PIXEL_FORMAT ||
> data->flags & TEST_BAD_ROTATION_90 ||
>- data->flags & TEST_NO_AUX_BUFFER) {
>+ data->flags & TEST_NO_AUX_BUFFER ||
>+ data->flags & TEST_BAD_CCS_HANDLE) {
> try_config(data, fb_flags | FB_COMPRESSED);
> }
>
>@@ -532,6 +540,9 @@ igt_main
> igt_subtest_f("pipe-%s-missing-ccs-buffer", pipe_name)
> test_output(&data);
>
>+ data.flags = TEST_BAD_CCS_HANDLE;
>+ igt_subtest_f("pipe-%s-ccs-on-another-bo", pipe_name)
>+ test_output(&data);
> }
>
> igt_fixture
>--
>2.11.0
>
>_______________________________________________
>Intel-gfx mailing list
>Intel-gfx@lists.freedesktop.org
>https://lists.freedesktop.org/mailman/listinfo/intel-gfx