On 04/29/2013 10:44 PM, Emmanuel Dreyfus wrote:
On Mon, Apr 29, 2013 at 05:01:46PM +0000, Emmanuel Dreyfus wrote:I found an uninitialized mutex and I am tracking another one.In xlators/mgmt/glusterd/src:gd_lock_op_phase() gd_syncargs_init() is not called, whereas it is called at other places to initialize args. Is it correct?
It is correct (and intentional). gd_lock_op_phase and gd_unlock_op_phase don't use args->dict, so don't use the args->lock_dict to protect it's access. ~krish
