#32220 [Asn]: [PATCH] thread_resources for thread not getting freed when apache kills thread

From: Date: Mon, 05 Dec 2005 08:03:16 +0000
Subject: #32220 [Asn]: [PATCH] thread_resources for thread not getting freed when apache kills thread
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-89743@lists.php.net to get a copy of this message 
 ID:               32220
 User updated by:  kameshj at fastmail dot fm
 Reported By:      kameshj at fastmail dot fm
 Status:           Assigned
 Bug Type:         Apache2 related
 Operating System: *
 PHP Version:      5CVS, 4CVS (2005-03-25)
 Assigned To:      dmitry
 New Comment:

Yes you are correct that only IIS sapi is making use of ts_free_thread.
But If you look at my patch you can see that I am registering this
ts_free_thread function to pthread_push_cleanup to make sure the
pthread subsystem to do the cleanup for us as soon as the thread is
getting killed not to wait for the main thread to free the full
thread_resources.


Previous Comments:
------------------------------------------------------------------------

[2005-12-01 15:51:41] dmitry@php.net

I confirm that PHP doesn't call ts_free_thread() if Web Server (except
IIS) kills some thread. As a result all thread resources are not freed
before server shutdown.

The proposed patch is invalid and even cannot be compiled on Linux.

------------------------------------------------------------------------

[2005-11-10 18:36:39] sniper@php.net

Yes, but did you check out the patch..?


------------------------------------------------------------------------

[2005-11-08 09:50:11] dmitry@php.net

I am not able to reproduce this. Seems like a mess in report. Apache2
worker MPM doesn't support StartThreads and MaxThreads directives.

------------------------------------------------------------------------

[2005-11-03 22:21:58] sniper@php.net

Dmitry, can you check this out?

------------------------------------------------------------------------

[2005-03-25 02:05:13] sniper@php.net

Zeev, here's another thread issue, can you take a look in the patch?


------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
    http://bugs.php.net/32220

-- 
Edit this bug report at http://bugs.php.net/?id=32220&edit=1

Thread (14 messages)

« previous php.bugs (#89743) next »