Re: CVS commit: src/sys/arch/amd64/include

[Antti Kantee <pooka%cs.hut.fi@localhost>]

On Fri Mar 04 2011 at 11:42:10 +0100, Joerg Sonnenberger wrote:
> On Fri, Mar 04, 2011 at 04:33:17AM +0000, Christos Zoulas wrote:
> > In article <20110304033425.0E010175DF%cvs.netbsd.org@localhost>,
> > Joerg Sonnenberger <source-changes-d%NetBSD.org@localhost> wrote:
> > >-=-=-=-=-=-
> > >
> > >Module Name:       src
> > >Committed By:      joerg
> > >Date:              Fri Mar  4 03:34:24 UTC 2011
> > >
> > >Modified Files:
> > >   src/sys/arch/amd64/include: vmparam.h
> > >
> > >Log Message:
> > >Reduce MAXSSIZ to 64MB, otherwise netbsd32 binaries crash in ld.elf_so,
> > >including the trivial main(){}. Add a warning to not modify this without
> > >testing compatibility mode.
> > >
> > 
> > Please revert this. The 32 bit emulation is supposed to be using MAXSSIZ32.
> 
> Feel free to revert the change after testing that it actually works.
> I've spend enough time looking for the regression.

If only we had some infrastructure for periodically testing that a program
compiled with -m32 works on amd64!  Good thing that it's not my time being
wasted or I'd try to prevent it from happening in the future.

-- 
älä karot toivorikkauttas, kyl rätei ja lumpui piisaa