dotnet/runtime

[mono] MonoImage->references reads should be atomic

Open

#32.889 geöffnet am 26. Feb. 2020

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)C# (5.445 Forks)batch import
area-VM-meta-monogood first issueruntime-mono

Repository-Metriken

Stars
 (17.886 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 12T 11h) (661 gemergte PRs in 30 T)

Beschreibung

As of https://github.com/dotnet/runtime/pull/32622 we will take the image lock when writing to references, but reads throughout the code are currently done directly and without locking. This should be changed to use accessor functions that read in the pointer atomically.

This isn't time-critical as I don't believe we've seen many (any?) crashes related to this, but is worth noting and changing for completeness' sake.

Contributor Guide

Research-Richtung
Untersuchen Sie alle Stellen, an denen `MonoImage >references` ohne Sperre gelesen wird, und ersetzen Sie sie durch atomare Zugriffsfunktionen. Sehen Sie sich das in PR #32622 für das Schreiben verwendete Muster an.
Tech Stack
csharp
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#concurrency
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60