Perform accuracy analysis only on hull points
[lttv.git] / lttv / lttv / sync / event_analysis_eval.c
index 3c1b110aeeee1e7f766cd7805952b60694c90566..45bad57c57ba3d907d48151929312e97c00c0c77 100644 (file)
@@ -1,19 +1,18 @@
 /* This file is part of the Linux Trace Toolkit viewer
- * Copyright (C) 2009 Benjamin Poirier <benjamin.poirier@polymtl.ca>
+ * Copyright (C) 2009, 2010 Benjamin Poirier <benjamin.poirier@polymtl.ca>
  *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License Version 2 as
- * published by the Free Software Foundation;
+ * This program is free software: you can redistribute it and/or modify it
+ * under the terms of the GNU Lesser General Public License as published by
+ * the Free Software Foundation, either version 2.1 of the License, or (at
+ * your option) any later version.
  *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Lesser General Public
+ * License for more details.
  *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston,
- * MA 02111-1307, USA.
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 #define _GNU_SOURCE
@@ -78,7 +77,6 @@ static void writeAnalysisTraceTraceForePlotsEval(SyncState* const syncState,
        const unsigned int i, const unsigned int j);
 
 // Functions specific to this module
-static void registerAnalysisEval() __attribute__((constructor (102)));
 static guint ghfRttKeyHash(gconstpointer key);
 static gboolean gefRttKeyEqual(gconstpointer a, gconstpointer b);
 static void gdnDestroyRttKey(gpointer data);
@@ -111,6 +109,7 @@ static void writeHistogram(FILE* graphsStream, const struct RttKey* rttKey,
 static void updateBounds(Bounds** const bounds, Event* const e1, Event* const
        e2);
 
+static void finalizeAnalysisEvalLP(SyncState* const syncState);
 // The next group of functions is only needed when computing synchronization
 // accuracy.
 #ifdef HAVE_LIBGLPK
@@ -122,6 +121,8 @@ static void calculateCompleteFactors(glp_prob* const lp, FactorsCHull*
        factors);
 static FactorsCHull** createAllFactors(const unsigned int traceNb);
 static inline void finalizeAnalysisEvalLP(SyncState* const syncState);
+static void gfAddAbsiscaToArray(gpointer data, gpointer user_data);
+static gint gcfCompareDouble(gconstpointer a, gconstpointer b);
 #else
 static void finalizeAnalysisEvalLP(SyncState* const syncState);
 #endif
@@ -150,7 +151,6 @@ static AnalysisModule analysisModuleEval= {
 static ModuleOption optionEvalRttFile= {
        .longName= "eval-rtt-file",
        .hasArg= REQUIRED_ARG,
-       {.arg= NULL},
        .optionHelp= "specify the file containing RTT information",
        .argHelp= "FILE",
 };
@@ -159,7 +159,7 @@ static ModuleOption optionEvalRttFile= {
 /*
  * Analysis module registering function
  */
-static void registerAnalysisEval()
+void registerAnalysisEval()
 {
        binBase= exp10(6. / (BIN_NB - 3));
 
@@ -210,7 +210,8 @@ static void initAnalysisEval(SyncState* const syncState)
        if (syncState->stats)
        {
                analysisData->stats= calloc(1, sizeof(AnalysisStatsEval));
-               analysisData->stats->broadcastDiffSum= 0.;
+               analysisData->stats->broadcastRangeMin= INFINITY;
+               analysisData->stats->broadcastRangeMax= -INFINITY;
 
                analysisData->stats->messageStats= malloc(syncState->traceNb *
                        sizeof(MessageStats*));
@@ -309,7 +310,7 @@ static AnalysisHistogramEval* constructAnalysisHistogramEval(const char* const
        convertIP(saddr, rttKey->saddr);
        convertIP(daddr, rttKey->daddr);
 
-       cwd= changeToGraphDir(graphsDir);
+       cwd= changeToGraphsDir(graphsDir);
 
        for (i= 0; i < sizeof(loopValues) / sizeof(*loopValues); i++)
        {
@@ -532,7 +533,7 @@ static void writeHistogram(FILE* graphsStream, const struct RttKey* rttKey,
  */
 static void destroyAnalysisEval(SyncState* const syncState)
 {
-       unsigned int i, j;
+       unsigned int i;
        AnalysisDataEval* analysisData;
 
        analysisData= (AnalysisDataEval*) syncState->analysisData;
@@ -582,10 +583,12 @@ static void destroyAnalysisEval(SyncState* const syncState)
 #ifdef HAVE_LIBGLPK
                for (i= 0; i < syncState->traceNb; i++)
                {
+                       unsigned int j;
+
                        for (j= 0; j < i; j++)
                        {
                                // There seems to be a memory leak in glpk, valgrind reports a
-                               // loss even if the problem is deleted
+                               // loss (reachable) even if the problem is deleted
                                glp_delete_prob(graphs->lps[i][j]);
                        }
                        free(graphs->lps[i]);
@@ -625,8 +628,7 @@ static void analyzeMessageEval(SyncState* const syncState, Message* const
        message)
 {
        AnalysisDataEval* analysisData= syncState->analysisData;
-       MessageStats* messageStats=
-               &analysisData->stats->messageStats[message->outE->traceNum][message->inE->traceNum];
+       MessageStats* messageStats;
        double* rtt;
        double tt;
        struct RttKey rttKey;
@@ -635,6 +637,8 @@ static void analyzeMessageEval(SyncState* const syncState, Message* const
 
        if (syncState->stats)
        {
+               messageStats=
+                       &analysisData->stats->messageStats[message->outE->traceNum][message->inE->traceNum];
                messageStats->total++;
        }
 
@@ -825,7 +829,40 @@ static void analyzeBroadcastEval(SyncState* const syncState, Broadcast* const
                        g_queue_get_length(broadcast->events), 2.);
                if (y > 0)
                {
-                       analysisData->stats->broadcastDiffSum+= sqrt(y);
+                       analysisData->stats->broadcastStdevSum+= sqrt(y);
+               }
+
+               if (syncState->traceNb == 2 && g_queue_get_length(broadcast->events)
+                       == 2)
+               {
+                       Event* e0, * e1;
+                       double dd;
+
+                       e0= g_queue_peek_head(broadcast->events);
+                       e1= g_queue_peek_tail(broadcast->events);
+                       if (e0->traceNum > e1->traceNum)
+                       {
+                               Event* tmp;
+
+                               tmp= e0;
+                               e0= e1;
+                               e1= tmp;
+                       }
+
+                       dd= wallTimeSub(&e1->wallTime, &e0->wallTime);
+
+                       analysisData->stats->broadcastPairNb++;
+                       if (dd < analysisData->stats->broadcastRangeMin)
+                       {
+                               analysisData->stats->broadcastRangeMin= dd;
+                       }
+                       if (dd > analysisData->stats->broadcastRangeMax)
+                       {
+                               analysisData->stats->broadcastRangeMax= dd;
+                       }
+
+                       analysisData->stats->broadcastSum+= dd;
+                       analysisData->stats->broadcastSumSquares+= pow(dd, 2);
                }
        }
 
@@ -925,11 +962,27 @@ static void printAnalysisStatsEval(SyncState* const syncState)
        printf("Synchronization evaluation analysis stats:\n");
        if (analysisData->stats->broadcastNb)
        {
-               printf("\tsum of broadcast differential delays: %g\n",
-                       analysisData->stats->broadcastDiffSum);
-               printf("\taverage broadcast differential delay: %g\n",
-                       analysisData->stats->broadcastDiffSum /
+               printf("\tBroadcast differential delay:\n");
+               printf("\t\tsum of standard deviations: %g\n",
+                       analysisData->stats->broadcastStdevSum);
+               printf("\t\taverage standard deviation: %g\n",
+                       analysisData->stats->broadcastStdevSum /
                        analysisData->stats->broadcastNb);
+
+               if (syncState->traceNb == 2)
+               {
+                       printf("\t\tdifferential delay range: [ %g .. %g ]\n",
+                               analysisData->stats->broadcastRangeMin,
+                               analysisData->stats->broadcastRangeMax);
+                       printf("\t\tdifferential delay average: %g\n",
+                               analysisData->stats->broadcastSum /
+                               analysisData->stats->broadcastPairNb);
+                       printf("\t\tdifferential delay standard deviation: %g\n",
+                               sqrt(analysisData->stats->broadcastSumSquares /
+                                       analysisData->stats->broadcastPairNb -
+                                       pow(analysisData->stats->broadcastSum /
+                                               analysisData->stats->broadcastPairNb, 2)));
+               }
        }
 
        printf("\tIndividual evaluation:\n"
@@ -953,13 +1006,13 @@ static void printAnalysisStatsEval(SyncState* const syncState)
                                        &analysisData->stats->messageStats[loopValues[k].t1][loopValues[k].t2];
 
                                printf("\t\t%3d - %-3d   ", loopValues[k].t1, loopValues[k].t2);
-                               printf("%u (%u%%)%n", messageStats->inversionNb, (unsigned
-                                               int) ceil((double) messageStats->inversionNb /
-                                               messageStats->total * 100), &charNb);
+                               printf("%u (%.2f%%)%n", messageStats->inversionNb, (double)
+                                       messageStats->inversionNb / messageStats->total * 100,
+                                       &charNb);
                                printf("%*s", 17 - charNb > 0 ? 17 - charNb + 1: 1, " ");
-                               printf("%u (%u%%)%n", messageStats->tooFastNb, (unsigned int)
-                                       ceil((double) messageStats->tooFastNb /
-                                               messageStats->total * 100), &charNb);
+                               printf("%u (%.2f%%)%n", messageStats->tooFastNb, (double)
+                                       messageStats->tooFastNb / messageStats->total * 100,
+                                       &charNb);
                                printf("%*s%-10u   %u\n", 17 - charNb > 0 ? 17 - charNb + 1:
                                        1, " ", messageStats->noRTTInfoNb, messageStats->total);
 
@@ -972,11 +1025,11 @@ static void printAnalysisStatsEval(SyncState* const syncState)
        }
 
        printf("\t\t  total     ");
-       printf("%u (%u%%)%n", totInversion, (unsigned int) ceil((double)
-                       totInversion / totTotal * 100), &charNb);
+       printf("%u (%.2f%%)%n", totInversion, (double) totInversion / totTotal *
+               100, &charNb);
        printf("%*s", 17 - charNb > 0 ? 17 - charNb + 1: 1, " ");
-       printf("%u (%u%%)%n", totTooFast, (unsigned int) ceil((double) totTooFast
-                       / totTotal * 100), &charNb);
+       printf("%u (%.2f%%)%n", totTooFast, (double) totTooFast / totTotal * 100,
+               &charNb);
        printf("%*s%-10u   %u\n", 17 - charNb > 0 ? 17 - charNb + 1: 1, " ",
                totNoInfo, totTotal);
 
@@ -985,6 +1038,7 @@ static void printAnalysisStatsEval(SyncState* const syncState)
        g_hash_table_foreach(analysisData->stats->exchangeRtt,
                &ghfPrintExchangeRtt, analysisData->rttInfo);
 
+#ifdef HAVE_LIBGLPK
        printf("\tConvex hull factors comparisons:\n"
                "\t\tTrace pair  Factors type  Differences (lp - chull)\n"
                "\t\t                          a0                    a1\n"
@@ -1021,6 +1075,7 @@ static void printAnalysisStatsEval(SyncState* const syncState)
                        }
                }
        }
+#endif
 }
 
 
@@ -1536,7 +1591,10 @@ static glp_prob* lpCreateProblem(GQueue* const lowerHull, GQueue* const
 
        // Create the LP problem
        glp_term_out(GLP_OFF);
-       glp_add_rows(lp, hullPointNb);
+       if (hullPointNb > 0)
+       {
+               glp_add_rows(lp, hullPointNb);
+       }
        glp_add_cols(lp, 2);
 
        glp_set_col_name(lp, 1, "a0");
@@ -1717,6 +1775,52 @@ static FactorsCHull** createAllFactors(const unsigned int traceNb)
 
        return factorsArray;
 }
+
+
+/*
+ * A GFunc for g_queue_foreach()
+ *
+ * Args:
+ *   data          Point*, a convex hull point
+ *   user_data     GArray*, an array of convex hull point absisca values, as
+ *                 double
+ */
+static void gfAddAbsiscaToArray(gpointer data, gpointer user_data)
+{
+       Point* p= data;
+       GArray* a= user_data;
+       double v= p->x;
+
+       g_array_append_val(a, v);
+}
+
+
+/*
+ * A GCompareFunc for g_array_sort()
+ *
+ * Args:
+ *   a, b          double*, absisca values
+ *
+ * Returns:
+ *   "returns less than zero for first arg is less than second arg, zero for
+ *   equal, greater zero if first arg is greater than second arg"
+ *   - the great glib documentation
+ */
+static gint gcfCompareDouble(gconstpointer a, gconstpointer b)
+{
+       if (*(double*) a < *(double*) b)
+       {
+               return -1;
+       }
+       else if (*(double*) a > *(double*) b)
+       {
+               return 1;
+       }
+       else
+       {
+               return 0;
+       }
+}
 #endif
 
 
@@ -1724,37 +1828,36 @@ static FactorsCHull** createAllFactors(const unsigned int traceNb)
  * Compute synchronization factors using a linear programming approach.
  * Compute the factors using analysis_chull. Compare the two.
  *
- * There are two definitions of this function. The empty one is used when the
- * solver library, glpk, is not available at build time. In that case, nothing
- * is actually produced.
+ * When the solver library, glpk, is not available at build time, only compute
+ * the factors using analysis_chull. This is to make sure that module runs its
+ * finalize function so that its graph functions can be called later.
  *
  * Args:
  *   syncState:    container for synchronization data
  */
-#ifndef HAVE_LIBGLPK
-static inline void finalizeAnalysisEvalLP(SyncState* const syncState)
-{
-}
-#else
 static void finalizeAnalysisEvalLP(SyncState* const syncState)
 {
-       unsigned int i, j;
        AnalysisDataEval* analysisData= syncState->analysisData;
+#ifdef HAVE_LIBGLPK
+       unsigned int i, j;
        AnalysisDataCHull* chAnalysisData= analysisData->chullSS->analysisData;
-       FactorsCHull** lpFactorsArray= createAllFactors(syncState->traceNb);
-       FactorsCHull* lpFactors;
+       FactorsCHull** lpFactorsArray;
 
        if (!syncState->stats && !syncState->graphsStream)
        {
                return;
        }
 
+       /* Because of matching_distributor, this analysis may be called twice.
+        * Only run it once */
        if ((syncState->graphsStream && analysisData->graphs->lps != NULL) ||
                (syncState->stats && analysisData->stats->chFactorsArray != NULL))
        {
                return;
        }
 
+       lpFactorsArray= createAllFactors(syncState->traceNb);
+
        if (syncState->stats)
        {
                analysisData->stats->chFactorsArray=
@@ -1794,15 +1897,14 @@ static void finalizeAnalysisEvalLP(SyncState* const syncState)
                        else
                        {
                                glp_delete_prob(lp);
-                               destroyFactorsCHull(lpFactors);
                        }
                }
        }
+#endif
 
        g_array_free(analysisData->chullSS->analysisModule->finalizeAnalysis(analysisData->chullSS),
                TRUE);
 }
-#endif
 
 
 /*
@@ -1836,16 +1938,13 @@ static void writeAnalysisTraceTimeBackPlotsEval(SyncState* const syncState,
                char* cwd;
                char fileName[40];
                FILE* fp;
-               double* xValues;
-               unsigned int xBegin, xEnd;
-               double interval;
-               const unsigned int graphPointNb= 1000;
+               GArray* xValues;
 
                // Open the data file
                snprintf(fileName, 40, "analysis_eval_accuracy-%03u_and_%03u.data", i, j);
                fileName[sizeof(fileName) - 1]= '\0';
 
-               cwd= changeToGraphDir(syncState->graphsDir);
+               cwd= changeToGraphsDir(syncState->graphsDir);
 
                if ((fp= fopen(fileName, "w")) == NULL)
                {
@@ -1861,51 +1960,27 @@ static void writeAnalysisTraceTimeBackPlotsEval(SyncState* const syncState,
                free(cwd);
 
                // Build the list of absisca values for the points in the accuracy graph
-               g_assert_cmpuint(graphPointNb, >=, 4);
-               xValues= malloc(graphPointNb * sizeof(double));
-               xValues[0]= graphs->bounds[j][i].min;
-               xValues[graphPointNb - 1]= graphs->bounds[j][i].max;
-               xValues[1]= MIN(((Point*) g_queue_peek_head(hullArray[i][j]))->x,
-                       ((Point*) g_queue_peek_head(hullArray[j][i]))->x);
-               xValues[graphPointNb - 2]= MAX(((Point*)
-                               g_queue_peek_tail(hullArray[i][j]))->x, ((Point*)
-                               g_queue_peek_tail(hullArray[j][i]))->x);
-
-               if (xValues[0] == xValues[1])
-               {
-                       xBegin= 0;
-               }
-               else
-               {
-                       xBegin= 1;
-               }
-               if (xValues[graphPointNb - 2] == xValues[graphPointNb - 1])
-               {
-                       xEnd= graphPointNb - 1;
-               }
-               else
-               {
-                       xEnd= graphPointNb - 2;
-               }
-               interval= (xValues[xEnd] - xValues[xBegin]) / (graphPointNb - 1);
+               xValues= g_array_sized_new(FALSE, FALSE, sizeof(double),
+                       g_queue_get_length(hullArray[i][j]) +
+                       g_queue_get_length(hullArray[j][i]));
 
-               for (it= xBegin; it <= xEnd; it++)
-               {
-                       xValues[it]= xValues[xBegin] + interval * (it - xBegin);
-               }
+               g_queue_foreach(hullArray[i][j], &gfAddAbsiscaToArray, xValues);
+               g_queue_foreach(hullArray[j][i], &gfAddAbsiscaToArray, xValues);
+
+               g_array_sort(xValues, &gcfCompareDouble);
 
                /* For each absisca value and each optimisation direction, solve the LP
                 * and write a line in the data file */
-               for (it= 0; it < graphPointNb; it++)
+               for (it= 0; it < xValues->len; it++)
                {
                        unsigned int it2;
                        int directions[]= {GLP_MIN, GLP_MAX};
-
                        glp_set_obj_coef(lp, 1, 1.);
-                       glp_set_obj_coef(lp, 2, xValues[it]);
+                       glp_set_obj_coef(lp, 2, g_array_index(xValues, double, it));
 
-                       fprintf(fp, "%25.9f %25.9f", xValues[it], lpFactors->approx->offset
-                               + lpFactors->approx->drift * xValues[it]);
+                       fprintf(fp, "%25.9f %25.9f", g_array_index(xValues, double, it),
+                               lpFactors->approx->offset + lpFactors->approx->drift *
+                               g_array_index(xValues, double, it));
                        for (it2= 0; it2 < sizeof(directions) / sizeof(*directions); it2++)
                        {
                                int status;
@@ -1920,7 +1995,7 @@ static void writeAnalysisTraceTimeBackPlotsEval(SyncState* const syncState,
                        fprintf(fp, "\n");
                }
 
-               free(xValues);
+               g_array_free(xValues, TRUE);
                fclose(fp);
 
                fprintf(syncState->graphsStream,
This page took 0.029155 seconds and 4 git commands to generate.