[igt-dev] [PATCH i-g-t v3 5/8] tests/vmwgfx: Add surface copy test
Zack Rusin
zack at kde.org
Wed Apr 12 02:33:26 UTC 2023
From: Maaz Mombasawala <mombasawalam at vmware.com>
This test checks for validation errors during several surface copies
in vmwgfx.
Signed-off-by: Zack Rusin <zackr at vmware.com>
Signed-off-by: Maaz Mombasawala <mombasawalam at vmware.com>
Reviewed-by: Martin Krastev <krastevm at vmware.com>
---
tests/vmwgfx/meson.build | 3 +-
tests/vmwgfx/vmw_surface_copy.c | 341 ++++++++++++++++++++++++++++++++
2 files changed, 343 insertions(+), 1 deletion(-)
create mode 100644 tests/vmwgfx/vmw_surface_copy.c
diff --git a/tests/vmwgfx/meson.build b/tests/vmwgfx/meson.build
index ce2969db..cc0718b2 100644
--- a/tests/vmwgfx/meson.build
+++ b/tests/vmwgfx/meson.build
@@ -1,7 +1,8 @@
vmwgfx_progs = [
'vmw_tri',
- 'vmw_execution_buffer'
+ 'vmw_execution_buffer',
+ 'vmw_surface_copy'
]
vmwgfx_deps = test_deps
diff --git a/tests/vmwgfx/vmw_surface_copy.c b/tests/vmwgfx/vmw_surface_copy.c
new file mode 100644
index 00000000..d7c2711a
--- /dev/null
+++ b/tests/vmwgfx/vmw_surface_copy.c
@@ -0,0 +1,341 @@
+// SPDX-License-Identifier: GPL-2.0 OR MIT
+/**********************************************************
+ * Copyright 2021-2022 VMware, Inc.
+ *
+ * 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 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 "igt_vmwgfx.h"
+
+IGT_TEST_DESCRIPTION("Test surface copies.");
+
+/**
+ * Compares two 2d surfaces to see if their size and contents are equal.
+ */
+static bool are_surfaces_identical(int fd, struct vmw_surface *s1,
+ struct vmw_surface *s2)
+{
+ struct vmw_mob mob1 = { .size = s1->base.buffer_size,
+ .handle = s1->base.buffer_handle,
+ .map_handle = s1->base.buffer_map_handle };
+ struct vmw_mob mob2 = { .size = s2->base.buffer_size,
+ .handle = s2->base.buffer_handle,
+ .map_handle = s2->base.buffer_map_handle };
+ char *readback1;
+ char *readback2;
+ bool is_equal = true;
+ uint32 i, j;
+ uint32 stride1, stride2, height, width;
+
+ if ((s1->params.base.base_size.width !=
+ s2->params.base.base_size.width) ||
+ (s1->params.base.base_size.height !=
+ s2->params.base.base_size.height)) {
+ return false;
+ }
+ width = s1->params.base.base_size.width;
+ height = s1->params.base.base_size.height;
+ stride1 = s1->params.buffer_byte_stride == 0 ?
+ width :
+ s1->params.buffer_byte_stride;
+ stride2 = s2->params.buffer_byte_stride == 0 ?
+ width :
+ s2->params.buffer_byte_stride;
+
+ readback1 = vmw_ioctl_mob_map(fd, &mob1);
+ readback2 = vmw_ioctl_mob_map(fd, &mob2);
+ for (i = 0; i < width; i++) {
+ for (j = 0; j < height; j++) {
+ if (readback1[j * stride1 + i] !=
+ readback2[j * stride2 + i]) {
+ is_equal = false;
+ goto are_surfaces_identical_end;
+ }
+ }
+ }
+are_surfaces_identical_end:
+ vmw_ioctl_mob_unmap(&mob1);
+ vmw_ioctl_mob_unmap(&mob2);
+ return is_equal;
+}
+
+static void set_surface_value(int fd, struct vmw_surface *surface, char value)
+{
+ struct vmw_mob mob = { .size = surface->base.buffer_size,
+ .handle = surface->base.buffer_handle,
+ .map_handle = surface->base.buffer_map_handle };
+ char *readback;
+
+ readback = vmw_ioctl_mob_map(fd, &mob);
+ memset(readback, value, mob.size);
+ vmw_ioctl_mob_unmap(&mob);
+}
+
+static void exec_surface_copy(struct vmw_execbuf *cmd_buf,
+ struct drm_vmw_fence_rep *cmd_fence,
+ SVGA3dSurfaceImageId src,
+ SVGA3dSurfaceImageId dest, SVGA3dCopyBox *box)
+{
+ vmw_cmd_surface_copy(cmd_buf, src, dest, box, 1);
+ vmw_execbuf_submit(cmd_buf, cmd_fence);
+ vmw_ioctl_fence_finish(cmd_buf->drm_fd, cmd_fence);
+}
+
+static void test_invalid_copies(int fd, int32 cid)
+{
+ struct vmw_surface *s1;
+ struct vmw_surface *s2;
+ struct vmw_execbuf *cmd_buf;
+ struct drm_vmw_fence_rep cmd_fence;
+ SVGA3dSize surface_size;
+ SVGA3dCopyBox box;
+ SVGA3dSurfaceImageId src;
+ SVGA3dSurfaceImageId dest;
+ SVGA3dSurfaceImageId bad_surface;
+
+ surface_size.width = 128;
+ surface_size.height = 128;
+ surface_size.depth = 1;
+
+ igt_require(
+ vmw_is_format_supported(fd, SVGA3D_DEVCAP_SURFACEFMT_A8R8G8B8));
+
+ s1 = vmw_create_surface_simple(fd, 0, SVGA3D_A8R8G8B8, surface_size,
+ NULL);
+ s2 = vmw_create_surface_simple(fd, 0, SVGA3D_A8R8G8B8, surface_size,
+ NULL);
+ cmd_buf = vmw_execbuf_create(fd, cid);
+
+ box.x = 0;
+ box.y = 0;
+ box.z = 0;
+ box.w = 1;
+ box.h = 1;
+ box.d = 1;
+ box.srcx = 0;
+ box.srcy = 0;
+ box.srcz = 0;
+
+ src.sid = s1->base.handle;
+ src.face = 0;
+ src.mipmap = 0;
+ dest.sid = s2->base.handle;
+ dest.face = 0;
+ dest.mipmap = 0;
+
+ /* Testing a valid copy first */
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(are_surfaces_identical(fd, s1, s2));
+
+ /* Setting surfaces to different values */
+ set_surface_value(fd, s1, 0);
+ set_surface_value(fd, s2, 16);
+
+ /* Testing invalid copies */
+
+ /* x */
+ box.x = 129;
+ box.w = 1;
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ box.x = 0;
+ box.w = 129;
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ box.srcx = 129;
+ box.w = 1;
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ box.srcx = 0;
+ box.w = 129;
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ /* y */
+ box.y = 129;
+ box.h = 1;
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ box.y = 0;
+ box.h = 129;
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ box.srcy = 129;
+ box.h = 1;
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ box.srcy = 0;
+ box.h = 129;
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ /* z */
+ box.z = 2;
+ box.d = 1;
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ box.z = 0;
+ box.d = 2;
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ box.srcz = 2;
+ box.d = 1;
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ box.srcz = 0;
+ box.d = 2;
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ /* Invalid surface id */
+ bad_surface.sid = src.sid + dest.sid + 1;
+ bad_surface.face = 0;
+ bad_surface.mipmap = 0;
+
+ box.x = 0;
+ box.y = 0;
+ box.z = 0;
+ box.w = 1;
+ box.h = 1;
+ box.d = 1;
+ box.srcx = 0;
+ box.srcy = 0;
+ box.srcz = 0;
+
+ vmw_cmd_surface_copy(cmd_buf, bad_surface, dest, &box, 1);
+ igt_assert(vmw_execbuf_submit(cmd_buf, &cmd_fence) != 0);
+
+ vmw_cmd_surface_copy(cmd_buf, src, bad_surface, &box, 1);
+ igt_assert(vmw_execbuf_submit(cmd_buf, &cmd_fence) != 0);
+ vmw_ioctl_fence_finish(fd, &cmd_fence);
+
+ bad_surface.sid = src.sid;
+ bad_surface.face = 2;
+
+ exec_surface_copy(cmd_buf, &cmd_fence, bad_surface, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ exec_surface_copy(cmd_buf, &cmd_fence, src, bad_surface, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ vmw_execbuf_destroy(cmd_buf);
+ vmw_ioctl_surface_unref(fd, s1);
+ vmw_ioctl_surface_unref(fd, s2);
+}
+
+static void test_invalid_copies_3d(int fd, int32 cid)
+{
+ struct vmw_surface *s1;
+ struct vmw_surface *s2;
+ struct vmw_execbuf *cmd_buf;
+ struct drm_vmw_fence_rep cmd_fence;
+ SVGA3dSize surface_size;
+ SVGA3dCopyBox box;
+ SVGA3dSurfaceImageId src;
+ SVGA3dSurfaceImageId dest;
+
+ surface_size.width = 128;
+ surface_size.height = 128;
+ surface_size.depth = 1;
+
+ igt_require(
+ vmw_is_format_supported(fd, SVGA3D_DEVCAP_SURFACEFMT_A8R8G8B8));
+ igt_require(vmw_is_format_supported(fd, SVGA3D_DEVCAP_DXFMT_Z_D32));
+
+ s1 = vmw_create_surface_simple(fd, 0, SVGA3D_A8R8G8B8, surface_size,
+ NULL);
+ s2 = vmw_create_surface_simple(fd, 0, SVGA3D_Z_D32, surface_size, NULL);
+ cmd_buf = vmw_execbuf_create(fd, cid);
+
+ box.x = 0;
+ box.y = 0;
+ box.z = 0;
+ box.w = 10;
+ box.h = 10;
+ box.d = 10;
+ box.srcx = 0;
+ box.srcy = 0;
+ box.srcz = 0;
+
+ src.sid = s1->base.handle;
+ src.face = 0;
+ src.mipmap = 0;
+ dest.sid = s2->base.handle;
+ dest.face = 0;
+ dest.mipmap = 0;
+
+ set_surface_value(fd, s1, 0);
+ set_surface_value(fd, s2, 16);
+
+ exec_surface_copy(cmd_buf, &cmd_fence, src, dest, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ exec_surface_copy(cmd_buf, &cmd_fence, dest, src, &box);
+ igt_assert(!are_surfaces_identical(fd, s1, s2));
+
+ vmw_execbuf_destroy(cmd_buf);
+ vmw_ioctl_surface_unref(fd, s1);
+ vmw_ioctl_surface_unref(fd, s2);
+}
+
+igt_main
+{
+ int fd;
+ int32 cid;
+
+ igt_fixture
+ {
+ fd = drm_open_driver_render(DRIVER_VMWGFX);
+ igt_require(fd != -1);
+
+ cid = vmw_ioctl_context_create(fd);
+ igt_require(cid != SVGA3D_INVALID_ID);
+ }
+
+ igt_describe("Test surface copies (valid and invalid ones).");
+ igt_subtest("test_invalid_copies")
+ {
+ test_invalid_copies(fd, cid);
+ }
+
+ igt_describe("Test surface copies on 3D enabled contexts.");
+ igt_subtest("test_invalid_copies_3d")
+ {
+ igt_require(vmw_ioctl_get_param(fd, DRM_VMW_PARAM_3D));
+ test_invalid_copies_3d(fd, cid);
+ }
+
+ igt_fixture
+ {
+ vmw_ioctl_context_destroy(fd, cid);
+ close(fd);
+ }
+}
--
2.39.2
More information about the igt-dev
mailing list