Interactive shell with framebuffer output #62
Merged
Conversation
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
Initial implementation of keyboard→stdin→userspace pipeline: Kernel infrastructure: - kernel/src/ipc/stdin.rs: Ring buffer for stdin with blocked reader tracking - keyboard.rs: Route decoded characters to stdin buffer - syscall/handlers.rs: Blocking read() for fd 0 with thread blocking/waking Userspace: - init_shell.rs: Minimal interactive shell with REPL loop - Built-in commands: help, echo, ps, uptime, clear, exit - Line editing with backspace support - Yields on EAGAIN and retries - stdin_test.rs: Test program for stdin reading Build system: - xtask: Added interactive mode and stdin test support - userspace build: Include new test programs Documentation: - docs/planning/interactive-shell/IMPLEMENTATION_PLAN.md: Full architecture and phased plan Status: Compiles but not yet tested. Thread blocking infrastructure needs verification. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Phase 2 of interactive shell implementation: - Add ShellFrameBuffer for direct framebuffer text rendering - Uses noto-sans-mono-bitmap for glyph rendering - Supports scrolling, backspace, line wrapping - Separate from kernel log output - Route shell output to framebuffer in interactive mode - sys_write() calls write_to_framebuffer() for stdout/stderr - Keyboard echo also goes to framebuffer - Kernel logs go to COM2 only (not framebuffer) - Remove debug markers from assembly hot paths - Removed all serial output from timer_entry.asm - Removed all serial output from syscall/entry.asm - Fixes log spam that was flooding the console - Add xtask interactive command - Uses -display cocoa for PS/2 keyboard input - Serial output goes to log files for debugging - Fix boot stage 54 regression - Added one-time scheduler boot marker 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Key Changes
Framebuffer System:
kernel/src/logger.rs: ShellFrameBuffer struct with font rendering, cursor tracking, scrollingkernel/src/syscall/handlers.rs: write_to_stdio routes to framebuffer in interactive modekernel/src/ipc/stdin.rs: Keyboard echo to framebuffer for user input visibilityBuild Configuration:
kernel/Cargo.toml: Added noto-sans-mono-bitmap with unicode-basic-latin, unicode-specials featuresinteractivegates all framebuffer code pathsBug Fixes:
<EE>!SYS,@@12Q!CR3) from assembly filesis_initialized()in per_cpu.rsTest plan
cargo run -p xtask -- interactive) shows shell output on QEMU display🤖 Generated with Claude Code