Re: [PATCH] Fix NLM reference count panic

[Wendy Cheng <wcheng@redhat.com>]

Forgot the signed-off line .. here it is ...
Signed-off-by: S. Wendy Cheng <wcheng@redhat.com>
>
> ------------------------------------------------------------------------
>
> This fixes the incorrect fclose call inside nlm_traverse_files() where 
> a posix lock could still be held by NFS client. Problem was found in a
> kernel panic inside locks_remove_flock() (fs/locks.c:2034) as part of 
> the fclose call due to NFS-NLM locks still hanging on inode->i_flock list. 
>
> Also see: http://people.redhat.com/wcheng/Patches/NFS/NLM/001.txt
>
>  svcsubs.c |    3 +--
>  1 files changed, 1 insertion(+), 2 deletions(-)
>
> --- gfs2-nmw/fs/lockd/svcsubs.c	2007-04-10 11:59:09.000000000 -0400
> +++ linux/fs/lockd/svcsubs.c	2007-04-18 10:01:23.000000000 -0400
> @@ -250,8 +250,7 @@ nlm_traverse_files(struct nlm_host *host
>  			mutex_lock(&nlm_file_mutex);
>  			file->f_count--;
>  			/* No more references to this file. Let go of it. */
> -			if (list_empty(&file->f_blocks) && !file->f_locks
> -			 && !file->f_shares && !file->f_count) {
> +			if (!nlm_file_inuse(file)) {
>  				hlist_del(&file->f_list);
>  				nlmsvc_ops->fclose(file->f_file);
>  				kfree(file);
>