[VOTE][RFC] Native TLS

php.internals

Anatol Belski

11 years ago
Hi, hereby the voting on https://wiki.php.net/rfc/native-tls#vote is opened. The vote starts on 12/10/2014 at 21:00 CET and ends on 12/17/2014 at 21:00 CET. Regards Anatol

Dmitry Stogov

11 years ago
+1 the patch is great. It doesn't affect non-ZTS build at all. ZTS build became even a bit faster. :) And we finally will able to forget about TSRMLS_ macros :) Non-ZTS ZTS master native-tls master native-tls bench.php [sec] 1.193 1.195 1.313 1.299 micro_bench.php [sec] 6.008 6.064 7.470 6.786 Wordpress-3.6 -T 1000 [sec] 13.308 13.293 14.997 14.611 bench.php [Mops] 7,055 7,055 7,885 7,342 Wordpress-3.6 -T 100 [Mops] 3,093 3,093 3,743 3,483 Mops - millions of instruction fetches measured using callgrind (the less the better) The solution requires minor changes in every third party extension, but it's really small cost for significant step forward. Thanks. Dmitry. On Wed, Dec 10, 2014 at 10:04 PM, Anatol Belski <anatol.php@belski.net> wrote:

Paul Dragoonis

11 years ago
On Wed, Dec 10, 2014 at 8:46 PM, Dmitry Stogov <dmitry@zend.com> wrote:
> > +1 > > the patch is great. > It doesn't affect non-ZTS build at all. > ZTS build became even a bit faster. :) >
> And we finally will able to forget about TSRMLS_ macros :) >
Say what? this is music to my ears :) Thanks for the patch.

Anatol Belski

11 years ago
Hi, On Wed, December 10, 2014 20:04, Anatol Belski wrote:
> Hi, > > > hereby the voting on https://wiki.php.net/rfc/native-tls#vote is opened. > The vote starts on 12/10/2014 at 21:00 CET and ends on 12/17/2014 at > 21:00 > CET. > >
this RFC was accepted, thanks everyone! I'm going to merge it next days. Regards Anatol

Paul Dragoonis

11 years ago
On Wed, Dec 17, 2014 at 8:27 PM, Anatol Belski <anatol.php@belski.net> wrote:
> > Hi, > > On Wed, December 10, 2014 20:04, Anatol Belski wrote: > > Hi, > > > > > > hereby the voting on https://wiki.php.net/rfc/native-tls#vote is opened. > > The vote starts on 12/10/2014 at 21:00 CET and ends on 12/17/2014 at > > 21:00 > > CET. > > > > > this RFC was accepted, thanks everyone! I'm going to merge it next days. >
Thank you for this improvement, it's appreciated.