igt/gem_exec_nop: Fix logical inversion for checking of valid execbuf

Only if the trial __gem_execbuf reports an error do we want to remove
the fancy LUT flags.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
This commit is contained in:
Chris Wilson 2016-03-10 11:50:53 +00:00
parent 544ba6ca88
commit 3e2443f838
2 changed files with 3 additions and 4 deletions

View File

@ -105,7 +105,6 @@ static void *gem_busyspin(void *arg)
nengine = 0;
for_each_engine(fd, engine)
if (!ignore_engine(fd, engine)) engines[nengine++] = engine;
igt_require(nengine);
memset(&obj, 0, sizeof(obj));
obj.handle = gem_create(fd, 4096);
@ -116,7 +115,7 @@ static void *gem_busyspin(void *arg)
execbuf.buffer_count = 1;
execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf) == 0) {
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = 0;
gem_execbuf(fd, &execbuf);
}

View File

@ -72,7 +72,7 @@ static void single(int fd, uint32_t handle, unsigned ring_id, const char *ring_n
execbuf.flags = ring_id;
execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf) == 0) {
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = ring_id;
gem_execbuf(fd, &execbuf);
}
@ -127,7 +127,7 @@ static void all(int fd, uint32_t handle)
execbuf.buffer_count = 1;
execbuf.flags |= LOCAL_I915_EXEC_HANDLE_LUT;
execbuf.flags |= LOCAL_I915_EXEC_NO_RELOC;
if (__gem_execbuf(fd, &execbuf) == 0) {
if (__gem_execbuf(fd, &execbuf)) {
execbuf.flags = 0;
gem_execbuf(fd, &execbuf);
}