Add Linux keyboard support via evdev#9
Open
killerra wants to merge 4 commits into
Open
Conversation
- Implement Process::state() via /proc/<pid>/stat: zombie/dead states map to ProcessState::Dead (with exit_code when readable), a vanished PID maps to Dead, and permission errors stay Unknown. - Use the module base address as the opaque module handle instead of an index into a freshly parsed maps list, eliminating the race between module_address_list_callback and module_by_address. - Resolve primary_module_address() through /proc/<pid>/exe instead of assuming the 0th mapping, falling back to the first mapping. - Report real in-process addresses for environment variables using env_start from /proc/<pid>/stat, and return the env block address from environment_block_address() (plugin API 2).
After a partial process_vm_readv/writev, the retry syscall resumes at iov offset `offset`, but the result-dispatch loop iterated the local/ remote iovecs and temp_meta from index 0. On every pass after the first, already-reported entries were re-reported with the wrong metadata and local slices, the entries the retry actually transferred were never reported, byte accounting used the wrong iov_lens, and out_fail flagged the wrong element. This corrupted result attribution for any batched read/write spanning an unmapped hole. Align dispatch and accounting with the syscall window by skipping the first `win` entries on all three iterators. Add a regression test that batches [valid, unmapped, valid] reads against our own PID; it fails against the previous code (first region duplicated, third dropped).
Follow-ups from the Linux backend soundness audit:
- Decode the waitpid(2)-style status word from /proc/<pid>/stat into a
real exit code in Process::state(): normal exits report the exit(3)
code, signal deaths report the negated signal number. Previously the
raw status word leaked through (exit(3) surfaced as Dead(768)).
- Derive OsInfo.arch and ProcessInfo::{sys_arch,proc_arch} from the
compile target (x86_64/x86/aarch64) instead of hardcoding x86-64, and
make the process module list callback emit the same arch field that
module_by_address keys on.
- Replace OS kernel-module handles (indices into a name-sorted snapshot
that shift on module load/unload) with a stable hash of the module
name (std DefaultHasher, fixed-seeded), resolved against the live
snapshot.
- Report the real process state in process_info_by_pid via the shared
process_state() helper instead of hardcoding Alive.
Implement OsKeyboard for LinuxOs with LinuxKeyboard/LinuxKeyboardState, polling key state through the EVIOCGKEY ioctl (evdev's get_key_state) on /dev/input/event* devices - the direct analog of GetKeyboardState on Windows. Requires root or membership in the 'input' group. is_down() accepts Microsoft virtual-key codes like the Windows backend, translated through a static VK -> evdev table (US layout). Side-agnostic modifiers check both sides, VK_RETURN covers both enter keys, and mouse buttons cover both side-button report styles (BTN_SIDE/BTN_EXTRA and BTN_BACK/BTN_FORWARD). Unmapped VKs report not-pressed, matching Windows. Key state is ORed across all keyboard-capable devices; unplugged devices are dropped on the fly with a one-shot re-enumeration when none remain.
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.
Stacked on #8 — review only the last commit (
Add Linux keyboard support via evdev); the earlier commits belong to #8 and this branch will be rebased once it merges.What
Implements
OsKeyboardforLinuxOs, bringing the Linux backend to feature parity with the Windows backend's keyboard support:LinuxKeyboard/LinuxKeyboardStatepoll the pressed-key bitmap via theEVIOCGKEYioctl (evdev'sget_key_state) on/dev/input/event*— the direct analog ofGetKeyboardState, reading state without consuming input events.is_down()accepts Microsoft virtual-key codes, same as the Windows backend, translated through a static VK → evdev table (US layout), so cross-platform consumers work unchanged. Unmapped VKs cleanly report not-pressed.VK_SHIFT/VK_CONTROL/VK_MENU) check both left and right keys;VK_RETURNcovers main and keypad enter.GetKeyStatedoes, including mouse4/mouse5 in both evdev report styles (BTN_SIDE/BTN_EXTRAandBTN_BACK/BTN_FORWARD).set_downstays a stub, matching the Windows backend.Permissions
Reading
/dev/input/event*requires root or membership in theinputgroup (udev default:root:input0660). With no readable device,keyboard()returns a descriptiveNotFounderror stating exactly that.Testing
cargo test --all).cargo buildwith--all-featuresand--no-default-features,cargo fmt --check,cargo clippy --all-targets --all-featuresall clean./dev/input.