diff options
| author | tang.junhui <tang.junhui@zte.com.cn> | 2016-11-24 15:11:48 +0800 |
|---|---|---|
| committer | Mike Snitzer <snitzer@redhat.com> | 2016-11-28 16:20:59 -0500 |
| commit | 1422b33eda2ffbc70eb11163bf7ac1e59ebba222 (patch) | |
| tree | 58c4afa09894266401985c2fba532c837884de96 | |
| parent | 6ab0ee62c51446420078804ecc657bd27cc460ad (diff) | |
| download | linux-dm-1422b33eda2ffbc70eb11163bf7ac1e59ebba222.tar.gz | |
dm mpath: use hw_handler_params if attached hw_handler is same as requested
Notice: this object is not reachable from any branch.
Let the requested m->hw_handler_params be used if the attached hardware
handler is the same handler as requested with m->hw_handler_name.
Signed-off-by: tang.junhui <tang.junhui@zte.com.cn>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
Notice: this object is not reachable from any branch.
| -rw-r--r-- | drivers/md/dm-mpath.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/drivers/md/dm-mpath.c b/drivers/md/dm-mpath.c index 0caab4bf35155..6400cffb986df 100644 --- a/drivers/md/dm-mpath.c +++ b/drivers/md/dm-mpath.c @@ -850,18 +850,22 @@ retain: attached_handler_name = scsi_dh_attached_handler_name(q, GFP_KERNEL); if (attached_handler_name) { /* + * Clear any hw_handler_params associated with a + * handler that isn't already attached. + */ + if (m->hw_handler_name && strcmp(attached_handler_name, m->hw_handler_name)) { + kfree(m->hw_handler_params); + m->hw_handler_params = NULL; + } + + /* * Reset hw_handler_name to match the attached handler - * and clear any hw_handler_params associated with the - * ignored handler. * * NB. This modifies the table line to show the actual * handler instead of the original table passed in. */ kfree(m->hw_handler_name); m->hw_handler_name = attached_handler_name; - - kfree(m->hw_handler_params); - m->hw_handler_params = NULL; } } |