David, Hmm.. I think this can be done by a simple replace of EOPNOTSUPP with EACCES. I will get back to you later today on this.. Thanks again for the testing! Thanks, Murali
On Fri, 18 Aug 2006, David Metheny wrote: > I ran this test on a RHEL4 32bit server. There are a few differences still > (between ext3/pvfs2) noted below by *** > > --------------------------------------------------- > EXT3 > --------------------------------------------------- > SUCCESS: Chown correct > ***setfacl: shared/symlinkdir1: Only directories can have default ACLs > > ***attr_set: Operation not permitted > Could not set "attrname3" for shared/team2/symlinkfile1 > > Do not follow symlinks. > but extended user attributes are disallowed for symbolic links > > attr -r to remove the named object > > --------------------------------------------------- > PVFS2 > --------------------------------------------------- > SUCCESS: Chown correct > > ***attr_set: Operation not supported > Could not set "attrname3" for shared/team2/symlinkfile1 > > Do not follow symlinks. > but extended user attributes are disallowed for symbolic links > ***getfattr: shared/team2/symlinkfile1: Operation not supported > ***FAILED: getfattr: Do not follow symlinks. > > attr -r to remove the named object > ***getfattr: shared/team1/symlinkfile1: Operation not supported > ***getfattr: shared/team2/symlinkfile1: Operation not supported > ***getfattr: shared/team1/symlinkfile1: Operation not supported > ***getfattr: shared/team2/symlinkfile1: Operation not supported > > > > -----Original Message----- > > From: Murali Vilayannur [mailto:[EMAIL PROTECTED] > > Sent: Thursday, August 17, 2006 7:15 PM > > To: David Metheny > > Cc: [email protected] > > Subject: RE: [Pvfs2-developers] ACL errors using LTP tests > > > > David, > > I committed a series of fixes to the tree the last couple of > > days that should fix the ACL test errors reported by the LTP script.. > > Could you give that a whirl and let us know if they fix the > > problem for you? > > Thanks for the pointer to the script. > > I guess, we will add it to the nightly test suites soon. > > Murali > > > > On Tue, 15 Aug 2006, David Metheny wrote: > > > > > I've attached an updated version of the tacl_xattr.sh > > script mentioned > > > below. This should now execute tests for both PVFS2 and EXT3. > > > > > > To run the script > > > - EXT3 or PVFS2 file system must be mounted with acl,user_xattr. > > > - must be root > > > - ./tacl_xattr.sh [ EXT3_DIR | PVFS2_DIR ] > > > > > > There are several tests failing on PVFS2 at the moment. > > > > > > > > > > -----Original Message----- > > > > From: Murali Vilayannur [mailto:[EMAIL PROTECTED] > > > > Sent: Tuesday, August 08, 2006 2:31 PM > > > > To: David Metheny > > > > Cc: [email protected] > > > > Subject: Re: [Pvfs2-developers] ACL errors using LTP tests > > > > > > > > Hi David, > > > > Thanks for bringing this up.. > > > > Sorry for taking time to respond to this. > > > > Before I got to the acl tests, I did uncover and fix a few other > > > > bugs some of which were in the LTP script (runltp) and > > some of which > > > > were in pvfs2. > > > > The runltp script requires a mkdir -m 777 when it creates > > the temp > > > > directory for many of the tests to work properly (Before that > > > > change, the runltp script was failing even on /tmp for me). > > > > > > > > The tests that continue to fail for me now are > > > > a) the nftw() tests when run as the user "nobody" on a directory > > > > owned by <nobody,root> > > > > b) the writev01 test which passes in a bogus address > > amongst valid > > > > addresses in the iovec and expects that writev() does not return > > > > -EFAULT. > > > > I think we had this discussion a long time back when the writev > > > > implementation was done and I still havent had a chance > > to fix this. > > > > > > > > The acl tests in the LTP suite does something very naive (I am > > > > referring to the acl_file_test and acl_link_test). They > > open a file, > > > > do a > > > > (l)listxattr() on it, do a (l)getxattr() on the first key > > and then > > > > do a > > > > (l)setxattr() on it. > > > > For a regular file, we return "system.pvfs2.dh" and > > "system.pvfs2.md" > > > > as part of listxattr(), and any attempt to do a setxattr() will > > > > obviously fail (currently with -EINVAL) since these are > > not settable > > > > xtended attributes. > > > > Quite frankly, these acl tests are not worth fixing. > > > > > > > > That said, it would be great if we could modify the > > tacl_xattr.sh to > > > > somehow work with pvfs2 (It is currently > > > > ext3 specific) since that is really a good acl test... > > > > If there are any other (Acl or otherwise) tests that fail > > for you, > > > > please let me know and I can take a shot at fixing them. > > > > thanks, > > > > Murali > > > > > > > > On Wed, 2 Aug 2006, David Metheny wrote: > > > > > > > > > We're seeing some acl errors in the LTP tests > > > > (http://ltp.sourceforge.net/). > > > > > This is on a SMP RHEL4 x86_64 server with kernel > > > > 2.6.9-34.0.2. This is > > > > > using the code in the CVS tree on July 19th. I've attached > > > > a test set > > > > > for LTP that runs this. > > > > > > > > > > To run it, do something like this: > > > > > > > > > > # su > > > > > # tar -xvzf ltp-full-20060717.tgz > > > > > # cd ltp-full-20060717 > > > > > # export CFLAGS="-g" > > > > > # make && make install > > > > > # mkdir /mnt/pvfs2/ltp-tmp > > > > > # chmod 777 /mnt/pvfs2/ltp-tmp > > > > > # cp ../pvfs-test-cases runtest/ > > > > > # cp ./testcases/kernel/fs/acls/acl_test01 ./testcases/bin/ > > > > # ./runltp > > > > > -p -l /tmp/ltp.log -d /mnt/pvfs2/ltp-tmp -f pvfs-test-cases 2>&1 > > > > > | tee /tmp/ltp.output > > > > > # tail /tmp/ltp.log > > > > > > > > > > some notes on the above steps: > > > > > ------------------------------- > > > > > - ltp needs to be built and run as root > > > > > - -g CFLAGS helps if you need to run a debugger to track > > > > down failures > > > > > - make install leaves binaries in ltp-full-20050307 directory > > > > > - I don't know if it is strictly required, but it seems > > like I had > > > > > better luck when running the tests in a subdir with 777 > > > > > permissions > > > > > - the end of the log file shows a summary (how many tests > > > > passed and > > > > > failed) > > > > > > > > > > > > > > > > _______________________________________________ Pvfs2-developers mailing list [email protected] http://www.beowulf-underground.org/mailman/listinfo/pvfs2-developers
