From owner-svn-src-all@FreeBSD.ORG Tue Nov 4 07:25:03 2014 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 651F55C0; Tue, 4 Nov 2014 07:25:03 +0000 (UTC) Received: from phk.freebsd.dk (phk.freebsd.dk [130.225.244.222]) by mx1.freebsd.org (Postfix) with ESMTP id A477ED81; Tue, 4 Nov 2014 07:25:02 +0000 (UTC) Received: from critter.freebsd.dk (unknown [192.168.60.3]) by phk.freebsd.dk (Postfix) with ESMTP id BCFE93BD2B; Tue, 4 Nov 2014 07:25:00 +0000 (UTC) Received: from critter.freebsd.dk (localhost [127.0.0.1]) by critter.freebsd.dk (8.14.9/8.14.9) with ESMTP id sA47Ox8I082149; Tue, 4 Nov 2014 07:24:59 GMT (envelope-from phk@phk.freebsd.dk) To: Mateusz Guzik Subject: Re: svn commit: r274017 - head/sys/kern In-reply-to: <20141104053520.GA4032@dft-labs.eu> From: "Poul-Henning Kamp" References: <201411030746.sA37kpPu037113@svn.freebsd.org> <54573AEE.9010602@freebsd.org> <54573B87.7000801@freebsd.org> <54573CD2.1000702@selasky.org> <20141103092132.GH29497@dft-labs.eu> <20141103100847.GK53947@kib.kiev.ua> <20141104045159.E1605@besplex.bde.org> <79454.1415043356@critter.freebsd.dk> <20141104053520.GA4032@dft-labs.eu> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <82147.1415085899.1@critter.freebsd.dk> Date: Tue, 04 Nov 2014 07:24:59 +0000 Message-ID: <82148.1415085899@critter.freebsd.dk> X-Mailman-Approved-At: Tue, 04 Nov 2014 12:39:36 +0000 Cc: Hans Petter Selasky , src-committers@freebsd.org, Mateusz Guzik , jmallett@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org, Bruce Evans , Konstantin Belousov , Julian Elischer X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 04 Nov 2014 07:25:03 -0000 -------- In message <20141104053520.GA4032@dft-labs.eu>, Mateusz Guzik writes: >On Mon, Nov 03, 2014 at 07:35:56PM +0000, Poul-Henning Kamp wrote: >> -------- >> In message <20141104045159.E1605@besplex.bde.org>, Bruce Evans writes: >> >> >This optimization is probably minor, but reminds me of other syscalls >> >that would benefit using a single largish allocation up front: >> >- all vfs calls that start with namei(). They allocate PATH_MAX (1K) >> > bytes and more. An 8K stack has plently to spare after allocating >> > 1K. However, if malloc() is used then the PATH_MAX limit shouldn't >> > exist. Only a limit to prevent denial of service is needed. >> >> We should actually roll a new rev of all syscalls which take a path >> and have them pass strlen(path)+1 into the kernel. >> >> That would allow both precise allocation and faster copyin, followed >> by a check that the last byte is (still) a NUL. >> > >I think we can speed up things on amd64 no problem without affecting >userspace. I think from a design point of view it makes much more sense to let userland do as much work as possible before we enter the kernel, rather than complicate the kernel with "oops we got a page fault" handling. -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 phk@FreeBSD.ORG | TCP/IP since RFC 956 FreeBSD committer | BSD since 4.3-tahoe Never attribute to malice what can adequately be explained by incompetence.