linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] btrfs-progs: add mount options to btrfs-mount.5
@ 2014-06-11 22:13 Eric Sandeen
  2014-06-11 22:14 ` [PATCH V2] " Eric Sandeen
  0 siblings, 1 reply; 10+ messages in thread
From: Eric Sandeen @ 2014-06-11 22:13 UTC (permalink / raw)
  To: linux-btrfs, Karel Zak

This is a straight cut and paste from the util-linux
mount manpage into btrfs-mount.5

It's pretty much impossible for util-linux to keep up
with every filesystem out there, and Karel has more than
once expressed a wish that mount options move into fs-specific
manpages.

So, here we go.

The way btrfs asciidoc is generated, there's not a trivial
way to have both btrfs(5) and btrfs(8) so I named it btrfs-mount(5)
for now.  A bit ick and I'm open to suggestions.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
---

diff --git a/Documentation/Makefile b/Documentation/Makefile
index 03a5cd5..95ecbf6 100644
--- a/Documentation/Makefile
+++ b/Documentation/Makefile
@@ -31,13 +31,21 @@ MAN8_TXT += btrfs-replace.txt
 MAN8_TXT += btrfs-restore.txt
 MAN8_TXT += btrfs-property.txt
 
-MAN_TXT = $(MAN8_TXT)
+# Mount manpage
+MAN5_TXT += btrfs-mount.txt
+
+MAN_TXT = $(MAN8_TXT) $(MAN5_TXT)
 MAN_XML = $(patsubst %.txt,%.xml,$(MAN_TXT))
+
+DOC_MAN5 = $(patsubst %.txt,%.5,$(MAN5_TXT))
+GZ_MAN5 = $(patsubst %.txt,%.5.gz,$(MAN5_TXT))
+
 DOC_MAN8 = $(patsubst %.txt,%.8,$(MAN8_TXT))
 GZ_MAN8 = $(patsubst %.txt,%.8.gz,$(MAN8_TXT))
 
 mandir ?= $(prefix)/share/man
 man8dir = $(mandir)/man8
+man5dir = $(mandir)/man5
 
 ASCIIDOC = asciidoc
 ASCIIDOC_EXTRA =
@@ -67,25 +75,35 @@ endif
 endif
 
 all: man
-man: man8
+man: man5 man8
+man5: $(GZ_MAN5)
 man8: $(GZ_MAN8)
 
 install: install-man
 
 install-man: man
 	$(INSTALL) -d -m 755 $(DESTDIR)$(man8dir)
+	$(INSTALL) -m 644 $(GZ_MAN5) $(DESTDIR)$(man5dir)
 	$(INSTALL) -m 644 $(GZ_MAN8) $(DESTDIR)$(man8dir)
 	$(LNS) btrfs-check.8.gz $(DESTDIR)$(man8dir)
 
 clean:
-	$(QUIET_RM)$(RM) *.xml *.xml+ *.8 *.8.gz
+	$(QUIET_RM)$(RM) *.xml *.xml+ *.5 *.5.gz *.8 *.8.gz
+
+%.5.gz : %.5
+	$(QUIET_GZIP)$(GZIP) -n -c $< > $@
 
 %.8.gz : %.8
 	$(QUIET_GZIP)$(GZIP) -n -c $< > $@
 
+%.5 : %.xml 
+	$(QUIET_XMLTO)$(RM) $@ && \
+	$(XMLTO) -m $(MANPAGE_XSL) $(XMLTO_EXTRA) man $<
+
 %.8 : %.xml 
 	$(QUIET_XMLTO)$(RM) $@ && \
 	$(XMLTO) -m $(MANPAGE_XSL) $(XMLTO_EXTRA) man $<
+
 %.xml : %.txt asciidoc.conf
 	$(QUIET_ASCIIDOC)$(RM) $@.tmp[12] $@ && \
 	sed -e "s/\(<[^>]\+>\)/'\1'/g" < $< > $@.tmp1 && \
diff --git a/Documentation/btrfs-mount.txt b/Documentation/btrfs-mount.txt
new file mode 100644
index 0000000..4433a78
--- /dev/null
+++ b/Documentation/btrfs-mount.txt
@@ -0,0 +1,186 @@
+btrfs-mount(5)
+==============
+
+NAME
+----
+btrfs-mount - mount options for the btrfs filesystem
+
+DESCRIPTION
+-----------
+This document describes mount options specific to the btrfs filesystem.
+Other generic mount options are available,and are described in the
+`mount`(8) manpage.
+
+MOUNT OPTIONS
+-------------
+*alloc_start='bytes'*::
+	Debugging option to force all block allocations above a certain
+	byte threshold on each block device.  The value is specified in
+	bytes, optionally with a K, M, or G suffix, case insensitive.
+	Default is 1MB.
+
+*autodefrag*::
+	Disable/enable auto defragmentation.
+	Auto defragmentation detects small random writes into files and queue
+	them up for the defrag process.  Works best for small files;
+	Not well suited for large database workloads.
+
+*check_int*|*check_int_data*|*check_int_print_mask='value'*::
+	These debugging options control the behavior of the integrity checking
+	module (the BTRFS_FS_CHECK_INTEGRITY config option required). +
+	+
+	`check_int` enables the integrity checker module, which examines all
+	block write requests to ensure on-disk consistency, at a large
+	memory and CPU cost. +
+	+
+	`check_int_data` includes extent data in the integrity checks, and
+	implies the check_int option. +
+	+
+	`check_int_print_mask` takes a bitmask of BTRFSIC_PRINT_MASK_* values
+	as defined in 'fs/btrfs/check-integrity.c', to control the integrity
+	checker module behavior. +
+	+
+	See comments at the top of 'fs/btrfs/check-integrity.c'
+	for more info.
+
+*commit='seconds'*::
+	Set the interval of periodic commit, 30 seconds by default. Higher
+	values defer data being synced to permanent storage with obvious
+	consequences when the system crashes. The upper bound is not forced,
+	but a warning is printed if it's more than 300 seconds (5 minutes).
+
+*compress*|*compress='type'*|*compress-force*|*compress-force='type'*::
+	Control BTRFS file data compression.  Type may be specified as "zlib"
+	"lzo" or "no" (for no compression, used for remounting).  If no type
+	is specified, zlib is used.  If compress-force is specified,
+	all files will be compressed, whether or not they compress well.
+	If compression is enabled, nodatacow and nodatasum are disabled.
+
+*degraded*::
+	Allow mounts to continue with missing devices.  A read-write mount may
+	fail with too many devices missing, for example if a stripe member
+	is completely missing.
+
+*device='devicepath'*::
+	Specify a device during mount so that ioctls on the control device
+	can be avoided.  Especially useful when trying to mount a multi-device
+	setup as root.  May be specified multiple times for multiple devices.
+
+*discard*::
+	Disable/enable discard mount option.
+	Discard issues frequent commands to let the block device reclaim space
+	freed by the filesystem.
+	This is useful for SSD devices, thinly provisioned
+	LUNs and virtual machine images, but may have a significant
+	performance impact.  (The fstrim command is also available to
+	initiate batch trims from userspace).
+
+*enospc_debug*::
+	Disable/enable debugging option to be more verbose in some ENOSPC conditions.
+
+*fatal_errors='action'*::
+	Action to take when encountering a fatal error. +
+		"bug" - BUG() on a fatal error.  This is the default. +
+		"panic" - panic() on a fatal error.
+
+*flushoncommit*::
+	The `flushoncommit` mount option forces any data dirtied by a write in a
+	prior transaction to commit as part of the current commit.  This makes
+	the committed state a fully consistent view of the file system from the
+	application's perspective (i.e., it includes all completed file system
+	operations).  This was previously the behavior only when a snapshot is
+	created.
+
+*inode_cache*:
+	Enable free inode number caching.   Defaults to off due to an overflow
+	problem when the free space crcs don't fit inside a single page.
+
+*max_inline='bytes'*::
+	Specify the maximum amount of space, in bytes, that can be inlined in
+	a metadata B-tree leaf.  The value is specified in bytes, optionally
+	with a K, M, or G suffix, case insensitive.  In practice, this value
+	is limited by the root sector size, with some space unavailable due
+	to leaf headers.  For a 4k sectorsize, max inline data is ~3900 bytes.
+
+*metadata_ratio='value'*::
+	Specify that 1 metadata chunk should be allocated after every
+	'value' data chunks.  Off by default.
+
+*noacl*::
+	Enable/disable support for Posix Access Control Lists (ACLs).  See the
+	`acl`(5) manual page for more information about ACLs.
+
+*nobarrier*::
+	ensure that certain IOs make it through the device cache and are on
+	persistent storage. If disabled on a device with a volatile
+	(non-battery-backed) write-back cache, nobarrier option will lead to
+	filesystem corruption on a system crash or power loss.
+
+*nodatacow*::
+	Enable/disable data copy-on-write for newly created files.
+	Nodatacow implies nodatasum, and disables all compression.
+
+*nodatasum*::
+	Enable/disable data checksumming for newly created files.
+	Datasum implies datacow.
+
+*notreelog*::
+	Enable/disable the tree logging used for fsync and O_SYNC writes.
+
+*recovery*::
+	Enable autorecovery attempts if a bad tree root is found at mount time.
+	Currently this scans a list of several previous tree roots and tries to
+	use the first readable.
+
+*rescan_uuid_tree*::
+	Force check and rebuild procedure of the UUID tree. This should not
+	normally be needed.
+
+*skip_balance*::
+	Skip automatic resume of interrupted balance operation after mount.
+	May be resumed with "btrfs balance resume."
+
+*nospace_cache*::
+	Disable freespace cache loading without clearing the cache.
+
+*clear_cache*::
+	Force clearing and rebuilding of the disk space cache if something
+	has gone wrong.
+
+*ssd*|*nossd*|*ssd_spread*::
+	Options to control ssd allocation schemes.  By default, BTRFS will
+	enable or disable ssd allocation heuristics depending on whether a
+	rotational or nonrotational disk is in use.  The ssd and nossd options
+	can override this autodetection. +
+	The ssd_spread mount option attempts to allocate into big chunks
+	of unused space, and may perform better on low-end ssds.  ssd_spread
+	implies ssd, enabling all other ssd heuristics as well.
+
+*subvol='path'*::
+	Mount subvolume at 'path' rather than the root subvolume. The
+	'path' is relative to the top level subvolume.
+
+*subvolid='ID'*::
+	Mount subvolume specified by an ID number rather than the root subvolume.
+	This allows mounting of subvolumes which are not in the root of the mounted
+	filesystem.
+	You can use "btrfs subvolume list" to see subvolume ID numbers.
+
+*subvolrootid='objectid' (deprecated)*::
+	Mount subvolume specified by 'objectid' rather than the root subvolume.
+	This allows mounting of subvolumes which are not in the root of the mounted
+	filesystem.
+	You can use "btrfs subvolume show" to see the object ID for a subvolume.
+
+*thread_pool='number'*::
+	The number of worker threads to allocate.  The default number is equal
+	to the number of CPUs + 2, or 8, whichever is smaller.
+
+*user_subvol_rm_allowed*::
+	Allow subvolumes to be deleted by a non-root user. Use with caution.
+
+SEE ALSO
+--------
+`mkfs.btrfs`(8),
+`mount`(8),
+`btrfs`(8)


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH V2] btrfs-progs: add mount options to btrfs-mount.5
  2014-06-11 22:13 [PATCH] btrfs-progs: add mount options to btrfs-mount.5 Eric Sandeen
@ 2014-06-11 22:14 ` Eric Sandeen
  2014-06-12 10:51   ` David Sterba
  0 siblings, 1 reply; 10+ messages in thread
From: Eric Sandeen @ 2014-06-11 22:14 UTC (permalink / raw)
  To: linux-btrfs, Karel Zak

This is a straight cut and paste from the util-linux
mount manpage into btrfs-mount.5

It's pretty much impossible for util-linux to keep up
with every filesystem out there, and Karel has more than
once expressed a wish that mount options move into fs-specific
manpages.

So, here we go.

The way btrfs asciidoc is generated, there's not a trivial
way to have both btrfs(5) and btrfs(8) so I named it btrfs-mount(5)
for now.  A bit ick and I'm open to suggestions.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
---

V2: whoops, have to $(INSTALL) -d -m 755 $(DESTDIR)$(man5dir) too...

diff --git a/Documentation/Makefile b/Documentation/Makefile
index 03a5cd5..be95fda 100644
--- a/Documentation/Makefile
+++ b/Documentation/Makefile
@@ -31,13 +31,21 @@ MAN8_TXT += btrfs-replace.txt
 MAN8_TXT += btrfs-restore.txt
 MAN8_TXT += btrfs-property.txt
 
-MAN_TXT = $(MAN8_TXT)
+# Mount manpage
+MAN5_TXT += btrfs-mount.txt
+
+MAN_TXT = $(MAN8_TXT) $(MAN5_TXT)
 MAN_XML = $(patsubst %.txt,%.xml,$(MAN_TXT))
+
+DOC_MAN5 = $(patsubst %.txt,%.5,$(MAN5_TXT))
+GZ_MAN5 = $(patsubst %.txt,%.5.gz,$(MAN5_TXT))
+
 DOC_MAN8 = $(patsubst %.txt,%.8,$(MAN8_TXT))
 GZ_MAN8 = $(patsubst %.txt,%.8.gz,$(MAN8_TXT))
 
 mandir ?= $(prefix)/share/man
 man8dir = $(mandir)/man8
+man5dir = $(mandir)/man5
 
 ASCIIDOC = asciidoc
 ASCIIDOC_EXTRA =
@@ -67,25 +75,36 @@ endif
 endif
 
 all: man
-man: man8
+man: man5 man8
+man5: $(GZ_MAN5)
 man8: $(GZ_MAN8)
 
 install: install-man
 
 install-man: man
+	$(INSTALL) -d -m 755 $(DESTDIR)$(man5dir)
 	$(INSTALL) -d -m 755 $(DESTDIR)$(man8dir)
+	$(INSTALL) -m 644 $(GZ_MAN5) $(DESTDIR)$(man5dir)
 	$(INSTALL) -m 644 $(GZ_MAN8) $(DESTDIR)$(man8dir)
 	$(LNS) btrfs-check.8.gz $(DESTDIR)$(man8dir)
 
 clean:
-	$(QUIET_RM)$(RM) *.xml *.xml+ *.8 *.8.gz
+	$(QUIET_RM)$(RM) *.xml *.xml+ *.5 *.5.gz *.8 *.8.gz
+
+%.5.gz : %.5
+	$(QUIET_GZIP)$(GZIP) -n -c $< > $@
 
 %.8.gz : %.8
 	$(QUIET_GZIP)$(GZIP) -n -c $< > $@
 
+%.5 : %.xml 
+	$(QUIET_XMLTO)$(RM) $@ && \
+	$(XMLTO) -m $(MANPAGE_XSL) $(XMLTO_EXTRA) man $<
+
 %.8 : %.xml 
 	$(QUIET_XMLTO)$(RM) $@ && \
 	$(XMLTO) -m $(MANPAGE_XSL) $(XMLTO_EXTRA) man $<
+
 %.xml : %.txt asciidoc.conf
 	$(QUIET_ASCIIDOC)$(RM) $@.tmp[12] $@ && \
 	sed -e "s/\(<[^>]\+>\)/'\1'/g" < $< > $@.tmp1 && \
diff --git a/Documentation/btrfs-mount.txt b/Documentation/btrfs-mount.txt
new file mode 100644
index 0000000..4433a78
--- /dev/null
+++ b/Documentation/btrfs-mount.txt
@@ -0,0 +1,186 @@
+btrfs-mount(5)
+==============
+
+NAME
+----
+btrfs-mount - mount options for the btrfs filesystem
+
+DESCRIPTION
+-----------
+This document describes mount options specific to the btrfs filesystem.
+Other generic mount options are available,and are described in the
+`mount`(8) manpage.
+
+MOUNT OPTIONS
+-------------
+*alloc_start='bytes'*::
+	Debugging option to force all block allocations above a certain
+	byte threshold on each block device.  The value is specified in
+	bytes, optionally with a K, M, or G suffix, case insensitive.
+	Default is 1MB.
+
+*autodefrag*::
+	Disable/enable auto defragmentation.
+	Auto defragmentation detects small random writes into files and queue
+	them up for the defrag process.  Works best for small files;
+	Not well suited for large database workloads.
+
+*check_int*|*check_int_data*|*check_int_print_mask='value'*::
+	These debugging options control the behavior of the integrity checking
+	module (the BTRFS_FS_CHECK_INTEGRITY config option required). +
+	+
+	`check_int` enables the integrity checker module, which examines all
+	block write requests to ensure on-disk consistency, at a large
+	memory and CPU cost. +
+	+
+	`check_int_data` includes extent data in the integrity checks, and
+	implies the check_int option. +
+	+
+	`check_int_print_mask` takes a bitmask of BTRFSIC_PRINT_MASK_* values
+	as defined in 'fs/btrfs/check-integrity.c', to control the integrity
+	checker module behavior. +
+	+
+	See comments at the top of 'fs/btrfs/check-integrity.c'
+	for more info.
+
+*commit='seconds'*::
+	Set the interval of periodic commit, 30 seconds by default. Higher
+	values defer data being synced to permanent storage with obvious
+	consequences when the system crashes. The upper bound is not forced,
+	but a warning is printed if it's more than 300 seconds (5 minutes).
+
+*compress*|*compress='type'*|*compress-force*|*compress-force='type'*::
+	Control BTRFS file data compression.  Type may be specified as "zlib"
+	"lzo" or "no" (for no compression, used for remounting).  If no type
+	is specified, zlib is used.  If compress-force is specified,
+	all files will be compressed, whether or not they compress well.
+	If compression is enabled, nodatacow and nodatasum are disabled.
+
+*degraded*::
+	Allow mounts to continue with missing devices.  A read-write mount may
+	fail with too many devices missing, for example if a stripe member
+	is completely missing.
+
+*device='devicepath'*::
+	Specify a device during mount so that ioctls on the control device
+	can be avoided.  Especially useful when trying to mount a multi-device
+	setup as root.  May be specified multiple times for multiple devices.
+
+*discard*::
+	Disable/enable discard mount option.
+	Discard issues frequent commands to let the block device reclaim space
+	freed by the filesystem.
+	This is useful for SSD devices, thinly provisioned
+	LUNs and virtual machine images, but may have a significant
+	performance impact.  (The fstrim command is also available to
+	initiate batch trims from userspace).
+
+*enospc_debug*::
+	Disable/enable debugging option to be more verbose in some ENOSPC conditions.
+
+*fatal_errors='action'*::
+	Action to take when encountering a fatal error. +
+		"bug" - BUG() on a fatal error.  This is the default. +
+		"panic" - panic() on a fatal error.
+
+*flushoncommit*::
+	The `flushoncommit` mount option forces any data dirtied by a write in a
+	prior transaction to commit as part of the current commit.  This makes
+	the committed state a fully consistent view of the file system from the
+	application's perspective (i.e., it includes all completed file system
+	operations).  This was previously the behavior only when a snapshot is
+	created.
+
+*inode_cache*:
+	Enable free inode number caching.   Defaults to off due to an overflow
+	problem when the free space crcs don't fit inside a single page.
+
+*max_inline='bytes'*::
+	Specify the maximum amount of space, in bytes, that can be inlined in
+	a metadata B-tree leaf.  The value is specified in bytes, optionally
+	with a K, M, or G suffix, case insensitive.  In practice, this value
+	is limited by the root sector size, with some space unavailable due
+	to leaf headers.  For a 4k sectorsize, max inline data is ~3900 bytes.
+
+*metadata_ratio='value'*::
+	Specify that 1 metadata chunk should be allocated after every
+	'value' data chunks.  Off by default.
+
+*noacl*::
+	Enable/disable support for Posix Access Control Lists (ACLs).  See the
+	`acl`(5) manual page for more information about ACLs.
+
+*nobarrier*::
+	ensure that certain IOs make it through the device cache and are on
+	persistent storage. If disabled on a device with a volatile
+	(non-battery-backed) write-back cache, nobarrier option will lead to
+	filesystem corruption on a system crash or power loss.
+
+*nodatacow*::
+	Enable/disable data copy-on-write for newly created files.
+	Nodatacow implies nodatasum, and disables all compression.
+
+*nodatasum*::
+	Enable/disable data checksumming for newly created files.
+	Datasum implies datacow.
+
+*notreelog*::
+	Enable/disable the tree logging used for fsync and O_SYNC writes.
+
+*recovery*::
+	Enable autorecovery attempts if a bad tree root is found at mount time.
+	Currently this scans a list of several previous tree roots and tries to
+	use the first readable.
+
+*rescan_uuid_tree*::
+	Force check and rebuild procedure of the UUID tree. This should not
+	normally be needed.
+
+*skip_balance*::
+	Skip automatic resume of interrupted balance operation after mount.
+	May be resumed with "btrfs balance resume."
+
+*nospace_cache*::
+	Disable freespace cache loading without clearing the cache.
+
+*clear_cache*::
+	Force clearing and rebuilding of the disk space cache if something
+	has gone wrong.
+
+*ssd*|*nossd*|*ssd_spread*::
+	Options to control ssd allocation schemes.  By default, BTRFS will
+	enable or disable ssd allocation heuristics depending on whether a
+	rotational or nonrotational disk is in use.  The ssd and nossd options
+	can override this autodetection. +
+	The ssd_spread mount option attempts to allocate into big chunks
+	of unused space, and may perform better on low-end ssds.  ssd_spread
+	implies ssd, enabling all other ssd heuristics as well.
+
+*subvol='path'*::
+	Mount subvolume at 'path' rather than the root subvolume. The
+	'path' is relative to the top level subvolume.
+
+*subvolid='ID'*::
+	Mount subvolume specified by an ID number rather than the root subvolume.
+	This allows mounting of subvolumes which are not in the root of the mounted
+	filesystem.
+	You can use "btrfs subvolume list" to see subvolume ID numbers.
+
+*subvolrootid='objectid' (deprecated)*::
+	Mount subvolume specified by 'objectid' rather than the root subvolume.
+	This allows mounting of subvolumes which are not in the root of the mounted
+	filesystem.
+	You can use "btrfs subvolume show" to see the object ID for a subvolume.
+
+*thread_pool='number'*::
+	The number of worker threads to allocate.  The default number is equal
+	to the number of CPUs + 2, or 8, whichever is smaller.
+
+*user_subvol_rm_allowed*::
+	Allow subvolumes to be deleted by a non-root user. Use with caution.
+
+SEE ALSO
+--------
+`mkfs.btrfs`(8),
+`mount`(8),
+`btrfs`(8)


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH V2] btrfs-progs: add mount options to btrfs-mount.5
  2014-06-11 22:14 ` [PATCH V2] " Eric Sandeen
@ 2014-06-12 10:51   ` David Sterba
  2014-06-12 14:39     ` Eric Sandeen
  0 siblings, 1 reply; 10+ messages in thread
From: David Sterba @ 2014-06-12 10:51 UTC (permalink / raw)
  To: Eric Sandeen; +Cc: linux-btrfs, Karel Zak

On Wed, Jun 11, 2014 at 05:14:55PM -0500, Eric Sandeen wrote:
> This is a straight cut and paste from the util-linux
> mount manpage into btrfs-mount.5
> 
> It's pretty much impossible for util-linux to keep up
> with every filesystem out there, and Karel has more than
> once expressed a wish that mount options move into fs-specific
> manpages.
> 
> So, here we go.
> 
> The way btrfs asciidoc is generated, there's not a trivial
> way to have both btrfs(5) and btrfs(8) so I named it btrfs-mount(5)
> for now.  A bit ick and I'm open to suggestions.

So what if the mount options are generated from btrfs-mount.txt but
installed under btrfs.5.gz name? If there are more section 5 manpages we
can make it more generic but for now hardcoding btrfs-mount.* ->
btrfs.5. sounds ok to me.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH V2] btrfs-progs: add mount options to btrfs-mount.5
  2014-06-12 10:51   ` David Sterba
@ 2014-06-12 14:39     ` Eric Sandeen
  2014-06-18 15:29       ` David Sterba
  0 siblings, 1 reply; 10+ messages in thread
From: Eric Sandeen @ 2014-06-12 14:39 UTC (permalink / raw)
  To: dsterba, linux-btrfs, Karel Zak

On 6/12/14, 5:51 AM, David Sterba wrote:
> On Wed, Jun 11, 2014 at 05:14:55PM -0500, Eric Sandeen wrote:
>> This is a straight cut and paste from the util-linux
>> mount manpage into btrfs-mount.5
>>
>> It's pretty much impossible for util-linux to keep up
>> with every filesystem out there, and Karel has more than
>> once expressed a wish that mount options move into fs-specific
>> manpages.
>>
>> So, here we go.
>>
>> The way btrfs asciidoc is generated, there's not a trivial
>> way to have both btrfs(5) and btrfs(8) so I named it btrfs-mount(5)
>> for now.  A bit ick and I'm open to suggestions.
> 
> So what if the mount options are generated from btrfs-mount.txt but
> installed under btrfs.5.gz name? If there are more section 5 manpages we
> can make it more generic but for now hardcoding btrfs-mount.* ->
> btrfs.5. sounds ok to me.

Yeah, that seemed like kind of nasty hard-coding, but I suppose it works
for now.  I wanted to make it more generic, I didn't have a better idea..

-Eric


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH V2] btrfs-progs: add mount options to btrfs-mount.5
  2014-06-12 14:39     ` Eric Sandeen
@ 2014-06-18 15:29       ` David Sterba
  2014-06-18 16:18         ` Eric Sandeen
  2014-06-18 20:59         ` WorMzy Tykashi
  0 siblings, 2 replies; 10+ messages in thread
From: David Sterba @ 2014-06-18 15:29 UTC (permalink / raw)
  To: Eric Sandeen; +Cc: dsterba, linux-btrfs, Karel Zak

On Thu, Jun 12, 2014 at 09:39:14AM -0500, Eric Sandeen wrote:
> > So what if the mount options are generated from btrfs-mount.txt but
> > installed under btrfs.5.gz name? If there are more section 5 manpages we
> > can make it more generic but for now hardcoding btrfs-mount.* ->
> > btrfs.5. sounds ok to me.
> 
> Yeah, that seemed like kind of nasty hard-coding, but I suppose it works
> for now.  I wanted to make it more generic, I didn't have a better idea..

Added:

--- a/Documentation/Makefile
+++ b/Documentation/Makefile
@@ -85,6 +85,9 @@ install-man: man
        $(INSTALL) -d -m 755 $(DESTDIR)$(man5dir)
        $(INSTALL) -d -m 755 $(DESTDIR)$(man8dir)
        $(INSTALL) -m 644 $(GZ_MAN5) $(DESTDIR)$(man5dir)
+       # the source file name of btrfs.5 clashes with section 8 page, but we
+       # want to keep the code generic
+       mv $(DESTDIR)$(man5dir)/btrfs-mount.5.gz $(DESTDIR)$(man5dir)/btrfs.5.gz
        $(INSTALL) -m 644 $(GZ_MAN8) $(DESTDIR)$(man8dir)
        $(LNS) btrfs-check.8.gz $(DESTDIR)$(man8dir)/btrfsck.8.gz

So the MAN5 code is generic, the exception is only at the installation time.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH V2] btrfs-progs: add mount options to btrfs-mount.5
  2014-06-18 15:29       ` David Sterba
@ 2014-06-18 16:18         ` Eric Sandeen
  2014-06-18 20:59         ` WorMzy Tykashi
  1 sibling, 0 replies; 10+ messages in thread
From: Eric Sandeen @ 2014-06-18 16:18 UTC (permalink / raw)
  To: dsterba, linux-btrfs, Karel Zak

On 6/18/14, 10:29 AM, David Sterba wrote:
> On Thu, Jun 12, 2014 at 09:39:14AM -0500, Eric Sandeen wrote:
>>> So what if the mount options are generated from btrfs-mount.txt but
>>> installed under btrfs.5.gz name? If there are more section 5 manpages we
>>> can make it more generic but for now hardcoding btrfs-mount.* ->
>>> btrfs.5. sounds ok to me.
>>
>> Yeah, that seemed like kind of nasty hard-coding, but I suppose it works
>> for now.  I wanted to make it more generic, I didn't have a better idea..
> 
> Added:
> 
> --- a/Documentation/Makefile
> +++ b/Documentation/Makefile
> @@ -85,6 +85,9 @@ install-man: man
>         $(INSTALL) -d -m 755 $(DESTDIR)$(man5dir)
>         $(INSTALL) -d -m 755 $(DESTDIR)$(man8dir)
>         $(INSTALL) -m 644 $(GZ_MAN5) $(DESTDIR)$(man5dir)
> +       # the source file name of btrfs.5 clashes with section 8 page, but we
> +       # want to keep the code generic
> +       mv $(DESTDIR)$(man5dir)/btrfs-mount.5.gz $(DESTDIR)$(man5dir)/btrfs.5.gz
>         $(INSTALL) -m 644 $(GZ_MAN8) $(DESTDIR)$(man8dir)
>         $(LNS) btrfs-check.8.gz $(DESTDIR)$(man8dir)/btrfsck.8.gz
> 
> So the MAN5 code is generic, the exception is only at the installation time.
> 

Looks good to me, thanks.

-Eric

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH V2] btrfs-progs: add mount options to btrfs-mount.5
  2014-06-18 15:29       ` David Sterba
  2014-06-18 16:18         ` Eric Sandeen
@ 2014-06-18 20:59         ` WorMzy Tykashi
  2014-06-18 23:05           ` David Sterba
  1 sibling, 1 reply; 10+ messages in thread
From: WorMzy Tykashi @ 2014-06-18 20:59 UTC (permalink / raw)
  To: dsterba, Eric Sandeen, linux-btrfs, Karel Zak

On 18 June 2014 16:29, David Sterba <dsterba@suse.cz> wrote:
> On Thu, Jun 12, 2014 at 09:39:14AM -0500, Eric Sandeen wrote:
>> > So what if the mount options are generated from btrfs-mount.txt but
>> > installed under btrfs.5.gz name? If there are more section 5 manpages we
>> > can make it more generic but for now hardcoding btrfs-mount.* ->
>> > btrfs.5. sounds ok to me.
>>
>> Yeah, that seemed like kind of nasty hard-coding, but I suppose it works
>> for now.  I wanted to make it more generic, I didn't have a better idea..
>
> Added:
>
> --- a/Documentation/Makefile
> +++ b/Documentation/Makefile
> @@ -85,6 +85,9 @@ install-man: man
>         $(INSTALL) -d -m 755 $(DESTDIR)$(man5dir)
>         $(INSTALL) -d -m 755 $(DESTDIR)$(man8dir)
>         $(INSTALL) -m 644 $(GZ_MAN5) $(DESTDIR)$(man5dir)
> +       # the source file name of btrfs.5 clashes with section 8 page, but we
> +       # want to keep the code generic
> +       mv $(DESTDIR)$(man5dir)/btrfs-mount.5.gz $(DESTDIR)$(man5dir)/btrfs.5.gz
>         $(INSTALL) -m 644 $(GZ_MAN8) $(DESTDIR)$(man8dir)
>         $(LNS) btrfs-check.8.gz $(DESTDIR)$(man8dir)/btrfsck.8.gz
>
> So the MAN5 code is generic, the exception is only at the installation time.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Hi David,

I think you forgot to apply the patch that adds
Documentation/btrfs-mount.5.txt before you tagged
integration-20140618, man5 (and consequently Documentation) can't be
made without it, causing a failed build.

make: *** No rule to make target 'btrfs-mount.5.gz', needed by 'man5'.  Stop.

Any chance of a quick fix and a -2 tag?

Cheers,


WorMzy

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH V2] btrfs-progs: add mount options to btrfs-mount.5
  2014-06-18 20:59         ` WorMzy Tykashi
@ 2014-06-18 23:05           ` David Sterba
  2014-06-19  7:43             ` WorMzy Tykashi
  0 siblings, 1 reply; 10+ messages in thread
From: David Sterba @ 2014-06-18 23:05 UTC (permalink / raw)
  To: WorMzy Tykashi; +Cc: Eric Sandeen, linux-btrfs, Karel Zak

On Wed, Jun 18, 2014 at 09:59:50PM +0100, WorMzy Tykashi wrote:
> I think you forgot to apply the patch that adds
> Documentation/btrfs-mount.5.txt before you tagged
> integration-20140618, man5 (and consequently Documentation) can't be
> made without it, causing a failed build.

You're right, sorry. The file was not added after a conflict with
git-am and I did not check afterwards. Local build test passed.
Updated integration pushed, though it's a bit reduced. The 'next'
sub-branch rebase had some suspicious glitch, I need to look closer
tomorrow.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH V2] btrfs-progs: add mount options to btrfs-mount.5
  2014-06-18 23:05           ` David Sterba
@ 2014-06-19  7:43             ` WorMzy Tykashi
  2014-06-19 12:26               ` David Sterba
  0 siblings, 1 reply; 10+ messages in thread
From: WorMzy Tykashi @ 2014-06-19  7:43 UTC (permalink / raw)
  To: dsterba, WorMzy Tykashi, Eric Sandeen, linux-btrfs, Karel Zak

On 19 June 2014 00:05, David Sterba <dsterba@suse.cz> wrote:
> On Wed, Jun 18, 2014 at 09:59:50PM +0100, WorMzy Tykashi wrote:
>> I think you forgot to apply the patch that adds
>> Documentation/btrfs-mount.5.txt before you tagged
>> integration-20140618, man5 (and consequently Documentation) can't be
>> made without it, causing a failed build.
>
> You're right, sorry. The file was not added after a conflict with
> git-am and I did not check afterwards. Local build test passed.
> Updated integration pushed, though it's a bit reduced. The 'next'
> sub-branch rebase had some suspicious glitch, I need to look closer
> tomorrow.

Thanks for the fast update. :)

Good luck battling that glitch!

Cheers,


WorMzy

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH V2] btrfs-progs: add mount options to btrfs-mount.5
  2014-06-19  7:43             ` WorMzy Tykashi
@ 2014-06-19 12:26               ` David Sterba
  0 siblings, 0 replies; 10+ messages in thread
From: David Sterba @ 2014-06-19 12:26 UTC (permalink / raw)
  To: WorMzy Tykashi; +Cc: Eric Sandeen, linux-btrfs, Karel Zak

On Thu, Jun 19, 2014 at 08:43:36AM +0100, WorMzy Tykashi wrote:
> > You're right, sorry. The file was not added after a conflict with
> > git-am and I did not check afterwards. Local build test passed.
> > Updated integration pushed, though it's a bit reduced. The 'next'
> > sub-branch rebase had some suspicious glitch, I need to look closer
> > tomorrow.
> 
> Thanks for the fast update. :)

The next branch merged via format-patch/git-am instead of rebase and did
not hit any conflicts. Branch readded and integration pushed with
today's tag, without other patches.

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2014-06-19 12:26 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-11 22:13 [PATCH] btrfs-progs: add mount options to btrfs-mount.5 Eric Sandeen
2014-06-11 22:14 ` [PATCH V2] " Eric Sandeen
2014-06-12 10:51   ` David Sterba
2014-06-12 14:39     ` Eric Sandeen
2014-06-18 15:29       ` David Sterba
2014-06-18 16:18         ` Eric Sandeen
2014-06-18 20:59         ` WorMzy Tykashi
2014-06-18 23:05           ` David Sterba
2014-06-19  7:43             ` WorMzy Tykashi
2014-06-19 12:26               ` David Sterba

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).