Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

shell/cmds: add udptty command #19742

Merged
merged 1 commit into from
Feb 2, 2024
Merged

Conversation

benpicco
Copy link
Contributor

Contribution description

To facilitate remote debugging in isolated networks, we need a RIOT client for stdio_udp.
This adds a udptty command that fulfills this role.

Testing procedure

On one board, enable the stdio_udp module, on the other board enable the shell_cmd_udptty module.

You can now connect to the remote board from within RIOT:

> ifconfig
Iface  6  HWaddr: 7A:37:FC:7D:1A:AF 
          L2-PDU:1500  MTU:1500  HL:64  Source address length: 6
          Link type: wired
          inet6 addr: fe80::7837:fcff:fe7d:1aaf  scope: link  VAL
          inet6 group: ff02::1
          inet6 group: ff02::1:ff7d:1aaf
          inet6 group: ff02::1a
          
          Statistics for Layer 2
            RX packets 5  bytes 542
            TX packets 2 (Multicast: 2)  bytes 140
            TX succeeded 0 errors 0
          Statistics for IPv6
            RX packets 5  bytes 472
            TX packets 2 (Multicast: 2)  bytes 112
            TX succeeded 2 errors 0

> udptty fe80::58b0:8ff:fe67:ad82 2323
> ifconfig
6 HWaddr: 5A:B0:08:67:AD:82 
          L2-PDU:1500  MTU:1500  HL:64  Source address length: 6
          Link type: wired
          inet6 addr: fe80::58b0:8ff:fe67:ad82  scope: link  VAL
          inet6 group: ff02::1
          inet6 group: ff02::1:ff67:ad82
          inet6 group: ff02::1a
                    Statistics for Layer 2
            RX packets 10  bytes 808
            TX packets 157 (Multicast: 4)  bytes 12044
            TX succeeded 0 errors 0
          Statistics for IPv6
            RX packets 10  bytes 668
            TX packets 158 (Multicast: 4)  bytes 10055
            TX succeeded 158 errors 0

Issues/PRs references

@github-actions github-actions bot added the Area: sys Area: System label Jun 18, 2023
Copy link
Member

@maribu maribu left a comment

Choose a reason for hiding this comment

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

some suggestions inline, otherwise lgtm

sys/shell/cmds/udptty.c Show resolved Hide resolved
sys/shell/cmds/udptty.c Outdated Show resolved Hide resolved
@github-actions github-actions bot added the Area: build system Area: Build system label Feb 2, 2024
@benpicco benpicco requested a review from fabian18 February 2, 2024 11:52
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Feb 2, 2024
@riot-ci
Copy link

riot-ci commented Feb 2, 2024

Murdock results

✔️ PASSED

13deed1 shell/cmds: add udptty command

Success Failures Total Runtime
8627 0 8627 08m:37s

Artifacts

@benpicco benpicco added this pull request to the merge queue Feb 2, 2024
Merged via the queue into RIOT-OS:master with commit 85dcc1b Feb 2, 2024
27 checks passed
@benpicco benpicco deleted the shell_cmd_udptty branch February 2, 2024 19:20
@MrKevinWeiss MrKevinWeiss added this to the Release 2024.04 milestone Apr 30, 2024
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: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants