Signed-off-by: Steve French <sfrench@us.ibm.com>
This commit is contained in:
parent
17cbbafe8e
commit
eb9bdaa3f3
1 changed files with 7 additions and 1 deletions
|
@ -1190,7 +1190,6 @@ static int cifs_writepages(struct address_space *mapping,
|
||||||
/* BB what if continued retry is
|
/* BB what if continued retry is
|
||||||
requested via mount flags? */
|
requested via mount flags? */
|
||||||
set_bit(AS_EIO, &mapping->flags);
|
set_bit(AS_EIO, &mapping->flags);
|
||||||
SetPageError(page);
|
|
||||||
} else {
|
} else {
|
||||||
cifs_stats_bytes_written(cifs_sb->tcon,
|
cifs_stats_bytes_written(cifs_sb->tcon,
|
||||||
bytes_written);
|
bytes_written);
|
||||||
|
@ -1198,6 +1197,13 @@ static int cifs_writepages(struct address_space *mapping,
|
||||||
}
|
}
|
||||||
for (i = 0; i < n_iov; i++) {
|
for (i = 0; i < n_iov; i++) {
|
||||||
page = pvec.pages[first + i];
|
page = pvec.pages[first + i];
|
||||||
|
/* Should we also set page error on
|
||||||
|
success rc but too little data written? */
|
||||||
|
/* BB investigate retry logic on temporary
|
||||||
|
server crash cases and how recovery works
|
||||||
|
when page marked as error */
|
||||||
|
if(rc)
|
||||||
|
SetPageError(page);
|
||||||
kunmap(page);
|
kunmap(page);
|
||||||
unlock_page(page);
|
unlock_page(page);
|
||||||
page_cache_release(page);
|
page_cache_release(page);
|
||||||
|
|
Loading…
Reference in a new issue