Skip to content

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

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

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

Conversation

@sangho2

@sangho2 Sangho 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
Contributor Author

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

@sangho2 Sangho Lee (sangho2) changed the title [DRAFT] LVBS/OP-TEE: Fallible Box/Vec allocations [DRAFT] LVBS/OP-TEE: Fallible Box/Vec allocation/usage Aug 20, 2026
@sangho2 Sangho Lee (sangho2) changed the title [DRAFT] LVBS/OP-TEE: Fallible Box/Vec allocation/usage [DRAFT] LVBS/OP-TEE: Fallible Box/Vec allocation Aug 20, 2026
@sangho2 Sangho 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
Contributor Author

#1219

@sangho2
Sangho Lee (sangho2) force-pushed the sanghle/lvbs/fallible branch 5 times, most recently from 6abf727 to cc27a13 Compare August 21, 2026 23:37
@sangho2 Sangho Lee (sangho2) removed the must-not-merge:blocked-on-other-changes Other changes/PRs to be handled first. Label not needed for non-main changes. label Aug 31, 2026
Comment thread litebox_service_heki/src/handlers.rs Dismissed
@sangho2 Sangho 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 31, 2026
@github-actions

github-actions Bot commented Sep 1, 2026

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:692

@sangho2 Sangho Lee (sangho2) removed the must-not-merge:blocked-on-other-changes Other changes/PRs to be handled first. Label not needed for non-main changes. label Sep 1, 2026
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