diff options
author | Alex Elder <elder@inktank.com> | 2012-11-13 22:11:15 -0500 |
---|---|---|
committer | Alex Elder <elder@inktank.com> | 2013-01-17 16:52:05 -0500 |
commit | d178a9e74006e80f568d87e29f2a68f14fc7cbb1 (patch) | |
tree | a7f59f104771c9ad4ed4f2d86d7e3a2299e932a8 /include/linux/ceph/osd_client.h | |
parent | e75b45cf36565fd8ba206a9d80f670a86e61ba2f (diff) |
libceph: don't set flags in ceph_osdc_alloc_request()
The only thing ceph_osdc_alloc_request() really does with the
flags value it is passed is assign it to the newly-created
osd request structure. Do that in the caller instead.
Both callers subsequently call ceph_osdc_build_request(), so have
that function (instead of ceph_osdc_alloc_request()) issue a warning
if a request comes through with neither the read nor write flags set.
Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Diffstat (limited to 'include/linux/ceph/osd_client.h')
-rw-r--r-- | include/linux/ceph/osd_client.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/include/linux/ceph/osd_client.h b/include/linux/ceph/osd_client.h index fe3a6e8db1f9..6ddda5bbd1a6 100644 --- a/include/linux/ceph/osd_client.h +++ b/include/linux/ceph/osd_client.h | |||
@@ -213,7 +213,6 @@ extern int ceph_calc_raw_layout(struct ceph_file_layout *layout, | |||
213 | struct ceph_osd_req_op *op); | 213 | struct ceph_osd_req_op *op); |
214 | 214 | ||
215 | extern struct ceph_osd_request *ceph_osdc_alloc_request(struct ceph_osd_client *osdc, | 215 | extern struct ceph_osd_request *ceph_osdc_alloc_request(struct ceph_osd_client *osdc, |
216 | int flags, | ||
217 | struct ceph_snap_context *snapc, | 216 | struct ceph_snap_context *snapc, |
218 | struct ceph_osd_req_op *ops, | 217 | struct ceph_osd_req_op *ops, |
219 | bool use_mempool, | 218 | bool use_mempool, |