Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable clone3 #1971

Merged
merged 3 commits into from
Dec 17, 2024
Merged

Enable clone3 #1971

merged 3 commits into from
Dec 17, 2024

Conversation

erthalion
Copy link
Contributor

@erthalion erthalion commented Nov 26, 2024

Description

Originally we've excluded clone3, which wasn't present on some older
platforms. But now it's getting used by newer glibc, which means that
we're missing information about threads. It plays some visible role only
when threads are exiting, e.g. in a weird situation if a thread will do
exec, all the threads in the thread group are going to be stopped,
leaving only one running exec. This is visible in the resolved file path
for the event.

Also includes a small CI improvement, allowing use of trace log level
in the debug mode.

Checklist

  • Investigated and inspected CI test results
  • Updated documentation accordingly

Automated testing

  • Added unit tests
  • Added integration tests
  • Added regression tests

Testing Performed

Local testing to verify which event we receive with clone and clone3.

@erthalion erthalion requested a review from a team as a code owner November 26, 2024 11:10
@erthalion erthalion marked this pull request as draft November 26, 2024 11:11
@erthalion erthalion changed the title Fix/clone3 Enable clone3 Nov 26, 2024
@erthalion erthalion force-pushed the fix/clone3 branch 2 times, most recently from f9ddeb5 to 6adb950 Compare November 26, 2024 14:06
@erthalion erthalion force-pushed the fix/clone3 branch 2 times, most recently from 53b7967 to 9673e01 Compare December 3, 2024 09:26
@erthalion erthalion force-pushed the fix/clone3 branch 2 times, most recently from 7b8eff9 to fa9fad9 Compare December 10, 2024 09:44
@erthalion erthalion added the run-multiarch-builds Run steps for non-x86 archs. label Dec 10, 2024
Originally we've excluded clone3, which wasn't present on some older
platforms. But now it's getting used by newer glibc, which means that
we're missing information about threads. It plays some visible role only
when threads are exiting, e.g. in a weird situation if a thread will do
exec, all the threads in the thread group are going to be stopped,
leaving only one running exec. This is visible in the resolved file path
for the event.
@erthalion erthalion marked this pull request as ready for review December 12, 2024 09:58
Copy link
Collaborator

@Stringy Stringy left a comment

Choose a reason for hiding this comment

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

LGTM!

@erthalion erthalion merged commit 4a9fccf into master Dec 17, 2024
76 of 78 checks passed
@erthalion erthalion deleted the fix/clone3 branch December 17, 2024 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run-multiarch-builds Run steps for non-x86 archs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants