Skip to content

Conversation

@softwarecki
Copy link
Collaborator

@softwarecki softwarecki commented Feb 26, 2024

Memory region must first be mapped as writable to allow data to be placed there. Then the target access flags are set, e.g. read-only and executable.

Updated zephyr to zephyrproject-rtos/zephyr@9cf3e08

@lgirdwood
Copy link
Member

@softwarecki I think this and others are blocked by the Zephyr upgrade. We should be able to fix this soon though.

@lgirdwood
Copy link
Member

SOFCI TEST

@lgirdwood lgirdwood added the blocked progress blocked by something else, applies to either feature or bug label Feb 29, 2024
@lgirdwood
Copy link
Member

@softwarecki will mark this as blocked on Zephyr update (impacts other PRs too). @lyakh now looking at resolving Zephyr update.

@lgirdwood lgirdwood added this to the v2.9 milestone Feb 29, 2024
@kv2019i kv2019i modified the milestones: v2.9, v2.10 Mar 4, 2024
@kv2019i
Copy link
Collaborator

kv2019i commented Mar 8, 2024

@softwarecki Zephyr is now updated, so this is unblocked now (but you do need to rebase).

@lyakh
Copy link
Collaborator

lyakh commented Mar 13, 2024

@softwarecki could you update this one, please?

@kv2019i
Copy link
Collaborator

kv2019i commented Mar 13, 2024

@softwarecki This adds a sparse warnings:
https://github.com/thesofproject/sof/actions/runs/8261860169/job/22600019216?pr=8882
/zep_workspace/sof/src/library_manager/lib_manager.c:142:47: warning: incorrect type in argument 1 (different address spaces)

can you check?

Memory region must first be mapped as writable to allow data to be placed
there. Then the target access flags are set, e.g. read-only and executable.

Signed-off-by: Adrian Warecki <adrian.warecki@intel.com>
@kv2019i kv2019i merged commit dff7332 into thesofproject:main Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked progress blocked by something else, applies to either feature or bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants