Emmanuel, I have submitted the fix for review: http://review.gluster.com/3380 I have not tested the fix with "/" having EA disabled. It would be great if you could confirm the looping forever doesn't happen with this fix.
thanks, krish ----- Original Message ----- From: "Emmanuel Dreyfus" <[email protected]> To: "Krishnan Parthasarathi" <[email protected]> Cc: [email protected] Sent: Sunday, May 20, 2012 1:41:04 PM Subject: Re: [Gluster-devel] 3.3 requires extended attribute on / Krishnan Parthasarathi <[email protected]> wrote: > It is strange that the you see glusterd_path_in_use() loop forever. If I > am not wrong, the inner loop checks for presence of trusted.gfid and > trusted.glusterfs.volume-id and should exit after that, and the outer loop > performs dirname on the path repeatedly and dirname(3) guarantees such an > operation should return "/" eventually, which we check. Here is my setup when I tried that: / with EA enabled /export/wd3a ibrick with EA enabled But I may have been testing with an untintended patch in glusterd_path_in_use(). I will retry with the right fix once it will be available. -- Emmanuel Dreyfus http://hcpnet.free.fr/pubz [email protected] _______________________________________________ Gluster-devel mailing list [email protected] https://lists.nongnu.org/mailman/listinfo/gluster-devel
