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

firmware: confs. for a better performance of RPL routing protocol. #344

Merged
merged 1 commit into from
Sep 13, 2022

Conversation

luisan00
Copy link
Contributor

Contribution description

A few C flags allow alternative parents and add space for increasing the number of nodes network nodes.

Testing procedure

Number of alternative parents, is set to 2, so the minimum setup needs at least 3 nodes:

  • 2 as DODAG, build and flash mesh4all_rpl_dodag
  • 1 as DAG, build and flash mesh4all_rpl_dag

Turn on all nodes and make sure the two DODAG nodes are using different IPV6 addresses.
Open a terminal on the DAG node and verify that it can use alternative parents.

Using zep create a network at least of 50 nodes acting as DAG and only one as dodag, turn all nodes on.
The DODAG node should show you 50 entries, it correspond to 50 DAG nodes in the network

Issues/PRs references

None

@luisan00 luisan00 added this to the 0.22.12-RC2 milestone Sep 12, 2022
@luisan00 luisan00 requested a review from CW-75 September 12, 2022 13:19
CW-75
CW-75 previously approved these changes Sep 12, 2022
Copy link
Collaborator

@CW-75 CW-75 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good for me, IMO is a good idea announce that only is supported the storing mode and all its limitations, also it's better knows to the nib storage limit and do it settable . Good job 👍

@CW-75 CW-75 self-requested a review September 12, 2022 15:55
@CW-75 CW-75 dismissed their stale review September 12, 2022 15:56

testing procedure

Copy link
Collaborator

@CW-75 CW-75 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Collaborator

@CW-75 CW-75 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@luisan00 luisan00 merged commit 82d107d into Mesh4all:main Sep 13, 2022
@luisan00 luisan00 deleted the feature/rpl_setup branch September 13, 2022 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants