diff options
| author | Sage Weil <[email protected]> | 2012-06-01 03:22:18 +0000 |
|---|---|---|
| committer | Alex Elder <[email protected]> | 2012-06-20 12:43:49 +0000 |
| commit | 88ed6ea0b295f8e2383d599a04027ec596cdf97b (patch) | |
| tree | e9a49af02373e76a9ce0ed94b4120ccc184a7461 /fs/jbd2/commit.c | |
| parent | libceph: osd_client: don't drop reply reference too early (diff) | |
| download | kernel-88ed6ea0b295f8e2383d599a04027ec596cdf97b.tar.gz kernel-88ed6ea0b295f8e2383d599a04027ec596cdf97b.zip | |
libceph: use con get/put ops from osd_client
There were a few direct calls to ceph_con_{get,put}() instead of the con
ops from osd_client.c. This is a bug since those ops aren't defined to
be ceph_con_get/put.
This breaks refcounting on the ceph_osd structs that contain the
ceph_connections, and could lead to all manner of strangeness.
The purpose of the ->get and ->put methods in a ceph connection are
to allow the connection to indicate it has a reference to something
external to the messaging system, *not* to indicate something
external has a reference to the connection.
[[email protected]: added that last sentence]
Signed-off-by: Sage Weil <[email protected]>
Reviewed-by: Alex Elder <[email protected]>
(cherry picked from commit 0d47766f14211a73eaf54cab234db134ece79f49)
Diffstat (limited to 'fs/jbd2/commit.c')
0 files changed, 0 insertions, 0 deletions