crush: try _all_ bucket items when doing exhaustive search

N-1 isn't exhaustive.

Fixes: #1594
Signed-off-by: Sage Weil <sage@newdream.net>

diff --git a/src/crush/mapper.c b/src/crush/mapper.c
index cb8195aa5756a9b3593ae43125a84f52d2428c88..8235ca2e15006f2069575a0ede4205c283cec437 100644 (file)
--- a/src/crush/mapper.c
+++ b/src/crush/mapper.c
@@ -415,7 +415,7 @@ reject:
                                        if (collide && flocal < 3)
                                                /* retry locally a few times */
                                                retry_bucket = 1;
-                                       else if (flocal < in->size + orig_tries)
+                                       else if (flocal <= in->size + orig_tries)
                                                /* exhaustive bucket search */
                                                retry_bucket = 1;
                                        else if (ftotal < 20)