Re: [PATCH] CIFS: Remove some obsolete comments

[Sachin Prabhu <sprabhu-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]

On Wed, 2016-05-18 at 20:57 -0500, Steve French wrote:
> Remove some obsolete comments in the cifs inode_operations
> structs that were pointed out by Stephen Rothwell.
> 
> CC: Stephen Rothwell <sfr-3FnU+UHB4dNDw9hX6IcOSA@public.gmane.org>
> CC: Al Viro <viro-RmSDqhL/yNMiFSDQTTA3OLVCufUGDwFn@public.gmane.org>
> Signed-off-by: Steve French <steve.french-7I+n7zu2hftEKMMhf/gKZA@public.gmane.org>

Acked-By: Sachin Prabhu <sprabhu-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

> ---
>  fs/cifs/cifsfs.c | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/fs/cifs/cifsfs.c b/fs/cifs/cifsfs.c
> index 67f622d..5d8b7ed 100644
> --- a/fs/cifs/cifsfs.c
> +++ b/fs/cifs/cifsfs.c
> @@ -890,7 +890,6 @@ const struct inode_operations cifs_dir_inode_ops
> = {
>  	.rmdir = cifs_rmdir,
>  	.rename2 = cifs_rename2,
>  	.permission = cifs_permission,
> -/*	revalidate:cifs_revalidate,   */
>  	.setattr = cifs_setattr,
>  	.symlink = cifs_symlink,
>  	.mknod   = cifs_mknod,
> @@ -901,9 +900,8 @@ const struct inode_operations cifs_dir_inode_ops
> = {
>  };
>  
>  const struct inode_operations cifs_file_inode_ops = {
> -/*	revalidate:cifs_revalidate, */
>  	.setattr = cifs_setattr,
> -	.getattr = cifs_getattr, /* do we need this anymore? */
> +	.getattr = cifs_getattr,
>  	.permission = cifs_permission,
>  	.setxattr = generic_setxattr,
>  	.getxattr = generic_getxattr,
> @@ -915,9 +913,6 @@ const struct inode_operations
> cifs_symlink_inode_ops = {
>  	.readlink = generic_readlink,
>  	.get_link = cifs_get_link,
>  	.permission = cifs_permission,
> -	/* BB add the following two eventually */
> -	/* revalidate: cifs_revalidate,
> -	   setattr:    cifs_notify_change, *//* BB do we need notify
> change */
>  	.setxattr = generic_setxattr,
>  	.getxattr = generic_getxattr,
>  	.listxattr = cifs_listxattr,