Skip to content

Commit

Permalink
[release/7.0] Managed EventSources do not show up by default in Event…
Browse files Browse the repository at this point in the history
…Pipe sessions (#75248) (#77811)

* Update provider_compute_event_enable_mask so EventSouces with no keywords show up by default (#75248)

Update ep-provider.c

* Update ep-provider.c

* Update src/native/eventpipe/ep-provider.c

Co-authored-by: Juan Hoyos <juan.hoyos@microsoft.com>

Co-authored-by: Juan Hoyos <juan.hoyos@microsoft.com>
  • Loading branch information
davmason and hoyosjs authored Nov 7, 2022
1 parent 4ecef5f commit 2a65e07
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions src/native/eventpipe/ep-provider.c
Original file line number Diff line number Diff line change
Expand Up @@ -245,6 +245,16 @@ ep_provider_add_event (

ep_requires_lock_not_held ();

// Keyword bits 44-47 are reserved for use by EventSources, and every EventSource sets them all.
// We filter out those bits here so later comparisons don't have to take them in to account. Without
// filtering, EventSources wouldn't show up with Keywords=0.
uint64_t session_mask = ~0xF00000000000;
// -1 is special, it means all keywords and gets used on internal error notifications. Don't change it.
uint64_t all_keywords = (uint64_t)(-1);
if (keywords != all_keywords) {
keywords &= session_mask;
}

EventPipeEvent *instance = ep_event_alloc (
provider,
keywords,
Expand Down

0 comments on commit 2a65e07

Please sign in to comment.