Merge branch 'num2str-patch' of https://github.com/gloit042/fio

* 'num2str-patch' of https://github.com/gloit042/fio:
  num2str: fix precision loss bug when the fractional part is close to 1

Merge branch 'cpu-engine' of https://github.com/bvanassche/fio

* 'cpu-engine' of https://github.com/bvanassche/fio:
  engines/cpu: Fix td_vmsg() call

engines/cpu: Fix td_vmsg() call

Make sure that the third td_vmsg() argument is a fully expanded string and
also that the fourth argument is a string argument. This was detected by
Coverity.

Fixes: b213922390fe ("engines/cpu: style cleanups")
Signed-off-by: Bart Van Assche <bvanassche@acm.org>

engines/cpu: style cleanups

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

Merge branch 'evelu-qsort' of https://github.com/ErwanAliasr1/fio

* 'evelu-qsort' of https://github.com/ErwanAliasr1/fio:
  engines/cpu: Adding qsort capabilities

engines/cpu: Adding qsort capabilities

This commit adds cpumode option into the cpuio engine.
By default, cpumode=noop to keep the current behavior.

If cpumode is set to qsort, fio will use a qsort algorithm
instead of the noop instructions to load the processor.
This mode will consume more cpu power and will be useful to
increase the pressure on the thermal and electrical components.

The expected cpu load is selected as per noop via the cpuload option.
qsort() consumes a lot of energy so the duration
of every loop will vary over time as the power management & cpu clock
changes. To ensure a proper calibration, the thinktime is adjusted after every
qsort() computation to be as precise as possible.

To give an order of magnitude, on an AMD 7502P (TDP=180W) :
cpuload=30, numjobs=64: packagewatt = 134W
cpuload=50, numjobs=64: packagewatt = 167W
cpuload=70, numjobs=64: packagewatt = 180W

The example file is updated to reflect this new capabilities.

The qsort code is coming from stress-qsort.c from stress-ng tool.
This software is also GPLv2 but author was informed and agreed with this usage.

Signed-off-by: Erwan Velu <e.velu@criteo.com>

Change ARRAY_SIZE to FIO_ARRAY_SIZE

Ensures that we don't clash with external symbols/macros.

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

Merge branch 'drop_xp' of https://github.com/sitsofe/fio

* 'drop_xp' of https://github.com/sitsofe/fio:
  windows: drop XP support

log: only compile log_prevalist() if FIO_INC_DEBUG is set

Only used for that case.

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

Merge branch 'fix-get-next-file' of https://github.com/aclamk/fio

* 'fix-get-next-file' of https://github.com/aclamk/fio:
  io_u: Fix bad interaction with --openfiles and non-sequential file selection policy

io_u: Fix bad interaction with --openfiles and non-sequential file selection policy

Problem happens when --openfiles is set and file_service_type != FIO_FSERVICE_SEQ.
In function __get_next_file, we decrement file_service_left and if 0, we select
next file to operate on.
However, get_next_file_rand can return -EBUSY if too many files are already opened,
and __get_next_file exits with error.

In next invocation of __get_next_file, we decrement file_service_left again (from 0),
wrapping around to 2^32-1, effectively locking __get_next_file to always select the same.

Algorithm to observe bad behavior:
fio --randseed=1 --ioengine=libaio --rw=randwrite --nrfiles=256 --bs=4k --size=256m \
--loops=50 --allow_file_create=1 --write_iolog=log.txt --file_service_type=normal:20 \
--filename_format=object.\$filenum --name=x --openfiles=100

cat log.txt |grep write |cut -f 1 -d " " |sort |uniq -c | sort -n | sed "s/[.]/ /" \
| while read a b c; do echo $c $b $a; done |sort -n
....
70 object 17
71 object 19
72 object 22
73 object 65296
74 object 65255
75 object 33
76 object 27
77 object 25
78 object 65243
79 object 36
80 object 49
81 object 47
....

Signed-off-by: Adam Kupczyk <akupczyk@redhat.com>

windows: drop XP support

Windows 2003 has been EOL since 2015 and Windows XP has been EOL since
2014. Technically fio hasn't been properly targetting XP anyway - (see
16d40a15783f9181d64082fc00a970dff485798f 'configure: be explicit about
"XP" Windows API version').

- Strip out support for XP and change the documentation to no longer
  mention it
- Stop targetting XP in the CI 32 bit build

Next stop, Windows 7...

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

Merge branch 'terse_units' of https://github.com/sitsofe/fio

* 'terse_units' of https://github.com/sitsofe/fio:
  docs: add missing units to terse headings

docs: add missing units to terse headings

In terse output many latency and bandwidth values have non-obvious
units. In the documentation of the format add a microsecond suffix to
latency values and a kb suffix to bandwidth values to clarify things.

Fixes: https://github.com/axboe/fio/issues/1094
Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

num2str: fix precision loss bug when the fractional part is close to 1

example:
The result of num2str(11999999999999, 4, 1, 0, N2S_NONE) should
be "12.0G", but current result is "11.0G".

Signed-off-by: Jiahao Li <gloit042@gmail.com>

Merge branch 'github_issue' of https://github.com/sitsofe/fio

* 'github_issue' of https://github.com/sitsofe/fio:
  docs: add new section to REPORTING-BUGS and github issue templates

docs: add new section to REPORTING-BUGS and github issue templates

- Add a section to REPORTING-BUGS to clarify what is required for a good
  GitHub issue
- Add a note to REPORTING-BUGS that questions should go to the mailing
  list
- Add a bug GitHub issue template that encourages people to read
  REPORTING-BUGS and include necessary information
- Add a GitHub issue enhancement template to try and set expectations
  reasonably
- Add a SUPPORT.md file because GitHub seems to link to it when a
  reporter first makes an issue

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

Merge branch 'evelu-examples' of https://github.com/ErwanAliasr1/fio

* 'evelu-examples' of https://github.com/ErwanAliasr1/fio:
  examples: Clarify group_reporting usage
  examples: Clarify thread usage
  examples: Clarify time_based usage
  examples/fsx: Removing deprecated rwmixcycle options

examples: Clarify group_reporting usage

When reading examples, many jobs assign a value to group_reporting.
No value is expected, so let's have all examples using the same syntax.

Signed-off-by: Erwan Velu <e.velu@criteo.com>

examples: Clarify thread usage

When reading examples, many jobs assign a value to thread.
No value is expected, so let's have all examples using the same syntax.

Signed-off-by: Erwan Velu <e.velu@criteo.com>

examples: Clarify time_based usage

When reading examples, many jobs assign a value to time_based.
It could even very misleading as some put the same value as for the runtime.
No value is expected, so let's have all examples using the same syntax.

Signed-off-by: Erwan Velu <e.velu@criteo.com>

examples/fsx: Removing deprecated rwmixcycle options

Let's remove deprecated option to avoid confusion if some reads it.

Signed-off-by: Erwan Velu <e.velu@criteo.com>

Merge branch 'wip-rbd-engine-tweaks' of https://github.com/dillaman/fio

* 'wip-rbd-engine-tweaks' of https://github.com/dillaman/fio:
  engines/rbd: issue initial flush to enable writeback/around mode
  engines/rbd: add support for "direct=1" option

flow: fix hang with flow control and zoned block devices

When flow control is specified between 2 or more threads executing a workload
with zonemode=zbd and flow_sleep is not specified by the user (default to zero),
then a job A can lock a zone Z with the flow ratio exceeded, resulting in the
job to not issue the IO until other jobs IOs balance the flow ratio.
Since job A will wait with the Zone Z locked, other jobs will deadlock waiting
for this zone lock if their workload resulted in this zone Z being chosen
as an IO target.

Executing io_u_quiesce() when the flow ratio is exceeded solves this potential
deadlock as job A will wait for the flow ratio to balance without holding the
zone A lock. To preserve the behavior for regular block devices and to avoid
extra overhead, io_u_quiesce() only needs to be executed when the workload
specifies zonedmode=zbd.

Signed-off-by: Aravind Ramesh <aravind.ramesh@wdc.com>
Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

Merge branch 'reword-toolarge' of https://github.com/sitsofe/fio

* 'reword-toolarge' of https://github.com/sitsofe/fio:
  filesetup: reword block size too large message

filesetup: reword block size too large message

Fixes https://github.com/axboe/fio/issues/1132

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

Merge branch 'cufile' of https://github.com/SystemFabricWorks/fio

* 'cufile' of https://github.com/SystemFabricWorks/fio:
  ioengine: Add libcufile I/O engine

ioengine: Add libcufile I/O engine

The libcufile I/O engine uses NVIDIA GPUDirect Storage (GDS) cuFile API to perform
synchronous I/O directly against GPU buffers via nvidia-fs and a GDS-supported
filesystem.

'configure --enable-libcufile' enables the libcufile engine.

CFLAGS must specify the location of CUDA and cuFile headers.
e.g. CFLAGS="-I/usr/local/cuda/include -I/usr/local/cuda/lib64"

LDFLAGS must specify the location of CUDA and cuFile libraries.
e.g. LDFLAGS="-L/usr/local/cuda/lib64"

The paths used in CFLAGS and LDFLAGS depend upon the build host's
CUDA installation.

libcufile adds the following optons: gpu_dev_ids, cuda_io
Usage is documented in HOWTO, fio.1, examples/libcufile-cufile.fio
and examples/libcufile-posix.fio.

Note that enabling verify when cuda_io=cufile necessitates
cudaMemcpy() to populate the GPU buffer on a write and populate the
CPU buffer on a read. The primary goal of GDS is to not copy data
between CPU and GPU buffers.

Signed-off-by: Brian T. Smith <bsmith@systemfabricworks.com>

Fio 3.25

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

Merge branch 'stat-int-creep3' of https://github.com/jeffreyalien/fio

* 'stat-int-creep3' of https://github.com/jeffreyalien/fio:
  stat: Prevent the BW and IOPS logging interval from creeping up

engines/rbd: issue initial flush to enable writeback/around mode

By default librbd will use a writethrough cache until it receives
the first flush. Send a no-op flush after opening the image to
ensure writearound (default) or writeback cache modes can be
properly enabled (unlesss direct=1).

Signed-off-by: Jason Dillaman <dillaman@redhat.com>

engines/rbd: add support for "direct=1" option

When direct IO is enabled, the librbd in-memory cache will be
automatically disabled.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>

stat: Prevent the BW and IOPS logging interval from creeping up

With the current FIO version, we're seeing the interval for writing
the BW and IOPS logs increase by 1 msec over time.  The --log_avg_msec
option is used to set the time interval for logging BW and IOPs.

This issue will cause dip ( low ) in IOPS and BW plot vs time because
parsing/plot tool can't get correct sum total IOPS and BW from
multiple jobs when there is 1-2us increment in timestamp.

The issue was introduced in version 3.18 with the following
commit - 31eca64.  And these changes made it slightly better: 0f77d30,
and PR #973 (0e59dd6), but the problem still exists.

This commit also contains the change from PR #982.  I believe we need
both changes to totally resolve the issue of the BW and IOPs logging
interval creep.

Signed-off-by: Jeff Lien <jeff.lien@wdc.com>
Inspired-by: Alex Merenstein <mmerenstein@cs.stonybrook.edu>

Merge branch 'regrow_agg_logs' of https://github.com/pmoust/fio

* 'regrow_agg_logs' of https://github.com/pmoust/fio:
  stat: allow bandwidth log stats to grow to MAX_LOG_ENTRIES

stat: allow bandwidth log stats to grow to MAX_LOG_ENTRIES

This commit enables agg logs to grow to MAX_LOG_ENTRIES, like any thread logs.

Fixes: https://github.com/axboe/fio/issues/1113
Co-authored-by: Sitsofe Wheeler <sitsofe@yahoo.com>
Tested-by: Panagiotis Moustafellos <pmoust@elastic.co>
Signed-off-by: Panagiotis Moustafellos <pmoust@elastic.co>

Merge branch 'update-fio-ioops-version' of https://github.com/diameter/fio

* 'update-fio-ioops-version' of https://github.com/diameter/fio:
  ioengines: increment FIO_IOOPS_VERSION

ioengines: increment FIO_IOOPS_VERSION

Signed-off-by: Ivan Andreyev <ewantlf@gmail.com>

error out if ENOSPC during file layout

Hi,

When I run fio with --create_only=1 and it runs out of space, it still
exits with status 0.  I could not figure out if this was intentional
(except for the fill_device case, where this is obviously the expected
behavior).

$ cat ~/fio
[global]
ioengine=posixaio
rw=readwrite
size=2g
directory=${HOME}/mounts/testdisk
thread=1

[trivial-readwrite-1g]
$ df -h .
Filesystem      Size  Used Avail Use% Mounted on
/dev/loop6      976M  2.6M  907M   1% /home/kushal/mounts/testdisk
$ fio ~/fio --create_only=1
trivial-readwrite-1g: (g=0): rw=rw, bs=(R) 4096B-4096B, (W) 4096B-4096B, (T) 4096B-4096B, ioengine=pos
ixaio, iodepth=1
fio-3.12
Starting 1 thread
trivial-readwrite-1g: Laying out IO file (1 file / 2048MiB)
fio: ENOSPC on laying out file, stopping

Run status group 0 (all jobs):

Disk stats (read/write):
  loop6: ios=0/0, merge=0/0, ticks=0/0, in_queue=0, util=0.00%
$ echo $?
0

A trivial patch for this gives me the behavior I expect.

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

Merge branch 'segmented-threads'

Pull in the segmented segments code. This avoids having to setup a
thread area segment for the max (4096) number of jobs. Instead it's
done in chunks of 8. This results in a net reduction of more than 1G
of memory used for the normal use case.

* segmented-threads:
  Kill off 'max_jobs'
  Add thread_segments as needed
  Wrap thread_data in thread_segment

Kill off 'max_jobs'

We don't really need this anymore - we cap the number of segments
separately, and since the chunk size is small enough, this removes the
need to have per-OS limits on the thread area size.

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

Add thread_segments as needed

Setup segments in units of 8, and add extra ones as needed. This avoids
having to setup one huge segment upfront for the maximum number of jobs.

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

Wrap thread_data in thread_segment

No functional changes in this patch, just in preparation for having
multiple shm segments for the thread_data structures. They are getting
very large, and since it's hard to know upfront how many we need, let's
instead refactor the code a bit to allow us to add chunks of them as
needed when parsing the job file (or command line).

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

t/latency_percentiles.py: tweak terse output parse

Seems like we still have some wonkiness on whether or not the fio version
string is printed. For now, let's just cut this down to '3;' for
detecting terse (v3) output, don't have time to fully investigate this
one right now.

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

Fio 3.24

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

Makefile: fix fio version gen

A previous commit moved the cflags override, but neglected to move the
version generation that is being used by it.

Fixes: f4bd2c3d80bc ("fix dynamic engine build")
Signed-off-by: Jens Axboe <axboe@kernel.dk>

t/latency_percentiles.py: correct terse parse

For some reason the script expects '3;fio-' when it should be latency.

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

Make sure we do libaio engine compatability names

This goes for both the dynamic engines, and the builtin ones. If
'aio' or 'linuxaio' is used, we want 'libaio'.

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

configure: remove libaio-uring remnant

This was missed in the previous patch.

Fixes: d4946e79e833 ("Remove the "libaio over io_uring" mode")
Signed-off-by: Jens Axboe <axboe@kernel.dk>

Makefile: ensure that external libs are linked properly with dynamic engine

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

Remove the "libaio over io_uring" mode

Not going to maintain this mode in liburing, so just get rid of it. It's
also caused some confusion where users think they need this enabled to
run io_uring, which isn't the case.

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

FIO_EXT_ENG_DIR should be default path

Distros will override this anyway, make sure the default is what
will work on a normal build out of git.

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

list all available dynamic ioengines with --enghelp

When dynamic engines are enabled, "fio --enghelp" does not list all
available engines as the man page says it will:

If no ioengine is given, list all available ioengines.

Fix this by opening FIO_EXT_ENG_DIR and attempting dlopen_ioengine
everything that's found there.

Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

fix dynamic engine loading for libaio engine etc

The dynamic engine loading for libaio (and some others) currently
fails because the dlopen routine is looking for ("lib%s", enginename)
which translates into "liblibiscsi.so":

openat(AT_FDCWD, "/usr/lib64/fio/liblibiscsi.so", O_RDONLY|O_CLOEXEC) = -1 ENOENT (No such file or directory)
Engine libiscsi not found; Either name is invalid, was not built, or fio-engine-libiscsi package is missing.
fio: engine libiscsi not loadable
IO engine libiscsi not found

The Makefile decide to name this engine "iscsi" instead of "libiscsi",
which leads to "libiscsi.so" not "liblibiscsi.so" hence the mismatch.

OTOH, "liblibiscsi.so" seems a bit bonkers.

Try to resolve all this by:

1) make all of the engine names match the documented engine names
   in the Makefile, i.e. "iscsi" -> "libiscsi"
2) change the created library filenames to "fio-$(ENGINENAME)"
   from "lib$(ENGINENAME)" to avoid the "liblib" prefix.

So now we consistently have the libraries named "fio-$(ENGINENAME).so"

fio-http.so
fio-libaio.so

etc.

Fixes: 5a8a6a03 ("configure: new --dynamic-libengines build option")
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

fix dynamic engine build

Builds with --dynamic-libengines are currently broken, because the rule
to build the libraries was moved out of the template that creates them:

make: *** No rule to make target 'engines/libpmem.so', needed by 'all'.  Stop.
make: *** Waiting for unfinished jobs....

Fix this by moving the rule back into the template, calling the template
to create rules only after FIO_CFLAGS has been incorporated into CFLAGS,
and using CFLAGS + -fPIC in that rule rather than FIO_CFLAGS.

Fixes: 8a2cf08d29ac ("Makefile: introduce FIO_CFLAGS")
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

Revert "Windows: update dobuild.cmd to run the configure/make"

This reverts commit 38c2f9384db8dbd93f59d965d70ab0d3a53343fa.

It's causing issues for CI, revert it for now.

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

Merge branch 'master' of https://github.com/albert-chang0/fio

* 'master' of https://github.com/albert-chang0/fio:
  engines/hdfs: swap fio_hdfsio_init and fio_hdfsio_io_u_init
  Makefile: fix usage of JAVA_HOME environmental variable

Merge branch 'patch-1' of https://github.com/gloit042/fio

* 'patch-1' of https://github.com/gloit042/fio:
  goptions: correct postfix

engines/hdfs: swap fio_hdfsio_init and fio_hdfsio_io_u_init

Commit 08dc3bd50 initialized the io engine before the io_u buffers. The
unintended consequence for the libhdfs io engine was that the HDFS
connection was not established before attempting to open HDFS file
handles. This caused a NPE in the java layer on start-up preventing the
libhdfs io engine from being able to be used.

Signed-off-by: Albert Chang <albert.chang@hitachivantara.com>

Makefile: fix usage of JAVA_HOME environmental variable

Signed-off-by: Albert Chang <albert.chang@hitachivantara.com>

Windows: update dobuild.cmd to run the configure/make

Add a new shell script, _domake.sh which runs under Cygwin. This
is invoked via dobuild.cmd to run the configure and make steps of
the build. In this way, a single command, run from a Developer Command
Prompt, can be used to go from a clean workspace to a final installer
binary.

Signed-off-by: Rebecca Cran <rebecca@bsdio.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

goptions: correct postfix

Signed-off-by: Jiahao Li <gloit042@gmail.com>

fio: fix dynamic engines soname definition

The SONAME of the engines should not match the name of the dependent
library. Otherwise it confuses the dynamic loader into thinking the
dependency is already resolved.

Prefixing the name with fio make more sense here.

Signed-off-by: Yigal Korman <ykorman@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

t/zbd: Fix test target size of test case #48

Option --size was not specified to the fio command of test case #48. It
resulted in write operations to all available sequential write required
zones and relaxed zone locking test condition. Specify the option to
limit test target to 16 zones so that zone locking is tested with
expected condition.

Fixes: 3bd2078bdd1c ("zbd: add test for stressing zone locking")
Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Reviewed-by: Dmitry Fomichev <dmitry.fomichev@wdc.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

zbd: Avoid excessive zone resets

When zbd_reset_zone() is called for a zone repeatedly, reset zone command
is issued multiple times to a single zone even though its status is
empty. This is excessive and meaningless. Especially when zones are reset
at file set up with multiple jobs, zone reset is repeated for each job
and delays fio work load start.

To avoid the repeated zone resets, check the write pointer of the zone
before issuing zone reset command. If the write pointer is at the zone
start, do not reset the zone and just return.

Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Reviewed-by: Dmitry Fomichev <dmitry.fomichev@wdc.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

Merge branch 'o_dsync' of https://github.com/anarazel/fio

* 'o_dsync' of https://github.com/anarazel/fio:
  extend --sync to allow {sync,dsync,0,1}, to support O_DSYNC

extend --sync to allow {sync,dsync,0,1}, to support O_DSYNC

Fixes: https://github.com/axboe/fio/issues/1033

Disallow offload IO mode for engines marked with FIO_NO_OFFLOAD

Previous commits did this for all async engines, this can potentially
break existing job files. There are only certain cases where it fails,
for now at least mark io_uring as one of those.

Reported-by: Jeff Furlong <jeff.furlong@wdc.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

Merge branch 'patch-1' of https://github.com/chengli-rutgers/fio into master

* 'patch-1' of https://github.com/chengli-rutgers/fio:
  getopt_long: avoid variable global initialization

getopt_long: avoid variable global initialization

Issue #1100 shows that an address sanitizer(ASAN) complains about a few
global variables that are initialized globally in the getopt_long.c file.
We look into these variables and found they do not need to be initialized
globally. This patch fixes the issue by cleaning up the global
initialization for these variables.

Fixes https://github.com/axboe/fio/issues/1100
Signed-off-by: Cheng Li <chenglii@cs.rutgers.edu>

Merge branch 'master' of https://github.com/bvanassche/fio into master

* 'master' of https://github.com/bvanassche/fio:
  helper_thread: Increase timer accuracy
  helper_thread: Rework the interval timer code
  Change the return value of two functions from 'void' into 'int'
  helper_thread: Introduce the wait_for_action() function
  helper_thread: Introduce a function for blocking Unix signals
  gettime: Introduce rel_time_since()
  gettime: Simplify get_cycles_per_msec()
  gettime: Introduce fio_get_mono_time()
  configure: Remove the CLOCK_MONOTONIC_PRECISE probe

simplify MB/s, IOPS interactive printout code

Use loops(!) instead of hardcoded if-else pairs.

Make it easier to add DDIR_APPEND.

Signed-off-by: Alexey Dobriyan (SK hynix) <adobriyan@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

flow: use unsigned long for the counter

Same size on 64-bit, it'll be 32-bit on 32-bit archs. But we can't
reliably use 64-bit atomics on 32-bit archs.

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

flow: avoid holes in struct fio_flow

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

Merge branch 'master' of https://github.com/chengli-rutgers/fio into master

* 'master' of https://github.com/chengli-rutgers/fio:
  td_var: avoid arithmetic on a pointer to void cast (#1096)

td_var: avoid arithmetic on a pointer to void cast (#1096)

Issue 1096 shows that a compiler complains an `arithmetic on a pointer to
void` error when `-Wpointer-arith` is turned on in Makefile. Although
there are many violating instances, we only fix the one in `parse.h`
because it is shared across many core source code files. This patch fixes
the `arithmetic on a pointer to void` issue by casting it to a uintptr_t
type first and perform manipulation, and then cast it back to void* type.

Signed-off-by: Cheng Li <chenglii@cs.rutgers.edu>

helper_thread: Increase timer accuracy

The helper_thread uses select() to wait for the next event. select() rounds
up its timeout argument to 1 / HZ. Increase timer accuracy by using a
high-res timer if possible. This patch has been tested by running the
following fio command and by verifying that the next_log and
msec_to_next_event were always above 100 ms:

./fio --debug=helperthread --bandwidth-log --write_bw_log=log --log_avg_msec=1000 --ioengine=null --name=null --size=1G --rw=randwrite --loops=999999 --runtime=60

Signed-off-by: Bart Van Assche <bvanassche@acm.org>

helper_thread: Rework the interval timer code

Instead of adding the number of milliseconds that have elapsed when
a timer fires, add the timer interval. Reduce code duplication by
introducing a helper function that verifies whether or not a timer
has expired.

Signed-off-by: Bart Van Assche <bvanassche@acm.org>

Change the return value of two functions from 'void' into 'int'

This patch does not change any functionality but makes the next patch in
this series easier to read.

Signed-off-by: Bart Van Assche <bvanassche@acm.org>

helper_thread: Introduce the wait_for_action() function

This patch makes helper_thread_main() easier to read.

Signed-off-by: Bart Van Assche <bvanassche@acm.org>

helper_thread: Introduce a function for blocking Unix signals

This patch reworks commit c31092b8ac06 ("Block signals for the helper
thread") but does not change any functionality.

Signed-off-by: Bart Van Assche <bvanassche@acm.org>

gettime: Introduce rel_time_since()

rel_time_since() subtracts two timespecs and returns the result as a
signed integer. Instead of using the implementation from mtime_since(),
use an implementation that only performs one division instead of two.

Signed-off-by: Bart Van Assche <bvanassche@acm.org>

gettime: Simplify get_cycles_per_msec()

Call fio_get_mono_time() directly instead of setting fio_clock_source and
calling __fio_gettime().

Signed-off-by: Bart Van Assche <bvanassche@acm.org>

gettime: Introduce fio_get_mono_time()

Introduce a new function for querying the monotonic clock, something that
is necessary in every context where relative time is measured and where
wall clock time jumps should have no effect. Remove fill_clock_gettime()
since the only contexts where CLOCK_MONOTONIC_RAW are used are
get_cycles_per_msec() and --clocksource=clock_gettime. I think both contexts
should use CLOCK_MONOTONIC instead of its raw variant such that the values
read from the clock are frequency adjusted (a computer clock crystal can
deviate up to 500 ppm from its nominal frequency).

This patch improves accuracy of the helper_thread code on Darwin. Darwin
supports CLOCK_MONOTONIC but not pthread_condattr_setclock(). In other
words, this patch causes the helper thread code to switch from the
real-time clock to the monotonic clock on Darwin.

Signed-off-by: Bart Van Assche <bvanassche@acm.org>

configure: Remove the CLOCK_MONOTONIC_PRECISE probe

CLOCK_MONOTONIC_PRECISE is not used in the fio source code. Hence use the
probe for that constant. The removed test was introduced by commit
67bf982340d9 ("Add configure script").

Signed-off-by: Bart Van Assche <bvanassche@acm.org>

Merge branch 'msys2' of https://github.com/sitsofe/fio into master

* 'msys2' of https://github.com/sitsofe/fio:
  travis: cleanup build script
  configure: cleanup lex/yacc tests
  memlock: avoid type confusion in format string
  Makefile: introduce FIO_CFLAGS
  appveyor: cleanup and add separate install script
  Makefile/ci: Don't pass CFLAGS when linking
  configure/Makefile: add option to generate pdb symbols
  configure: be explicit about "XP" Windows API version
  windows: fix DWORD format string complaints
  windows: fix wrong format strings
  net: coerce the result of htonl before printing
  configure: pass non-null pointer to (v)asprintf

Merge branch 'backend' of https://github.com/bvanassche/fio into master

* 'backend' of https://github.com/bvanassche/fio:
  backend: Use asprintf() instead of strlen() + sprintf()
  backend: Remove two superfluous casts

travis: cleanup build script

- Standardise on spaces for indentation
- Exit on error or using variables before they are defined
- Fix up shellcheck complaints by exporting some variables
- Drop printing of python 2 information as we don't use it
- Use type -p rather than which

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

configure: cleanup lex/yacc tests

- Switch to prebuilt function for finding executable
- Search for the fl library from flex too
- Test lex command output directly

This allows lexing to be more likely to be found (e.g. MSYS2 but sadly
not MSYS2 Mingw-w64).

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

memlock: avoid type confusion in format string

Windows GCC has the followng grumble:

t/memlock.c: In function 'worker':
t/memlock.c:25:26: error: format '%lu' expects argument of type 'long unsigned int', but argument 3 has type 'size_t' {aka 'long long unsigned int'} [-Werror=format=]
   25 |    printf("loop%d: did %lu MiB\n", i+1, size/(1024UL*1024UL));
      |                        ~~^              ~~~~~~~~~~~~~~~~~~~~
      |                          |                  |
      |                          long unsigned int  size_t {aka long long unsigned int}
      |                        %llu

Fix it up by just using the original variable that was already in
megabytes.

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

Makefile: introduce FIO_CFLAGS

Build on the work  done in 4c0b3d98f2d05ddd3f16262c466dcedb22158065
("configure/Makefile: don't override user CFLAGS") by builing up all the
internal flags in a temporary variable.

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

appveyor: cleanup and add separate install script

- Rename PLATFORM environment variable to ARCHITECTURE to avoid clash
  with Appveyor variable
  (https://www.appveyor.com/docs/environment-variables/ )
- Introduce support for using MSYS2
- Perform clang builds that also package PDB symbols
- Drop PACKAGE_ARCH variable since it can be derived
- Switch some lines to just call native binaries directly

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

Makefile/ci: Don't pass CFLAGS when linking

There was an issue where a lld (the linker from LLVM) could fail because
a C compiler flag it didn't understand was passed to it. Fix this by no
longer passing CFLAGS to the linker.

As a result of the above, the travis "cross compiled" 32 bit Linux build
was changed to set LDFLAGS (so the linker can know about the cross
compiling).

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

configure/Makefile: add option to generate pdb symbols

Recent versions of Windows clang and LLVM's lld have the ability to
build Windows PDB symbols for debugging so add an explicit option to
test for support and use it so.

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

configure: be explicit about "XP" Windows API version

With a Cygwin Mingw-w64 environment, when an explicit Windows API
version define isn't set the Windows Server 2003 API is targeted
(https://github.com/mirror/mingw-w64/blob/a25dc933f3d3ed6c8529fb24b7ae393f8792ca69/mingw-w64-headers/include/sdkddkver.h#L166
). This means the XP build was actually misnamed and have never been
fully compatible with plain XP for many years (I suspect such binaries
will likely still work on XP bar trying to use the net ioengine)...

With an MSYS2 Mingw-w64 environment, the default Windows API version is
Windows 7
(https://github.com/msys2/MINGW-packages/blob/e9f271c61a39076ab5de56299f7fd86020c280ae/mingw-w64-headers-git/PKGBUILD#L55
) which breaks an "XP" build because we were were relying on the default
being 2003. Fix this by making the "XP" build explicitly target the
Windows Server 2003 API.

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

windows: fix DWORD format string complaints

Printing DWORD variables causes Windows clang to grumble about types:

os/windows/cpu-affinity.c:263:5: error: format specifies type 'int' but the argument has type 'DWORD' (aka 'unsigned long') [-Werror,-Wformat]
                         GetLastError());
                         ^~~~~~~~~~~~~~

Since a DWORD is defined to be an unsigned 32 bit integer and is
declared to be an unsigned long
(https://docs.microsoft.com/en-us/openspecs/windows_protocols/ms-dtyp/262627d8-3418-4627-9218-4ffe110850b2
) just use long as the format specifier. When printing the cpu mask in
hex on legacy Windows builds, cast to unsigned long long which should be
safe because there won't be more than 64 CPUs with old versions of
Windows.

(As some format strings are longer than the wrapping width, make an
exception and avoid splitting/preserve them to make greping easier)

Suggested-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

windows: fix wrong format strings

Found by Windows clang which mysteriously seemed able to ignore
-Wno-format...

(As some format strings are longer than the wrapping width, make an
exception and avoid splitting/preserve them to make greping easier)

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

net: coerce the result of htonl before printing

Technically htonl() returns a long on Windows
(https://docs.microsoft.com/en-us/windows/win32/api/winsock/nf-winsock-htonl
) which upsets clang.

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

configure: pass non-null pointer to (v)asprintf

Windows clang warns about redefinition of (v)asprintf when targeting
Windows 7 because the configure test failed due to passing NULL when to
those functions when they tag key parameters with the nonnull attribute.

Signed-off-by: Sitsofe Wheeler <sitsofe@yahoo.com>

backend: Use asprintf() instead of strlen() + sprintf()

This patch does not change any functionality but makes exec_string() easier
to read.

Signed-off-by: Bart Van Assche <bvanassche@acm.org>

backend: Remove two superfluous casts

Casting the type of a string constant to (const char *) is not necessary.
These casts were introduced by commit ce4864950d2c ("backend: Logging
exec_{pre|post}run").

Signed-off-by: Bart Van Assche <bvanassche@acm.org>