diff options
author | Ilya Dryomov <ilya.dryomov@inktank.com> | 2013-12-24 21:19:26 +0200 |
---|---|---|
committer | Ilya Dryomov <ilya.dryomov@inktank.com> | 2013-12-31 20:32:22 +0200 |
commit | f18650ace38ef200dd1578257c75e9407297953c (patch) | |
tree | 252f8675255569d1ab376b652cbb3c17332f36ca /net/dcb | |
parent | be3226acc5544bcc91e756eb3ee6ca7b74f6f0a8 (diff) | |
download | linux-f18650ace38ef200dd1578257c75e9407297953c.tar.gz linux-f18650ace38ef200dd1578257c75e9407297953c.tar.bz2 linux-f18650ace38ef200dd1578257c75e9407297953c.zip |
crush: apply chooseleaf_tries to firstn mode too
Parameterize the attempts for the _firstn choose method, and apply the
rule-specified tries count to firstn mode as well. Note that we have
slightly different behavior here than with indep:
If the firstn value is not specified for firstn, we pass through the
normal attempt count. This maintains compatibility with legacy behavior.
Note that this is usually *not* actually N^2 work, though, because of the
descend_once tunable. However, descend_once is unfortunately *not* the
same thing as 1 chooseleaf try because it is only checked on a reject but
not on a collision. Sigh.
In contrast, for indep, if tries is not specified we default to 1
recursive attempt, because that is simply more sane, and we have the
option to do so. The descend_once tunable has no effect for indep.
Reflects ceph.git commit 64aeded50d80942d66a5ec7b604ff2fcbf5d7b63.
Signed-off-by: Ilya Dryomov <ilya.dryomov@inktank.com>
Reviewed-by: Sage Weil <sage@inktank.com>
Diffstat (limited to 'net/dcb')
0 files changed, 0 insertions, 0 deletions