[Intel-gfx] [PATCH] test/gem_userptr_*: Fix compile fail
Daniel Vetter
daniel.vetter at ffwll.ch
Fri Apr 25 17:55:57 CEST 2014
Also shut up warnings. Those revealed incorrect usage of local
variables in conjunction with igt_fixture/igt_subtest. Since those use
longjmps we need to move the out of the stackframe those magic blocks
are declared in.
Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
---
benchmarks/gem_userptr_benchmark.c | 12 +++++++-----
tests/gem_userptr_blits.c | 14 +++++++++++---
2 files changed, 18 insertions(+), 8 deletions(-)
diff --git a/benchmarks/gem_userptr_benchmark.c b/benchmarks/gem_userptr_benchmark.c
index 9ad6e4a35687..bdfce126e960 100644
--- a/benchmarks/gem_userptr_benchmark.c
+++ b/benchmarks/gem_userptr_benchmark.c
@@ -42,8 +42,11 @@
#include <sys/stat.h>
#include <sys/time.h>
#include <sys/mman.h>
+#include <signal.h>
+
#include "drm.h"
#include "i915_drm.h"
+
#include "drmtest.h"
#include "intel_bufmgr.h"
#include "intel_batchbuffer.h"
@@ -288,7 +291,6 @@ static void test_ptr_read(void *ptr)
unsigned long iter = 0;
volatile unsigned long *p;
unsigned long i, loops;
- register unsigned long v;
loops = BO_SIZE / sizeof(unsigned long) / 4;
@@ -297,10 +299,10 @@ static void test_ptr_read(void *ptr)
while (run_test) {
p = (unsigned long *)ptr;
for (i = 0; i < loops; i++) {
- v = *p++;
- v = *p++;
- v = *p++;
- v = *p++;
+ (void)*p++;
+ (void)*p++;
+ (void)*p++;
+ (void)*p++;
}
iter++;
}
diff --git a/tests/gem_userptr_blits.c b/tests/gem_userptr_blits.c
index 03af58ea7fe5..cec8c8fd4f04 100644
--- a/tests/gem_userptr_blits.c
+++ b/tests/gem_userptr_blits.c
@@ -46,8 +46,11 @@
#include <sys/stat.h>
#include <sys/time.h>
#include <sys/mman.h>
+#include <signal.h>
+
#include "drm.h"
#include "i915_drm.h"
+
#include "drmtest.h"
#include "intel_bufmgr.h"
#include "intel_batchbuffer.h"
@@ -1039,17 +1042,22 @@ static int test_unmap_cycles(int fd, int expected)
return 0;
}
+unsigned int total_ram;
+uint64_t aperture_size;
+int fd, count, size = 0, ret;
+
+
int main(int argc, char **argv)
{
- uint64_t aperture_size;
- unsigned int total_ram;
- int fd = -1, count = 0, size = 0, ret;
+ int size = sizeof(linear);
igt_skip_on_simulation();
igt_subtest_init(argc, argv);
igt_fixture {
+ int ret;
+
fd = drm_open_any();
igt_assert(fd >= 0);
--
1.9.2
More information about the Intel-gfx
mailing list