linux-kernel - Re: Broken xarray testing in v6.9-rc1

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: <ZifvRx0-aGQoDQgO@bombadil.infradead.org>
Date: Tue, 23 Apr 2024 10:26:31 -0700
From: Luis Chamberlain <mcgrof@...nel.org>
To: "Liam R. Howlett" <Liam.Howlett@...cle.com>,
	linux-fsdevel@...r.kernel.org, linux-mm@...ck.org,
	linux-kernel@...r.kernel.org, hare@...e.com, p.raghav@...sung.com,
	da.gomez@...sung.com, djwong@...nel.org, david@...morbit.com,
	willy@...radead.org, akpm@...ux-foundation.org
Subject: Re: Broken xarray testing in v6.9-rc1

On Tue, Apr 23, 2024 at 10:18:36AM -0400, Liam R. Howlett wrote:
> Hello,
> 
> Testing of the xarray using the radix tree test suite is currently
> broken in v6.9-rc1 and beyond.  A bisect resulted in your commit failing
> to build:
> 
> a60cc288a1a2604bd86d3df129f269887018c3cb is the first bad commit
> commit a60cc288a1a2604bd86d3df129f269887018c3cb
> Author: Luis Chamberlain <mcgrof@...nel.org>
> 
> ...
> 
> Building in the test suite now fails in linux/tools/testing/radix-tree:
> 
> {bisect/bad(bisect)//radix-tree} $ make
> cc  -O2 -g -Wall -I. -I../../include -I../../../lib -g -Og -Wall -D_LGPL_SOURCE -fsanitize=address -fsanitize=undefined   -c -o xarray.o xarray.c
> In file included from xarray.c:18:
> ../../../lib/test_xarray.c: In function ‘test_get_entry’:
> ../../../lib/test_xarray.c:750:17: warning: implicit declaration of function ‘schedule’ [-Wimplicit-function-declaration]
>   750 |                 schedule();
>       |                 ^~~~~~~~
> ../../../lib/test_xarray.c: In function ‘check_xa_multi_store_adv’:
> ../../../lib/test_xarray.c:767:24: error: ‘PAGE_SHIFT’ undeclared (first use in this function)
>   767 |         index = pos >> PAGE_SHIFT;
>       |                        ^~~~~~~~~~
> ../../../lib/test_xarray.c:767:24: note: each undeclared identifier is reported only once for each function it appears in
> make: *** [<builtin>: xarray.o] Error 1
> 
> Can you please have a look?

Sure, I'll post patches right away, this also demonstrated an issue in
the test code too which I've fixed and will post a fix for too.

Thanks for reporting it.

  Luis