On Sun, Oct 07, 2018 at 10:36:41AM +0200, Christophe Varoqui wrote: I'll send a v4, hopefully today. -Ben > Hi Ben, > this v3 patchset is ready for merge or do you plan a v4 ? > Thanks, > Christophe > On Sat, Sep 22, 2018 at 1:05 AM Benjamin Marzinski > <[1]bmarzins@xxxxxxxxxx> wrote: > > This batch of patches is a resend of the non-merged pathes from my last > set, along with 2 new ones. It would be really great if we could get > a version bug after the outstanding commits go it. > > Patches 0001-0005 are a number of fixes to the tur checker.These are > the ones that should get the most attention. > Patches 0006-0017 are minor issues found by coverity. > Not all of them are bugs that could actually occur in practice, > but they are simple and hopefully non-controversial changes. > Patches 0018-0019 are new and related to changing path wwids > > Changes in v3 > added patches 0018-0019 > > Changes in v2 > 0002-libmultipath-fix-tur-checker-double-locking.patch now sets > ct->devt when initially creating the tur_checker_context, while > that structure is still only referenced by a local variable. > After that, ct->devt is only ever read. This should remove any > issues with it needing locking. > > Benjamin Marzinski (19): > libmultipath: fix tur checker timeout > libmultipath: fix tur checker double locking > libmultipath: fix tur memory misuse > libmultipath: cleanup tur locking > libmultipath: fix tur checker timeout issue > libmultipath: fix set_int error path > libmultipath: fix length issues in get_vpd_sgio > libmultipath: _install_keyword cleanup > libmultipath: remove unused code > libmultipath: fix memory issue in path_latency prio > libmultipath: fix null dereference int alloc_path_group > libmutipath: don't use malformed uevents > multipath: fix max array size in print_cmd_valid > multipathd: function return value tweaks > multipathd: minor fixes > multipathd: remove useless check and fix format > multipathd: fix memory leak on error in configure > libmultipath: Don't blank intialized paths > libmultipath: Fixup updating paths > > libmultipath/checkers/tur.c | 168 > +++++++++++-------------------- > libmultipath/dict.c | 5 +- > libmultipath/discovery.c | 18 ++-- > libmultipath/parser.c | 12 ++- > libmultipath/print.c | 8 -- > libmultipath/prioritizers/path_latency.c | 3 +- > libmultipath/structs.c | 2 +- > libmultipath/uevent.c | 6 ++ > multipath/main.c | 2 +- > multipathd/cli_handlers.c | 11 +- > multipathd/main.c | 50 +++++---- > 11 files changed, 131 insertions(+), 154 deletions(-) > > -- > 2.7.4 > > References > > Visible links > 1. mailto:bmarzins@xxxxxxxxxx -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
