Re: git: 4f741801d860 - main - t_setrlimit: Adjust resource limit to 20M

From: Kyle Evans <kevans_at_freebsd.org>
Date: Mon, 20 Dec 2021 07:54:22 UTC
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 anyway or do whatever you decide.
>
> Regards,
> Wojtek
>
> pon., 20 gru 2021 o 07:51 Kyle Evans <kevans@freebsd.org> napisaƂ(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=4f741801d86089a1c5d631ba1e0f1421cdcf7a7e
>> >
>> > 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 fact
>> >     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, adjusting
>> >     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