Skip to content

Add processing of code coverage for optee core #4192

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

Add processing of code coverage for optee core #4192

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
cb8cd11
core: add configuration flags for gcov
lenormandfranck Nov 2, 2020
89e521c
compil: Fix link error TEE_RAM_VA_SIZE is too small
lenormandfranck Nov 2, 2020
ad40ae8
libgcov: Add library adding the gcov support
lenormandfranck Oct 6, 2020
ce426d3
libgcov: Add implementation of gcov for GCC
lenormandfranck Oct 6, 2020
db065b3
core: Add writer of coverage data to REE FS
lenormandfranck Oct 6, 2020
e5d442f
pta: gcov: Add gcov pta
lenormandfranck Oct 6, 2020
92afa01
core: Add support for code coverage compilation
lenormandfranck Nov 2, 2020
08d97f1
coverage.py: Add script to process code coverage
lenormandfranck Oct 6, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions core/arch/arm/plat-imx/conf.mk
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -425,5 +425,17 @@ CFG_IMX_CAAM ?= y
endif
endif

ifeq ($(CFG_CORE_GCOV_SUPPORT), y)
ifneq ($(CFG_WITH_LPAE), y)
# Default:
# TEE_RAM_VA_SIZE = CORE_MMU_PGDIR_SIZE
# = BIT(CORE_MMU_PGDIR_SHIFT)
# with LPAE disabled: #define CORE_MMU_PGDIR_SHIFT 20
# = BIT(20)
# = 0x80000 (512 Ko)
Copy link
Contributor

@jenswi-linaro jenswi-linaro Nov 2, 2020

Choose a reason for hiding this comment

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

That should be 0x100000 (1MB)

Copy link
Contributor Author

@lenormandfranck lenormandfranck Dec 1, 2020

Choose a reason for hiding this comment

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

In effect, my bad

CFG_TEE_RAM_VA_SIZE := 0x200000 # 2MB
endif
endif

# Cryptographic configuration
include core/arch/arm/plat-imx/crypto_conf.mk