← Back to team overview

yade-dev team mailing list archive

Re: [Branch ~yade-dev/yade/trunk] Rev 2626: 1. Clean up NewtonIntegrator following the discussion on the list; adds 1st order damping for asp...

 


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.




References