svn commit: r320892 - head/etc/defaults

[Ravi Pokala]

I appreciate the spirit of this change; thanks Trasz!

A question though: you're telling the generic `fsck' to pass "-R" to either `fsck_ffs' or `fsck_ufs', as needed. But those are both names for the same executable. Won't the generic `fsck' always end up invoking (per sbin/fsck/fsck.c::ptype_map[]) `fsck_ffs'? In which case, is the `fsck_ufs' case needed here?

Thanks,

Ravi (rpokala@)

-----Original Message-----
From: <owner-src-committers at freebsd.org> on behalf of Edward Tomasz Napierala <trasz at FreeBSD.org>
Date: 2017-07-11, Tuesday at 05:32
To: <src-committers at freebsd.org>, <svn-src-all at freebsd.org>, <svn-src-head at freebsd.org>
Subject: svn commit: r320892 - head/etc/defaults

Author: trasz
Date: Tue Jul 11 12:32:40 2017
New Revision: 320892
URL: https://svnweb.freebsd.org/changeset/base/320892

Log:
  Make fsck_y_enable default to passing pass -R to fsck_ffs(8) in addition
  to -y.  To me, fsck_y_enable means "try as hard as possible", and without
  -R, it... well, doesn't.
  
  Reviewed by:	mckusick
  Obtained from:	CheriBSD
  MFC after:	2 weeks
  Sponsored by:	DARPA, AFRL
  Differential Revision:	https://reviews.freebsd.org/D11490

Modified:
  head/etc/defaults/rc.conf

Modified: head/etc/defaults/rc.conf
==============================================================================
--- head/etc/defaults/rc.conf	Tue Jul 11 06:39:12 2017	(r320891)
+++ head/etc/defaults/rc.conf	Tue Jul 11 12:32:40 2017	(r320892)
@@ -92,7 +92,7 @@ geli_autodetach="YES"	# Automatically detach on last c
 root_rw_mount="YES"	# Set to NO to inhibit remounting root read-write.
 root_hold_delay="30"	# Time to wait for root mount hold release.
 fsck_y_enable="NO"	# Set to YES to do fsck -y if the initial preen fails.
-fsck_y_flags=""		# Additional flags for fsck -y
+fsck_y_flags="-T ffs:-R -T ufs:-R"	# Additional flags for fsck -y
 background_fsck="YES"	# Attempt to run fsck in the background where possible.
 background_fsck_delay="60" # Time to wait (seconds) before starting the fsck.
 netfs_types="nfs:NFS smbfs:SMB" # Net filesystems.