From: Martin Wilck <mwilck@suse.com>
To: Christophe Varoqui <christophe.varoqui@opensvc.com>
Cc: Martin Wilck <mwilck@suse.com>, dm-devel@redhat.com
Subject: [PATCH v4 00/22] libmultipath: checkers overhaul
Date: Wed, 31 Oct 2018 11:59:37 +0100 [thread overview]
Message-ID: <20181031105939.22202-1-mwilck@suse.com> (raw)
Hi Christophe,
This is v4 of my "checkers overhaul" series. In v3, I missed
one minor suggestion from Ben. Changed wrt v3 are only 03/22
and 12/22; I kept Ben's Reviewed-by: tag in 12/22.
This series starts with a few minor fixes and then attempts
an overhaul of the checker code.
First, there's a block of patches to get rid of the "message"
char array in struct checker, replacing it with an integer.
This topic had been touched in recent discussion between Ben
and myself.
The next larger block fixes problems with checkers that try
to check unsupported devices. It's an interesting experience
to configure wrong checkers for the existing devices and see
what happens. With these patches, paths won't be falsely
teared down any more in such situations.
The last patch cleans up the checker data structure by splitting
it into a "checker class" and the path "checker instance".
There's more work to do in this area, but this is a start.
Regards,
Martin
----
Changes v3->v4:
- 03/22: renamed CHECKER_LAST_GENERIC_MSGID -> CHECKER_GENERIC_MSGTABLE_SIZE
(Ben).
- 12/22: rebased on top of changed 03/22.
Changes v2->v3:
- 03/22: fixed one minor issue mentioned by Ben;
reverted the const-ification of checker_message(),
as it will be reverted in 22/22 anyway.
- 13/22: fix clariion checker semantics (Ben).
- 21/22: rebased on top of updated 03/22.
- 22/22: fix thread-safety issue from 03/22 (Ben).
Changes v1->v2:
- 11/22: rebased on top of "various multipath-tools patches" series
Martin Wilck (22):
libmultipath: fix use of uninitialized memory in write()
libmultipath: fix memory leaks from scandir() use
libmultipath/checkers: replace message by msgid
libmultipath/checkers: cciss_tur: use message id
libmultipath/checkers: directio: use message id
libmultipath/checkers: emc_clariion: use message id
libmultipath/checkers: hp_sw: use message id
libmultipath/checkers: rdac: use message id
libmultipath/checkers: readsector0: use message id
libmultipath/checkers: tur: use message id
multipathd: improve checker message logging
libmultipath/checkers: support unsupported paths
libmultipath: clariion checker: leave unsupported paths alone
libmultipath: hp_sw checker: leave unsupported paths alone
libmultipath: rdac checker: leave unsupported paths alone
libmultipath: tur checker: leave unsupported paths alone
libmultipath: pathinfo: don't blank wwid if checker fails
multipathd: check_path: improve logging for "unusable path" case
libmultipath: coalesce_paths: improve logging of orphaned paths
libmultipath: sync_map_state: log failing paths
libmultipath/checkers: cleanup class/instance model
libmultipath: make checker_message thread safe
libmultipath/checkers.c | 209 ++++++++++++++++++---------
libmultipath/checkers.h | 67 ++++++---
libmultipath/checkers/cciss_tur.c | 13 +-
libmultipath/checkers/directio.c | 29 ++--
libmultipath/checkers/emc_clariion.c | 114 ++++++++++++---
libmultipath/checkers/hp_sw.c | 39 +++--
libmultipath/checkers/rdac.c | 92 ++++++++----
libmultipath/checkers/readsector0.c | 7 +-
libmultipath/checkers/tur.c | 60 ++++----
libmultipath/config.c | 10 +-
libmultipath/configure.c | 10 +-
libmultipath/discovery.c | 16 +-
libmultipath/foreign.c | 5 +-
libmultipath/foreign/nvme.c | 6 +-
libmultipath/print.c | 2 +-
libmultipath/propsel.c | 19 +--
libmultipath/structs_vec.c | 5 +-
libmultipath/sysfs.c | 5 +-
libmultipath/util.c | 9 ++
libmultipath/util.h | 9 ++
multipathd/main.c | 35 +++--
21 files changed, 519 insertions(+), 242 deletions(-)
--
2.19.1
next reply other threads:[~2018-10-31 10:59 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-10-31 10:59 Martin Wilck [this message]
2018-10-31 10:59 ` [PATCH v4 03/22] libmultipath/checkers: replace message by msgid Martin Wilck
2018-11-01 19:48 ` Benjamin Marzinski
2018-10-31 10:59 ` [PATCH v4 12/22] libmultipath/checkers: support unsupported paths Martin Wilck
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181031105939.22202-1-mwilck@suse.com \
--to=mwilck@suse.com \
--cc=christophe.varoqui@opensvc.com \
--cc=dm-devel@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.