From owner-cvs-all@FreeBSD.ORG  Mon Mar  7 08:42:51 2005
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id 1FBF616A4CE; Mon,  7 Mar 2005 08:42:51 +0000 (GMT)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 3A4D443D5E; Mon,  7 Mar 2005 08:42:50 +0000 (GMT)
	(envelope-from delphij@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.13.1/8.13.1) with ESMTP id j278gn4a062004;
	Mon, 7 Mar 2005 08:42:49 GMT
	(envelope-from delphij@repoman.freebsd.org)
Received: (from delphij@localhost)
	by repoman.freebsd.org (8.13.1/8.13.1/Submit) id j278gns3062003;
	Mon, 7 Mar 2005 08:42:49 GMT
	(envelope-from delphij)
Message-Id: <200503070842.j278gns3062003@repoman.freebsd.org>
From: Xin LI <delphij@FreeBSD.org>
Date: Mon, 7 Mar 2005 08:42:49 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org,
	cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Subject: cvs commit: src/sbin/fsck_ffs fsck.h pass5.c setup.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: Mon, 07 Mar 2005 08:42:51 -0000

delphij     2005-03-07 08:42:49 UTC

  FreeBSD src repository

  Modified files:
    sbin/fsck_ffs        fsck.h pass5.c setup.c 
  Log:
  Make background fsck based summary adjustments actually work by
  initializing the sysctl mibs data before actually using them.
  
  The original patchset (which is the actual version that is running
  on my testboxes) have checked whether all of these sysctls and
  refuses to do background fsck if we don't have them.  Kirk has
  pointed out that refusing running fsck on old kernels is pointless,
  as old kernels will recompute the summary at mount time, so I
  have removed these checks.
  
  Unfortunatelly, as the checks will initialize the mib values of
  those sysctl's, and which are vital for the runtime summary
  adjustment to work, we can not simply remove the check, which
  will lead to problem when running background fsck over a dirty
  volume.  Add these checks in a different way: give a warning rather
  than refusing to work, and complain if the functionality is not
  available when adjustments are necessary.
  
  Noticed by:     A power failure at my lab
  Pointy hat:     me
  MFC After:      3 days
  
  Revision  Changes    Path
  1.36      +1 -0      src/sbin/fsck_ffs/fsck.h
  1.42      +5 -5      src/sbin/fsck_ffs/pass5.c
  1.49      +18 -0     src/sbin/fsck_ffs/setup.c