Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

shootout-spectralnorm resurection with parallelization #10704

Merged
merged 1 commit into from
Nov 28, 2013

Conversation

TeXitoi
Copy link
Contributor

@TeXitoi TeXitoi commented Nov 28, 2013

No description provided.

bors added a commit that referenced this pull request Nov 28, 2013
@bors bors closed this Nov 28, 2013
@bors bors merged commit 6bd2249 into rust-lang:master Nov 28, 2013
@TeXitoi
Copy link
Contributor Author

TeXitoi commented Nov 29, 2013

Le 2013-11-28 20:52, Daniel Micay a écrit :

Isn't using RWArc for this really slow?

(strange, I can't find this comment on github)

In fact, if you look at the code carefully (see fn mult), in parallel,
1 RWArc (v)is only used as read during a long time by multiple tasks
(100), and 1 RWArc (out) is used in write mode only to set a value
in a vector at the end of each tasks (RWArc::write concurrency is on a
very short time). As RWArc::read is almost the same thing as using
Arc, this code works quite well. I tried to use Arc in tasks for
read only data, and writing in one unique task reading messages, but it
is slower (by memory, 10% slower) and more complicated.

I didn't profile the code, but it seems that most of the time spend is
in the A function.

@TeXitoi TeXitoi deleted the spectralnorm-resurected branch December 1, 2013 18:14
flip1995 pushed a commit to flip1995/rust that referenced this pull request May 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants