Date: Sun, 10 Mar 2013 19:42:15 -0500 From: Alan Cox <alc@rice.edu> To: Andre Oppermann <andre@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r248031 - in head/sys: kern sys Message-ID: <513D2867.2020206@rice.edu> In-Reply-To: <513D0F80.80405@freebsd.org> References: <201303081014.r28AEwet053196@svn.freebsd.org> <513A2B3E.2070804@rice.edu> <513D0F80.80405@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 03/10/2013 17:56, Andre Oppermann wrote: > On 08.03.2013 19:17, Alan Cox wrote: >> On 03/08/2013 04:14, Andre Oppermann wrote: >>> Author: andre >>> Date: Fri Mar 8 10:14:58 2013 >>> New Revision: 248031 >>> URL: http://svnweb.freebsd.org/changeset/base/248031 >>> >>> Log: >>> Move the auto-sizing of the callout array from init_param2() to >>> kern_timeout_callwheel_alloc() where it is actually used. >>> >>> This is a mechanical move and no tuning parameters are changed. >>> >>> The pre-allocated callout array is only used for legacy timeout(9) >>> calls and is only allocated and active on cpu0. Eventually all >>> remaining users of timeout(9) should switch to the callout_* API. >>> >> >> In the meantime, until all legacy timeout(9) users are updated, I think >> that it would be wise to retain the comment that describes where the >> magic number 18508 comes from. > > A valid point. Reintroduced in r248141. Thank you!
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?513D2867.2020206>