32 lines
1.0 KiB
Diff
32 lines
1.0 KiB
Diff
|
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
||
|
|
From: Benjamin Marzinski <bmarzins@redhat.com>
|
||
|
|
Date: Thu, 18 May 2017 10:11:07 -0500
|
||
|
|
Subject: [PATCH] libmultipath: add comment about resuming
|
||
|
|
|
||
|
|
The reason for the second resume in my commit "libmultipath: fix
|
||
|
|
suspended devs from failed reloads" is not obvious from the multipath
|
||
|
|
code, so add a comment.
|
||
|
|
|
||
|
|
Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
|
||
|
|
---
|
||
|
|
libmultipath/devmapper.c | 3 +++
|
||
|
|
1 file changed, 3 insertions(+)
|
||
|
|
|
||
|
|
diff --git a/libmultipath/devmapper.c b/libmultipath/devmapper.c
|
||
|
|
index 69b634b..ee83e0f 100644
|
||
|
|
--- a/libmultipath/devmapper.c
|
||
|
|
+++ b/libmultipath/devmapper.c
|
||
|
|
@@ -399,6 +399,9 @@ int dm_addmap_reload(struct multipath *mpp, char *params, int flush)
|
||
|
|
if (r)
|
||
|
|
return r;
|
||
|
|
|
||
|
|
+ /* If the resume failed, dm will leave the device suspended, and
|
||
|
|
+ * drop the new table, so doing a second resume will try using
|
||
|
|
+ * the original table */
|
||
|
|
if (dm_is_suspended(mpp->alias))
|
||
|
|
dm_simplecmd(DM_DEVICE_RESUME, mpp->alias, !flush, 1,
|
||
|
|
udev_flags, 0);
|
||
|
|
--
|
||
|
|
2.7.4
|
||
|
|
|