Skip to content

Support new message CMD_VM_GET_SYSTEM_INFORMATION#127337

Merged
thaystg merged 1 commit intodotnet:mainfrom
thaystg:dev/thays/support_getsysteminformation
Apr 24, 2026
Merged

Support new message CMD_VM_GET_SYSTEM_INFORMATION#127337
thaystg merged 1 commit intodotnet:mainfrom
thaystg:dev/thays/support_getsysteminformation

Conversation

@thaystg
Copy link
Copy Markdown
Member

@thaystg thaystg commented Apr 23, 2026

No description provided.

@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @steveisok, @thaystg, @dotnet/dotnet-diag
See info in area-owners.md if you want to be subscribed.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds support in Mono’s debugger wire protocol for a new VM command (CMD_VM_GET_SYSTEM_INFORMATION) so debugger clients can query basic system characteristics from the runtime.

Changes:

  • Bump debugger protocol minor version to 2.68.
  • Add processor architecture constants (matching winnt.h) and a new VM command ID for GET_SYSTEM_INFORMATION.
  • Implement the new command in the debugger agent to return processor architecture and OS page size.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
src/mono/mono/component/debugger-protocol.h Bumps protocol version; defines architecture constants; adds new VM command ID.
src/mono/mono/component/debugger-engine.h Exposes the new VM command as an engine-level CMD_ macro.
src/mono/mono/component/debugger-agent.c Implements command handling and returns (processor_architecture, page_size).

Comment thread src/mono/mono/component/debugger-agent.c
Comment thread src/mono/mono/component/debugger-protocol.h
@thaystg thaystg merged commit ad1912b into dotnet:main Apr 24, 2026
86 of 88 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants