diff options
| author | Alex Elder <elder@inktank.com> | 2012-11-13 22:11:15 -0500 |
|---|---|---|
| committer | Alex Elder <elder@inktank.com> | 2013-01-17 17:34:57 -0500 |
| commit | 54a5400721da7fa5a16cea151aade5bdfee74111 (patch) | |
| tree | 285ba131da6c5b700c0fda64858de938f45959c6 /include/linux/ceph | |
| parent | d178a9e74006e80f568d87e29f2a68f14fc7cbb1 (diff) | |
libceph: don't set pages or bio in ceph_osdc_alloc_request()
Only one of the two callers of ceph_osdc_alloc_request() provides
page or bio data for its payload. And essentially all that function
was doing with those arguments was assigning them to fields in the
osd request structure.
Simplify ceph_osdc_alloc_request() by having the caller take care of
making those assignments
Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Diffstat (limited to 'include/linux/ceph')
| -rw-r--r-- | include/linux/ceph/osd_client.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/include/linux/ceph/osd_client.h b/include/linux/ceph/osd_client.h index 6ddda5bbd1a6..75f56d372d44 100644 --- a/include/linux/ceph/osd_client.h +++ b/include/linux/ceph/osd_client.h | |||
| @@ -216,9 +216,7 @@ extern struct ceph_osd_request *ceph_osdc_alloc_request(struct ceph_osd_client * | |||
| 216 | struct ceph_snap_context *snapc, | 216 | struct ceph_snap_context *snapc, |
| 217 | struct ceph_osd_req_op *ops, | 217 | struct ceph_osd_req_op *ops, |
| 218 | bool use_mempool, | 218 | bool use_mempool, |
| 219 | gfp_t gfp_flags, | 219 | gfp_t gfp_flags); |
| 220 | struct page **pages, | ||
| 221 | struct bio *bio); | ||
| 222 | 220 | ||
| 223 | extern void ceph_osdc_build_request(struct ceph_osd_request *req, | 221 | extern void ceph_osdc_build_request(struct ceph_osd_request *req, |
| 224 | u64 off, u64 len, | 222 | u64 off, u64 len, |
