linux-kernel - Re: linux-next: build failure after merge of almost all the trees

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]

Date:   Tue, 2 Nov 2021 08:06:10 +0100
From:   Arnd Bergmann <arnd@...db.de>
To:     Stephen Rothwell <sfr@...b.auug.org.au>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Linux Next Mailing List <linux-next@...r.kernel.org>,
        Anton Altaparmakov <anton@...era.com>,
        Joel Stanley <joel@....id.au>,
        Michael Ellerman <mpe@...erman.id.au>,
        Arnd Bergmann <arnd@...db.de>
Subject: Re: linux-next: build failure after merge of almost all the trees

On Tue, Nov 2, 2021 at 7:44 AM Stephen Rothwell <sfr@...b.auug.org.au> wrote:
>
> Hi all,
>
> On Thu, 28 Oct 2021 23:38:44 +1100 Stephen Rothwell <sfr@...b.auug.org.au> wrote:
> >
> > Hi all,
> >
> > On Thu, 28 Oct 2021 21:26:51 +1100 Stephen Rothwell <sfr@...b.auug.org.au> wrote:
> > >
> > > Today's linux-next build (powerpc allyesconfig) failed like this:
> > >
> > > fs/ntfs/aops.c: In function 'ntfs_write_mst_block':
> > > fs/ntfs/aops.c:1311:1: error: the frame size of 2304 bytes is larger than 2048 bytes [-Werror=frame-larger-than=]
> > >  1311 | }
> > >       | ^
> > > cc1: all warnings being treated as errors
> > >
> > > I have no idea what has caused this.
> >
> > With a nudge from Arnd, it seems the immediate case was commit
> >
> >   f22969a66041 ("powerpc/64s: Default to 64K pages for 64 bit book3s")
> >
> > from the powerpc tree switching the allyesconfig build from 4k pages to
> > 64k pages which expanded a few arrays on the stack in that function.
>
> Can we do something about this, please?

I submitted a workaround a while ago. Anton didn't like it, but has not
come up with a proper fix in ntfs either:

https://lore.kernel.org/lkml/20210927141815.1711736-1-arnd@kernel.org/

It does need to be changed a bit as I realized it depends on a rework of
the Kconfig logic that I had in my randconfig build tree to have a common
page size symbol across architectures. Without my other patch, it also
needs to check for PPC_64K_PAGES.

Should I send an updated version of the patch?

       Arnd