[igt-dev] [PATCH i-g-t v2 1/2] tests/kms_plane_multiple: PCU messaging test
B S, Karthik
karthik.b.s at intel.com
Fri Sep 27 02:55:04 UTC 2019
On 9/25/2019 7:25 PM, Ville Syrjälä wrote:
> On Mon, Aug 05, 2019 at 12:24:53PM +0530, Karthik B S wrote:
>> Add a subtest that gives flips which alternate between least bandwidth
>> requirement and highest bandwidth requirement.
> And what are we trying to achieve with such a test? Also the patch
> subject talks about PCU communication which is nowhere mentioned in the
> rest of the patch.
It is a stress test, which intends to stress the system by providing
flips with continuously changing bandwidth requirements.
I agree that PCU communication is not mentioned in the patch.
Will update the commit message accordingly, by removing PCU communication.
>
>> v2: Make the test more generic. (Ville)
>> Rebase.
>>
>> Cc: Ville Syrjälä <ville.syrjala at linux.intel.com>
>> Signed-off-by: Karthik B S <karthik.b.s at intel.com>
>> Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
>> Reviewed-by: Mika Kahola <mika.kahola at intel.com>
>> ---
>> tests/kms_plane_multiple.c | 151 +++++++++++++++++++++++++++++++++++++++------
>> 1 file changed, 133 insertions(+), 18 deletions(-)
>>
>> diff --git a/tests/kms_plane_multiple.c b/tests/kms_plane_multiple.c
>> index 81ed45d..bed337c 100644
>> --- a/tests/kms_plane_multiple.c
>> +++ b/tests/kms_plane_multiple.c
>> @@ -34,6 +34,11 @@ IGT_TEST_DESCRIPTION("Test atomic mode setting with multiple planes.");
>>
>> #define SIZE_PLANE 256
>> #define SIZE_CURSOR 128
>> +#define SIZE_PLANE_LOW 10
>> +#define SIZE_PANE 5
>> +#define SMALL_SCREEN 0
>> +#define FULL_SCREEN 1
>> +#define TEST_BANDWIDTH 1
>> #define LOOP_FOREVER -1
>>
>> typedef struct {
>> @@ -48,9 +53,17 @@ typedef struct {
>> igt_crc_t ref_crc;
>> igt_pipe_crc_t *pipe_crc;
>> igt_plane_t **plane;
>> + unsigned int flag;
>> struct igt_fb *fb;
>> } data_t;
>>
>> +enum bandwidth {
>> + BW_PRIMARY_LOW,
>> + BW_PRIMARY_HIGH,
>> + BW_PRIMARY_LOW2,
>> + BW_HIGH,
>> + BW_INVALID,
>> +};
>> /* Command line parameters. */
>> struct {
>> int iterations;
>> @@ -97,7 +110,7 @@ static void test_fini(data_t *data, igt_output_t *output, int n_planes)
>>
>> static void
>> get_reference_crc(data_t *data, igt_output_t *output, enum pipe pipe,
>> - color_t *color, uint64_t tiling)
>> + color_t *color, uint64_t tiling, int BW)
>> {
>> drmModeModeInfo *mode;
>> igt_plane_t *primary;
>> @@ -110,7 +123,9 @@ get_reference_crc(data_t *data, igt_output_t *output, enum pipe pipe,
>>
>> mode = igt_output_get_mode(output);
>>
>> - igt_create_color_fb(data->drm_fd, mode->hdisplay, mode->vdisplay,
>> + igt_create_color_fb(data->drm_fd,
>> + BW ? mode->hdisplay : SIZE_PLANE_LOW,
>> + BW ? mode->vdisplay : SIZE_PLANE_LOW,
>> DRM_FORMAT_XRGB8888,
>> LOCAL_DRM_FORMAT_MOD_NONE,
>> color->red, color->green, color->blue,
>> @@ -270,6 +285,76 @@ prepare_planes(data_t *data, enum pipe pipe_id, color_t *color,
>> free((void*)suffle);
>> }
>>
>> +static void
>> +prepare_planes2(data_t *data, enum pipe pipe_id, color_t *color,
>> + uint64_t tiling, int max_planes, igt_output_t *output,
>> + enum bandwidth bandwidth)
>> +{
>> + drmModeModeInfo *mode;
>> + int hsize, vsize;
>> + int i;
>> + cairo_t *cr;
>> +
>> + igt_output_set_pipe(output, pipe_id);
>> + mode = igt_output_get_mode(output);
>> +
>> + switch (bandwidth) {
>> + case BW_PRIMARY_LOW:
>> + case BW_PRIMARY_LOW2:
>> + hsize = SIZE_PLANE_LOW;
>> + vsize = SIZE_PLANE_LOW;
>> + break;
>> + case BW_PRIMARY_HIGH:
>> + case BW_HIGH:
>> + hsize = mode->hdisplay;
>> + vsize = mode->vdisplay;
>> + break;
>> + default:
>> + igt_warn("Invalid BW\n");
>> + }
>> +
>> + for (i = 0; i < max_planes; i++) {
>> + igt_plane_t *plane = igt_output_get_plane(output, i);
>> + uint32_t format = DRM_FORMAT_XRGB8888;
>> +
>> + if (plane->type == DRM_PLANE_TYPE_CURSOR) {
>> + format = DRM_FORMAT_ARGB8888;
>> + tiling = LOCAL_DRM_FORMAT_MOD_NONE;
>> + hsize = SIZE_CURSOR;
>> + vsize = SIZE_CURSOR;
>> + }
>> +
>> + data->plane[i] = plane;
>> + igt_create_color_fb(data->drm_fd,
>> + hsize, vsize,
>> + format, tiling,
>> + color->red, color->green,
>> + color->blue,
>> + &data->fb[i]);
>> + igt_plane_set_position(data->plane[i], 0, 0);
>> +
>> + hsize -= SIZE_PANE;
>> +
>> + /* Create black color holes in all planes other than the cursor
>> + * and the topmost plane, so that all planes put together
>> + * produces a solid blue screen that matches with
>> + * the reference CRC.
>> + */
>> + if (i < (max_planes - 2) && bandwidth == BW_HIGH) {
>> + cr = igt_get_cairo_ctx(data->drm_fd, &data->fb[i]);
>> + igt_paint_color(cr, 0, 0, hsize, vsize, 0.0, 0.0, 0.0);
>> + igt_put_cairo_ctx(data->drm_fd, &data->fb[i], cr);
>> + }
>> +
>> + igt_plane_set_fb(data->plane[i], &data->fb[i]);
>> +
>> + if (bandwidth != BW_HIGH &&
>> + plane->type == DRM_PLANE_TYPE_PRIMARY)
>> + break;
>> +
>> + }
>> +}
>> +
>> /*
>> * Multiple plane position test.
>> * - We start by grabbing a reference CRC of a full blue fb being scanned
>> @@ -306,8 +391,6 @@ test_plane_position_with_output(data_t *data, enum pipe pipe,
>>
>> test_init(data, pipe, n_planes);
>>
>> - get_reference_crc(data, output, pipe, &blue, tiling);
>> -
>> /* Find out how many planes are allowed simultaneously */
>> do {
>> c++;
>> @@ -325,27 +408,51 @@ test_plane_position_with_output(data_t *data, enum pipe pipe,
>> c--;
>>
>> igt_info("Testing connector %s using pipe %s with %d planes %s with seed %d\n",
>> - igt_output_name(output), kmstest_pipe_name(pipe), c,
>> - info, opt.seed);
>> + igt_output_name(output), kmstest_pipe_name(pipe), c,
>> + info, opt.seed);
>> +
>> + if (data->flag == TEST_BANDWIDTH) {
>> + for (i = BW_PRIMARY_LOW; i < BW_INVALID; i++) {
>> + if (i == BW_PRIMARY_LOW || i == BW_PRIMARY_LOW2)
>> + get_reference_crc(data, output, pipe, &blue,
>> + tiling, SMALL_SCREEN);
>> + else
>> + get_reference_crc(data, output, pipe, &blue,
>> + tiling, FULL_SCREEN);
>> +
>> + prepare_planes2(data, pipe, &blue, tiling,
>> + c, output, i);
>> + igt_display_commit2(&data->display, COMMIT_ATOMIC);
>> +
>> + igt_pipe_crc_get_current(data->display.drm_fd,
>> + data->pipe_crc, &crc);
>> +
>> + igt_assert_crc_equal(&data->ref_crc, &crc);
>> + }
>> + } else {
>> + i = 0;
>> + get_reference_crc(data, output, pipe, &blue,
>> + tiling, FULL_SCREEN);
>>
>> - i = 0;
>> - while (i < iterations || loop_forever) {
>> - /* randomize planes and set up the holes */
>> - prepare_planes(data, pipe, &blue, tiling, c, output);
>> + while (i < iterations || loop_forever) {
>> + prepare_planes(data, pipe, &blue, tiling,
>> + c, output);
>>
>> - igt_display_commit2(&data->display, COMMIT_ATOMIC);
>> + igt_display_commit2(&data->display, COMMIT_ATOMIC);
>>
>> - igt_pipe_crc_get_current(data->display.drm_fd, data->pipe_crc, &crc);
>> + igt_pipe_crc_get_current(data->display.drm_fd,
>> + data->pipe_crc, &crc);
>>
>> - for_each_plane_on_pipe(&data->display, pipe, plane)
>> - igt_plane_set_fb(plane, NULL);
>> + for_each_plane_on_pipe(&data->display, pipe, plane)
>> + igt_plane_set_fb(plane, NULL);
>>
>> - for (int x = 0; x < c; x++)
>> - igt_remove_fb(data->drm_fd, &data->fb[x]);
>> + for (int x = 0; x < c; x++)
>> + igt_remove_fb(data->drm_fd, &data->fb[x]);
>>
>> - igt_assert_crc_equal(&data->ref_crc, &crc);
>> + igt_assert_crc_equal(&data->ref_crc, &crc);
>>
>> - i++;
>> + i++;
>> + }
>> }
>>
>> test_fini(data, output, n_planes);
>> @@ -377,6 +484,14 @@ run_tests_for_pipe(data_t *data, enum pipe pipe)
>> igt_require(data->display.pipes[pipe].n_planes > 0);
>> }
>>
>> +
>> + data->flag = TEST_BANDWIDTH;
>> +
>> + igt_subtest_f("atomic-pipe-%s-tiling-none_bw", kmstest_pipe_name(pipe))
>> + test_plane_position(data, pipe, LOCAL_DRM_FORMAT_MOD_NONE);
>> +
>> + data->flag = 0;
>> +
>> igt_subtest_f("atomic-pipe-%s-tiling-x", kmstest_pipe_name(pipe))
>> test_plane_position(data, pipe, LOCAL_I915_FORMAT_MOD_X_TILED);
>>
>> --
>> 2.7.4
More information about the igt-dev
mailing list