Re: [PATCH 1/2] vfs: make real_lookup do dentry revalidation with i_mutex held

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 25, 2009 at 01:52:32PM +0800, Ian Kent wrote:
> Sage Weil wrote:
> > 
> > As you can see I haven't spent much time looking at this code.  :)  I'm 
> > happy to do so, though, or spend some time testing to get this resolved.. 
> > just let me know how I can help.
> 
> You can't just yet as it's completely different from the original patch.
> Let me get this to a sensible point then we can both go over it in detail.

Just thought I'd let you know how this is going.

I had to stop work on this for a few weeks becuase it was driving me crazy.
The good news is that after returning to it I think I've got it working, at
least mostly.

The testing is lengthy as I've had to increase the number of iterations
significantly to investigate the failures. Also, I've focused on the most
problematic configuration so there may still be difficulties as the testing
goes further.

The test I'm using has two basic configurations, one is for automount maps
using the autofs pseudo mount option "nobrowse" which just means mount
points are created at mount time. The other is with the pseudo option
"browse" that just means mount point directories are pre-created. For
these two configurations there are various sub-cases so the directory
pre-creation is not always done. But that's detail.

The current state is that I'm not seeing failures from the kernel module
with the "browse" configuration, which was the main source of difficulty
to begin with. One remaining issue is that, by the time I reach around
2000 iterations, I'm seeing an increase of just over 1 second in around
28 seconds, per iteration. I'm not sure if this is a real probem or not.

Anyway, I'm hopeing the "nobrowse" configuration testing will not show
further problems. 

So, it's probably time to post my patch series to get some more eyes
looking at them. Who on the recipient list of this mail should I include
for the post?

Ian
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux