[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR(XS): JDK-8062037 java/lang/instrument/RetransformBigClass.sh should be quarantined
From:       "olivier.lagneau () oracle ! com" <olivier ! lagneau () oracle ! com>
Date:       2014-11-13 17:21:53
Message-ID: 5464E8B1.8090801 () oracle ! com
[Download RAW message or body]

Thanks Dan and Serguey for the quick reply.

I will change 8062037 reference to 8061177 in ProblemList.txt before 
pushing.

Olivier.

On 12/11/2014 19:22, Daniel D. Daugherty wrote:
> Thumbs up on the quarantine.
>
> However, I think the bug reference in ProblemList.txt should be
> 8061177 and not the bug used to do the quarantine.
>
> Dan
>
>
> On 11/12/14 11:19 AM, olivier.lagneau@oracle.com wrote:
>> Please review this addition to ProblemList.txt
>>
>> Issue: https://bugs.openjdk.java.net/browse/JDK-8062037
>> Initial Failure: https://bugs.openjdk.java.net/browse/JDK-8061177
>> Webrev: http://cr.openjdk.java.net/~olagneau/8062037/webrev.00/
>>
>> Retransform/RedefineBigClass.sh have been showing again random 
>> failure(s) with crash and 134 exitcode.
>> These are stress tests and showed they need a safe and not overloaded 
>> running context to avoid random
>> errors. They should be moved to a stress suite. Quarantining them is 
>> a first step to such move.
>>
>> When further moving them to some stress suite they should also be 
>> refactored (back) using memory leak
>> detection through OOM (rather than delta measurement on free memory) 
>> , and tuned to the expected running
>> context.
>>
>> Thanks,
>> Olivier.
>>
>


[Attachment #3 (text/html)]

<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#C0C0C0" text="#000099">
    <tt>Thanks Dan and Serguey for the quick reply.<br>
      <br>
      I will change </tt><tt><tt>8062037 </tt>reference to 8061177 in
      ProblemList.txt before pushing.<br>
      <br>
      Olivier.<br>
      <br>
    </tt>
    <div class="moz-cite-prefix">On 12/11/2014 19:22, Daniel D.
      Daugherty wrote:<br>
    </div>
    <blockquote cite="mid:5463A580.2040302@oracle.com" type="cite">
      <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
      <tt>Thumbs up on the quarantine.<br>
        <br>
        However, I think the bug reference in ProblemList.txt should be<br>
        8061177 and not the bug used to do the quarantine.<br>
        <br>
        Dan<br>
        <br>
        <br>
      </tt>
      <div class="moz-cite-prefix">On 11/12/14 11:19 AM, <a
          moz-do-not-send="true" class="moz-txt-link-abbreviated"
          href="mailto:olivier.lagneau@oracle.com">olivier.lagneau@oracle.com</a>
        wrote:<br>
      </div>
      <blockquote cite="mid:5463A49F.7030404@oracle.com" type="cite">
        <meta http-equiv="content-type" content="text/html;
          charset=utf-8">
        <tt>Please review this addition to ProblemList.txt<br>
          <br>
          Issue:                                   <a moz-do-not-send="true"
            class="moz-txt-link-freetext"
            href="https://bugs.openjdk.java.net/browse/JDK-8062037">https://bugs.openjdk.java.net/browse/JDK-8062037</a><br>
  Initial Failure:               <a moz-do-not-send="true"
            class="moz-txt-link-freetext"
            href="https://bugs.openjdk.java.net/browse/JDK-8061177">https://bugs.openjdk.java.net/browse/JDK-8061177</a><br>
                
          Webrev:                                 <a moz-do-not-send="true"
            class="moz-txt-link-freetext"
            href="http://cr.openjdk.java.net/%7Eolagneau/8062037/webrev.00/">http://cr.openjdk.java.net/~olagneau/8062037/webrev.00/</a><br>
  <br>
          Retransform/RedefineBigClass.sh have been showing again random
          failure(s) with crash and 134 exitcode.<br>
          These are stress tests and showed they need a </tt><tt><tt>safe

            and not overloaded </tt>running context to avoid random<br>
          errors. They should be moved to a stress suite. Quarantining
          them is a first step to such move.<br>
          <br>
          When further moving them to some stress suite they should also
          be refactored (back) using memory leak<br>
          detection through OOM (rather than delta measurement on free
          memory) , and tuned to the expected running<br>
          context.<br>
          <br>
          Thanks,<br>
          Olivier.<br>
          <br>
        </tt> </blockquote>
      <br>
    </blockquote>
    <br>
  </body>
</html>



[prev in list] [next in list] [prev in thread] [next in thread]

Configure | About | News | Add a list | Sponsored by KoreLogic