diff options
author | Alex Elder <elder@inktank.com> | 2013-02-19 13:25:56 -0500 |
---|---|---|
committer | Alex Elder <elder@inktank.com> | 2013-02-25 16:37:18 -0500 |
commit | 154171678989950f6c392e126fa8006a145ed1cc (patch) | |
tree | ec223720a5dac808685e4a49fe570055a9007987 /net/ceph/messenger.c | |
parent | 2c3dd4ff595e604cd4c4c51cff7a208f23148c2d (diff) |
libceph: eliminate sparse warnings
Eliminate most of the problems in the libceph code that cause sparse
to issue warnings.
- Convert functions that are never referenced externally to have
static scope.
- Pass NULL rather than 0 for a pointer argument in one spot in
ceph_monc_delete_snapid()
This partially resolves:
http://tracker.ceph.com/issues/4184
Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Diffstat (limited to 'net/ceph/messenger.c')
-rw-r--r-- | net/ceph/messenger.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ceph/messenger.c b/net/ceph/messenger.c index 771d4c904469..ed9e237d967c 100644 --- a/net/ceph/messenger.c +++ b/net/ceph/messenger.c | |||
@@ -223,7 +223,7 @@ static void encode_my_addr(struct ceph_messenger *msgr) | |||
223 | */ | 223 | */ |
224 | static struct workqueue_struct *ceph_msgr_wq; | 224 | static struct workqueue_struct *ceph_msgr_wq; |
225 | 225 | ||
226 | void _ceph_msgr_exit(void) | 226 | static void _ceph_msgr_exit(void) |
227 | { | 227 | { |
228 | if (ceph_msgr_wq) { | 228 | if (ceph_msgr_wq) { |
229 | destroy_workqueue(ceph_msgr_wq); | 229 | destroy_workqueue(ceph_msgr_wq); |