On 03/05/2016 01:33 PM, Daniel Vetter wrote:
> On Wed, Mar 02, 2016 at 03:00:19PM +0100, Tomeu Vizoso wrote:
>> Move tests requiring tiled BOs to the end so they don't cause unrelated
>> subtests to be skipped when testing drivers with only dumb buffer
>> support.
>
> This uncovers a deficiency in igt skip infrastructure when you have
> disjoint sets of subtests with common requirements, e.g.
>
> {
> igt_require_intel();
>
> ... intel tiling subtests;
> }
>
> {
> igt_require_amd();
>
> ... amd tiling subtests;
> }
>
> The current rule is that a skip outside of subtest skips _all_ remaining
> subtests. We might want to introduce some kind of igt_subtest_group block
> so that only that group is skipped.
>
> Anyway, just a thought, nothing you'll have to tackle here ;-)
I think it's likely I will find more problematic cases as I get more
tests to run on !i915, so will think of it.
Thanks,
Tomeu
> -Daniel
>
>>
>> Signed-off-by: Tomeu Vizoso <tomeu.vizoso@xxxxxxxxxxxxx>
>> ---
>>
>> tests/kms_addfb_basic.c | 50 ++++++++++++++++++++++++++++++++++++++++++++-----
>> 1 file changed, 45 insertions(+), 5 deletions(-)
>>
>> diff --git a/tests/kms_addfb_basic.c b/tests/kms_addfb_basic.c
>> index e61d2502d78b..6dfaecfc38a7 100644
>> --- a/tests/kms_addfb_basic.c
>> +++ b/tests/kms_addfb_basic.c
>> @@ -156,16 +156,50 @@ static void pitch_tests(int fd)
>> }
>> }
>>
>> + igt_fixture
>> + gem_close(fd, gem_bo);
>> +}
>> +
>> +
>> +static void tiling_tests(int fd)
>> +{
>> + struct drm_mode_fb_cmd2 f = {};
>> + uint32_t tiled_x_bo;
>> + uint32_t tiled_y_bo;
>> +
>> + f.width = 512;
>> + f.height = 512;
>> + f.pixel_format = DRM_FORMAT_XRGB8888;
>> f.pitches[0] = 1024*4;
>>
>> + igt_fixture {
>> + tiled_x_bo = igt_create_bo_with_dimensions(fd, 1024, 1024,
>> + DRM_FORMAT_XRGB8888, LOCAL_I915_FORMAT_MOD_X_TILED,
>> + 1024*4, NULL, NULL, NULL);
>> + igt_assert(tiled_x_bo);
>> +
>> + tiled_y_bo = igt_create_bo_with_dimensions(fd, 1024, 1024,
>> + DRM_FORMAT_XRGB8888, LOCAL_I915_FORMAT_MOD_Y_TILED,
>> + 1024*4, NULL, NULL, NULL);
>> + igt_assert(tiled_y_bo);
>> +
>> + gem_bo = igt_create_bo_with_dimensions(fd, 1024, 1024,
>> + DRM_FORMAT_XRGB8888, 0, 0, NULL, NULL, NULL);
>> + igt_assert(gem_bo);
>> + }
>> +
>> + f.pitches[0] = 1024*4;
>> igt_subtest("basic-X-tiled") {
>> - gem_set_tiling(fd, gem_bo, I915_TILING_X, 1024*4);
>> + f.handles[0] = tiled_x_bo;
>> +
>> igt_assert(drmIoctl(fd, DRM_IOCTL_MODE_ADDFB2, &f) == 0);
>> igt_assert(drmIoctl(fd, DRM_IOCTL_MODE_RMFB, &f.fb_id) == 0);
>> f.fb_id = 0;
>> }
>>
>> igt_subtest("framebuffer-vs-set-tiling") {
>> + f.handles[0] = gem_bo;
>> +
>> gem_set_tiling(fd, gem_bo, I915_TILING_X, 1024*4);
>> igt_assert(drmIoctl(fd, DRM_IOCTL_MODE_ADDFB2, &f) == 0);
>> igt_assert(__gem_set_tiling(fd, gem_bo, I915_TILING_X, 512*4) == -EBUSY);
>> @@ -176,20 +210,24 @@ static void pitch_tests(int fd)
>>
>> f.pitches[0] = 512*4;
>> igt_subtest("tile-pitch-mismatch") {
>> - gem_set_tiling(fd, gem_bo, I915_TILING_X, 1024*4);
>> + f.handles[0] = tiled_x_bo;
>> +
>> igt_assert(drmIoctl(fd, DRM_IOCTL_MODE_ADDFB2, &f) == -1 &&
>> errno == EINVAL);
>> }
>>
>> f.pitches[0] = 1024*4;
>> igt_subtest("basic-Y-tiled") {
>> - gem_set_tiling(fd, gem_bo, I915_TILING_Y, 1024*4);
>> + f.handles[0] = tiled_y_bo;
>> +
>> igt_assert(drmIoctl(fd, DRM_IOCTL_MODE_ADDFB2, &f) == -1 &&
>> errno == EINVAL);
>> }
>>
>> - igt_fixture
>> - gem_close(fd, gem_bo);
>> + igt_fixture {
>> + gem_close(fd, tiled_x_bo);
>> + gem_close(fd, tiled_y_bo);
>> + }
>> }
>>
>> static void size_tests(int fd)
>> @@ -448,6 +486,8 @@ igt_main
>>
>> addfb25_ytile(fd, gen);
>>
>> + tiling_tests(fd);
>> +
>> igt_fixture
>> close(fd);
>> }
>> --
>> 2.5.0
>>
>> _______________________________________________
>> Intel-gfx mailing list
>> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
