Received-SPF: pass (google.com: domain of linux-kernel+bounces-155675-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Tue, 23 Apr 2024 10:26:31 -0700
From: Luis Chamberlain <mcgrof@kernel.org>
To: "Liam R. Howlett" <Liam.Howlett@oracle.com>,
	linux-fsdevel@vger.kernel.org, linux-mm@kvack.org,
	linux-kernel@vger.kernel.org, hare@suse.com, p.raghav@samsung.com,
	da.gomez@samsung.com, djwong@kernel.org, david@fromorbit.com,
	willy@infradead.org, akpm@linux-foundation.org
Subject: Re: Broken xarray testing in v6.9-rc1
Message-ID: <ZifvRx0-aGQoDQgO@bombadil.infradead.org>
References: <vm5b7c2jwmvptcnpgofwunjg4supq3snn63xqklidudnzlnhuv@kanproehzmdr>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <vm5b7c2jwmvptcnpgofwunjg4supq3snn63xqklidudnzlnhuv@kanproehzmdr>
Sender: Luis Chamberlain <mcgrof@infradead.org>

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@kernel.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