From 744e75ad9163cbee9c1f998c83473007467ae491 Mon Sep 17 00:00:00 2001
From: Bart Van Assche <bvanassche@acm.org>
Date: Thu, 21 Mar 2024 14:25:49 -0700
Subject: [PATCH] block/035: Test shared queue fairness

Test whether both requests queues process I/O if the tag set is shared
and if the completion times of the two request queues differ
significantly.

Signed-off-by: Bart Van Assche <bvanassche@acm.org>
---
 tests/block/035     | 82 +++++++++++++++++++++++++++++++++++++++++++++
 tests/block/035.out |  2 ++
 2 files changed, 84 insertions(+)
 create mode 100755 tests/block/035
 create mode 100644 tests/block/035.out

diff --git a/tests/block/035 b/tests/block/035
new file mode 100755
index 00000000..76696338
--- /dev/null
+++ b/tests/block/035
@@ -0,0 +1,82 @@
+#!/bin/bash
+# SPDX-License-Identifier: GPL-3.0+
+# Copyright 2024 Google LLC
+#
+# Test fairness if hardware queues share a tag set across request queues.
+
+. tests/block/rc
+. common/null_blk
+
+DESCRIPTION="shared tag set fairness"
+TIMED=1
+
+requires() {
+	_have_fio
+	_have_module null_blk
+	_have_module_param null_blk shared_tags
+}
+
+test() {
+	local runtime=${TIMEOUT:-30}
+
+	echo "Running ${TEST_NAME}"
+
+	local nullb_params=(
+		blocking=1
+		irqmode=2
+		nr_devices=0
+		shared_tags=1
+	)
+	if ! _init_null_blk "${nullb_params[@]}"; then
+		echo "Loading null_blk failed"
+		return 1
+	fi
+	local nullb_params=(
+		completion_nsec=$((10**6)) # 1 ms
+		hw_queue_depth=64
+		memory_backed=1
+		size=1                     # MiB
+		submit_queues=1
+		power=1
+	)
+	if ! _configure_null_blk nullb0 "${nullb_params[@]}"; then
+		echo "Configuring null_blk failed (1/2)"
+		return 1
+	fi
+	local nullb_params=(
+		completion_nsec=$((10**8)) # 100 ms
+		hw_queue_depth=64
+		memory_backed=1
+		size=1                     # MiB
+		submit_queues=1
+		power=1
+	)
+	if ! _configure_null_blk nullb1 "${nullb_params[@]}"; then
+		echo "Configuring null_blk failed (2/2)"
+		return 1
+	fi
+	local fio_output=${RESULTS_DIR}/block/fio-output-block-035.txt
+	fio --rw=randwrite --ioengine=io_uring --iodepth=64 \
+		--direct=1 --runtime="${runtime}" --time_based=1 \
+		--significant_figures=9 \
+		--name=nullb0 --filename=/dev/nullb0 \
+		--name=nullb1 --filename=/dev/nullb1 \
+		--output="${fio_output}" \
+		>>"$FULL" 2>&1
+	local fio_status=$?
+	rmdir /sys/kernel/config/nullb/nullb*
+	_exit_null_blk
+	if [ $fio_status != 0 ]; then
+		echo "Failed (fio status = $fio_status)"
+		return
+	fi
+	local iops1 iops2 rest
+	read -r iops1 iops2 rest \
+	     <<<"$(sed -n 's/.*IOPS=\([0-9]*\).*/\1/p' <"${fio_output}" | xargs)"
+	if [ -z "$iops1" ] || [ -z "$iops2" ] ||
+			[ "$iops1" -lt "$iops2" ]; then
+		echo "Error: IOPS too low ($iops1; $iops2)"
+		return 1
+	fi
+	echo "Passed"
+}
diff --git a/tests/block/035.out b/tests/block/035.out
new file mode 100644
index 00000000..6020deac
--- /dev/null
+++ b/tests/block/035.out
@@ -0,0 +1,2 @@
+Running block/035
+Passed