Commit 7e1ee54a authored by Yan, Zheng's avatar Yan, Zheng Committed by Ilya Dryomov

ceph: fix capsnap dirty pages accounting

writepages_finish() calls ceph_put_wrbuffer_cap_refs() once for
all pages, parameter snapc is set to req->r_snapc. So writepages()
shouldn't write dirty pages associated with different snapc in
one OSD request.
Signed-off-by: default avatar"Yan, Zheng" <zyan@redhat.com>
Signed-off-by: default avatarIlya Dryomov <idryomov@gmail.com>
parent 2a2d927e
...@@ -930,8 +930,8 @@ static int ceph_writepages_start(struct address_space *mapping, ...@@ -930,8 +930,8 @@ static int ceph_writepages_start(struct address_space *mapping,
/* only if matching snap context */ /* only if matching snap context */
pgsnapc = page_snap_context(page); pgsnapc = page_snap_context(page);
if (pgsnapc->seq > snapc->seq) { if (pgsnapc != snapc) {
dout("page snapc %p %lld > oldest %p %lld\n", dout("page snapc %p %lld != oldest %p %lld\n",
pgsnapc, pgsnapc->seq, snapc, snapc->seq); pgsnapc, pgsnapc->seq, snapc, snapc->seq);
unlock_page(page); unlock_page(page);
continue; continue;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment