linux-kernel - Re: [PATCH] padata: do not leak refcount in reorder

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <01bc9f1e-5a49-4387-b0cb-c2f8647e3b6e@sirena.org.uk>
Date: Thu, 22 May 2025 15:02:14 +0100
From: Mark Brown <broonie@...nel.org>
To: Dominik Grzegorzek <dominik.grzegorzek@...cle.com>
Cc: "aishwarya.tcv@....com" <aishwarya.tcv@....com>,
	"chenridong@...wei.com" <chenridong@...wei.com>,
	Daniel Jordan <daniel.m.jordan@...cle.com>,
	"herbert@...dor.apana.org.au" <herbert@...dor.apana.org.au>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-crypto@...r.kernel.org" <linux-crypto@...r.kernel.org>,
	"steffen.klassert@...unet.com" <steffen.klassert@...unet.com>
Subject: Re: [PATCH] padata: do not leak refcount in reorder_work

On Thu, May 22, 2025 at 01:44:29PM +0000, Dominik Grzegorzek wrote:
> On Thu, 2025-05-22 at 14:10 +0100, Aishwarya wrote:

> > self->n1, self->n2) (-2) == 0 (0)
> > 11213 03:29:55.022519  # # # shared_anon_thp: Test terminated by assertion
> > 11214 03:29:55.022834  # # #          FAIL  migration.shared_anon_thp
> > 11215 03:29:55.027864  # # not ok 4 migration.shared_anon_thp
> > 11216 03:29:55.028156  # # #  RUN           migration.private_anon_htlb ...
> > 11217 03:30:14.595327  # # #            OK  migration.private_anon_htlb
> > 11218 03:30:14.595777  # # ok 5 migration.private_anon_htlb
> > 11219 03:30:14.596398  # # #  RUN           migration.shared_anon_htlb ...
> > 11220 03:30:34.595239  # # #            OK  migration.shared_anon_htlb
> > 11221 03:30:34.595623  # # ok 6 migration.shared_anon_htlb

> Looking at the test, I don't think this is related. The test allocates some
> pages and attempts to migrate them between two NUMA nodes. It seems to be a
> selftest for memory management code, and I don't see how this patch could affect
> its outcome.

The hugepages code does make use of padata and it's a hugepages test so
there's some potential connection there, definitely not an obvious one
though.

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)