tests/srp/rc: Avoid triggering "QP creation failed" error messages

Instead of trying to log in to each target port from each initiator
port, only log in to the target ports associated with the RDMA HCA
from which the log in request is sent. This avoids that the SRP
initiator driver logs the following error message when e.g.
attempting to log in to interface lo from an Ethernet interface:
"QP creation failed".

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

tests/srp/rc: Consider all network interfaces

Instead of giving up if a login attempt fails, try all network interfaces.
This patch ensures that login succeeds if the first network interface that
is encountered is down and another network interface is up.

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

tests/srp/rc: Simplify log_in()

Since the 'sysfsdir' variable is identical to the 'd' variable, leave
out 'sysfsdir' and use 'd' instead. This patch does not change any
functionality.

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

tests/srp/rc: Make the output of srp_single_login() less confusing

Displaying $PWD is not useful inside srp_single_login(). Hence leave
$PWD out.

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

tests/srp/rc: Make debugging do_rdma_cm_login() easier

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

nvmeof-mp, srp: Check multipathd version

The srp tests reliably report data corruption with multipathd versions
before 0.7.0 due to a multipathd bug. Hence verify that the multipathd
version is at least 0.7.0.

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

loop/004: Need to wait for drop caches if block_size is changed

When i test blktests, loop/004 will be fail. The Key test steps
are as follows:
1. losetup -f --show  /dev/sda
2. src/loblksize /dev/loop0 4096

step 1 will create /dev/loop0. after that, daemon systemd-udevd
will visit and close /dev/loop0 who will add and delete
i_mapping->nrpages.
PS: the operation of systemd-udevd is in the background.

step 2 will set /dev/loop0 block size, linux kernel function
loop_set_block_size has been changed since commit 5db470e229e2
("loop: drop caches if offset or block_size are changed")
+ if (lo->lo_queue->limits.logical_block_size != arg &&
+     lo->lo_device->bd_inode->i_mapping->nrpages) {
+               err = -EAGAIN;
+               pr_warn("%s: loop%d (%s) has still dirty pages\n",
+                       __func__, lo->lo_number, lo->lo_file_name,
+                       lo->lo_device->bd_inode->i_mapping->nrpages);
+               goto out_unfreeze;
+       }
--->maybe systemd-udevd just visited /dev/loop0, this leads to failure

Add a step between 1 and 2.

Signed-off-by: zhengbin <zhengbin13@huawei.com>

zbd/004: Add zone condition "Closed" for sequential write required zones

The test case zbd/004 executes write operations for two sequential write
required zones across the zone boundary between them. After the write
operation, the second zone has non-zero write pointer. At that status,
the zone can have not only "Implicit Open" condition but also "Closed"
condition based on zone status management logic of the block zoned device.

Add "Closed" condition to the zone condition check logic in zbd/004. Add
ZONE_COND_CLOSED constant definition in zbd/rc.

Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>

block/024: Increase I/O time

The total time spent on write I/Os for the 3 dd executions will be at most
1500 * 3 * 0.5ms = 2250ms, that is, 2.25s. Due to system overhead and
timer triggers, a total write I/O time larger than this value is generally
reported through I/O stat. However, for a system with very low overhead
and/or very precise timers, the reported value can be under 2.5s, leading
to a rounded value equal to 2s, and not the 3s expected.

To avoid the test to fail due to this problem, increase the number of write
I/Os executed so that the total exact I/O time exceeds 2.5s, leading to a
rounded value equal to 3s. The change increases the number of I/Os of the
second and third dd calls to 1800, leading to an exact write I/O time of
(1500 + 1800 * 2) * 0.5 ms = 2.55s.

Signed-off-by: Masato Suzuki <masato.suzuki@wdc.com>

travis: streamline config

- Reorder the testing matrix to more or less the order of importance.
- Don't install libc and libstdc++ explicitly; it's not necessary and
  for some reason it installs a bunch of other packages.
- We only test on Linux, so get rid of the conditional.
- Don't error out if apt update fails, since that seems to happen every
  now and then because some repository changed keys.

Signed-off-by: Omar Sandoval <osandov@fb.com>

travis: run make check

We want to run shellcheck and the other sanity checks on all pushes.
Install the latest stable version of shellcheck, as old versions are
known to have false positives.

Signed-off-by: Omar Sandoval <osandov@fb.com>

CONTRIBUTING: add my email address

Most people cc me on blktests patches, but let's document it, too, since
I'm not always watching the block mailing list.

Signed-off-by: Omar Sandoval <osandov@fb.com>

src: check if header exists with -E

The current HAVE_C_HEADER function creates a -.o file. We don't need to
compile, just run the C preprocessor, so use -E instead of -c.

Signed-off-by: Omar Sandoval <osandov@fb.com>

README: add Travis status badge

Now that Travis is set up for the repository, let's make it easily
discoverable.

Signed-off-by: Omar Sandoval <osandov@fb.com>

Merge pull request #44 from bvanassche/master

Multiple srp and nvmeof-mp patches

tests/srp/013: Provide detailed information about the progress of this test

This makes it easier to determine what went wrong if this test fails.

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

tests/srp/rc: Increase maximum request size

A new feature for the SRP initiator and target drivers in kernel v5.0-rc1
is that both the initiator and target drivers support immediate data.
Since the default maximum immediate data size is 8 KB, increase the target
maximum request size such that the SRP initiator can send write requests
with up to 8 KB of immediate data.

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

tests/nvmeof-mp/rc: Fix simulate_network_failure_loop()

Avoid that the following is reported:

+tests/nvmeof-mp/rc: line 118: /dev/nvme0n1/device/reset_controller: Not a directory

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

tests/nvmeof-mp/rc: Remove sg_reset dependency

Since sg_reset is not used by any of the nvmeof-mp tests, do not require
presence of that executable.

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

common/multipath-over-rdma: Skip teardown if $debug has been set

A convenient way to debug the srp and nvmeof-mp tests scripts is to set
the debug variable in common/multipath-over-rdma. Examining what went
wrong during a test is only possible if teardown() is skipped after a
test failure. Make sure that this is the case.

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

common/multipath-over-rdma: Fix held_by()

Use the proper shell operator to remove the /dev/ prefix from a device
path.

Fixes: a7f15d1141e4 ("Add tests for the SRP initiator and target drivers")
Signed-off-by: Bart Van Assche <bvanassche@acm.org>

Unbreak the continuous integration build

The Travis environment in which the continous integration build is run
does not have the <linux/blkzoned.h> header file. Make sure that the
zbdioctl.c source file builds fine without that header file.

Cc: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Cc: Chaitanya Kulkarni <Chaitanya.Kulkarni@wdc.com>
Fixes: d42cc3f85e0d ("src: Introduce zbdioctl program")
Signed-off-by: Bart Van Assche <bvanassche@acm.org>

zbdioctl: copy ioctl definitions from kernel

blkzoned.h may not have the recent BLKGETZONESZ or BLKGETNRZONES ioctls,
so just copy the definition.

Signed-off-by: Omar Sandoval <osandov@fb.com>

zbd/005: Test write ordering

Run a high queue depth direct sequential write fio job to check that
write requests are not being reordered when the deadline scheduler is
used. This test allowed to catch a bug fixed with commit
80e02039721 "block: mq-deadline: Fix write completion handling".

Signed-off-by: Masato Suzuki <masato.suzuki@wdc.com>

zbd/004: Check write split accross sequential zones

Check that write operations spanning a zone boundary are correctly
processed as 2 different write operations each fully within a single
zone.

Signed-off-by: Masato Suzuki <masato.suzuki@wdc.com>

zbd/003: Test sequential zones reset

Test zone reset operation to make sure that the BLKRESETZONE ioctl call
works as expected but also that the zone sector remapping that may be
done for logical devices (partitions or dm-linear devices) is correct.

Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Signed-off-by: Masato Suzuki <masato.suzuki@wdc.com>

zbd/002: report zone test

Get a report for all zones and confirm that the reported values are
valid and consistent with regard to the device capacity and zone size.

Signed-off-by: Masato Suzuki <masato.suzuki@wdc.com>

zbd/001: sysfs and ioctl consistency test

Obtain a zoned block device attributes from sysfs and using ioctl calls
and confirm the consistency of the values.

Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Signed-off-by: Masato Suzuki <masato.suzuki@wdc.com>

tests: Introduce zbd test group

The zoned block device (zbd) test group is used to gather all tests
specific to zoned block devices (null_blk device with zoned mode enabled,
SMR disks, dm-linear on top of zoned devices, etc). Execution of this group
requires that the kernel be compiled with the block layer
CONFIG_BLK_DEV_ZONED option enabled and also requires the null_blk driver
to have zoned mode support (added in kernel 4.19).

This group rc script implements _fallback_null_blk_zoned() helper function
which initailize a null_blk device with zoned mode. Each of the zbd group
test cases calls it in fallback_device() function. This allows the zbd
group test cases fallback to the null_blk device even if the TEST_DEVS
is empty. With this, all tests scripts can be written by only defining
the test_device() function while allowing operation on both null_blk and
user specified devices.

Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Signed-off-by: Masato Suzuki <masato.suzuki@wdc.com>

src: Introduce zbdioctl program

zbdioctl implements calls to zoned block devices ioctls that are not
supported currently by sys-utils blkzone utility, namely BLKGETZONESZ
and BLKGETNRZONES.

Reviewed-by: Chaitanya Kulkarni <Chaitanya.Kulkarni@wdc.com>
Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>

check: Introduce fallback_device() and cleanup_fallback_device()

These optional functions can be defined by a test case script. When
defined and TEST_DEVS is empty, the fallback_device() is executed before
runing the test case. The fallback_device() function intializes a virtual
device to run the test case and return the device to be set in TEST_DEVS.
After running the test case, cleanup_fallback_device() is executed to
clean up the device.

This feature allows to run test cases with test_device() function even if
TEST_DEVS is not set in the config, using virtaul devices such as null_blk.

Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>

block: Whitelist tests supporting zoned block devices

Define CAN_BE_ZONED=1 in block/005, block/006, block/010, block/011,
block/016, block/017, block/020, block/021 and block/023 as all these
tests should execute without any problem against null_blk with zoned
mode enabled or zoned block devices specified in TEST_DEVS.

Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>

block/004: Adjust fio conditions for zoned block devices

For a random write pattern to a zoned block device, fio requires --direct=1
and --zonemode=zbd options as well as deadline I/O scheduler to be
specified. Specify these options and set the I/O scheduler if the target
device is a zoned block device. Before doing that, also make sure that the
deadline scheduler is available and that fio supports the zbd zone mode.
Set CAN_BE_ZONED flag to run this test case for zoned block devices.

Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>

common: Introduce _have_fio_zbd_zonemode() helper function

Fio zbd zone mode is necessary for zoned block devices. Introduce the
helper function _have_fio_zbd_zonemode() to check that the installed
fio version supports the option --zonemode=zbd.

Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>

config: Introduce RUN_ZONED_TESTS variable and CAN_BE_ZONED flag

To allow running tests using a null_blk device with the zoned mode
disabled (current setup) as well as enabled, introduce the config
the RUN_ZONED_TESTS config variable and the per-test flag CAN_BE_ZONED.

RUN_ZONED_TESTS=1 indicates that tests run against null_blk will be
executed twice, first with null_blk as a regular block device
(RUN_FOR_ZONED=0) and a second time with null_blk set as a zoned block
device (RUN_FOR_ZONED=1). This applies only to tests cases that have the
variable CAN_BE_ZONED set to 1, indicating that the test case applies to
zoned block devices. If CAN_BE_ZONED is not defined by a test case, the
test is executed only with the regular null_blk device.

_init_null_blk is modified to prepare null_blk as a zoned blocked device
if RUN_FOR_ZONED is set and as a regular block device otherwise. To avoid
"modprobe -r null_blk" failures, rmdir calls on all sysfs nullbX
directories is added.

When a zoned block device is specified in TEST_DEVS, failures of test
cases that do not set CAN_BE_ZONED are avoided by automatically skipping
the test. The new helper function _test_dev_is_zoned() is introduced to
implement this.

The use of the RUN_ZONED_TESTS variable requires that the kernel be
compiled with CONFIG_BLK_DEV_ZONED enabled.

Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>

loop: Add test for changing capacity when filesystem is mounted

Add test for changing capacity of a loop device when a filesystem with
non-default block size is mounted on it. This is a regression test for
"blockdev: Fix livelocks on loop device".

Signed-off-by: Jan Kara <jack@suse.cz>
Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
[Omar: mount under $TMPDIR]
Signed-off-by: Omar Sandoval <osandov@fb.com>

block: 028: block integrity funtion test

Use scsi_debug's dif/dix to cover block layer's integrity function
test, then it can serve as block integrity regeression test.

Signed-off-by: Ming Lei <ming.lei@redhat.com>
[Omar: fix some shellcheck errors]
Signed-off-by: Omar Sandoval <osandov@fb.com>

README: update required bash version

Bash 5 was recently released, so the bash 4 requirement is misleading.
Additionally, we also depend on some features in bash 4.2.

Signed-off-by: Omar Sandoval <osandov@fb.com>

cgroup: test if CGROUP2_DIR is set with -v instead of -n

common/multipath-over-rdma does set -u, so -n "$CGROUP2_DIR" fails with
an unbound variable error. Instead, use -v to test if the variable was
set.

Signed-off-by: Omar Sandoval <osandov@fb.com>

Merge pull request #43 from tytso/master

src/sg/syzkaller1.c: fix portability problem for syscall(__NR_mmap, ...)

block/008: avoid race with CPU offline and sched_setaffinity() in fio

This is a better fix for 4a2933fd0386 ("block/008: fix race between CPU
offline and fio startup"). Closes #41.

Reported-by: Signed-off-by: Ivan Mironov <mironov.ivan@gmail.com>
Signed-off-by: Omar Sandoval <osandov@fb.com>

src/sg/syzkaller1.c: fix portability problem for syscall(__NR_mmap, ...)

How mmap is mapped to a raw system call varies across different
architectures.  On some architectures (such as 32-bit ARM), __NR_mmap
may not exist at all; glibc will use __NR_mmap2 to implement mmap(2).
Syzkaller is using mmap() as a non-portable version of malloc(3), so
it should be safe to use the glibc's mmap wrapper instead of trying to
directly call the system call.

Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Reviewed-by: Bart Van Assche <bvanassche@acm.org>

blktests: add Ming Lei's scsi-stress-remove

This test exposed a race condiiton when shutting down a request_queue
with active IO against it and blkg association for the IOs [1]. The
issue ended up being that while the request_queue will just start
failing requests, blkg destruction sets the q->root_blkg to %NULL. This
caused a NPE. This was fixed in [2].

So to help prevent this from happening again, integrate Ming's test into
blktests so that it can more easily be ran. Here I've ported it to fit
better into the blktests framework.

[1] https://lore.kernel.org/lkml/20181205171039.73066-1-dennis@kernel.org/
[2] https://lore.kernel.org/lkml/20181211230308.66276-1-dennis@kernel.org/

Signed-off-by: Dennis Zhou <dennis@kernel.org>
Cc: Ming Lei <ming.lei@redhat.com>

blktests: split out cgroup2 controller and file check

This is a prep patch for a new test that will race blkg association and
request_queue cleanup. As blkg association is a underlying cgroup io
controller feature, we need the ability to check if the controller is
available.

Signed-off-by: Dennis Zhou <dennis@kernel.org>

blktests: add cgroup2 infrastructure

In order to test io.latency and other cgroup related things we need some
supporting helpers to setup and tear down cgroup2.  This adds support
for checking that we can even configure cgroup2 things, set them up if
need be, and then add the cleanup stuff to the main cleanup function so
everything is always in a clean state.

Signed-off-by: Josef Bacik <josef@toxicpanda.com>
[Omar: split into separate file, fix shellcheck errors, rework
 cleanup/exit]
Signed-off-by: Omar Sandoval <osandov@fb.com>

blktests: Fix compilation warning

Fix strncpy length to "sizeof(buf) - 1" to avoid the compiler warning:
cc  -O2 -Wall -Wshadow  -o sg/syzkaller1 sg/syzkaller1.c
sg/syzkaller1.c: In function ‘syz_open_dev.constprop’:
sg/syzkaller1.c:204:16: warning: ‘strncpy’ specified bound 1024 equals
destination size [-Wstringop-truncation]
     NONFAILING(strncpy(buf, (char*)a0, sizeof(buf)));
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
sg/syzkaller1.c:143:7: note: in definition of macro ‘NONFAILING’
       __VA_ARGS__;                                                     \
       ^~~~~~~~~~~

Since the last character of buf is forced set to 0, there is no
functional change introduced by this patch.

Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>

scsi/006: allow changing cache_type to fail

Some devices don't support all cache types. Allow setting the cache type
to fail with EINVAL. On success, make sure it was changed to the desired
value.

Signed-off-by: Omar Sandoval <osandov@fb.com>

Merge pull request #39 from bvanassche/master

Two improvements for the srp tests

common/multipath-over-rdma: Retry unloading rdma_rxe if necessary

If any context, e.g. queued work, holds a reference on an rdma_rxe
device then it can happen that the first unload attempt fails. Try
several times to unload that kernel module if the first unload
attempt fails.

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

srp and nvmeof-mp: Check whether legacy dm is supported

Kernel commit 6a23e05c2fe3 ("dm: remove legacy request-based IO path")
retained dm-mq but removed the legacy dm code. Skip those tests that
require the legacy dm code.

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

Change _have_module -> _have_modules

This way, we can report all missing modules at once.

Signed-off-by: Omar Sandoval <osandov@fb.com>

tests/srp: add ib_uverbs to list of modules

While we're here, sort the list.

Signed-off-by: Omar Sandoval <osandov@fb.com>

block/025: test discard sector alignement and sector size overflow

This test covers the following two issues:

1) discard sector need to be aligned with logical block size

2) make sure 'sector_t' instead of 'unsigned int' is used when comparing
with discard sector size

Signed-off-by: Ming Lei <ming.lei@redhat.com>
[Omar: reword description]
Signed-off-by: Omar Sandoval <osandov@fb.com>

Add use of logger so that syslog files show when each test starts

Signed-off-by: Theodore Ts'o <tytso@mit.edu>

Check permissions of output files in make check

Output files should not be executable.

Signed-off-by: Omar Sandoval <osandov@fb.com>

Fix permissions of golden output files

Four of the golden outputs had execute bits set. However, like all the
other .out-files, their permission should be restricted to 644.

Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Michael Moese <mmoese@suse.de>

Merge pull request #37 from bvanassche/master

Fix 32-bit build and add a Travis configuration file

makefile: Add install rule

Add rule to install to a target directory, /usr/local/blktests by
default.

Signed-off-by: Gwendal Grignou <gwendal@chromium.org>

.travis.yml: Add to repository

Automatically build binaries upon 'git push' with -Werror to avoid that
new build errors or warnings get introduced.

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

src/sg/syzkaller1.c: Fix a 32-bit compiler warning

Avoid that clang reports the following warning when building in 32-bit mode:

sg/syzkaller1.c:405:34: error: implicit conversion from 'unsigned long long' to
      'uintptr_t' (aka 'unsigned int') changes value from 18446744073709551615
      to 4294967295 [-Werror,-Wconstant-conversion]
                         0x32ul, 0xfffffffffffffffful, 0x0ul, 0, 0, 0);
                                 ^~~~~~~~~~~~~~~~~~~~

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

src/discontiguous-io: Do not shadow variables

Avoid using variables in an inner scope with the same name as variables in
an outer scope. Enable the -Wshadow compiler flag for C and C++ source
files such that in the future the compiler will complain about shadowing.

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

Fix build failure for discontiguous-io on 32-bit platforms

Avoid that building with a 32-bit compiler fails as follows:
discontiguous-io.cpp:294:34: error: no matching function for call to 'min(long unsigned int, size_t)'
         std::min(4ul, len - i * 4));
                                  ^
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
[bvanassche: elaborated commit message]
Signed-off-by: Bart Van Assche <bvanassche@acm.org>

loop/006: Add test for oops during backing file verification

Add regression test for patch "block/loop: Use global lock for ioctl()
operation." where we can oops while traversing list of loop devices
backing newly created device.

Signed-off-by: Jan Kara <jack@suse.cz>
[Omar: rename to 006, change description]
Signed-off-by: Omar Sandoval <osandov@fb.com>

blktest: remove instances of null_blk queue_mode=1

This is no longer supported in recent kernels, get rid of any testing of
queue_mode=1. queue_mode=1 tested the legacy IO path, which is going
away completely. As such, there's no point in doing anymore testing with
it.

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

Convert license headers to SPDX tags

This gets rid of the noisy boilerplate at the beginning of every file.

Signed-off-by: Omar Sandoval <osandov@fb.com>

Add NVMeOF dm-mpath tests

Add a series of tests for the NVMeOF drivers on top of the dm-mpath
driver. These tests are similar to the tests under tests/srp. Both
tests use the dm-mpath driver for multipath and the loopback
functionality of the rdma_rxe driver. The only difference is that the
nvmeof-mp tests use the NVMeOF initiator and target drivers instead
of the SRP initiator and target drivers.

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

common/multipath-over-rdma: Log fio exit code

Log the fio exit code to make it easier to analyze test failures.

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

common/multipath-over-rdma: Improve robustness

Detect all test setup failures and report any failures.

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

common/multipath-over-rdma: Make teardown() shut down the client

This ensures that no multipath definitions created during a test remain
after a test has finished.

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

common/multipath-over-rdma: Fix is_qinp_def()

Make is_qinp_def() recognize the queue_if_no_path keyword correctly.

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

tests/srp/rc, common/multipath-over-rdma: Introduce setup_rdma()

Move the code that will be reused by the NVMeOF tests into a new
function. This patch does not change any functionality.

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

tests/srp: Move reusable code into common/multipath-over-rdma

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

tests/srp/rc: Move the multipathd reconfiguration command into the get_bdev() loop

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

tests/srp/rc: Rework get_bdev()

Replace the function get_bdev_uid() by the function get_bdev_path().
Call that new function every iteration instead of only once.

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

tests/srp: Remove /etc/multipath.conf after a test has finished

Instead of removing /etc/multipath.conf before a test starts, remove it
after a test has finished. This change is needed to let the nvmeof-mp
tests run after the srp tests have been run.

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

tests/srp: Remove unused code

Remove all code that depends on $nvme != "" and also the $nvme variable
itself.

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

tests/srp/012: Do not fail if not all schedulers are available

Avoid that test 012 returns exit code 1 (failure) even if fio succeeded
due to an I/O scheduler not being available.

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

block: add a test for iostats time with fast I/Os

This is similar to block/018, but all of the I/Os take less than a
jiffy, which tests b57e99b4b8b0 ("block: use nanosecond resolution for
iostat").

Signed-off-by: Omar Sandoval <osandov@fb.com>

block/018: use _init_null_blk

Signed-off-by: Omar Sandoval <osandov@fb.com>

nvme: add NVMeOF list-subsys test for file-ns

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme: add NVMeOF ns-rescan test for file-ns

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme: add NVMeOF ns-desc test for file-ns

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme: add NVMeOF effects-log test for file-ns

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme: add NVMeOF smart-log test for file-ns

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme: add NVMeOF smart-log test for bdev-ns

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme: add NVMeOF reset test for file-ns

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme: add NVMeOF list test for file-ns

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme: add NVMeOF DSM discard test for file-ns

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

nvme: add NVMeOF DSM discard test for bdev-ns

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

block: add smoke test for queue_mode=0

Linux kernel commit ca4b2a011948 ("null_blk: add zone support") broke
null_blk queue_mode=0. None of the existing tests use bio mode, so add a
test which does a very basic test of all modes.

Signed-off-by: Omar Sandoval <osandov@fb.com>

scsi: add test that toggles cache_type

This is Jens' easy reproducer for
https://www.spinics.net/lists/linux-block/msg29942.html.

Signed-off-by: Omar Sandoval <osandov@fb.com>

blktests: remove unused null_blk parameter for _init_null_blk in block/016

Reviewed-by: Gioh Kim <gi-oh.kim@profitbricks.com>
Signed-off-by: Yi Zhang <yi.zhang@redhat.com>

Delete nvme/001

Johannes pointed out that the format of this tracepoint is going to
change in 4.19, which will make this test fail. Now that we have a slew
of real NVMe tests, we can do without this one. We can always add it
back if we decide it's useful later.

Signed-off-by: Omar Sandoval <osandov@fb.com>

blktests: add NVMeOF many file-ns creation test

This patch adds a new testcase to exercise a large number of ns
creation and deletion along with discovery command.

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

blktests: add NVMeOF many bdev-ns creation test

This patch adds a new testcase to exercise a large number of ns
creation and deletion along with discovery command.

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

blktests: add NVMeOF flush test for file-ns

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

blktests: add NVMeOF flush test for bdev-ns

Signed-off-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

check: add --output parameter

In case we want to store the results somewhere other than ./results.
Based on a patch from Hannes Reinecke.

Signed-off-by: Omar Sandoval <osandov@fb.com>

src: add -std=c++11 to C++ build flags

Certain distributions like to ship ancient relics for compilers.
Fixes #31.

Signed-off-by: Omar Sandoval <osandov@fb.com>

common/rc: search harder for pci device

Some multipathed NVMe devices have a bit of a different sysfs layout
then we've expoected them to have in the first place.

Try harder searching for a pci device if we encounter such a device.

This fixes issue #26.

Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>