On Mar 27, 2007, at 3:53 PM, Phil Carns wrote:
I like the structure of the patch in that it cleans the xattr namespace translation goop out of the state machines.I think that this may break something, though. Could you confirm with the test/automated/tacl_xattr.sh script on your end? I may have done something wrong, but this test appears to work fine without the attached patch, but then has some failures if I apply it.
Hi Phil,There were some bugs in that patch, yeah. I've attached another patch that fixes them. Even without the patch, tacl_xattr.sh reports failures, but now the failures with and without the patch are the same. I've attached the output of the tacl script for with and without the patch cases. Do you get those failures?
As a side note, it looks like there are still some noisy messages coming from the kernel related to ACLs regardless of whether the patches are applied or not. The tacl_xattr.sh script generates several of these in dmesg for me:pvfs2_acl_chmod: get acl (access) failed with 0 They also pop out during LTP test runs.
Attached patch should also fix those. I think the error checking was just a little bit wrong.
-sam
eattr-callouts4.patch
Description: Binary data
tacl-out-no-patch
Description: Binary data
tacl-out-with-patch
Description: Binary data
-Phil
_______________________________________________ Pvfs2-developers mailing list [email protected] http://www.beowulf-underground.org/mailman/listinfo/pvfs2-developers
