Report swapinfo by ben-grande · Pull Request #140 · QubesOS/qubes-linux-utils

ben-grande · 2026-06-25T15:14:51Z

Allows distinguishing how much memory the qube is using from what is assigned by using "meminfo - swapinfo".

Tested briefly.

codecov-commenter · 2026-06-25T15:34:00Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.24%. Comparing base (0dc52ab) to head (d12a416).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #140   +/-   ##
=======================================
  Coverage   71.24%   71.24%           
=======================================
  Files           6        6           
  Lines         991      991           
=======================================
  Hits          706      706           
  Misses        285      285

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

For: QubesOS/qubes-linux-utils#140

marmarek · 2026-07-10T10:18:49Z

+		sprintf(used_mem_buf, "%lld", used_mem);
+		sprintf(used_swap_buf, "%lld", used_swap);


Redundant calls.

marmarek · 2026-07-10T10:22:52Z

+		syslog(LOG_DAEMON | LOG_ERR, "error writing meminfo to xenstore ?");
+		exit(1);
+	}
+	if (!xs_write(xs, XBT_NULL, "memory/swapinfo", swap, strlen(swap))) {


Note to self: this can be merged only after core-admin part, which sets necessary xenstore access.

marmarek · 2026-07-10T10:37:23Z

+    char *swap;
+} UsedMem;
+
+UsedMem parse(const char *meminfo_buf, const char* dom_current_buf)


FWIW the usual method in C for returning more results is to either return a pointer to a structure (usually dynamically allocated via malloc), or add output parameters (pointers to where function should write result). In case of a simple structure like this, compiler will do the latter for you here, so it can stay this way, but keep in mind it wont be efficient code in more complex cases.

ben-grande · 2026-07-13T09:40:55Z

+		return (UsedMem){ .mem = used_mem_buf, .swap = used_swap_buf };
 	}
-	return NULL;
+	return (UsedMem){ .mem = NULL, .swap = NULL };


Report swap change if above threshold despite memory being below its threshold.

ben-grande · 2026-07-13T09:41:06Z

-	if (meminfo_data)
-		send_to_qmemman(xs, meminfo_data);
+	UsedMem meminfo_data = parse(meminfo_buf, dom_current_buf);
+	if (meminfo_data.mem && meminfo_data.mem[0])


https://github.com/QubesOS/qubes-linux-utils/pull/140/changes#r3569568589

Allows distinguishing how much memory the qube is using from what is assigned by using "meminfo - swapinfo". For: QubesOS/qubes-core-admin#827

Although both values can be lied by the qube, we can distinguish the "memory types" being used in client tools. For: QubesOS/qubes-linux-utils#140

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jun 25, 2026

EXPERIMENTAL Report used memory without swap

f670ea7

For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jun 25, 2026

EXPERIMENTAL Report used memory without swap

f82da90

For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 3, 2026

EXPERIMENTAL Report used memory without swap

90c0c5b

For: QubesOS/qubes-linux-utils#140

ben-grande mentioned this pull request Jul 3, 2026

Report more statistics QubesOS/qubes-core-admin#827

Open

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 6, 2026

EXPERIMENTAL Report used memory without swap

86e699c

For: QubesOS/qubes-linux-utils#140

marmarek requested changes Jul 10, 2026

View reviewed changes

ben-grande commented Jul 13, 2026

View reviewed changes

Report swapinfo

d12a416

Allows distinguishing how much memory the qube is using from what is assigned by using "meminfo - swapinfo". For: QubesOS/qubes-core-admin#827

ben-grande force-pushed the share-swap branch from 49418ad to d12a416 Compare July 13, 2026 13:00

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 14, 2026

Report used memory without swap

34387aa

Although both values can be lied by the qube, we can distinguish the "memory types" being used in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 14, 2026

Report used memory without swap

e498748

Although both values can be lied by the qube, we can distinguish the "memory types" being used in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 16, 2026

Report used memory without swap

af7a111

Although both values can be lied by the qube, we can distinguish the "memory types" being used in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 17, 2026

Report amount of swap used

5ca395b

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 17, 2026

Report amount of swap used

e3d9365

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 21, 2026

Report amount of swap used

e710eec

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 22, 2026

Report amount of swap used

70e4e99

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 22, 2026

Report amount of swap used

4bdedb2

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 22, 2026

Report amount of swap used

d76da46

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 22, 2026

Report amount of swap used

c04a242

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 22, 2026

Report amount of swap used

7474282

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 23, 2026

Report amount of swap used

5d7a23e

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 23, 2026

Report amount of swap used

becbaaa

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 24, 2026

Report amount of swap used

d11fb09

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

ben-grande added a commit to ben-grande/qubes-core-admin that referenced this pull request Jul 24, 2026

Report amount of swap used

40b8979

Although both values can be lied by the qube, we can distinguish the "memory types" in client tools. For: QubesOS/qubes-linux-utils#140

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Report swapinfo#140

Report swapinfo#140
ben-grande wants to merge 1 commit into
QubesOS:mainfrom
ben-grande:share-swap

ben-grande commented Jun 25, 2026

Uh oh!

codecov-commenter commented Jun 25, 2026 •

edited

Loading

Uh oh!

marmarek Jul 10, 2026

Uh oh!

marmarek Jul 10, 2026

Uh oh!

marmarek Jul 10, 2026

Uh oh!

ben-grande Jul 13, 2026

Uh oh!

ben-grande Jul 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		sprintf(used_mem_buf, "%lld", used_mem);
		sprintf(used_swap_buf, "%lld", used_swap);

Uh oh!

Uh oh!

Conversation

ben-grande commented Jun 25, 2026

Uh oh!

codecov-commenter commented Jun 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

marmarek Jul 10, 2026

Choose a reason for hiding this comment

Uh oh!

marmarek Jul 10, 2026

Choose a reason for hiding this comment

Uh oh!

marmarek Jul 10, 2026

Choose a reason for hiding this comment

Uh oh!

ben-grande Jul 13, 2026

Choose a reason for hiding this comment

Uh oh!

ben-grande Jul 13, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

codecov-commenter commented Jun 25, 2026 •

edited

Loading