Make verify_dump off by default

It's a change of behaviour, so it should be off by default.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

diff --git a/HOWTO b/HOWTO
index ef8f9599942bd08e63a96c8f32a24c6f176892e7..41edcf1a8ff820e09d915a4e69ceca11d8bb619c 100644 (file)
--- a/HOWTO
+++ b/HOWTO
@@ -981,7 +981,7 @@ verify_fatal=bool   Normally fio will keep checking the entire contents
 verify_dump=bool       If set, dump the contents of both the original data
                block and the data block we read off disk to files. This
                allows later analysis to inspect just what kind of data
-               corruption occurred. On by default.
+               corruption occurred. Off by default.
 
 verify_async=int       Fio will normally verify IO inline from the submitting
                thread. This option takes an integer describing how many

diff --git a/fio.1 b/fio.1
index 3b1112256f77595fd6c696ca54bb2d83be051738..aae7657281a893f97b8cede4f4b5e59b23d3a5d8 100644 (file)
--- a/fio.1
+++ b/fio.1
@@ -780,7 +780,7 @@ false.
 .BI verify_dump \fR=\fPbool
 If set, dump the contents of both the original data block and the data block we
 read off disk to files. This allows later analysis to inspect just what kind of
-data corruption occurred. On by default.
+data corruption occurred. Off by default.
 .TP
 .BI verify_async \fR=\fPint
 Fio will normally verify IO inline from the submitting thread. This option

diff --git a/options.c b/options.c
index 4207537e9c4a4bf4958213b6dd3cf6783582b766..bb46dc9df8cbf707a87671c4b1f0b1867dd79f4b 100644 (file)
--- a/options.c
+++ b/options.c
@@ -1581,7 +1581,7 @@ static struct fio_option options[FIO_MAX_OPTS] = {
                .name   = "verify_dump",
                .type   = FIO_OPT_BOOL,
                .off1   = td_var_offset(verify_dump),
-               .def    = "1",
+               .def    = "0",
                .help   = "Dump contents of good and bad blocks on failure",
                .parent = "verify",
        },