From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Layton Subject: [PATCH 08/11] cifs: fix handling of signing with writepages Date: Tue, 20 Apr 2010 16:07:16 -0400 Message-ID: <1271794039-22787-9-git-send-email-jlayton@redhat.com> References: <1271794039-22787-1-git-send-email-jlayton@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: smfrench@gmail.com To: linux-cifs-client@lists.samba.org, linux-fsdevel@vger.kernel.org Return-path: In-Reply-To: <1271794039-22787-1-git-send-email-jlayton@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-cifs-client-bounces@lists.samba.org Errors-To: linux-cifs-client-bounces@lists.samba.org List-Id: linux-fsdevel.vger.kernel.org Get a reference to the file early so we can base the decision about signing on the correct tcon. If that doesn't work for some reason, then fall back to generic_writepages. That's just as likely to fail, but it simplifies the error handling. In truth, I'm not sure how that could occur anyway, so maybe a NULL open_file here ought to be a BUG()? Signed-off-by: Jeff Layton --- fs/cifs/file.c | 85 ++++++++++++++++++++++++++----------------------------- 1 files changed, 40 insertions(+), 45 deletions(-) diff --git a/fs/cifs/file.c b/fs/cifs/file.c index b56913a..12495c7 100644 --- a/fs/cifs/file.c +++ b/fs/cifs/file.c @@ -1398,8 +1398,16 @@ static int cifs_writepages(struct address_space *mapping, int scanned = 0; int xid, long_op; + /* + * BB: Is this meaningful for a non-block-device file system? + * If it is, we should test it again after we do I/O + */ + if (wbc->nonblocking && bdi_write_congested(bdi)) { + wbc->encountered_congestion = 1; + return 0; + } + cifs_sb = CIFS_SB(mapping->host->i_sb); - tcon = cifs_sb_master_tcon(cifs_sb); /* * If wsize is smaller that the page cache size, default to writing @@ -1408,25 +1416,26 @@ static int cifs_writepages(struct address_space *mapping, if (cifs_sb->wsize < PAGE_CACHE_SIZE) return generic_writepages(mapping, wbc); - /* FIXME: not quite right -- we need to determine this once we know - * what tcon we're going to use. - */ - if (tcon->ses->sign && !experimEnabled) - return generic_writepages(mapping, wbc); - iov = kmalloc(32 * sizeof(struct kvec), GFP_KERNEL); if (iov == NULL) return generic_writepages(mapping, wbc); - /* - * BB: Is this meaningful for a non-block-device file system? - * If it is, we should test it again after we do I/O + * if there's no open file, then this is likely to fail too, + * but it'll at least handle the return. Maybe it should be + * a BUG() instead? */ - if (wbc->nonblocking && bdi_write_congested(bdi)) { - wbc->encountered_congestion = 1; + open_file = find_writable_file(CIFS_I(mapping->host), false); + if (!open_file) { kfree(iov); - return 0; + return generic_writepages(mapping, wbc); + } + + tcon = open_file->tcon; + if (tcon->ses->sign && !experimEnabled) { + kfree(iov); + cifsFileInfo_put(open_file); + return generic_writepages(mapping, wbc); } xid = GetXid(); @@ -1532,39 +1541,24 @@ retry: break; } if (n_iov) { - /* Search for a writable handle every time we call - * CIFSSMBWrite2. We can't rely on the last handle - * we used to still be valid - */ - open_file = find_writable_file(CIFS_I(mapping->host), - false); - if (!open_file) { - cERROR(1, ("No writable handles for inode")); - rc = -EBADF; + long_op = cifs_write_timeout(cifsi, offset); + rc = CIFSSMBWrite2(xid, tcon, open_file->netfid, + bytes_to_write, offset, + &bytes_written, iov, n_iov, + long_op); + cifs_update_eof(cifsi, offset, bytes_written); + + if (rc || bytes_written < bytes_to_write) { + cERROR(1, ("Write2 ret %d, wrote %d", + rc, bytes_written)); + /* BB what if continued retry is + requested via mount flags? */ + if (rc == -ENOSPC) + set_bit(AS_ENOSPC, &mapping->flags); + else + set_bit(AS_EIO, &mapping->flags); } else { - tcon = open_file->tcon; - long_op = cifs_write_timeout(cifsi, offset); - rc = CIFSSMBWrite2(xid, tcon, - open_file->netfid, - bytes_to_write, offset, - &bytes_written, iov, n_iov, - long_op); - cifsFileInfo_put(open_file); - cifs_update_eof(cifsi, offset, bytes_written); - - if (rc || bytes_written < bytes_to_write) { - cERROR(1, ("Write2 ret %d, wrote %d", - rc, bytes_written)); - /* BB what if continued retry is - requested via mount flags? */ - if (rc == -ENOSPC) - set_bit(AS_ENOSPC, &mapping->flags); - else - set_bit(AS_EIO, &mapping->flags); - } else { - cifs_stats_bytes_written(tcon, - bytes_written); - } + cifs_stats_bytes_written(tcon, bytes_written); } for (i = 0; i < n_iov; i++) { page = pvec.pages[first + i]; @@ -1601,6 +1595,7 @@ retry: if (wbc->range_cyclic || (range_whole && wbc->nr_to_write > 0)) mapping->writeback_index = index; + cifsFileInfo_put(open_file); FreeXid(xid); kfree(iov); return rc; -- 1.6.6.1