Re: [PATCH] igt/kms_rotation_crc: Add a new subtest to exhaustively test for fence leaks (v2)

[Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>]

On 04/11/15 02:25, Vivek Kasireddy wrote:
> In this subtest, as a first step, MAX_FENCES+1 number of framebuffers are
> created backed up by objects that have multiple GGTT views (normal and
> rotated). Next, we have the i915 driver instantiate a normal view followed
> by a rotated view. We continue doing the above MAX_FENCES + 1 times.
>
> v2:
> - Add a igt_require() to check if there is enough GTT space left for
>    MAX_FENCES+1 framebuffers. (Tvrtko)
> - Make data2 local to test_plane_rotation_exhaust_fences(). (Tvrtko)
> - If there is a failure, deallocate all the previously allocated
>    framebuffers before asserting.
>
> Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
> Signed-off-by: Vivek Kasireddy <vivek.kasireddy@intel.com>
> ---
>   tests/kms_rotation_crc.c | 106 +++++++++++++++++++++++++++++++++++++++++++++++
>   1 file changed, 106 insertions(+)
>
> diff --git a/tests/kms_rotation_crc.c b/tests/kms_rotation_crc.c
> index ed6eeef..154c6a1 100644
> --- a/tests/kms_rotation_crc.c
> +++ b/tests/kms_rotation_crc.c
> @@ -25,6 +25,7 @@
>   #include "igt.h"
>   #include <math.h>
>
> +#define MAX_FENCES 32
>
>   typedef struct {
>   	int gfx_fd;
> @@ -376,6 +377,106 @@ static void test_plane_rotation_ytiled_obj(data_t *data, enum igt_plane plane_ty
>   	igt_assert(ret == 0);
>   }
>
> +static void test_plane_rotation_exhaust_fences(data_t *data, enum igt_plane plane_type)
> +{
> +	igt_display_t *display = &data->display;
> +	uint64_t tiling = LOCAL_I915_FORMAT_MOD_Y_TILED;
> +	uint32_t format = DRM_FORMAT_XRGB8888;
> +	int bpp = igt_drm_format_to_bpp(format);
> +	enum igt_commit_style commit = COMMIT_LEGACY;
> +	int fd = data->gfx_fd;
> +	igt_output_t *output = &display->outputs[0];
> +	igt_plane_t *plane;
> +	drmModeModeInfo *mode;
> +	data_t data2[MAX_FENCES+1] = {};
> +	unsigned int stride, size, w, h;
> +	uint32_t gem_handle;
> +	uint64_t total_aperture_size, total_fbs_size;
> +	int i, ret;
> +
> +	igt_require(output != NULL && output->valid == true);
> +
> +	plane = igt_output_get_plane(output, plane_type);
> +	igt_require(igt_plane_supports_rotation(plane));
> +
> +	if (plane_type == IGT_PLANE_PRIMARY || plane_type == IGT_PLANE_CURSOR) {
> +		igt_require(data->display.has_universal_planes);
> +		commit = COMMIT_UNIVERSAL;
> +	}
> +
> +	mode = igt_output_get_mode(output);
> +	w = mode->hdisplay;
> +	h = mode->vdisplay;
> +
> +	for (stride = 512; stride < (w * bpp / 8); stride *= 2)
> +		;
> +	for (size = 1024*1024; size < stride * h; size *= 2)
> +		;
> +
> +	/*
> +	 * Make sure there is atleast 90% of the available GTT space left
> +	 * for creating (MAX_FENCES+1) framebuffers.
> +	 */
> +	total_fbs_size = size * (MAX_FENCES + 1);
> +	total_aperture_size = gem_available_aperture_size(fd);
> +	igt_require(total_fbs_size < total_aperture_size * 0.9);
> +
> +	igt_plane_set_fb(plane, NULL);
> +	igt_display_commit(display);
> +
> +	for (i = 0; i < MAX_FENCES + 1; i++) {
> +		gem_handle = gem_create(fd, size);
> +		ret = __gem_set_tiling(fd, gem_handle, I915_TILING_Y, stride);
> +		if (ret) {
> +			igt_warn("failed to set tiling\n");
> +			goto err_alloc;

If it hits this path it leaks the gem_handle I think.

> +		}
> +
> +		ret = (__kms_addfb(fd, gem_handle, w, h, stride,
> +		       format, tiling, LOCAL_DRM_MODE_FB_MODIFIERS,
> +		       &data2[i].fb.fb_id));
> +		if (ret) {
> +			igt_warn("failed to create framebuffer\n");
> +			goto err_alloc;

And here.

> +		}
> +
> +		data2[i].fb.width = w;
> +		data2[i].fb.height = h;
> +		data2[i].fb.gem_handle = gem_handle;
> +
> +		igt_plane_set_fb(plane, &data2[i].fb);
> +		igt_plane_set_rotation(plane, IGT_ROTATION_0);
> +
> +		ret = igt_display_try_commit2(display, commit);
> +		if (ret) {
> +			igt_warn("failed to commit unrotated fb\n");
> +			goto err_commit;
> +		}
> +
> +		igt_plane_set_rotation(plane, IGT_ROTATION_90);
> +
> +		drmModeObjectSetProperty(fd, plane->drm_plane->plane_id,
> +					 DRM_MODE_OBJECT_PLANE,
> +					 plane->rotation_property,
> +					 plane->rotation);
> +		ret = igt_display_try_commit2(display, commit);
> +		if (ret) {
> +			igt_warn("failed to commit hardware rotated fb\n");
> +			goto err_commit;
> +		}
> +	}
> +
> +err_alloc:
> +	i--;
> +err_commit:
> +	kmstest_restore_vt_mode();
> +
> +	for (; i >= 0; i--)
> +		igt_remove_fb(fd, &data2[i].fb);
> +
> +	igt_assert(ret == 0);
> +}
> +
>   igt_main
>   {
>   	data_t data = {};
> @@ -471,6 +572,11 @@ igt_main
>   		test_plane_rotation_ytiled_obj(&data, IGT_PLANE_PRIMARY);
>   	}
>
> +	igt_subtest_f("exhaust-fences") {
> +		igt_require(gen >= 9);
> +		test_plane_rotation_exhaust_fences(&data, IGT_PLANE_PRIMARY);
> +	}
> +
>   	igt_fixture {
>   		igt_display_fini(&data.display);
>   	}
>

Regards,

Tvrtko

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx