RFR: 8000780 back ported Fix zero fail to build in JDK7u-dev [updated]

[Alejandro E Murillo]

oh yeah, hadn't seen your email.
Can reach the link now, thanks.
Once this is reviewed  either Chris (Thalinger) or myself  will get it 
into hs24
Note that there won't be any snapshot for hs24 in the next few weeks due to
the holidays, so it won't be in the 7u master for a while

Thanks
Alejandro

On 12/22/2012 11:15 AM, Chris Phillips @ T O wrote:
> Hi Alejandro,
> Oops emails crossed in the net ...
> See my latest email , I had to regenerate the webrev and patch -
> This webrev s/b ok:
>
> Webrev:
> http://integral-portal.lgonqn.org/temp/ChrisPhi/jdk7u-dev-zero-hs24b29_20121222/
>
>
> Chris
>
> On 22/12/12 01:03 PM, Alejandro E Murillo wrote:
>> Hi Chris,
>> In that case is not necessary.
>>   I initially thought the webrev was against jdk7u/jdk7u-dev/hotspot
>> but looks like it is against hsx/hsx24/hotspot.
>> BTW, I went to check the webrev link and is unreachable, can you check
>> that?
>> Thanks
>> Alejandro
>>
>> On 12/21/2012 9:53 PM, Chris Phillips @ T O wrote:
>>> Hi Alejandro,
>>>
>>> The only change since I tested this afternoon is the change
>>> for the hotspot_version. I will re-run tests if you wish but it seems
>>> unnecessary.
>>>
>>> Cheers!
>>> Chris
>>>
>>> On 21/12/12 07:40 PM, Alejandro E Murillo wrote:
>>>> On 12/21/2012 4:34 PM, Chris Phillips @ T O wrote:
>>>>> Hi
>>>>>
>>>>> {Not sure if this needs a new bug or not, it definitely needs a
>>>>> sponsor
>>>>> - Twisti ? }
>>>>>
>>>>> Attached is a backport (essentially resurrection of the Permgen code)
>>>>> of 8000780 now updated with Roman Kennke's comments. Checked and built
>>>>> against hsx24b28. (Current jdk7u-dev hs)
>>>>>
>>>>> Please review.
>>>>>
>>>>> Cheers!
>>>>> Chris
>>>>>     PS
>>>>> Webrev is here:
>>>>> http://integral-portal.lgonqn.org/temp/ChrisPhi/jdk7u-dev-zero/hs24b28/
>>>>>
>>>> Hi
>>>> jdk7u-dev/hotspot corresponds to hs24-b27.
>>>> I just got the hs24 snapshot for b28 and bumped the number to b29,
>>>> Can you rework the patch/webrev against this repo:
>>>>
>>>>    http://hg.openjdk.java.net/hsx/hsx24/hotspot
>>>>
>>>> once it's reviewed I can push it
>>>>
>>>> Thanks
>>>>
>>
>>