-
Notifications
You must be signed in to change notification settings - Fork 859
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #285 from bosilca/master
Reenable high accuracy timers
- Loading branch information
Showing
10 changed files
with
109 additions
and
90 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
8cae899
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bosilca it looks like this commit is causing "ILLEGAL INSTRUCTION" failure at start.
8cae899
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FWIW: it seems to be working okay on my cluster (x86, CentOS7).
@elenash can you provide further info on the environment and failure?
8cae899
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rhc54 @bosilca Linux version 2.6.32-358.23.2.el6.x86_64
no output at all at start, just Illegal instruction (core dumped)
8cae899
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
8cae899
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bosilca
gcc version 4.4.7 20120313 (Red Hat 4.4.7-3) (GCC)
It looks you're right, see the stack trace:
Program received signal SIGILL, Illegal instruction.
opal_sys_timer_get_cycles () at ../../opal/include/opal/sys/amd64/timer.h:46
46 asm volatile ("rdtscp\n\t"
Missing separate debuginfos, use: debuginfo-install glibc-2.12-1.107.el6.x86_64 libpciaccess-0.13.1-2.el6.x86_64 numactl-2.0.7-6.el6.x86_64
(gdb) bt
#0 opal_sys_timer_get_cycles () at ../../opal/include/opal/sys/amd64/timer.h:46
#1 0x00007ffff7a0cd93 in opal_timer_base_get_usec () at ../../opal/mca/timer/linux/timer_linux.h:43
#2 0x00007ffff7a0d12c in opal_progress_set_event_poll_rate (polltime=10000) at ../../opal/runtime/opal_progress.c:271
#3 0x00007ffff7a0cddd in opal_progress_init () at ../../opal/runtime/opal_progress.c:98
#4 0x00007ffff7a0ddd1 in opal_init (pargc=0x7fffffffde1c, pargv=0x7fffffffde10) at ../../opal/runtime/opal_init.c:461
#5 0x00000000004044f5 in orterun (argc=12, argv=0x7fffffffe1f8) at ../../../../orte/tools/orterun/orterun.c:687
#6 0x0000000000403a74 in main (argc=12, argv=0x7fffffffe1f8) at ../../../../orte/tools/orterun/main.c:13
8cae899
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bosilca
Dump of assembler code for function opal_sys_timer_get_cycles:
0x00007ffff7a0cd62 <+0>: push %rbp
0x00007ffff7a0cd63 <+1>: mov %rsp,%rbp
0x00007ffff7a0cd66 <+4>: push %rbx
=> 0x00007ffff7a0cd67 <+5>: rdtscp
0x00007ffff7a0cd6a <+8>: mov %edx,%edi
0x00007ffff7a0cd6c <+10>: mov %eax,%esi
0x00007ffff7a0cd6e <+12>: cpuid
0x00007ffff7a0cd70 <+14>: mov %edi,-0x10(%rbp)
0x00007ffff7a0cd73 <+17>: mov %esi,-0xc(%rbp)
0x00007ffff7a0cd76 <+20>: mov -0x10(%rbp),%eax
0x00007ffff7a0cd79 <+23>: mov -0xc(%rbp),%edx
0x00007ffff7a0cd7c <+26>: shl $0x20,%rdx
0x00007ffff7a0cd80 <+30>: or %rdx,%rax
0x00007ffff7a0cd83 <+33>: pop %rbx
0x00007ffff7a0cd84 <+34>: leaveq
0x00007ffff7a0cd85 <+35>: retq
8cae899
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
8cae899
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.