Skip to content

Reduction of memory footprint - #1203

Open
Slamy wants to merge 1 commit into
MiSTer-devel:masterfrom
Slamy:memory_footprint
Open

Reduction of memory footprint#1203
Slamy wants to merge 1 commit into
MiSTer-devel:masterfrom
Slamy:memory_footprint

Conversation

@Slamy

@Slamy Slamy commented May 25, 2026

Copy link
Copy Markdown
Contributor
  • Removes ~100kB of static memory per CD supporting core for CUE file loading

This change might be controversial. It already got negative feedback on the MiSTer Discord from other devs since it uses C++14 methods of memory allocation.

I could think of an alternative, like a global buffer for this task. Many of the CD cores don't really share code, but I believe this is a small change to at least reduce memory usage in case it might be a problem in the future.

@Kitrinx

Kitrinx commented May 25, 2026

Copy link
Copy Markdown
Member

The question I would ask here is what actual problem is this solving? Are we getting close to running out of RAM? Changes for the sake of changes without solving a real issue are usually undesirable and can potentially introduce unforseen bugs.

@Slamy

Slamy commented May 25, 2026

Copy link
Copy Markdown
Contributor Author

I do have to admit that this change actually fixes no real life issues right now since we probably have enough RAM for now. It was mostly a suggestion based on a sudden notice when I've discovered that this kind of buffer existed multiple times in the MiSTer application. I don't even know how many of these buffers even exist that have a single use purpose and then never again.

Edit: It was meant to show that there are nicer solutions.

- Removes 100kB of memory per CD supporting core for CUE file loading
@Slamy
Slamy force-pushed the memory_footprint branch from 50a9b29 to e9af983 Compare May 31, 2026 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants