Project

General

Profile

Bug #1210

log "Wall t" column incorrect

Added by Szilárd Páll over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
mdrun
Target version:
Affected version - extra info:
4.6.1
Affected version:
Difficulty:
uncategorized
Close

Description

A recent change to the timing statistics code broke the "Wall t" column of the performance statistics table in the log, the sum of all entries adds up to double of what the total wall time is (see below).

    R E A L   C Y C L E   A N D   T I M E   A C C O U N T I N G

   Computing:         Nodes   Th.     Count  Wall t (s)     G-Cycles       %
  -----------------------------------------------------------------------------
   Neighbor search        1    1        401       0.671        1.079     4.1
   Launch GPU ops.        1    1      10001       0.589        0.946     3.6
   Force                  1    1      10001       0.065        0.105     0.4
   PME mesh               1    1      10001      11.573       18.604    70.9
   Wait GPU local         1    1      10001       0.053        0.086     0.3
   NB X/F buffer ops.     1    1      19601       0.307        0.494     1.9
   Update                 1    1      10001       0.648        1.042     4.0
   Constraints            1    1      10001       2.069        3.325    12.7
   Rest                   1                       0.172        0.554     2.1
  -----------------------------------------------------------------------------
   Total                  1                       8.160       26.235   100.0
  -----------------------------------------------------------------------------

Associated revisions

Revision 063e5658 (diff)
Added by Berk Hess over 4 years ago

corrected log file wall time breakdown

The recent commit 873b9854 doubled the wall times in the breakdown
table in the log file when not using separate PME nodes.
Note that this didn't affect cycle count or total timing prints.
Fixes #1210

Change-Id: Ia2b1d5e6c21543990465b7c7a4a8a1e687c41528

Revision 180a71a1 (diff)
Added by Berk Hess over 4 years ago

corrected log file wall time breakdown

The recent commit 873b9854 doubled the wall times in the breakdown
table in the log file when not using separate PME nodes.
Note that this didn't affect cycle count or total timing prints.
Fixes #1210

Change-Id: Ia2b1d5e6c21543990465b7c7a4a8a1e687c41528

History

#1 Updated by Berk Hess over 4 years ago

  • Status changed from New to Closed

This only affected runs without separate PME nodes.
Fix uploaded to gerrit.

Also available in: Atom PDF