RunCPM: loopback TELNET console on 127.0.0.1:8677 - #1403
Open
peterkaczorowski wants to merge 3 commits into
Open
RunCPM: loopback TELNET console on 127.0.0.1:8677#1403peterkaczorowski wants to merge 3 commits into
peterkaczorowski wants to merge 3 commits into
Conversation
All lib/device/*/cpm devices were near-identical and each embedded its own copy of the RunCPM engine. Introduce a single lib/device/cpm base class (cpmDevice, plus cpmQueueDevice for buses whose engine runs on its own task) and a single shared engine TU (runcpm_core.cpp via runcpm_session.h). Per-bus devices now override only four console-endpoint primitives. Buses without a working CP/M console (rc2014, rs232) inherit a default clean-exit endpoint instead of returning hardcoded values, so CP/M cold-boots and exits cleanly everywhere. Fold N:CPM:// onto the same shared engine. Built against the current RunCPM 5.8 (no engine version change).
Bumps the vendored RunCPM engine from 5.8 to 6.9 while keeping the cpmDevice base-class architecture introduced in the previous commit. The engine-side files are a matched set taken from upstream RunCPM 6.9: ccp.h, cpm.h, cpu.h (+cpu_mhz.h, +debug.h), disk.h, console.h, host.h, ram.h, resource.h and globals.h, plus the 6.9 single-core glue (runcpm_core.cpp, runcpm_session.h, abstraction_fujinet_core.h, which replaces abstraction_fujinet.h). 6.9's console interface adds a getche primitive (blocking read + echo) and widens getch/getche/kbhit to int. Rather than add a per-bus override, cpmDevice synthesises getche generically (read endpoint byte, mask to 7 bits, echo via putch) so no transport had to change: the SIO, IWM, DriveWire, RS232 and RC2014 devices keep their existing ep_* overrides untouched. The N:CPM:// adapter (network-protocol/CPM.cpp), which is not a cpmDevice, gets the same small getche shim. Builds clean on desktop ATARI and APPLE (cpmQueueDevice path) and on the ESP32 fujinet-atari-v1 target (RAM 36.6%, Flash 24.2%).
Add a second access path to the shared RunCPM core: a TELNET console that the Atari reaches via the FujiNet N:TELNET://127.0.0.1:8677 device. The acceptor binds to loopback only and is started from fn_service_loop() after fnWiFi.start() has brought up the lwIP stack (starting it earlier panicked in tcpip_send_msg_wait_sem() with an uninitialised mbox), and is torn down in main() before static destruction so detached worker threads do not race the queue mutexes at exit. Also lets the reset button preempt an active session and yields core 1 in the idle kbhit poll to avoid an ESP32 watchdog hang.
This was referenced Jun 29, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a second access path to the shared RunCPM core: a loopback TELNET
console that the Atari reaches via
N:TELNET://127.0.0.1:8677. Rebased ontothe cpmDevice + 6.9 stack.
Stacks on #1402 (engine 6.9 on cpmDevice), which stacks on #1401
(cpmDevice consolidation). Merge in order #1401 → #1402 → this.
What changed
lib/device/sio/vm_telnet.{cpp,h}— a loopback acceptor (binds127.0.0.1:8677 only) that bridges a TELNET client to the shared RunCPM core
via two byte queues exposed as a
runcpm_console_ops(the same consoleinterface every transport uses, including the new
getche).src/main.cpp— the acceptor is started fromfn_service_loop()afterfnWiFi.start()has brought up the lwIP stack (starting earlier panicked intcpip_send_msg_wait_sem()with an uninitialised mbox), and torn down inmain()before static destruction so detached workers don't race the queuemutexes at exit.
kbhitpoll yieldscore 1 to avoid an ESP32 watchdog hang.
Build / test
fujinet-atari-v1: builds clean, RAM 36.6%, Flash 24.2%.N:TELNET://127.0.0.1:8677and runs a full CP/M session through the sharedcore.