On 12/17/2010 08:38 PM, Jeff Layton wrote: > The client should not be timing out on individual SMB requests. Too much > of the state between client and server is tied to the state of the > socket. If we time out requests and issue spurious disconnects then that > comprimises data integrity. > > Instead of doing this complicated dance where we try to decide how long > to wait for a response for particular requests, have the client instead > wait indefinitely for a response. Also, use a TASK_KILLABLE sleep here > so that fatal signals will break out of this waiting. > > Later patches will add support for detecting dead peers and forcing > reconnects based on that. > > Signed-off-by: Jeff Layton <jlayton@xxxxxxxxxx> > --- > fs/cifs/transport.c | 110 ++++++++------------------------------------------- > 1 files changed, 17 insertions(+), 93 deletions(-) > Looks good to me. Reviewed-by: Suresh Jayaraman <sjayaraman@xxxxxxx> -- 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
