From owner-freebsd-current@FreeBSD.ORG  Fri Mar 18 00:03:27 2005
Return-Path: <owner-freebsd-current@FreeBSD.ORG>
Delivered-To: freebsd-current@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 279C716A4CF
	for <current@freebsd.org>; Fri, 18 Mar 2005 00:03:27 +0000 (GMT)
Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id BFB2843D31; Fri, 18 Mar 2005 00:03:26 +0000 (GMT)
	(envelope-from davidxu@freebsd.org)
Received: from [127.0.0.1] (davidxu@localhost [127.0.0.1])
	by freefall.freebsd.org (8.13.3/8.13.3) with ESMTP id j2I03PTa019767;
	Fri, 18 Mar 2005 00:03:26 GMT
	(envelope-from davidxu@freebsd.org)
Message-ID: <423A1AD4.2050207@freebsd.org>
Date: Fri, 18 Mar 2005 08:03:32 +0800
From: David Xu <davidxu@freebsd.org>
User-Agent: Mozilla/5.0 (X11; U; FreeBSD amd64; en-US;
	rv:1.7.2) Gecko/20041004
X-Accept-Language: en-us, en
MIME-Version: 1.0
To: Poul-Henning Kamp <phk@phk.freebsd.dk>
References: <5641.1111050847@critter.freebsd.dk>
In-Reply-To: <5641.1111050847@critter.freebsd.dk>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
cc: current@freebsd.org
Subject: Re: [TEST/REVIEW/PERF] kern_thread.c tid allocation patch
X-BeenThere: freebsd-current@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: Discussions about the use of FreeBSD-current
	<freebsd-current.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>,
	<mailto:freebsd-current-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-current>
List-Post: <mailto:freebsd-current@freebsd.org>
List-Help: <mailto:freebsd-current-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>,
	<mailto:freebsd-current-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 18 Mar 2005 00:03:27 -0000

Poul-Henning Kamp wrote:

>While investigating a Coverity report (false alarm btw) I noticed
>that kern_thread has a lot of code for allocating thread id's.
>
>This patch uses subr_unit instead.
>
>I would expect thread create/destroy performance to increase on
>systems where there are many threads, and would be interested
>if anybody can measure any difference.
>
>Poul-Henning
>
>  
>
I have tested it, yes, it is slight faster than current code.

David Xu