Skip to content

Conversation

@hakehuang
Copy link
Contributor

disable MPU for cases exhaust MPU region on twr_ke18f
update ram size for benchmarks

Signed-off-by: Hake Huang [email protected]

disable MPU for cases exhaust MPU region on twr_ke18f
update ram size for benchmarks

Signed-off-by: Hake Huang <[email protected]>
@github-actions github-actions bot added the area: Tests Issues related to a particular existing or missing test label Jan 12, 2021
@henrikbrixandersen
Copy link
Member

henrikbrixandersen commented Jan 12, 2021

The KE1xF MPU region exhaustion is not limited to test cases. I submitted a a PR a few days back to address this issue in a more generic way: #31148

benchmark.kernel.core:
arch_exclude: nios2 riscv32 xtensa
min_ram: 32
min_ram: 36
Copy link
Member

Choose a reason for hiding this comment

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

I think this belongs in a separate commit in a separate PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@henrikbrixandersen this case is not buildable in twr_ke18f, add this will skip this case in board testing.

Copy link
Member

Choose a reason for hiding this comment

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

@henrikbrixandersen this case is not buildable in twr_ke18f, add this will skip this case in board testing.

I understand that. I still think it belongs in a separate commit with a dedicated commit log.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I see, let me drop the PR, and create another one

@hakehuang hakehuang closed this Jan 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Tests Issues related to a particular existing or missing test platform: NXP NXP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants