flywithu/ntel-gpu-tools
1
0
Fork 0
mirror of https://github.com/tiagovignatti/intel-gpu-tools.git synced 2025-06-19 05:46:25 +00:00
Code Issues Releases Wiki Activity

igt/gem_exec_alignment: Fix off-by-one in buffer objects

Browse Source 
When reducing the buffer count to fit into the aperture whilst aligned,
remember to adjust the pointer so that the batch is the last object!

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
This commit is contained in:
Chris Wilson 2016-02-05 15:55:40 +00:00
parent b081257d74
commit 6f7372f664
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tests/gem_exec_alignment.c
View File

@ -119,7 +119,7 @@ static void many(int fd)
uint64_t factor = alignment / max_alignment; uint64_t factor = alignment / max_alignment;
execbuf.buffer_count = 2*count / factor; execbuf.buffer_count = 2*count / factor;
execbuf.buffers_ptr = execbuf.buffers_ptr =
(uintptr_t)(execobj + count - execbuf.buffer_count); (uintptr_t)(execobj + count - execbuf.buffer_count + 1);
} }
igt_debug("testing %lld x alignment=%#llx [%db]\n", igt_debug("testing %lld x alignment=%#llx [%db]\n",
@ -127,7 +127,7 @@ static void many(int fd)
(long long)alignment, (long long)alignment,
find_last_bit(alignment)-1); find_last_bit(alignment)-1);
gem_execbuf(fd, &execbuf); gem_execbuf(fd, &execbuf);
for(i = count - execbuf.buffer_count; i < count; i++) { for(i = count - execbuf.buffer_count + 1; i < count; i++) {
igt_assert_eq_u64(execobj[i].alignment, alignment); igt_assert_eq_u64(execobj[i].alignment, alignment);
igt_assert_eq_u64(execobj[i].offset % alignment, 0); igt_assert_eq_u64(execobj[i].offset % alignment, 0);
} }
@ -197,7 +197,7 @@ static void single(int fd)
igt_main igt_main
{ {
int fd; int fd = -1;
igt_fixture igt_fixture
fd = drm_open_driver(DRIVER_INTEL); fd = drm_open_driver(DRIVER_INTEL);