Skip to content
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

common: object_map_update fails with -EINVAL return code #5559

Merged
2 commits merged into from Aug 30, 2015

Conversation

Projects
None yet
2 participants
@dillaman
Copy link
Contributor

commented Aug 13, 2015

dillaman added some commits Aug 10, 2015

common: bit_vector extent calculation incorrect for last page
It's highly probable that the last page in the bit vector will not
be a full page size.  As a result, the computed extents will extend
beyond the data portion of the bit vector, resulting in a end_of_buffer
exception.

Fixes: #12611
Backport: hammer
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit c6d9899)
tests: increase test coverage for partial encodes/decodes
Multiple combinations of offsets/lengths are now tested when
performing partial encodes/decodes of the bit vector.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit 3e145f7)

@dillaman dillaman added this to the hammer milestone Aug 13, 2015

@ghost ghost self-assigned this Aug 13, 2015

ghost pushed a commit that referenced this pull request Aug 30, 2015

Loic Dachary
Merge pull request #5559 from ceph/wip-12682-hammer
object_map_update fails with -EINVAL return code

Reviewed-by: Loic Dachary <ldachary@redhat.com>

@ghost ghost merged commit a39aeee into hammer Aug 30, 2015

@dillaman dillaman deleted the wip-12682-hammer branch Sep 7, 2015

@ghost ghost changed the title object_map_update fails with -EINVAL return code common: object_map_update fails with -EINVAL return code Sep 13, 2015

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.