X-Git-Url: https://git.lttng.org/?a=blobdiff_plain;f=lttv%2Flttv%2Ftraceset-process.c;h=5f6df65257a612ca7cc97f99515b0a7a76097cc0;hb=8284135eaaf40b6d7897c63a59a7df0b7f0cd2c7;hp=8ce68981882dd0d76c5a441ab9a845aa9043bbbc;hpb=9a366873e0ed6ecc7f23e2d3a959ecddd4f2f91d;p=lttv.git

diff --git a/lttv/lttv/traceset-process.c b/lttv/lttv/traceset-process.c
index 8ce68981..5f6df652 100644
--- a/lttv/lttv/traceset-process.c
+++ b/lttv/lttv/traceset-process.c
@@ -65,7 +65,11 @@ guint lttv_process_traceset_middle(LttvTraceset *traceset,
 	struct bt_ctf_event *bt_event;
 	
 	LttvEvent event;
-  
+	LttTime endPositionTime;
+
+	if(end_position) {
+		endPositionTime = lttv_traceset_position_get_time(end_position);
+	}
 	while(TRUE) {
 
 		if(last_ret == TRUE || ((count >= nb_events) && (nb_events != G_MAXULONG))) {
@@ -74,22 +78,26 @@ guint lttv_process_traceset_middle(LttvTraceset *traceset,
 
 		if((bt_event = bt_ctf_iter_read_event(traceset->iter)) != NULL) {
 
-			LttTime time = ltt_time_from_uint64(bt_ctf_get_timestamp_raw(bt_event));
+			LttTime time = ltt_time_from_uint64(bt_ctf_get_timestamp(bt_event));
 			if(ltt_time_compare(end, time) <= 0) {
 				break;
 			}
-			
-			currentPos = lttv_traceset_create_position(traceset);
+			/*
+			currentPos = lttv_traceset_create_current_position(traceset);
 			if(lttv_traceset_position_compare(currentPos,end_position ) == 0){
 				lttv_traceset_destroy_position(currentPos);
 				break;
 			}
 			lttv_traceset_destroy_position(currentPos);
+			*/
+			if(end_position && (ltt_time_compare(endPositionTime, time) <= 0)) {
+				break;
+			}
 			count++;
 
 			event.bt_event = bt_event;
 
-			/* Retreive the associated state */
+			/* Retrieve the associated state */
 			event.state = g_ptr_array_index(traceset->state_trace_handle_index,
 							bt_ctf_event_get_handle_id(bt_event));
 			
@@ -214,7 +222,6 @@ guint lttv_process_traceset_seek_n_forward(LttvTraceset *traceset,
                 LttvFilter *filter3,
                 gpointer data)
 {
-        struct bt_ctf_event *bt_event;
         unsigned count = 0;
         while(count < n) {
 		if(bt_iter_next(bt_ctf_get_iter(traceset->iter)) < 0) {
@@ -239,16 +246,21 @@ guint lttv_process_traceset_seek_n_backward(LttvTraceset *ts,
         guint i, count, ret;
         gint extraEvent = 0;
         guint64 initialTimeStamp, previousTimeStamp;
-        LttvTracesetPosition *initialPos, *previousPos, *currentPos;
-        
+        LttvTracesetPosition *initialPos, *previousPos, *currentPos, beginPos;
+        struct bt_iter_pos pos;
+        beginPos.bt_pos = &pos;
+        beginPos.iter = ts->iter;
+        beginPos.bt_pos->type = BT_SEEK_BEGIN;
+        beginPos.timestamp = G_MAXUINT64;
+        beginPos.cpu_id = INT_MAX;
         /*Save initial position of the traceset*/
-        initialPos = lttv_traceset_create_position (ts);
+        initialPos = lttv_traceset_create_current_position (ts);
         
         /*Get the timespan of the initial position*/
         initialTimeStamp = lttv_traceset_position_get_timestamp(initialPos);
         /* 
          * Create a position before the initial timestamp according
-         * to the ratio of nanosecond/nanosecond hopefully before the
+         * to the ratio of nanosecond/event hopefully before the
          * the desired seek position
          */
         while(1){
@@ -259,25 +271,42 @@ guint lttv_process_traceset_seek_n_backward(LttvTraceset *ts,
 			break;
                 
                 currentPos = lttv_traceset_create_time_position(ts,ltt_time_from_uint64(previousTimeStamp));
+                /*Corner case: When we are near the beginning of the trace and the previousTimeStamp is before
+                        * the beginning of the trace. We have to seek to the first event.
+                        */
+                if((lttv_traceset_position_compare(currentPos,&beginPos ) == 0)){
+                                lttv_traceset_seek_to_position(&beginPos);
+                                break;
+                        }
                 /*move traceset position */
-                lttv_traceset_seek_to_position(previousPos);
+                lttv_state_traceset_seek_position(ts, previousPos);
                 /* iterate to the initial position counting the number of event*/
                 count = 0;
                 do {
                         if((ret = lttv_traceset_position_compare(currentPos,initialPos)) == 1){       
-                                bt_iter_next(bt_ctf_get_iter(ts->iter));
-                                lttv_traceset_destroy_position(currentPos);
-                                currentPos = lttv_traceset_create_position(ts);
-                                count++;
+				if(bt_iter_next(bt_ctf_get_iter(ts->iter)) == 0) {
+					if(bt_ctf_iter_read_event(ts->iter) > 0) {
+						lttv_traceset_destroy_position(currentPos);
+						currentPos = lttv_traceset_create_current_position(ts);
+						count++;
+					} else  {
+						break;
+					}
+					
+				} else {
+
+					//No more event available
+					break;
+				}
                         }
                 }while(ret != 0);
-                lttv_traceset_destroy_position(currentPos);
+                
                 /*substract the desired number of event to the count*/
                 extraEvent = count - n;
-                if(extraEvent >= 0){
+                if (extraEvent >= 0) {
                         //if the extraEvent is over 0 go back to previousPos and 
                         //move forward the value of extraEvent times
-                        lttv_traceset_seek_to_position(previousPos);
+                        lttv_state_traceset_seek_position(ts, previousPos);
                         
                         for(i = 0 ; i < extraEvent ; i++){
                                 if(bt_iter_next(bt_ctf_get_iter(ts->iter)) < 0){
@@ -288,9 +317,11 @@ guint lttv_process_traceset_seek_n_backward(LttvTraceset *ts,
                         }
                         break; /* we successfully seeked backward */
                 }
-                else{ /* if the extraEvent is below 0 create a position before and start over*/  
-                        ratio = ratio * 16;
+                else{ 
+                        /* if the extraEvent is below 0 create a position before and start over*/  
+			ratio = ratio * 16;
                 }
+                lttv_traceset_destroy_position(currentPos);
         }
         return 0;
 }