Hi All,
I have noticed one more problem with device mapper today.
Summary: If there are more than one path in the path group, device mapper does not activate the all the paths during the failover. DM only activates the first path in the path group. It does not call the function activate_path for the second path (or third so on). This seems like a major problem for me. Current code does not keep track of whether the path is already activated or not. This leads DM using only first path and all other paths become unusable. This happens during the failover.
Test case.
1. Start the IO with all the paths.
2. Fail the active path.
3. Failover (mode select) will happen and passive path will be activated
4. Only first path in path group is activated and other paths will be failed. I would expect the DM to activate all the paths in the path group. However this does not happen.
Output of multipath -ll before the test.
mpathf (3600a0b80000f519c0000cc8a48fc7d0b) dm-2 LSI,INF-01-00
[size=2.0G][features=0][hwhandler=1 rdac][rw]
\_ round-robin 0 [prio=4][enabled]
\_ 3:0:0:0 sde 8:64 [active][undef] running
\_ 3:0:2:0 sdg 8:96 [active][undef] running
\_ round-robin 0 [prio=2][enabled]
\_ 3:0:1:0 sdf 8:80 [active][undef] running
\_ 3:0:3:0 sdh 8:112 [active][undef] running
Output of multipath -ll after the test. Notice that second path has failed.
mpathf (3600a0b80000f519c0000cc8a48fc7d0b) dm-2 LSI,INF-01-00
[size=2.0G][features=0][hwhandler=1 rdac][rw]
\_ round-robin 0 [prio=2][enabled]
\_ 3:0:1:0 sdf 8:80 [active][undef] running
\_ 3:0:3:0 sdh 8:112 [failed][undef] running
Here is the patch, I have used to work-around the problem. I am sure this is not the place we want to add the fix. However, this patch should give you understanding of the problem. This patch will set the flags
m->pg_init_required and m->queue_io whenever D-M select new path in the path group whenever the repeat_count is exhausted.
--- linux-2.6.28-rc4/drivers/md/dm-mpath.c.orig 2008-11-10 17:50:24.000000000 -0600
+++ linux-2.6.28-rc4/drivers/md/dm-mpath.c 2008-11-10 17:51:36.000000000 -0600
@@ -245,6 +245,10 @@ static int __choose_path_in_pg(struct mu
if (!path)
return -ENXIO;
+ /* Set the pg_init_required flag to activate this path */
+ m->pg_init_required = 1;
+ m->queue_io = 1;
+
m->current_pgpath = path_to_pgpath(path);
if (m->current_pg != pg)
--
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
