Re: com php-src: Use shorter php.ini directive name "opcache.huge_code_pages" and disable it by default.: ext/opcache/ZendAccelerator.c
ext/opcache/ZendAccelerator.h ext/opcache/zend_accelerator_module.c

From: Date: Tue, 15 Sep 2015 01:46:55 +0000
Subject: Re: com php-src: Use shorter php.ini directive name "opcache.huge_code_pages" and disable it by default.: ext/opcache/ZendAccelerator.c
ext/opcache/ZendAccelerator.h ext/opcache/zend_accelerator_module.c
References: 1  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-89026@lists.php.net to get a copy of this message
Looks like you forgot to add these into the actual php.ini-development and php.ini-production. The option itself should at least be included with the default value, even though it isn't documented yet. -Hannes On Mon, Sep 14, 2015 at 3:54 PM, Dmitry Stogov <dmitry@php.net> wrote: > Commit: c104d329e1e3b0fe10b1291b6a3704a1f6ce4d26 > Author: Dmitry Stogov <dmitry@zend.com> Tue, 15 Sep 2015 01:54:04 +0300 > Parents: f2480582fbf8b15de934cceaeee86da7313e8acf > Branches: PHP-7.0 master > > Link: > http://git.php.net/?p=php-src.git;a=commitdiff;h=c104d329e1e3b0fe10b1291b6a3704a1f6ce4d26 > > Log: > Use shorter php.ini directive name "opcache.huge_code_pages" and disable it by > default. > > Changed paths: > M ext/opcache/ZendAccelerator.c > M ext/opcache/ZendAccelerator.h > M ext/opcache/zend_accelerator_module.c > > > Diff: > diff --git a/ext/opcache/ZendAccelerator.c b/ext/opcache/ZendAccelerator.c > index d6fd262..2bbfb44 100644 > --- a/ext/opcache/ZendAccelerator.c > +++ b/ext/opcache/ZendAccelerator.c > @@ -2585,7 +2585,7 @@ static int accel_startup(zend_extension *extension) > #endif > > #ifdef HAVE_HUGE_CODE_PAGES > - if (ZCG(accel_directives).enable_huge_code_pages && > + if (ZCG(accel_directives).huge_code_pages && > (strcmp(sapi_module.name, "cli") == 0 || > strcmp(sapi_module.name, "cli-server") == 0 || > strcmp(sapi_module.name, "cgi-fcgi") == 0 || > diff --git a/ext/opcache/ZendAccelerator.h b/ext/opcache/ZendAccelerator.h > index da3fc66..93344ec 100644 > --- a/ext/opcache/ZendAccelerator.h > +++ b/ext/opcache/ZendAccelerator.h > @@ -220,7 +220,7 @@ typedef struct _zend_accel_directives { > zend_bool file_cache_consistency_checks; > #endif > #ifdef HAVE_HUGE_CODE_PAGES > - zend_bool enable_huge_code_pages; > + zend_bool huge_code_pages; > #endif > } zend_accel_directives; > > diff --git a/ext/opcache/zend_accelerator_module.c b/ext/opcache/zend_accelerator_module.c > index 5dfa8c1..11d631c 100644 > --- a/ext/opcache/zend_accelerator_module.c > +++ b/ext/opcache/zend_accelerator_module.c > @@ -310,7 +310,7 @@ ZEND_INI_BEGIN() > STD_PHP_INI_ENTRY("opcache.file_cache_consistency_checks" , "1" , > PHP_INI_SYSTEM, OnUpdateBool, accel_directives.file_cache_consistency_checks, > zend_accel_globals, accel_globals) > #endif > #ifdef HAVE_HUGE_CODE_PAGES > - STD_PHP_INI_BOOLEAN("opcache.enable_huge_code_pages" , "1" , > PHP_INI_SYSTEM, OnUpdateBool, accel_directives.enable_huge_code_pages, > zend_accel_globals, accel_globals) > + STD_PHP_INI_BOOLEAN("opcache.huge_code_pages" , "0" , > PHP_INI_SYSTEM, OnUpdateBool, accel_directives.huge_code_pages, > zend_accel_globals, accel_globals) > #endif > ZEND_INI_END() > > > -- > PHP CVS Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php >

« previous php.cvs (#89026) next »