Re: [PATCH] cifs: implement drop_inode superblock op

Raja R Harinath <[email protected]> Wed, 26 May 2010 07:46:58 +0530
Newsgroups gmane.linux.file-systems,gmane.linux.file-systems.cifs
Message-ID <[email protected]>
Hi,

Jeff Layton <[email protected]> writes:

[snip]
>  static const struct super_operations cifs_super_ops = {
>  	.put_super = cifs_put_super,
>  	.statfs = cifs_statfs,
>  	.alloc_inode = cifs_alloc_inode,
>  	.destroy_inode = cifs_destroy_inode,
> -/*	.drop_inode	    = generic_delete_inode,
> -	.delete_inode	= cifs_delete_inode,  */  /* Do not need above two
> +	.drop_inode	= cifs_drop_inode,
> +/*	.delete_inode	= cifs_delete_inode,  */  /* Do not need above two
>  	functions unless later we add lazy close of inodes or unless the
>  	kernel forgets to call us with the same number of releases (closes)
>  	as opens */

I think the comment needs to be updated, at least to decrement the "two
functions".

- Hari

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