[Lustre-devel] Setting attributes at open.

James Simmons jsimmons at infradead.org
Fri Jan 14 12:04:11 PST 2011


> Hello,
> 
> just quick idea. The mdd_open_sanity_check() is called under  
> mdd_write_lock():
> >         mdd_write_lock(env, mdd_obj, MOR_TGT_CHILD);
> > -        rc = mdd_open_sanity_check(env, mdd_obj, flags);
> > +        rc = mdd_open_sanity_check(env, obj, ma, flags);
> 
> You've added mdd_attr_set(env, o, ma); in it causing deadlock it seems,
> because it calls mdd_attr_set_internal_locked() which is taking the same  
> lock.

Perhaps mdd_attr_set is to heavy of a function. I wonder would calling
mdd_setattr_txn_param_build be light enough to do what we want or do we 
really need to write to the change logs that happs in mdd_attr_set.



More information about the lustre-devel mailing list