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

Add native EventPipe event source generation into Mono build. #52844

Conversation

lateralusX
Copy link
Member

Adjust existing genEventing.py/genEventPipe.py to work together with Mono C build. Gives Mono ability to use same set of generated event serializers already used in CoreCLR.

PR adds support for a optional inclusion file when calling eventing scripts, only adding specific events into generated source files. Since Mono won't handle all native events supported by CoreCLR (just a small amount initially) using the inclusion file will dramatically reduce the size of eventpipe static library, if inclusion file is not used, all events will be included (current behaviour).

PR also switch existing hand written native events currently used in Mono over to the native events generated by the eventing scripts.

src/mono/mono/eventpipe/gen-eventing.txt Outdated Show resolved Hide resolved
src/mono/mono/eventpipe/gen-eventing.txt Outdated Show resolved Hide resolved
src/coreclr/scripts/genEventing.py Show resolved Hide resolved
@lateralusX
Copy link
Member Author

lateralusX commented May 19, 2021

@josalem @sywhang are you also OK with this (mainly Mono related changes and changes in scripts shouldn't affect functionality of CoreClr generated sources)? Would like to get this PR merged since we have a couple of Mono specific follow up PR's building on top of this work.

@lateralusX
Copy link
Member Author

Merging this to unblock other workstreams.

@lateralusX lateralusX merged commit 542ef8b into dotnet:main May 20, 2021
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jun 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants