<div dir="ltr">Merged.<div>Thanks.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 21, 2017 at 12:03 AM, Benjamin Marzinski <span dir="ltr"><<a href="mailto:bmarzins@redhat.com" target="_blank">bmarzins@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sun, Mar 26, 2017 at 03:49:47PM +0200, Martin Wilck wrote:<br>
> Use the global variable "udev" - the internal one is not<br>
> initialized, causing current libudev calls to fail.<br>
> In the main program "mpathpersist", use a globally visible<br>
> variable "udev" rather than a local variable in main().<br>
> This imitates the way the global variable "udev" is used<br>
> in multipath and multipathd.<br>
><br>
> Removed the "udev" parameter from mpath_lib_init() to<br>
> clarify that it isn't used.<br>
><br>
<br>
</span>ACK<br>
<br>
-Ben<br>
<div><div class="h5"><br>
> Fixes: b87454988 "libmultipath: separate out 'udev' config entry"<br>
> Signed-off-by: Martin Wilck <<a href="mailto:mwilck@suse.com">mwilck@suse.com</a>><br>
> ---<br>
>  libmpathpersist/mpath_persist.<wbr>c | 4 ++--<br>
>  libmpathpersist/mpath_persist.<wbr>h | 2 +-<br>
>  mpathpersist/main.c             | 5 +++--<br>
>  3 files changed, 6 insertions(+), 5 deletions(-)<br>
><br>
> diff --git a/libmpathpersist/mpath_<wbr>persist.c b/libmpathpersist/mpath_<wbr>persist.c<br>
> index 982c7954..9bca7764 100644<br>
> --- a/libmpathpersist/mpath_<wbr>persist.c<br>
> +++ b/libmpathpersist/mpath_<wbr>persist.c<br>
> @@ -34,10 +34,10 @@<br>
><br>
>  #define __STDC_FORMAT_MACROS 1<br>
><br>
> -struct udev *udev;<br>
> +extern struct udev *udev;<br>
><br>
>  struct config *<br>
> -mpath_lib_init (struct udev *udev)<br>
> +mpath_lib_init (void)<br>
>  {<br>
>       struct config *conf;<br>
><br>
> diff --git a/libmpathpersist/mpath_<wbr>persist.h b/libmpathpersist/mpath_<wbr>persist.h<br>
> index 79de5b5b..7422322d 100644<br>
> --- a/libmpathpersist/mpath_<wbr>persist.h<br>
> +++ b/libmpathpersist/mpath_<wbr>persist.h<br>
> @@ -174,7 +174,7 @@ struct prout_param_descriptor {           /* PROUT parameter descriptor */<br>
>   *<br>
>   * RETURNS: struct config ->Success, NULL->Failed.<br>
>   */<br>
> -extern struct config * mpath_lib_init (struct udev *udev);<br>
> +extern struct config * mpath_lib_init (void);<br>
><br>
><br>
>  /*<br>
> diff --git a/mpathpersist/main.c b/mpathpersist/main.c<br>
> index 2e0aba3c..e1aac8fa 100644<br>
> --- a/mpathpersist/main.c<br>
> +++ b/mpathpersist/main.c<br>
> @@ -57,6 +57,8 @@ void rcu_register_thread_memb(void) {}<br>
><br>
>  void rcu_unregister_thread_memb(<wbr>void) {}<br>
><br>
> +struct udev *udev;<br>
> +<br>
>  int main (int argc, char * argv[])<br>
>  {<br>
>       int fd, c, res;<br>
> @@ -86,7 +88,6 @@ int main (int argc, char * argv[])<br>
>       int num_transport =0;<br>
>       void *resp = NULL;<br>
>       struct transportid * tmp;<br>
> -     struct udev *udev = NULL;<br>
>       struct config *conf;<br>
><br>
>       if (optind == argc)<br>
> @@ -104,7 +105,7 @@ int main (int argc, char * argv[])<br>
>       }<br>
><br>
>       udev = udev_new();<br>
> -     conf = mpath_lib_init(udev);<br>
> +     conf = mpath_lib_init();<br>
>       if(!conf) {<br>
>               udev_unref(udev);<br>
>               exit(1);<br>
> --<br>
> 2.12.0<br>
><br>
</div></div>> --<br>
> dm-devel mailing list<br>
> <a href="mailto:dm-devel@redhat.com">dm-devel@redhat.com</a><br>
> <a href="https://www.redhat.com/mailman/listinfo/dm-devel" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/dm-devel</a><br>
<br>
--<br>
dm-devel mailing list<br>
<a href="mailto:dm-devel@redhat.com">dm-devel@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/dm-devel" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/dm-devel</a><br>
</blockquote></div><br></div>