Skip to content

[DRAFT] LVBS/OP-TEE: Fallible Box/Vec allocation - #1212

Draft
Sangho Lee (sangho2) wants to merge 2 commits into
mainfrom
sanghle/lvbs/fallible
Draft

[DRAFT] LVBS/OP-TEE: Fallible Box/Vec allocation#1212
Sangho Lee (sangho2) wants to merge 2 commits into
mainfrom
sanghle/lvbs/fallible

Conversation

@sangho2

@sangho2Sangho Lee (sangho2) commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

This PR replaces most of LVBS/OP-TEE's use of infallible Box and Vec with fallible ones, avoiding kernel panics due to out of memory.

@sangho2

Sangho Lee (sangho2) commented Aug 20, 2026

Copy link
Copy Markdown
ContributorAuthor

To resolve the issue #846 . Plan to handle other complex data structures like map and do some recoverable OOM in upcoming PRs.

@sangho2Sangho Lee (sangho2) changed the title [DRAFT] LVBS/OP-TEE: Fallible Box/Vec allocations[DRAFT] LVBS/OP-TEE: Fallible Box/Vec allocation/usageAug 20, 2026
@sangho2Sangho Lee (sangho2) changed the title [DRAFT] LVBS/OP-TEE: Fallible Box/Vec allocation/usage[DRAFT] LVBS/OP-TEE: Fallible Box/Vec allocationAug 20, 2026
@sangho2Sangho Lee (sangho2) added the must-not-merge:blocked-on-other-changes Other changes/PRs to be handled first. Label not needed for non-main changes. label Aug 20, 2026
@sangho2

Copy link
Copy Markdown
ContributorAuthor

#1219

@sangho2
Sangho Lee (sangho2)force-pushed the sanghle/lvbs/fallible branch 3 times, most recently from 5840af0 to 6abf727CompareAugust 21, 2026 23:20
@github-actions

Copy link
Copy Markdown

🤖 SemverChecks 🤖 ⚠️ Potential breaking API changes detected ⚠️

Click for details
--- failure enum_variant_added: enum variant added on exhaustive enum ---
Description:
A publicly-visible enum without #[non_exhaustive] has a new variant.
ref: https://doc.rust-lang.org/cargo/reference/semver.html#enum-variant-new
impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.50.0/src/lints/enum_variant_added.ron
Failed in:
variant ElfParseError:AllocationFailed in /home/runner/work/litebox/litebox/litebox_common_linux/src/loader.rs:138
--- failure derive_trait_impl_removed: built-in derived trait no longer implemented ---
Description:
A public type has stopped deriving one or more traits. This can break downstream code that depends on those types implementing those traits.
ref: https://doc.rust-lang.org/reference/attributes/derive.html#derive
impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.50.0/src/lints/derive_trait_impl_removed.ron
Failed in:
type ShmInfo no longer derives Clone, in /home/runner/work/litebox/litebox/litebox_shim_optee/src/msg_handler.rs:696

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

must-not-merge:blocked-on-other-changesOther changes/PRs to be handled first. Label not needed for non-main changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@sangho2