yade-dev team mailing list archive

[Jerome Duriez <Jerome.Duriez@xxxxxxxxxxx>]

So I finally removed all parallel loops from capillary Law2 : https://github.com/yade/trunk/commit/4fc1e9a023f03d7678a59908039c17b3f8f4c364

Thanks a lot Anton for you remark, this is very useful for me to improve my OpenMP skills. Here I took the shortest path since this code is, as we said, going to be deprecated.

Jerome


________________________________
From: Anton Gladky [gladky.anton@xxxxxxxxx]
Sent: December 4, 2014 11:49 PM
To: Jerome Duriez
Cc: yade-dev@xxxxxxxxxxxxxxxxxxx
Subject: Re: [Yade-dev] [Branch ~yade-pkg/yade/git-trunk] Rev 3486: Parallelization of interaction loops in Law2_..._Capillarity (which is in fact not a LawFunctor h...

Hi guys,

sorry for not being active these days. BodiesMenisciiList is definitely not a
thread-safe container. You can have a look at implementation of addIntrs,
which is thread-safe. In other words you need to to use OpenMPVector
instead of Vector.

Cheers

[1] https://github.com/yade/trunk/blob/master/core/Scene.hpp#L66

Anton