From owner-cvs-all@FreeBSD.ORG  Thu Oct 18 07:26:21 2007
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id B69FB16A41A;
	Thu, 18 Oct 2007 07:26:21 +0000 (UTC) (envelope-from bde@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org
	[IPv6:2001:4f8:fff6::29])
	by mx1.freebsd.org (Postfix) with ESMTP id A05E913C4BB;
	Thu, 18 Oct 2007 07:26:21 +0000 (UTC) (envelope-from bde@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.14.1/8.14.1) with ESMTP id l9I7QLoG017376;
	Thu, 18 Oct 2007 07:26:21 GMT (envelope-from bde@repoman.freebsd.org)
Received: (from bde@localhost)
	by repoman.freebsd.org (8.14.1/8.14.1/Submit) id l9I7QLo7017375;
	Thu, 18 Oct 2007 07:26:21 GMT (envelope-from bde)
Message-Id: <200710180726.l9I7QLo7017375@repoman.freebsd.org>
From: Bruce Evans <bde@FreeBSD.org>
Date: Thu, 18 Oct 2007 07:26:21 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Cc: 
Subject: cvs commit: src/sys/fs/msdosfs msdosfs_vnops.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
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, 18 Oct 2007 07:26:21 -0000

bde         2007-10-18 07:26:21 UTC

  FreeBSD src repository

  Modified files:
    sys/fs/msdosfs       msdosfs_vnops.c 
  Log:
  In msdosfs_settattr(), don't do synchronous updates of the denode
  (except indirectly for the size pseudo-attribute).  If anything deserves
  a sync update, then it is ids and immutable flags, since these are
  related to security, but ffs never synced these and msdosfs doesn't
  support them.  (ufs_setattr() only does an update in one case where
  it is least needed (for timestamps); it did pessimal sync updates for
  timestamps until 1998/03/08 but was changed for unlogged reasons related
  to soft updates.)
  
  Now msdosfs calls deupdat() with waitfor == 0, which normally gives a
  delayed update to disk but always gives a sync update of timestamps
  in core, while for ffs everything is delayed until the syncer daemon
  or other activity causes an update (except for timestamps).
  
  This gives a large optimization mainly for things like cp -p, where
  attribute adjustment could easily triple the number of physical I/O's
  if it is done synchronously (but cp -p to msdosfs is not as bad as
  that, since msdosfs doesn't support many attributes so null adjustments
  are more common, and msdosfs doesn't support ctimes so even if cp
  doesn't weed out null adjustments they don't become non-null after
  clobbering the ctime).
  
  Revision  Changes    Path
  1.180     +1 -1      src/sys/fs/msdosfs/msdosfs_vnops.c