Hello Phil,
Thanks for the review. Yes, I have run jtreg on it. Regards, Rajeev Chamyal From: Philip Race Sent: 20 October 2015 03:21 To: Rajeev Chamyal Cc: Sergey Bylokhov; Alexander Scherbatiy; [email protected] Subject: Re: Review request for JDK-8030099 Memory usage of java process increases after pressing start button in test window The code change looks fine. The test is delightfully complicated. Did you run it under jtreg ? -phil. On 10/19/15, 5:10 AM, Rajeev Chamyal wrote: Hello, Please review the following fix for Jdk9: Bug: https://bugs.openjdk.java.net/browse/JDK-8030099 Webrev: HYPERLINK "http://cr.openjdk.java.net/%7Erchamyal/8030099/webrev.00/"http://cr.openjdk.java.net/~rchamyal/8030099/webrev.00/ Issue: The memory usage of java process goes on increasing if we call ShellFolder:listFiles API aggressively on some folder with large number of files/folders. Cause: Win32ShellFolder class registers its instances and native data with sun.java2d.Disposer class for later disposal. Win32ShellFolder instances are getting registered as PhantomReferences with Disposer and are not getting cleared immediately. This leads to increase of memory usage by java process. Fix: Instead of registering the complete Win32ShellFolder object reference with disposer we are now adding only a sentinel object and also using sun.java2d.Disposer :: addObjectRecord API which adds the references as WeakReferences. Regards, Rajeev Chamyal
