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

task04: change timings for large payload test #121

Merged
merged 2 commits into from
Apr 16, 2019

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Apr 10, 2019

Due to fragmentation the average round trip time of a link-local ping
is about 140ms (depending on distance). This puts a strain on both
packet buffer and reassembly buffer on the sending nodes as it has to
handle at least two packets while it is also working on a reply.

Add to that that the at86rf2xx driver is only able to send when it is
not in receive state (and blocks until it leaves) the pinged node has
problems to get out the data it wants to send the echo replies while it
is being spammed with fragments containing follow-up echo requests.

Because the test is mainly about testing fragmentation, not
fragmentation under stress, I suggest we tweak the test parameters a
bit so the test takes the same time, but with less stress on the node.

Also see #113 (comment)

Due to fragmentation the average round trip time of a link-local ping
is about 140ms (depending on distance). This puts a strain on both
packet buffer and reassembly buffer on the sending nodes as it has to
handle at least two packets while it is also working on a reply.

Add to that that the `at86rf2xx` driver is only able to send when it is
not in receive state (and blocks until it leaves) the pinged node has
problems to get out the data it wants to send the echo replies while it
is being spammed with fragments containing follow-up echo requests.

Because the test is mainly about testing fragmentation, not
fragmentation under stress, I suggest we tweak the test parameters a
bit so the test takes the same time, but with less stress on the node.
@miri64 miri64 added the bug label Apr 10, 2019
@miri64 miri64 requested a review from danpetry April 10, 2019 15:27
@danpetry
Copy link
Contributor

With the automated script (i.e. using IoT-LAB), this works if you put the delay up to 300ms. But not 200 ms. I guess there's a longer round trip. Suggest this time instead?

@miri64
Copy link
Member Author

miri64 commented Apr 15, 2019

With the automated script (i.e. using IoT-LAB), this works if you put the delay up to 300ms. But not 200 ms. I guess there's a longer round trip. Suggest this time instead?

Mh... Would have been good to provide some output with that, so I could assess. But I guess 300ms should be fine. Will fix

@miri64
Copy link
Member Author

miri64 commented Apr 15, 2019

Done

@danpetry
Copy link
Contributor

By my reckoning, for the same reason, the timing should be increased also on:

  • test 05 task 02 (static routing test single hop, iotlab-m3, default route)
  • test 05 task 04 (static routing test single hop, iotlab-m3, static route)

Re the iot-lab scripts, I'm having a look at it. But 300ms interval works ok for on-desk tests.

@miri64
Copy link
Member Author

miri64 commented Apr 15, 2019

By my reckoning, for the same reason, the timing should be increased also on:

  • test 05 task 02 (static routing test single hop, iotlab-m3, default route)
  • test 05 task 04 (static routing test single hop, iotlab-m3, static route)

Let's do that in a separate PR

@danpetry danpetry merged commit 22ce710 into RIOT-OS:master Apr 16, 2019
@danpetry
Copy link
Contributor

Let's do that in a separate PR

Will open in a minute

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants