netdev - Re: [herbert-cryptodev-2.6:master] [rhashtable] e1d3422c95: stress-ng.syscall.ops_per

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: <20250102-bizarre-griffin-of-attraction-c1f728@leitao>
Date: Thu, 2 Jan 2025 07:45:53 -0800
From: Breno Leitao <leitao@...ian.org>
To: Herbert Xu <herbert@...dor.apana.org.au>
Cc: Tejun Heo <tj@...nel.org>, kernel test robot <oliver.sang@...el.com>,
	oe-lkp@...ts.linux.dev, lkp@...el.com, linux-crypto@...r.kernel.org,
	netdev@...r.kernel.org
Subject: Re: [herbert-cryptodev-2.6:master] [rhashtable]  e1d3422c95:
 stress-ng.syscall.ops_per_sec 98.9% regression

On Thu, Jan 02, 2025 at 09:10:15AM +0800, Herbert Xu wrote:
> On Sun, Dec 29, 2024 at 12:54:47PM -1000, Tejun Heo wrote:
> >
> > Hmm... the only meaningful behavior difference would be that after the
> > patch, rht_grow_above_75() test is done regardless of the return value while
> > before it was done only when the return value is zero. Breno, can you please
> > look into whether this report is valid and whether restoring the NULL check
> > makes it go away?
> 
> Actually I fixed that when committing the patch.  It should be
> conditional on whether the insertion succeeds or not.

Thanks.

I am finally back from vacation. I will try to reproduce the issue
reported in this report, and double-check the regression.

--breno