Skip to content

Commit

Permalink
Merge pull request #33030 from nextcloud/backport/32968/stable23
Browse files Browse the repository at this point in the history
[stable23] Do not display the buffer_size info if the size is already set to a c…
  • Loading branch information
szaimen authored Jun 27, 2022
2 parents 3aff3ef + 6d0d38a commit 46642ca
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion apps/settings/lib/Controller/CheckSetupController.php
Original file line number Diff line number Diff line change
Expand Up @@ -525,7 +525,11 @@ protected function getOpcacheSetupRecommendations(): array {

if (
empty($status['interned_strings_usage']['free_memory']) ||
($status['interned_strings_usage']['used_memory'] / $status['interned_strings_usage']['free_memory'] > 9)
(
($status['interned_strings_usage']['used_memory'] / $status['interned_strings_usage']['free_memory'] > 9) &&
// Do not recommend to raise the interned strings buffer size above a quarter of the total OPcache size
($this->iniGetWrapper->getNumeric('opcache.interned_strings_buffer') < $this->iniGetWrapper->getNumeric('opcache.memory_consumption') / 4)
)
) {
$recommendations[] = $this->l10n->t('The OPcache interned strings buffer is nearly full. To assure that repeating strings can be effectively cached, it is recommended to apply <code>opcache.interned_strings_buffer</code> to your PHP configuration with a value higher than <code>%s</code>.', [($this->iniGetWrapper->getNumeric('opcache.interned_strings_buffer') ?: 'currently')]);
}
Expand Down

0 comments on commit 46642ca

Please sign in to comment.