Skip to content

add: binding for set_autoattach#1106

Merged
danielocfb merged 1 commit intolibbpf:masterfrom
syogaraj:issues/#1085
Mar 3, 2025
Merged

add: binding for set_autoattach#1106
danielocfb merged 1 commit intolibbpf:masterfrom
syogaraj:issues/#1085

Conversation

@syogaraj
Copy link
Contributor

@syogaraj syogaraj commented Mar 1, 2025

Add binding for bpf_program__set_autoattach as discussed in #1085 .
bpf_program__set_autoattach is of type void. So, no return checks are needed.

Thanks to @mmullin-halcyon

Signed-off-by: yogaraj.s <yogarajsivaprakasam@gmail.com>
Copy link
Contributor

@chenhengqi chenhengqi left a comment

Choose a reason for hiding this comment

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

LGTM but the commit message could be improved though.

Copy link
Collaborator

@danielocfb danielocfb left a comment

Choose a reason for hiding this comment

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

I agree with @chenhengqi, but probably not worth doing another round trip. Please stay consistent with existing commit message styles in the future. Ideally we'd also have a test for this functionality, but given that we don't have one for autoload not going to push for that either. Thanks!

@danielocfb danielocfb merged commit 3ead30d into libbpf:master Mar 3, 2025
14 checks passed
danielocfb pushed a commit to d-e-s-o/libbpf-rs that referenced this pull request Mar 3, 2025
Add a CHANGELOG entry for pull request libbpf#1106, which added the
OpenProgramMut::set_autoattach() functionality.

Signed-off-by: Daniel Müller <deso@posteo.net>
d-e-s-o added a commit that referenced this pull request Mar 3, 2025
Add a CHANGELOG entry for pull request #1106, which added the
OpenProgramMut::set_autoattach() functionality.

Signed-off-by: Daniel Müller <deso@posteo.net>
@syogaraj syogaraj deleted the issues/#1085 branch March 4, 2025 04:09
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.

3 participants