diff options
author | Sage Weil <sage@newdream.net> | 2009-11-01 20:53:24 -0500 |
---|---|---|
committer | Sage Weil <sage@newdream.net> | 2009-11-01 20:53:24 -0500 |
commit | 33aa96e7430d215e2ee779f65cdad0f6d4571fe1 (patch) | |
tree | 00309ddde7506b0c2a86a20f5b579a60623005ab /fs/ceph/crush | |
parent | 63ff78b25c4b204075b5b98afcac6ad3639d43fe (diff) |
crush: always return a value from crush_bucket_choose
Even when we encounter a corrupt bucket. We still BUG(). This fixes
the warning
fs/ceph/crush/mapper.c: In function 'crush_choose':
fs/ceph/crush/mapper.c:352: warning: control may reach end of non-void function
'crush_bucket_choose' being inlined
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'fs/ceph/crush')
-rw-r--r-- | fs/ceph/crush/mapper.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ceph/crush/mapper.c b/fs/ceph/crush/mapper.c index c268393adfc..54f3f402af6 100644 --- a/fs/ceph/crush/mapper.c +++ b/fs/ceph/crush/mapper.c | |||
@@ -253,7 +253,7 @@ static int crush_bucket_choose(struct crush_bucket *in, int x, int r) | |||
253 | x, r); | 253 | x, r); |
254 | default: | 254 | default: |
255 | BUG_ON(1); | 255 | BUG_ON(1); |
256 | /* return in->items[0] */; | 256 | return in->items[0]; |
257 | } | 257 | } |
258 | } | 258 | } |
259 | 259 | ||