From f4e401ea363ad974f4e286c7b8fe8be190b7ae5d Mon Sep 17 00:00:00 2001 From: paul luse Date: Tue, 27 Aug 2019 11:25:15 -0400 Subject: [PATCH] module/compress: Clear vol element in comp_bdev struct on vol unload When a vol is unloaded, the vol element in comp_bdev element needs to be NULL'd so that when the comp_bdev destruct entry point is called, we won't try to unload the vol again. fixes issue #928 Signed-off-by: paul luse Change-Id: If267335cbe2234c62351cfc39d33b0ea698ca893 Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/466441 Tested-by: SPDK CI Jenkins Reviewed-by: Karol Latecki Reviewed-by: Jim Harris Reviewed-by: Shuhei Matsumoto Reviewed-by: Tomasz Zawadzki --- module/bdev/compress/vbdev_compress.c | 1 + 1 file changed, 1 insertion(+) diff --git a/module/bdev/compress/vbdev_compress.c b/module/bdev/compress/vbdev_compress.c index cc6978e67..60ad7d4ac 100644 --- a/module/bdev/compress/vbdev_compress.c +++ b/module/bdev/compress/vbdev_compress.c @@ -1194,6 +1194,7 @@ bdev_hotremove_vol_unload_cb(void *cb_arg, int reduce_errno) SPDK_ERRLOG("number %d\n", reduce_errno); } + comp_bdev->vol = NULL; spdk_bdev_unregister(&comp_bdev->comp_bdev, NULL, NULL); }