On 28.01.2025 10:29, Krzysztof Karas wrote:
Hi Maciej,
The locked==true looks OK.
thanks for review!
However, I fear that there is some corner case with locked==false. 1 or 2
calls back in chain looks good.
CI failures needs to be analyzed.
Yup, I already did that recently. I thought those were
suspicious, but I could not reproduce them locally on the
same platforms (I ran singluar tests and whole test suites from
igt_runner*.txt files as well). I re-triggered the tests for
that patch and the failures did not appear again:
https://patchwork.freedesktop.org/series/143402/#rev2
Krzysztof
Looks good to me.
Reviewed-by: Maciej Patelczyk <maciej.patelczyk@xxxxxxxxx>
