Conversation
|
Hello there, We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process. Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6 Thank you for contributing to Nextcloud and we hope to hear from you soon! (If you believe you should not receive this message, you can add yourself to the blocklist.) |
Single Nextcloud instances have shown to use less than 2 MiB of the configured JIT buffer size, so that 8 MiB is sufficient by a large margin. Using any larger value pre-occupies the `opcache.memory_consumption` accurdingly and 128 MiB would hence fill its default 128 MiB completely, without any benefit. The example JIT size is hence reduced to 8 MiB. Enabling JIT however has shown to raise overall OPcache usage by a larger amount than the configured JIT size, so that `opcache.memory_consumption` might still need to be raised in some cases. This information is added to the documentation: nextcloud/all-in-one#2755 (comment) Additionally, the info is added that JIT is currently supported on x86 platforms only. Signed-off-by: MichaIng <[email protected]>
pulsejet
left a comment
pulsejet
left a comment
There was a problem hiding this comment.
Never tested this in depth, but makes sense.
|
/backport to stable30 |
|
/backport to stable29 |
2 participants
Single Nextcloud instances have shown to use less than 2 MiB of the configured JIT buffer size, so that 8 MiB is sufficient by a large margin. Using any larger value pre-occupies the
opcache.memory_consumptionaccurdingly and 128 MiB would hence fill its default 128 MiB completely, without any benefit.The example JIT size is hence reduced to 8 MiB. Enabling JIT however has shown to raise overall OPcache usage by a larger amount than the configured JIT size, so that
opcache.memory_consumptionmight still need to be raised in some cases. This information is added to the documentation: nextcloud/all-in-one#2755 (comment)Additionally, the info is added that JIT is currently supported on x86 platforms only.
☑️ Resolves
🖼️ Screenshots