On 07/27/2016 01:09 PM, Mike Snitzer wrote:
In addition to the above patch, please apply this patch and retest your
4.7 kernel:
diff --git a/drivers/md/dm-mpath.c b/drivers/md/dm-mpath.c
index 287caa7..16583c1 100644
--- a/drivers/md/dm-mpath.c
+++ b/drivers/md/dm-mpath.c
@@ -1416,12 +1416,14 @@ static void multipath_postsuspend(struct dm_target *ti)
static void multipath_resume(struct dm_target *ti)
{
struct multipath *m = ti->private;
+ unsigned long flags;
+ spin_lock_irqsave(&m->lock, flags);
if (test_bit(MPATHF_SAVED_QUEUE_IF_NO_PATH, &m->flags))
set_bit(MPATHF_QUEUE_IF_NO_PATH, &m->flags);
else
clear_bit(MPATHF_QUEUE_IF_NO_PATH, &m->flags);
- smp_mb__after_atomic();
+ spin_unlock_irqrestore(&m->lock, flags);
}
/*
Hello Mike,
Thanks again for having made this patch available. I will test it as
soon as I have the time. BTW, in the meantime I ran a few tests with
DM_MQ_DEFAULT=n since until now I ran all tests with DM_MQ_DEFAULT=y.
The result of these tests is as follows:
* v4.6.0, v4.6.5 and v4.7.0 with DM_MQ_DEFAULT=y: first simulated path
removal triggers I/O errors.
* v4.6.4, v4.6.5 and v4.7.0 with DM_MQ_DEFAULT=n: test passes more than
100 iterations.
I have not yet run any tests with kernel v4.5.x because in the test I
ran the ib_srp and ib_srpt drivers are loaded on the same system and
because I need five v4.7 LIO patches to run this test pass but
unfortunately these patches do not apply cleanly on the v4.5.x code base.
Please let me know if you need more information.
Bart.
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
