Bypassing the unnecessary zend_stream_open() in the ZEND_INCLUDE_OR_EVAL_*_HANDER() rtns about zendoptimizerplus HOT 3 CLOSED

Hi Terry,

You probably missed something, but O+ already overrides zend_stream_open() and zend_resolve_path() and eliminates almost all syscalls. You can see strace output for a second request for a script that includes two other files through include_once() (PHP-5.5, FastCGI sapi).

accept(4, {sa_family=AF_FILE, NULL}, [2]) = 5
poll([{fd=5, events=POLLIN}], 1, 5000) = 1 ([{fd=5, revents=POLLIN}])
read(5, "\1\1\334l\0\10\0\0", 8) = 8
read(5, "\0\1\0\0\0\0\0\0", 8) = 8
read(5, "\1\4\334l\2t\4\0", 8) = 8
read(5, "\r\30DOCUMENT_ROOT/usr/local/apache"..., 632) = 632
read(5, "\1\4\334l\0\0\0\0", 8) = 8
time(NULL) = 1362502136
gettimeofday({1362502136, 602388}, NULL) = 0
setitimer(ITIMER_PROF, {it_interval={0, 0}, it_value={600, 0}}, NULL) = 0
rt_sigaction(SIGPROF, {0x8384ae0, [PROF], SA_RESTART}, {0x8384ae0, [PROF], SA_RESTART}, 8) = 0
rt_sigprocmask(SIG_UNBLOCK, [PROF], NULL, 8) = 0
getcwd("/home/dmitry/php/php5.5/CGI-RELEASE", 4095) = 36
chdir("/usr/local/apache/htdocs/test") = 0
fcntl64(3, F_SETLK, {type=F_RDLCK, whence=SEEK_SET, start=1, len=1}) = 0
getcwd("/var/www/html/test", 4096) = 19
chdir("/home/dmitry/php/php5.5/CGI-RELEASE") = 0
setitimer(ITIMER_PROF, {it_interval={0, 0}, it_value={0, 0}}, NULL) = 0
fcntl64(3, F_SETLK, {type=F_UNLCK, whence=SEEK_SET, start=0, len=0}) = 0
write(5, "\1\6\334l\0D\4\0X-Powered-By: PHP/5.5.0-"..., 96) = 96
shutdown(5, SHUT_WR) = 0
recv(5, "\1\5\334l\0\0\0\0", 8, 0) = 8
close(5) = 0
setitimer(ITIMER_PROF, {it_interval={0, 0}, it_value={0, 0}}, NULL) = 0

No open(), no even stat().

Thanks. Dmitry.

from zendoptimizerplus.

:oops: I am still working statically 'cos I am setting up to refactor in my CLI / CGI changes in a test fork with a view to offering them back once shaken down. I am still feeling my way around the code. I should have checked, of course. At least I suggested the solution that you guys have already implemented. :-(

Sorry Dmitry. Durrrhhh.

from zendoptimizerplus.

no problem, you are welcome with ideas :)

from zendoptimizerplus.