On Wed, 16 Mar 2011 15:15:31 -0400 Jeff Layton <[email protected]> wrote:
> If a process has a dirty page mapped into its page tables, then it has > the ability to change it while the client is trying to write the data > out to the server. If that happens after the signature has been > calculated then that signature will then be wrong, and the server will > likely reset the TCP connection. > > This patch adds a page_mkwrite handler for CIFS that simply takes the > page lock. Because the page lock is held over the life of writepage and > writepages, this prevents the page from becoming writeable until > the write call has completed. > > With this, we can also remove the "sign_zero_copy" module option and > always inline the pages when writing. > > Signed-off-by: Jeff Layton <[email protected]> > --- > fs/cifs/cifsfs.c | 4 --- > fs/cifs/cifsglob.h | 1 - > fs/cifs/file.c | 62 > ++++++++++++++++++++++++++-------------------------- > 3 files changed, 31 insertions(+), 36 deletions(-) > Steve, can you let me know where we are with this patch? It's not merged and you haven't commented on it. -- Jeff Layton <[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
