Skip to content

Conversation

@mikem8361
Copy link
Contributor

Customer Impact

The Native AOT Watson and SOS support needs to the module's base address containing the runtime for adequate performance. Add the module base address in the crash info as "runtime_base". Changed the "runtime" to the clearer "runtime_version" property.

Testing

Verified the address in local testing. Modified the !crashinfo command in SOS to display this new base address.

Risk

Very low. Native AOT unhandled exception crash info.

The Native AOT Watson and SOS support needs to the module's base address containing
the runtime for adequate performance. Add the module base address in the crash info
as "runtime_base". Changed the "runtime" to the clearer "runtime_version" property.

Verified the address in local testing. Modified the !crashinfo command in SOS to display this new base address.

Very low. Native AOT unhandled exception crash info.
@mikem8361 mikem8361 requested a review from jkotas October 20, 2023 20:23
@mikem8361 mikem8361 self-assigned this Oct 20, 2023
@ghost
Copy link

ghost commented Oct 20, 2023

Tagging subscribers to this area: @agocke, @MichalStrehovsky, @jkotas
See info in area-owners.md if you want to be subscribed.

Issue Details

Customer Impact

The Native AOT Watson and SOS support needs to the module's base address containing the runtime for adequate performance. Add the module base address in the crash info as "runtime_base". Changed the "runtime" to the clearer "runtime_version" property.

Testing

Verified the address in local testing. Modified the !crashinfo command in SOS to display this new base address.

Risk

Very low. Native AOT unhandled exception crash info.

Author: mikem8361
Assignees: mikem8361
Labels:

area-NativeAOT-coreclr

Milestone: -

@mikem8361
Copy link
Contributor Author

/backport to release/8.0

@github-actions
Copy link
Contributor

Started backporting to release/8.0: https://github.com/dotnet/runtime/actions/runs/6592429962

Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@mikem8361 mikem8361 merged commit 34d6ecf into dotnet:main Oct 20, 2023
@mikem8361 mikem8361 deleted the aotcrashinfo branch October 20, 2023 23:07
@ghost ghost locked as resolved and limited conversation to collaborators Nov 20, 2023
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.

2 participants