Skip to content

userspace: llext: enable cache for loadable modules#10625

Open
lyakh wants to merge 1 commit intothesofproject:mainfrom
lyakh:user
Open

userspace: llext: enable cache for loadable modules#10625
lyakh wants to merge 1 commit intothesofproject:mainfrom
lyakh:user

Conversation

@lyakh
Copy link
Collaborator

@lyakh lyakh commented Mar 16, 2026

Modules, loaded via LLEXT and running in userspace should have all their sections mapped with enabled cache. This fixes performance degradation in userspace with DP.

Modules, loaded via LLEXT and running in userspace should have all
their sections mapped with enabled cache. This fixes performance
degradation in userspace with DP.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
Copilot AI review requested due to automatic review settings March 16, 2026 09:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Enables cached mappings for userspace LLEXT-loaded module sections to address performance degradation (DP) by ensuring .text, .rodata, .data, and cold sections are mapped with write-back caching.

Changes:

  • ORs XTENSA_MMU_CACHED_WB into the memory partition attributes when adding module sections to a memory domain.
  • Mirrors the same cache attribute when removing partitions (including error cleanup paths).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@jsarha jsarha left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@softwarecki softwarecki left a comment

Choose a reason for hiding this comment

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

Thats exactly what I thought was causing the performance issues

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.

6 participants