Changeset 2226

Timestamp:

Jul 6, 2005, 6:39:45 AM (20 years ago)

Author:

bird

Message:

o Reap embryos when the waiter thread is querying spm about

the death certificate. This, together with a tiny fix
in spawnve() cures the bad spm process leaks when spawning
non-libc processes seen with make when building libc.

Location:

trunk/src/emx

Files:

• ChangeLog.LIBC (modified) (1 diff, 1 prop)
• src/lib/sys/sharedpm.c (modified) (1 diff, 1 prop)

trunk/src/emx/ChangeLog.LIBC

@@ -8 +8 @@
           orphants are adopted and killed by the init process.)
         o Added signal handler types for POSIX, BSD and GNU conformance.
+        o Reap embryos when the waiter thread is querying spm about
+          the death certificate. This, together with a tiny fix
+          in __spawnve() cures the bad spm process leaks when spawning
+          non-libc processes seen with make when building libc.
 
 2005-07-04: knut st. osmundsen <bird-gccos2-spam@anduin.net>

trunk/src/emx/src/lib/sys/sharedpm.c

@@ -266 +266 @@
             pPrev = pNotify;
             pNotify = pNotify->pNext;
+        }
+
+        /*
+         * If we didn't find the process, let's check the embryo list
+         * since it's likely to be a non-libc process.
+         *
+         * This ASSUMES that this function is only called after death.
+         */
+        if (rc)
+        {
+            for (__LIBC_PSPMPROCESS pEmbryo = gpSPMHdr->apHeads[__LIBC_PROCSTATE_EMBRYO]; pEmbryo; pEmbryo = pEmbryo->pNext)
+                if (pEmbryo->pid == pid)
+                {
+                    if (pEmbryo->cReferences == 0)
+                        spmFreeProcess(pEmbryo);
+                    else
+                    {   /* Make zombie - needs a reference to work right. */
+                        pEmbryo->cReferences++;
+                        spmZombieOrFree(pEmbryo);
+                    }
+                    break;
+                }
         }