[Intel-gfx] [PATCH v3] i-g-t: Adding test case to test background color.
Chandra Konduru
chandra.konduru at intel.com
Mon Mar 30 13:44:32 PDT 2015
From: chandra konduru <chandra.konduru at intel.com>
Adding i-g-t test case to test display crtc background color.
v2:
- Added IGT_TEST_DESCRIPTION() (Thomas Wood)
- Added to .gitignore (Thomas Wood)
- Added additional details to function header (Thomas Wood)
- Simplified igt_main (Thomas Wood)
v3:
- rebased to latest master (me)
- took sleep calls out (Daniel)
- use new tiled types when calling igt_create_fb (me)
Signed-off-by: chandra konduru <chandra.konduru at intel.com>
---
lib/igt_kms.c | 61 ++++++++++++
lib/igt_kms.h | 4 +
tests/.gitignore | 1 +
tests/Android.mk | 1 +
tests/Makefile.sources | 1 +
tests/kms_crtc_background_color.c | 195 ++++++++++++++++++++++++++++++++++++++
6 files changed, 263 insertions(+)
create mode 100644 tests/kms_crtc_background_color.c
diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index 9c131f0..9c46e18 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -940,6 +940,22 @@ igt_plane_set_property(igt_plane_t *plane, uint32_t prop_id, uint64_t value)
DRM_MODE_OBJECT_PLANE, prop_id, value);
}
+static bool
+get_crtc_property(int drm_fd, uint32_t crtc_id, const char *name,
+ uint32_t *prop_id /* out */, uint64_t *value /* out */,
+ drmModePropertyPtr *prop /* out */)
+{
+ return kmstest_get_property(drm_fd, crtc_id, DRM_MODE_OBJECT_CRTC,
+ name, prop_id, value, prop);
+}
+
+static void
+igt_crtc_set_property(igt_output_t *output, uint32_t prop_id, uint64_t value)
+{
+ drmModeObjectSetProperty(output->display->drm_fd,
+ output->config.crtc->crtc_id, DRM_MODE_OBJECT_CRTC, prop_id, value);
+}
+
/*
* Walk a plane's property list to determine its type. If we don't
* find a type property, then the kernel doesn't support universal
@@ -1097,6 +1113,7 @@ void igt_display_init(igt_display_t *display, int drm_fd)
igt_assert(display->outputs);
for (i = 0; i < display->n_outputs; i++) {
+ int j;
igt_output_t *output = &display->outputs[i];
/*
@@ -1108,6 +1125,19 @@ void igt_display_init(igt_display_t *display, int drm_fd)
output->display = display;
igt_output_refresh(output);
+
+ for (j = 0; j < display->n_pipes; j++) {
+ uint64_t prop_value;
+ igt_pipe_t *pipe = &display->pipes[j];
+ if (output->config.crtc) {
+ get_crtc_property(display->drm_fd, output->config.crtc->crtc_id,
+ "background_color",
+ &pipe->background_property,
+ &prop_value,
+ NULL);
+ pipe->background = (uint32_t)prop_value;
+ }
+ }
}
drmModeFreePlaneResources(plane_resources);
@@ -1527,6 +1557,13 @@ static int igt_output_commit(igt_output_t *output,
pipe = igt_output_get_driving_pipe(output);
+ if (pipe->background_changed) {
+ igt_crtc_set_property(output, pipe->background_property,
+ pipe->background);
+
+ pipe->background_changed = false;
+ }
+
for (i = 0; i < pipe->n_planes; i++) {
igt_plane_t *plane = &pipe->planes[i];
@@ -1779,6 +1816,30 @@ void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation)
plane->rotation_changed = true;
}
+/**
+ * igt_crtc_set_background:
+ * @pipe: pipe pointer to which background color to be set
+ * @background: background color value in BGR 16bpc
+ *
+ * Sets background color for requested pipe. Color value provided here
+ * will be actually submitted at output commit time via "background_color"
+ * property.
+ * For example to get red as background, set background = 0x00000000FFFF.
+ */
+void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background)
+{
+ igt_display_t *display = pipe->display;
+
+ LOG(display, "%s.%d: crtc_set_background(%lu)\n",
+ kmstest_pipe_name(pipe->pipe),
+ pipe->pipe, background);
+
+ pipe->background = background;
+
+ pipe->background_changed = true;
+}
+
+
void igt_wait_for_vblank(int drm_fd, enum pipe pipe)
{
drmVBlank wait_vbl;
diff --git a/lib/igt_kms.h b/lib/igt_kms.h
index 565df14..84e8456 100644
--- a/lib/igt_kms.h
+++ b/lib/igt_kms.h
@@ -207,6 +207,9 @@ struct igt_pipe {
#define IGT_MAX_PLANES 4
int n_planes;
igt_plane_t planes[IGT_MAX_PLANES];
+ uint64_t background; /* Background color MSB BGR 16bpc LSB */
+ uint32_t background_changed : 1;
+ uint32_t background_property;
};
typedef struct {
@@ -252,6 +255,7 @@ void igt_plane_set_position(igt_plane_t *plane, int x, int y);
void igt_plane_set_size(igt_plane_t *plane, int w, int h);
void igt_plane_set_panning(igt_plane_t *plane, int x, int y);
void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation);
+void igt_crtc_set_background(igt_pipe_t *pipe, uint64_t background);
void igt_wait_for_vblank(int drm_fd, enum pipe pipe);
diff --git a/tests/.gitignore b/tests/.gitignore
index 402e062..0f08ce0 100644
--- a/tests/.gitignore
+++ b/tests/.gitignore
@@ -141,6 +141,7 @@ kms_setmode
kms_sink_crc_basic
kms_universal_plane
kms_vblank
+kms_crtc_background_color
multi-tests.txt
pm_lpsp
pm_rc6_residency
diff --git a/tests/Android.mk b/tests/Android.mk
index 814b846..96a32b9 100644
--- a/tests/Android.mk
+++ b/tests/Android.mk
@@ -75,6 +75,7 @@ else
kms_rotation_crc \
kms_force_connector \
kms_flip_event_leak \
+ kms_crtc_background_color \
kms_pwrite_crc
IGT_LOCAL_CFLAGS += -DANDROID_HAS_CAIRO=0
endif
diff --git a/tests/Makefile.sources b/tests/Makefile.sources
index a165978..5235b5f 100644
--- a/tests/Makefile.sources
+++ b/tests/Makefile.sources
@@ -82,6 +82,7 @@ TESTS_progs_M = \
kms_setmode \
kms_universal_plane \
kms_vblank \
+ kms_crtc_background_color \
pm_lpsp \
pm_rpm \
pm_rps \
diff --git a/tests/kms_crtc_background_color.c b/tests/kms_crtc_background_color.c
new file mode 100644
index 0000000..fb26411
--- /dev/null
+++ b/tests/kms_crtc_background_color.c
@@ -0,0 +1,195 @@
+/*
+ * Copyright © 2013,2014 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ *
+ */
+
+#include <math.h>
+
+#include "drmtest.h"
+#include "igt_debugfs.h"
+#include "igt_kms.h"
+#include "igt_core.h"
+
+IGT_TEST_DESCRIPTION("Test crtc background color feature");
+
+typedef struct {
+ int gfx_fd;
+ igt_display_t display;
+ struct igt_fb fb;
+ igt_crc_t ref_crc;
+ igt_pipe_crc_t *pipe_crc;
+} data_t;
+
+#define BLACK 0x000000 /* BGR 8bpc */
+#define CYAN 0xFFFF00 /* BGR 8bpc */
+#define PURPLE 0xFF00FF /* BGR 8bpc */
+#define WHITE 0xFFFFFF /* BGR 8bpc */
+
+#define BLACK64 0x000000000000 /* BGR 16bpc */
+#define CYAN64 0xFFFFFFFF0000 /* BGR 16bpc */
+#define PURPLE64 0xFFFF0000FFFF /* BGR 16bpc */
+#define YELLOW64 0x0000FFFFFFFF /* BGR 16bpc */
+#define WHITE64 0xFFFFFFFFFFFF /* BGR 16bpc */
+#define RED64 0x00000000FFFF /* BGR 16bpc */
+#define GREEN64 0x0000FFFF0000 /* BGR 16bpc */
+#define BLUE64 0xFFFF00000000 /* BGR 16bpc */
+
+static void
+paint_background(data_t *data, struct igt_fb *fb, drmModeModeInfo *mode,
+ uint32_t background, double alpha)
+{
+ cairo_t *cr;
+ int w, h;
+ double r, g, b;
+
+ w = mode->hdisplay;
+ h = mode->vdisplay;
+
+ cr = igt_get_cairo_ctx(data->gfx_fd, &data->fb);
+
+ /* Paint with background color */
+ r = (double) (background & 0xFF) / 255.0;
+ g = (double) ((background & 0xFF00) >> 8) / 255.0;
+ b = (double) ((background & 0xFF0000) >> 16) / 255.0;
+ igt_paint_color_alpha(cr, 0, 0, w, h, r, g, b, alpha);
+
+ cairo_destroy(cr);
+}
+
+static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe,
+ igt_plane_t *plane, int opaque_buffer, int plane_color,
+ uint64_t pipe_background_color)
+{
+ drmModeModeInfo *mode;
+ igt_display_t *display = &data->display;
+ int fb_id;
+ double alpha;
+
+ igt_output_set_pipe(output, pipe);
+
+ /* create the pipe_crc object for this pipe */
+ igt_pipe_crc_free(data->pipe_crc);
+ data->pipe_crc = igt_pipe_crc_new(pipe, INTEL_PIPE_CRC_SOURCE_AUTO);
+
+ mode = igt_output_get_mode(output);
+
+ fb_id = igt_create_fb(data->gfx_fd,
+ mode->hdisplay, mode->vdisplay,
+ DRM_FORMAT_XRGB8888,
+ LOCAL_DRM_FORMAT_MOD_NONE, /* tiled */
+ &data->fb);
+ igt_assert(fb_id);
+
+ /* To make FB pixel win with background color, set alpha as full opaque */
+ igt_crtc_set_background(plane->pipe, pipe_background_color);
+ if (opaque_buffer)
+ alpha = 1.0; /* alpha 1 is fully opque */
+ else
+ alpha = 0.0; /* alpha 0 is fully transparent */
+ paint_background(data, &data->fb, mode, plane_color, alpha);
+
+ igt_plane_set_fb(plane, &data->fb);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+}
+
+static void cleanup_crtc(data_t *data, igt_output_t *output, igt_plane_t *plane)
+{
+ igt_display_t *display = &data->display;
+
+ igt_pipe_crc_free(data->pipe_crc);
+ data->pipe_crc = NULL;
+
+ igt_remove_fb(data->gfx_fd, &data->fb);
+
+ igt_crtc_set_background(plane->pipe, BLACK64);
+ igt_plane_set_fb(plane, NULL);
+ igt_output_set_pipe(output, PIPE_ANY);
+
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+}
+
+static void test_crtc_background(data_t *data)
+{
+ igt_display_t *display = &data->display;
+ igt_output_t *output;
+ enum pipe pipe;
+ int valid_tests = 0;
+
+ igt_require(data->display.has_universal_planes);
+
+ for_each_connected_output(display, output) {
+ igt_plane_t *plane;
+
+ pipe = output->config.pipe;
+ igt_output_set_pipe(output, pipe);
+
+ plane = igt_output_get_plane(output, IGT_PLANE_PRIMARY);
+ igt_require(plane->pipe->background_property);
+
+ prepare_crtc(data, output, pipe, plane, 1, PURPLE, BLACK64);
+
+ /* Now set background without using a plane, i.e.,
+ * Disable the plane to let hw background color win blend. */
+ igt_plane_set_fb(plane, NULL);
+ igt_crtc_set_background(plane->pipe, PURPLE64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ /* Try few other background colors */
+ igt_crtc_set_background(plane->pipe, CYAN64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ igt_crtc_set_background(plane->pipe, YELLOW64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ igt_crtc_set_background(plane->pipe, RED64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ igt_crtc_set_background(plane->pipe, GREEN64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ igt_crtc_set_background(plane->pipe, BLUE64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ igt_crtc_set_background(plane->pipe, WHITE64);
+ igt_display_commit2(display, COMMIT_UNIVERSAL);
+
+ valid_tests++;
+ cleanup_crtc(data, output, plane);
+ }
+ igt_require_f(valid_tests, "no valid crtc/connector combinations found\n");
+}
+
+igt_main
+{
+ data_t data = {};
+
+ igt_skip_on_simulation();
+
+ data.gfx_fd = drm_open_any();
+ igt_require_pipe_crc();
+ igt_display_init(&data.display, data.gfx_fd);
+
+ igt_subtest("pipe-background")
+ test_crtc_background(&data);
+
+ igt_display_fini(&data.display);
+}
--
1.9.1
More information about the Intel-gfx
mailing list