Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
release 3.3.3
authormquinson <mquinson@48e7efb5-ca39-0410-a469-dd3cf9ba447f>
Thu, 20 Aug 2009 15:39:56 +0000 (15:39 +0000)
committermquinson <mquinson@48e7efb5-ca39-0410-a469-dd3cf9ba447f>
Thu, 20 Aug 2009 15:39:56 +0000 (15:39 +0000)
git-svn-id: svn+ssh://scm.gforge.inria.fr/svn/simgrid/simgrid/trunk@6619 48e7efb5-ca39-0410-a469-dd3cf9ba447f

ChangeLog
configure.ac

index fcce2b7..2630539 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,6 +1,35 @@
-SimGrid (3.3.3-svn) unstable; urgency=low
+SimGrid (3.3.3stable; urgency=low
 
- The "Desktop grid also matter" release.
+ The "Need for Speed" release.
+ The timings done to validate the 3.3.2 were faulty. 
+ Instead of being 5% faster, it was 15% slower (compared to 3.3.1).
+   
+ The problem was a conversion from a manually handled vector to
+   xbt_dynar_t on the critical path. 
+ xbt_dynar_foreach calls functions, inducing stack management crap.
+
+ We inlined these functions and xbt_dynar_foreach is now breath taking.
+ We also inlined xbt_swag_belong on the way.
+
+ Here are some approximate speedup measurements (on master/slaves
+  simulations lasting between 10s and 20s each):
+   3.3.1                   -> 3.3.2: -15%
+   3.3.2                   -> 3.3.3: +40%
+   3.3.1                   -> 3.3.3: +40%
+   3.3.1 with inline patch -> 3.3.3: +30%
+   
+ Our reading is that the refactoring which occurred in 3.3.2 made us
+  suffer much more from the xbt_dynar_foreach low performance, but
+  once we solved this, this refactoring proved to be very performance
+  effective.
+
+ That's a pitty that gcc cannot inline functions placed in other files
+  alone. We have to choose between:
+  - break the encapsulation (by putting private data structures and
+    accessors in headers files to help gcc)
+  - live with low performance 
+  - switch to a decent compiler such as icc (not quite possible).
 
  -- Da SimGrid team <simgrid-devel@lists.gforge.inria.fr> 
 
index a00c64e..bf3e689 100644 (file)
@@ -9,7 +9,7 @@ dnl GNU LGPL (v2.1) licence.
 ##
 
 AC_PREREQ(2.59)
-AC_INIT([simgrid],[3.3.3-svn],[simgrid-devel@lists.gforge.inria.fr])
+AC_INIT([simgrid],[3.3.3],[simgrid-devel@lists.gforge.inria.fr])
 AC_CONFIG_SRCDIR([include/gras.h])
 AC_CONFIG_HEADERS([src/gras_config.h])
 # A CI_PREREQ(2003.01.16) # We need a recent ACI when having sub-modules