[Intel-gfx] [PATCH i-g-t v2] tests/kms_concurrent: Concurrent and interruptible subtests for atomic
Maarten Lankhorst
maarten.lankhorst at linux.intel.com
Tue Apr 11 08:18:45 UTC 2017
Op 11-04-17 om 09:42 schreef Kahola, Mika:
>
>> -----Original Message-----
>> From: Maarten Lankhorst [mailto:maarten.lankhorst at linux.intel.com]
>> Sent: Monday, April 10, 2017 3:39 PM
>> To: Kahola, Mika <mika.kahola at intel.com>; intel-gfx at lists.freedesktop.org
>> Subject: Re: [PATCH i-g-t v2] tests/kms_concurrent: Concurrent and interruptible
>> subtests for atomic
>>
>> Op 10-04-17 om 14:11 schreef Mika Kahola:
>>> On Wed, 2017-03-29 at 11:53 +0200, Maarten Lankhorst wrote:
>>>> Op 15-03-17 om 09:43 schreef Mika Kahola:
>>>>> This test case introduces concurrently running test cases for atomic
>>>>> modesetting.
>>>>>
>>>>> The first test or thread draws blue backround with black holes on
>>>>> it.
>>>>> These holes are covered by rectangular, blue planes that are placed
>>>>> randomly like in test case 'kms_plane_multiple'.
>>>>>
>>>>> The second thread changes resolution from higher to lower one and
>>>>> back.
>>>>> The delay between resolution changes is randomly selected between
>>>>> 20 and
>>>>> 50 milliseconds.
>>>>>
>>>>> The test runs by default 32 iterations to increase the coverage.
>>>>>
>>>>> v2: use igt_fork instead of pthreads to create concurrent test runs
>>>>> (Maarten)
>>>>>
>>>>> Signed-off-by: Mika Kahola <mika.kahola at intel.com>
>>>>> ---
>>>>> tests/Makefile.sources | 1 +
>>>>> tests/kms_concurrent.c | 630
>>>>> +++++++++++++++++++++++++++++++++++++++++++++++++
>>>>> 2 files changed, 631 insertions(+)
>>>>> create mode 100644 tests/kms_concurrent.c
>>>>>
>>>>> <snip>
>>>>> +static int
>>>>> +display_commit_mode(data_t *data, igt_crc_t *crc) {
>>>>> + char buf[256];
>>>>> + struct drm_event *e = (void *)buf;
>>>>> + int n, ret;
>>>>> + int flags = DRM_MODE_PAGE_FLIP_EVENT |
>>>>> DRM_MODE_ATOMIC_ALLOW_MODESET |
>> DRM_MODE_ATOMIC_NONBLOCK;
>>>>> +
>>>>> + ret = igt_display_try_commit_atomic(&data->display,
>>>>> + flags,
>>>>> + NULL);
>>>>> + igt_skip_on(ret != 0);
>>>>> +
>>>>> + igt_set_timeout(1, "Stuck on page flip");
>>>>> + ret = read(data->display.drm_fd, buf, sizeof(buf));
>>>>> + igt_assert(ret >= 0);
>>>>> +
>>>>> + igt_assert_eq(e->type, DRM_EVENT_FLIP_COMPLETE);
>>>>> + igt_reset_timeout();
>>>>> +
>>>>> + return n;
>>>>> +}
>>>>> +
>>>>> +static void
>>>>> +test_grab_crc(data_t *data, color_t *color, igt_crc_t *crc /* out
>>>>> */)
>>>>> +{
>>>>> + drmModeModeInfo *mode;
>>>>> + igt_plane_t *primary;
>>>>> + int ret;
>>>>> +
>>>>> + igt_output_set_pipe(data->output, data->pipe);
>>>>> +
>>>>> + primary = igt_output_get_plane_type(data->output,
>>>>> DRM_PLANE_TYPE_PRIMARY);
>>>>> + data->plane[primary->index] = primary;
>>>>> +
>>>>> + mode = igt_output_get_mode(data->output);
>>>>> +
>>>>> + igt_create_color_fb(data->drm_fd, mode->hdisplay, mode-
>>>>>> vdisplay,
>>>>> + DRM_FORMAT_XRGB8888,
>>>>> + LOCAL_I915_FORMAT_MOD_X_TILED,
>>>>> + color->red, color->green, color->blue,
>>>>> + &data->fb[primary->index]);
>>>>> +
>>>>> + igt_plane_set_fb(data->plane[primary->index], &data-
>>>>>> fb[primary->index]);
>>>>> +
>>>>> + ret = igt_display_try_commit2(&data->display,
>>>>> COMMIT_ATOMIC);
>>>>> + igt_skip_on(ret != 0);
>>>>> +}
>>>>> +
>>>>> +/*
>>>>> + * Multiple plane position test.
>>>>> + * - We start by grabbing a reference CRC of a full blue fb
>>>>> being scanned
>>>>> + * out on the primary plane
>>>>> + * - Then we scannout number of planes:
>>>>> + * * the primary plane uses a blue fb with a black rectangle
>>>>> hole
>>>>> + * * planes, on top of the primary plane, with a blue fb that
>>>>> is set-up
>>>>> + * to cover the black rectangles of the primary plane fb
>>>>> + * The resulting CRC should be identical to the reference CRC
>>>>> + */
>>>>> +
>>>>> +static void
>>>>> +create_fb_for_mode_position(data_t *data, drmModeModeInfo *mode,
>>>>> + color_t *color, int *rect_x, int
>>>>> *rect_y,
>>>>> + int *rect_w, int *rect_h, uint64_t
>>>>> tiling,
>>>>> + int max_planes)
>>>>> +{
>>>>> + unsigned int fb_id;
>>>>> + cairo_t *cr;
>>>>> + igt_plane_t *primary;
>>>>> +
>>>>> + primary = igt_output_get_plane_type(data->output,
>>>>> DRM_PLANE_TYPE_PRIMARY);
>>>>> +
>>>>> + fb_id = igt_create_fb(data->drm_fd,
>>>>> + mode->hdisplay, mode->vdisplay,
>>>>> + DRM_FORMAT_XRGB8888,
>>>>> + tiling,
>>>>> + &data->fb[primary->index]);
>>>>> + igt_assert(fb_id);
>>>>> +
>>>>> + cr = igt_get_cairo_ctx(data->drm_fd, &data->fb[primary-
>>>>>> index]);
>>>>> + igt_paint_color(cr, rect_x[0], rect_y[0],
>>>>> + mode->hdisplay, mode->vdisplay,
>>>>> + color->red, color->green, color->blue);
>>>>> +
>>>>> + for (int i = 0; i < max_planes; i++) {
>>>>> + if (data->plane[i]->type ==
>>>>> DRM_PLANE_TYPE_PRIMARY)
>>>>> + continue;
>>>>> + igt_paint_color(cr, rect_x[i], rect_y[i],
>>>>> + rect_w[i], rect_h[i], 0.0, 0.0,
>>>>> 0.0);
>>>>> + }
>>>>> +
>>>>> + igt_assert(cairo_status(cr) == 0);
>>>>> + cairo_destroy(cr);
>>>>> +}
>>>>> +
>>>>> +static void
>>>>> +prepare_planes(data_t *data, color_t *color, int max_planes) {
>>>>> + drmModeModeInfo *mode;
>>>>> + igt_pipe_t *pipe;
>>>>> + igt_plane_t *primary;
>>>>> + int *x;
>>>>> + int *y;
>>>>> + int *size;
>>>>> + int i;
>>>>> +
>>>>> + igt_output_set_pipe(data->output, data->pipe);
>>>>> + primary = igt_output_get_plane_type(data->output,
>>>>> DRM_PLANE_TYPE_PRIMARY);
>>>>> + pipe = primary->pipe;
>>>>> +
>>>>> + x = malloc(pipe->n_planes * sizeof(*x));
>>>>> + igt_assert_f(x, "Failed to allocate %ld bytes for variable
>>>>> x\n", (long int) (pipe->n_planes * sizeof(*x)));
>>>>> + y = malloc(pipe->n_planes * sizeof(*y));
>>>>> + igt_assert_f(y, "Failed to allocate %ld bytes for variable
>>>>> y\n", (long int) (pipe->n_planes * sizeof(*y)));
>>>>> + size = malloc(pipe->n_planes * sizeof(*size));
>>>>> + igt_assert_f(size, "Failed to allocate %ld bytes for
>>>>> variable size\n", (long int) (pipe->n_planes * sizeof(*size)));
>>>>> +
>>>>> + mode = igt_output_get_mode(data->output);
>>>>> +
>>>>> + /* planes with random positions */
>>>>> + x[primary->index] = 0;
>>>>> + y[primary->index] = 0;
>>>>> + for (i = 0; i < max_planes; i++) {
>>>>> + igt_plane_t *plane = igt_output_get_plane(data-
>>>>>> output, i);
>>>>> +
>>>>> + if (plane->type == DRM_PLANE_TYPE_PRIMARY)
>>>>> + continue;
>>>>> + else if (plane->type == DRM_PLANE_TYPE_CURSOR)
>>>>> + size[i] = SIZE_CURSOR;
>>>>> + else
>>>>> + size[i] = SIZE_PLANE;
>>>>> +
>>>>> + x[i] = rand() % (mode->hdisplay - size[i]);
>>>>> + y[i] = rand() % (mode->vdisplay - size[i]);
>>>>> +
>>>>> + data->plane[i] = plane;
>>>>> +
>>>>> + igt_create_color_fb(data->drm_fd,
>>>>> + size[i], size[i],
>>>>> + data->plane[i]->type ==
>>>>> DRM_PLANE_TYPE_CURSOR ? DRM_FORMAT_ARGB8888 :
>> DRM_FORMAT_XRGB8888,
>>>>> + data->plane[i]->type ==
>>>>> DRM_PLANE_TYPE_CURSOR ? LOCAL_DRM_FORMAT_MOD_NONE :
>>>>> LOCAL_I915_FORMAT_MOD_X_TILED,
>>>>> + color->red, color->green,
>>>>> color->blue,
>>>>> + &data->fb[i]);
>>>>> +
>>>>> + igt_plane_set_position(data->plane[i], x[i],
>>>>> y[i]);
>>>>> + igt_plane_set_fb(data->plane[i], &data->fb[i]);
>>>>> + }
>>>>> +
>>>>> + /* primary plane */
>>>>> + data->plane[primary->index] = primary;
>>>>> + create_fb_for_mode_position(data, mode, color, x, y, size,
>>>>> size,
>>>>> + LOCAL_I915_FORMAT_MOD_X_TILED,
>>>>> max_planes);
>>>>> + igt_plane_set_fb(data->plane[primary->index], &data-
>>>>>> fb[primary->index]);
>>>>> +}
>>>>> +
>>>>> +static void
>>>>> +test_plane_position_with_output(data_t *data) {
>>>>> + igt_crc_t *crc;
>>>>> + color_t blue = { 0.0f, 0.0f, 1.0f };
>>>>> + int i;
>>>>> + int iterations = opt.iterations < 1 ? 1 : opt.iterations;
>>>>> + bool loop_forever;
>>>>> + char info[256];
>>>>> + int max_planes = data->display.pipes[data->pipe].n_planes;
>>>>> +
>>>>> + if (opt.iterations == LOOP_FOREVER) {
>>>>> + loop_forever = true;
>>>>> + sprintf(info, "forever");
>>>>> + } else {
>>>>> + loop_forever = false;
>>>>> + sprintf(info, "for %d %s",
>>>>> + iterations, iterations > 1 ? "iterations"
>>>>> : "iteration");
>>>>> + }
>>>>> +
>>>>> + igt_info("Testing connector %s using pipe %s with %d
>>>>> planes %s with seed %d\n",
>>>>> + igt_output_name(data->output),
>>>>> kmstest_pipe_name(data->pipe), max_planes,
>>>>> + info, opt.seed);
>>>>> +
>>>>> + i = 0;
>>>>> + while (i < iterations || loop_forever) {
>>>>> + prepare_planes(data, &blue, max_planes);
>>>>> +
>>>>> + display_commit_mode(data, crc);
>>>>> +
>>>>> + i++;
>>>>> + }
>>>>> +}
>>>>> +
>>>>> +static drmModeConnector *
>>>>> +get_connector_connected(int drm_fd) {
>>>>> + drmModeRes *resources = drmModeGetResources(drm_fd);
>>>>> + drmModeConnector *connector;
>>>>> + int i;
>>>>> +
>>>>> + for (i = 0; i < resources->count_connectors; i++) {
>>>>> + connector = drmModeGetConnector(drm_fd,
>>>>> + resources-
>>>>>> connectors[i]);
>>>>> +
>>>>> + if (connector->connection == DRM_MODE_CONNECTED)
>>>>> + break;
>>>>> +
>>>>> + drmModeFreeConnector(connector);
>>>>> + connector = NULL;
>>>>> + }
>>>>> +
>>>>> + drmModeFreeResources(resources);
>>>>> +
>>>>> + return connector;
>>>>> +}
>>>>> +
>>>>> +static int get_crtc_for_encoder(drmModeRes *resources,
>>>>> + drmModeEncoder *encoder)
>>>>> +{
>>>>> + int mask, id;
>>>>> + int i;
>>>>> + bool found;
>>>>> +
>>>>> + for (i = 0; i < resources->count_crtcs; i++) {
>>>>> + mask = 1 << i;
>>>>> + id = resources->crtcs[i];
>>>>> +
>>>>> + found = encoder->possible_crtcs & mask;
>>>>> + if (found)
>>>>> + break;
>>>>> + }
>>>>> +
>>>>> + if (!found)
>>>>> + return -EINVAL;
>>>>> +
>>>>> + return id;
>>>>> +}
>>>>> +
>>>>> +static int get_crtc_for_connector(int drm_fd, drmModeRes
>>>>> *resources,
>>>>> + drmModeConnector *connector)
>>>>> +{
>>>>> + drmModeEncoder *encoder;
>>>>> + int encoder_id;
>>>>> + int crtc_id;
>>>>> + int i;
>>>>> +
>>>>> + for (i = 0; i < connector->count_encoders; i++) {
>>>>> + encoder_id = connector->encoders[i];
>>>>> + encoder = drmModeGetEncoder(drm_fd, encoder_id);
>>>>> +
>>>>> + if (encoder) {
>>>>> + crtc_id = get_crtc_for_encoder(resources,
>>>>> encoder);
>>>>> +
>>>>> + drmModeFreeEncoder(encoder);
>>>>> +
>>>>> + if (crtc_id != 0)
>>>>> + break;
>>>>> + }
>>>>> + }
>>>>> +
>>>>> + if (crtc_id == 0)
>>>>> + return -EINVAL;
>>>>> +
>>>>> + return crtc_id;
>>>>> +}
>>>>> +
>>>>> +static int get_crtc(int drm_fd)
>>>>> +{
>>>>> + drmModeRes *resources;
>>>>> + drmModeConnector *connector;
>>>>> + drmModeEncoder *encoder;
>>>>> + int i, id;
>>>>> +
>>>>> + resources = drmModeGetResources(drm_fd);
>>>>> + connector = get_connector_connected(drm_fd);
>>>>> +
>>>>> + for (i = 0; i < resources->count_encoders; i++) {
>>>>> + encoder = drmModeGetEncoder(drm_fd, resources-
>>>>>> encoders[i]);
>>>>> +
>>>>> + if (encoder->encoder_id == connector->encoder_id)
>>>>> + break;
>>>>> +
>>>>> + drmModeFreeEncoder(encoder);
>>>>> + encoder = NULL;
>>>>> + }
>>>>> +
>>>>> + if (encoder)
>>>>> + return encoder->crtc_id;
>>>>> +
>>>>> + id = get_crtc_for_connector(drm_fd, resources, connector);
>>>>> + igt_assert(id != 0);
>>>>> +
>>>>> + return id;
>>>>> +}
>>>>> +
>>>>> +static drmModeModeInfo *
>>>>> +get_lowres_mode(data_t *data, drmModeModeInfo *mode_default) {
>>>>> + drmModeRes *resources = drmModeGetResources(data->drm_fd);
>>>>> + drmModeModeInfo std_1024_mode = {
>>>>> + .clock = 65000,
>>>>> + .hdisplay = 1024,
>>>>> + .hsync_start = 1048,
>>>>> + .hsync_end = 1184,
>>>>> + .htotal = 1344,
>>>>> + .hskew = 0,
>>>>> + .vdisplay = 768,
>>>>> + .vsync_start = 771,
>>>>> + .vsync_end = 777,
>>>>> + .vtotal = 806,
>>>>> + .vscan = 0,
>>>>> + .vrefresh = 60,
>>>>> + .flags = 0xA,
>>>>> + .type = 0x40,
>>>>> + .name = "Custom 1024x768",
>>>>> + };
>>>>> + drmModeModeInfo *mode;
>>>>> + drmModeConnector *connector;
>>>>> + bool found;
>>>>> + int limit = mode_default->vdisplay-SIZE_PLANE;
>>>>> + int i, j;
>>>>> +
>>>>> + if (!resources) {
>>>>> + igt_warn("drmModeGetResources failed: %s\n",
>>>>> strerror(errno));
>>>>> + return mode;
>>>>> + }
>>>>> +
>>>>> + found = false;
>>>>> + for (i = 0; i < resources->count_connectors; i++) {
>>>>> +
>>>>> + connector = drmModeGetConnectorCurrent(data-
>>>>>> drm_fd,
>>>>> + resources-
>>>>>> connectors[i]);
>>>>> +
>>>>> + if (!connector) {
>>>>> + igt_warn("could not get connector %i:
>>>>> %s\n",
>>>>> + resources->connectors[i],
>>>>> strerror(errno));
>>>>> + continue;
>>>>> + }
>>>>> +
>>>>> + if (!connector->count_modes)
>>>>> + continue;
>>>>> +
>>>>> + for (j = 0; j < connector->count_modes; j++) {
>>>>> + mode = &connector->modes[j];
>>>>> +
>>>>> + if (mode->vdisplay < limit) {
>>>>> + found = true;
>>>>> + break;
>>>>> + }
>>>>> + }
>>>>> +
>>>>> + drmModeFreeConnector(connector);
>>>>> + }
>>>>> +
>>>>> + drmModeFreeResources(resources);
>>>>> +
>>>>> + if (!found)
>>>>> + mode = &std_1024_mode;
>>>>> +
>>>>> + return mode;
>>>>> +}
>>>> Still not completely happy about this part.
>>>>
>>>> I would look at data->output->config.connector for modes, and return
>>>> 1024 mode if nothing's found.
>>>>
>>>> Saves a whole lot of drm calls.
>>> True. A lot of lines can be cleaned out.
>>>>> +static void
>>>>> +test_resolution_with_output(data_t *data) {
>>>>> + drmModeModeInfo *mode_hi, *mode_lo;
>>>>> + int ret;
>>>>> + int iterations = opt.iterations < 1 ? 1 : opt.iterations;
>>>>> + bool loop_forever;
>>>>> + int i;
>>>>> +
>>>>> + if (opt.iterations == LOOP_FOREVER)
>>>>> + loop_forever = true;
>>>>> + else
>>>>> + loop_forever = false;
>>>>> +
>>>>> + igt_info("Testing resolution with connector %s using pipe
>>>>> %s with seed %d\n",
>>>>> + igt_output_name(data->output),
>>>>> kmstest_pipe_name(data->pipe), opt.seed);
>>>>> +
>>>>> + mode_hi = igt_output_get_mode(data->output);
>>>>> + mode_lo = get_lowres_mode(data, mode_hi);
>>>>> +
>>>>> + i = 0;
>>>>> + while (i < iterations || loop_forever) {
>>>>> + /* switch to lower resolution */
>>>>> + ret = drmModeSetCrtc(data->drm_fd, data->crtc_id,
>>>>> data->fb->fb_id, 0, 0,
>>>>> + &data->connector_id, 1,
>>>>> mode_lo);
>>>>> + igt_assert_eq(ret, 0);
>>>>> +
>>>>> + /* switch back to higher resolution */
>>>>> + ret = drmModeSetCrtc(data->drm_fd, data->crtc_id,
>>>>> data->fb->fb_id, 0, 0,
>>>>> + &data->connector_id, 1,
>>>>> mode_hi);
>>>>> + igt_assert_eq(ret, 0);
>>>>> +
>>>>> + i++;
>>>>> + }
>>>>> +}
>>>> Use igt_output_override_mode + igt_display calls? With the thread
>>>> being a fork now, you can use igt_display. :)
>>> For some reason I keep receiving an error when using
>>> igt_output_override_mode + igt_display calls. The error is
>>>
>>> igt-kms-CRITICAL: Failed assertion:
>>> drmModeDestroyPropertyBlob(display-
>>>> drm_fd, *blob) == 0
>>> igt-kms-CRITICAL: Last errno: 1, Operation not permitted
>>>
>>> Maybe both threads try to access this and this request is not granted?
>>> With drmModeSetCrtc() call I don't see this behavior. Maybe stick with
>>> raw calls instead?
>> data->display.pipes[pipe].mode_blob = 0; in one of the forks of the test should
>> fix it.
> This seems to fix the issue. I'll throw a revised patch of the test shortly.
>
>> -EPERM means that the blob was already destroyed by this fd, but still used
>> globally.
>>
>> kms_rmfb does similar, maybe we should ignore failures?
> Maybe we could ignore these failures. Any harm done elsewhere if we do so?
I'm hesitant to ignore them, for the same reason as calling free() twice is a bad idea, we may end up freeing a different unknown blob.
~Maarten
More information about the Intel-gfx
mailing list