New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

os/bluestore: fix BitMapAllocator assert on out-of-bound hint value #15289

Merged
merged 1 commit into from May 26, 2017

Conversation

Projects
None yet
3 participants
@ifed01
Contributor

ifed01 commented May 25, 2017

BlueFS might provide a hint pointing to the end of the disk and BitMapAllocator asserts on that.

Signed-off-by: Igor Fedotov ifedotov@mirantis.com

@ifed01

This comment has been minimized.

Show comment
Hide comment
@ifed01

ifed01 May 25, 2017

Contributor

jenkins retest this please

Contributor

ifed01 commented May 25, 2017

jenkins retest this please

@liewegas liewegas requested a review from varadakari May 26, 2017

@varadakari

Except the nit, rest looks good.

Show outdated Hide outdated src/os/bluestore/BitMapAllocator.cc
Igor Fedotov
os/bluestore: fix BitMapAllocator assert on out-of-bound hint value
Signed-off-by: Igor Fedotov <ifedotov@mirantis.com>
@varadakari

This comment has been minimized.

Show comment
Hide comment
@varadakari

varadakari May 26, 2017

Contributor

LGTM

Contributor

varadakari commented May 26, 2017

LGTM

@liewegas liewegas added the needs-qa label May 26, 2017

@liewegas liewegas merged commit 5f003d6 into ceph:master May 26, 2017

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

@ifed01 ifed01 deleted the ifed01:wip-bluestore-fix-bmap-assert branch Jun 13, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment