-
Notifications
You must be signed in to change notification settings - Fork 609
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
[EPIC] v1 data path performance enhancement #6600
[EPIC] v1 data path performance enhancement #6600
Comments
The primary improvement would be:
The purpose is to make v1 appropriate for running under constrained resource environments like Edge. |
@Kampadais improved the v1 data path based on my previous PoC. I think we can work on the improvement in v1.8.0 together and replace the iSCSI frontend with the new one. cc @innobead |
Agree that we can implement it in 1.8.0. Because I am working on v1 performance related topics, I can help to drive this ticket if it is ok Derek |
I would like to work on it more. I am currently experimenting and investigating why with my multiple connections approach can't achieve ublk's performance. |
Surely @Kampadais . I am happy to follow your lead! |
Recently Phan and me tried to make some improvements to the v1 backend data path:
Notice that only the first two ideas work as expected. And the remaining 3 ideas cannot get a better result in the fio test, regardless that the CPU profiling shows a CPU usage percentage decrement for the R/W function. We suspect that iSCSI tgt may be the main bottleneck. It may waste all CPU resources saved by the last 3 ideas, which finally leads to 0 improvements in the fio test. If we plan to introduce
cc @PhanLe1010 |
Maybe the bottleneck can be the revision counter itself? I mean the writing of the revision counter on the disk. |
longhorn/longhorn-engine#1109 by @Toutou98 , alternative based on nbd. |
longhorn/longhorn-engine#1067 draft PR by @Kampadais |
Is your feature request related to a problem? Please describe (👍 if you like this request)
This epic is used to track improvements of the v1 data path.
Describe the solution you'd like
Describe alternatives you've considered
Additional context
cc @longhorn/dev-data-plane
The text was updated successfully, but these errors were encountered: