svn commit: r250710 - head/sbin/dumpfs

[Kirk McKusick]

Author: mckusick
Date: Thu May 16 20:07:08 2013
New Revision: 250710
URL: http://svnweb.freebsd.org/changeset/base/250710

Log:
  When running the -m option to generate a newfs(8) command suitable for
  recreating the filesystem, check for and output the -i, -k, and -l
  options if appropriate.
  
  Note the remaining deficiencies of the -m option in the dumpfs(8)
  manual page. Specifically that newfs(8) options -E, -R, -S, and -T
  options are not handled and that -p is not useful so is omitted.
  Also document that newfs(8) options -n and -r are neither checked
  for nor output but should be. The -r flag is needed if the filesystem
  uses gjournal(8).
  
  PR:           bin/163992
  Reported by:  Dieter <freebsd at sopwith.solgatos.com>
  Submitted by: Andy Kosela <akosela at andykosela.com>
  MFC after:    1 week

Modified:
  head/sbin/dumpfs/dumpfs.8
  head/sbin/dumpfs/dumpfs.c

Modified: head/sbin/dumpfs/dumpfs.8
==============================================================================
--- head/sbin/dumpfs/dumpfs.8	Thu May 16 19:44:51 2013	(r250709)
+++ head/sbin/dumpfs/dumpfs.8	Thu May 16 20:07:08 2013	(r250710)
@@ -28,7 +28,7 @@
 .\"     @(#)dumpfs.8	8.1 (Berkeley) 6/5/93
 .\" $FreeBSD$
 .\"
-.Dd Jul 14, 2011
+.Dd May 16, 2013
 .Dt DUMPFS 8
 .Os
 .Sh NAME
@@ -76,6 +76,27 @@ is specified, a
 .Xr newfs 8
 command is printed that can be used to generate a new file system
 with equivalent settings.
+Please note that
+.Xr newfs 8
+options
+.Fl E ,
+.Fl R ,
+.Fl S ,
+and
+.Fl T
+are not handled and
+.Fl p
+is not useful in this case so is omitted.
+.Xr Newfs 8
+options
+.Fl n
+and
+.Fl r
+are neither checked for nor output but should be.
+The
+.Fl r
+flag is needed if the filesystem uses
+.Xr gjournal 8 .
 .Sh SEE ALSO
 .Xr disktab 5 ,
 .Xr fs 5 ,

Modified: head/sbin/dumpfs/dumpfs.c
==============================================================================
--- head/sbin/dumpfs/dumpfs.c	Thu May 16 19:44:51 2013	(r250709)
+++ head/sbin/dumpfs/dumpfs.c	Thu May 16 20:07:08 2013	(r250710)
@@ -417,12 +417,15 @@ marshal(const char *name)
 	printf("-f %d ", fs->fs_fsize);
 	printf("-g %d ", fs->fs_avgfilesize);
 	printf("-h %d ", fs->fs_avgfpdir);
-	/* -i is dumb */
+	printf("-i %jd ", fragroundup(fs, lblktosize(fs, fragstoblks(fs,
+	    fs->fs_fpg)) / fs->fs_ipg));
 	if (fs->fs_flags & FS_SUJ)
 		printf("-j ");
 	if (fs->fs_flags & FS_GJOURNAL)
 		printf("-J ");
-	/* -k..l unimplemented */
+	printf("-k %jd ", fs->fs_metaspace);
+	if (fs->fs_flags & FS_MULTILABEL)
+		printf("-l ");
 	printf("-m %d ", fs->fs_minfree);
 	/* -n unimplemented */
 	printf("-o ");