On Mon, Feb 01, 2021 at 10:24:31PM -0500, Philippe Meunier wrote:
> Anyone?
Fixing a particluar issue is fine, but more important is an assessment
it does not break other things. In particular, does this limit the VM
for data available to any program (which is already quite limited on
i386)?
-Otto
>
> Philippe
>
>
> Philippe Meunier wrote:
> >Jonathan Gray wrote:
> >>MAXTSIZ is 128 MB on i386
> >>see sys/arch/i386/include/vmparam.h
> >
> >Mark Kettenis wrote:
> >>sys/arch/i386/include/vmparam.h has:
> >>#define MAXTSIZ (128*1024*1024) /* max text size */
> >
> >Thanks to both of you for the pointer!
> >
> >So what about the patch below? I've checked with a new kernel that it
> >fixes the problem with chrome (even when using the default limits in
> >/etc/login.conf).
> >
> >Philippe
> >
> >
> >Index: sys/arch/i386/include/vmparam.h
> >===================================================================
> >RCS file: /cvs/src/sys/arch/i386/include/vmparam.h,v
> >retrieving revision 1.56
> >diff -u -r1.56 vmparam.h
> >--- sys/arch/i386/include/vmparam.h 17 Apr 2018 15:50:05 -0000 1.56
> >+++ sys/arch/i386/include/vmparam.h 31 Jan 2021 09:41:00 -0000
> >@@ -46,7 +46,7 @@
> > /*
> > * Virtual memory related constants, all in bytes
> > */
> >-#define MAXTSIZ (128*1024*1024) /* max text size */
> >+#define MAXTSIZ (256*1024*1024) /* max text size */
> > #ifndef DFLDSIZ
> > #define DFLDSIZ (64*1024*1024) /* initial data size
> > limit */
> > #endif
> >
> >
>
