From dd6994b1b8796848223b1b6b9a7c7455fb0469a9 Mon Sep 17 00:00:00 2001 From: Zhigang Gong Date: Mon, 12 Aug 2013 13:34:15 +0800 Subject: [PATCH] Driver: Fix the incorrect size of surface 1. According to Ben's comments, the surface 0 and 1 should be exactly match each other, and the only reason why we need two surfaces rather than 1 is that for the fulsim usage. Thus we should set surface 1 and 0 with the same memory size. This patch fixes the flat_address_space unit test case and also a randome failure reported by yang rong. Signed-off-by: Zhigang Gong Tested-by: "Yang, Rong R" --- src/intel/intel_gpgpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/intel_gpgpu.c b/src/intel/intel_gpgpu.c index e553a55..8e90641 100644 --- a/src/intel/intel_gpgpu.c +++ b/src/intel/intel_gpgpu.c @@ -478,7 +478,7 @@ intel_gpgpu_map_address_space(intel_gpgpu_t *gpgpu) ss1->ss2.width = ss0->ss2.width = 127; /* bits 6:0 of sz */ ss1->ss2.height = ss0->ss2.height = 16383; /* bits 20:7 of sz */ ss0->ss3.depth = 1023; /* bits 30:21 of sz */ - ss1->ss3.depth = 510; /* bits 30:21 of sz */ + ss1->ss3.depth = 1023; /* bits 30:21 of sz */ ss1->ss5.cache_control = ss0->ss5.cache_control = cc_llc_l3; heap->binding_table[0] = offsetof(surface_heap_t, surface); heap->binding_table[1] = sizeof(gen7_surface_state_t) + offsetof(surface_heap_t, surface); -- 2.7.4