]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
smb3: remove unused flag passed into close functions
authorSteve French <stfrench@microsoft.com>
Mon, 2 Dec 2019 23:55:41 +0000 (17:55 -0600)
committerSteve French <stfrench@microsoft.com>
Tue, 3 Dec 2019 00:07:17 +0000 (18:07 -0600)
close was relayered to allow passing in an async flag which
is no longer needed in this path.  Remove the unneeded parameter
"flags" passed in on close.

Signed-off-by: Steve French <stfrench@microsoft.com>
Reviewed-by: Pavel Shilovsky <pshilov@microsoft.com>
Reviewed-by: Ronnie Sahlberg <lsahlber@redhat.com>
fs/cifs/smb2pdu.c
fs/cifs/smb2proto.h

index be0de8a63e57601b00c0bf836898a55c0e7f2df5..cec53eb8a6daf1bd71eb7010a37640b16201514e 100644 (file)
@@ -2959,8 +2959,8 @@ SMB2_close_free(struct smb_rqst *rqst)
 }
 
 int
-SMB2_close_flags(const unsigned int xid, struct cifs_tcon *tcon,
-                u64 persistent_fid, u64 volatile_fid, int flags)
+SMB2_close(const unsigned int xid, struct cifs_tcon *tcon,
+                u64 persistent_fid, u64 volatile_fid)
 {
        struct smb_rqst rqst;
        struct smb2_close_rsp *rsp = NULL;
@@ -2969,6 +2969,7 @@ SMB2_close_flags(const unsigned int xid, struct cifs_tcon *tcon,
        struct kvec rsp_iov;
        int resp_buftype = CIFS_NO_BUFFER;
        int rc = 0;
+       int flags = 0;
 
        cifs_dbg(FYI, "Close\n");
 
@@ -3007,27 +3008,17 @@ SMB2_close_flags(const unsigned int xid, struct cifs_tcon *tcon,
 close_exit:
        SMB2_close_free(&rqst);
        free_rsp_buf(resp_buftype, rsp);
-       return rc;
-}
-
-int
-SMB2_close(const unsigned int xid, struct cifs_tcon *tcon,
-          u64 persistent_fid, u64 volatile_fid)
-{
-       int rc;
-       int tmp_rc;
-
-       rc = SMB2_close_flags(xid, tcon, persistent_fid, volatile_fid, 0);
 
        /* retry close in a worker thread if this one is interrupted */
        if (rc == -EINTR) {
+               int tmp_rc;
+
                tmp_rc = smb2_handle_cancelled_close(tcon, persistent_fid,
                                                     volatile_fid);
                if (tmp_rc)
                        cifs_dbg(VFS, "handle cancelled close fid 0x%llx returned error %d\n",
                                 persistent_fid, tmp_rc);
        }
-
        return rc;
 }
 
index d21a5fcc8d060f6e86014e9404110a37f50df9d1..5caeaa99cb7c4d5f1b1116351853738eea26a9a1 100644 (file)
@@ -157,8 +157,6 @@ extern int SMB2_change_notify(const unsigned int xid, struct cifs_tcon *tcon,
 
 extern int SMB2_close(const unsigned int xid, struct cifs_tcon *tcon,
                      u64 persistent_file_id, u64 volatile_file_id);
-extern int SMB2_close_flags(const unsigned int xid, struct cifs_tcon *tcon,
-                           u64 persistent_fid, u64 volatile_fid, int flags);
 extern int SMB2_close_init(struct cifs_tcon *tcon, struct smb_rqst *rqst,
                      u64 persistent_file_id, u64 volatile_file_id);
 extern void SMB2_close_free(struct smb_rqst *rqst);