Re: [PATCH 0/8] cifs: patches for 2.6.35

On 06/01/2010 08:24 PM, Jeff Layton wrote:
> Hi Steve,
> 
> The following is a comprehensive set of patches that I have queued up
> for 2.6.35. They fix a number of important bugs, including:
> 
> - a page refcount leak introduced in a recent patch by Nick P.
> 
> - a long-standing bug involving busy-file renames
> 
> - a set of fixes for all known "Busy inodes after umount..." problems
>   (see https://bugzilla.samba.org/show_bug.cgi?id=7433)
> 
> It also contains the "drop_inode" patch which helps reduce memory
> utilization when server inode numbers aren't used.
> 
> It may be easiest to pull these from my git repo on kernel.org. Pull
> request follows:
> 
> The following changes since commit 67a3e12b05e055c0415c556a315a3d3eb637e29e:
>   Linus Torvalds (1):
>         Linux 2.6.35-rc1
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/jlayton/linux.git for-sfrench
> 
> Jeff Layton (7):
>       cifs: fix page refcount leak
>       cifs: don't attempt busy-file rename unless it's in same directory
>       cifs: implement drop_inode superblock op
>       cifs: move cifs_new_fileinfo call out of cifs_posix_open
>       cifs: pass instantiated filp back after open call
>       cifs: clean up arguments to cifs_open_inode_helper
>       cifs: don't call cifs_new_fileinfo unless cifs_open succeeds

I have reviewed and tested [4/8], [5/8] [6/8] and [8/8] and I could no
longer reproduce the "VFS busy inodes" errors using my reproducer and I
don't see the leaks anymore..

Reviewed-and-Tested-by: Suresh Jayaraman <[email protected]>


Thanks,

-- 
Suresh Jayaraman