NFS: Don't corrupt the value of pg_bytes_written in nfs_do_recoalesce()
The value of mirror->pg_bytes_written should only be updated after a
successful attempt to flush out the requests on the list.
Fixes: a7d42ddb30
("nfs: add mirroring support to pgio layer")
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
This commit is contained in:
parent
56517ab958
commit
0d0ea30935
|
@ -1128,17 +1128,16 @@ static void nfs_pageio_doio(struct nfs_pageio_descriptor *desc)
|
|||
{
|
||||
struct nfs_pgio_mirror *mirror = nfs_pgio_current_mirror(desc);
|
||||
|
||||
|
||||
if (!list_empty(&mirror->pg_list)) {
|
||||
int error = desc->pg_ops->pg_doio(desc);
|
||||
if (error < 0)
|
||||
desc->pg_error = error;
|
||||
else
|
||||
if (list_empty(&mirror->pg_list)) {
|
||||
mirror->pg_bytes_written += mirror->pg_count;
|
||||
}
|
||||
if (list_empty(&mirror->pg_list)) {
|
||||
mirror->pg_count = 0;
|
||||
mirror->pg_base = 0;
|
||||
mirror->pg_count = 0;
|
||||
mirror->pg_base = 0;
|
||||
mirror->pg_recoalesce = 0;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -1228,7 +1227,6 @@ static int nfs_do_recoalesce(struct nfs_pageio_descriptor *desc)
|
|||
|
||||
do {
|
||||
list_splice_init(&mirror->pg_list, &head);
|
||||
mirror->pg_bytes_written -= mirror->pg_count;
|
||||
mirror->pg_count = 0;
|
||||
mirror->pg_base = 0;
|
||||
mirror->pg_recoalesce = 0;
|
||||
|
|
Loading…
Reference in New Issue