Brian Robbins

Excellent. Then I'll keep this issue around. Do post if you have any questions or run into issues. Thanks.

@windhandel, collection of kernel events including CPU samples has always required administrative privileges, and last I spoke with the ETW folks, there is no plan to change this. I think...

@MagicAndre1981 thanks for pinging this. From a quick search, it looks like we would need to add a manifest to the PerfView repo, and then we'd want to upload the...

There was an issue around this code a while back which resulted in this change: https://github.com/microsoft/perfview/commit/9d05617bba91a22807777a9a725353781ecb651b#diff-1f06bc7c79946634f1cf3b0eea5ef2e74db9cea1b22a251df4b9173f5c5136abR1643. It looks like the behavior observed here is slightly different. From the stack, it...

@copilot, can you please add a regression test for this to the `PerfView.Tests` project.

@copilot, this code looks correct. I'd now like for you to update `UniversalEventsTraceEventParser` to inherit from `PredefinedDynamicTraceEventParser`. Be sure to update each of the events as well. The event payloads...

@copilot please update the `cpu` and `cswitch` events in `UniversalEventsTraceEventParser` to properly use the registered dynamic events.

Copilot Post-Mortem: - Copilot did a good job getting started initially, but ultimately was not able to produce code that worked. - I believe a big issue here is that...

> Looks good. I have a preference for new C# features that reduce the boilerplate on alot of this, but this seems like it should work. Thanks much! I was...

> Two reasons come to my mind. First is simply the resolution of sampling (I'm assuming you used sampling). You can try cranking up the sampling from default in VS...