diff options
| author | Sage Weil <sage@newdream.net> | 2011-12-12 18:35:22 +0100 |
|---|---|---|
| committer | Sage Weil <sage@newdream.net> | 2012-01-10 17:56:57 +0100 |
| commit | e11b05d31f21f0ea39ea288af667887cd6c21c80 (patch) | |
| tree | 59768fe6ec9ef15f5e01817d76257f7489df1d5e | |
| parent | ceph: remove unnecessary d_fsdata conditional checks (diff) | |
| download | linux-e11b05d31f21f0ea39ea288af667887cd6c21c80.tar.xz linux-e11b05d31f21f0ea39ea288af667887cd6c21c80.zip | |
crush: fix force for non-root TAKE
Signed-off-by: Sage Weil <sage@newdream.net>
| -rw-r--r-- | net/ceph/crush/mapper.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/net/ceph/crush/mapper.c b/net/ceph/crush/mapper.c index 3a94eae7abe9..b79747c4b645 100644 --- a/net/ceph/crush/mapper.c +++ b/net/ceph/crush/mapper.c @@ -510,10 +510,15 @@ int crush_do_rule(struct crush_map *map, switch (rule->steps[step].op) { case CRUSH_RULE_TAKE: w[0] = rule->steps[step].arg1; - if (force_pos >= 0) { - BUG_ON(force_context[force_pos] != w[0]); + + /* find position in force_context/hierarchy */ + while (force_pos >= 0 && + force_context[force_pos] != w[0]) force_pos--; - } + /* and move past it */ + if (force_pos >= 0) + force_pos--; + wsize = 1; break; |