zellij-org/zellij
1
1
Fork 0
mirror of https://github.com/zellij-org/zellij.git synced 2024-11-29 23:31:03 +03:00
Code Issues Projects Releases Wiki Activity

fix(performance): only refresh relevant prorcess in sysinfo when serializing sessions (#2920)

Browse Source 
* optimize: update refresh prorcess for sysinfo to reduce cpu usage

* fix: fix format

Signed-off-by: xuanyuan300 <xuanyuan300@gmail.com>

---------

Signed-off-by: xuanyuan300 <xuanyuan300@gmail.com>
This commit is contained in:
pierce 2023-11-09 16:41:01 +08:00 committed by GitHub
parent a553188601
commit c325eb3af9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 13 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
zellij-server/src/os_input_output.rs
View File

@ -756,12 +756,14 @@ impl ServerOsApi for ServerOsInputOutput {
fn get_cwds(&self, pids: Vec<Pid>) -> HashMap<Pid, PathBuf> {
let mut system_info = System::new();
let mut cwds = HashMap::new();
for pid in pids {
// Update by minimizing information.
// See https://docs.rs/sysinfo/0.22.5/sysinfo/struct.ProcessRefreshKind.html#
system_info.refresh_processes_specifics(ProcessRefreshKind::default());
let mut cwds = HashMap::new();
for pid in pids {
let is_found =
system_info.refresh_process_specifics(pid.into(), ProcessRefreshKind::default());
if is_found {
if let Some(process) = system_info.process(pid.into()) {
let cwd = process.cwd();
let cwd_is_empty = cwd.iter().next().is_none();
@ -770,6 +772,8 @@ impl ServerOsApi for ServerOsInputOutput {
}
}
}
}
cwds
}
fn get_all_cmds_by_ppid(&self) -> HashMap<String, Vec<String>> {