Hi

On 11/01/13 12:29 PM, Andrew Hughes wrote:
> ----- Original Message -----
>> On 21/12/2012 9:51 AM, Coleen Phillimore wrote:
>>> I am checking these changes into hsx24 now.
>> Thanks Coleen.
>>
> Yes, thanks Coleen for taking this on.  I see they've now appeared in 
> 7u/hotspot.
>
> http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/c707a5af0d71
> http://hg.openjdk.java.net/jdk7u/jdk7u/hotspot/rev/9d98c1eb82b0
>
> Any chance of getting:
>
> 8000780: make Zero build and run with JDK8
> http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/a3e2f723f2a5
This required a backport that has been reviewed by Roman and
Christian (Thanks!).

  See attached email's from Alejandro and Christian.


>
> in too?  Or is it not appropriate?
>
> CCing the original reviewers & author.
>
Chris

--- Begin Message ---

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







--- End Message ---
--- Begin Message ---
On Dec 22, 2012, at 10:15 AM, Chris Phillips @ T O <chris...@lgonqn.org> 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/

That looks good.  -- Chris

> 
> 
> 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
>>>> 
>> 
>> 
>> 
> 





--- End Message ---

Reply via email to