]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commit
md: Prevent IO hold during accessing to faulty raid5 array
authorAlexey Obitotskiy <aleksey.obitotskiy@intel.com>
Wed, 3 Aug 2016 08:02:56 +0000 (10:02 +0200)
committerShaohua Li <shli@fb.com>
Sat, 6 Aug 2016 05:03:10 +0000 (22:03 -0700)
commit11367799f3d12a5074c4a3c0fa4ea8da2a21a2a4
treef81a5ec15049ec652ead141893f40f5550c47138
parentd9dd26b20cff88b45d861ec786d86b1c9bd2ee60
md: Prevent IO hold during accessing to faulty raid5 array

After array enters in faulty state (e.g. number of failed drives
becomes more then accepted for raid5 level) it sets error flags
(one of this flags is MD_CHANGE_PENDING). For internal metadata
arrays MD_CHANGE_PENDING cleared into md_update_sb, but not for
external metadata arrays. MD_CHANGE_PENDING flag set prevents to
finish all new or non-finished IOs to array and hold them in
pending state. In some cases this can leads to deadlock situation.

For example, we have faulty array (2 of 4 drives failed) and
udev handle array state changes and blkid started (or other
userspace application that used array to read/write) but unable
to finish reads due to IO hold. At the same time we unable to get
exclusive access to array (to stop array in our case) because
another external application still use this array.

Fix makes possible to return IO with errors immediately.
So external application can finish working with array and
give exclusive access to other applications to perform
required management actions with array.

Signed-off-by: Alexey Obitotskiy <aleksey.obitotskiy@intel.com>
Signed-off-by: Shaohua Li <shli@fb.com>
drivers/md/raid5.c