swung0x48/MobileGL
1
0
Fork 0
mirror of https://github.com/MobileGL-Dev/MobileGL synced 2026-11-07 01:53:24 +09:00
Code Issues Packages Projects Releases Wiki Activity

[Fix] (Remote): the create window starts empty with each session

Browse Source 
The window of creates whose answers are read later is process-global, and every entry names a
sequence number of the session that wrote it.  A process can run several sessions one after
another - eglinfo terminates one platform display and initializes the next - and an entry left by
the previous session made the new session's first create wait, in DrainCreateWindow, for a
sequence number the new server would not reach: the second session hung on its first texture
create.

InstallClientWireTables, which runs at the start of every session, now empties the window and the
create suspects; their answers and their objects died with the session that wrote them.
This commit is contained in:
BZLZHH
2026-10-01 10:31:20 -04:00
parent c6f828532a
commit 69728a4f9b
1 changed files with 13 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
MobileGL/MG_Remote/Client/WireTables.cpp
+13
View File
@@ -513,6 +513,18 @@ namespace MobileGL::MG_Remote::Client {
if (CreateIsSuspect(handle) || g_createSuspects.size() >= kCreateSuspectMax) return;
g_createSuspects.push_back(handle);
}
// THE WINDOW IS A SESSION'S, NOT THE PROCESS'S. Every entry names a sequence number of the
// session that wrote it, and a process can run several sessions one after another (eglinfo
// terminates one display and initializes the next). An entry left by the previous session
// made the new session's first create wait for a sequence number the new server would not
// reach for a long time (measured: the second session hung on its first texture create, in
// DrainCreateWindow's blocking WaitForApplied). Its answer, and the object it was about,
// died with that session; the suspects are that session's handles too.
void ForgetPreviousSessionCreates() {
g_createWindowHead = 0;
g_createWindowCount = 0;
g_createSuspects.clear();
}
void ClearCreateSuspect(MG_Pipe::MGPipeHandle handle) {
for (SizeT i = 0; i < g_createSuspects.size(); ++i) {
if (g_createSuspects[i].Slot != handle.Slot || g_createSuspects[i].Gen != handle.Gen)
@@ -1209,6 +1221,7 @@ namespace MobileGL::MG_Remote::Client {
void InstallClientWireTables() {
using namespace MG_Pipe;
ForgetPreviousSessionCreates();
// DirectGLES normally installs this notice while creating its handle
// backend. An independent client never creates that backend: without a