Bug 701 - inline HighPrecision Max () and Min () functions
inline HighPrecision Max () and Min () functions
Status: RESOLVED FIXED
Product: ns-3
Classification: Unclassified
Component: core
ns-3-dev
All All
: P3 enhancement
Assigned To: ns-bugs
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-10-01 11:01 UTC by Andrey Mazo
Modified: 2009-11-24 04:45 UTC (History)
2 users (show)

See Also:


Attachments
Proposed patch (1.53 KB, patch)
2009-10-01 11:02 UTC, Andrey Mazo
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Mazo 2009-10-01 11:01:22 UTC
0) Original code
 Command being timed: "build/optimized/examples/wifi-wired-bridging --nStas=4"
 User time (seconds): 403.23
 System time (seconds): 5.57
 Percent of CPU this job got: 98%
 Elapsed (wall clock) time (h:mm:ss or m:ss): 6:55.31

 Command being timed: "build/optimized/examples/mesh"
 User time (seconds): 256.20
 System time (seconds): 0.02
 Percent of CPU this job got: 99%
 Elapsed (wall clock) time (h:mm:ss or m:ss): 4:16.49

1) Modified code
 Command being timed: "build/optimized/examples/wifi-wired-bridging --nStas=4"
 User time (seconds): 389.68
 System time (seconds): 5.76
 Percent of CPU this job got: 98%
 Elapsed (wall clock) time (h:mm:ss or m:ss): 6:41.36

 Command being timed: "build/optimized/examples/mesh"
 User time (seconds): 232.67
 System time (seconds): 0.03
 Percent of CPU this job got: 99%
 Elapsed (wall clock) time (h:mm:ss or m:ss): 3:52.83


So, we're getting about 3-10% speed improvement.
Comment 1 Andrey Mazo 2009-10-01 11:02:16 UTC
Created attachment 616 [details]
Proposed patch
Comment 2 Andrey Mazo 2009-10-01 11:13:14 UTC
(In reply to comment #0)
> So, we're getting about 3-10% speed improvement.
Forgot to say, that:
1) revision -- 0ba73cdd2a43
2) compiler -- gcc-4.3.2
3) build -- static optimized
4) machine -- Intel Xeon @ 2.80GHz
5) examples are modified with attachment 608 [details] from bug 695 [1]


[1] http://www.nsnam.org/bugzilla/attachment.cgi?id=608
Comment 3 Mathieu Lacage 2009-11-24 04:45:26 UTC
changeset: bb1eea10412f