Skip to content

WIP io parallel copy#834

Closed
mmatczuk wants to merge 3 commits intomainfrom
mmt/io_parallel_copy
Closed

WIP io parallel copy#834
mmatczuk wants to merge 3 commits intomainfrom
mmt/io_parallel_copy

Conversation

@mmatczuk
Copy link
Copy Markdown
Contributor

@mmatczuk mmatczuk commented Jun 6, 2024

No description provided.

@mmatczuk mmatczuk requested a review from Choraden as a code owner June 6, 2024 14:58
@mmatczuk mmatczuk marked this pull request as draft June 6, 2024 14:58
@mmatczuk
Copy link
Copy Markdown
Contributor Author

I tested this patch with Forwarder and sc build, both locally and with dedicated Linux box 4 CPU Intel in local network.
I see no stable improvement that would justify the change.
In the test against gnet hello world, leading to this work, the new implementation (both local and over sc tunnel) does not change anything. In the test I'm able to get 20MiB/s throughput from a single core running gnet while forwarder and sc peak at 8,5MiB.

Testing against httpbin does not really work due to high variation in latencies stddev ranging to 70-100% both with wrk and vegeta.
In vegeta this can be flattened out by using HDR but one can note how stable gnet latencies are.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant