Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:84372
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.220.49 as permitted sender)
Date: Fri, 6 Mar 2015 09:20:17 +0800
To: Xinchen Hui <laruence@php.net>
Cc: PHP Internals <internals@lists.php.net>
Message-ID: <CDC62DEF06224315B091903CE27B2022@gmail.com>
In-Reply-To: <CABBJUpfhGtB2JPA7VS1c5Syw0OcZyTp--rnA+qCaiCjpxDCDtg@mail.gmail.com>
References: <CABBJUpfhGtB2JPA7VS1c5Syw0OcZyTp--rnA+qCaiCjpxDCDtg@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/alternative; boundary="54f90149_32fff902_36a5"
Subject: =?utf-8?Q?=E5=9B=9E=E5=A4=8D=EF=BC=9A_?=[PHP-DEV] Enable Zend
 Signals Handler By Default
From: zxcvdavid@gmail.com (Wei Dai)

--54f90149_32fff902_36a5
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

=E5=9C=A8 2015=E5=B9=B43=E6=9C=885=E6=97=A5 =E6=98=9F=E6=9C=9F=E5=9B=9B=EF=
=BC=8C=E4=B8=8B=E5=8D=889:27=EF=BC=8CXinchen Hui =E5=86=99=E9=81=93=EF=BC=
=9A

Hi,  =20
> Hey internal:
> =20
> =20
> There is also a problem we have been suffering from for a quite
> long time, that is signal raised at OOM/Timeout when execution is in a
> critical area will lead PHP crash or even more big troubles if it is
> in shared memory.
> =20
> You may still remember we disabled zend signal handlers before
> PHP-5.4 was released because it doesn't work well as mod=5Fapache. whic=
h
> is described in =2361083.
> =20
> Zend signal is a good way to fix the problem, thus I did some
> research then fixed zend signals(make it works with apache2handler),
> committed in https://github.com/php/php-src/commit/fd5a756ad44124ffa7e9=
a5810a31ff49e91708cf
> =20
> And no obviously performance affects in reallife app(benched with
> wordpress).
> =20
> Next, We may do some further optimizationes based on that.
> =20
> =46or now, I want to make zend signals handler enable by default
> (in configure) for PHP7, or maybe totally removed the
> --enable-zend-signals option=3F thus make zend=5Fsignal.c always be
> compiled and linked.
> =20
> Is there any concerns or objections on this=3F
I=E2=80=99m +1 on this, it sounds good and helpful.
Thank you for your work.
> =20
> thanks
> =20
> -- =20
> Xinchen Hui
> =40Laruence
> http://www.laruence.com/
> =20
> =20
> =20


--
Wei Dai
=40demon

--54f90149_32fff902_36a5--