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

Synchronous P2P Request Latency Too Large #67

Closed
andrei3131 opened this issue May 29, 2019 · 3 comments
Closed

Synchronous P2P Request Latency Too Large #67

andrei3131 opened this issue May 29, 2019 · 3 comments
Assignees

Comments

@andrei3131
Copy link
Collaborator

Please find below the latencies measured for the synchronous case of P2P Model Request and Store update. This is the current bottleneck of the system which needs to be resolved in order to improve the throughput of the system.

[127.0.0.1/02/02-of-04::stdout] Model store update took 16.012414ms
[127.0.0.1/03/03-of-04::stdout] Model store update took 276.625µs
[127.0.0.1/02/02-of-04::stdout] Model store update took 176.388µs
[127.0.0.1/02/02-of-04::stdout] Request took 673.288µs
[127.0.0.1/03/03-of-04::stdout] Request took 15.947665ms
[127.0.0.1/00/00-of-04::stdout] Model store update took 3.733306ms
[127.0.0.1/01/01-of-04::stdout] Model store update took 175.905µs
[127.0.0.1/00/00-of-04::stdout] Request took 14.822951ms
[127.0.0.1/03/03-of-04::stdout] Model store update took 195.484µs
[127.0.0.1/01/01-of-04::stdout] Request took 20.67839ms
[127.0.0.1/02/02-of-04::stdout] Model store update took 216.966µs
[127.0.0.1/00/00-of-04::stdout] Model store update took 235.257µs
[127.0.0.1/01/01-of-04::stdout] Request took 833.436µs
@luomai
Copy link
Member

luomai commented May 29, 2019

We benchmark the requesting model latency on playtpus2. The peer actually requests a local peer (though through domain socket). The latency is high.

The requested model size is 2MB. In the worst case if using a 10 Gbps link, the actual data transmission should not take more than 2ms.

Could you look into this if you have time? @lgarithm

@lgarithm
Copy link
Collaborator

Commit id for benchmarks: e97969d

@lgarithm
Copy link
Collaborator

lgarithm commented Jun 9, 2019

The latency varies from t to 2t (#83), I think it is stable, considering the memory bandwidth is shared.

Screenshot 2019-06-09 at 10 52 54 PM

@luomai luomai closed this as completed Jul 11, 2019
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

No branches or pull requests

3 participants