Skip to content

shell/cmds: add udptty command#19742

Merged
benpicco merged 1 commit intoRIOT-OS:masterfrom
benpicco:shell_cmd_udptty
Feb 2, 2024
Merged

shell/cmds: add udptty command#19742
benpicco merged 1 commit intoRIOT-OS:masterfrom
benpicco:shell_cmd_udptty

Conversation

@benpicco
Copy link
Copy Markdown
Contributor

@benpicco benpicco commented Jun 18, 2023

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
maribu
maribu reviewed Jun 19, 2023
View reviewed changes
Copy link
Copy Markdown
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

@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
Copy Markdown

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
@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

Reviewers

@maribu maribu maribu approved these changes

@fabian18 fabian18 Awaiting requested review from fabian18

Assignees

No one assigned

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

Milestone

Release 2024.04

Development

Successfully merging this pull request may close these issues.

4 participants

@benpicco @riot-ci @maribu @MrKevinWeiss