Skip to content

Comments

btf: fix Spec.Copy() for split BTF#1775

Merged
lmb merged 1 commit intocilium:mainfrom
lmb:btf-decoder-copy
May 7, 2025
Merged

btf: fix Spec.Copy() for split BTF#1775
lmb merged 1 commit intocilium:mainfrom
lmb:btf-decoder-copy

Conversation

@lmb
Copy link
Contributor

@lmb lmb commented May 6, 2025

Make decoder.Copy do a deep copy of the base, so that types which are inflated in the future also point at the correct copy.

@lmb lmb force-pushed the btf-decoder-copy branch 2 times, most recently from bf70f6e to cd2768e Compare May 6, 2025 16:20
Make decoder.Copy do a deep copy of the base, so that types which are
inflated from the base in the future also point at the correct copy.

Do not copy base when constructing a new decoder, so that multiple
decoders can share a single base.

Signed-off-by: Lorenz Bauer <[email protected]>
@lmb lmb force-pushed the btf-decoder-copy branch from cd2768e to be20b2f Compare May 6, 2025 16:33
@lmb lmb marked this pull request as ready for review May 6, 2025 16:52
@lmb lmb requested a review from dylandreimerink as a code owner May 6, 2025 16:52
@lmb lmb merged commit ee99c66 into cilium:main May 7, 2025
17 checks passed
@lmb lmb deleted the btf-decoder-copy branch May 7, 2025 10:20
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.

1 participant