Re: [PATCH i-g-t 0/3] Few fixes for igt_display_reset_outputs() in lib/igt_kms

Ville Syrjälä
2026-04-27T12:24:57Z

On Fri, Apr 24, 2026 at 05:13:52PM +0200, Kamil Konieczny wrote:
> Discussion around a fix for asserts in
>
> https://patchwork.freedesktop.org/series/165136/
> tests/kms: correct index validation logic
>
> lead to conclusion that the missing part was additional
> swap of pointers. While looking at code I also spot one
> more potential problem, fixed in 3rd commit.
>
> Cc: Mark Yacoub <markyacoub@chromium.org>
> Cc: Lyude Paul <lyude@redhat.com>
> Cc: Jani Nikula <jani.nikula@intel.com>
> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> Kamil Konieczny (3):
> Revert "tests/kms: correct index validation logic"
> lib/igt_kms: Fix assert in igt_display_reset_outputs()
> lib/igt_kms: Fix cleanup in igt_display_reset_outputs()

For the series
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

>
> lib/igt_kms.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> --
> 2.54.0

--
Ville Syrjälä
Intel