X-Git-Url: http://git.liburcu.org/?a=blobdiff_plain;f=lttv%2Flttv%2Ftraceset-process.c;h=ef9d80a8a5c310cdc50cdaa8b08e9d8c5dfc44d7;hb=b9ce0bad7daf7c0a2333c91fdb1e35d602afe17f;hp=b49542f9fb7aea400464c472fbb562f6931fdc49;hpb=88bf15f0f2a239186b5cf6fcf8711786907ca483;p=lttv.git

diff --git a/lttv/lttv/traceset-process.c b/lttv/lttv/traceset-process.c
index b49542f9..ef9d80a8 100644
--- a/lttv/lttv/traceset-process.c
+++ b/lttv/lttv/traceset-process.c
@@ -13,8 +13,8 @@
  *
  * 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.
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, 
+ * MA 02110-1301, USA.
  */
 #ifdef HAVE_CONFIG_H
 #include <config.h>
@@ -60,12 +60,16 @@ guint lttv_process_traceset_middle(LttvTraceset *traceset,
 {
 	unsigned count = 0;
 	gint last_ret = 0;
-	LttvTracesetPosition *currentPos;
         
 	struct bt_ctf_event *bt_event;
 	
 	LttvEvent event;
-  
+	LttTime endPositionTime;
+
+	//TODO ybrosseau 2013-10-17: Compare with end_position directly when its possible
+	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,19 @@ 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_current_position(traceset);
-			if(lttv_traceset_position_compare(currentPos,end_position ) == 0){
-				lttv_traceset_destroy_position(currentPos);
+			//TODO ybrosseau 2013-10-17: Compare with end_position directly when its possible
+			if(end_position && (ltt_time_compare(endPositionTime, time) <= 0)) {
 				break;
 			}
-			lttv_traceset_destroy_position(currentPos);
 			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));
 			
@@ -243,6 +244,8 @@ guint lttv_process_traceset_seek_n_backward(LttvTraceset *ts,
         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_current_position (ts);
         
@@ -274,10 +277,20 @@ guint lttv_process_traceset_seek_n_backward(LttvTraceset *ts,
                 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_current_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);