cifs: minor cleanup of some headers
authorSteve French <stfrench@microsoft.com>
Thu, 8 Dec 2022 22:11:00 +0000 (16:11 -0600)
committerSteve French <stfrench@microsoft.com>
Mon, 12 Dec 2022 19:08:06 +0000 (13:08 -0600)
checkpatch showed formatting problems with extra spaces,
and extra semicolon and some missing blank lines in some
cifs headers.

Reviewed-by: Paulo Alcantara (SUSE) <pc@cjr.nz>
Reviewed-by: Germano Percossi <germano.percossi@gmail.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
fs/cifs/cifs_ioctl.h
fs/cifs/cifsfs.h
fs/cifs/cifsglob.h

index d86d78d5bfdc1fea4d12b521cc6e37a3428b9009..332588e77c311fb4fde12a2a683e450b79cac6ab 100644 (file)
@@ -108,7 +108,7 @@ struct smb3_notify_info {
 #define CIFS_IOC_NOTIFY _IOW(CIFS_IOCTL_MAGIC, 9, struct smb3_notify)
 #define CIFS_DUMP_FULL_KEY _IOWR(CIFS_IOCTL_MAGIC, 10, struct smb3_full_key_debug_info)
 #define CIFS_IOC_NOTIFY_INFO _IOWR(CIFS_IOCTL_MAGIC, 11, struct smb3_notify_info)
-#define CIFS_IOC_SHUTDOWN _IOR ('X', 125, __u32)
+#define CIFS_IOC_SHUTDOWN _IOR('X', 125, __u32)
 
 /*
  * Flags for going down operation
index 388b745a978e21a26844661fdadab0a1bf1f87ba..00a573e0ad0e19acdab2f42e950d828bf4dd29a5 100644 (file)
@@ -105,8 +105,8 @@ extern int cifs_lock(struct file *, int, struct file_lock *);
 extern int cifs_fsync(struct file *, loff_t, loff_t, int);
 extern int cifs_strict_fsync(struct file *, loff_t, loff_t, int);
 extern int cifs_flush(struct file *, fl_owner_t id);
-extern int cifs_file_mmap(struct file * , struct vm_area_struct *);
-extern int cifs_file_strict_mmap(struct file * , struct vm_area_struct *);
+extern int cifs_file_mmap(struct file *file, struct vm_area_struct *vma);
+extern int cifs_file_strict_mmap(struct file *file, struct vm_area_struct *vma);
 extern const struct file_operations cifs_dir_ops;
 extern int cifs_dir_open(struct inode *inode, struct file *file);
 extern int cifs_readdir(struct file *file, struct dir_context *ctx);
index 1420acf987f03d3ff1ab63a16b15e260fe87f37f..cd3a173e65b1cf89e5bba4269c8b99d2cdc156f1 100644 (file)
@@ -785,6 +785,7 @@ static inline unsigned int
 in_flight(struct TCP_Server_Info *server)
 {
        unsigned int num;
+
        spin_lock(&server->req_lock);
        num = server->in_flight;
        spin_unlock(&server->req_lock);
@@ -795,6 +796,7 @@ static inline bool
 has_credits(struct TCP_Server_Info *server, int *credits, int num_credits)
 {
        int num;
+
        spin_lock(&server->req_lock);
        num = *credits;
        spin_unlock(&server->req_lock);
@@ -1025,7 +1027,7 @@ struct cifs_ses {
        struct TCP_Server_Info *server; /* pointer to server info */
        int ses_count;          /* reference counter */
        enum ses_status_enum ses_status;  /* updates protected by cifs_tcp_ses_lock */
-       unsigned overrideSecFlg;  /* if non-zero override global sec flags */
+       unsigned int overrideSecFlg; /* if non-zero override global sec flags */
        char *serverOS;         /* name of operating system underlying server */
        char *serverNOS;        /* name of network operating system of server */
        char *serverDomain;     /* security realm of server */
@@ -1381,7 +1383,7 @@ struct cifsFileInfo {
        __u32 pid;              /* process id who opened file */
        struct cifs_fid fid;    /* file id from remote */
        struct list_head rlist; /* reconnect list */
-       /* BB add lock scope info here if needed */ ;
+       /* BB add lock scope info here if needed */
        /* lock scope id (0 if none) */
        struct dentry *dentry;
        struct tcon_link *tlink;
@@ -1769,6 +1771,7 @@ static inline void free_dfs_info_array(struct dfs_info3_param *param,
                                       int number_of_items)
 {
        int i;
+
        if ((number_of_items == 0) || (param == NULL))
                return;
        for (i = 0; i < number_of_items; i++) {