-
Notifications
You must be signed in to change notification settings - Fork 273
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
[Bug] Packets send date drift #724
Comments
Attempt to recreate on Debian 10 (Buster). Not seeing any difference in performance between #660 (b53d916) and 4.4.1. Reopen if you have steps to reproduce. Here is commit b53d916:
here is 4.4.1:
|
Hi @fklassen. You've tested two versions that are impacted by the issue. You should test the commit before b53d916 if you want to see a difference. Beware, mutliplier option was broken with #631, so previous version with this specific command line will have a different behavior, but will be broken as well. Anyway, if you look at the duration it took to send the packets, you can see something is wrong : you are trying to send 100000 packets at a rate of 10000 packets a second, and it takes 11.39 seconds. This is the issue: it should take 10 seconds. It's not a performance issue. #631 was fixing that, by waiting to reach packet date to send a packet, instead of waiting to reach duration since previous packet send date to send a packet (which introduces drift). But it looks like #631 also broke multiplier option in tcpreplay, and was subsequently reverted, fixing multiplier, but recreating drift issue. Can you reopen the issue or do you need more information ? Regards, Mathieu |
I'll check it out. |
Tried to recreate, and I cannot find any versions pre #699 that doesn't hang when reproducing #630 setup. That includes #630 (b53d916), and pre #630 (b53d916~1). It appears that #631 introduced bugs that have since been fixed. Question. Is there any commit that works, i.e. reproduces #630 setup with no drift? I'll see if I can make a routine that has less drift, but doesn't break with extreme multiplies, e.g. 10000 and 0.000001. |
Labeling as a feature due to the fact that we cannot find a version where this worked better than it does now. Moving out to feature version 4.5. |
Describe the bug
Bug that was described in #630 was fixed with #631 and #645 , but subsequently reverted with #699 to address #674
To Reproduce
See #630
Expected behavior
See #630
System
The text was updated successfully, but these errors were encountered: