diff options
author | Alex Elder <elder@inktank.com> | 2013-03-01 19:00:15 -0500 |
---|---|---|
committer | Sage Weil <sage@inktank.com> | 2013-05-02 00:16:21 -0400 |
commit | cf7b7e1492e97dd0c44479239742eb4cb752eeed (patch) | |
tree | 83ed8068f3b0c1138601a763657aa3a2335fd24e /fs/ceph | |
parent | 54ae0756e3847f4350ba24271a2a38be1263dd67 (diff) |
ceph: use calc_pages_for() in start_read()
There's a spot that computes the number of pages to allocate for a
page-aligned length by just shifting it. Use calc_pages_for()
instead, to be consistent with usage everywhere else. The result
is the same.
The reason for this is to make it clearer in an upcoming patch that
this calculation is duplicated.
Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Diffstat (limited to 'fs/ceph')
-rw-r--r-- | fs/ceph/addr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ceph/addr.c b/fs/ceph/addr.c index 2a571fb4803b..e53f24b15b12 100644 --- a/fs/ceph/addr.c +++ b/fs/ceph/addr.c | |||
@@ -314,7 +314,7 @@ static int start_read(struct inode *inode, struct list_head *page_list, int max) | |||
314 | return PTR_ERR(req); | 314 | return PTR_ERR(req); |
315 | 315 | ||
316 | /* build page vector */ | 316 | /* build page vector */ |
317 | nr_pages = len >> PAGE_CACHE_SHIFT; | 317 | nr_pages = calc_pages_for(0, len); |
318 | pages = kmalloc(sizeof(*pages) * nr_pages, GFP_NOFS); | 318 | pages = kmalloc(sizeof(*pages) * nr_pages, GFP_NOFS); |
319 | ret = -ENOMEM; | 319 | ret = -ENOMEM; |
320 | if (!pages) | 320 | if (!pages) |