Skip to content

sys/net/ipv4: add header print function - #22678

Draft
basilfx wants to merge 6 commits into
RIOT-OS:masterfrom
basilfx:feature/ipv4_header_print
Draft

sys/net/ipv4: add header print function#22678
basilfx wants to merge 6 commits into
RIOT-OS:masterfrom
basilfx:feature/ipv4_header_print

Conversation

@basilfx

@basilfx basilfx commented Sep 8, 2026

Copy link
Copy Markdown
Member

Contribution description

Note

This PR is still draft, because it also contains the commits of #22677.

This adds a header print utility, just like ipv6_hdr_print. This will eventually be used by gnrc_pktdump.

Testing procedure

This PR does not add any tests for this.

Issues/PRs references

#22676

Declaration of AI-Tools / LLMs usage:

AI-Tools / LLMs that were used are:

  • Claude Code Opus 5, which I then reviewed and reworked.

The internet header length occupies the four least significant bits,
of the combined version-internet header length field. Its value (
multiplied by four) indicates the length of the IPv4 header.
This adds the header checksum and the internet checksum methods.
@github-actions github-actions Bot added Area: network Area: Networking Area: tests Area: tests and testing framework Area: sys Area: System labels Sep 8, 2026
@basilfx basilfx mentioned this pull request Sep 8, 2026
@AnnsAnns AnnsAnns added the AI: Co-Authored PR/Issue relies on AI. Documentation / Code was partly generated by AI. label Sep 9, 2026
@crasbe crasbe added Type: new feature The issue requests / The PR implemements a new feature for RIOT CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Sep 9, 2026
Comment thread sys/include/net/ipv4/hdr.h
Comment on lines +6 to +10
/**
* @{
*
* @file
*/

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This could be extended a bit.

@crasbe crasbe added the State: waiting for other PR State: The PR requires another PR to be merged first label Sep 9, 2026
@riot-ci

riot-ci commented Sep 9, 2026

Copy link
Copy Markdown

Murdock results

✔️ PASSED

4739230 sys/net/ipv4: add header print function

Success Failures Total Runtime
11292 0 11293 12m:32s

Artifacts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI: Co-Authored PR/Issue relies on AI. Documentation / Code was partly generated by AI. Area: network Area: Networking Area: sys Area: System Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR State: waiting for other PR State: The PR requires another PR to be merged first 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.

4 participants