Skip to content

Release logging thread continuations - #13466

Merged
bneradt merged 1 commit into
apache:masterfrom
bneradt:fix-log-thread-leak
Jul 31, 2026
Merged

Release logging thread continuations#13466
bneradt merged 1 commit into
apache:masterfrom
bneradt:fix-log-thread-leak

Conversation

@bneradt

@bneradt bneradt commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Dedicated logging continuations remain allocated after their worker loops return during shutdown. Once shutdown reliably completes, LeakSanitizer can report the flush continuation as an 80-byte leak.

Release both continuations after their loops finish. This matches the ownership model used by other dedicated event threads.

Dedicated logging continuations remain allocated after their worker
loops return during shutdown. Once shutdown reliably completes,
LeakSanitizer can report the flush continuation as an 80-byte leak.

Release both continuations after their loops finish. This matches the
ownership model used by other dedicated event threads.
@bneradt bneradt added this to the 11.0.0 milestone Jul 31, 2026
Copilot AI review requested due to automatic review settings July 31, 2026 16:20
@bneradt bneradt added Bug ASan Address Sanitizer Logging labels Jul 31, 2026
@bneradt bneradt self-assigned this Jul 31, 2026

Copilot AI left a comment

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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@cmcfarlen cmcfarlen left a comment

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.

Ah, ez fix

@bneradt
bneradt merged commit b080c83 into apache:master Jul 31, 2026
15 checks passed
@bneradt
bneradt deleted the fix-log-thread-leak branch July 31, 2026 22:04
@github-project-automation github-project-automation Bot moved this to For v10.2.0 in ATS v10.2.x Jul 31, 2026
cmcfarlen pushed a commit that referenced this pull request Aug 4, 2026
Dedicated logging continuations remain allocated after their worker
loops return during shutdown. Once shutdown reliably completes,
LeakSanitizer can report the flush continuation as an 80-byte leak.

Release both continuations after their loops finish. This matches the
ownership model used by other dedicated event threads.

(cherry picked from commit b080c83)
@cmcfarlen cmcfarlen moved this from For v10.2.0 to Picked v10.2.0 in ATS v10.2.x Aug 4, 2026
@cmcfarlen cmcfarlen modified the milestones: 11.0.0, 10.2.0 Aug 4, 2026
@cmcfarlen

Copy link
Copy Markdown
Contributor

Cherry-picked to the 10.2.x branch as ca5f3a3 for the 10.2.0 release.

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

Labels

ASan Address Sanitizer Bug Logging

Projects

Status: Picked v10.2.0

Development

Successfully merging this pull request may close these issues.

3 participants