Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 04 Nov 2012 21:13:36 +0900 (JST)
From:      HATANO Tomomi <hatanou@infolab.ne.jp>
To:        freebsd-stable@freebsd.org
Cc:        jdc@koitsu.org, b.smeelen@ose.nl, fnwhitehorn@freebsd.org, freebsd-current@freebsd.org
Subject:   Re: SU+J on 9.1-RC2 ISO
Message-ID:  <20121104.211336.244701604.hatanou@infolab.ne.jp>
In-Reply-To: <5095B89F.4070705@freebsd.org>
References:  <20121103190930.GA23145@icarus.home.lan> <5095B89F.4070705@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
----Next_Part(Sun_Nov__4_21_13_36_2012_719)--
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit

Hi all.

The point is:

    There is completely no way to take a snapshot of SU+J partition
    unless modify one's kernel.

Whether some issue still exist or not,
how about enabling snapshoting SU+J partition
through sysctl variable?

Would you mind to see patch attached?

1. Taking a snapshot of SU+J partition is controlled through sysctl variable.

2. Default to disable.
   One who want to enable it should set the variable manually.

3. The default value in bsdinstall(8) may be left as is.
--
HATANO Tomomi.
----Next_Part(Sun_Nov__4_21_13_36_2012_719)--
Content-Type: Text/X-Patch; charset=us-ascii
Content-Transfer-Encoding: 7bit
Content-Disposition: inline; filename="snapsuj.patch"

--- src/sys/ufs/ffs/ffs_snapshot.c.orig	2012-11-04 11:01:58.000000000 +0900
+++ src/sys/ufs/ffs/ffs_snapshot.c	2012-11-04 11:13:32.000000000 +0900
@@ -182,8 +182,10 @@
  */
 int dopersistence = 0;
 
-#ifdef DEBUG
 #include <sys/sysctl.h>
+int snapsuj = 0;
+SYSCTL_INT(_debug, OID_AUTO, snapsuj, CTLFLAG_RW, &snapsuj, 0, "");
+#ifdef DEBUG
 SYSCTL_INT(_debug, OID_AUTO, dopersistence, CTLFLAG_RW, &dopersistence, 0, "");
 static int snapdebug = 0;
 SYSCTL_INT(_debug, OID_AUTO, snapdebug, CTLFLAG_RW, &snapdebug, 0, "");
@@ -230,7 +232,7 @@
 	 * At the moment, journaled soft updates cannot support
 	 * taking snapshots.
 	 */
-	if (MOUNTEDSUJ(mp)) {
+	if (MOUNTEDSUJ(mp) && (snapsuj == 0)) {
 		vfs_mount_error(mp, "%s: Snapshots are not yet supported when "
 		    "running with journaled soft updates", fs->fs_fsmnt);
 		return (EOPNOTSUPP);

----Next_Part(Sun_Nov__4_21_13_36_2012_719)----



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20121104.211336.244701604.hatanou>