diff options
| author | Alex Elder <elder@inktank.com> | 2013-04-05 08:27:11 +0200 |
|---|---|---|
| committer | Sage Weil <sage@inktank.com> | 2013-05-02 06:18:15 +0200 |
| commit | c99d2d4abb6c405ef52e9bc1da87b382b8f41739 (patch) | |
| tree | 073ad414ec82f706a38300d38fe5c5a710d3098a /fs/ceph | |
| parent | libceph: add data pointers in osd op structures (diff) | |
| download | linux-c99d2d4abb6c405ef52e9bc1da87b382b8f41739.tar.xz linux-c99d2d4abb6c405ef52e9bc1da87b382b8f41739.zip | |
libceph: specify osd op by index in request
An osd request now holds all of its source op structures, and every
place that initializes one of these is in fact initializing one
of the entries in the the osd request's array.
So rather than supplying the address of the op to initialize, have
caller specify the osd request and an indication of which op it
would like to initialize. This better hides the details the
op structure (and faciltates moving the data pointers they use).
Since osd_req_op_init() is a common routine, and it's not used
outside the osd client code, give it static scope. Also make
it return the address of the specified op (so all the other
init routines don't have to repeat that code).
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 0ac3a37753cb..cc57104a7266 100644 --- a/fs/ceph/addr.c +++ b/fs/ceph/addr.c @@ -926,7 +926,7 @@ get_more_pages: /* Update the write op length in case we changed it */ - osd_req_op_extent_update(&req->r_ops[0], len); + osd_req_op_extent_update(req, 0, len); vino = ceph_vino(inode); ceph_osdc_build_request(req, offset, snapc, vino.snap, |