Skip to content

rocknix-memory-manager integration#7

Open
aenertia wants to merge 1 commit into
ROCKNIX:masterfrom
aenertia:meman
Open

rocknix-memory-manager integration#7
aenertia wants to merge 1 commit into
ROCKNIX:masterfrom
aenertia:meman

Conversation

@aenertia

@aenertia aenertia commented Feb 5, 2026

Copy link
Copy Markdown

No description provided.

@porschemad911

porschemad911 commented Feb 12, 2026

Copy link
Copy Markdown
Contributor

I will make a local build to give this a test on my OGU

if (currSwap.empty()) currSwap = "0";
swap_size->add(_("DISABLE"), "0", currSwap == "0");
swap_size->add(_("AUTO (RAM SIZE)"), "Auto", currSwap != "0" && currSwap != "256" && currSwap != "512" && currSwap != "768" && currSwap != "1024" && currSwap != "2048" && currSwap != "4096" && currSwap != "6144" && currSwap != "8192" && currSwap != "12288" && currSwap != "16384");
swap_size->add(_("RESUME (HIBERNATION)"), "Resume", false);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please remove the RESUME (HIBERNATION) option

vm_oc->add(_("ALWAYS (GAME COMPAT)"), "1", false);
vm_oc->add(_("HEURISTIC (DEFAULT)"), "0", false);
p->addWithLabel(_("VM OVERCOMMIT"), vm_oc);
p->addSaveFunc([vm_oc] { system(("rocknix-memory-manager --vm-overcommit \"" + vm_oc->getSelected() + "\"").c_str()); });

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is not currently a valid rocknix-memory-manager action

@porschemad911 porschemad911 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The main problem in my testing is that changing settings in this menu and selecting Apply & Reload Manager did not actually modify /storage/.config/swap.conf and change any settings (checked rocknix-memory-manager --status over SSH).

@aenertia

aenertia commented Feb 12, 2026

Copy link
Copy Markdown
Author

Thanks for the feedback - I decided to hold off doing the GUI integration until I had played around with ES bindings more and become a bit more familiar with it so that this doesn't create inadvertently annoying rebase issues later on. Will incorporate into testing before next PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants