Re: [dm-devel] [PATCH] libmpathpersist: fix thread safety of default functions

2021-01-26 Thread Benjamin Marzinski
On Tue, Jan 26, 2021 at 10:04:28AM +, Martin Wilck wrote: > On Mon, 2021-01-25 at 23:31 -0600, Benjamin Marzinski wrote: > > commit a839e39e ("libmpathpersist: factor out initialization and > > teardown") made mpath_presistent_reserve_{in,out} use share variables > > for curmp and pathvec. 

Re: [dm-devel] [PATCH] libmpathpersist: fix thread safety of default functions

2021-01-26 Thread Benjamin Marzinski
On Tue, Jan 26, 2021 at 09:36:59AM +, Martin Wilck wrote: > On Mon, 2021-01-25 at 23:31 -0600, Benjamin Marzinski wrote: > > commit a839e39e ("libmpathpersist: factor out initialization and > > teardown") made mpath_presistent_reserve_{in,out} use share variables > > for curmp and pathvec. 

Re: [dm-devel] [PATCH] libmpathpersist: fix thread safety of default functions

2021-01-26 Thread Martin Wilck
On Mon, 2021-01-25 at 23:31 -0600, Benjamin Marzinski wrote: > commit a839e39e ("libmpathpersist: factor out initialization and > teardown") made mpath_presistent_reserve_{in,out} use share variables > for curmp and pathvec.  There are users of this library that call > these > functions in a

Re: [dm-devel] [PATCH] libmpathpersist: fix thread safety of default functions

2021-01-26 Thread Martin Wilck
On Mon, 2021-01-25 at 23:31 -0600, Benjamin Marzinski wrote: > commit a839e39e ("libmpathpersist: factor out initialization and > teardown") made mpath_presistent_reserve_{in,out} use share variables > for curmp and pathvec.  There are users of this library that call > these > functions in a

[dm-devel] [PATCH] libmpathpersist: fix thread safety of default functions

2021-01-25 Thread Benjamin Marzinski
commit a839e39e ("libmpathpersist: factor out initialization and teardown") made mpath_presistent_reserve_{in,out} use share variables for curmp and pathvec. There are users of this library that call these functions in a multi-threaded process, and this change causes their application to crash.