xf86-video-intel: 2 commits - src/sna/sna_accel.c test/basic-copyplane.c test/.gitignore test/Makefile.am
Chris Wilson
ickle at kemper.freedesktop.org
Wed Jul 29 01:46:16 PDT 2015
src/sna/sna_accel.c | 6 +-
test/.gitignore | 1
test/Makefile.am | 1
test/basic-copyplane.c | 99 +++++++++++++++++++++++++++++++++++++++++++++++++
4 files changed, 105 insertions(+), 2 deletions(-)
New commits:
commit 4246c63347290390a2104739c719f5ff6a05a0e2
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date: Wed Jul 29 09:42:26 2015 +0100
sna: Fix off by one in constructing XCopyPlane on bdw
Broadwell expanded all the relocations and we needed to adjust our
command construction to match. I missed offsetting the XY_SRC_COPY_IMM
used for XCopyPlane resulting in garbage for small copies on Broadwell.
Reported-by: Omar Sandoval
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91499
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
diff --git a/src/sna/sna_accel.c b/src/sna/sna_accel.c
index 38ca366..a816b77 100644
--- a/src/sna/sna_accel.c
+++ b/src/sna/sna_accel.c
@@ -8349,6 +8349,8 @@ sna_copy_bitmap_blt(DrawablePtr _bitmap, DrawablePtr drawable, GCPtr gc,
}
br13 |= blt_depth(drawable->depth) << 24;
br13 |= copy_ROP[gc->alu] << 16;
+ DBG(("%s: target-depth=%d, alu=%d, bg=%08x, fg=%08x\n",
+ __FUNCTION__, drawable->depth, gc->alu, gc->bgPixel, gc->fgPixel));
kgem_set_mode(&sna->kgem, KGEM_BLT, arg->bo);
assert(kgem_bo_can_blt(&sna->kgem, arg->bo));
@@ -8397,8 +8399,8 @@ sna_copy_bitmap_blt(DrawablePtr _bitmap, DrawablePtr drawable, GCPtr gc,
I915_GEM_DOMAIN_RENDER |
KGEM_RELOC_FENCED,
0);
- b[5] = gc->bgPixel;
- b[6] = gc->fgPixel;
+ b[6] = gc->bgPixel;
+ b[7] = gc->fgPixel;
dst = (uint8_t *)&b[8];
sna->kgem.nbatch += 8 + src_stride;
commit 66e16d97ee29595922cd7e38f172ffcc4b87724e
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date: Wed Jul 29 08:58:44 2015 +0100
test: Exercise copyplane
Reported-by: Omar Sandoval
References: https://bugs.freedesktop.org/show_bug.cgi?id=91499
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
diff --git a/test/.gitignore b/test/.gitignore
index 2a68c3d..3eea32e 100644
--- a/test/.gitignore
+++ b/test/.gitignore
@@ -1,5 +1,6 @@
basic-copyarea
basic-copyarea-size
+basic-copyplane
basic-fillrect
basic-putimage
basic-lines
diff --git a/test/Makefile.am b/test/Makefile.am
index 50f2126..7d88810 100644
--- a/test/Makefile.am
+++ b/test/Makefile.am
@@ -5,6 +5,7 @@ stress_TESTS = \
basic-rectangle \
basic-string \
basic-copyarea \
+ basic-copyplane \
basic-copyarea-size \
basic-putimage \
basic-lines \
diff --git a/test/basic-copyplane.c b/test/basic-copyplane.c
new file mode 100644
index 0000000..f049b82
--- /dev/null
+++ b/test/basic-copyplane.c
@@ -0,0 +1,99 @@
+#include <stdint.h>
+#include <stdio.h>
+#include <stdlib.h>
+
+#include <X11/Xutil.h> /* for XDestroyImage */
+#include <pixman.h> /* for pixman blt functions */
+
+#include "test.h"
+
+static uint8_t clock_bits[] = {0x3C, 0x5E, 0xEF, 0xF7, 0x87, 0xFF, 0x7E, 0x3C};
+
+/* https://bugs.freedesktop.org/show_bug.cgi?id=91499 */
+static void draw_clock(struct test_display *t, Drawable d,
+ uint8_t alu, int x, int y, uint32_t fg, uint32_t bg)
+{
+ Pixmap pixmap;
+ XGCValues val;
+ GC gc;
+
+ val.graphics_exposures = 0;
+ val.function = alu;
+ val.foreground = fg;
+ val.background = fg;
+
+ gc = XCreateGC(t->dpy, d,
+ GCGraphicsExposures | GCForeground | GCBackground | GCFunction,
+ &val);
+ pixmap = XCreateBitmapFromData(t->dpy, d, (char *)clock_bits, 8, 8);
+
+ XCopyPlane(t->dpy, pixmap, d, gc, 0, 0, 8, 8, x, y, 1);
+
+ XFreePixmap(t->dpy, pixmap);
+ XFreeGC(t->dpy, gc);
+}
+
+static void clear(struct test_display *dpy, struct test_target *tt)
+{
+ XRenderColor render_color = {0};
+ XRenderFillRectangle(dpy->dpy, PictOpClear, tt->picture, &render_color,
+ 0, 0, tt->width, tt->height);
+}
+
+static void clock_tests(struct test *t, int reps, int sets, enum target target)
+{
+ struct test_target out, ref;
+ int r, s;
+
+ printf("Testing clock (%s): ", test_target_name(target));
+ fflush(stdout);
+
+ test_target_create_render(&t->out, target, &out);
+ clear(&t->out, &out);
+
+ test_target_create_render(&t->ref, target, &ref);
+ clear(&t->ref, &ref);
+
+ for (s = 0; s < sets; s++) {
+ for (r = 0; r < reps; r++) {
+ int x = rand() % (out.width - 8);
+ int y = rand() % (out.height - 8);
+ uint8_t alu = rand() % (GXset + 1);
+ uint32_t bg = rand();
+ uint32_t fg = rand();
+
+ draw_clock(&t->out, out.draw, alu, x, y, fg, bg);
+ draw_clock(&t->ref, ref.draw, alu, x, y, fg, bg);
+ }
+
+ test_compare(t,
+ out.draw, out.format,
+ ref.draw, ref.format,
+ 0, 0, out.width, out.height,
+ "");
+ }
+
+ printf("passed [%d iterations x %d]\n", reps, sets);
+
+ test_target_destroy_render(&t->out, &out);
+ test_target_destroy_render(&t->ref, &ref);
+}
+
+int main(int argc, char **argv)
+{
+ struct test test;
+ int i;
+
+ test_init(&test, argc, argv);
+
+ for (i = 0; i <= DEFAULT_ITERATIONS; i++) {
+ int reps = REPS(i), sets = SETS(i);
+ enum target t;
+
+ for (t = TARGET_FIRST; t <= TARGET_LAST; t++) {
+ clock_tests(&test, reps, sets, t);
+ }
+ }
+
+ return 0;
+}
More information about the xorg-commit
mailing list