RE: [Ext2-devel] Re: ext3 htree brelse problems look to be fixed!

[Christopher Li]

I post a patch for comment on ext2-devel for the
NFS cookie bug.  Did not get any feedback yet.
As Ted suggested, it set the cookie to -1 on EOF,
even though it is not seek able to there.

I only test it with Stephen's "readdir.c".
Not have chance to run it on a NFS server yet.

Do you have more information about the cache trashing
bug?

Regards,
Chris

===== dir.c 1.5 vs edited =====
--- 1.5/fs/ext3/dir.c	Wed Oct  2 01:24:11 2002
+++ edited/dir.c	Sat Mar  1 23:45:04 2003
@@ -450,8 +450,10 @@
 						   &info->next_hash);
 			if (ret < 0)
 				return ret;
-			if (ret == 0)
+			if (ret == 0) {
+				filp->f_pos = -1;
 				break;
+			}
 			info->curr_node = rb_get_first(&info->root);
 		}
 

> -----Original Message-----
> From: Daniel Phillips [mailto:[e-mail]]
> Sent: Wednesday, March 05, 2003 12:25 AM
> To: James H. Cloos Jr.; [e-mail]
> Cc: [e-mail]; [e-mail]
> Subject: [Ext2-devel] Re: ext3 htree brelse problems look to be fixed!
> 
> 
> On Wed 05 Mar 03 00:57, James H. Cloos Jr. wrote:
> > I beleive (with this patch) htree is now ready for prime time.
> 
> Good that it's working for you, but it's not quite the last 
> issue.  There is 
> some apparent cache thrashing to track down, and I believe 
> there's still an 
> outstanding NFS issue.  It's getting there, though.
> 
> Regards,
> 
> Daniel
> 
> 
> -------------------------------------------------------
> This SF.net email is sponsored by: Etnus, makers of 
> TotalView, The debugger 
> for complex code. Debugging C/C++ programs can leave you 
> feeling lost and 
> disoriented. TotalView can help you find your way. Available 
> on major UNIX 
> and Linux platforms. Try it free. www.etnus.com
> _______________________________________________
> Ext2-devel mailing list
> [e-mail]
> https://lists.sourceforge.net/lists/listinfo/ext2-devel
>