Allow ETW to use thread's activity ID #286
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit is contains several related changes that ultimately allow ETW events to set their activity ID to the current thread's value. By setting the thread's activity ID, win32 API calls from that thread will be able to access it and either continue using it or set it as the related activity ID, improving tracing on Windows. Update mkwisyscall to import arbitrary other packages, so generated functions can types defined elsewhere. Switch ETW functions to use `guid.GUID` instead of `windows.GUID`. Add convenience `IsEmpty` function for GUIDs. Add `EventActivityIdControl` function to manipulate the thread's activity ID, and related `*ThreadActivityID` functions to set the thread activity ID to a non-empty value. Update `writeEventRaw` to only pass (related) activity ID to `eventWriteTransfer` if they are non-empty. This is not a breaking change, since related activity ID will remain the same, and the activity ID will only be changed when not specified if `InitializeThreadActivityID` or `SetThreadActivityID` are called before-hand. Signed-off-by: Hamza El-Saawy <hamzaelsaawy@microsoft.com>
|
What is the motivation behind supporting thread activity ID? Our previous thinking has been this doesn't give much value given the nature of goroutines, and it makes more sense (and is more idiomatic Go) to just carry activity ID around in a context. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This commit is contains several related changes that ultimately allow ETW events to set their activity ID to the current thread's value. By setting the thread's activity ID, win32 API calls from that thread will be able to access it and either continue using it or set it as the related activity ID, improving tracing on Windows.
Update mkwisyscall to import arbitrary other packages, so generated functions can types defined elsewhere.
Switch ETW functions to use
guid.GUIDinstead ofwindows.GUID.Add convenience
IsEmptyfunction for GUIDs.Add
EventActivityIdControlfunction to manipulate the thread's activity ID, and related*ThreadActivityIDfunctions to set the thread activity ID to a non-empty value.Update
writeEventRawto only pass (related) activity ID toeventWriteTransferif they are non-empty.This is not a breaking change, since related activity ID will remain the same, and the activity ID will only be changed when not specified if
InitializeThreadActivityIDorSetThreadActivityIDare called before-hand.