Compare commits
1 commit
master
...
perf/throt
| Author | SHA1 | Date | |
|---|---|---|---|
| 77262dd0af |
1 changed files with 31 additions and 3 deletions
|
|
@ -1,21 +1,49 @@
|
||||||
// Copyright 2025 System76 <info@system76.com>
|
// Copyright 2025 System76 <info@system76.com>
|
||||||
// SPDX-License-Identifier: MPL-2.0
|
// SPDX-License-Identifier: MPL-2.0
|
||||||
|
|
||||||
|
use std::cell::Cell;
|
||||||
use std::os::raw::c_int;
|
use std::os::raw::c_int;
|
||||||
|
use std::time::{Duration, Instant};
|
||||||
|
|
||||||
const M_MMAP_THRESHOLD: c_int = -3;
|
const M_MMAP_THRESHOLD: c_int = -3;
|
||||||
|
|
||||||
|
/// Minimum interval between two actual `malloc_trim` calls.
|
||||||
|
///
|
||||||
|
/// `trim` is called at the end of every `update()` and `view()`, which can
|
||||||
|
/// reach 60-200 Hz during typical scrolling, resize, or animation. Each
|
||||||
|
/// `malloc_trim` walks the glibc heap (10 µs to several ms depending on
|
||||||
|
/// fragmentation), so calling it at render frequency can consume a
|
||||||
|
/// substantial fraction of the frame budget. Throttling to 1 Hz keeps RSS
|
||||||
|
/// bounded while removing the per-frame syscall from the hot path.
|
||||||
|
const TRIM_MIN_INTERVAL: Duration = Duration::from_millis(1000);
|
||||||
|
|
||||||
|
thread_local! {
|
||||||
|
static LAST_TRIM: Cell<Option<Instant>> = const { Cell::new(None) };
|
||||||
|
}
|
||||||
|
|
||||||
unsafe extern "C" {
|
unsafe extern "C" {
|
||||||
fn malloc_trim(pad: usize);
|
fn malloc_trim(pad: usize);
|
||||||
|
|
||||||
fn mallopt(param: c_int, value: c_int) -> c_int;
|
fn mallopt(param: c_int, value: c_int) -> c_int;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// Throttled wrapper over `malloc_trim`. Safe to call at render frequency:
|
||||||
|
/// consecutive calls within `TRIM_MIN_INTERVAL` (per-thread) skip the syscall.
|
||||||
#[inline]
|
#[inline]
|
||||||
pub fn trim(pad: usize) {
|
pub fn trim(pad: usize) {
|
||||||
unsafe {
|
LAST_TRIM.with(|last| {
|
||||||
malloc_trim(pad);
|
let now = Instant::now();
|
||||||
}
|
let should_trim = match last.get() {
|
||||||
|
None => true,
|
||||||
|
Some(prev) => now.duration_since(prev) >= TRIM_MIN_INTERVAL,
|
||||||
|
};
|
||||||
|
if should_trim {
|
||||||
|
last.set(Some(now));
|
||||||
|
unsafe {
|
||||||
|
malloc_trim(pad);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Prevents glibc from hoarding memory via memory fragmentation.
|
/// Prevents glibc from hoarding memory via memory fragmentation.
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue