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 for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <f41505e6-62da-480c-86ed-5eccd309cc2d@yandex-team.ru>
Date: Thu, 2 Oct 2025 15:57:46 +0300
From: Maksim Davydov <davydov-max@...dex-team.ru>
To: Pedro Falcato <pfalcato@...e.de>
Cc: "H. Peter Anvin" <hpa@...or.com>,
 "Peter Zijlstra (Intel)" <peterz@...radead.org>,
 linux-kernel@...r.kernel.org, Thomas Gleixner <tglx@...utronix.de>,
 Ingo Molnar <mingo@...hat.com>, Borislav Petkov <bp@...en8.de>,
 Dave Hansen <dave.hansen@...ux.intel.com>, x86@...nel.org
Subject: Re: [PATCH RESEND] x86/split_lock: Warn for bus locks once for each
 task



On 9/17/25 18:37, Pedro Falcato wrote:
> On Wed, Sep 17, 2025 at 05:41:11PM +0300, Maksim Davydov wrote:
>> Hi!
> 
> Hi!
> 
>> I've tested this patch (VM w/ bus lock detection and w/o split lock
>> detection). The warn mode works fine and as expected with only one
>> notification per task. However, the ratelimit mode has been changed too:
>> only one notification per task will be in dmesg, because this mode reuses
>> notification code of warn mode. But in the documentation for ratelimit mode
>> there is nothing about the limit of notifications: "Limit bus lock rate to N
>> bus locks per second system wide and warn on bus locks.". Thus, I think that
>> ratelimit mode should remain old behaviour.
>>
> 
> So, you're suggesting different "warn" behaviors depending on =warn vs
> =ratelimit? I don't have a particularly strong opinion here, but I was assuming
> that "warn on bus locks" means "do whatever =warn is supposed to do". I'm not
> sure warning every time gains you much, though.
> 

For me, it's ok to reduce the number of messages to one per task.

By the way, I read the patch again and noticed that split_lock_warn() 
and handle_bus_lock() print messages in the same way. Maybe, it'll be 
better to implement an auxiliary wrapper for printing 
split_lock/bus_lock warning messages (for example, 
print_sl_warn_once()), what do you think?

-- 
Best regards,
Maksim Davydov


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ