svn commit: r214409 - head/sys/kern

Andriy Gapon avg at freebsd.org
Wed Oct 27 16:22:41 UTC 2010


[patch attachment was lost]

on 27/10/2010 19:07 Garrett Cooper said the following:
>     How about this patch? I implemented this as a readonly tunable and

I don't think that it's correct to call it a tunable or use CTLFLAG_RDTUN.
As I understand it is a read-only sysctl.

> sysconf tunable, because (AFAIK) the value that is being tested
> shouldn't change during runtime after the system has been booted up,
> and figuring that the value wasn't going to change it was better to
> lose 4/8 bytes on the kernel stack instead of having to recompute the
> value every time in a function call, with the associated lost heap /
> stack memory in the process, as the assumption is that this libcall
> was going to be called frequently by some programs.
> Thanks!

-- 
Andriy Gapon


More information about the svn-src-all mailing list