FAQ

On 2013/01/03 15:29:31, dvyukov wrote:
Hello mailto:golang-dev@googlegroups.com,
I'd like you to review this change to
https://dvyukov%2540google.com%40code.google.com/p/go/
For my ray tracer stuff the behaviour is very similar to what I had with
previous patch.

Machine: amd64/linux on i5-3470 CPU.

Before (tip d0d76b7fb219):
Rendering took 2m 34.5s
Sys: 85928184
StackInuse: 102400
StackSys: 31981568
After (tip d0d76b7fb219 + issue7029044_6007.diff):
Rendering took 2m 35.2s
Sys: 55128064
StackInuse: 12885004288
StackSys: 2621440

This time the actual runtime.MemStats numbers instead of staring at
process' RES (resident memory size). Rendering time is similar (less
than 1% difference is not statistically significant). Oh and this time I
was using 50 rays per pixel instead of 100, just to make tests quicker.
Also note the anomally high StackInuse number in your patch. Is it a
miscalculation?


https://codereview.appspot.com/7029044/

Search Discussions

Discussion Posts

Previous

Follow ups

Related Discussions

Discussion Navigation
viewthread | post
posts ‹ prev | 3 of 12 | next ›
Discussion Overview
groupgolang-dev @
categoriesgo
postedJan 4, '13 at 1:17a
activeJan 4, '13 at 8:36p
posts12
users4
websitegolang.org

People

Translate

site design / logo © 2022 Grokbase