Skip to content

Commit

Permalink
---
Browse files Browse the repository at this point in the history
yaml
---
r: 311212
b: refs/heads/master
c: 642c0db
h: refs/heads/master
v: v3
  • Loading branch information
Sage Weil authored and Alex Elder committed Jun 20, 2012
1 parent f6a022f commit 4ba897b
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 8 deletions.
2 changes: 1 addition & 1 deletion [refs]
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
---
refs/heads/master: b132cf4c733f91bb4dd2277ea049243cf16e8b66
refs/heads/master: 642c0dbde32f34baa7886e988a067089992adc8f
7 changes: 0 additions & 7 deletions trunk/net/ceph/ceph_common.c
Original file line number Diff line number Diff line change
Expand Up @@ -504,13 +504,6 @@ void ceph_destroy_client(struct ceph_client *client)
/* unmount */
ceph_osdc_stop(&client->osdc);

/*
* make sure osd connections close out before destroying the
* auth module, which is needed to free those connections'
* ceph_authorizers.
*/
ceph_msgr_flush();

ceph_monc_stop(&client->monc);

ceph_debugfs_client_cleanup(client);
Expand Down
8 changes: 8 additions & 0 deletions trunk/net/ceph/mon_client.c
Original file line number Diff line number Diff line change
Expand Up @@ -847,6 +847,14 @@ void ceph_monc_stop(struct ceph_mon_client *monc)

mutex_unlock(&monc->mutex);

/*
* flush msgr queue before we destroy ourselves to ensure that:
* - any work that references our embedded con is finished.
* - any osd_client or other work that may reference an authorizer
* finishes before we shut down the auth subsystem.
*/
ceph_msgr_flush();

ceph_auth_destroy(monc->auth);

ceph_msg_put(monc->m_auth);
Expand Down

0 comments on commit 4ba897b

Please sign in to comment.