Re: [Outreachy kernel] [PATCH] Staging: lustre: Remove unneeded variable and replace its uses

[Greg KH <gregkh@linuxfoundation.org>]

On Tue, Mar 10, 2015 at 07:12:42PM +0530, Vatika Harlalka wrote:
> Variable dotdot is only used as a function parameter and
> can be replaced there directly to improve code compactness.
> 
> Signed-off-by: Vatika Harlalka <vatikaharlalka@gmail.com>
> ---
>  drivers/staging/lustre/lustre/llite/llite_nfs.c | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/staging/lustre/lustre/llite/llite_nfs.c b/drivers/staging/lustre/lustre/llite/llite_nfs.c
> index 6a67e71..b8ced3d 100644
> --- a/drivers/staging/lustre/lustre/llite/llite_nfs.c
> +++ b/drivers/staging/lustre/lustre/llite/llite_nfs.c
> @@ -287,7 +287,6 @@ static struct dentry *ll_get_parent(struct dentry *dchild)
>  	struct ll_sb_info     *sbi;
>  	struct dentry	 *result = NULL;
>  	struct mdt_body       *body;
> -	static const char	   dotdot[] = "..";
>  	struct md_op_data     *op_data;
>  	int		   rc;
>  	int		      lmmsize;
> @@ -303,8 +302,8 @@ static struct dentry *ll_get_parent(struct dentry *dchild)
>  	if (rc != 0)
>  		return ERR_PTR(rc);
>  
> -	op_data = ll_prep_md_op_data(NULL, dir, NULL, dotdot,
> -				     strlen(dotdot), lmmsize,
> +	op_data = ll_prep_md_op_data(NULL, dir, NULL, "..",
> +				     2, lmmsize,
>  				     LUSTRE_OPC_ANY, NULL);
>  	if (IS_ERR(op_data))
>  		return (void *)op_data;
> @@ -321,7 +320,7 @@ static struct dentry *ll_get_parent(struct dentry *dchild)
>  	CDEBUG(D_INFO, "parent for "DFID" is "DFID"\n",
>  		PFID(ll_inode2fid(dir)), PFID(&body->fid1));
>  
> -	result = ll_iget_for_nfs(dir->i_sb, &body->fid1, NULL);
> +	

Why did you delete this whole function call?  That does not seem to
match up with what you said this patch did.

I can't take this as-is, sorry.

greg k-h