Hi Bart,
Does the attached patch help this issue any?
-Phil
Phil Carns wrote:
Hi Bart,
I can't get those specific commands to cause a problem for me (using
trunk at least, I haven't tried 2.7.1 yet). However, I definitely see
some problems in LTP. For me it eventually hangs on the rename06 test,
but valgrind is complaining about some memory problems in the
pvfs2-client-core symlink path as well before reaching that point.
I'll look into it and let you know what turns up.
I might also see if I can cook up a script to include LTP in our nightly
tinderbox builds as well; there are some good tests in here.
-Phil
Bart Taylor wrote:
Hello all,
I have run into a few problems when running some tests against the
2.7.1 release on RHEL4. They all seem to hinge around large file names
of 256 characters or greater. The commands fail gracefully - reporting
an error that the file names are too long - but the next access of
PVFS (an 'ls' for instance) hangs the system so that it has to be
rebooted. I ran into them using the LTP test suite, but they can be
reliably repeated with simple commands. Create, open, rename, and stat
are the operations that fail, and some example commands are provided
below. The file does not need to exist for the open or stat commands.
Occasionally it takes a few seconds after command has been executed
for the problem to show itself. Do these commands cause problems for
anyone else, or is it something unique to my setup or environment?
Bart.
create: $echo "Hello World" >
1234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456
open: $cat
1234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456
rename: $mv test.file
1234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456
stat: $stat -f
/mnt/pvfs2/test/1234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123456
------------------------------------------------------------------------
_______________________________________________
Pvfs2-developers mailing list
[email protected]
http://www.beowulf-underground.org/mailman/listinfo/pvfs2-developers
_______________________________________________
Pvfs2-developers mailing list
[email protected]
http://www.beowulf-underground.org/mailman/listinfo/pvfs2-developers
---------------------
PatchSet 6937
Date: 2008/06/30 09:28:04
Author: pcarns
Branch: HEAD
Tag: (none)
Log:
backing out i_mutex lock within d_revalidate due to deadlock issues
Members:
src/kernel/linux-2.6/dcache.c:1.41->1.42
Index: pvfs2-1/src/kernel/linux-2.6/dcache.c
diff -u pvfs2-1/src/kernel/linux-2.6/dcache.c:1.41 pvfs2-1/src/kernel/linux-2.6/dcache.c:1.42
--- pvfs2-1/src/kernel/linux-2.6/dcache.c:1.41 Fri Apr 11 13:05:27 2008
+++ pvfs2-1/src/kernel/linux-2.6/dcache.c Mon Jun 30 09:28:04 2008
@@ -109,9 +109,7 @@
gossip_debug(GOSSIP_DCACHE_DEBUG,
"%s: doing getattr: inode: %p, handle: %llu)\n",
__func__, inode, llu(get_handle_from_ino(inode)));
- pvfs2_inode_lock(inode);
ret = pvfs2_inode_getattr(inode, PVFS_ATTR_SYS_ALL_NOHINT);
- pvfs2_inode_unlock(inode);
gossip_debug(GOSSIP_DCACHE_DEBUG,
"%s: getattr %s (ret = %d), returning %s for dentry\n",
__func__,
_______________________________________________
Pvfs2-developers mailing list
[email protected]
http://www.beowulf-underground.org/mailman/listinfo/pvfs2-developers