It seems to me that this kind of loop parallelizes badly since one
thread will access a member directly while another one will be updating
it from inside the clump inner loop. So 1 loop would be better than 2.
I think that is not relevant, since 1. yade parallellizes very badly in general (seriously), 2. the other loop would skip that body immediately; since openmp distributes load in chunks, then each thread usually works on id-wise distant bodies, while clumps have ids typically adjacent (with O.bodies.appendClumped).

Anyway, I factored away that function so that there is just one loop now.

Cheers, v.


_______________________________________________
Mailing list: https://launchpad.net/~yade-dev
Post to     : yade-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~yade-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to