Skip to content

refactor(ebpf): use AsyncFd in PerfEventReader#601

Open
mccormickt wants to merge 1 commit intoaurae-runtime:mainfrom
mccormickt:push-rpslkxrysktu
Open

refactor(ebpf): use AsyncFd in PerfEventReader#601
mccormickt wants to merge 1 commit intoaurae-runtime:mainfrom
mccormickt:push-rpslkxrysktu

Conversation

@mccormickt
Copy link
Copy Markdown
Contributor

@mccormickt mccormickt commented May 6, 2026

AsyncPerfEventArray and the accompanying async_tokio is removed in future versions of aya. This change refactors our PerfBufferReader to use an AsyncFd for async support to reduce friction in migrating to a future release.

aya-rs/aya#1292

@@ -12,26 +12,6 @@
* Copyright 2022 - 2024, the aurae contributors *
* SPDX-License-Identifier: Apache-2.0 *
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we need to adapt this header for the code since it must be GPL-compatible? Cillium (Apache 2.0) has their ebpf code licensed Dual BSD/GPL, for example.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it looks like your change here is removing the GPL-2.0 and MIT license identifiers that were here for exactly that reason. (it was dual license MIT/GPL2.0 prior to this PR)

@mccormickt mccormickt force-pushed the push-rpslkxrysktu branch from e7527e4 to 001f6bc Compare May 8, 2026 01:35
* Copyright 2022 - 2024, the aurae contributors *
* SPDX-License-Identifier: Apache-2.0 *
\* -------------------------------------------------------------------------- */
/* -------------------------------------------------------------------------- *\
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here.. don't remove this :)

* Copyright 2022 - 2024, the aurae contributors *
* SPDX-License-Identifier: Apache-2.0 *
\* -------------------------------------------------------------------------- */
/* -------------------------------------------------------------------------- *\
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants