Skip to content

gnrc_sixloenc: introduce pseudo-module to send 6Lo frames over Ethernet#10665

Merged
cgundogan merged 5 commits intoRIOT-OS:masterfrom
miri64:gnrc_sixloenc/feat/initial
Jan 7, 2019
Merged

gnrc_sixloenc: introduce pseudo-module to send 6Lo frames over Ethernet#10665
cgundogan merged 5 commits intoRIOT-OS:masterfrom
miri64:gnrc_sixloenc/feat/initial

Conversation

@miri64
Copy link
Copy Markdown
Member

@miri64 miri64 commented Dec 27, 2018

Contribution description

See title (it implements RFC 7973). Also activates capabilities, once this module is compiled in. I will add a way to configure the 6Lo capabilities of an Ethernet interface dynamically.

Testing procedure

  • Compile gnrc_networking for native with USEMODULE += gnrc_sixloenc,
  • use dist/tools/tapsetup/tapsetup to create two bridged TAP interfaces,
  • start a wireshark sniffing the bridge and two native instances, and
  • try to ping one of the native instances from the other.

Wireshark should show a 6Lo dispatch within the Ethernet frame (not sure why it is showing "wrong checksum", RIOT is happy with it):

image

Issues/PRs references

Redoing #4861.

Resolves #4857.

@miri64 miri64 added Area: network Area: Networking Type: new feature The issue requests / The PR implemements a new feature for RIOT labels Dec 27, 2018
@miri64 miri64 requested review from bergzand and cgundogan December 27, 2018 21:30
@miri64 miri64 force-pushed the gnrc_sixloenc/feat/initial branch from 75b8fd1 to 481f742 Compare January 4, 2019 12:33
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Jan 4, 2019

Rebased to current master

Copy link
Copy Markdown
Member

@cgundogan cgundogan left a comment

Choose a reason for hiding this comment

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

ACK. I can confirm that the test works as specified. The wrong checksum is weird, though. Might be a wireshark issue. Btw, with the current approach no 6Lo fragmentation is used for packets larger than the configured MTU.

@cgundogan cgundogan added Reviewed: 1-fundamentals The fundamentals of the PR were reviewed according to the maintainer guidelines Reviewed: 2-code-design The code design of the PR was reviewed according to the maintainer guidelines Reviewed: 3-testing The PR was tested according to the maintainer guidelines CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jan 7, 2019
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Jan 7, 2019

Btw, with the current approach no 6Lo fragmentation is used for packets larger than the configured MTU.

I am aware of that and plan to do that later in the future.

@cgundogan cgundogan merged commit aa2de01 into RIOT-OS:master Jan 7, 2019
@miri64 miri64 deleted the gnrc_sixloenc/feat/initial branch January 7, 2019 11:00
@miri64 miri64 added this to the Release 2019.01 milestone Jan 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: network Area: Networking CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Reviewed: 1-fundamentals The fundamentals of the PR were reviewed according to the maintainer guidelines Reviewed: 2-code-design The code design of the PR was reviewed according to the maintainer guidelines Reviewed: 3-testing The PR was tested according to the maintainer guidelines Type: new feature The issue requests / The PR implemements a new feature for RIOT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature request: 6lo over Ethernet

2 participants