Date: Mon, 20 Dec 2021 01:54:22 -0600 From: Kyle Evans <kevans@freebsd.org> To: Wojciech Macek <wma@semihalf.com> Cc: Marcin Wojtas <mw@freebsd.org>, Wojciech Macek <wma@freebsd.org>, src-committers <src-committers@freebsd.org>, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, dev-commits-src-main@freebsd.org Subject: Re: git: 4f741801d860 - main - t_setrlimit: Adjust resource limit to 20M Message-ID: <CACNAnaG83XFumDDVP378JAheOANoYBsi0ADEYE4LUAhGJtbtdA@mail.gmail.com> In-Reply-To: <CANsEV8erfPbgTVNGqfmfS9Rv6H_PdwMd2ubXBvJfQc2=U4yzTQ@mail.gmail.com> References: <202112200529.1BK5T0np047358@gitrepo.freebsd.org> <CACNAnaHMg5WpXQT_TEVPq3STmtgjwL1Mhzc3c02zvS1UR-Epmg@mail.gmail.com> <CANsEV8erfPbgTVNGqfmfS9Rv6H_PdwMd2ubXBvJfQc2=U4yzTQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
I found the PR that I was thinking of, that you folks might want to join if you haven't already: https://bugs.freebsd.org/260303 I don't think a revert's necessary, just something to consider if this ends up becoming a non-issue due to other low-level changes. Thanks, Kyle Evans On Mon, Dec 20, 2021 at 1:32 AM Wojciech Macek <wma@semihalf.com> wrote: > > Hi Kyle, > > Thanks for the comment, I thought this one is discussed but apparently I = was mistaken. > Adding Marcin to handle this on our side. I'm fine with reverting it anyw= ay or do whatever you decide. > > Regards, > Wojtek > > pon., 20 gru 2021 o 07:51 Kyle Evans <kevans@freebsd.org> napisa=C5=82(a)= : >> >> On Sun, Dec 19, 2021 at 11:29 PM Wojciech Macek <wma@freebsd.org> wrote: >> > >> > The branch main has been updated by wma: >> > >> > URL: https://cgit.FreeBSD.org/src/commit/?id=3D4f741801d86089a1c5d631b= a1e0f1421cdcf7a7e >> > >> > commit 4f741801d86089a1c5d631ba1e0f1421cdcf7a7e >> > Author: Dawid Gorecki <dgr@semihalf.com> >> > AuthorDate: 2021-12-20 05:27:12 +0000 >> > Commit: Wojciech Macek <wma@FreeBSD.org> >> > CommitDate: 2021-12-20 05:28:20 +0000 >> > >> > t_setrlimit: Adjust resource limit to 20M >> > >> > With ASLR enabled by default, RLIMIT_STACK test fails due to the f= act >> > that default stack gap can be as big as 15M. Because of that the >> > resource limit of 4M results in test program receiving SIGSEGV >> > immediately after exiting the setrlimit syscall. Since the idea of= this >> > test is to check if rlim_cur does not extend past rlim_max, adjust= ing >> > the resource limit to 20M should not invalidate the test results. >> > >> >> I can't seem to find the PR at the moment, but isn't this exactly the >> kind of thing markj@ talked about trying to avoid elsewhere? It seems >> like this probably should've been involved in the discussion rather >> than hacked around independently. >> >> Thanks, >> >> Kyle Evans
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CACNAnaG83XFumDDVP378JAheOANoYBsi0ADEYE4LUAhGJtbtdA>