diff options
author | Greg Farnum <gregory.farnum@dreamhost.com> | 2012-03-27 12:57:14 -0700 |
---|---|---|
committer | Greg Farnum <gregory.farnum@dreamhost.com> | 2012-04-03 13:22:19 -0700 |
commit | e966c39df69028df88625e08e5571144aa50186d (patch) | |
tree | e293d7d5f7f4036d158142ed4b6ada17a303aedc | |
parent | 096971d4c39f2a73c345bee50d3987085073d828 (diff) | |
download | ceph-e966c39df69028df88625e08e5571144aa50186d.tar.gz |
msgr: make a bunch of stuff private.
Why were all these data members public? They're accessed by Pipes
and the Accepter and stuff, so maybe that's why...but that's all
internal interface stuff.
Signed-off-by: Greg Farnum <gregory.farnum@dreamhost.com>
-rw-r--r-- | src/msg/SimpleMessenger.h | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/src/msg/SimpleMessenger.h b/src/msg/SimpleMessenger.h index 0cfc1240bca..8b9de3b3215 100644 --- a/src/msg/SimpleMessenger.h +++ b/src/msg/SimpleMessenger.h @@ -460,7 +460,6 @@ private: void dispatch_throttle_release(uint64_t msize); // SimpleMessenger stuff - public: Mutex lock; Cond wait_cond; // for wait() bool did_bind; @@ -483,6 +482,13 @@ private: Policy default_policy; map<int, Policy> policy_map; // entity_name_t::type -> Policy + // --- pipes --- + set<Pipe*> pipes; + list<Pipe*> pipe_reap_queue; + + Mutex global_seq_lock; + __u32 global_seq; +public: Policy& get_policy(int t) { if (policy_map.count(t)) return policy_map[t]; @@ -490,15 +496,7 @@ private: return default_policy; } - // --- pipes --- - set<Pipe*> pipes; - list<Pipe*> pipe_reap_queue; - - Mutex global_seq_lock; - __u32 global_seq; - Pipe *connect_rank(const entity_addr_t& addr, int type); - virtual void mark_down(const entity_addr_t& addr); virtual void mark_down(Connection *con); virtual void mark_down_on_empty(Connection *con); |