2022-04-30 17:29:01

by Liam R. Howlett

[permalink] [raw]
Subject: Re: [PATCH] mapletree-vs-khugepaged

* Heiko Carstens <[email protected]> [220429 09:19]:
> On Fri, Apr 29, 2022 at 01:01:53PM +0000, Liam Howlett wrote:
> > * Heiko Carstens <[email protected]> [220429 08:10]:
> > > On Thu, Apr 28, 2022 at 10:20:40AM -0700, Guenter Roeck wrote:
> > > > On Wed, Apr 27, 2022 at 03:10:45PM -0700, Andrew Morton wrote:
> > > > > Fix mapletree for patch series "Make khugepaged collapse readonly FS THP
> > > > > more consistent", v3.
> > > > >
> > > > > Cc: Liam R. Howlett <[email protected]>
> > > > > Signed-off-by: Andrew Morton <[email protected]>
> > > >
> > > > This patch causes all my sparc64 boot tests to fail. Bisect and crash logs
> > > > attached.
> > > >
> > > > Guenter
> ...
> > >
> > > FWIW, same on s390 - linux-next is completely broken. Note: I didn't
> > > bisect, but given that the call trace, and even the failing address
> > > match, I'm quite confident it is the same reason.
> >
> > This is worth a lot to me. Thanks for the report and the testing.
>
> Not sure if it is of any relevance, and you are probably aware if it
> anyway, but both sparc64 and s390 are big endian; and there was no
> report from little endian architectures yet.

I was aware they are big endian, but thanks - the more info the better.
sparc64 is technically bi-endian but I think everyone runs it in big
endian mode? Is alpha the same? There was a report of alpha having
issues too. m68k is also big endian - but also nommu, so that makes
testing difficult.

What I liked about the s390 report is that the s390 is very good at
finding vma issues since it seems to use move_vma (among others) a lot
more than other arch. I've built my maple tree + v5.18-rc2 and
successfully booted with KASAN and poison on the s390. Andrew asked me
to respin maple on top of one of his branches with the fixes rolled in
so I'm going to work on that while the m68k buildroot compiles.

In parallel, I'm running a (very slow qemu) install of sparc64 and
trying to figure out how to get a qemu setup for those. I'm trying to
follow what Guenter has in his repo[1] and have found debian ISOs[2]
that may help with some of these targets.


1. https://github.com/groeck/linux-build-test
2. https://cdimage.debian.org/cdimage/ports/snapshots/


2022-05-03 00:15:02

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [PATCH] mapletree-vs-khugepaged

Hi Liam,

On Sat, Apr 30, 2022 at 1:58 AM Liam Howlett <[email protected]> wrote:
> * Heiko Carstens <[email protected]> [220429 09:19]:
> > On Fri, Apr 29, 2022 at 01:01:53PM +0000, Liam Howlett wrote:
> > > * Heiko Carstens <[email protected]> [220429 08:10]:
> > > > On Thu, Apr 28, 2022 at 10:20:40AM -0700, Guenter Roeck wrote:
> > > > > On Wed, Apr 27, 2022 at 03:10:45PM -0700, Andrew Morton wrote:
> > > > > > Fix mapletree for patch series "Make khugepaged collapse readonly FS THP
> > > > > > more consistent", v3.
> > > > > >
> > > > > > Cc: Liam R. Howlett <[email protected]>
> > > > > > Signed-off-by: Andrew Morton <[email protected]>
> > > > >
> > > > > This patch causes all my sparc64 boot tests to fail. Bisect and crash logs
> > > > > attached.
> > > > >
> > > > > Guenter
> > ...
> > > >
> > > > FWIW, same on s390 - linux-next is completely broken. Note: I didn't
> > > > bisect, but given that the call trace, and even the failing address
> > > > match, I'm quite confident it is the same reason.
> > >
> > > This is worth a lot to me. Thanks for the report and the testing.
> >
> > Not sure if it is of any relevance, and you are probably aware if it
> > anyway, but both sparc64 and s390 are big endian; and there was no
> > report from little endian architectures yet.
>
> I was aware they are big endian, but thanks - the more info the better.
> sparc64 is technically bi-endian but I think everyone runs it in big

Sparc64 is big-endian. It has support for accessing little endian data
in memory, but that's merely an optimization.

> endian mode? Is alpha the same? There was a report of alpha having

Alpha is little-endian.

> issues too. m68k is also big endian - but also nommu, so that makes
> testing difficult.

M68k exists with and without MMU.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds