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

gnrc/ipv6_auto_subnets: add CONFIG_GNRC_IPV6_AUTO_SUBNETS_STATIC option #17429

Merged
merged 1 commit into from
Oct 26, 2022

Conversation

benpicco
Copy link
Contributor

@benpicco benpicco commented Dec 20, 2021

Contribution description

In situations with high packet loss, if all sync packets are lost, conflicting subnets will be configured.

If the network is static, this can be prevented by always using the highest number of subnets that has been observed.

This assumes no nodes are physically added / removed from the network. This is typically given in a wired sensor network.

Testing procedure

Add

CFLAGS += -DCONFIG_GNRC_IPV6_AUTO_SUBNETS_STATIC=1

to the Makefile of examples/gnrc_networking_subnets.

Configure a topology with multiple routers on the same subnet. Now even if you remove a router, the subnet assignments should not change.

Issues/PRs references

@github-actions github-actions bot added Area: network Area: Networking Area: sys Area: System labels Dec 20, 2021
@benpicco benpicco removed the Area: sys Area: System label Dec 20, 2021
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jan 4, 2022
@benpicco benpicco force-pushed the gnrc/ipv6_auto_subnets-static branch from 329b770 to 67bf76b Compare May 24, 2022 21:34
@github-actions github-actions bot added the Area: sys Area: System label May 24, 2022
@miri64
Copy link
Member

miri64 commented May 25, 2022

Testing procedure

Add

CFLAGS += -DCONFIG_GNRC_IPV6_AUTO_SUBNETS_STATIC=1

to the Makefile of examples/gnrc_networking_subnets.

Would be great if RIOT_CONFIG_GNRC_IPV6_AUTO_SUBNETS_STATIC=y would also work ;-) Read: Could you provide a Kconfig variable as well, please?

@benpicco benpicco force-pushed the gnrc/ipv6_auto_subnets-static branch from 67bf76b to e484241 Compare October 20, 2022 14:13
@benpicco benpicco removed the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Oct 20, 2022
@benpicco
Copy link
Contributor Author

For that I would first need to add Kconfig to that subsystem

In situations with high packet loss, if all sync packets are lost, conflicting
subnets will be configured.

If the network is static, this can be prevented by always using the highest number
of subnets that has been observed.

This assumes no nodes are physically added / removed from the network.
@benpicco benpicco force-pushed the gnrc/ipv6_auto_subnets-static branch from e484241 to b068bd3 Compare October 25, 2022 17:46
Copy link
Member

@miri64 miri64 left a comment

Choose a reason for hiding this comment

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

ACK

@miri64 miri64 added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Oct 25, 2022
@miri64 miri64 enabled auto-merge October 25, 2022 17:56
@riot-ci
Copy link

riot-ci commented Oct 25, 2022

Murdock results

✔️ PASSED

b068bd3 gnrc/ipv6_auto_subnets: add CONFIG_GNRC_IPV6_AUTO_SUBNETS_STATIC option

Success Failures Total Runtime
1991 0 1991 07m:20s

Artifacts

This only reflects a subset of all builds from https://ci-prod.riot-os.org. Please refer to https://ci.riot-os.org for a complete build for now.

@miri64 miri64 merged commit ff884dd into RIOT-OS:master Oct 26, 2022
@benpicco benpicco deleted the gnrc/ipv6_auto_subnets-static branch October 26, 2022 13:09
@kaspar030 kaspar030 added this to the Release 2023.01 milestone Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: network Area: Networking Area: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants