From owner-svn-src-head@FreeBSD.ORG  Sun Mar 10 22:56:02 2013
Return-Path: <owner-svn-src-head@FreeBSD.ORG>
Delivered-To: svn-src-head@freebsd.org
Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115])
 by hub.freebsd.org (Postfix) with ESMTP id B6E88FAD
 for <svn-src-head@freebsd.org>; Sun, 10 Mar 2013 22:56:02 +0000 (UTC)
 (envelope-from andre@freebsd.org)
Received: from c00l3r.networx.ch (c00l3r.networx.ch [62.48.2.2])
 by mx1.freebsd.org (Postfix) with ESMTP id 2E79563F
 for <svn-src-head@freebsd.org>; Sun, 10 Mar 2013 22:56:02 +0000 (UTC)
Received: (qmail 97466 invoked from network); 11 Mar 2013 00:08:53 -0000
Received: from c00l3r.networx.ch (HELO [127.0.0.1]) ([62.48.2.2])
 (envelope-sender <andre@freebsd.org>)
 by c00l3r.networx.ch (qmail-ldap-1.03) with SMTP
 for <alc@rice.edu>; 11 Mar 2013 00:08:53 -0000
Message-ID: <513D0F80.80405@freebsd.org>
Date: Sun, 10 Mar 2013 23:56:00 +0100
From: Andre Oppermann <andre@freebsd.org>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64;
 rv:17.0) Gecko/20130107 Thunderbird/17.0.2
MIME-Version: 1.0
To: Alan Cox <alc@rice.edu>
Subject: Re: svn commit: r248031 - in head/sys: kern sys
References: <201303081014.r28AEwet053196@svn.freebsd.org>
 <513A2B3E.2070804@rice.edu>
In-Reply-To: <513A2B3E.2070804@rice.edu>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org,
 src-committers@freebsd.org
X-BeenThere: svn-src-head@freebsd.org
X-Mailman-Version: 2.1.14
Precedence: list
List-Id: SVN commit messages for the src tree for head/-current
 <svn-src-head.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-head>
List-Post: <mailto:svn-src-head@freebsd.org>
List-Help: <mailto:svn-src-head-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 10 Mar 2013 22:56:02 -0000

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.

I was investigating the current users of timeout() and couldn't find any
remaining direct relation to maxproc or maxfiles.

The use of timeout(9) is almost entirely limited to drivers in the CAM/SCSI
area to manage CCB's.  The highest number of outstanding transactions and
timers seems to be limited to around 256 per driver.  The majority of those
also seem to be rather easy and mostly mechanical callout_* conversion targets.
This is only a preliminary finding though, I didn't attempt to convert any of
those.

-- 
Andre