Re: [dm-devel] [PATCH] mpathpersistent: segment faulty occured in mpath_persistent_reserve_in()

2016-11-20 Thread Christophe Varoqui
Applied, thanks. On Mon, Nov 7, 2016 at 7:35 AM, wrote: > From: "tang.wenji" > > Segment faulty occured when executing "mpathpersist -i -k > /dev/mapper/mpath1" command.The reason is that an uninitialized global > variable conf is used in mpath_persistent_reserve_in(). The same > pr

Re: [dm-devel] [PATCH] multipath-tools: remove Pure modifications

2016-11-20 Thread Christophe Varoqui
Applied, thanks. On Mon, Nov 7, 2016 at 3:40 PM, Xose Vazquez Perez wrote: > There is no reason to deviate from standard values. > > Default: > .fast_io_fail = 5 > .dev_loss = 600 > > Cc: Christophe Varoqui > Cc: device-mapper development > Signed-off-by: Xose Vazquez Perez > --- > l

Re: [dm-devel] [PATCH] multipath-tools: increase the print length of the multipath fields

2016-11-20 Thread Christophe Varoqui
Applied, thanks. On Thu, Nov 10, 2016 at 1:32 PM, wrote: > From: "tang.wenji" > > The maximum field length is set to 64, which may be truncated when > displayed. > E.g:the multipath's 'features' field sets as "6 queue_if_no_path > pg_init_retries 3 pg_init_delay_msecs 30 retain_attached_hw_hand

Re: [dm-devel] [PATCH 0/3] more multipath config options

2016-11-20 Thread Christophe Varoqui
Applied, thanks. On Fri, Nov 11, 2016 at 11:58 PM, Benjamin Marzinski wrote: > Here are two new mutipath config options, and the documentation for > disable_changed_wwids. > > Benjamin Marzinski (3): > document "disable_changed_wwids" config parameter > allow multipath to retry removes of in

Re: [dm-devel] [PATCH] multipath-tools: remove Pure modifications

2016-11-20 Thread Xose Vazquez Perez
On 11/20/2016 10:15 AM, Christophe Varoqui wrote: > Applied, > thanks. No sure, if you skipped or are delaying these patches: Subject: [PATCH] multipath-tools: remove old multipath/{01_udev,02_multipath} scripts Subject: [PATCH] multipath-tools: merge nvme in similar blacklist regex Subject: [PA

Re: [dm-devel] [PATCH] multipath-tools: remove Pure modifications

2016-11-20 Thread Christophe Varoqui
Hi Xose, I think I'll be able to empty the patches backlog today. These patches are still candidates, I just had to interrupt my merge session this morning. Thanks for the digest anyway. On Sun, Nov 20, 2016 at 12:41 PM, Xose Vazquez Perez wrote: > On 11/20/2016 10:15 AM, Christophe Varoqui w

Re: [dm-devel] [PATCH] multipath-tools: update bl info for nvme in multipath.conf.5

2016-11-20 Thread Christophe Varoqui
Applied, thanks. On Wed, Nov 16, 2016 at 5:20 PM, Xose Vazquez Perez wrote: > Cc: Christophe Varoqui > Cc: device-mapper development > Signed-off-by: Xose Vazquez Perez > --- > multipath/multipath.conf.5 | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/multipath/multi

Re: [dm-devel] [PATCH] multipath-tools: merge nvme in similar blacklist regex

2016-11-20 Thread Christophe Varoqui
Applied, thanks. On Wed, Nov 16, 2016 at 5:13 PM, Xose Vazquez Perez wrote: > Similar patterns, /dev/nvme[0-9] . > > Cc: Christophe Varoqui > Cc: device-mapper development > Signed-off-by: Xose Vazquez Perez > --- > libmultipath/blacklist.c | 8 +--- > 1 file changed, 1 insertion(+), 7 d

Re: [dm-devel] [PATCH] libmultipath: remove unnecessary condition in checker_repair

2016-11-20 Thread Christophe Varoqui
Merged, thanks. On Mon, Nov 7, 2016 at 9:57 AM, wrote: > From: PengLiang > > The checker's validity is an unnecessary checking. It will be checked in > checker_selected. > > Signed-off-by: PengLiang > --- > libmultipath/checkers.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > di

Re: [dm-devel] [PATCH] Makefiles: Fix function availability checks

2016-11-20 Thread Christophe Varoqui
Very nice, thanks. Merged. On Fri, Nov 18, 2016 at 8:52 PM, Bart Van Assche wrote: > The current implementation of the code that checks for function > presence is not correct because it checks for a prefix match only. > Introduce a function that checks for the exact function name. > Additionally

Re: [dm-devel] [PATCH] Remove superfluous instances of the "extern" keyword

2016-11-20 Thread Christophe Varoqui
Merged as commit b573878f76a893cece197b5969e54099a5fac905. Merge rejects committed as 9c4be4c0d1b06c066098dafb5aeb914bae1f110b. Thanks. On Fri, Nov 18, 2016 at 9:50 PM, Bart Van Assche wrote: > This avoids that the sparse static analyzer complains about > the "extern" keyword. > > Signed-off-by

Re: [dm-devel] [PATCH] libmultipath: fix missing dm_task_destroy() in dm_reassign()

2016-11-20 Thread Christophe Varoqui
Merged. Thanks. On Mon, Nov 7, 2016 at 3:01 AM, wrote: > From: "zhou.guichun" > > dm_task should be destoryed before going out of dm_reassign() > > Signed-off-by: zhou.guichun > --- > libmultipath/devmapper.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/libmultip

[dm-devel] [PATCH] multipath-tools: enable exec bit for kpartx/kpartx_id script

2016-11-20 Thread Xose Vazquez Perez
Cc: Christophe Varoqui Cc: device-mapper development Signed-off-by: Xose Vazquez Perez --- kpartx/kpartx_id | 0 1 file changed, 0 insertions(+), 0 deletions(-) mode change 100644 => 100755 kpartx/kpartx_id diff --git a/kpartx/kpartx_id b/kpartx/kpartx_id old mode 100644 new mode 100755 -- 2