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

Evaluate Direct I/O for better performance #7793

Open
Bukhtawar opened this issue May 27, 2023 · 1 comment
Open

Evaluate Direct I/O for better performance #7793

Bukhtawar opened this issue May 27, 2023 · 1 comment
Labels
enhancement Enhancement or improvement to existing feature or request Storage:Durability Issues and PRs related to the durability framework Storage Issues and PRs relating to data and metadata storage

Comments

@Bukhtawar
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
The primary benefit of direct I/O is to reduce CPU utilization for file reads and writes by eliminating the copy from the cache to the user buffer. Many operations like peer recovery, segrep transferring large merged segments to replica for instance w/o direct I/O can cause other segments(since refreshes no longer leverage shard idle #7736 ), being queried heavily to be pushed out of the cache even if. The remote store interactions needs to be revisited to explore any opportunities for improvements

Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@Bukhtawar Bukhtawar added enhancement Enhancement or improvement to existing feature or request untriaged labels May 27, 2023
@gbbafna gbbafna added Storage:Durability Issues and PRs related to the durability framework and removed untriaged labels May 30, 2023
@msproact
Copy link

msproact commented Jun 7, 2023

Another approach here would be to use posiv_fadvice() with POSIX_FADV_DONTNEED (see manpage posix_fadvise(2))¹. However I bet Direct I/O could even be more efficient.

@Bukhtawar Bukhtawar added the Storage Issues and PRs relating to data and metadata storage label Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement or improvement to existing feature or request Storage:Durability Issues and PRs related to the durability framework Storage Issues and PRs relating to data and metadata storage
Projects
Status: 🆕 New
Development

No branches or pull requests

3 participants