Request for review: 7197627: There are issues with shared data on windows

Ioi Lam ioi.lam at
Wed Jan 30 08:55:43 PST 2013

On 01/30/2013 06:20 AM, Daniel D. Daugherty wrote:
> On 1/30/13 6:39 AM, harold seigel wrote:
>> Hi,
>> Please review the following change to fix bug 7197627.
>> Summary:
>> The original fix for this bug created the CDS classes.jsa file with 
>> read/write protection.  This change creates the file with read-only 
>> protection and only changes the file/s protection to read/write when 
>> deleting the file.  This is a safer implementation.
>> This was tested on Windows 7 and Windows XP and sanity tested on Linux.
>> Open webrev at 
>> <>
>> Bug link at
>> Thanks!  Harold
> Thumbs up on the safer change.
> This comment:
> 217   // Remove the existing file in case another process has it open.
> bothers me. On Windows, if another process has the classes.jsa file
> open, then you won't be able to remove it (or move it or...) so the
> comment is misleading. Or I'm wrong about Windows which could easily
> be true...
> Dan
This comment still applies to *nix. The remove() call will remove the 
file from the file system, but the other process holding a file ID to it 
will still be able to access the old contents. A good test case would be 
one app running with -Xshare:on while you run -Xshare:dump from a 
different terminal.

Maybe we need to do an #ifdef around this comment?

- Ioi

More information about the hotspot-runtime-dev mailing list