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 <[email protected]> > --- > fs/cifs/transport.c | 110 > ++++++++------------------------------------------- > 1 files changed, 17 insertions(+), 93 deletions(-) >
Looks good to me. Reviewed-by: Suresh Jayaraman <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
