Skip to content

Fix incorrect memory display in the language servers menu on Linux - #48245

Merged
reflectronic merged 1 commit into
zed-industries:mainfrom
feeiyu:fix-lsp-memory
Feb 3, 2026
Merged

reflectronic merged 1 commit into
zed-industries:mainfrom
feeiyu:fix-lsp-memory

Conversation

@feeiyu

@feeiyu feeiyu commented Feb 3, 2026

Copy link
Copy Markdown
Collaborator

While trying out the new feature introduced in #48226, I noticed an issue on Linux where the reported memory usage is incorrect.

On Linux, even when using ProcessRefreshKind::nothing, thread information is still collected. To fix this, the memory calculation needs to explicitly exclude task/thread data by using without_tasks.

Before

After

Release Notes:

  • N/A

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Feb 3, 2026

@reflectronic reflectronic left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@reflectronic
reflectronic merged commit 6600154 into zed-industries:main Feb 3, 2026
29 checks passed
adb-sh pushed a commit to adb-sh/zed that referenced this pull request Feb 5, 2026
…ed-industries#48245)

While trying out the new feature introduced in zed-industries#48226, I noticed an
issue on Linux where the reported memory usage is incorrect.

On Linux, even when using ProcessRefreshKind::nothing, thread
information is still collected. To fix this, the memory calculation
needs to explicitly exclude task/thread data by using without_tasks.

Before
<img width="544" height="186"
src="https://github.com/user-attachments/assets/438c15b2-b1f1-42df-9ffe-dea2f5b1b6ce"
/>

After
<img width="531" height="139"
src="https://github.com/user-attachments/assets/a0e5aae5-9558-4bfc-b368-8306bbc7c37e"
/>




Release Notes:

- N/A
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
…ed-industries#48245)

While trying out the new feature introduced in zed-industries#48226, I noticed an
issue on Linux where the reported memory usage is incorrect.

On Linux, even when using ProcessRefreshKind::nothing, thread
information is still collected. To fix this, the memory calculation
needs to explicitly exclude task/thread data by using without_tasks.

Before
<img width="544" height="186"
src="https://github.com/user-attachments/assets/438c15b2-b1f1-42df-9ffe-dea2f5b1b6ce"
/>

After
<img width="531" height="139"
src="https://github.com/user-attachments/assets/a0e5aae5-9558-4bfc-b368-8306bbc7c37e"
/>




Release Notes:

- N/A
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
…ed-industries#48245)

While trying out the new feature introduced in zed-industries#48226, I noticed an
issue on Linux where the reported memory usage is incorrect.

On Linux, even when using ProcessRefreshKind::nothing, thread
information is still collected. To fix this, the memory calculation
needs to explicitly exclude task/thread data by using without_tasks.

Before
<img width="544" height="186"
src="https://github.com/user-attachments/assets/438c15b2-b1f1-42df-9ffe-dea2f5b1b6ce"
/>

After
<img width="531" height="139"
src="https://github.com/user-attachments/assets/a0e5aae5-9558-4bfc-b368-8306bbc7c37e"
/>




Release Notes:

- N/A
@feeiyu
feeiyu deleted the fix-lsp-memory branch August 9, 2026 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants