Re: [PATCH] drm/i915/selftests: avoid using uninitialized context

2025-01-28 Thread Krzysztof Karas
Hi Sebastian, thanks for review. > I don't thing it's a best idea to just initialize ww here, you still have > incorrect path that try to fini ww before it was initialize. Fair point - we still call i915_gem_ww_ctx_fini(), which is useless without actual ww. > > I would probably do something lik

Re: [PATCH] drm/i915/selftests: avoid using uninitialized context

2025-01-27 Thread Sebastian Brzezinka
Hi Krzysztof, On Mon Jan 27, 2025 at 11:46 AM UTC, Krzysztof Karas wrote: > There is an error path in igt_ppgtt_alloc(), which leads to ww > object being passed down to i915_gem_ww_ctx_fini() without > initialization. Correct that by zeroing the struct. > > Fixes: 480ae79537b2 ("drm/i915/selftests