Skip to content

[Userspace LL] llext: adjust for MMU flag application - #11064

Open
lyakh wants to merge 1 commit into
thesofproject:mainfrom
lyakh:coldalign
Open

[Userspace LL] llext: adjust for MMU flag application#11064
lyakh wants to merge 1 commit into
thesofproject:mainfrom
lyakh:coldalign

Conversation

@lyakh

@lyakh lyakh commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

The .cold section contains executable code, while .coldrodata contains data. Therefore their MMU flags have to be different, particularly when mapping for userspace. Add a page-size alignment to account for such cases.

The .cold section contains executable code, while .coldrodata
contains data. Therefore their MMU flags have to be different,
particularly when mapping for userspace. Add a page-size alignment to
account for such cases.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
Copilot AI lite review requested due to automatic review settings August 5, 2026 13:00

Copilot AI 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.

Pull request overview

This PR updates the LLEXT link helper to ensure executable DRAM sections and DRAM read-only data sections (.cold vs .coldrodata) do not end up sharing the same MMU page, enabling correct per-page permission flags when mapping for userspace.

Changes:

  • Removes an outdated comment about .cold being marked executable.
  • Inserts a 4KiB page alignment boundary between linked executable DRAM sections and subsequent .coldrodata DRAM sections.

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.

3 participants