-
Notifications
You must be signed in to change notification settings - Fork 258
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
crashdump: add crash dump for sbl build #2031
crashdump: add crash dump for sbl build #2031
Conversation
bhe4
commented
Sep 5, 2024
- add crashdump stack and heap define
- optimize the sbl_cd heap to 2M and static to 128K
- only add needed drivers in dedicated efiwrapper
CRASHDUMP_BASE_ADDRESS := 0x12800000 | ||
CRASHDUMP_HEAP_SIZE := 2097152 | ||
CRASHDUMP_STACK_SIZE := 131072 | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove 1 blank line
c4ec106
to
138d775
Compare
138d775
to
fbf82a4
Compare
Improper Commit Message |
Expected syntax:
|
1. add crashdump stack and heap define 2. optimize the sbl_cd heap to 2M and static to 128K 3. only add needed drivers in dedicated efiwrapper Test Done: 1. android boot 2. crashdump function works Tracked-On: OAM-126307 Signed-off-by: Bo He <[email protected]> Signed-off-by: Haoyu Tang <[email protected]>
cdcf69f
to
de5d7e7
Compare
Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web for more details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
SUCCESS: Android CI has completed Engineering Build for this issue.Please check the linked Tracked-On issue/Android CI Web for more details. |
Android CI has started MERGE Build for this pr ,Please check the linked Tracked-On issue/Android CI Web for more details. |
35b88d2
into
projectceladon:celadon/u/mr0/master
Android CI has completed MERGE Build for this pr, build is SUCCESS. Please check the linked Tracked-On issue/Android CI Web for more details. For Binaries: /cactus-absp-or-local/celadon_umr0_master-merge/168 |