Skip to content

Remove our macOS-only task-info crate#41132

Merged
mrobinson merged 1 commit intoservo:mainfrom
mrobinson:remove-rust-task-info
Dec 8, 2025
Merged

Remove our macOS-only task-info crate#41132
mrobinson merged 1 commit intoservo:mainfrom
mrobinson:remove-rust-task-info

Conversation

@mrobinson
Copy link
Copy Markdown
Member

This crate is just using system APIs to get the resident and virtual
memory size of the current process. We can do this directly with
mach2, which also allows more flexibility if we want to fetch other
values in the future.

This does require duplicating mach2 as the version used by gilrs is
older. Presumably, some future release of gilrs will upgrade soon.

Testing: There aren't really tests for this, but I tested it manually by
running the memory reporter and ensuring that both the old and new values were
the same.

@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Dec 8, 2025
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Dec 8, 2025
This crate is just using system APIs to get the resident and virtual
memory size of the current process. We can do this directly with
`mach2`, which also allows more flexibility if we want to fetch other
values in the future.

This does require duplicating `mach2` as the version used by `gilrs` is
older. Presumably, some future release of `gilrs` will upgrade soon.

Signed-off-by: Martin Robinson <[email protected]>
@mrobinson mrobinson force-pushed the remove-rust-task-info branch from 5ac3d2d to b735364 Compare December 8, 2025 11:57
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Dec 8, 2025
@mrobinson mrobinson enabled auto-merge December 8, 2025 11:58
@mrobinson mrobinson added this pull request to the merge queue Dec 8, 2025
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Dec 8, 2025
Merged via the queue into servo:main with commit 1e8d1cf Dec 8, 2025
32 checks passed
@mrobinson mrobinson deleted the remove-rust-task-info branch December 8, 2025 12:51
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Dec 8, 2025
mrobinson added a commit to servo/book that referenced this pull request Dec 9, 2025
This crate was removed in servo/servo#41132.

Signed-off-by: Martin Robinson <[email protected]>
github-merge-queue bot pushed a commit to servo/book that referenced this pull request Dec 9, 2025
This crate was removed in servo/servo#41132.

Signed-off-by: Martin Robinson <[email protected]>

Signed-off-by: Martin Robinson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-awaiting-review There is new code that needs to be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants