cifs: don't allow mmap'ed pages to be dirtied while under writeback (try #3)
authorJeff Layton <jlayton@redhat.com>
Tue, 12 Apr 2011 13:13:44 +0000 (09:13 -0400)
committerSteve French <sfrench@us.ibm.com>
Tue, 12 Apr 2011 14:19:55 +0000 (14:19 +0000)
commitca83ce3d5b9ad321ee24f5870a77f0b21ac5a5de
tree7added4d982e03c89599ef7c78c9b69845a7504d
parentd9b942013730c38ac83564d6669c6d0ecf6d754d
cifs: don't allow mmap'ed pages to be dirtied while under writeback (try #3)

This is more or less the same patch as before, but with some merge
conflicts fixed up.

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 <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
fs/cifs/cifsfs.c
fs/cifs/cifsglob.h
fs/cifs/file.c