On 10/07/2018 22:02, Brent Christian wrote:

Please review this change to disable the file canonicalization cache by default.


From the bug report:
"The file canonicalization cache was added back in JDK 1.4.2 in order to improve startup time.

The cache has long-standing correctness issues (JDK-7066948, for example). For this reason, it has been desired to disable the cache by default, but the increase to startup time was undesirable.

Recent JDK releases have removed usages of the cache, in particular from FilePermission (JDK-8164705 in JDK 9). This reduced usage of the canonicalization cache should also reduce the startup effect of disabling the cache by default. Measurements support this. Previous measurements showed the startup effect of disabling the cache on Linux to be 3-6%, depending on the specific benchmark. The same comparison performed last month now shows no startup change on Linux.

The file canonicalization cache can still be enabled by setting the "sun.io.useCanonCaches" system property. This is merely a change to the default value."
Happy to see this disabled as it has always been problematic. The patch looks okay although you don't of couse need to set the initial values to false.


Reply via email to