RFR: 8059625 - JEP-JDK-8043304: Test task: DTrace- tests for segmented codecache feature

[serguei.spitsyn at oracle.com]

Hi Dmitry,


It looks good in general.
Some minor comments are below.

test/testlibrary/com/oracle/java/testlibrary/dtrace/DtraceRunner.java

   42     public static final String PERMIT_DESCTUCTIVE_ACTIONS_DTRACE_OPTION = "w";

    A typo in the constant name: DESCTUCTIVE => DESTRUCTIVE


test/compiler/codecache/dtrace/SegmentedCodeCacheDtraceTest.java

   84                 DtraceRunner.PERMIT_DESCTUCTIVE_ACTIONS_DTRACE_OPTION,

    A typo in the constant name: DESCTUCTIVE => DESTRUCTIVE


   60     private static final String WORKER_CLASS_NAME
   61             = SegmentedCodeCacheDtraceTestWorker.class.getName();
  ...
   80         runner.runDtrace(JDKToolFinder.getTestJDKTool("java"), JAVA_OPTS,
   81                 SegmentedCodeCacheDtraceTestWorker.class.getName(), params,


   The WORKER_CLASS_NAME can be used at 81.


   75         String params = SegmentedCodeCacheDtraceTestWorker.TESTED_METHODS_LIST.stream()
  ...
  179             List<Executable> tml
  180                     = SegmentedCodeCacheDtraceTestWorker.TESTED_METHODS_LIST;
  ...
  235             List<Executable> mlist
  236                     = SegmentedCodeCacheDtraceTestWorker.TESTED_METHODS_LIST;


    The TESTED_METHODS_LIST is used three times.
    It can be cached at the top and re-used.


Thanks,
Serguei



On 12/19/14 11:03 AM, Dmitrij Pochepko wrote:
> Hi all,
>
> Please review changes for 
> https://bugs.openjdk.java.net/browse/JDK-8059625 - JEP-JDK-8043304: 
> Test task: DTrace- tests for segmented codecache feature
>
> Description: this fix introduce dtrace test, which verify that 
> different combinations of available compile levels(and, in case 
> compile levels allows it, different code heaps as result)  doesn't 
> affect callstack shown by dtrace. There is a control class 
> SegmentedCodeCacheDtraceTest.java and class for running via dtrace 
> SegmentedCodeCacheDtraceTestWorker.java. A dtrace d script is also 
> present (SegmentedCodeCacheDtraceTestScript.d). A control class is 
> using DtraceRunner.java to run dtrace and then analyzing results using 
> class SegmentedCodeCacheDtraceResultsAnalyzer with 
> DtraceResultsAnalyzer interface.
> There is also a small class CompilerUtils.java created for usefull 
> common code.
>
> webrev: http://cr.openjdk.java.net/~iignatyev/dpochepk/8059625/webrev.00/
>
> Additional note: Please note that this path assumes that fix for 
> JDK-8066440 - Various changes in testlibrary for JDK-8059613 is also 
> applied.
>
> Thanks,
> Dmitrij

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20141222/5a8c681f/attachment-0001.html>