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>

Add tests for the SRP initiator and target drivers

This patch adds the following tests:
001: Create and remove LUNs
002: File I/O on top of multipath concurrently with logout and login (mq)
003: File I/O on top of multipath concurrently with logout and login (sq)
004: File I/O on top of multipath concurrently with logout and login (sq-on-mq)
005: Direct I/O with large transfer sizes, cmd_sg_entries=255 and bs=4M
006: Direct I/O with large transfer sizes, cmd_sg_entries=255 and bs=8M
007: Direct I/O with large transfer sizes, cmd_sg_entries=1 and bs=4M
008: Direct I/O with large transfer sizes, cmd_sg_entries=1 and bs=8M
009: Buffered I/O with large transfer sizes, cmd_sg_entries=255 and bs=4M
010: Buffered I/O with large transfer sizes, cmd_sg_entries=255 and bs=8M
011: Block I/O on top of multipath concurrently with logout and login
012: dm-mpath on top of multiple I/O schedulers
013: Direct I/O using a discontiguous buffer

Other changes in this patch are:
- Add function _have_kernel_option to common/rc.
- Add file tests/srp/rc with shell functions that are used by multiple SRP
  tests.

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>

Add the discontiguous-io test program

This test program will be used by one of the srp tests.

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>

src/Makefile: Move -Wall from the C compilation rule to the $(CFLAGS) variable

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>

src/Makefile: Introduce the C_TARGETS variable

This patch does not change any functionality but will make it easier
to add support for building C++ code.

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>

check: Make $FULL an absolute path

This makes it safe to use $FULL in shell code that modifies (and
restores) the working directory.

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>

dir-locals.el: Add c-mode settings

Add c-mode settings for the files in the src directory. Additionally,
make indent-tabs-mode global such that it also applies to the
.dir-locals.el file itself.

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>

block/008: fix race between CPU offline and fio startup

If fio is still setting up when we start hotplugging CPUs, fio can fail
with "clock setaffinity failed: Invalid argument". That comes from
calling sched_setaffinity() on an offline CPU. We can make this much
less likely to happen by sleeping before we start hotplugging.

Fixes #28.

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

Merge pull request #27 from gurugio/master

Add null_blk helpers and settle udev before unloading null_blk

dir-locals.el: fix indenation with Emacs

A recent update of Emacs broke .dir-locals.el's shell indentation
setting.

Also set 'sh-indentation' not just 'sh-basic-offset' for this.

Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>
Reviewed-by: Bart Van Assche <bart.vanassche@wdc.com>

block: apply null_blk helpers

use null_blk helpers to load/unload null_blk module

Signed-off-by: Gioh Kim <gi-oh.kim@profitbricks.com>

common: make helpers for null_blk module

Common helpers for loading/unloading null_blk module.
It settles udev before loading/unloading.

Signed-off-by: Gioh Kim <gi-oh.kim@profitbricks.com>

block: settle udev before removing null_blk module

prevent "modprobe: FATAL: Module null_blk is in use." errors.

Signed-off-by: Gioh Kim <gi-oh.kim@profitbricks.com>

blktests: add timeout smoketest for !mq

Identical to block/014, except we test the legacy IO path instead
of blk-mq.

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

block/014: make it explicit that we're testing blk-mq timeouts

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

scsi: regression test for SCSI device blacklisting

SCSI device blacklisting seems to be a tricky subject, with lots of
potential for messing up the selection algorithm. This adds a test for
catching regressions here.

Signed-off-by: Hannes Reinecke <hare@suse.com>
[Omar: updated to use improved scsi_debug helpers]
Signed-off-by: Omar Sandoval <osandov@fb.com>

check: fix TEST_DEVS being overwritten

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

check: allow setting config options as environment variables

Closes #25.

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

new: sort group names

And replace the unnecessary bash loop with more find flags.

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

Make group/rc and common/rc sources explicit and reenable SC2034

SC2034 (unused variable) is a useful warning, but we disabled it because
we set a bunch of global variables as test metadata. However, this is
easy to work around: as Bart demonstrated, an echo "$VAR" > /dev/null
does the trick.

However, we don't want to copy-and-paste this everywhere, so we need to
source something everywhere. Bart's idea was to put this in
common/shellcheck and source that everywhere. Sourcing a file just to
appease the linter is silly, though, so instead, this adds explicit
sources of tests/*/rc to each test, which in turn sources common/rc,
which in turn sources common/shellcheck.

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

Rename tests/*/group to tests/*/rc

The following change is going to add an explicit source of these to the
tests themselves, and this naming makes more sense in that context.

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

check: Make TMPDIR absolute

This avoids that ${TMPDIR} gets misinterpreted if a test script
modifies the working directory.

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>

make check: Also check tests/*/group files

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>

tests/meta/group: Fix shellcheck complaints

Avoid that shellcheck complains about unused global variables.
Avoid that shellcheck complains about using cat instead of
redirection.

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>