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: align reclaim size to bluefs_alloc_size #14744

Merged
merged 1 commit into from Apr 25, 2017

Conversation

Projects
None yet
3 participants
@yuyuyu101
Member

yuyuyu101 commented Apr 24, 2017

otherwise it will crash because of unaligned alloc size

Signed-off-by: Haomai Wang haomai@xsky.com

os/bluestore: align reclaim size to bluefs_alloc_size
otherwise it will crash because of unaligned alloc size

Signed-off-by: Haomai Wang <haomai@xsky.com>

@yuyuyu101 yuyuyu101 requested review from liewegas and ifed01 Apr 24, 2017

@tchaikov

This comment has been minimized.

Show comment
Hide comment
@tchaikov

tchaikov Apr 24, 2017

Contributor

@yuyuyu101 could you create ci branches in ceph-ci and create PR branches from your own repo next time?

Contributor

tchaikov commented Apr 24, 2017

@yuyuyu101 could you create ci branches in ceph-ci and create PR branches from your own repo next time?

@yuyuyu101

This comment has been minimized.

Show comment
Hide comment
@yuyuyu101

yuyuyu101 Apr 24, 2017

Member

@tchaikov yep.... I'm a little lazy on one line change to creating new branch and push..just editing on github. will do next time

Member

yuyuyu101 commented Apr 24, 2017

@tchaikov yep.... I'm a little lazy on one line change to creating new branch and push..just editing on github. will do next time

@liewegas liewegas merged commit 79a7c21 into master Apr 25, 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

@liewegas liewegas deleted the wip-bluefs-alloc-size branch Apr 25, 2017

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