Re: [dm-devel] [PATCH 9/9] multipathd: use update_path_groups instead of reload_map

2019-02-27 Thread Martin Wilck
On Tue, 2019-02-26 at 16:32 -0600, Benjamin Marzinski wrote: > On Tue, Feb 26, 2019 at 11:47:29AM +0100, Martin Wilck wrote: > > On Fri, 2019-02-22 at 10:58 -0600, Benjamin Marzinski wrote: > > > reload_map() doesn't do the work to sync the state after > > > reloading > > > the > > > map. Instead

Re: [dm-devel] [PATCH 9/9] multipathd: use update_path_groups instead of reload_map

2019-02-26 Thread Benjamin Marzinski
On Tue, Feb 26, 2019 at 11:47:29AM +0100, Martin Wilck wrote: > On Fri, 2019-02-22 at 10:58 -0600, Benjamin Marzinski wrote: > > reload_map() doesn't do the work to sync the state after reloading > > the > > map. Instead of calling it directly, cli_reload() and > > uev_update_path() > > should cal

Re: [dm-devel] [PATCH 9/9] multipathd: use update_path_groups instead of reload_map

2019-02-26 Thread Martin Wilck
On Fri, 2019-02-22 at 10:58 -0600, Benjamin Marzinski wrote: > reload_map() doesn't do the work to sync the state after reloading > the > map. Instead of calling it directly, cli_reload() and > uev_update_path() > should call update_path_groups(), which calls reload_map() with all > the > necessar

[dm-devel] [PATCH 9/9] multipathd: use update_path_groups instead of reload_map

2019-02-22 Thread Benjamin Marzinski
reload_map() doesn't do the work to sync the state after reloading the map. Instead of calling it directly, cli_reload() and uev_update_path() should call update_path_groups(), which calls reload_map() with all the necessary syncing. Signed-off-by: Benjamin Marzinski --- multipathd/cli_handlers