Skip to content

Conversation

@tommcdon
Copy link
Member

Port #117843 from main to .NET 10 Preview 7 branch

  • Update DBG_MAX_EXECUTABLE_ALLOC_SIZE and EXPECTED_CHUNKSIZE to account recent SharedPatchBypassBuffer changes

…17843)

* Update DBG_MAX_EXECUTABLE_ALLOC_SIZE and EXPECTED_CHUNKSIZE to account recent SharedPatchBypassBuffer changes

* Comment-only update

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Improve EXPECTED_CHUNKSIZE comment

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@tommcdon tommcdon self-assigned this Jul 22, 2025
Copilot AI review requested due to automatic review settings July 22, 2025 02:41
Copy link
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

This PR updates debugger heap constants to accommodate recent changes to the SharedPatchBypassBuffer. The changes adjust memory allocation limits and chunk sizes to ensure proper functionality of the debugger's executable memory management.

  • Updates maximum executable allocation size from 112 to 120 bytes to match the size of SharedPatchBypassBuffer
  • Doubles the expected chunk size from 128 to 256 bytes to accommodate the larger allocation requirements
  • Adds detailed comments explaining the rationale and requirements for these constants

@tommcdon tommcdon added the Servicing-consider Issue for next servicing release review label Jul 23, 2025
@artl93 artl93 added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Jul 23, 2025
@jeffschwMSFT jeffschwMSFT enabled auto-merge (squash) July 23, 2025 17:43
@jeffschwMSFT jeffschwMSFT disabled auto-merge July 23, 2025 17:48
@jeffschwMSFT jeffschwMSFT merged commit af84ec9 into dotnet:release/10.0-preview7 Jul 23, 2025
15 of 76 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Aug 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants