diff mbox

[i-g-t,v2] gem_largeobject: Fix for clang compiler

Message ID 2D016F5BA44C744783BA96B1D3EBC5BD14002BBC@IRSMSX102.ger.corp.intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Derek Morton Aug. 10, 2016, 8:30 a.m. UTC 
Bump,

Can this be merged? It is a simple fix for an issue with the clang compiler.

//Derek

-----Original Message-----
From: Morton, Derek J 
Sent: Thursday, August 4, 2016 3:37 PM
To: intel-gfx@lists.freedesktop.org
Cc: Gore, Tim <tim.gore@intel.com>; Morton, Derek J <derek.j.morton@intel.com>
Subject: [PATCH i-g-t v2] gem_largeobject: Fix for clang compiler

The clang compiler generates a build error for binaries with >128MB data segments because "The gap between stack and binary is only guarenteed to be 128MB on x86_64"

Simple solution is to allocate the required memory from the heap.

[v2: moved malloc to after skip_on_simulation]

Signed-off-by: Derek Morton <derek.j.morton@intel.com>
---
 tests/gem_largeobject.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

 
 	igt_skip_on_simulation();
 
+	data = malloc(OBJ_SIZE);
+	igt_assert(data);
+
 	fd = drm_open_driver(DRIVER_INTEL);
 
 	test_large_object(fd);
+
+	free(data);
 }
--
1.9.1
diff mbox

Patch

diff --git a/tests/gem_largeobject.c b/tests/gem_largeobject.c index c001c17..518396f 100644
--- a/tests/gem_largeobject.c
+++ b/tests/gem_largeobject.c
@@ -39,7 +39,7 @@ 
 /* Should take 64 pages to store the page pointers on 64 bit */  #define OBJ_SIZE (128 * 1024 * 1024)
 
-unsigned char data[OBJ_SIZE];
+unsigned char *data;
 
 static void
 test_large_object(int fd)
@@ -79,7 +79,12 @@  igt_simple_main