From: Ilya Dryomov Date: Mon, 16 Mar 2020 16:16:28 +0000 (+0100) Subject: rbd: don't test rbd_dev->opts in rbd_dev_image_release() X-Git-Tag: v5.10.7~2792^2~2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b8776051529230f76e464d5ffc5d1cf8465576bf;p=platform%2Fkernel%2Flinux-rpi.git rbd: don't test rbd_dev->opts in rbd_dev_image_release() rbd_dev->opts is used to distinguish between the image that is being mapped and a parent. However, because we no longer establish watch for read-only mappings, this test is imprecise and results in unnecessary rbd_unregister_watch() calls. Make it consistent with need_watch in rbd_dev_image_probe(). Fixes: b9ef2b8858a0 ("rbd: don't establish watch for read-only mappings") Signed-off-by: Ilya Dryomov Reviewed-by: Jason Dillaman --- diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c index 7aec8bc..205192a 100644 --- a/drivers/block/rbd.c +++ b/drivers/block/rbd.c @@ -6898,7 +6898,7 @@ static void rbd_print_dne(struct rbd_device *rbd_dev, bool is_snap) static void rbd_dev_image_release(struct rbd_device *rbd_dev) { - if (rbd_dev->opts) + if (!rbd_is_ro(rbd_dev)) rbd_unregister_watch(rbd_dev); rbd_dev_unprobe(rbd_dev);