diff options
author | Ilya Dryomov <ilya.dryomov@inktank.com> | 2014-03-19 15:58:37 +0100 |
---|---|---|
committer | Sage Weil <sage@inktank.com> | 2014-04-05 06:07:25 +0200 |
commit | 6ed1002f368c63ef79d7f659fcb4368a90098132 (patch) | |
tree | 04ee2f8ea095ea791a3d99f19a28a4c064b6f720 /net/ceph/crush/mapper.c | |
parent | crush: fix off-by-one errors in total_tries refactor (diff) | |
download | linux-6ed1002f368c63ef79d7f659fcb4368a90098132.tar.xz linux-6ed1002f368c63ef79d7f659fcb4368a90098132.zip |
crush: allow crush rules to set (re)tries counts to 0
These two fields are misnomers; they are *retry* counts.
Reflects ceph.git commit f17caba8ae0cad7b6f8f35e53e5f73b444696835.
Signed-off-by: Ilya Dryomov <ilya.dryomov@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Diffstat (limited to 'net/ceph/crush/mapper.c')
-rw-r--r-- | net/ceph/crush/mapper.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/ceph/crush/mapper.c b/net/ceph/crush/mapper.c index 074bb2a5e675..b3fb84903b30 100644 --- a/net/ceph/crush/mapper.c +++ b/net/ceph/crush/mapper.c @@ -684,12 +684,12 @@ int crush_do_rule(const struct crush_map *map, break; case CRUSH_RULE_SET_CHOOSE_LOCAL_TRIES: - if (curstep->arg1 > 0) + if (curstep->arg1 >= 0) choose_local_retries = curstep->arg1; break; case CRUSH_RULE_SET_CHOOSE_LOCAL_FALLBACK_TRIES: - if (curstep->arg1 > 0) + if (curstep->arg1 >= 0) choose_local_fallback_retries = curstep->arg1; break; |