Skip to content

Add setitimer/getitimer#858

Merged
CvvT merged 3 commits into
mainfrom
weiteng/sys_itimer
May 29, 2026
Merged

Add setitimer/getitimer#858
CvvT merged 3 commits into
mainfrom
weiteng/sys_itimer

Conversation

@CvvT
Copy link
Copy Markdown
Contributor

@CvvT CvvT commented May 20, 2026

Implement syscalls setitimer and getitime, but setitimer does not support periodic timers yet (i.e., it is equivalent to alarm for now).

Also add a new feature alarm_fallback to enable/disable the fallback alarm check.

@CvvT CvvT marked this pull request as ready for review May 21, 2026 00:05
Comment thread litebox_shim_linux/src/syscalls/process.rs Outdated
Copy link
Copy Markdown
Member

@wdcui wdcui left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@CvvT CvvT force-pushed the weiteng/sys_itimer branch from 1cc07e9 to f9ab441 Compare May 28, 2026 23:33
@github-actions
Copy link
Copy Markdown

🤖 SemverChecks 🤖 ⚠️ Potential breaking API changes detected ⚠️

Click for details
--- failure enum_no_repr_variant_discriminant_changed: enum variant had its discriminant change value ---

Description:
The enum's variant had its discriminant value change. This breaks downstream code that used its value via a numeric cast like `as isize`.
        ref: https://doc.rust-lang.org/reference/items/enumerations.html#assigning-discriminant-values
       impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.47.0/src/lints/enum_no_repr_variant_discriminant_changed.ron

Failed in:
  variant SyscallRequest::Statx 98 -> 99 in /home/runner/work/litebox/litebox/litebox_common_linux/src/lib.rs:2380

@CvvT CvvT added this pull request to the merge queue May 29, 2026
Merged via the queue into main with commit cf8328c May 29, 2026
24 of 26 checks passed
@CvvT CvvT deleted the weiteng/sys_itimer branch May 29, 2026 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants