[PATCH] drm/i915: replace in_atomic() with manually set flag

Maciej Patelczyk maciej.patelczyk at intel.com
Mon Feb 3 11:46:50 UTC 2025


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 at intel.com>



More information about the Intel-gfx mailing list