Quoting Antonio Argenziano (2017-10-13 21:49:28)
> static void gem_require_context(int fd)
> {
> - igt_require(__gem_context_create(fd));
> + uint32_t ctx_id = 0;
> + __gem_context_create(fd, &ctx_id);
> + igt_require(ctx_id);
Still that leaks.
uint32_t has_contexts = 0;
__gem_context_create(fd, &has_contexts);
if (has_contexts)
gem_context_destroy(fd, has_contexts);
igt_require(has_contexts);
Note that before we can drop the require from gem_context_create() we
have to go throw and teach all users to do the manual
gem_require_context() first.
Second note,
static bool gem_has_contexts(int fd)
{
uint32_t ctx_id = 0;
__gem_context_create(fd, &ctx_id);
if (ctx_id)
gem_context_destroy(fd, ctx_id);
return ctx_id;
}
void gem_require_contexts(int fd)
{
igt_require(gem_has_contexts(fd));
}
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
