]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
tests: do not check for invalid k/m combinations
authorKefu Chai <kchai@redhat.com>
Mon, 12 Nov 2018 01:55:14 +0000 (09:55 +0800)
committerKefu Chai <kchai@redhat.com>
Thu, 15 Nov 2018 06:34:03 +0000 (14:34 +0800)
commit2b13c7f7e9697be2d3a0188d185addea2553539e
tree230c6feb648b73ccf5a5a3142596fc0bfa9f46ca
parent83762041a4b71d899bda2c0aaa219cb6ea99622e
tests: do not check for invalid k/m combinations

This reverts commit c5bafae6fd10282f408695c7763b1277469123f6.

in 37f73607 and 3ec41030, we detect invalid "k". and we should always
detect invalid combinations of parameters in
ErasureCodeInterface::init(), and return -EINVAL instead of "autotune" them
as put in the commit message of c5bafae6, if the profile is invalid for
given erasure plugin. so ErasureCodeBench::encode() and
ErasureCodeBench::decode() will bail out with the error message returned by
ErasureCodeInterface::init() if the parameters are not valid.

also, i checked all ErasureCodeInterface::init() implementations, none
of them is overwriting the settings without returning a non-zero return
code. so we are not overriding or autotuning the settings specified by
user.

Fixes: http://tracker.ceph.com/issues/16500
Signed-off-by: Kefu Chai <kchai@redhat.com>
src/test/erasure-code/ceph_erasure_code_benchmark.cc