Improve logging accuracy

If we do windowed logging (over a period of time), then we can
take the logging out of the IO worker fast path, and just have
our helper thread do it. This means we'll reduce the overhead
for the IO worker (which is important), and that we'll be able
to provide better accuracy of when we log. Before this change,
we'd log when we had exceeded the window of time at a completion
even. This means that if you had a logging time of 500 msec,
you'd always exceed that and even tend to skew it to being much
later. With this change, logging should be very accurate time
wise:

$ ./fio --inflate-log=fsrandW4-8-iopslog_iops.1.log.fz | head -n10
500, 470, 0, 0
1000, 492, 0, 0
1500, 484, 0, 0
2000, 492, 0, 0
2500, 454, 0, 0
3000, 462, 0, 0
3500, 480, 0, 0
4000, 462, 0, 0
4500, 472, 0, 0
5000, 496, 0, 0

and not exhibit any drift at all.

Signed-off-by: Jens Axboe <axboe@fb.com>

libfio: clear iops/bw sample times on stats reset

Signed-off-by: Jens Axboe <axboe@fb.com>

iolog: add define for default number of log entries

We use two different ones, depending on gz or not...

Signed-off-by: Jens Axboe <axboe@fb.com>

iolog: free memory on gz init failure

Signed-off-by: Jens Axboe <axboe@fb.com>

iolog: don't use the workqueue for sync work

Just run it inline. We'll need this going forward, since we can't
always rely on the proper shared memory from the helper thread.

Signed-off-by: Jens Axboe <axboe@fb.com>

iolog: fix use-after-free of iolog_flush_data

If we run with compressed logs, drop_data_unlock() will free 'data'
before killing the condvar and mutex.

Signed-off-by: Jens Axboe <axboe@fb.com>

hash: import Linux sparse hash fix

This is commit 689de1d6ca95 in the upstream Linux kernel tree.

Signed-off-by: Jens Axboe <axboe@fb.com>

Update RBD documentation

Sync them between HOWTO and fio.1 as well.

Signed-off-by: Jens Axboe <axboe@fb.com>

rbd: add clustername option

As the Ceph Jewel is released, ceph cluster name
may not be the default "ceph". So we should add
the clustername for rbd engine to support that.

Signed-off-by: Tianqing <tianqing@unitedstack.com>
Fixed up various coding style violations by me.

Signed-off-by: Jens Axboe <axboe@fb.com>

Fio 2.9

Signed-off-by: Jens Axboe <axboe@fb.com>

oslib/getopt_long: allow (unique) short match

The GNU version allows short matches, if they are unique. Our version
does not, since it's weird, but let's at least make them have the
same behavior. Otherwise users will have different behavior for
command line parsing for edge cases, on different platforms.

Signed-off-by: Jens Axboe <axboe@fb.com>

include sys/sysmacros.h for major/minor

These functions have always been defined in sys/sysmacros.h under
Linux C libraries.  For some, including sys/types.h implicitly
includes that as well, but glibc wants to deprecate that, and some
others already have.  Include the header explicitly for the funcs.

The mtd change is already in upstream mtd-utils too.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
Signed-off-by: Jens Axboe <axboe@fb.com>

Documentation: it's 'log_max_value', not 'log_max'

Signed-off-by: Jens Axboe <axboe@fb.com>

t/fio-verify-state: show completions in order

We currently show them in array order, but that's logged in
reverse. So show them from newest to latest.

Signed-off-by: Jens Axboe <axboe@fb.com>

filesetup: assign fileno at setup time

For duplicate jobs, we don't always get fileno assigned if we have
multiple files per job. Ensure that we do.

Signed-off-by: Jens Axboe <axboe@fb.com>

t/fio-verify-state: allow multiple input files

Signed-off-by: Jens Axboe <axboe@fb.com>

client: bool conversion

Signed-off-by: Jens Axboe <axboe@fb.com>

t/fio-verify-state: pretty up output a bit

Also parseable now.

Signed-off-by: Jens Axboe <axboe@fb.com>

Cleanup last write logging

Signed-off-by: Jens Axboe <axboe@fb.com>

Fix verify state for multiple files

Fio currently only logs the completions on a per-job basis, but
that breaks down if the job uses multiple files and we rely
on the verify state save/load to check when to stop verifying.

This patch extends the verify state format to be version 3, and
removes support for v1/v2. The new version has support for
multiple files.

Signed-off-by: Jens Axboe <axboe@fb.com>

Modify RDMA engine to print strerror messages.

In order to ease debugging it would be nice to know why certain
rdma/ib functions fail. I've added %m messages to a large
number of log_err lines.

fio: register pvsync2 engine correctly

I'm experimenting with the new syscalls and noticed fio's support was
incomplete. This patch fixes some of those issues.

Signed-off-by: Jon Derrick <jonathan.derrick@intel.com>
Signed-off-by: Jens Axboe <axboe@fb.com>

HOWTO/man: clarify that the usr/sys utilization numbers are averages

Signed-off-by: Jens Axboe <axboe@fb.com>

Fix crash with group_reporting and one group member that never started

If we have a job that nevers gets started, then we crash in
setting up the group stats, if group_reporting is enabled. This
can be reproduced like this:

fio --name=iwontstart --name=null --ioengine=null --size=1g --group_reporting=1
Starting 2 processes
iwontstart: you need to specify size=
fio: pid=0, err=22/file:filesetup.c:828, func=total_file_size, error=Invalid argument
Segmentation fault (core dumped)

Check for -1 groupid, and just skip the job if so.

Signed-off-by: Jens Axboe <axboe@fb.com>

t/read-to-pipe-async: use gettimeofday() instead of clock_gettime()

We don't have clock_gettime() on all supported platforms, we can
just use gettimeofday() and convert to timespec.

Signed-off-by: Jens Axboe <axboe@fb.com>

t/read-to-pipe-async: synchronization fixes

Signed-off-by: Jens Axboe <axboe@fb.com>

t/memlock: sample utility to use X memory from Y threads

Signed-off-by: Jens Axboe <axboe@fb.com>

Makefile: disable build of t/read-to-pipe-async by default

It's not part of fio, if you need it, you can just do:

$ make t/read-to-pipe-async

and it should build.

Signed-off-by: Jens Axboe <axboe@fb.com>

t/read-to-pipe-async: needs time.h on some platforms

Signed-off-by: Jens Axboe <axboe@fb.com>

t/read-to-pipe-async: standalone test app

Reads from a file through reads, spawning more threads if necessary
to keep the latency target. Writes the result to a pipe, and outputs
the latencies at the end.

Signed-off-by: Jens Axboe <axboe@fb.com>

backend: ensure that we run verification for short time based jobs

We need to break out of the main loop for the case where we did
exceed our bytes issued, if we have verification to run. For
time based jobs, we'll get back in this loop after verification.

Fixes c04e4661e4da ("time_based: Avoid restarting main I/O loop")
Signed-off-by: Jens Axboe <axboe@fb.com>

travis.yml: ensure we have libaio-dev and numa dev libs

Signed-off-by: Jens Axboe <axboe@fb.com>

Makefile: add empty 'test' target

Signed-off-by: Jens Axboe <axboe@fb.com>

Add .travis.yml

Signed-off-by: Jens Axboe <axboe@fb.com>

gen-rand: fix dependency on strcasestr.o

Fails on systems where we have to pull that in (eg Windows).

Signed-off-by: Jens Axboe <axboe@fb.com>

init: seed repeatable jobs differently

Factor in the job number for the stock random seeds, so we don't
generate the same patterns for identical jobs.

Signed-off-by: Jens Axboe <axboe@fb.com>

Fio 2.8

Signed-off-by: Jens Axboe <axboe@fb.com>

verify: clear IO verify state all_io_list before writing

We only fill it partially sometimes, so ensure that we clear
the whole thing before writing it out. This fixes valgrind
complaining about writing unitialized memory.

Signed-off-by: Jens Axboe <axboe@fb.com>

Revert "options: move pattern_fmt_desc where we need it"

This reverts commit 5f3bd0fd67254e3750681f0700a0cc162faee500.

We do need it to be globally reachable, otherwise we assign a
pointer to an on-stack structure. This causes the pattern filling
to read garbage, and segfault.

Signed-off-by: Jens Axboe <axboe@fb.com>

lib/rand: make __init_randX() static

Signed-off-by: Jens Axboe <axboe@fb.com>

Use 32-bit rand for parts that use rand_between()

We don't need a full 32-bit space for these, and the 32-bit generator
is faster.

Signed-off-by: Jens Axboe <axboe@fb.com>

t/gen-rand: use 32-bit random generator

This was inadvertently changed.

Signed-off-by: Jens Axboe <axboe@fb.com>

Fix compile of test programs on archs that use arch_flags at runtime

SuperH compile currently fails with:

gettime.o: In function fio_gettime':
/home/thomas/projets/buildroot/output/build/fio-fio-2.7/gettime.c:163: undefined reference to arch_flags'
gettime.o: In function utime_since_now':
/home/thomas/projets/buildroot/output/build/fio-fio-2.7/gettime.c:164: undefined reference to arch_flags'
gettime.o: In function mtime_since_now':
/home/thomas/projets/buildroot/output/build/fio-fio-2.7/gettime.c:164: undefined reference to arch_flags'
gettime.o: In function time_since_now':
/home/thomas/projets/buildroot/output/build/fio-fio-2.7/gettime.c:164: undefined reference to arch_flags'
mutex.o: In function fio_mutex_up':
/home/thomas/projets/buildroot/output/build/fio-fio-2.7/mutex.c:189: undefined reference to arch_flags'
collect2: error: ld returned 1 exit status
Makefile:375: recipe for target 't/stest' failed
make[2]: *** [t/stest] Error 1
make[2]: *** Waiting for unfinished jobs....

Fix that by ensuring we have a stub arch.o with the necessary arch flags
for the standalone test programs.

Signed-off-by: Jens Axboe <axboe@fb.com>

t/gen-rand: remove compile warning on 32-bit

Signed-off-by: Jens Axboe <axboe@fb.com>

Fio 2.7

Signed-off-by: Jens Axboe <axboe@fb.com>

.gitignore: ignore vim undo files

Signed-off-by: Jens Axboe <axboe@fb.com>

Add t/gen-rand to test random generator

Signed-off-by: Jens Axboe <axboe@fb.com>

Update documentation on log file formats

Signed-off-by: Jens Axboe <axboe@fb.com>

rand: use bools

Signed-off-by: Jens Axboe <axboe@fb.com>

Add the sample JESD219 job file

Based on the posting from Jeff.

Signed-off-by: Jens Axboe <axboe@fb.com>

io_u: speed up __get_next_buflen()

Change the divide-by-100 into a multiplier instead.

Signed-off-by: Jens Axboe <axboe@fb.com>

options: improvements to parse dry run

Signed-off-by: Jens Axboe <axboe@fb.com>

options: finish merge of bssplit/rand zone code

Signed-off-by: Jens Axboe <axboe@fb.com>

options: unify the bssplit/zone split code

Most of that parsing code was duplicated, apply a bit of abstraction
to it. Especially because it's not that pretty, so rather have one
copy of that nastiness.

Signed-off-by: Jens Axboe <axboe@fb.com>

Fix double free of td zone state index

Signed-off-by: Jens Axboe <axboe@fb.com>

Log to parent instead of twice to child td

options: clean number of zones if we fail parsing

Leaves us in a cleaner state, since the zone generation will never
attempt to touch uninitialized data then.

Signed-off-by: Jens Axboe <axboe@fb.com>

Add support for zones of random IO, with varying frequency of access

Let's say you want to ensure that 50% of the IO falls in the first 5%
of the flie, with the remaining 50% over the last 95%. Now that's
possible with random_distribution=zoned. For this particular case,
you would do:

random_distribution=zoned:50/5:50/95

Up to 64 ranges can be specified, and multiple data directions can
be given as well. The above would apply to both reads, writes, and
trims. If you wanted to have 50% of the writes fall in the first
10%, 25% in the next 10%, and the last 25% over the remaining 80%,
you could extend the above ala:

random_distribution=zoned:50/5:50/95,50/10:25/10:25/80

Signed-off-by: Jens Axboe <axboe@fb.com>

Update documentation for random_distribution=gauss

For some reason this was never committed, add it now.

Signed-off-by: Jens Axboe <axboe@fb.com>

Add support for preadv2/pwritev2

This is coming on Linux, and supports a high priority flag that will
use polling on the completion side.

Basic support is there, we'll need to add wrappers on x86_64 linux
to use them before it shows up in the glibc devel headers.

Signed-off-by: Jens Axboe <axboe@fb.com>

ioengines: account any queued IO on the engine side

Currently we don't account if it's not a read/write/trim, but that
can screw up with SYNC, for instance. Account any IO that the
IO engine has returned a queued status for.

Signed-off-by: Jens Axboe <axboe@fb.com>

backend: ensure that fio_io_sync() commits IN on queued status

If we get a FIO_Q_QUEUED back, we need to commit pending IO
before we wait for it. Otherwise we trigger this bug:

fio: ioengines.c:257: td_io_getevents: Assertion `td->io_u_in_flight >= r' failed.

Reported-by: Andrey Kuzmin <andrey.v.kuzmin@gmail.com>
Signed-off-by: Jens Axboe <axboe@fb.com>

Allow IO engine driven allocations of IO structures

Based on a patch from Andrey Kuzmin, but modified so that we error
out if the user has specified a specific memory backing type
through the use of mem= or iomem=.

Signed-off-by: Jens Axboe <axboe@fb.com>

Allow for the include file specification to be relative.

Simplifies include file grouping, with same directory file included
via file name. Checked only if the lookup by the file name specified
fails.

Signed-off-by: Jens Axboe <axboe@fb.com>

fio.1: man page fixes

Signed-off-by: Jens Axboe <axboe@fb.com>

Update documentation

Section numbering duplicated in one place in the HOWTO.
Add the missing sections to the man page.

Signed-off-by: Jens Axboe <axboe@fb.com>

server: don't make SO_REUSEPORT errors fatal

Signed-off-by: Jens Axboe <axboe@fb.com>

diskutil: don't print terse disk util twice for json,terse output format

Signed-off-by: Jens Axboe <axboe@fb.com>

diskutil: fix segfault for both json and terse output

Signed-off-by: Jens Axboe <axboe@fb.com>

io_ddir: io_ddir_name array should be static

Signed-off-by: Jens Axboe <axboe@fb.com>

fio: fix copy/paste error in io_ddir.h

The last 'write' should be 'wait', to match with DDIR_WAIT.

Signed-off-by: Jens Axboe <axboe@fb.com>

log: add support for logging max instead of averages

If we log with lov_avg_msec, fio will log the average over that
period of time to the log. Add log_max_value to change this to
logging maximums over the windowed period instead.

Signed-off-by: Jens Axboe <axboe@fb.com>

stat: cleanup average logging

Signed-off-by: Jens Axboe <axboe@fb.com>

fio: fix ignore_error regression

Commit e9d512d8e7d2 introduced a regression affecting handling for
the ignore_error parameter.  The common code for io_u event handling
pulled from do_io() and do_verify() into io_queue_event() does not
check for and process errors that are to be ignored in the main I/O
path as I/Os are completed.  Prior to that commit, do_io() called
break_on_this_error() to handle that case.

The change causes fio to return a non-zero exit status after
encountering an error it should ignore while performing I/O, even
though the job does run to completion. This regression causes
failures for some fstests that use fio, including ext4/302, ext4/303,
and generic/300.

This patch restores the original error handling behavior.

Signed-off-by: Eric Whitney <enwlinux@gmail.com>
Signed-off-by: Jens Axboe <axboe@fb.com>

libmtd: compile fix for linux header changes.

In the Linux kernel source, mtd/mtd-user.h had a change
'mtd: mtd-user: remove stdint.h include'
(137d36af4a53858b8db7ca83c8480247118b8bdf)

This causes the uint8_t/uint64_t types to be undefined in libmtd.h
now, as they were implicitly coming from it before.

Import stdint.h explicitly into libmtd.h to resolve this.

X-URL: https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=137d36af4a53858b8db7ca83c8480247118b8bdf
Signed-off-by: Robin H. Johnson <robbat2@gentoo.org>

Makefile: false positive on _FORTIFY_SOURCE

If the compiler is very pedantic and includes _FORTIFY_SOURCE, then you
get the following warning about redefinition:
<command-line>:0:0: warning: "_FORTIFY_SOURCE" redefined

To prevent it, undefine and redefine _FORTIFY_SOURCE.

Gentoo's gcc does this for -O2.

Signed-off-by: Robin H. Johnson <robbat2@gentoo.org>

Add a note of what is the default value for random_generator

Signed-off-by: Jens Axboe <axboe@fb.com>

server: send trigger state inline

If we don't send it inline, we can't be sure that it's transmitted
if the trigger event is a hard reboot or similar.

Signed-off-by: Jens Axboe <axboe@fb.com>

client: fix double removal of client on job file open failure

If fio is invoked as a client and the job file does not exist, it
will exit ala:

fio: job file <x.fio> open: No such file or directory
fio: client.c:240: remove_client: Assertion `client->refs' failed.

because the break path ends up doing the removal twice. Fix that
to only remove it once, so we don't trigger the assert.

Signed-off-by: Jens Axboe <axboe@fb.com>

Fio 2.6

Signed-off-by: Jens Axboe <axboe@fb.com>

server: split vectored commands into fragments

This fixes the fio_send_cmd_ext_pdu() path to split commands
into properly sized fragments. This also reverts the
previous commit, that bypassed the check since this path
was broken.

Signed-off-by: Jens Axboe <axboe@fb.com>

server: disable pdu length check for now

Vectored entries don't always honor it. Allow longer payloads, until
we fix the breakdown again.

Signed-off-by: Jens Axboe <axboe@fb.com>

client: remove debug printf() for compressed iolog

Signed-off-by: Jens Axboe <axboe@fb.com>

powerpc: fix cpus_online() to get correct max CPU number for powerpc64

This patch tries to fix cpus_online() for powerpc64 when running fio
by using parameter "cpus_allowed", in order to obtain correct max CPU
number on powerpc64 architecture.

When changing SMT settings on powerpc architecture, the way of cpu
numbering is changing as well. For example, when SMT=off, CPUs are
numbered like 0,8,16,..., while when SMT=2, CPUs are numbered like
0,1,8,9,.... Therefore, sysconf(_SC_NPROCESSORS_ONLN) is not applied
to get exact max CPU number on powerpc. And the max_cpu value can be
correctly ontained by using sysconf(_SC_NPROCESSORS_CONF).

Signed-off-by: Dan Qing Zhang <dqzhang@linux.vnet.ibm.com>
Signed-off-by: Jens Axboe <axboe@fb.com>

Fix Windows build: define rand_r()

Makefile: fix duplicate strcasestr for windows

Signed-off-by: Jens Axboe <axboe@fb.com>

server: use the wrapped MSG_DONTWAIT

Signed-off-by: Jens Axboe <axboe@fb.com>

ioengine: remove duplicate queue handler check

Signed-off-by: David Disseldorp <ddiss@suse.de>

Fio 2.5

2.4 was short lived, 2.5 fixes an important regression that
happened between fio-2.2.12 and fio-2.2.13.

Signed-off-by: Jens Axboe <axboe@fb.com>

mutex: fix double unlock in fio_mutex_down_timeout()

In case of error, we unlock twice.

Fixes: 09400a60042e ("mutex: fixup fio_mutex_down_timeout()")
Signed-off-by: Jens Axboe <axboe@fb.com>

init: fix compile for FIO_INC_DEBUG not being set

Signed-off-by: Jens Axboe <axboe@fb.com>

Fio 2.4

Signed-off-by: Jens Axboe <axboe@fb.com>

init: increment stat count if parent == default thread

We skip doing the stat increment if group_reporting is set,
but that means we can end up with stat_number == 0. That
isn't correct, increment for group_reporting as well, but
only if the parent is the default thread. That means we
still won't do it for sub-jobs.

Signed-off-by: Jens Axboe <axboe@fb.com>

client: fix non-text command leak

Signed-off-by: Jens Axboe <axboe@fb.com>

client: drain text output when exiting

If we don't, we can lose error messages from the server side. One
example would be for protocol version mismatch - the server will
notify the client, but the client doesn't check for these messages
before exiting.

Signed-off-by: Jens Axboe <axboe@fb.com>

Fix issue with termination before io_size has been reached

If the size of the device isn't a multiple of the block size,
then we can exit prematurely for cases where an io_limit has
been set higher than the device size. This is because the file(s)
get marked completely done if bs is a multiple of the device size,
but doesn't for the opposite case.

We should not factor this in for deciding to keep running or not.

Signed-off-by: Jens Axboe <axboe@fb.com>

io_u: ensure that we align new start offset properly for time_based

If io_size isn't a multiple of the block size, weird things can
happen, which usually means that fio will exit with an unaligned
IO error:

fio: io_u error on file /dev/nvme0n1: Invalid argument: write offset=125830, buflen=1048576

where offset=125830 really should have been aligned to the block
size. Ensure that we do that.

Signed-off-by: Jens Axboe <axboe@fb.com>

t/verify-state: one more printf type fix

Signed-off-by: Jens Axboe <axboe@fb.com>

t/verify-state: fix type for printf

Signed-off-by: Jens Axboe <axboe@fb.com>

Destroy sk_out server key when we are done

Signed-off-by: Jens Axboe <axboe@fb.com>