Commit 40c594b6 authored by Hyunchul Lee's avatar Hyunchul Lee Committed by Namjae Jeon
Browse files

cifsd: enclose macro variables in parenthesis 



checkpatch.pl complains as the following:

CHECK: Macro argument 'fp' may be better as '(fp)' to avoid
precedence issues.

Signed-off-by: default avatarHyunchul Lee <hyc.lee@gmail.com>
Signed-off-by: default avatarNamjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: default avatarSteve French <stfrench@microsoft.com>
parent 152de8c6

fs/cifsd/vfs_cache.h

+8 −8
Original line number Diff line number Diff line
@@ -25,14 +25,14 @@ 
#define KSMBD_NO_FID		(UINT_MAX)

#define SMB2_NO_FID		(0xFFFFFFFFFFFFFFFFULL)



#define FP_FILENAME(fp)		fp->filp->f_path.dentry->d_name.name

#define FP_INODE(fp)		d_inode(fp->filp->f_path.dentry)

#define PARENT_INODE(fp)	d_inode(fp->filp->f_path.dentry->d_parent)



#define ATTR_FP(fp) (fp->attrib_only && \

		(fp->cdoption != FILE_OVERWRITE_IF_LE && \

		fp->cdoption != FILE_OVERWRITE_LE && \

		fp->cdoption != FILE_SUPERSEDE_LE))

#define FP_FILENAME(fp)		((fp)->filp->f_path.dentry->d_name.name)

#define FP_INODE(fp)		d_inode((fp)->filp->f_path.dentry)

#define PARENT_INODE(fp)	d_inode((fp)->filp->f_path.dentry->d_parent)



#define ATTR_FP(fp) ((fp)->attrib_only && \

		     ((fp)->cdoption != FILE_OVERWRITE_IF_LE && \

		      (fp)->cdoption != FILE_OVERWRITE_LE && \

		      (fp)->cdoption != FILE_SUPERSEDE_LE))



struct ksmbd_conn;

struct ksmbd_session;