OK, the top 2 commits here: https://github.com/malahal/ntirpc/commits/duplex-12
Regards, Malahal. Matt Benjamin [mbenja...@redhat.com] wrote: > sorry, i meant pull request > > -- > Matt Benjamin > Red Hat, Inc. > 315 West Huron Street, Suite 140A > Ann Arbor, Michigan 48103 > > http://www.redhat.com/en/technologies/storage > > tel. 734-707-0660 > fax. 734-769-8938 > cel. 734-216-5309 > > ----- Original Message ----- > > From: "Malahal Naineni" <mala...@us.ibm.com> > > To: "Matt Benjamin" <mbenja...@redhat.com> > > Cc: nfs-ganesha-devel@lists.sourceforge.net > > Sent: Monday, October 19, 2015 10:30:04 AM > > Subject: Re: Subject: [PATCH] Call destroy on mutexes and condition > > variables > > > > Matt Benjamin [mbenja...@redhat.com] wrote: > > > ok > > > > > > (PR?) > > > > You mean problem report? valgrind drd tool! > > > > > -- > > > Matt Benjamin > > > Red Hat, Inc. > > > 315 West Huron Street, Suite 140A > > > Ann Arbor, Michigan 48103 > > > > > > http://www.redhat.com/en/technologies/storage > > > > > > tel. 734-707-0660 > > > fax. 734-769-8938 > > > cel. 734-216-5309 > > > > > > ----- Original Message ----- > > > > From: "Malahal Naineni" <mala...@us.ibm.com> > > > > To: nfs-ganesha-devel@lists.sourceforge.net > > > > Cc: mbenja...@redhat.com > > > > Sent: Saturday, October 17, 2015 9:38:47 AM > > > > Subject: Subject: [PATCH] Call destroy on mutexes and condition > > > > variables > > > > > > > > From 510492a32b6427d265b812b3923de5910d96d310 Mon Sep 17 00:00:00 2001 > > > > From: Malahal Naineni <mala...@us.ibm.com> > > > > Date: Sat, 17 Oct 2015 08:29:26 -0500 > > > > Subject: [PATCH] Call destroy on mutexes and condition variables > > > > > > > > We should destroy them before freeing the memory. Supresses valgrind > > > > errors. > > > > --- > > > > src/clnt_vc.c | 3 +++ > > > > src/rpc_ctx.c | 4 ++++ > > > > 2 files changed, 7 insertions(+) > > > > > > > > diff --git a/src/clnt_vc.c b/src/clnt_vc.c > > > > index 4c614d1..c893605 100644 > > > > --- a/src/clnt_vc.c > > > > +++ b/src/clnt_vc.c > > > > @@ -316,6 +316,7 @@ clnt_vc_ncreate2(int fd, /* open file descriptor > > > > */ > > > > if (cs) { > > > > mem_free(cs, sizeof(struct ct_serialized)); > > > > } > > > > + mutex_destroy(&clnt->cl_lock); > > > > mem_free(clnt, sizeof(CLIENT)); > > > > } > > > > if (rec) { > > > > @@ -800,6 +801,7 @@ clnt_vc_release(CLIENT *clnt, u_int flags) > > > > mem_free(clnt->cl_netid, strlen(clnt->cl_netid) > > > > + 1); > > > > if (clnt->cl_tp && clnt->cl_tp[0]) > > > > mem_free(clnt->cl_tp, strlen(clnt->cl_tp) + 1); > > > > + mutex_destroy(&clnt->cl_lock); > > > > mem_free(clnt, sizeof(CLIENT)); > > > > > > > > REC_LOCK(rec); > > > > @@ -861,6 +863,7 @@ clnt_vc_destroy(CLIENT *clnt) > > > > mem_free(clnt->cl_netid, strlen(clnt->cl_netid) > > > > + 1); > > > > if (clnt->cl_tp && clnt->cl_tp[0]) > > > > mem_free(clnt->cl_tp, strlen(clnt->cl_tp) + 1); > > > > + mutex_destroy(&clnt->cl_lock); > > > > mem_free(clnt, sizeof(CLIENT)); > > > > > > > > if (xd_refcnt == 0) { > > > > diff --git a/src/rpc_ctx.c b/src/rpc_ctx.c > > > > index 3590422..6c99927 100644 > > > > --- a/src/rpc_ctx.c > > > > +++ b/src/rpc_ctx.c > > > > @@ -85,6 +85,8 @@ alloc_rpc_call_ctx(CLIENT *clnt, rpcproc_t proc, > > > > xdrproc_t > > > > xdr_args, > > > > "%s: call ctx insert failed (xid %d client %p)", > > > > __func__, ctx->xid, clnt); > > > > REC_UNLOCK(rec); > > > > + mutex_destroy(&ctx->we.mtx); > > > > + cond_destroy(&ctx->we.cv); > > > > mem_free(ctx, sizeof(rpc_ctx_t)); > > > > ctx = NULL; > > > > goto out; > > > > @@ -248,5 +250,7 @@ free_rpc_call_ctx(rpc_ctx_t *ctx, uint32_t flags) > > > > > > > > if (ctx->msg) > > > > free_rpc_msg(ctx->msg); > > > > + mutex_destroy(&ctx->we.mtx); > > > > + cond_destroy(&ctx->we.cv); > > > > mem_free(ctx, sizeof(rpc_ctx_t)); > > > > } > > > > -- > > > > 1.8.3.1 > > > > > > > > > > > > > > > > ------------------------------------------------------------------------------ _______________________________________________ Nfs-ganesha-devel mailing list Nfs-ganesha-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
