Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:110232
To: Derick Rethans <derick@php.net>
CC: "internals@lists.php.net" <internals@lists.php.net>
Thread-Topic: [PHP-DEV] [RFC] opcache.no_cache: Opcache optimization without
 any caching
Thread-Index: AQHWK5wzTowRpqnQX0mkb/2wfLlg3airAgEAgAAIULyAB423UA==
Date: Thu, 21 May 2020 14:07:28 +0000
Message-ID:
 <DM6PR07MB661827F45AC68DE3DCAA8FCBF9B70@DM6PR07MB6618.namprd07.prod.outlook.com>
References:
 <DM6PR07MB6618FDC90008510CEFFB9B19F9BA0@DM6PR07MB6618.namprd07.prod.outlook.com>,<alpine.DEB.2.22.394.2005161904160.3289@singlemalt.home.derickrethans.nl>,<DM6PR07MB661867BE0ECCE854D7ED7E2FF9BA0@DM6PR07MB6618.namprd07.prod.outlook.com>
In-Reply-To:
 <DM6PR07MB661867BE0ECCE854D7ED7E2FF9BA0@DM6PR07MB6618.namprd07.prod.outlook.com>
Accept-Language: en-CA, en-US
Content-Language: en-CA
x-ms-exchange-messagesentrepresentingtype: 1
x-tmn: [a1Y0NFeWrPLVPkPbp1MUK+DRCiCTcZSzLUffZqguXfKNe9l6Pbp0G4cRtSGb3kHF]
x-ms-publictraffictype: Email
x-incomingheadercount: 46
x-eopattributedmessage: 0
x-ms-office365-filtering-correlation-id: 6bc4681f-19e8-42be-3e93-08d7fd904bd5
x-ms-traffictypediagnostic: SN1NAM04HT153:
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info:
 la+qUjTyCbawkg0lkv6c+1bVhjIzAwBbdbINPfuCtCCsMvo9jytilanPlmfIRpF5eCABrQ7OgdHjgF58N2MJHfGgne9P+R3BwIDS57M+dAPBEEG1PkAwwPc5tqMEsOQ1IC7fOF+VaVcqPxmKa/QhhPEoxHFp6PCURs0S3vroBt038DzRh1pT7iiRE9cU/rZi/tJ0lhhJQL9iTUPPf3lU7dQkjJ2rQruNAgi4tUTH3tpJ3tmSwixw8ccC4DDm5AiE
x-forefront-antispam-report:
 CIP:255.255.255.255;CTRY:;LANG:en;SCL:0;SRV:;IPV:NLI;SFV:NSPM;H:DM6PR07MB6618.namprd07.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:;DIR:OUT;SFP:1901;
x-ms-exchange-antispam-messagedata:
 7E7eCxbNbIDq4hFhWHhL8F9TmXfglwLtWBdcdBGwywGT3IhPNZRWo66QR5ywnOM9Kf/t3GzaV638ea7ZTZMcGuyydGo4ScvOFrqN4yMvT5e8XM8n+zdizEuG17EVu20iQ24tCUw23w50BM6Ty/cOQYVwRFlbkwISUUGyTkpfOQfJy7apipkQEX+gccvFWgOJi0vMI3hPNxaCuC7ceysAzg==
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: hotmail.com
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-CrossTenant-Network-Message-Id: 6bc4681f-19e8-42be-3e93-08d7fd904bd5
X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-CrossTenant-originalarrivaltime: 21 May 2020 14:07:28.4679
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Internet
X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN1NAM04HT153
Subject: Re: [PHP-DEV] [RFC] opcache.no_cache: Opcache optimization without
 any caching
From: tysonandre775@hotmail.com (tyson andre)

Hi internals,=0A=
=0A=
https://wiki.php.net/rfc/opcache.no_cache=A0has=A0been updated.=0A=
Instead of opcache.no_cache=3D1, opcache.enable_cache=3D0 is now used to di=
sable caching.=0A=
=0A=
In the RFC, I wrote that opcache_get_status() now includes the following ne=
w booleans=0A=
=0A=
- `optimizations_enabled`, which is true if any optimization passes will ge=
t run.=0A=
- `allow_cache`, which will be true if any form of opcode caching is enable=
d.=0A=
  Maybe `cache_opcodes` or `opcode_cache_enabled` would be a better name.=
=0A=
=0A=
Any thoughts on whether opcache_get_status() should be changed at all here?=
=0A=
It's probably possible but inconvenient to determine what's being done with=
 extension_loaded() and multiple calls to ini_get(),=0A=
and the intended meaning of existing flags such as `opcache_enabled` was ne=
ver documented=0A=
in https://www.php.net/opcache_get_status=0A=
=0A=
Any other thoughts on the RFC?=0A=
=0A=
> > In my opinion, configuration names should never be "in the negative".=
=0A=
>=A0> Instead of defaulting to opcache.no_cache=3D0, default to opcache.cac=
he=3D1.=0A=
> =0A=
> ... the setting isn't actually enabling cache, it's allowing cache (i.e. =
opposite of disabling),=0A=
> so making this "opcache.allow_cache" might be the best. I'll probably go =
with that.=0A=
> (similar to allow_url_fopen, allow_url_include)=0A=
> =0A=
> ... `opcache.cache` could be confused with the caching strategy or the pa=
th to the cache,=0A=
> or being sufficient on its own to enable caching=0A=
=0A=
Thanks,=0A=
- Tyson=0A=