code review 4438041: test/bench: parallel matrix multiplication benchmark

test/bench: parallel matrix multiplication benchmark

Includes naive version, recursive version,
parallel recursive version that uses channels for synchronization,
and parallel recursive version that uses shared mem for synchronization (racy).
Stresses scheduler, channels and GC.

[dvyukov, 2011-04-15 10:07:39]

Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com),

I'd like you to review this change to
https://go.googlecode.com/hg/

[r2, 2011-04-15 15:19:33]

That directory contains only responses to the "computer language benchmarks
game".  I'm not sure I want to start adding outside programs, but I'm also not
sure I should care.  Please discuss, after the next weekly.

-rob

[dvyukov, 2011-04-17 08:50:17]

On 2011/04/15 15:19:33, r2 wrote:
> That directory contains only responses to the "computer language benchmarks
> game".  I'm not sure I want to start adding outside programs, but I'm also not
> sure I should care.  Please discuss, after the next weekly.

Hi Rob,

What is "next weekly"? What should I do? Move it to another folder? I am
planning to add more similar benchmarks (for channels, condition variables,
semaphores).

[peterGo, 2011-04-17 12:04:23]

On 2011/04/17 08:50:17, dvyukov wrote:
Dmitriy,

> What is "next weekly"?

Changes to the Release Process
http://groups.google.com/group/golang-dev/browse_thread/thread/620d31f727fd3869

Aiming for "release"
http://groups.google.com/group/golang-dev/browse_thread/thread/2167930955919123

Peter

[rsc1, 2011-04-17 14:28:31]

> What is "next weekly"? What should I do? Move it to another folder? I am
> planning to add more similar benchmarks (for channels, condition
> variables, semaphores).

Microbenchmarks typically belong next to the code
they are testing.  So these would belong in runtime
or sync's tests.  The matrix multiply could go there too
for that matter, if it's meant as a scheduling benchmark.

See the documentation for the testing package for
more about benchmarks.

Russ

[dvyukov, 2011-04-18 06:11:18]

Peter, Russ,

Thank you. So, I will put it into /src/pkg/runtime, and adopt it for usage with
gotest.

[dvyukov, 2011-06-28 10:19:36]

*** Abandoned ***