diff options
author | Sage Weil <sage@newdream.net> | 2010-06-10 18:50:10 -0400 |
---|---|---|
committer | Sage Weil <sage@newdream.net> | 2010-08-01 23:11:40 -0400 |
commit | 796d6955a51ce6768d0e033f27a2f8f5be6cb39a (patch) | |
tree | 03672d617a25c43ff90f7f627aa2ed0c3f3ed1db | |
parent | 37151668bad3fd058368752bee476f2ba3645596 (diff) |
ceph: only set num_pages in calc_layout
Setting it elsewhere is unnecessary and more fragile.
Signed-off-by: Sage Weil <sage@newdream.net>
-rw-r--r-- | fs/ceph/osd_client.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/ceph/osd_client.c b/fs/ceph/osd_client.c index e38522347898..707d2dbd8776 100644 --- a/fs/ceph/osd_client.c +++ b/fs/ceph/osd_client.c | |||
@@ -1276,8 +1276,6 @@ int ceph_osdc_readpages(struct ceph_osd_client *osdc, | |||
1276 | 1276 | ||
1277 | /* it may be a short read due to an object boundary */ | 1277 | /* it may be a short read due to an object boundary */ |
1278 | req->r_pages = pages; | 1278 | req->r_pages = pages; |
1279 | num_pages = calc_pages_for(off, *plen); | ||
1280 | req->r_num_pages = num_pages; | ||
1281 | 1279 | ||
1282 | dout("readpages final extent is %llu~%llu (%d pages)\n", | 1280 | dout("readpages final extent is %llu~%llu (%d pages)\n", |
1283 | off, *plen, req->r_num_pages); | 1281 | off, *plen, req->r_num_pages); |
@@ -1319,7 +1317,6 @@ int ceph_osdc_writepages(struct ceph_osd_client *osdc, struct ceph_vino vino, | |||
1319 | 1317 | ||
1320 | /* it may be a short write due to an object boundary */ | 1318 | /* it may be a short write due to an object boundary */ |
1321 | req->r_pages = pages; | 1319 | req->r_pages = pages; |
1322 | req->r_num_pages = calc_pages_for(off, len); | ||
1323 | dout("writepages %llu~%llu (%d pages)\n", off, len, | 1320 | dout("writepages %llu~%llu (%d pages)\n", off, len, |
1324 | req->r_num_pages); | 1321 | req->r_num_pages); |
1325 | 1322 | ||