That's great, thanks a lot Staffan!

Best regards,
  Goetz.

-----Original Message-----
From: Staffan Larsen [mailto:staffan.lar...@oracle.com] 
Sent: Montag, 26. Januar 2015 09:43
To: Lindenmaier, Goetz
Cc: serviceability-dev (serviceability-dev@openjdk.java.net); 
hotspot-...@openjdk.java.net Developers
Subject: Re: RFR(XS): 8068778: [TESTBUG] 
CompressedClassSpaceSizeInJmapHeap.java fails if SA not available

Change looks good to me - I’ll sponsor it!

/Staffan

> On 26 jan 2015, at 09:20, Lindenmaier, Goetz <goetz.lindenma...@sap.com> 
> wrote:
> 
> Hi,
> 
> could please somebody from servicability sponsor this tiny change?
> 
> (My first mail didn't go through, because I wasn't yet registered to 
> servicability-dev).
> 
> Thanks,
>  Goetz.
> 
> -----Original Message-----
> From: hotspot-dev [mailto:hotspot-dev-boun...@openjdk.java.net] On Behalf Of 
> Coleen Phillimore
> Sent: Freitag, 16. Januar 2015 23:56
> To: hotspot-...@openjdk.java.net
> Subject: Re: RFR(XS): 8068778: [TESTBUG] 
> CompressedClassSpaceSizeInJmapHeap.java fails if SA not available
> 
> 
> This seems fine.  Someone from the serviceability group should probably 
> sponsor.
> 
> Thanks,
> Coleen
> 
> On 1/16/15, 6:15 AM, Lindenmaier, Goetz wrote:
>> Hi,
>> 
>> could I please get a review for this small fix?  I also please need a 
>> sponsor.
>> 
>> Thanks,
>>   Goetz.
>> 
>> From: goetz.lindenma...@sap.com
>> Sent: Montag, 12. Januar 2015 09:23
>> To: hotspot-...@openjdk.java.net
>> Subject: RFR(XS): 8068778: [TESTBUG] CompressedClassSpaceSizeInJmapHeap.java 
>> fails if SA not available
>> 
>> Hi,
>> 
>> This test uses SA, thus fails on platforms where that's not implemented.
>> I see problems on mac and aix.
>> Call Platform.shouldSAAttach() to check whether SA should work.
>> 
>> Please review this small fix.  I please need a sponsor.
>> http://cr.openjdk.java.net/~goetz/webrevs/8068778-jtregSA/webrev.01/
>> 
>> Best regards,
>>   Goetz.
> 

Reply via email to