You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This PR ensures that the (last) default route added with gnrc_ipv6_nib_ft_add() is the primary default route. This way the user (or the routing protocol) has full control over which route is the primary one without changing the API syntax (just the semantics).
miri64
added
Process: API change
Integration Process: PR contains or issue proposes an API change. Should be handled with care.
Type: enhancement
The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
GNRC
Area: network
Area: Networking
CI: ready for build
If set, CI server will compile all applications for all available boards for the labeled PR
labels
Nov 19, 2017
@miri64 looks like I need this PR for the RPL port. Currently (on native), the native instances set default routes to each other because of Router Advertisements (gnrc_networking). I think that messes with RPL ..
it shouldn't just be the primary one, it also shouldn't be replaced by ND internals? Should we disable Router Advertisements if an actual routing protocol is compiled in?
it shouldn't just be the primary one, it also shouldn't be replaced by ND internals? Should we disable Router Advertisements if an actual routing protocol is compiled in?
At least for RPL this might be a good idea in the long run (and as a short term thing as well). But we still don't have a replacement for 6Lo Context Options in RPL, right?
nope we don't .. I will handle this 6co thing as soon as I'm back. Takes too long for that to be standardized.
Awfully! Until then: deactivating RAs is the tiniest thing. You can even do that via ifconfig before starting RPL ;-).
cgundogan
added
CI: ready for build
If set, CI server will compile all applications for all available boards for the labeled PR
and removed
CI: ready for build
If set, CI server will compile all applications for all available boards for the labeled PR
labels
Nov 19, 2017
Not yet, but basically every router in a 6LoWPAN has that flag deactivated, as per RFC 6775, except for the border router (which should be the RPL root) so it isn't that tedious to deactivate it manually ;-). But will think about that config thingy :-).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Area: networkArea: NetworkingCI: ready for buildIf set, CI server will compile all applications for all available boards for the labeled PRProcess: API changeIntegration Process: PR contains or issue proposes an API change. Should be handled with care.Type: enhancementThe issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR ensures that the (last) default route added with
gnrc_ipv6_nib_ft_add()is the primary default route. This way the user (or the routing protocol) has full control over which route is the primary one without changing the API syntax (just the semantics).