Re: [PATCH] drm/i915: Fix vm reference leak in mock_gem_device()

From: Joonas Lahtinen

Date: Thu Sep 17 2026 - 03:03:35 EST


Quoting Wentao Liang (2026-09-16 20:19:35)
> mock_gem_device() takes a reference on the GGTT vm for gt->vm, but
> neither mock_device_release() nor the setup error paths drop it, unlike
> the real teardown where intel_gt_driver_release() releases gt->vm.
> Drop the reference before mock_fini_ggtt() and on the error paths.
>
> Fixes: cdeea858d8df ("drm/i915: Remove unused i915->ggtt")
> Cc: stable@xxxxxxxxxxxxxxx

We're not putting Fixes and definitely not Cc: stable for selftest
improvements.

> Signed-off-by: Wentao Liang <vulab@xxxxxxxxxxx>

You clearly did not try to address any of the issues per the surrounding
coding style or understand the severity before Cc stable, but just generated
some static code fixes and commit messages with LLM.

Looking at your history of spamming the LKML with 30+ patches per day
and not a single response for requests to fix things or add the
Assisted-by tag, don't really expect you or even your LLM to read this.
So this reply is more for documentation purposes.

Doesn't really inspire any confidence to merge anything or look at any
further of your multiple patches which you send separately and not in
series in the hopes of getting some merged even when pushback in others.

If I happen to be wrong, let me know.

Regards, Joonas