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-prev] [day] [month] [year] [list]
Message-ID: <wox3agrrleq5ba24kjm3iyobelzo5ag45qukqouqs3k5mxyu2y@jfbyou4kuy4g>
Date: Tue, 14 Oct 2025 01:36:48 +0530
From: Brahmajit Das <listout@...tout.xyz>
To: Yonghong Song <yonghong.song@...ux.dev>
Cc: Sahil Chandna <chandna.linuxkernel@...il.com>, ast@...nel.org, 
	daniel@...earbox.net, andrii@...nel.org, martin.lau@...ux.dev, song@...nel.org, 
	john.fastabend@...il.com, haoluo@...gle.com, jolsa@...nel.org, bpf@...r.kernel.org, 
	netdev@...r.kernel.org, david.hunter.linux@...il.com, skhan@...uxfoundation.org, 
	khalid@...nel.org
Subject: Re: [PATCH v3] bpf: test_run: fix atomic context in timer path
 causing sleep-in-atomic BUG

On 14.10.2025 01:31, Brahmajit Das wrote:
> On 13.10.2025 11:35, Yonghong Song wrote:
> > 
> > 
> > On 10/13/25 10:11 AM, Sahil Chandna wrote:
> > > The timer mode is initialized to NO_PREEMPT mode by default,
...snip...
> Yeah, my bad. The v2 is mine, which I send few mins before Sahil
> 
> https://lore.kernel.org/all/20251013171122.1403859-1-listout@listout.xyz/T/
https://lore.kernel.org/all/20251013171122.1403859-1-listout@listout.xyz/
> > 
> > In the future, please submit new patch set only after some reviews on the old patch.
> > 
> > I also recommend to replace e.g. [PATCH v3] to [PATCH bpf v3] (or [PATCH bpf-next v3])
> > so CI can do proper testing for either bpf or bpf-next.
> > 
> > For the title:
> >   bpf: test_run: fix atomic context in timer path causing sleep-in-atomic BUG
> > Change to:
> >   bpf: Fix sleep-in-atomic BUG in timer path with RT kernel
> > 
> > The code change LGTM.
> > 
> > Acked-by: Yonghong Song <yonghong.song@...ux.dev>
> > 
> > > 
> > > ---
> > > Changes since v2:
> > > - Fix uninitialized struct bpf_test_timer
...snip...

-- 
Regards,
listout

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ