2011/10/4 Jeff Layton <jlayton@xxxxxxxxxx>: > On Tue, 4 Oct 2011 13:04:19 +0400 > Pavel Shilovsky <piastryyy@xxxxxxxxx> wrote: > >> 2011/8/27 Pavel Shilovsky <piastryyy@xxxxxxxxx>: >> > 2011/8/26 Jeff Layton <jlayton@xxxxxxxxxx>: >> >> This patchset does a fairly major cleanup and overhaul of the receive >> >> codepath for cifs. Aside from basic cleanup, there are two main goals: >> >> >> >> 1) allow the receive codepath to identify the mid before receiving all >> >> of the data for a particular SMB. >> >> >> >> ...and in turn... >> >> >> >> 2) add the ability for certain calls to receive their responses into >> >> their into their own set of buffers. >> >> >> >> These two changes allow cifs to break the CIFSMaxBufSize barrier for >> >> receives. This patchset just adds the necessary infrastructure to do >> >> the above. >> >> >> >> A separate patchset will follow that will overhaul cifs_readpages to use >> >> this infrastructure to allow for a larger rsize. >> >> >> >> Jeff Layton (16): >> >> cifs: clean up checkSMB >> >> cifs: consolidate signature generating code >> >> cifs: trivial: remove obsolete comment >> >> cifs: make smb_msg local to read_from_socket >> >> cifs: check for unresponsive server every time we call kernel_recvmsg >> >> cifs: simplify read_from_socket >> >> cifs: turn read_from_socket into a wrapper around a vectorized >> >> version >> >> cifs: keep a reusable kvec array for receives >> >> cifs: clean up check_rfc1002_header >> >> cifs: add a third receive phase to cifs_demultiplex_thread >> >> cifs: move mid finding into separate routine >> >> cifs: eliminate is_multi_rsp parm to find_cifs_mid >> >> cifs: move buffer pointers into TCP_Server_Info >> >> cifs: find mid earlier in receive codepath >> >> cifs: break out 3rd receive phase into separate function >> >> cifs: add a callback function to receive the rest of the frame >> >> >> >> fs/cifs/cifsencrypt.c | 103 ++-------- >> >> fs/cifs/cifsglob.h | 29 +++- >> >> fs/cifs/cifsproto.h | 7 +- >> >> fs/cifs/cifssmb.c | 5 +- >> >> fs/cifs/connect.c | 519 ++++++++++++++++++++++++++++--------------------- >> >> fs/cifs/misc.c | 51 +++--- >> >> fs/cifs/transport.c | 16 +- >> >> 7 files changed, 391 insertions(+), 339 deletions(-) >> >> >> >> -- >> >> 1.7.6 >> >> >> >> -- >> >> 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 >> >> >> > >> > The patchset looks good to me at the first glance. I am going to look >> > at this more carefully and test it in a few days. >> > >> > -- >> > Best regards, >> > Pavel Shilovsky. >> > >> >> Sorry for the long delay on this. As patchwork.kernel.org isn't >> available now, can you point me to public git repo, where I can fetch >> that changes, please? >> > > > Yes, I've moved my git repo to samba.org for the time being: > > http://git.samba.org/?p=jlayton/linux.git;a=summary > > See the "cifs-3.2" branch. > > -- > Jeff Layton <jlayton@xxxxxxxxxx> > I successfully tested this - the performance was closed to write one in my case - about 11.5 MB on 100Mbit LAN against Windows 7 server. Seems like a very good work! -- 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
