From owner-cvs-all@FreeBSD.ORG  Thu Mar 27 03:16:02 2003
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: by hub.freebsd.org (Postfix, from userid 683)
	id E0A9B37B401; Thu, 27 Mar 2003 03:16:02 -0800 (PST)
Date: Thu, 27 Mar 2003 03:16:02 -0800
From: Eivind Eklund <eivind@FreeBSD.org>
To: Tor Egge <tegge@FreeBSD.org>
Message-ID: <20030327031602.C77894@FreeBSD.org>
References: <200303262340.h2QNegJg065498@repoman.freebsd.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.2.5.1i
In-Reply-To: <200303262340.h2QNegJg065498@repoman.freebsd.org>;
	from tegge@FreeBSD.org on Wed, Mar 26, 2003 at 03:40:42PM -0800
X-Spam-Status: No, hits=-32.5 required=5.0
	tests=EMAIL_ATTRIBUTION,IN_REP_TO,QUOTED_EMAIL_TEXT,REFERENCES,
	      REPLY_WITH_QUOTES,USER_AGENT_MUTT
	autolearn=ham	version=2.50
X-Spam-Level: 
X-Spam-Checker-Version: SpamAssassin 2.50 (1.173-2003-02-20-exp)
cc: cvs-src@FreeBSD.org
cc: src-committers@FreeBSD.org
cc: cvs-all@FreeBSD.org
Subject: Re: cvs commit: src/sys/conf NOTES files options src/sys/kern
	vfs_bio.c src/sys/ufs/ffs ffs_rawread.c ffs_vnops.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 27 Mar 2003 11:16:05 -0000

On Wed, Mar 26, 2003 at 03:40:42PM -0800, Tor Egge wrote:
> tegge       2003/03/26 15:40:42 PST
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/conf             NOTES files options 
>     sys/kern             vfs_bio.c 
>     sys/ufs/ffs          ffs_vnops.c 
>   Added files:
>     sys/ufs/ffs          ffs_rawread.c 
>   Log:
>   Add support for reading directly from file to userland buffer when the
>   O_DIRECT descriptor status flag is set and both offset and length is a
>   multiple of the physical media sector size.

Why only in the O_DIRECT case?  O_DIRECT is normally used to avoid caching
effects *in order to avoid side effects of caching*, at the cost of speed.
This patch seems to do the opposite - speed up the codepath significantly.

Not that it isn't nice - it would just be even nicer to have available some
way to just say "max speed, please".

Eivind.