Re: 6.11/regression/bisected - after commit 1b04dcca4fb1, launching some RenPy games causes computer hang

[Leo Li <sunpeng.li@amd.com>]

On 2024-09-04 18:21, Mikhail Gavrilov wrote:
> On Wed, Sep 4, 2024 at 4:15 AM Leo Li <sunpeng.li@amd.com> wrote:
>> Hi Mike,
>>
>> Super sorry for the ridiculous wait. Your first two emails slipped by my inbox,
>> which is really silly, given I'm first in the to field...
>>
>> Thanks for bisecting and finding a free game to reproduce it on. I did not have
>> luck reproducing this today, but I am on sway and not gnome. While I get gnome
>> set up, will you be able to test which one of these reverts fixes the hang for
>> you? Whether just 1/2 is enough, or both 1/2 and 2/2 is required?
>>
>> I applied them on top of Linus's v6.11-rc6 tag, so hopefully they'll git am
>> cleanly for you:
>>
>> 1/2:
>> https://gist.github.com/leeonadoh/69147b5fa8d815b39c5f4c3e005cca28#file-0001-revert-drm-amd-display-move-primary-plane-zpos-highe-patch
>> 2/2:
>> https://gist.github.com/leeonadoh/69147b5fa8d815b39c5f4c3e005cca28#file-0002-revert-drm-amd-display-introduce-overlay-cursor-mode-patch
>>
> 
> The first patch is not enough.
> Yes, it fixes the system hang when I launch the game "Find the Orange Narwhal".
> But it does not fix the issue completely.
> Some RenPy games still can lead the system to hang.
> For example "Innocence Or Money Season 1"
> https://store.steampowered.com/app/1958390/Innocence_Or_Money_Season_1__Episodes_1_to_3/
> on the language selection screen.
> 
> Unfortunately the kernel is not builded with both patches.
> I have got compilation error after applying second patch:
> 
>    CC [M]  drivers/gpu/drm/nouveau/nvkm/engine/fifo/chid.o
> drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c: In
> function ‘amdgpu_dm_atomic_check’:
> drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:11003:69:
> error: unused variable ‘new_cursor_state’ [-Werror=unused-variable]
> 11003 |         struct drm_plane_state *old_plane_state,
> *new_plane_state, *new_cursor_state;

Can you delete ", new_cursor_state" on that line and try again? Seems to be a
unused variable warning being elevated to an error.

Thanks,
Leo

>        |
>       ^~~~~~~~~~~~~~~~
>    CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/basics/conversion.o
> ***
>    CC [M]  drivers/gpu/drm/nouveau/nvkm/engine/gr/tu102.o
> cc1: all warnings being treated as errors
>    CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/dml/calcs/dcn_calc_auto.o
>    CC [M]  drivers/gpu/drm/nouveau/nvkm/engine/gr/ga102.o
>    CC [M]  drivers/gpu/drm/nouveau/nvkm/engine/gr/ad102.o
>    CC [M]  drivers/gpu/drm/nouveau/nvkm/engine/gr/r535.o
>    CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/clk_mgr/clk_mgr.o
>    CC [M]  drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxnv40.o
>    CC [M]  drivers/gpu/drm/amd/amdgpu/../display/dc/clk_mgr/dce60/dce60_clk_mgr.o
> make[6]: *** [scripts/Makefile.build:244:
> drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.o] Error 1
> make[6]: *** Waiting for unfinished jobs....
>    CC [M]  drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxnv50.o
> ***
> make[5]: *** [scripts/Makefile.build:485: drivers/gpu/drm/amd/amdgpu] Error 2
> make[4]: *** [scripts/Makefile.build:485: drivers/gpu/drm] Error 2
> make[3]: *** [scripts/Makefile.build:485: drivers/gpu] Error 2
> make[2]: *** [scripts/Makefile.build:485: drivers] Error 2
> make[1]: *** [/home/mikhail/packaging-work/git/linux-3/Makefile:1925: .] Error 2
> make: *** [Makefile:224: __sub-make] Error 2
>