diff mbox series

fs: cifs: Replace _free_xid call in cifs_root_iget function

Message ID 20180317210656.GA14436@pathfinder
State New
Headers show
Series fs: cifs: Replace _free_xid call in cifs_root_iget function | expand

Commit Message

Phillip Potter March 17, 2018, 9:06 p.m. UTC
Modify end of cifs_root_iget function in fs/cifs/inode.c to call
free_xid(xid) instead of _free_xid(xid), thereby allowing debug
notification of this action when enabled.

Signed-off-by: Phillip Potter <phil@philpotter.co.uk>
---
 fs/cifs/inode.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Comments

Steve French March 17, 2018, 9:14 p.m. UTC | #1
merged into cifs-2.6.git for-next

thx

On Sat, Mar 17, 2018 at 4:06 PM, Phillip Potter <phil@philpotter.co.uk> wrote:
> Modify end of cifs_root_iget function in fs/cifs/inode.c to call
> free_xid(xid) instead of _free_xid(xid), thereby allowing debug
> notification of this action when enabled.
>
> Signed-off-by: Phillip Potter <phil@philpotter.co.uk>
> ---
>  fs/cifs/inode.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/fs/cifs/inode.c b/fs/cifs/inode.c
> index 8f9a8cc7cc62..c98a852180b1 100644
> --- a/fs/cifs/inode.c
> +++ b/fs/cifs/inode.c
> @@ -1066,10 +1066,7 @@ struct inode *cifs_root_iget(struct super_block *sb)
>
>  out:
>         kfree(path);
> -       /* can not call macro free_xid here since in a void func
> -        * TODO: This is no longer true
> -        */
> -       _free_xid(xid);
> +       free_xid(xid);
>         return inode;
>  }
>
> --
> 2.14.3
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox series

Patch

diff --git a/fs/cifs/inode.c b/fs/cifs/inode.c
index 8f9a8cc7cc62..c98a852180b1 100644
--- a/fs/cifs/inode.c
+++ b/fs/cifs/inode.c
@@ -1066,10 +1066,7 @@  struct inode *cifs_root_iget(struct super_block *sb)
 
 out:
 	kfree(path);
-	/* can not call macro free_xid here since in a void func
-	 * TODO: This is no longer true
-	 */
-	_free_xid(xid);
+	free_xid(xid);
 	return inode;
 }