Re: [PATCH rdma-core] iwpmd: fix double mutex unlock

[Doug Ledford <dledford@xxxxxxxxxx>]

On Wed, 2018-01-31 at 10:04 -0600, Steve Wise wrote:
> > > 
> > > pthread_mutex_unlock() is used twice and this patch fixes it.
> > > 
> > > Fixes: be3fbf85b ("iwpmd: use ccan list.h for pending_messages")
> > > Signed-off-by: Potnuri Bharat Teja <bharat@xxxxxxxxxxx>
> > > ---
> > >  iwpmd/iwarp_pm_helper.c | 1 -
> > >  1 file changed, 1 deletion(-)
> > > 
> > > diff --git a/iwpmd/iwarp_pm_helper.c b/iwpmd/iwarp_pm_helper.c
> > > index bebc151f329b..b8c230189d50 100644
> > > --- a/iwpmd/iwarp_pm_helper.c
> > > +++ b/iwpmd/iwarp_pm_helper.c
> > > @@ -578,7 +578,6 @@ int add_iwpm_pending_msg(iwpm_send_msg
> > > *send_msg)
> > > 
> > >  	pthread_mutex_lock(&pending_msg_mutex);
> > >  	list_add(&pending_messages, &pending_msg->entry);
> > > - 	pthread_mutex_unlock(&pending_msg_mutex);
> > >  	pthread_mutex_unlock(&pending_msg_mutex);
> > >  	/* signal the thread that a new message has been posted */
> > >  	pthread_cond_signal(&cond_pending_msg);
> > 
> > Good catch.
> > 
> > Reviewed-by: Steve Wise <swise@xxxxxxxxxxxxxxxxxxxxx>
> 
> And this definitely needs to flow to stable.
> 

Applied to master branch, thanks.

-- 
Doug Ledford <dledford@xxxxxxxxxx>
    GPG KeyID: B826A3330E572FDD
    Key fingerprint = AE6B 1BDA 122B 23B4 265B  1274 B826 A333 0E57 2FDD

Attachment: signature.asc
Description: This is a digitally signed message part