Date: Mon, 20 Dec 2021 08:32:10 +0100 From: Wojciech Macek <wma@semihalf.com> To: Kyle Evans <kevans@freebsd.org>, Marcin Wojtas <mw@freebsd.org> Cc: 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: <CANsEV8erfPbgTVNGqfmfS9Rv6H_PdwMd2ubXBvJfQc2=U4yzTQ@mail.gmail.com> In-Reply-To: <CACNAnaHMg5WpXQT_TEVPq3STmtgjwL1Mhzc3c02zvS1UR-Epmg@mail.gmail.com> References: <202112200529.1BK5T0np047358@gitrepo.freebsd.org> <CACNAnaHMg5WpXQT_TEVPq3STmtgjwL1Mhzc3c02zvS1UR-Epmg@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--0000000000009f256a05d38ee393 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable 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 anyway 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=3D4f741801d86089a1c5d631ba1e0f142= 1cdcf7a7e > > > > 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 fa= ct > > 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, adjusti= ng > > 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 > --0000000000009f256a05d38ee393 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr">Hi Kyle,<div><br></div><div>Thanks for the comment, I thou= ght this one is discussed=C2=A0but apparently I was mistaken.</div><div>Add= ing Marcin to handle this on our side. I'm fine with reverting it anywa= y or do whatever you decide.</div><div><br></div><div>Regards,</div><div>Wo= jtek</div></div><br><div class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gm= ail_attr">pon., 20 gru 2021 o 07:51=C2=A0Kyle Evans <<a href=3D"mailto:k= evans@freebsd.org">kevans@freebsd.org</a>> napisa=C5=82(a):<br></div><bl= ockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-lef= t:1px solid rgb(204,204,204);padding-left:1ex">On Sun, Dec 19, 2021 at 11:2= 9 PM Wojciech Macek <<a href=3D"mailto:wma@freebsd.org" target=3D"_blank= ">wma@freebsd.org</a>> wrote:<br> ><br> > The branch main has been updated by wma:<br> ><br> > URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D4f741801d860= 89a1c5d631ba1e0f1421cdcf7a7e" rel=3D"noreferrer" target=3D"_blank">https://= cgit.FreeBSD.org/src/commit/?id=3D4f741801d86089a1c5d631ba1e0f1421cdcf7a7e<= /a><br> ><br> > commit 4f741801d86089a1c5d631ba1e0f1421cdcf7a7e<br> > Author:=C2=A0 =C2=A0 =C2=A0Dawid Gorecki <<a href=3D"mailto:dgr@sem= ihalf.com" target=3D"_blank">dgr@semihalf.com</a>><br> > AuthorDate: 2021-12-20 05:27:12 +0000<br> > Commit:=C2=A0 =C2=A0 =C2=A0Wojciech Macek <wma@FreeBSD.org><br> > CommitDate: 2021-12-20 05:28:20 +0000<br> ><br> >=C2=A0 =C2=A0 =C2=A0t_setrlimit: Adjust resource limit to 20M<br> ><br> >=C2=A0 =C2=A0 =C2=A0With ASLR enabled by default, RLIMIT_STACK test fai= ls due to the fact<br> >=C2=A0 =C2=A0 =C2=A0that default stack gap can be as big as 15M. Becaus= e of that the<br> >=C2=A0 =C2=A0 =C2=A0resource limit of 4M results in test program receiv= ing SIGSEGV<br> >=C2=A0 =C2=A0 =C2=A0immediately after exiting the setrlimit syscall. Si= nce the idea of this<br> >=C2=A0 =C2=A0 =C2=A0test is to check if rlim_cur does not extend past r= lim_max, adjusting<br> >=C2=A0 =C2=A0 =C2=A0the resource limit to 20M should not invalidate the= test results.<br> ><br> <br> I can't seem to find the PR at the moment, but isn't this exactly t= he<br> kind of thing markj@ talked about trying to avoid elsewhere? It seems<br> like this probably should've been involved in the discussion rather<br> than hacked around independently.<br> <br> Thanks,<br> <br> Kyle Evans<br> </blockquote></div> --0000000000009f256a05d38ee393--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANsEV8erfPbgTVNGqfmfS9Rv6H_PdwMd2ubXBvJfQc2=U4yzTQ>