RFR(XXL): 8079792: GC directory structure cleanup

Coleen Phillimore coleen.phillimore at oracle.com
Tue May 12 18:25:12 UTC 2015


On 5/12/15, 2:08 PM, Derek White wrote:
> On 5/12/15 6:16 AM, Per Liden wrote:
>> Hi,
>>
>> As previously mentioned [1], the GC team is doing a cleanup of the 
>> directory structure for the GC code. Here's the patch for that cleanup.
>>
>> First, a recap of the new directory structure:
>>
>> - A single "top-level" directory for GC code:
>>     src/share/vm/gc/
>>
>> - One sub-directory per GC:
>>     src/share/vm/gc/cms/
>>     src/share/vm/gc/g1/
>>     src/share/vm/gc/parallel/
>>     src/share/vm/gc/serial/
>>
>> - A single directory for common/shared GC code:
>>     src/share/gc/shared/
>
> Typo? I think you meant:
>       src/share/vm/gc/shared/
>
> That's what the webrev looks like.
>
> Looking forward to the simpler structure (luckily my fingers don't 
> have the old structure memorized :-)

My fingers had the old structure somewhat memorized since it never made 
mental sense to me, but even so, I'm very pleased to see this change!

I can't tell from the giant webrev, but did you reorganize the SA 
duplicated directory structure?  It seems like some files moved but not 
all of them.  If not with this change, can you do a follow-on an RFE to 
fix it also?

Thanks,
Coleen

>
>  - Derek



More information about the hotspot-dev mailing list