misc/173301: [patch] bsdinstaller default to SU instead of SU+J

[Bas Smeelen]

>Number:         173301
>Category:       misc
>Synopsis:       [patch] bsdinstaller default to SU instead of SU+J
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Fri Nov 02 20:00:00 UTC 2012
>Closed-Date:
>Last-Modified:
>Originator:     Bas Smeelen
>Release:        9.1-RC3 , 9-STABLE and CURRENT
>Organization:
OverNite Software Europe (OSE)
>Environment:
FreeBSD sys.ose.nl 9.1-RC3 FreeBSD 9.1-RC3 #0: Wed Oct 31 11:56:55 CET 2012     root at sys.ose.nl:/usr/obj/usr/src/sys/SYS  amd64

>Description:
SU+J is default on new installs.
This is a problem when using snapshot functionality e.g. dump a live filesystem and also for SSD this is not recommended.
I would like to have the installer to default to just SU and experienced users can turn journaling on.
See also the discussion on the list: http://lists.freebsd.org/pipermail/freebsd-stable/2012-November/070360.html
>How-To-Repeat:
Install from 9.1-RC2 ISO (even 9.0 or later) and do a dump -0Lauf
snapshot is not supported with journaled soft updates
>Fix:
Workaround is to disable this afterwards, but a better solution is to not default to this, experienced users can enable this, since most installs will be on SSD nowadays and a lot of users will be using dump -L so this default would be not desired.


Patch attached with submission follows:

--- gpart_ops.c	2012-08-06 01:54:33.000000000 +0200
+++ gpart_ops.cnew	2012-11-02 19:07:45.000000000 +0100
@@ -90,8 +90,8 @@
 			{"SU", "Softupdates",
 			    "Enable softupdates (default)", 1 },
 			{"SUJ", "Softupdates journaling",
-			    "Enable file system journaling (default - "
-			    "turn off for SSDs)", 1 },
+			    "Disable file system journaling (default - "
+			    "turn on if you don't use snapshots or SSD)", 0 },
 			{"TRIM", "Enable SSD TRIM support",
 			    "Enable TRIM support, useful on solid-state drives",
 			    0 },


>Release-Note:
>Audit-Trail:
>Unformatted: