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.

Sign up for GitHub

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

Jump to bottom

[RFC] osd: EC Partial Stripe Reads (Retry of #23138) #52746

Closed
wants to merge 1 commit into from
Closed

[RFC] osd: EC Partial Stripe Reads (Retry of #23138) #52746

wants to merge 1 commit into from
+248 −47

Commits on Aug 2, 2023

  1. osd: EC Partial Stripe Reads (Retry of ceph#23138) 

    This is a re-implementation of PR ceph#23138 rebased on main with a couple of nitpicky changes to make the code a little more clear (to me at least).  Credit goes to Xiaofei Cui [cuixiaofei@sangfor.com.cn](mailto:cuixiaofei@sangfor.com.cn) for the original implementation.

Looking at the original PR's review, it does not appear that we can use the same technique as in ceph@468ad4b.  We don't have the ReadOp yet.  I'm not sure if @gregsforytwo's idea to query the plugin works, but it's clear we are not doing the efficient thing from the get-go here.

The performance and efficiency benefits for small random reads appears to be quite substantial, especially for large stripe widths.

Signed-off-by: Mark Nelson <mark.nelson@clyso.com>
    Mark Nelson authored and Mark Nelson committed Aug 2, 2023
    Configuration menu
    Copy the full SHA
    8f53dac View commit details
    Browse the repository at this point in the history