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

reef: rgw: improve buffer list utilization in the chunkupload scenario #53773

Merged
merged 1 commit into from Oct 16, 2023

Conversation

cbodley
Copy link
Contributor

@cbodley cbodley commented Oct 2, 2023

backport tracker: https://tracker.ceph.com/issues/63057


backport of #53266
parent tracker: https://tracker.ceph.com/issues/62681

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

Reduced waste of buffer::ptr by receiving multiple chunks and filling them into the buffer

AWSv4ComplMulti::recv_body() just receive one chunk and fill it into buffer.
Each 4MB buffer is actually only utilizing 64KB, leading to frequent buffer allocations.
~800GB virtual memory consumption has been observed.

Signed-off-by: liubingrun <liubr1@chinatelecom.cn>
(cherry picked from commit e47bb22)
@cbodley cbodley requested a review from a team as a code owner October 2, 2023 20:22
@cbodley cbodley added this to the reef milestone Oct 2, 2023
@cbodley
Copy link
Contributor Author

cbodley commented Oct 16, 2023

jenkins test make check

Copy link
Contributor

@yuriw yuriw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed-by: Casey Bodley cbodley@redhat.com

Copy link
Contributor

@yuriw yuriw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed-by: Casey Bodley cbodley@redhat.com

@yuriw yuriw merged commit fd4ac19 into ceph:reef Oct 16, 2023
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants