erasure-code: synchronize with upstream gf-complete #12382

Merged
merged 1 commit into from Dec 14, 2016

Projects

None yet

5 participants

@dachary
Member
dachary commented Dec 8, 2016

Fixes: http://tracker.ceph.com/issues/18092

Signed-off-by: Loic Dachary loic@dachary.org

@ldachary ldachary erasure-code: synchronize with upstream gf-complete
Fixes: http://tracker.ceph.com/issues/18092

Signed-off-by: Loic Dachary <loic@dachary.org>
4d7b81a
@dachary dachary added this to the kraken milestone Dec 8, 2016
@dachary
Member
dachary commented Dec 8, 2016

@bassamtabbara the make check passed and the submodule was updated to a commit including the fix, meaning it does not break anything. There is no test showing the problem unfortunately. Although we could try to simulate the absence of the PCLMUL flag to verify tests pass as they should, making it so it would fail with an illegal instruction would be challenging and too fragile to be maintainable.

@bassam
Member
bassam commented Dec 8, 2016

@dachary and @dx9 thanks for fixing this!

@bassam
bassam approved these changes Dec 8, 2016 View changes
@dx9
Contributor
dx9 commented Dec 9, 2016

high five!

@liewegas liewegas merged commit 6b6af6c into ceph:master Dec 14, 2016

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment