Skip to content
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.

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

symbolize.py: symbolize TA MMU regions #1778

Merged
merged 2 commits into from
Sep 18, 2017

Conversation

jforissier
Copy link
Contributor

No description provided.

@jenswi-linaro
Copy link
Contributor

For "core: print rwx flags for each MMU region when a user TA aborts"
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>

@jforissier jforissier changed the title Symbolize regions symbolize.py: symbolize TA MMU regions Sep 4, 2017
@jforissier
Copy link
Contributor Author

Can anyone please check the Python script patch(es)?

@jforissier
Copy link
Contributor Author

R-b added to first patch and all rebased on top of master. The Shippable CI should run and (hopefully) pass.

@jenswi-linaro
Copy link
Contributor

For "scripts/symbolize.py: print ELF sections after MMU region information"
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
When processing a user TA abort dump, list the ELF sections that are
mapped in each MMU region. For example (the lines modified by this
patch are prefixed with >>):

   User TA undef-abort at address 0x10574e
    fsr 0x00000000  ttbr0 0x0e07a06a  ttbr1 0x0e07406a  cidr 0x1
    cpu #0          cpsr 0x60000030
    r0 0x20000013      r4 0x0013a6bc    r8 0x00000000   r12 0x0e07dd88
    r1 0x00000033      r5 0x00121fd3    r9 0x00000000    sp 0x001026cc
    r2 0x0010581f      r6 0x00102590   r10 0x00000000    lr 0x00105823
    r3 0x00000043      r7 0x001026cc   r11 0x00000000    pc 0x0010574e
   Status of TA 5b9e0e40-2636-11e1-ad9e-0002a5d5c51b (0xe073b70) (active)
    arch: arm  load address: 0x103000  ctx-idr: 1
    stack: 0x100000 10240
    region 0: va 0x100000 pa 0xe21e000 size 0x3000 flags rw-
>>  region 1: va 0x103000 pa 0xe100000 size 0x2e000 flags r-x .ta_head .text .rodata
>>  region 2: va 0x131000 pa 0xe12e000 size 0xa000 flags r-- .rodata .ARM.extab .ARM.exidx .got .dynsym .rel.got .dynamic .dynstr .hash .rel.dyn
>>  region 3: va 0x13b000 pa 0xe138000 size 0xe6000 flags rw- .data .bss
    region 4: va 0 pa 0 size 0 flags ---
    region 5: va 0 pa 0 size 0 flags ---
    region 6: va 0 pa 0 size 0 flags ---
    region 7: va 0 pa 0 size 0 flags ---
   User TA undef-abort at address 0x10574e undef_instr+6 .text+10030
   Call stack:
    0x0010574e undef_instr at optee_test/ta/os_test/os_test.c:880
    0x00105823 ta_entry_bad_mem_access at optee_test/ta/os_test/os_test.c:917
    0x00105e75 TA_InvokeCommandEntryPoint at optee_test/ta/os_test/ta_entry.c:101
    0x00121fb7 entry_invoke_command at optee_os/lib/libutee/arch/arm/user_ta_entry.c:207
    0x00122013 __utee_entry at optee_os/lib/libutee/arch/arm/user_ta_entry.c:235

Suggested-by: Zeng Tao <prime.zeng@hisilicon.com>
Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
@jforissier
Copy link
Contributor Author

Shippable fails due to a checkpatch warning (line too long in commit message). Let's ignore that since it is a copy/paste of the tool output.

@jforissier jforissier merged commit 3099912 into OP-TEE:master Sep 18, 2017
@jforissier jforissier deleted the symbolize-regions branch November 13, 2018 13:14
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.

2 participants