[0/2] multipath cleanup patches

Message

Benjamin Marzinski Jan. 5, 2021, 3:59 a.m. UTC

These patches are cleanups that Martin mentioned for my last patchset.
The first is some code that should have been in patch 4/6 (libmultipath:
ignore multipaths sections without wwid option). The second is V2 of
patch 6/6 (multipath.conf.5: Improve checker_timeout description), which
was updated based on Martin's comments.

Benjamin Marzinski (2):
  libmultipath: check for null wwid before strcmp
  multipath.conf.5: Improve checker_timeout description

 libmultipath/config.c      |  2 +-
 multipath/multipath.conf.5 | 20 ++++++++++++--------
 2 files changed, 13 insertions(+), 9 deletions(-)

Comments

Martin Wilck Jan. 5, 2021, 10:57 a.m. UTC | #1

On Mon, 2021-01-04 at 21:59 -0600, Benjamin Marzinski wrote:
> These patches are cleanups that Martin mentioned for my last
> patchset.
> The first is some code that should have been in patch 4/6
> (libmultipath:
> ignore multipaths sections without wwid option). The second is V2 of
> patch 6/6 (multipath.conf.5: Improve checker_timeout description),
> which
> was updated based on Martin's comments.
> 
> Benjamin Marzinski (2):
>   libmultipath: check for null wwid before strcmp
>   multipath.conf.5: Improve checker_timeout description
> 
>  libmultipath/config.c      |  2 +-
>  multipath/multipath.conf.5 | 20 ++++++++++++--------
>  2 files changed, 13 insertions(+), 9 deletions(-)
> 

For the set:

Reviewed-by: Martin Wilck <[e-mail]>