From: "Benjamin Marzinski" <bmarzins@redhat.com>
To: Gris Ge <fge@redhat.com>
Cc: dm-devel@redhat.com
Subject: Re: [PATCH V6 0/3] Introducing multipath C API
Date: Fri, 15 Jul 2016 16:36:45 -0500 [thread overview]
Message-ID: <20160715213645.GQ8095@octiron.msp.redhat.com> (raw)
In-Reply-To: <20160712065038.17565-1-fge@redhat.com>
On Tue, Jul 12, 2016 at 02:50:35PM +0800, Gris Ge wrote:
ACK, with my comment on patch 1/3
-Ben
> Changes since V5:
> * Fix commit message typo of patch 1/3:
> 'EINVA vs EINVAL' and 'dedicate vs dedicated'
> * Use $(LN) and $(RM) in Makefile in patch 3/3.
> * Rebased to current master(c9aef428b1b16b8128c9fbed1cdefe30bed4ac6f).
>
> Changes since V4:
>
> * Remove the unused constant incorrectly added to libmpathcmd in V3.
> * The patch 3/3 could also be found in below link if dm-devel mailing list
> discard that patch again:
> https://github.com/cathay4t/multipath-tools/commit/b992056a447b90251a65aa4919055c70aa62a498.patch
>
> Changes since V3:
> * New way to limit the IPC command length.
> * Treat IPC connection refuse error as DMMP_ERR_NO_DAEMON.
>
> Gris Ge (3):
> multipath-tools: New way to limit the IPC command length.
> multipath-tools: Set errno mpath_recv_reply() when failure
> multipath-tools: Introducing multipath C API
>
> .gitignore | 4 +
> Makefile | 1 +
> Makefile.inc | 3 +
> libdmmp/DEV_NOTES | 41 +
> libdmmp/Makefile | 78 +
> libdmmp/docs/kernel-doc | 2971 +++++++++++++++++++++++++++++++++
> libdmmp/docs/libdmmp.h.3 | 113 ++
> libdmmp/docs/split-man.pl | 41 +
> libdmmp/libdmmp.c | 286 ++++
> libdmmp/libdmmp.pc.in | 9 +
> libdmmp/libdmmp/libdmmp.h | 607 +++++++
> libdmmp/libdmmp_misc.c | 87 +
> libdmmp/libdmmp_mp.c | 150 ++
> libdmmp/libdmmp_path.c | 115 ++
> libdmmp/libdmmp_pg.c | 208 +++
> libdmmp/libdmmp_private.h | 208 +++
> libdmmp/test/Makefile | 30 +
> libdmmp/test/libdmmp_speed_test.c | 49 +
> libdmmp/test/libdmmp_test.c | 144 ++
> libmpathcmd/mpath_cmd.c | 6 +-
> libmpathcmd/mpath_cmd.h | 2 -
> libmpathpersist/mpath_updatepr.c | 6 +-
> libmultipath/Makefile | 2 +-
> libmultipath/alias.c | 1 -
> libmultipath/configure.c | 5 +-
> libmultipath/file.c | 24 +-
> libmultipath/file.h | 1 +
> libmultipath/uxsock.h | 6 -
> libmultipath/wwids.c | 1 -
> multipath/main.c | 1 -
> multipathd/Makefile | 2 +-
> multipathd/uxclnt.c | 13 +-
> multipathd/uxlsnr.c | 12 +-
> {libmultipath => multipathd}/uxsock.c | 69 +-
> multipathd/uxsock.h | 13 +
> 35 files changed, 5210 insertions(+), 99 deletions(-)
> create mode 100644 libdmmp/DEV_NOTES
> create mode 100644 libdmmp/Makefile
> create mode 100644 libdmmp/docs/kernel-doc
> create mode 100644 libdmmp/docs/libdmmp.h.3
> create mode 100644 libdmmp/docs/split-man.pl
> create mode 100644 libdmmp/libdmmp.c
> create mode 100644 libdmmp/libdmmp.pc.in
> create mode 100644 libdmmp/libdmmp/libdmmp.h
> create mode 100644 libdmmp/libdmmp_misc.c
> create mode 100644 libdmmp/libdmmp_mp.c
> create mode 100644 libdmmp/libdmmp_path.c
> create mode 100644 libdmmp/libdmmp_pg.c
> create mode 100644 libdmmp/libdmmp_private.h
> create mode 100644 libdmmp/test/Makefile
> create mode 100644 libdmmp/test/libdmmp_speed_test.c
> create mode 100644 libdmmp/test/libdmmp_test.c
> delete mode 100644 libmultipath/uxsock.h
> rename {libmultipath => multipathd}/uxsock.c (67%)
> create mode 100644 multipathd/uxsock.h
>
> --
> 2.9.0
>
> --
> dm-devel mailing list
> dm-devel@redhat.com
> https://www.redhat.com/mailman/listinfo/dm-devel
next prev parent reply other threads:[~2016-07-15 21:36 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-28 3:52 [PATCH] Introducing multipath C API <libdmmp/libdmmp.h> Gris Ge
2016-01-28 9:15 ` Hannes Reinecke
2016-01-28 9:40 ` Gris Ge
2016-02-01 13:13 ` Todd Gill
2016-02-01 13:36 ` Hannes Reinecke
2016-02-12 8:10 ` [PATCH V2] Introducing multipath C API Gris Ge
2016-02-12 8:10 ` [PATCH V2] Introducing multipath C API <libdmmp/libdmmp.h> Gris Ge
2016-03-04 16:06 ` Benjamin Marzinski
2016-03-05 9:46 ` Gris Ge
2016-07-01 12:46 ` [PATCH V3 0/3] Introducing multipath C API Gris Ge
2016-07-01 12:46 ` [PATCH V3 1/3] multipath-tools: Increase MAX_REPLY_LEN Gris Ge
2016-07-01 14:46 ` Hannes Reinecke
2016-07-02 0:25 ` Gris Ge
2016-07-04 6:05 ` Hannes Reinecke
2016-07-04 9:11 ` Gris Ge
2016-07-04 9:17 ` [PATCH V4 0/3] Introducing multipath C API Gris Ge
2016-07-04 9:17 ` [PATCH V4 1/3] multipath-tools: New way to limit the IPC command length Gris Ge
2016-07-04 9:17 ` [PATCH V4 2/3] multipath-tools: Set errno mpath_recv_reply() when failure Gris Ge
2016-07-04 9:17 ` [PATCH V4 3/3] multipath-tools: Introducing multipath C API Gris Ge
2016-07-04 9:27 ` [PATCH V4 0/3] " Gris Ge
2016-07-04 9:29 ` Please ignore this patch set. ([PATCH V4 0/3] Introducing multipath C API) Gris Ge
2016-07-04 9:40 ` [PATCH V5 0/3] Introducing multipath C API Gris Ge
2016-07-04 9:40 ` [PATCH V5 1/3] multipath-tools: New way to limit the IPC command length Gris Ge
2016-07-04 9:40 ` [PATCH V5 2/3] multipath-tools: Set errno mpath_recv_reply() when failure Gris Ge
2016-07-04 9:40 ` [PATCH V5 3/3] multipath-tools: Introducing multipath C API Gris Ge
2016-07-01 12:46 ` [PATCH V3 2/3] multipath-tools: Set errno mpath_recv_reply() when failure Gris Ge
2016-07-01 12:46 ` [PATCH V3 3/3] multipath-tools: Introducing multipath C API <libdmmp/libdmmp.h> Gris Ge
2016-07-01 12:55 ` [PATCH " Gris Ge
2016-07-01 13:06 ` [PATCH V3 " Gris Ge
2016-07-12 6:50 ` [PATCH V6 0/3] Introducing multipath C API Gris Ge
2016-07-12 6:50 ` [PATCH V6 1/3] multipath-tools: New way to limit the IPC command length Gris Ge
2016-07-15 21:35 ` Benjamin Marzinski
2016-07-18 12:38 ` Gris Ge
2016-08-12 15:57 ` Bart Van Assche
2016-08-12 21:35 ` Benjamin Marzinski
2016-08-12 21:49 ` Bart Van Assche
2016-07-12 6:50 ` [PATCH V6 2/3] multipath-tools: Set errno mpath_recv_reply() when failure Gris Ge
2016-07-12 6:50 ` [PATCH V6 3/3] multipath-tools: Introducing multipath C API Gris Ge
2016-07-15 21:36 ` Benjamin Marzinski [this message]
2016-08-12 12:12 ` [PATCH V7 0/4] multipath: " Gris Ge
2016-08-12 12:12 ` [PATCH V7 1/4] libmpathcmd: Block SIGPIPE when write() Gris Ge
2016-08-12 16:01 ` Bart Van Assche
2016-08-12 12:12 ` [PATCH V7 2/4] multipath-tools: New way to limit the IPC command length Gris Ge
2016-08-12 15:48 ` Bart Van Assche
2016-08-12 21:53 ` Benjamin Marzinski
2016-08-12 12:12 ` [PATCH V7 3/4] multipath-tools: Set errno mpath_recv_reply() when failure Gris Ge
2016-08-12 12:12 ` [PATCH V7 4/4] multipath-tools: Introducing multipath C API Gris Ge
2017-02-24 12:50 ` [PATCH V5] " Gris Ge
2017-02-27 5:56 ` Christophe Varoqui
2017-02-24 13:07 ` Gris Ge
2016-08-12 13:26 ` [PATCH V7 0/4] multipath: " Gris Ge
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=20160715213645.GQ8095@octiron.msp.redhat.com \
--to=bmarzins@redhat.com \
--cc=dm-devel@redhat.com \
--cc=fge@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).