EnC: Add support for updating ManifestResource table

Open
#115,910 6 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
28/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Stale
Tech stack
csharp

Research direction

Begin with the EnC/Hot Reload implementation that handles metadata deltas and the ManifestResource table; the issue does not name files or tests. Reproduce by updating a .resx during an EnC session, then verify the three Assembly resource APIs return updated data without a restart and account for cache invalidation described in issue #115911.

Written by the indexing model from the issue text.

Description

area-Diagnostics-coreclr enhancement
Description

Adding, updating or deleting a managed resource during Hot Reload/EnC currently requires restarting the application. To avoid that we need the runtime to allow changes to ManifestResource table.

Embedded resources listed in the ManifestResource table are stored in the resource section of the PE file, at an offset stored in the ManifestResource table. Similarly to FieldRVA implementation, we can store the blob in the IL delta stream in EnC delta and interpret the offset as a position in the IL stream.

Once implemented a new EnC capability ChangeManifestResources should be added.

The resource string cache also needs to be invalidated when Hot Reload change is applied: https://github.com/dotnet/runtime/issues/115911

The following APIs should return the updated resources after change is applied:
https://learn.microsoft.com/en-us/dotnet/api/system.reflection.assembly.getmanifestresourceinfo?view=net-9.0
https://learn.microsoft.com/en-us/dotnet/api/system.reflection.assembly.getmanifestresourcenames?view=net-9.0
https://learn.microsoft.com/en-us/dotnet/api/system.reflection.assembly.getmanifestresourcestream?view=net-9.0

Any Reflection caches that store the data need to be invalidated on Hot Reload update.

Reproduction Steps

Update .resx file during Hot Reload/EnC session.

Expected behavior

The change is applied without restarting application.

Actual behavior

The change is ignored.

Regression?

No

Known Workarounds

No response

Configuration

No response

Other information

No response

Dominant language
C#
Stars
18.3k
Forks
5.6k
PR merge metrics
PR metrics pending

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from dotnet/runtime

All issues in dotnet/runtime

Similar issues

More C# issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.