From owner-cvs-sys  Tue Dec 10 22:00:48 1996
Return-Path: <owner-cvs-sys>
Received: (from root@localhost)
          by freefall.freebsd.org (8.8.4/8.8.4) id WAA18437
          for cvs-sys-outgoing; Tue, 10 Dec 1996 22:00:48 -0800 (PST)
Received: from time.cdrom.com (time.cdrom.com [204.216.27.226])
          by freefall.freebsd.org (8.8.4/8.8.4) with ESMTP id WAA18424;
          Tue, 10 Dec 1996 22:00:42 -0800 (PST)
Received: from time.cdrom.com (localhost [127.0.0.1]) by time.cdrom.com (8.8.4/8.6.9) with ESMTP id WAA02423; Tue, 10 Dec 1996 22:00:36 -0800 (PST)
To: John Dyson <dyson@freefall.freebsd.org>
cc: CVS-committers@freefall.freebsd.org, cvs-all@freefall.freebsd.org,
        cvs-sys@freefall.freebsd.org
Subject: Re: cvs commit: src/sys/ufs/ufs ufs_readwrite.c 
In-reply-to: Your message of "Tue, 10 Dec 1996 21:17:26 PST."
             <199612110517.VAA14335@freefall.freebsd.org> 
Date: Tue, 10 Dec 1996 22:00:36 -0800
Message-ID: <2419.850284036@time.cdrom.com>
From: "Jordan K. Hubbard" <jkh@time.cdrom.com>
Sender: owner-cvs-sys@FreeBSD.ORG
X-Loop: FreeBSD.org
Precedence: bulk

>   makes MADV_SEQUENTIAL much more effective.  I suggest that
>   we start using MADV_SEQUENTIAL on system utilities that mmap
>   their input files, and the I/O is predominantely sequential.

I was wondering whether or not cp(1) would benefit from this, then I
noticed that its mmap() handling had been made conditional on
VM_AND_BUFFER_CACHE_SYNCHRONIZED, which is not defined.  I thought
they had been for a long time. :-)

Or is this #ifdef just poorly named?  Would

#ifdef MMAP_ACTUALLY_BOUGHT_US_PERFORMANCE
..
#endif

Be more appropriate?

					Jordan