Print this page
rev 6900 : 8048169: Change 8037816 breaks HS build on PPC64 and CPP-Interpreter platforms
Summary: Fix the matching of format string parameter types to the actual argument types for the PPC64 and CPP-Interpreter files in the same way as 8037816 already did it for all the other files
Reviewed-by: stefank, coleenp, dholmes

Split Split Close
Expand all
Collapse all
          --- old/hotspot/src/cpu/ppc/vm/vm_version_ppc.cpp
          +++ new/hotspot/src/cpu/ppc/vm/vm_version_ppc.cpp
↓ open down ↓ 344 lines elided ↑ open up ↑
 345  345    start1 = os::current_thread_cpu_time(false);
 346  346    (*test2)();
 347  347    stop1 = os::current_thread_cpu_time(false);
 348  348  
 349  349    loop2_seconds = (stop1 - start1) / (1000 *1000 *1000.0);
 350  350  
 351  351    rel_diff = (loop2_seconds - loop1_seconds) / loop1_seconds *100;
 352  352  
 353  353    if (PrintAssembly) {
 354  354      ttyLocker ttyl;
 355      -    tty->print_cr("Decoding section size detection stub at " INTPTR_FORMAT " before execution:", code);
      355 +    tty->print_cr("Decoding section size detection stub at " INTPTR_FORMAT " before execution:", p2i(code));
 356  356      Disassembler::decode((u_char*)code, (u_char*)code_end, tty);
 357  357      tty->print_cr("Time loop1 :%f", loop1_seconds);
 358  358      tty->print_cr("Time loop2 :%f", loop2_seconds);
 359  359      tty->print_cr("(time2 - time1) / time1 = %f %%", rel_diff);
 360  360  
 361  361      if (rel_diff > 12.0) {
 362  362        tty->print_cr("Section Size 8 Instructions");
 363  363      } else{
 364  364        tty->print_cr("Section Size 32 Instructions or Power5");
 365  365      }
↓ open down ↓ 62 lines elided ↑ open up ↑
 428  428    a->dcbz(R3_ARG1); // R3_ARG1 = addr
 429  429    a->blr();
 430  430  
 431  431    uint32_t *code_end = (uint32_t *)a->pc();
 432  432    a->flush();
 433  433    _features = VM_Version::unknown_m;
 434  434  
 435  435    // Print the detection code.
 436  436    if (PrintAssembly) {
 437  437      ttyLocker ttyl;
 438      -    tty->print_cr("Decoding cpu-feature detection stub at " INTPTR_FORMAT " before execution:", code);
      438 +    tty->print_cr("Decoding cpu-feature detection stub at " INTPTR_FORMAT " before execution:", p2i(code));
 439  439      Disassembler::decode((u_char*)code, (u_char*)code_end, tty);
 440  440    }
 441  441  
 442  442    // Measure cache line size.
 443  443    memset(test_area, 0xFF, BUFFER_SIZE); // Fill test area with 0xFF.
 444  444    (*zero_cacheline_func_ptr)(mid_of_test_area); // Call function which executes dcbz to the middle.
 445  445    int count = 0; // count zeroed bytes
 446  446    for (int i = 0; i < BUFFER_SIZE; i++) if (test_area[i] == 0) count++;
 447  447    guarantee(is_power_of_2(count), "cache line size needs to be a power of 2");
 448  448    _measured_cache_line_size = count;
↓ open down ↓ 12 lines elided ↑ open up ↑
 461  461    if (code[feature_cntr++]) features |= cmpb_m;
 462  462    //if(code[feature_cntr++])features |= mftgpr_m;
 463  463    if (code[feature_cntr++]) features |= popcntb_m;
 464  464    if (code[feature_cntr++]) features |= popcntw_m;
 465  465    if (code[feature_cntr++]) features |= fcfids_m;
 466  466    if (code[feature_cntr++]) features |= vand_m;
 467  467  
 468  468    // Print the detection code.
 469  469    if (PrintAssembly) {
 470  470      ttyLocker ttyl;
 471      -    tty->print_cr("Decoding cpu-feature detection stub at " INTPTR_FORMAT " after execution:", code);
      471 +    tty->print_cr("Decoding cpu-feature detection stub at " INTPTR_FORMAT " after execution:", p2i(code));
 472  472      Disassembler::decode((u_char*)code, (u_char*)code_end, tty);
 473  473    }
 474  474  
 475  475    _features = features;
 476  476  }
 477  477  
 478  478  
 479  479  static int saved_features = 0;
 480  480  
 481  481  void VM_Version::allow_all() {
 482  482    saved_features = _features;
 483  483    _features      = all_features_m;
 484  484  }
 485  485  
 486  486  void VM_Version::revert() {
 487  487    _features = saved_features;
 488  488  }
    
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX