]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - STEER/AliTriggerRunScalers.cxx
Fixes for Coverity warnings - M. Sitta
[u/mrichter/AliRoot.git] / STEER / AliTriggerRunScalers.cxx
index d5a9bb34164fac74f03a7104544af6e21c7cd496..a14e1009572bc99633e3c739530a35f181757850 100644 (file)
@@ -1,4 +1,4 @@
- /**************************************************************************
+/**************************************************************************
  * Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
  *                                                                        *
  * Author: The ALICE Off-line Project.                                    *
 ///////////////////////////////////////////////////////////////////////////////
 //
 // Class to define the ALICE Trigger Scalers Record per Run.
-//
+// ReadScalers(): read the txt file (rXXXX.cnt) provided by CTP and creates array of records.
+// ConsistencyCheck(): checks if the last scaler added to record is consistent, 
+// i.e. time increases and valued are not decreasing in time.
+// 
 //
 //////////////////////////////////////////////////////////////////////////////
 
+#include <stdlib.h>
 #include <Riostream.h>
-#include <TObject.h>
+
+//#include <TObject.h>
+//#include <TArrayC.h>
+//#include <TFile.h>
 #include <TClass.h>
-#include <TArrayC.h>
 #include <TSystem.h>
 #include <TObjString.h>
 #include <TObjArray.h>
-#include <TSystem.h>
-#include <TFile.h>
 
 #include "AliLog.h"  
 #include "AliTimeStamp.h"
 #include "AliTriggerScalers.h"
+#include "AliTriggerScalersESD.h"
 #include "AliTriggerScalersRecord.h"
+#include "AliTriggerScalersRecordESD.h"
 #include "AliTriggerRunScalers.h"
 
 ClassImp( AliTriggerRunScalers )
@@ -47,23 +53,87 @@ AliTriggerRunScalers::AliTriggerRunScalers():
   fRunNumber(0),
   fnClasses(0),
   fClassIndex(),                    
-  fScalersRecord()
+  fScalersRecord(),
+  fScalersRecordESD()
 {
   // Default constructor
 }
-
+//______________________________________________________________________________
+AliTriggerRunScalers::~AliTriggerRunScalers() 
+{
+ // Destructor
+ fScalersRecord.SetOwner(); 
+ fScalersRecord.Delete(); 
+ fScalersRecordESD.SetOwner(); 
+ fScalersRecordESD.Delete(); 
+}
 //_____________________________________________________________________________
 void AliTriggerRunScalers::AddTriggerScalers( AliTriggerScalersRecord* scaler ) 
 { 
-  //
-  fScalersRecord.AddLast( scaler ); 
-  fScalersRecord.Sort(); 
+  // Add scaler and check consistency
+  fScalersRecord.AddLast( scaler );
+  UInt_t* overflow[6];
+  for(Int_t i=0; i<6; i++) {
+     overflow[i] = new UInt_t[fnClasses];
+     for(Int_t j=0; j<fnClasses; j++) overflow[i][j] = 0;
+  }
+  if (AliTriggerRunScalers::ConsistencyCheck(fScalersRecord.GetEntriesFast()-1,kFALSE,overflow)){
+   AliErrorClass("Trigger counters not in the right order or decreasing!");
+  //  scaler->Print();
+  //  fScalersRecord.Sort(); 
+ }
 }
+//_____________________________________________________________________________
+
+AliTriggerRunScalers::AliTriggerRunScalers(const AliTriggerRunScalers &run) :
+ TObject(),
+ fVersion(run.fVersion),
+ fRunNumber(run.fRunNumber),
+ fnClasses(run.fnClasses),
+ fClassIndex(),                    
+ fScalersRecord(),
+ fScalersRecordESD()
+{
+// copy constructor
+  fClassIndex.Set(run.fClassIndex.GetSize());
+for (Int_t i = 0; i < run.fClassIndex.GetSize(); i++) {
+    if (run.fClassIndex[i]) fClassIndex.AddAt(run.fClassIndex[i], i);
+  }
+for (Int_t i = 0; i < run.fScalersRecord.GetEntriesFast(); i++) {
+    if (run.fScalersRecord[i]) fScalersRecord.Add(run.fScalersRecord[i]->Clone());
+  }
+for (Int_t i = 0; i < run.fScalersRecordESD.GetEntriesFast(); i++) {
+    if (run.fScalersRecordESD[i]) fScalersRecordESD.Add(run.fScalersRecordESD[i]->Clone());
+  }
 
+}
+//_____________________________________________________________________________
+AliTriggerRunScalers &AliTriggerRunScalers::operator=(const AliTriggerRunScalers& run)
+{
+// assignment operator
+if(&run == this) return *this;
+((TObject *)this)->operator=(run);
+
+fVersion = run.fVersion;
+fRunNumber = run.fRunNumber;
+fnClasses = run.fnClasses;
+
+for (Int_t i = 0; i < run.fClassIndex.GetSize(); i++) {
+    if (run.fClassIndex[i]) fClassIndex.AddAt(run.fClassIndex[i], i);
+  }
+for (Int_t i = 0; i < run.fScalersRecord.GetEntriesFast(); i++) {
+    if (run.fScalersRecord[i]) fScalersRecord.Add(run.fScalersRecord[i]->Clone());
+  }
+for (Int_t i = 0; i < run.fScalersRecordESD.GetEntriesFast(); i++) {
+    if (run.fScalersRecordESD[i]) fScalersRecordESD.Add(run.fScalersRecordESD[i]->Clone());
+  }
+return *this;
+} 
 //_____________________________________________________________________________
 AliTriggerRunScalers* AliTriggerRunScalers::ReadScalers( TString & filename )
 {
-  
+  // Read scalers from text file(.cnt) provided by CTP 
+  // for given run and convert it to root format 
   if( gSystem->AccessPathName( filename.Data() ) ) {
      AliErrorClass( Form( "file (%s) not found", filename.Data() ) );
      return NULL;
@@ -137,13 +207,14 @@ AliTriggerRunScalers* AliTriggerRunScalers::ReadScalers( TString & filename )
                             filename.Data(), strLine.Data() )); 
       return NULL;
     }
-    UInt_t orbit     = ((TObjString*)tokens->At(0))->String().Atoi();
-    UInt_t period    = ((TObjString*)tokens->At(1))->String().Atoi();
-    UInt_t seconds   = ((TObjString*)tokens->At(2))->String().Atoi();
-    UInt_t microSecs = ((TObjString*)tokens->At(3))->String().Atoi();
+
+    UInt_t orbit     = strtoul(((TObjString*)tokens->At(0))->String(), NULL, 10);
+    UInt_t period    = strtoul(((TObjString*)tokens->At(1))->String(), NULL, 10);
+    UInt_t seconds   = strtoul(((TObjString*)tokens->At(2))->String(), NULL, 10);
+    UInt_t microSecs = strtoul(((TObjString*)tokens->At(3))->String(), NULL, 10);
+
     AliTriggerScalersRecord * rec = new AliTriggerScalersRecord();
     rec->SetTimeStamp( orbit, period, seconds, microSecs );
-    
     TString strLine1;
     for (Int_t i=0; i<nclass; ++i) {
       strLine1.ReadLine(*file);
@@ -151,19 +222,24 @@ AliTriggerRunScalers* AliTriggerRunScalers::ReadScalers( TString & filename )
       TObjArray *tokens1 = strLine1.Tokenize(" \t");
       Int_t ntokens1 = tokens1->GetEntriesFast();
       if( ntokens1 != 6 ) {
-        AliErrorClass( Form( "Error reading scalers from (%s): line (%s)\n", 
-                              filename.Data(), strLine1.Data() )); 
+        AliErrorClass( Form( "Error reading scalers from (%s): line (%s)", 
+                            filename.Data(), strLine1.Data() ));
+       delete rec;
+       return rScaler;
       }
-      UInt_t LOCB = ((TObjString*)tokens1->At(0))->String().Atoi();
-      UInt_t LOCA = ((TObjString*)tokens1->At(1))->String().Atoi();
-      UInt_t L1CB = ((TObjString*)tokens1->At(2))->String().Atoi();
-      UInt_t L1CA = ((TObjString*)tokens1->At(3))->String().Atoi();
-      UInt_t L2CB = ((TObjString*)tokens1->At(4))->String().Atoi();
-      UInt_t L2CA = ((TObjString*)tokens1->At(5))->String().Atoi();
+
+      UInt_t lOCB = strtoul(((TObjString*)tokens1->At(0))->String(), NULL, 10);
+      UInt_t lOCA = strtoul(((TObjString*)tokens1->At(1))->String(), NULL, 10);
+      UInt_t l1CB = strtoul(((TObjString*)tokens1->At(2))->String(), NULL, 10);
+      UInt_t l1CA = strtoul(((TObjString*)tokens1->At(3))->String(), NULL, 10);
+      UInt_t l2CB = strtoul(((TObjString*)tokens1->At(4))->String(), NULL, 10);
+      UInt_t l2CA = strtoul(((TObjString*)tokens1->At(5))->String(), NULL, 10);
+
       rScaler->GetClass(i);
       rec->AddTriggerScalers( rScaler->GetClass(i),
-                              LOCB, LOCA, L1CB,
-                              L1CA, L2CB, L2CA );
+                              lOCB, lOCA, l1CB,
+                              l1CA, l2CB, l2CA );
+
       delete tokens1;
     } 
     rScaler->AddTriggerScalers( rec );
@@ -177,7 +253,7 @@ AliTriggerRunScalers* AliTriggerRunScalers::ReadScalers( TString & filename )
 }
   
 //_____________________________________________________________________________
-Int_t  AliTriggerRunScalers::FindNearestScalersRecord( AliTimeStamp * stamp )
+Int_t  AliTriggerRunScalers::FindNearestScalersRecord( const AliTimeStamp *stamp ) const
 {
    // Find Trigger scaler record with the closest timestamp <= "stamp"
    // using a binary search. 
@@ -185,33 +261,197 @@ Int_t  AliTriggerRunScalers::FindNearestScalersRecord( AliTimeStamp * stamp )
    // is out of range return -1
 
    Int_t   base, position=-1, last, result = 0;
-   AliTimeStamp *op2 = NULL;
+   Int_t op2 = 0;
    
-   fScalersRecord.Sort();
+   //fScalersRecord.Sort();
 
    base = 0;
    last = fScalersRecord.GetEntriesFast();
 
    while (last >= base) {
       position = (base+last) / 2;
-      cout << "pos " <<   position<< " base " <<   base << "last " <<   last << endl;
+      AliDebug(1, Form("position= %d   base= %d    last= %d  ",position,base,last));
       AliTriggerScalersRecord* rec = (AliTriggerScalersRecord*)fScalersRecord.At(position);
-      if( rec ) op2 = rec->GetTimeStamp();
-      if( op2 && (result = stamp->Compare(op2)) == 0  )
-         return position;  // exact match
-      cout << "result " <<   result << " op2 " << op2 << " rec "<< rec << endl;
+      if( rec && rec->GetTimeStamp()) op2 = 1;
+      if( op2 && (result = stamp->Compare(rec->GetTimeStamp())) == 0  )
+         return position;  // exact match 
       if (!op2 || result < 0)
          last = position-1;
       else
          base = position+1;
-      op2 = NULL;  
+      op2 = 0;  
    }
    if( (position == 0 && result < 0) || position >= fScalersRecord.GetEntriesFast() ) 
     return -1;  // out of range
    else 
     return (result < 0 ) ? position-1 : position; // nearst < stamp   
 }
+//_____________________________________________________________________________
+Int_t AliTriggerRunScalers::ConsistencyCheck(Int_t position,Bool_t correctOverflow, UInt_t** overflow)
+{
+   //Check if counters are consistent(increase). Example: lOCB(n) < lOCB(n+1) and lOCB > lOCA
+   // scalers coding 0,1,2,3,4,5=0b,0a,1b,1a,2b,2a
+   // returns: 
+   //         1 = decresing time 
+   //         2 = too big jump in scalers, looks like some readings are missing
+   //         3 = (level+1) > (level)
+   if (position == 0){
+      if(correctOverflow){
+        AliErrorClass("position=0\n");
+        return 1;
+      }else return 0; // to work correctlu in AddScalers
+   };
+   UInt_t c2[6], c1[6];
+   ULong64_t c64[6]; 
+   Bool_t increase[6];  
+   for(Int_t i=0;i<6;i++){increase[i]=0;}
+   ULong64_t const max1 = 4294967295ul;  //32bit counters overflow after 4294967295
+   ULong64_t const max2 = 1000000000ul;  //when counters overflow they seem to be decreasing. Assume decrease cannot be smaller than max2.
+
+   AliTriggerScalersRecord* scalers2 = (AliTriggerScalersRecord*)fScalersRecord.At(position);
+   AliTriggerScalersRecord* scalers1 = (AliTriggerScalersRecord*)fScalersRecord.At(position-1);
+   if (scalers2->Compare((AliTriggerScalersRecord*)fScalersRecord.At(position-1)) == -1) return 1;
+   
+   AliTriggerScalersRecordESD* recESD = 0;
+   if(correctOverflow){
+     recESD = new AliTriggerScalersRecordESD();
+     recESD->SetTimeStamp(scalers2->GetTimeStamp());
+   }
+   for( Int_t ic=0; ic<fnClasses; ++ic ){
+      TObjArray* scalersArray2 = (TObjArray*)scalers2->GetTriggerScalers();
+      AliTriggerScalers* counters2 = (AliTriggerScalers*)scalersArray2->At(ic);
+      UChar_t iclass = counters2->GetClassIndex();
+      counters2->GetAllScalers(c2);
+      TObjArray* scalersArray1 = (TObjArray*)scalers1->GetTriggerScalers();
+      AliTriggerScalers* counters1 = (AliTriggerScalers*)scalersArray1->At(ic);
+      counters1->GetAllScalers(c1);
+      for(Int_t i=0;i<5;i++){
+         if ( c2[i] >= c1[i] ) increase[i]=1;
+         else if ( c2[i] < c1[i] && (c1[i] - c2[i]) > max2) overflow[i][ic]++;
+         else return 2;
+      }
+      for(Int_t i=0;i<5;i++){
+         if ((c2[i] - c1[i]) < (c2[i+1] - c1[i+1]) && increase[i] && increase[i+1] ) {
+                 if ( ((c2[i+1] - c1[i+1]) - (c2[i] - c1[i])) < 16 ) {AliWarningClass("Trigger scaler Level[i+1] > Level[i]. Diff < 16!");}
+                 else return 3; }
+         else if ( (max1 - c1[i]+c2[i]) < (c2[i+1] - c1[i+1]) && overflow[i][ic] && increase[i+1] ) {
+                 if ( ((c2[i+1] - c1[i+1]) - (max1 - c1[i]+c2[i])) < 16 ) {AliWarningClass("Trigger scaler Level[i+1] > Level[i]. Diff < 16!");}
+                 else return 3; }
+         else if ( (c2[i] - c1[i]) < (max1 - c1[i+1] + c2[i+1]) && increase[i] && overflow[i+1][ic] ) {
+                 if ( ((max1 - c1[i+1] + c2[i+1]) - (c2[i] - c1[i])) < 16 ) {AliWarningClass("Trigger scaler Level[i+1] > Level[i]. Diff < 16!");}
+                 else return 3; }
+         else if ( (max1 - c1[i] + c2[i] ) < (max1 - c1[i+1] + c2[i+1]) && overflow[i][ic] && overflow[i+1][ic] ) {
+                 if ( ((max1 - c1[i+1] + c2[i+1]) - (max1 - c1[i] + c2[i] )) < 16 ) {AliWarningClass("Trigger scaler Level[i+1] > Level[i]. Diff < 16!");}
+                 else return 3; }
+      }
+      if(correctOverflow){ 
+        for(Int_t i=0;i<6;i++){ c64[i]=c2[i]+max1*overflow[i][ic]; }
+        AliTriggerScalersESD* s= new AliTriggerScalersESD(iclass,c64);
+        recESD->AddTriggerScalers(s);
+         }
+
+ }
+ if(correctOverflow)fScalersRecordESD.AddLast(recESD);
+ return 0;
+}
+//____________________________________________________________________________
+Int_t AliTriggerRunScalers::CorrectScalersOverflow()
+{
+ // Run over fScalersRecord, check overflow using CheckConsistency methos
+ // and save corrected result in fScalersRecordESD.
 
+ // Temporary fix for the OCDB entries written with v4-16-Release
+ // where the wrong sorting was used
+ fScalersRecord.Sort();
+ UInt_t c1[6];
+ ULong64_t c64[6];
+ AliTriggerScalersRecordESD* recESD = new AliTriggerScalersRecordESD();
+ // add 0
+ AliTriggerScalersRecord* scalers = (AliTriggerScalersRecord*)fScalersRecord.At(0);
+ for( Int_t ic=0; ic<fnClasses; ++ic ){
+    TObjArray* scalersArray = (TObjArray*)scalers->GetTriggerScalers();
+    AliTriggerScalers* counters = (AliTriggerScalers*)scalersArray->At(ic);
+    counters->GetAllScalers(c1);
+    UChar_t iclass = counters->GetClassIndex();
+    for(Int_t i=0; i<6; i++)c64[i]=c1[i];
+    AliTriggerScalersESD* s= new AliTriggerScalersESD(iclass,c64);
+    recESD->AddTriggerScalers(s);
+ }
+ fScalersRecordESD.AddLast(recESD);
+
+ UInt_t* overflow[6];
+ for(Int_t i=0; i<6; i++) {
+    overflow[i] = new UInt_t[fnClasses];
+    for(Int_t j=0; j<fnClasses; j++) overflow[i][j] = 0;
+ }
+
+ for(Int_t i=1;i<fScalersRecord.GetEntriesFast(); i++){
+  if(ConsistencyCheck(i,kTRUE,overflow)){
+    fScalersRecord.At(i)->Print();
+    fScalersRecord.At(i-1)->Print();
+    fScalersRecordESD.SetOwner(); 
+    fScalersRecordESD.Delete(); 
+    AliErrorClass("Inconsistent scalers, they will not be provided.\n");
+    return 1;
+  }
+ }
+ if(fScalersRecordESD.GetEntriesFast() != fScalersRecord.GetEntriesFast()){
+    AliErrorClass("Internal error: #scalers ESD != #scalers \n");
+    return 1;
+ }
+ return 0;
+}
+//_____________________________________________________________________________
+AliTriggerScalersESD* AliTriggerRunScalers::GetScalersForEventClass(const AliTimeStamp* stamp,const Int_t classIndex) const
+{
+ // Find scalers for event for class in fScalersRecordESD
+ // Assumes that fScalerRecord = fScalerRecordESD
+ Int_t position = FindNearestScalersRecord(stamp);
+ if ( position == -1 ) { 
+  AliErrorClass("Event AliTimeStamp out of range!");
+  return 0; 
+ }
+ // check also position=max
+ AliTriggerScalersRecordESD* scalrec1 = (AliTriggerScalersRecordESD*)fScalersRecordESD.At(position);
+ AliTriggerScalersRecordESD* scalrec2 = (AliTriggerScalersRecordESD*)fScalersRecordESD.At(position+1);
+ TObjArray* scalers1 = (TObjArray*)scalrec1->GetTriggerScalers();
+ TObjArray* scalers2 = (TObjArray*)scalrec2->GetTriggerScalers();
+
+ if(scalers1->GetEntriesFast() != fnClasses){
+  AliErrorClass("Internal error: #classes in RecordESD != fnClasses\n");
+  return 0; 
+ }
+
+ AliTriggerScalersESD *s1,*s2;
+ for ( Int_t ic=0; ic < (Int_t)fnClasses; ++ic ){
+
+      s1 = (AliTriggerScalersESD*)scalers1->At(ic);
+      s2 = (AliTriggerScalersESD*)scalers2->At(ic);
+
+      Bool_t classfound = (s1->GetClassIndex() == classIndex) && (s2->GetClassIndex() == classIndex);
+      if(classfound){
+        ULong64_t max = 4294967295ul;
+        AliTriggerScalersRecordESD* scalrec0 = (AliTriggerScalersRecordESD*)fScalersRecordESD.At(0);
+        TObjArray* scalers0 = (TObjArray*)scalrec0->GetTriggerScalers();
+        AliTriggerScalersESD *s0 = (AliTriggerScalersESD*)scalers0->At(ic);
+       ULong64_t base[6],c1[6],c2[6],cint[6];
+        ULong64_t orbit = max*(stamp->GetPeriod()) + stamp->GetOrbit();
+       s0->GetAllScalers(base);
+       s1->GetAllScalers(c1);
+       s2->GetAllScalers(c2);
+       ULong64_t orbit1 = max*(scalrec1->GetTimeStamp()->GetPeriod())+scalrec1->GetTimeStamp()->GetOrbit();
+       ULong64_t orbit2 = max*(scalrec2->GetTimeStamp()->GetPeriod())+scalrec2->GetTimeStamp()->GetOrbit();
+        for(Int_t i=0;i<6;i++){
+          Double_t slope=Double_t(c2[i]-c1[i])/Double_t(orbit2-orbit1);
+          cint[i]=ULong64_t(slope*(orbit-orbit1)) +c1[i] -base[i];
+       }
+       AliTriggerScalersESD* result = new AliTriggerScalersESD(classIndex,cint);
+        return result;
+      }
+ }
+ AliErrorClass(Form("Classindex %i not found.\n",classIndex));
+ return 0;
+}
 //_____________________________________________________________________________
 void AliTriggerRunScalers::Print( const Option_t* ) const
 {