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
crimson/os/seastore/.../btree_range_pin: fix is_parent_of #43301
crimson/os/seastore/.../btree_range_pin: fix is_parent_of #43301
Conversation
…ansaction_direct Signed-off-by: Samuel Just <sjust@redhat.com>
The LBA tree implementation only requires that the start addr of a logical extent be contained within the leaf range. It's entirely possible for the end of a logical extent to extend past the end addr of the containing leaf node. Fixes: https://tracker.ceph.com/issues/52709 Signed-off-by: Samuel Just <sjust@redhat.com>
5b19b13
to
49affeb
Compare
@@ -21,7 +21,7 @@ struct lba_node_meta_t { | |||
bool is_parent_of(const lba_node_meta_t &other) const { | |||
return (depth == other.depth + 1) && | |||
(begin <= other.begin) && | |||
(end >= other.end); | |||
(end > other.begin); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so finally, we will update parent node meta to include leaf node meta, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry? No, this is just fixing the is_parent_of predicate to be correct.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.