shell_commands: provide command for 6Lo context administration at 6LBR by miri64 · Pull Request #3750 · RIOT-OS/RIOT

miri64 · 2015-08-31T14:35:12Z

Provides a shell command to administrate 6LoWPAN compression contexts at the border router.

Depends on ~~#3134~~ (merged) (and #3525 to work properly).

miri64 · 2015-08-31T23:32:21Z

Rebased to current #3134

miri64 · 2015-09-02T22:38:46Z

Rebased to current #3134.

miri64 · 2015-09-03T08:35:52Z

Rebased to current #3134

miri64 · 2015-09-09T10:13:28Z

Rebased to current #3134

miri64 · 2015-09-10T13:31:39Z

Rebased to current #3134

miri64 · 2015-09-11T18:00:05Z

Rebased to current master

OlegHahm · 2015-09-11T20:16:01Z

What do I have to do to get this building with gnrc_networking example? Just replacing gnrc_ipv6_router_default by gnrc_sixlowpan_nd_border_router or adding it doesn't seem to work.

miri64 · 2015-09-12T10:42:27Z

gnrc_ipv6_border_router_default

OlegHahm · 2015-09-13T13:32:52Z

make sense ;)

OlegHahm · 2015-09-13T17:21:52Z

For the record: the (pseudo) module is called gnrc_sixlowpan_border_router_default.

A dependency to xtimer is missing.

OlegHahm · 2015-09-13T17:24:54Z

sys/shell/commands/sc_gnrc_6ctx.c

I think you can shorten this to something like: "'del' will only invalidate the context for compression which can be reassigned after 5 min."

miri64 · 2015-09-14T11:43:00Z

A dependency to xtimer is missing.

Should I add it as a dependency for shell_commands? That seems weird to me... Because the xtimer is only used by the shell command, not the border router itself (yet).

OlegHahm · 2015-09-14T13:59:25Z

Hm, maybe we just wait for switch to xtimer then...

kaspar030 · 2015-09-14T14:02:55Z

+1

miri64 · 2015-09-14T14:10:05Z

As stated in the description ;-)

OlegHahm · 2015-09-16T20:44:46Z

I guess a rebase is required.

miri64 · 2015-09-16T21:49:09Z

Rebased

OlegHahm · 2015-09-17T09:06:51Z

please squash

miri64 · 2015-09-17T09:09:17Z

Squashed

miri64 · 2015-09-17T11:59:34Z

(but not pushed >.<)

OlegHahm · 2015-09-18T21:59:41Z

How can I test and how and when should the configured context get propagated?

OlegHahm · 2015-09-18T22:01:54Z

After configuring a context for my prefix, pinging to or via the 6LBR doesn't work any more.

OlegHahm · 2015-09-18T22:02:28Z

However, that's probably not related to the shell commands themselves.

miri64 · 2015-09-21T13:58:39Z

Rebased to current master.

miri64 · 2015-09-22T12:26:06Z

Added immediate advertisement for contexts (and fixed a bug that was revealed by that in net/gnrc/ndp/internal.h)

miri64 · 2015-09-22T12:26:29Z

Fix for #3750 (comment) will follow. I think I know what's happening.

miri64 · 2015-09-22T21:47:52Z

Fix for #3750 (comment) will follow. I think I know what's happening.

it's #3928 if that wasn't clear.

OlegHahm · 2015-09-23T22:39:55Z

ACK

OlegHahm · 2015-09-23T22:41:08Z

Please squash and merge as soon as Travis is green. With #3949 this is working. (However, this PR iteelf is sane anyway.)

miri64 · 2015-09-23T23:37:21Z

Squashed and rebased.

shell_commands: provide command for 6Lo context administration at 6LBR

miri64 added Area: network Area: Networking Type: new feature The issue requests / The PR implemements a new feature for RIOT NSTF labels Aug 31, 2015

miri64 assigned OlegHahm Aug 31, 2015

miri64 added this to the Release 2015.08 milestone Aug 31, 2015

OlegHahm added the State: waiting for other PR State: The PR requires another PR to be merged first label Aug 31, 2015

miri64 force-pushed the shell/feat/6ctx branch from 4ddc248 to ffca8e8 Compare August 31, 2015 23:32

miri64 force-pushed the shell/feat/6ctx branch 2 times, most recently from ce5160c to ad7e203 Compare September 2, 2015 22:38

miri64 force-pushed the shell/feat/6ctx branch 2 times, most recently from 659faea to 58cb27f Compare September 3, 2015 08:33

miri64 force-pushed the shell/feat/6ctx branch from 58cb27f to cf36737 Compare September 9, 2015 10:07

miri64 force-pushed the shell/feat/6ctx branch 2 times, most recently from 6eae09e to 2e96ec1 Compare September 11, 2015 17:59

OlegHahm reviewed Sep 13, 2015
View reviewed changes

OlegHahm removed the State: waiting for other PR State: The PR requires another PR to be merged first label Sep 16, 2015

miri64 force-pushed the shell/feat/6ctx branch from 2e96ec1 to d9b9b04 Compare September 16, 2015 21:49

OlegHahm added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Sep 17, 2015

miri64 force-pushed the shell/feat/6ctx branch from 56ed209 to c6c4841 Compare September 17, 2015 11:59

miri64 mentioned this pull request Sep 17, 2015

examples: provide example application for a 6LoWPAN border router #3886

Merged

miri64 force-pushed the shell/feat/6ctx branch from c6c4841 to ba0e1bd Compare September 21, 2015 13:58

miri64 mentioned this pull request Sep 22, 2015

gnrc_sixlowpan_nd: fix 6ctx option check #3928

Merged

OlegHahm added the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Sep 23, 2015

miri64 added 2 commits September 24, 2015 01:36

shell_commands: provide command for 6Lo context administration

cc9aa35

gnrc_ndp_internal: add missing include

dd69fd0

miri64 removed the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Sep 23, 2015

miri64 force-pushed the shell/feat/6ctx branch from 2bd606d to dd69fd0 Compare September 23, 2015 23:37

miri64 added a commit that referenced this pull request Sep 24, 2015

Merge pull request #3750 from authmillenon/shell/feat/6ctx

dd7b96c

shell_commands: provide command for 6Lo context administration at 6LBR

miri64 merged commit dd7b96c into RIOT-OS:master Sep 24, 2015

miri64 deleted the shell/feat/6ctx branch September 24, 2015 00:08

Conversation

miri64 commented Aug 31, 2015

Uh oh!

miri64 commented Aug 31, 2015

Uh oh!

miri64 commented Sep 2, 2015

Uh oh!

miri64 commented Sep 3, 2015

Uh oh!

miri64 commented Sep 9, 2015

Uh oh!

miri64 commented Sep 10, 2015

Uh oh!

miri64 commented Sep 11, 2015

Uh oh!

OlegHahm commented Sep 11, 2015

Uh oh!

miri64 commented Sep 12, 2015

Uh oh!

OlegHahm commented Sep 13, 2015

Uh oh!

OlegHahm commented Sep 13, 2015

Uh oh!

OlegHahm Sep 13, 2015

Choose a reason for hiding this comment

Uh oh!

OlegHahm Sep 17, 2015

Choose a reason for hiding this comment

Uh oh!

miri64 Sep 17, 2015

Choose a reason for hiding this comment

Uh oh!

miri64 commented Sep 14, 2015

Uh oh!

OlegHahm commented Sep 14, 2015

Uh oh!

kaspar030 commented Sep 14, 2015

Uh oh!

miri64 commented Sep 14, 2015

Uh oh!

OlegHahm commented Sep 16, 2015

Uh oh!

miri64 commented Sep 16, 2015

Uh oh!

OlegHahm commented Sep 17, 2015

Uh oh!

miri64 commented Sep 17, 2015

Uh oh!

miri64 commented Sep 17, 2015

Uh oh!

OlegHahm commented Sep 18, 2015

Uh oh!

OlegHahm commented Sep 18, 2015

Uh oh!

OlegHahm commented Sep 18, 2015

Uh oh!

miri64 commented Sep 21, 2015

Uh oh!

miri64 commented Sep 22, 2015

Uh oh!

miri64 commented Sep 22, 2015

Uh oh!

miri64 commented Sep 22, 2015

Uh oh!

OlegHahm commented Sep 23, 2015

Uh oh!

OlegHahm commented Sep 23, 2015

Uh oh!

miri64 commented Sep 23, 2015

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants