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 [[email protected]] Sent: December 4, 2014 11:49 PM To: Jerome Duriez Cc: [email protected] 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
_______________________________________________ Mailing list: https://launchpad.net/~yade-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~yade-dev More help : https://help.launchpad.net/ListHelp
