Hello Lukas, Wednesday, March 11, 2009, 5:10:57 PM, you wrote:
> Dmitry Stogov wrote: >> Hi Shire, >> >> I run patched APC on a number of real-life applications and got more >> than 30% speedup on XOOPS (99 req/sec instead of 60%) and 20% on >> ZendFramework (41 req/sec instead of 32), however most applications >> (drupal, qdig, typo3, wordpress) didn't show significant speed >> difference. As was expected the patch doesn't affects PHP without APC or >> with APC and lazy loading disabled. >> >> I also got APC warning with Zend Framewoek based blog application, but I >> didn't try to look deeper. >> >> [Wed Mar 11 17:53:02 2009] [apc-warning] [Wed Mar 11 17:53:02 2009] >> [apc-warning] apc_lookup_class_hook: could not install blogrow in >> /var/www/html/bench/fw/ZendFramework-1.5.0RC3/library/Zend/Loader.php on >> line 86 >> >> I didn't look careful into APC code, just into PHP patch and I see the >> following issues: >> >> 1) I would prefer to add additional hash_value argument into >> lookup_function_hook() and lookup_class_hook to prevent multiple >> calculation. >> >> 2) function_exists() should use lookup_function_hook(). >> >> 3) interface_exists() and class_alias() should use lookup_class_hook(). >> >> 4) ext/soap, ext/reflection, ext/wddx and ext/spl autoload support >> >> Anyway, it's very good job and 20-30% speedup on some real-life >> applications makes sense to include it into 5.3 (from my point of view). > Makes sense to me as well, especially since I don't see any drawbacks > for non-accelerated scripts. > I also think some of those applications that didn't show any gains could > be trivially modified to make use of it. Moving from conditionally > loaded stuff to lazy-loaded is likely to speed things up. Depending of > course on how granular the conditional loading is. Right, basically the patch adresses what we say at conferences. Autload can do an amazing job but also harm you. The patch on the other hand somehow combines the advantages of both worlds. You can be explicit because you know it better and for that you do not get any punishment from Autoload execution time. And then again, you don't get punished by including too much. Becuase after all the compiler knows it even better. And of course, given that some people gain over 20%, I don't see why we even need to discuss putting this in. Last but not least, Lukas, what happened, to putting APC into core? marcus Best regards, Marcus -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
