Skip to content

sys/net/gnrc: experimental gnrc_schc implementation#15040

Closed
bartmoons wants to merge 2 commits intoRIOT-OS:masterfrom
bartmoons:master
Closed

sys/net/gnrc: experimental gnrc_schc implementation#15040
bartmoons wants to merge 2 commits intoRIOT-OS:masterfrom
bartmoons:master

Conversation

@bartmoons
Copy link
Copy Markdown
Contributor

Added libschc as a pkg, which can be run with 2 examples; gnrc_schc (native) and gnrc_schc_lorawan (B-L072Z-LRWAN1).
The README of both examples gives information on how to run the examples.
Also added sys/net/gnrc/schc, holding the gnrc_schc.c file.

The implementation is very experimental!

@kaspar030 kaspar030 added Area: network Area: Networking Type: new feature The issue requests / The PR implemements a new feature for RIOT labels Sep 17, 2020
@kaspar030
Copy link
Copy Markdown
Contributor

@bartmoons pretty substantial first contribution, congrats!

@kaspar030 kaspar030 changed the title experimental gnrc_schc implementation sys/net/gnrc: experimental gnrc_schc implementation Sep 17, 2020
@jia200x
Copy link
Copy Markdown
Member

jia200x commented Sep 17, 2020

I'm eager to try this!

@cgundogan
Copy link
Copy Markdown
Member

@bartmoons great! I'll also have a look at this. At first glance, it seems you are copying some of the applications in examples? We could probably deduplicate a lot of the code (e.g., extend the Makefiles of the existing examples to use gnrc_schc, instead of creating new apps). But I need to take a closer look at this first.

@benpicco
Copy link
Copy Markdown
Contributor

We could probably deduplicate a lot of the code (e.g., extend the Makefiles of the existing examples to use gnrc_schc, instead of creating new apps).

But then CI would not be able to build this.

@cgundogan
Copy link
Copy Markdown
Member

But then CI would not be able to build this.

maybe a good candidate for iterating Kconfig configurations with incremental builds in the CI, @leandrolanzieri ? (:

In any way, we can also have a small compile-test application (or a more sophisticated one) in tests/. The example could stay minimal.

@bartmoons
Copy link
Copy Markdown
Contributor Author

@bartmoons great! I'll also have a look at this. At first glance, it seems you are copying some of the applications in examples? We could probably deduplicate a lot of the code (e.g., extend the Makefiles of the existing examples to use gnrc_schc, instead of creating new apps). But I need to take a closer look at this first.

Yes, indeed. Probably running the gnrc_networking example with the gnrc_schc module is a better approach. IDK about the LoRaWAN example; was more to see if it would work with a different netif.

@jia200x
Copy link
Copy Markdown
Member

jia200x commented Sep 17, 2020

@bartmoons do you have by chance any recommendations on what to run close the the Application Server in LoRaWAN in order to send/receive IPv6 data over LoRaWAN?

@bartmoons
Copy link
Copy Markdown
Contributor Author

bartmoons commented Sep 17, 2020

@jia200x I'm currently using click for this. However, should be fairly easy to transform the gnrc_border_router to route the decompressed packets. But that doesn't implement an MQTT client ofc..

@jia200x
Copy link
Copy Markdown
Member

jia200x commented Sep 17, 2020

I will give it a try, thanks!

@emmanuelsearch
Copy link
Copy Markdown
Member

@bartmoons any news on this? In particular is libSCHC on compliant with https://tools.ietf.org/html/rfc9011 ? Or is the plan that it will be?

@bartmoons
Copy link
Copy Markdown
Contributor Author

We plan to provide full support for both RFC 8724 and RFC 9011, if time permits at least. This could be ready by this summer.

@MrKevinWeiss MrKevinWeiss added this to the Release 2021.07 milestone Jun 21, 2021
@MrKevinWeiss MrKevinWeiss removed this from the Release 2021.07 milestone Jul 15, 2021
@stale
Copy link
Copy Markdown

stale bot commented Mar 2, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions.

@stale stale bot added the State: stale State: The issue / PR has no activity for >185 days label Mar 2, 2022
@stale stale bot closed this Apr 16, 2022
@miri64 miri64 reopened this Apr 16, 2022
@stale stale bot removed the State: stale State: The issue / PR has no activity for >185 days label Apr 16, 2022
@github-actions github-actions bot added Area: build system Area: Build system Area: doc Area: Documentation Area: examples Area: Example Applications Area: Kconfig Area: Kconfig integration Area: LoRa Area: LoRa radio support Area: pkg Area: External package ports Area: sys Area: System labels Apr 16, 2022
@miri64
Copy link
Copy Markdown
Member

miri64 commented Jun 17, 2022

If nobody else is already working on this, I'd like to pick this up in the next few weeks, as it seem to have become stale. Anyone against that?

@miri64 miri64 mentioned this pull request Aug 25, 2022
3 tasks
@miri64
Copy link
Copy Markdown
Member

miri64 commented Aug 25, 2022

I started my picking up in #18515. I decided to split this up into two PRs: first the package import, then the integration into GNRC.

@miri64 miri64 mentioned this pull request Oct 6, 2022
3 tasks
@miri64
Copy link
Copy Markdown
Member

miri64 commented Oct 6, 2022

With #18515 and #18700 the contents of this PR are now adopted in new PRs.

@miri64 miri64 closed this Oct 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: build system Area: Build system Area: doc Area: Documentation Area: examples Area: Example Applications Area: Kconfig Area: Kconfig integration Area: LoRa Area: LoRa radio support Area: network Area: Networking Area: pkg Area: External package ports Area: sys Area: System 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.

8 participants