linux-kernel - regression/bisected/6.9 commit 587d67fd929ad89801bcc429675bda90d53f6592 decrease 30% of gaming performance

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <CABXGCsNmEtrN9DK-XmESaPm_1xpXm8A+juE+44Jf6AK5JE0+TQ@mail.gmail.com>
Date: Mon, 1 Apr 2024 01:21:20 +0500
From: Mikhail Gavrilov <mikhail.v.gavrilov@...il.com>
To: Takashi Iwai <tiwai@...e.de>, Linux List Kernel Mailing <linux-kernel@...r.kernel.org>, 
	linux-sound@...r.kernel.org
Subject: regression/bisected/6.9 commit 587d67fd929ad89801bcc429675bda90d53f6592
 decrease 30% of gaming performance

Hi,

I spotted that FPS was around 122 [1] in the Shadow of the Tomb Raider
benchmark at commit f6cef5f8c37f but after moving to commit
4ae3dc83b047 it decreased to 84 [2].

I bisected it and the first bad commit was 587d67fd929a.
Author: Takashi Iwai <tiwai@...e.de>
Date:   Fri Mar 15 11:14:42 2024 +0100

    ALSA: timer: Fix missing irq-disable at closing

    The conversion to guard macro dropped the irq-disablement at closing
    mistakenly, which may lead to a race.  Fix it.

    Fixes: beb45974dd49 ("ALSA: timer: Use guard() for locking")
    Reported-by: syzbot+28c1a5a5b041a754b947@...kaller.appspotmail.com
    Closes: http://lore.kernel.org/r/0000000000000b9a510613b0145f@google.com
    Message-ID: <20240315101447.18395-1-tiwai@...e.de>
    Signed-off-by: Takashi Iwai <tiwai@...e.de>

 sound/core/timer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

To make sure of this I builded kernel at commit 712e14250dd2 with
revert 587d67fd929a and the benchmarking result was returned to best
observed by me.

[1] good results - https://postimg.cc/G4NPHMyk
[2] bad results - https://postimg.cc/1n0D7sYH

I attached here my build .config and kernel log.
Is it possible to find a better approach than 587d67fd929a?

-- 
Best Regards,
Mike Gavrilov.

Download attachment ".config.zip" of type "application/zip" (66041 bytes)

Download attachment "6.9.0-rc1-test-rev-587d67fd929ad89801bcc429675bda90d53f6592.zip" of type "application/zip" (57417 bytes)