[PATCH 08/10] tests/prime_nv_pcopy: Drop return values from tests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
---
 tests/prime_nv_pcopy.c | 121 +++++++++++++++----------------------------------
 1 file changed, 36 insertions(+), 85 deletions(-)

diff --git a/tests/prime_nv_pcopy.c b/tests/prime_nv_pcopy.c
index 74388349da3d..6aa48716348e 100644
--- a/tests/prime_nv_pcopy.c
+++ b/tests/prime_nv_pcopy.c
@@ -574,7 +574,7 @@ static void check1_macro(uint32_t *p, uint32_t w, uint32_t h)
 }
 
 /* test 1, see if we can copy from linear to intel Y format safely */
-static int test1_macro(void)
+static void test1_macro(void)
 {
 	int prime_fd = -1;
 	struct nouveau_bo *nvbo = NULL, *nvbi = NULL;
@@ -625,8 +625,6 @@ static int test1_macro(void)
 	nouveau_bo_ref(NULL, &nvbo);
 	nouveau_bo_ref(NULL, &nvbi);
 	close(prime_fd);
-
-	return 0;
 }
 
 static void dump_line(uint8_t *map)
@@ -663,11 +661,10 @@ static void check1_micro(void *map, uint32_t pitch, uint32_t lines,
 }
 
 /* test 1, but check micro format, should be unaffected by bit9 swizzling */
-static int test1_micro(void)
+static void test1_micro(void)
 {
 	struct nouveau_bo *bo_intel = NULL, *bo_nvidia = NULL, *bo_linear = NULL;
 	rect intel, nvidia, linear;
-	int ret = -1;
 	uint32_t tiling = I915_TILING_Y;
 
 	uint32_t src_x = 0, src_y = 0;
@@ -678,22 +675,13 @@ static int test1_micro(void)
 	int prime_fd;
 
 	test_intel_bo = drm_intel_bo_alloc(bufmgr, "test bo", w * h, 4096);
-	if (!test_intel_bo)
-		return -1;
+	igt_assert(test_intel_bo);
 	drm_intel_bo_set_tiling(test_intel_bo, &tiling, w);
-	if (tiling != I915_TILING_Y) {
-		fprintf(stderr, "Couldn't set y tiling\n");
-		goto out;
-	}
-	ret = drm_intel_gem_bo_map_gtt(test_intel_bo);
-	if (ret)
-		goto out;
+	igt_assert(tiling == I915_TILING_Y);
+	igt_assert(drm_intel_gem_bo_map_gtt(test_intel_bo) == 0);
 
 	drm_intel_bo_gem_export_to_prime(test_intel_bo, &prime_fd);
-	if (prime_fd < 0) {
-		drm_intel_bo_unreference(test_intel_bo);
-		goto out;
-	}
+	igt_assert(prime_fd >= 0);
 	noop_intel(test_intel_bo);
 
 	nv_bo_alloc(&bo_intel, &intel, w, h, tile_intel_y, prime_fd, 0);
@@ -724,12 +712,10 @@ static int test1_micro(void)
 	noop_intel(test_intel_bo);
 	check1_micro(test_intel_bo->virtual, intel.pitch, intel.h, dst_x, dst_y, w, h);
 
-out:
 	nouveau_bo_ref(NULL, &bo_linear);
 	nouveau_bo_ref(NULL, &bo_nvidia);
 	nouveau_bo_ref(NULL, &bo_intel);
 	drm_intel_bo_unreference(test_intel_bo);
-	return ret;
 }
 
 #if 0 /* nv can't deswizzle into all possible versions of Intel BO objects ... */
@@ -870,7 +856,7 @@ out:
  * as a reference for potential future tests. Software tiling is
  * used for now
  */
-static int test2(void)
+static void test2(void)
 {
 	struct nouveau_bo *nvbo = NULL, *nvbi = NULL;
 	rect dst, src;
@@ -915,8 +901,6 @@ static int test2(void)
 
 	nouveau_bo_ref(NULL, &nvbo);
 	nouveau_bo_ref(NULL, &nvbi);
-
-	return 0;
 }
 
 static void check3(const uint32_t *p, uint32_t pitch, uint32_t lines,
@@ -947,7 +931,7 @@ static void check3(const uint32_t *p, uint32_t pitch, uint32_t lines,
 }
 
 /* copy from nvidia bo to intel bo and copy to a linear bo to check if tiling went succesful */
-static int test3_base(int tile_src, int tile_dst)
+static void test3_base(int tile_src, int tile_dst)
 {
 	struct nouveau_bo *bo_intel = NULL, *bo_nvidia = NULL, *bo_linear = NULL;
 	rect intel, nvidia, linear;
@@ -961,14 +945,10 @@ static int test3_base(int tile_src, int tile_dst)
 	int prime_fd;
 
 	test_intel_bo = drm_intel_bo_alloc(bufmgr, "test bo", 2048 * cpp * 768, 4096);
-	if (!test_intel_bo)
-		return -1;
+	igt_assert(test_intel_bo);
 
 	drm_intel_bo_gem_export_to_prime(test_intel_bo, &prime_fd);
-	if (prime_fd < 0) {
-		drm_intel_bo_unreference(test_intel_bo);
-		return -1;
-	}
+	igt_assert(prime_fd >= 0);
 
 	nv_bo_alloc(&bo_intel, &intel, 2048 * cpp, 768, tile_dst, prime_fd, 0);
 	nv_bo_alloc(&bo_nvidia, &nvidia, 300 * cpp, 300, tile_src, -1, NOUVEAU_BO_VRAM);
@@ -998,38 +978,36 @@ static int test3_base(int tile_src, int tile_dst)
 	nouveau_bo_ref(NULL, &bo_nvidia);
 	nouveau_bo_ref(NULL, &bo_intel);
 	drm_intel_bo_unreference(test_intel_bo);
-
-	return 0;
 }
 
-static int test3_1(void)
+static void test3_1(void)
 {
 	/* nvidia tiling to intel */
-	return test3_base(0x40, tile_intel_y);
+	test3_base(0x40, tile_intel_y);
 }
 
-static int test3_2(void)
+static void test3_2(void)
 {
 	/* intel tiling to nvidia */
-	return test3_base(tile_intel_y, 0x40);
+	test3_base(tile_intel_y, 0x40);
 }
 
-static int test3_3(void)
+static void test3_3(void)
 {
 	/* intel tiling to linear */
-	return test3_base(tile_intel_y, 0);
+	test3_base(tile_intel_y, 0);
 }
 
-static int test3_4(void)
+static void test3_4(void)
 {
 	/* linear tiling to intel */
-	return test3_base(0, tile_intel_y);
+	test3_base(0, tile_intel_y);
 }
 
-static int test3_5(void)
+static void test3_5(void)
 {
 	/* linear to linear */
-	return test3_base(0, 0);
+	test3_base(0, 0);
 }
 
 /* Acquire when == SEQUENCE */
@@ -1043,44 +1021,33 @@ static int test3_5(void)
 #define SEMA_ACQUIRE_GEQUAL 4
 
 /* Test only new style semaphores, old ones are AWFUL */
-static int test_semaphore(void)
+static void test_semaphore(void)
 {
 	drm_intel_bo *test_intel_bo = NULL;
 	struct nouveau_bo *sema_bo = NULL;
-	int ret = -1, prime_fd;
+	int prime_fd;
 	uint32_t *sema;
 	struct nouveau_pushbuf *push = npush;
 
-	if (ndev->chipset < 0x84)
-		return -1;
+	igt_skip_on(ndev->chipset < 0x84);
 
 	/* Should probably be kept in sysmem */
 	test_intel_bo = drm_intel_bo_alloc(bufmgr, "semaphore bo", 4096, 4096);
-	if (!test_intel_bo)
-		goto out;
+	igt_assert(test_intel_bo);
 
 	drm_intel_bo_gem_export_to_prime(test_intel_bo, &prime_fd);
-	if (prime_fd < 0)
-		goto out;
-	ret = nouveau_bo_prime_handle_ref(ndev, prime_fd, &sema_bo);
+	igt_assert(prime_fd >= 0);
+	igt_assert(nouveau_bo_prime_handle_ref(ndev, prime_fd, &sema_bo) == 0);
 	close(prime_fd);
-	if (ret < 0)
-		goto out;
 
-	ret = drm_intel_gem_bo_map_gtt(test_intel_bo);
-	if (ret != 0) {
-		fprintf(stderr,"failed to map bo\n");
-		goto out;
-	}
+	igt_assert(drm_intel_gem_bo_map_gtt(test_intel_bo) == 0);
 	sema = test_intel_bo->virtual;
 	sema++;
 	*sema = 0;
 
-	ret = -1;
-	if (nouveau_pushbuf_space(push, 64, 0, 0) ||
-	    nouveau_pushbuf_refn(push, &(struct nouveau_pushbuf_refn)
-	    { sema_bo, NOUVEAU_BO_GART|NOUVEAU_BO_RDWR }, 1))
-		goto out;
+	igt_assert(nouveau_pushbuf_space(push, 64, 0, 0) == 0);
+	igt_assert(nouveau_pushbuf_refn(push, &(struct nouveau_pushbuf_refn)
+					{ sema_bo, NOUVEAU_BO_GART|NOUVEAU_BO_RDWR }, 1) == 0);
 
 	if (ndev->chipset < 0xc0) {
 		struct nv04_fifo *nv04_fifo = nchannel->data;
@@ -1119,38 +1086,22 @@ static int test_semaphore(void)
 	nouveau_pushbuf_kick(push, push->channel);
 
 	usleep(1000);
-	if (*sema != 2) {
-		fprintf(stderr, "new sema should be 2 is %u\n", *sema);
-		goto out;
-	}
+	igt_assert(*sema == 2);
 
 	*sema = 3;
 	usleep(1000);
-	if (*sema != 4) {
-		fprintf(stderr, "new sema should be 4 is %u\n", *sema);
-		goto out;
-	}
+	igt_assert(*sema == 4);
 
 	*sema = 5;
 	usleep(1000);
-	if (*sema != 6) {
-		fprintf(stderr, "new sema should be 6 is %u\n", *sema);
-		goto out;
-	}
+	igt_assert(*sema == 6);
 
 	*sema = 8;
 	usleep(1000);
-	if (*sema != 9) {
-		fprintf(stderr, "new sema should be 9 is %u\n", *sema);
-		goto out;
-	}
-	ret = 0;
+	igt_assert(*sema == 9);
 
-out:
 	nouveau_bo_ref(NULL, &sema_bo);
-	if (test_intel_bo)
-		drm_intel_bo_unreference(test_intel_bo);
-	return ret;
+	drm_intel_bo_unreference(test_intel_bo);
 }
 
 igt_main
@@ -1178,7 +1129,7 @@ igt_main
 
 #define xtest(x, args...) \
 	igt_subtest( #x ) \
-		igt_assert(((x)(args)) == 0); \
+		(x)(args);
 
 	xtest(test1_macro);
 	xtest(test1_micro);
-- 
1.9.2

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux