Enable Zend Signals Handler By Default

9 years ago by Xinchen Hui — view source

unread

Hey internal:

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.

 You may still remember we disabled zend signal handlers before

PHP-5.4 was released because it doesn't work well as mod_apache. which
is described in #61083.

 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/fd5a756ad44124ffa7e9a5810a31ff49e91708cf

 And no obviously performance affects in reallife app(benched with

wordpress).

 Next, We may do some further optimizationes based on that.

 For now, I want to make zend signals handler enable by default

(in configure) for PHP7, or maybe totally removed the
--enable-zend-signals option? thus make zend_signal.c always be
compiled and linked.

 Is there any concerns or objections on this?

thanks

--
Xinchen Hui
@Laruence
http://www.laruence.com/

9 years ago by Pierre Joye — view source

unread

Hey internal:
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.
 You may still remember we disabled zend signal handlers before
PHP-5.4 was released because it doesn't work well as mod_apache. which
is described in #61083.
 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/fd5a756ad44124ffa7e9a5810a31ff49e91708cf
 And no obviously performance affects in reallife app(benched with
wordpress).
 Next, We may do some further optimizationes based on that.

 For now, I want to make zend signals handler enable by default
(in configure) for PHP7, or maybe totally removed the
--enable-zend-signals option? thus make zend_signal.c always be
compiled and linked.
 Is there any concerns or objections on this?

Generally speaking no, but I would be very careful about the
portability parts here, and not only about windows.

I would go in two steps, first enable it by default and see which
issues exist on the various supported platforms then once everything
is ready, we can consider removing the setting. But removing it
straight away may force users to simply do not use 7 anymore until
things are fixed, if necessary.

--
Pierre

@pierrejoye | http://www.libgd.org

9 years ago by Xinchen Hui — view source

unread

Hey internal:
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.
 You may still remember we disabled zend signal handlers before
PHP-5.4 was released because it doesn't work well as mod_apache. which
is described in #61083.
 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/fd5a756ad44124ffa7e9a5810a31ff49e91708cf
 And no obviously performance affects in reallife app(benched with
wordpress).
 Next, We may do some further optimizationes based on that.

 For now, I want to make zend signals handler enable by default
(in configure) for PHP7, or maybe totally removed the
--enable-zend-signals option? thus make zend_signal.c always be
compiled and linked.
 Is there any concerns or objections on this?
Generally speaking no, but I would be very careful about the
portability parts here, and not only about windows.
Here is the problem, if --enable-zend-signals == no

then the whole zend-signals won't be compiled and linked..

I am going to do it in another way. always compile it. but let's make
--enable-zend-sgianls means use or not use.

thanks

I would go in two steps, first enable it by default and see which
issues exist on the various supported platforms then once everything
is ready, we can consider removing the setting. But removing it
straight away may force users to simply do not use 7 anymore until
things are fixed, if necessary.

--
Pierre

@pierrejoye | http://www.libgd.org

--
Xinchen Hui
@Laruence
http://www.laruence.com/

9 years ago by Wei Dai — view source

unread

在 2015年3月5日星期四，下午9:27，Xinchen Hui 写道：

Hi,

Hey internal:

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.

You may still remember we disabled zend signal handlers before
PHP-5.4 was released because it doesn't work well as mod_apache. which
is described in #61083.

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/fd5a756ad44124ffa7e9a5810a31ff49e91708cf

And no obviously performance affects in reallife app(benched with
wordpress).

Next, We may do some further optimizationes based on that.

For now, I want to make zend signals handler enable by default
(in configure) for PHP7, or maybe totally removed the
--enable-zend-signals option? thus make zend_signal.c always be
compiled and linked.

Is there any concerns or objections on this?
I’m +1 on this, it sounds good and helpful.
Thank you for your work.

thanks

--
Xinchen Hui
@Laruence
http://www.laruence.com/

--
Wei Dai
@demon