]> www.infradead.org Git - users/hch/blktests.git/commitdiff
Revert "nbd/rc: load nbd module explicitly"
authorShin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Thu, 1 Sep 2022 02:32:17 +0000 (11:32 +0900)
committerShin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Fri, 2 Sep 2022 03:37:36 +0000 (12:37 +0900)
This reverts commit 78271b8bb8c939e1d0b9cfa3ea321a4ed06635bd.

Once I thought explicit nbd module load in nbd/rc is required due to the
commit 06a0ba866d90 ("common/rc: avoid module load in _have_driver()").
However, it was not a good solution and _have_driver() was modified
again to load module. Hence, revert explicit nbd module load in nbd/rc.

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

index 32eea45d0426bd6a26cf6c1ab4d86c81bac61889..9c1c15b1a96164f7f1fe9442c4f5f804665e3c1c 100644 (file)
@@ -28,21 +28,17 @@ _have_nbd() {
 }
 
 _have_nbd_netlink() {
-       local ret=0
-
        if ! _have_nbd; then
                return 1
        fi
        if ! _have_program genl-ctrl-list; then
                return 1
        fi
-       modprobe -q nbd
        if ! genl-ctrl-list | grep -q nbd; then
                SKIP_REASONS+=("nbd does not support netlink")
-               ret=1
+               return 1
        fi
-       modprobe -qr nbd
-       return $ret
+       return 0
 }
 
 _wait_for_nbd_connect() {
@@ -66,7 +62,6 @@ _wait_for_nbd_disconnect() {
 }
 
 _start_nbd_server() {
-       modprobe -q nbd
        truncate -s 10G "${TMPDIR}/export"
        cat > "${TMPDIR}/nbd.conf" << EOF
 [generic]
@@ -78,20 +73,17 @@ EOF
 
 _stop_nbd_server() {
        kill -SIGTERM "$(cat "${TMPDIR}/nbd.pid")"
-       modprobe -qr nbd
        rm -f "${TMPDIR}/nbd.pid"
        rm -f "${TMPDIR}/export"
 }
 
 _start_nbd_server_netlink() {
-       modprobe -q nbd
        truncate -s 10G "${TMPDIR}/export"
        nbd-server 8000 "${TMPDIR}/export" >/dev/null 2>&1
 }
 
 _stop_nbd_server_netlink() {
        killall -SIGTERM nbd-server
-       modprobe -qr nbd
        rm -f "${TMPDIR}/export"
 }