You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
45 lines
1.5 KiB
45 lines
1.5 KiB
4 years ago
|
From 91c97c5432028875db5f8abeddb5cb5f31902001 Mon Sep 17 00:00:00 2001
|
||
|
|
From: Mariusz Tkaczyk <mariusz.tkaczyk@intel.com>
|
||
|
|
Date: Mon, 15 Jul 2019 09:25:35 +0200
|
||
|
|
Subject: [RHEL7.8 PATCH V2 34/47] imsm: close removed drive fd.
|
||
|
|
|
||
|
|
When member drive fails, managemon prepares metadata update and adds
|
||
|
|
the drive to disk_mgmt_list with DISK_REMOVE flag. It fills only
|
||
|
|
minor and major. It is enough to recognize the device later.
|
||
|
|
|
||
|
|
Monitor thread while processing this update will remove the drive from
|
||
|
|
super only if it is a spare. It never removes failed member from
|
||
|
|
disks list. As a result, it still keeps opened descriptor to
|
||
|
|
non-existing device.
|
||
|
|
|
||
|
|
If removed drive is not a spare fill fd in disk_cfg structure
|
||
|
|
(prepared by managemon), monitor will close fd during freeing it.
|
||
|
|
|
||
|
|
Also set this drive fd to -1 in super to avoid double closing because
|
||
|
|
monitor will close the fd (if needed) while replacing removed drive
|
||
|
|
in array.
|
||
|
|
|
||
|
|
Signed-off-by: Mariusz Tkaczyk <mariusz.tkaczyk@intel.com>
|
||
|
|
Signed-off-by: Jes Sorensen <jsorensen@fb.com>
|
||
|
|
---
|
||
|
|
super-intel.c | 3 +++
|
||
|
|
1 file changed, 3 insertions(+)
|
||
|
|
|
||
|
|
diff --git a/super-intel.c b/super-intel.c
|
||
|
|
index d7e8a65..a103a3f 100644
|
||
|
|
--- a/super-intel.c
|
||
|
|
+++ b/super-intel.c
|
||
|
|
@@ -9200,6 +9200,9 @@ static int add_remove_disk_update(struct intel_super *super)
|
||
|
|
remove_disk_super(super,
|
||
|
|
disk_cfg->major,
|
||
|
|
disk_cfg->minor);
|
||
|
|
+ } else {
|
||
|
|
+ disk_cfg->fd = disk->fd;
|
||
|
|
+ disk->fd = -1;
|
||
|
|
}
|
||
|
|
}
|
||
|
|
/* release allocate disk structure */
|
||
|
|
--
|
||
|
|
2.7.5
|
||
|
|
|