On Mon, Dec 10, 2018 at 10:49:35AM +0100, Martin Wilck wrote: > Changes in v3: > > Resent full series on Christophe's request. All patches except 22/24, 23/24 are > the same as before. Added Ben's "Reviewed-by:" where appropriate. Belated ACKs on 23/24 and v4 of 22/24 -Ben > > - 22/24: added one fix for a non-symbolic reference to a domap() return value, > which I'd overlooked before. > - 23/24: removed on Ben's review. The new 23/24 is just a code cleanup > without functional change. > > Changes in v2: > > The first 19 patches are identical to v1 as ACK'd by Ben, except 16/24 > "libmultipath: coalesce_paths: fix size mismatch handling". > No. 8/24 "libmultipath: decrease log level of word splitting" > (not yet ACKd by Ben) also stays the same; the issue Ben raised > in his review is addressed in a separate patch, 20/24. > 21/24 addresses implements Ben's suggestion to use named constants > as return values in coalesce_paths(). 22, 23, 24 do the same for > other important, related functions, as I found it strange to make > this change just for coalesce_paths() alone. > > Martin Wilck (24): > tests/hwtable: set multipath_dir in local configuration > tests/hwtable: adjust to new checker API > multipath-tools: decrease verbosity of state messages > libmultipath: decrease verbosity of pathinfo messages > libmultipath: decrease verbosity of TUR checker messages > libmultipath: avoid frequent messages from filter_property() > libmultipath: decrease log level of "disassembled" messages > libmultipath: decrease log level of word splitting > libmultipath: increase log level of map removal > multipathd: decrease log level of checker timing > libmultipath: decrease log level of "prioritizer refcount" message > libmpathpersist/update_map_pr: decrease log level for nop > libmultipath: simplify devt2devname() > libmultipath: decrease log level for failed VPD c9 > libmultipath: adopt_paths: check for size match > libmultipath: coalesce_paths: fix size mismatch handling > tests: add unit tests for bitmask functions > multipathd: uev_remove_path: remove redundant orphan_paths call > libmultipath: improve logging from orphan_paths > libmultipath: avoid syslog loglevel > LOG_DEBUG > coalesce_paths(): use symbolic return value > domap(): use symbolic return value > multipathd: simplify retry logic in ev_add_path() > multipath: use symbolic return value and exit code > > libmpathpersist/mpath_persist.c | 3 +- > libmultipath/blacklist.c | 54 +++++++------- > libmultipath/blacklist.h | 2 +- > libmultipath/checkers/tur.c | 6 +- > libmultipath/configure.c | 48 +++++++------ > libmultipath/configure.h | 23 ++++++ > libmultipath/discovery.c | 20 +++--- > libmultipath/dmparser.c | 6 +- > libmultipath/log_pthread.c | 3 + > libmultipath/prio.c | 2 +- > libmultipath/structs_vec.c | 18 +++-- > libmultipath/structs_vec.h | 3 +- > libmultipath/util.c | 7 +- > libmultipath/util.h | 16 +++++ > multipath/main.c | 121 ++++++++++++++++++-------------- > multipathd/cli_handlers.c | 5 +- > multipathd/main.c | 45 ++++++------ > tests/Makefile | 7 +- > tests/blacklist.c | 7 +- > tests/hwtable.c | 89 ++++++++++++----------- > tests/util.c | 98 ++++++++++++++++++++++++++ > 21 files changed, 383 insertions(+), 200 deletions(-) > > -- > 2.19.2 -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
