diff mbox

btrfs-progs: check: add experimental flag for lowmem mode

Message ID 20180703114237.11053-1-ddiss@suse.de (mailing list archive)
State New, archived
Headers show

Commit Message

David Disseldorp July 3, 2018, 11:42 a.m. UTC
The experimental flag is already carried in the manpage, but was removed
from the btrfs check usage message as part of refactoring via
87c1bd13c1fca430c3dbf0da62e9aa33bde609c8. Add it back.

Signed-off-by: David Disseldorp <ddiss@suse.de>
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
---
 check/main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Su Yue July 4, 2018, 1:19 a.m. UTC | #1
On 07/03/2018 07:42 PM, David Disseldorp wrote:
> The experimental flag is already carried in the manpage, but was removed
> from the btrfs check usage message as part of refactoring via
> 87c1bd13c1fca430c3dbf0da62e9aa33bde609c8. Add it back.
> 
> Signed-off-by: David Disseldorp <ddiss@suse.de>
> Reviewed-by: Nikolay Borisov <nborisov@suse.com>

Reviewed-by: Su Yue <suy.fnst@cn.fujitsu.com>
> ---
>   check/main.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/check/main.c b/check/main.c
> index 3190b5d4..9aca0c09 100644
> --- a/check/main.c
> +++ b/check/main.c
> @@ -9386,7 +9386,7 @@ const char * const cmd_check_usage[] = {
>   	"                            original - read inodes and extents to memory (requires",
>   	"                                       more memory, does less IO)",
>   	"                            lowmem   - try to use less memory but read blocks again",
> -	"                                       when needed",
> +	"                                       when needed (experimental)",
>   	"--check-data-csum           verify checksums of data blocks",
>   	"-Q|--qgroup-report          print a report on qgroup consistency",
>   	"-E|--subvol-extents <subvolid>",
> 


--
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
David Sterba July 4, 2018, 11:55 a.m. UTC | #2
On Tue, Jul 03, 2018 at 01:42:37PM +0200, David Disseldorp wrote:
> The experimental flag is already carried in the manpage, but was removed
> from the btrfs check usage message as part of refactoring via
> 87c1bd13c1fca430c3dbf0da62e9aa33bde609c8. Add it back.
> 
> Signed-off-by: David Disseldorp <ddiss@suse.de>
> Reviewed-by: Nikolay Borisov <nborisov@suse.com>

Applied, thanks.
--
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
diff mbox

Patch

diff --git a/check/main.c b/check/main.c
index 3190b5d4..9aca0c09 100644
--- a/check/main.c
+++ b/check/main.c
@@ -9386,7 +9386,7 @@  const char * const cmd_check_usage[] = {
 	"                            original - read inodes and extents to memory (requires",
 	"                                       more memory, does less IO)",
 	"                            lowmem   - try to use less memory but read blocks again",
-	"                                       when needed",
+	"                                       when needed (experimental)",
 	"--check-data-csum           verify checksums of data blocks",
 	"-Q|--qgroup-report          print a report on qgroup consistency",
 	"-E|--subvol-extents <subvolid>",