<br><br>
<div class="gmail_quote">On Fri, Sep 11, 2009 at 8:43 AM, Jeff Layton <span dir="ltr"><<a href="mailto:jlayton@redhat.com">jlayton@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">
<div>
<div></div>
<div class="h5">On Wed, 9 Sep 2009 03:40:13 -0500<br>Shirish Pargaonkar <<a href="mailto:shirishpargaonkar@gmail.com">shirishpargaonkar@gmail.com</a>> wrote:<br><br>> - Enable such that partial/shorts sends can be resent/retried_for_send<br>
> - Keep the midQ entry by marking it for resend/retry_sending in case<br>> of EAGAIN error<br>><br>><br>> diff --git a/fs/cifs/transport.c b/fs/cifs/transport.c<br>> index 1da4ab2..92fa1ad 100644<br>> --- a/fs/cifs/transport.c<br>
> +++ b/fs/cifs/transport.c<br>> @@ -269,6 +269,7 @@ smb_sendv(struct TCP_Server_Info *server, struct<br>> kvec *iov, int n_vec)<br>> to kill the socket so the server throws away the partial<br>
> SMB */<br>> server->tcpStatus = CifsNeedReconnect;<br>> + rc = -EAGAIN;<br>> }<br>><br>> if (rc < 0) {<br>> @@ -505,8 +506,13 @@ SendReceive2(const unsigned int xid, struct<br>
> cifsSesInfo *ses,<br>> mutex_unlock(&ses->server->srv_mutex);<br>> cifs_small_buf_release(in_buf);<br>><br>> - if (rc < 0)<br>> - goto out;<br>> + if (rc < 0) {<br>
> + if (rc == -EAGAIN) {<br>> + midQ->midState = MID_RETRY_NEEDED;<br>> + goto outagain;<br>> + } else<br>> + goto out;<br>
> + }<br>><br>> if (long_op == CIFS_STD_OP)<br>> timeout = 15 * HZ;<br>> @@ -623,6 +629,7 @@ SendReceive2(const unsigned int xid, struct<br>> cifsSesInfo *ses,<br>><br>> out:<br>
> DeleteMidQEntry(midQ);<br>> +outagain:<br>> atomic_dec(&ses->server->inFlight);<br>> wake_up(&ses->server->request_q);<br>><br>> @@ -697,8 +704,13 @@ SendReceive(const unsigned int xid, struct<br>
> cifsSesInfo *ses,<br>> #endif<br>> mutex_unlock(&ses->server->srv_mutex);<br>><br>> - if (rc < 0)<br>> - goto out;<br>> + if (rc < 0) {<br>> + if (rc == -EAGAIN) {<br>
> + midQ->midState = MID_RETRY_NEEDED;<br>> + goto outagain;<br>> + } else<br>> + goto out;<br>> + }<br>><br>> if (long_op == CIFS_STD_OP)<br>
> timeout = 15 * HZ;<br>> @@ -807,6 +819,7 @@ SendReceive(const unsigned int xid, struct cifsSesInfo *ses,<br>><br>> out:<br>> DeleteMidQEntry(midQ);<br>> +outagain:<br>> atomic_dec(&ses->server->inFlight);<br>
> wake_up(&ses->server->request_q);<br><br></div></div>Patch looks reasonably sane. Could you outline what problem this is<br>intended to solve?<br><br>Also what about SendReceiveBlockingLock? Doesn't it need a similar<br>
change?<br><font color="#888888"><br>--<br>Jeff Layton <<a href="mailto:jlayton@redhat.com">jlayton@redhat.com</a>><br></font></blockquote></div>
<div> </div>
<div> </div>
<div>We were abandoing the command that was partially sent, now a partially sent</div>
<div>command, on reconnect, will be treated like other commands that timed out</div>
<div>and get marked for retry.</div>
<div> </div>
<div><br> </div>