From: Christoph Anton Mitterer <calestyo@scientia.org>
To: linux-btrfs@vger.kernel.org
Subject: [PATCH] btrfs-progs: docs: improve space cache documentation
Date: Mon, 16 Jan 2023 15:07:49 +0100 [thread overview]
Message-ID: <afabf6cea649902733684575037e718f151a2ce1.camel@scientia.org> (raw)
In-Reply-To: <bddd724d-649b-aa3b-9a97-f415fe7b6afd@gmx.com>
- "Wipe" in storage terms is often understood as some kind of secure deletion.
Use "remove" instead in order to indicate that the space cache is fully
removed (and not just cleared and then e.g. automatically rebuild).
- The --clear-space-cache option for btrfs check actually clears the whole space
cache, just as documented.
Thus move any documentation about the clear_cache mount option not doing so
for v1 to that.
Instead, refer to the mount option.
- Also note that when clear_cache is used with v1, the free space cache for
block groups that are modified gets always cleared, but rebuilt only if
nospace_cache is not used.
Signed-off-by: Christoph Anton Mitterer <mail@christoph.anton.mitterer.name>
---
Documentation/btrfs-check.rst | 12 ++----------
Documentation/ch-mount-options.rst | 18 +++++++++++++++---
2 files changed, 17 insertions(+), 13 deletions(-)
diff --git a/Documentation/btrfs-check.rst b/Documentation/btrfs-check.rst
index bca0e7f8..1bd35fe2 100644
--- a/Documentation/btrfs-check.rst
+++ b/Documentation/btrfs-check.rst
@@ -79,17 +79,9 @@ SAFE OR ADVISORY OPTIONS
superblock is damaged.
--clear-space-cache v1|v2
- completely wipe all free space cache of given type
+ completely remove the free space cache of the given version
- For free space cache *v1*, the *clear_cache* kernel mount option only rebuilds
- the free space cache for block groups that are modified while the filesystem is
- mounted with that option. Thus, using this option with *v1* makes it possible
- to actually clear the entire free space cache.
-
- For free space cache *v2*, the *clear_cache* kernel mount option destroys
- the entire free space cache. This option, with *v2* provides an alternative
- method of clearing the free space cache that doesn't require mounting the
- filesystem.
+ See also the *clear_cache* mount option.
--clear-ino-cache
remove leftover items pertaining to the deprecated inode map feature
diff --git a/Documentation/ch-mount-options.rst b/Documentation/ch-mount-options.rst
index 3536a5bb..c9b64f19 100644
--- a/Documentation/ch-mount-options.rst
+++ b/Documentation/ch-mount-options.rst
@@ -86,8 +86,19 @@ check_int, check_int_data, check_int_print_mask=<value>
for more information.
clear_cache
- Force clearing and rebuilding of the disk space cache if something
- has gone wrong. See also: *space_cache*.
+ Force clearing and rebuilding of the free space cache if something
+ has gone wrong.
+
+ For free space cache *v1*, this only clears (and, unless *nospace_cache* is
+ used, rebuilds) the free space cache for block groups that are modified while
+ the filesystem is mounted with that option. To actually clear an entire free
+ space cache *v1*, see ``btrfs check --clear-space-cache v1``.
+
+ For free space cache *v2*, this clears the entire free space cache.
+ To do so without requiring to mounting the filesystem, see
+ ``btrfs check --clear-space-cache v2``.
+
+ See also: *space_cache*.
commit=<seconds>
(since: 3.12, default: 30)
@@ -348,7 +359,8 @@ space_cache, space_cache=<version>, nospace_cache
implementation, which adds a new b-tree called the free space tree, addresses
this issue. Once enabled, the *v2* space cache will always be used and cannot
be disabled unless it is cleared. Use *clear_cache,space_cache=v1* or
- *clear_cache,nospace_cache* to do so. If *v2* is enabled, kernels without *v2*
+ *clear_cache,nospace_cache* to do so. If *v2* is enabled, and *v1* space
+ cache will be cleared (at the first mount) and kernels without *v2*
support will only be able to mount the filesystem in read-only mode.
The :doc:`btrfs-check(8)<btrfs-check>` and `:doc:`mkfs.btrfs(8)<mkfs.btrfs>` commands have full *v2* free space
--
2.39.0
next prev parent reply other threads:[~2023-01-16 14:24 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-16 10:22 Correct way to clear a free space cache file invalid error Martin Steigerwald
2023-01-16 10:41 ` Qu Wenruo
2023-01-16 14:07 ` Christoph Anton Mitterer [this message]
2023-01-16 15:24 ` [PATCH] btrfs-progs: docs: improve space cache documentation Christoph Anton Mitterer
2023-01-16 15:30 ` Correct way to clear a free space cache file invalid error Martin Steigerwald
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=afabf6cea649902733684575037e718f151a2ce1.camel@scientia.org \
--to=calestyo@scientia.org \
--cc=linux-btrfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).