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.

Sign up for GitHub

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

Jump to bottom

Firmware implementation of OpenBenchmark API v0.0.1. #470

Open
wants to merge 42 commits into
base: develop
Choose a base branch
from
Open

Firmware implementation of OpenBenchmark API v0.0.1. #470

wants to merge 42 commits into from

Conversation

malishav
Copy link
Contributor

@malishav malishav commented Mar 23, 2019 •
edited
Loading

This PR implements the firmware components of the OpenBenchmark API and their interface with the OpenVisualizer. The PR:

  • introduces a CoAP app called cbenchmark that can be triggered over serial to send packets and that reacts to the CoAP packets adhering to the OpenBenchmark API.
  • Implements the RFC7967 CoAP No Response option in order to enable CoAP requests without a corresponding response being generated by the network
  • Generation of asynchronous events over serial with the parameters as needed by OpenBenchmark platform
    -Extends the radio.h API in order to provide support for configuring TX power. This is a necessary extension in order to enable an external agent, e.g. OpenVisualizer through CLI or OpenBenchmark platform, to configure the TX power of a given mote. New radio_setTxPower API is implemented for CC2420, AT86RF231, and CC2538 -based boards. Mock up functions are added for other radio chips that are not supported at the moment.
  • Adds support for CCM at iotlab-m3 nodes and their derivatives. It implements the CCM using hardware acceleration available in the AT86RF231 chip. While the CCM implementation passes all the unit tests available, its performance is bad due to the SPI transfer. For CCM to complete on a 127-byte frame, it takes ~5ms, making it impossible to use it for link-layer security processing. The provided CCM implementation enables the secure join protocol to complete, so it is therefore deemed useful.
  • Removes the trailing whitespaces in all touched files.

The PR resolves the following issues:

@malishav malishav force-pushed the develop_FW-808 branch 3 times, most recently from 2283091 to e98ad71 Compare March 31, 2019 21:29
@malishav malishav changed the title WIP: Implement OpenBenchmark API v0.0.1 OV-7. Implement OpenBenchmark API v0.0.1. Mar 31, 2019
@malishav malishav changed the title OV-7. Implement OpenBenchmark API v0.0.1. OV-7. Firmware implementation of OpenBenchmark API v0.0.1. Apr 1, 2019
@malishav malishav changed the title OV-7. Firmware implementation of OpenBenchmark API v0.0.1. Firmware implementation of OpenBenchmark API v0.0.1. Apr 1, 2019
This was referenced Apr 1, 2019
Closed
Closed
@malishav
FW-807. Extend the radio API with setTxPower() function.
39f407e
@malishav
FW-807. Implement setTxPower for CC2420.
e5e288a 
Remove trailing whitespaces.
@malishav
FW-807. Imlement setTxPower for openmote-cc2538.
da948b0
@malishav
FW-807. Implement setTxPower for AT86RF231.
5e961c1
@malishav
FW-807. use cc2538 implementation of setTxPower for openmote-b at 2.4…
f9cd289 
… GHz.

Remove trailing whitespaces.
@malishav
FW-807. Add setTxPower mockups for all radio chips.
1a8f479
@malishav
FW-806. Add placeholders for new command handling needed by OpenBench…
b574870 
…mark.
@malishav
FW-806. Handle setTxPower command over serial.
f2f52b0
@malishav
FW-806. Add hop limit to packet metadata.
5dda5ce 
This is useful in order to calculate how many hops a packet traveled.
@malishav
FW-806. Add serial handling of sendPacket command and invoke a callback.
384fdcc
@malishav
FW-806. Remove sendPacket parsing logic from openserial.
23f37de
@malishav
FW-806. Do not print join event periodically.
1c2b997
@malishav
FW-806. Add special serial frame type for benchmark data.
cc5bff2
@malishav
FW-806. Remove remnants and separate STATUS enums from benchmark.
376c486
@malishav
FW-806. Align event IDs with OV.
8f912d1
@malishav
FW-806. When generating benchmark serial events, include full EUI-64.
811cf60
@malishav
FW-806. Send sync event asynchronously as well.
5fea32e
@malishav
FW-806. Special event for dag root sending packets.
0d2ded6
@malishav
FW-806. Generate the event immediately after sending the packet at da…
a9c4e21 
…g root.
@malishav
FW-829. Implement No Response option handling as per RFC7967.
3675d04
@malishav
FW-808. Add cbenchmark app files and placeholders.
9dfdc8e
@malishav
FW-808. cbenchmark init and placeholders.
37e0b15
@malishav
FW-808. Add missing functions to python build.
c945b13
@malishav
FW-808. Implement sendPacket callback.
5899a31
@malishav
FW-808. Implement OpenBenchmark confirmable through No Response option.
d6cbfcc
@malishav
FW-808. Implement burst handling.
b3b6061
@malishav
FW-808. Implement sendPacket parsing logic within cbenchmark callback.
3827b32
@malishav
FW-808. Mote detailed error logging in cbenchmark.
7b9e3d6 
+ Bug fix.
@malishav
FW-808. Use port 5684 to communicate with DAG root over the air.
3dcbbf6
@malishav
FW-808. Parse the received packet token in the payload.
826d39b
@malishav
FW-808. Implement packet sent/received event notifications in firmware.
4d522dd
@malishav
FW-808. Revert to using default CoAP port for openbenchmark.
239d4b1
@malishav
FW-808. Fix warnings in cbenchmark.c.
16403ce
@malishav
FW-808. Adapt cbenchmark functions to the new serial printing API.
e12c4c2
@malishav
FW-808. Align cbenchmark with new events.
3408bd9
@malishav
FW-808. packetToken as the last bytes in the app payload.
cbcfdf5
@malishav
FW-808. Send responses when needed and generate events.
3e3a69f 
Packet token in the responses is the token from the request, with the
last byte incremented by one.
@malishav
cjoin: Reduce join time by polling more frequently.
593fcfc
@malishav
iotlab: Reduce delayTx for better synchro.
931418f
@malishav
FW-808. Add desync event in firmware.
cdbb695
@changtengfei
Copy link
Member

Can one of the admins verify this patch?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@malishav @changtengfei