2018-04-09 1:06 GMT-07:00 Ronnie Sahlberg <lsahlber@xxxxxxxxxx>:
> and get rid of some more calls to get_rfc1002_length()
>
> Signed-off-by: Ronnie Sahlberg <lsahlber@xxxxxxxxxx>
> ---
> fs/cifs/cifsglob.h | 1 +
> fs/cifs/connect.c | 1 +
> fs/cifs/smb2transport.c | 2 +-
> fs/cifs/transport.c | 2 +-
> 4 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/fs/cifs/cifsglob.h b/fs/cifs/cifsglob.h
> index 92ba96fae09e..0bdcadda762d 100644
> --- a/fs/cifs/cifsglob.h
> +++ b/fs/cifs/cifsglob.h
> @@ -1375,6 +1375,7 @@ struct mid_q_entry {
> mid_handle_t *handle; /* call handle mid callback */
> void *callback_data; /* general purpose pointer for callback */
> void *resp_buf; /* pointer to received SMB header */
> + unsigned int resp_buf_size;
> int mid_state; /* wish this were enum but can not pass to wait_event */
> unsigned int mid_flags;
> __le16 command; /* smb command code */
> diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c
> index 48012a0addf1..e8830f076a7f 100644
> --- a/fs/cifs/connect.c
> +++ b/fs/cifs/connect.c
> @@ -928,6 +928,7 @@ cifs_demultiplex_thread(void *p)
>
> server->lstrp = jiffies;
> if (mid_entry != NULL) {
> + mid_entry->resp_buf_size = server->pdu_size;
> if ((mid_entry->mid_flags & MID_WAIT_CANCELLED) &&
> mid_entry->mid_state == MID_RESPONSE_RECEIVED &&
> server->ops->handle_cancelled_mid)
> diff --git a/fs/cifs/smb2transport.c b/fs/cifs/smb2transport.c
> index bf49cb73b9e6..8806f3f76c1d 100644
> --- a/fs/cifs/smb2transport.c
> +++ b/fs/cifs/smb2transport.c
> @@ -604,7 +604,7 @@ int
> smb2_check_receive(struct mid_q_entry *mid, struct TCP_Server_Info *server,
> bool log_error)
> {
> - unsigned int len = get_rfc1002_length(mid->resp_buf);
> + unsigned int len = mid->resp_buf_size;
> struct kvec iov[2];
> struct smb_rqst rqst = { .rq_iov = iov,
> .rq_nvec = 2 };
> diff --git a/fs/cifs/transport.c b/fs/cifs/transport.c
> index 279718dcb2ed..8f6f25918229 100644
> --- a/fs/cifs/transport.c
> +++ b/fs/cifs/transport.c
> @@ -790,7 +790,7 @@ cifs_send_recv(const unsigned int xid, struct cifs_ses *ses,
>
> buf = (char *)midQ->resp_buf;
> resp_iov->iov_base = buf;
> - resp_iov->iov_len = get_rfc1002_length(buf) +
> + resp_iov->iov_len = midQ->resp_buf_size +
> ses->server->vals->header_preamble_size;
> if (midQ->large_buf)
> *resp_buf_type = CIFS_LARGE_BUFFER;
> --
> 2.13.3
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
Reviewed-by: Pavel Shilovsky <pshilov@xxxxxxxxxxxxx>
--
Best regards,
Pavel Shilovsky
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
