[igt-dev] [PATCH i-g-t] igt: Use lib gem_execbuf where possible
Antonio Argenziano
antonio.argenziano at intel.com
Fri Feb 9 15:25:33 UTC 2018
On 09/02/18 01:43, Chris Wilson wrote:
> Quoting Antonio Argenziano (2018-02-09 00:21:57)
>> diff --git a/tests/gem_busy.c b/tests/gem_busy.c
>> index c349c291..444c1dd1 100644
>> --- a/tests/gem_busy.c
>> +++ b/tests/gem_busy.c
>> @@ -332,11 +332,6 @@ static void alarm_handler(int sig)
>> {
>> }
>>
>> -static int __execbuf(int fd, struct drm_i915_gem_execbuffer2 *execbuf)
>> -{
>> - return ioctl(fd, DRM_IOCTL_I915_GEM_EXECBUFFER2, execbuf);
>> -}
>
> Wrong. There's an important difference here, can you see it? It's all to
> do with signal handling...
I guess we do not want to keep trying doing the same ioctl once the
ringbuffer is full...
Thanks for finding this, it would have been a nasty one.
Antonio
> -Chris
>
More information about the igt-dev
mailing list