[Ocfs2-devel] [PATCH V2] Fix the nested PR lock calling issue

Sunil Mushran sunil.mushran at oracle.com
Fri Jul 23 16:08:45 PDT 2010


ok. Then review Jiaju's earlier patch.

On 07/23/2010 03:42 PM, Tiger Yang wrote:
> Hi, Sunil,
>
> I think put them in ocfs2_check_acl() is better.
> First, check mount option in ocfs2_check_acl()  is more clear than in 
> _ocfs2_get_acl().
> Second, we already have ocfs2_get_acl and ocfs2_get_acl_nolock, so it 
> seems _ocfs2_get_acl is redundant and could cause confusing for 
> reading the code.
>
> Regards,
> tiger
>
> On 07/21/2010 02:26 PM, Sunil Mushran wrote:
>> Why not add _ocfs2_get_acl() that does the same without
>> taking the cluster locks?
>




More information about the Ocfs2-devel mailing list