MonoGame/MonoGame

Async dispose causes concurrent mod exception

Open

#3.760 geöffnet am 23. Apr. 2015

Auf GitHub ansehen
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (3.071 Forks)batch import
Help WantedOpenGLbughelp wantediOSplatform: iosscope: graphicsscope: opengl

Repository-Metriken

Stars
 (13.886 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 25T 3h) (22 gemergte PRs in 30 T)

Beschreibung

During the async dispose of resources in iOS (OpenGL) you will get a concurrent modification exception during the PlatformDeleteRenderTarget. I have fixed it in our fork of MG. You can see my bug report here:

https://github.com/Cocos2DXNA/cocos2d-xna/issues/471

and you can see my changes to the OpenGL GraphicsDevice here:

https://github.com/Cocos2DXNA/MonoGame/commit/47bec75605efabd17656356e955affec0248f57a

The Activate/Dispose/Resolve methods all access or make changes to the glFrameBuffers dictionary, and they do so without any regard to protection during an async dispose. I found that this exception occurs about 1 in 200 activate/dispose operations.

Contributor Guide

Research-Richtung
Identifizieren Sie die spezifischen Stellen in den Methoden Activate, Dispose und Resolve, die auf glFrameBuffers zugreifen oder diese ändern, und implementieren Sie eine geeignete Synchronisierung (z. B. Lock), um gleichzeitige Änderungen zu verhindern.
Tech Stack
csharp
Domain
mobilecomputer graphics
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
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#OpenGLthreading
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40