X-Git-Url: http://git.lttng.org./?a=blobdiff_plain;f=ltt%2Fbranches%2Fpoly%2Flttv%2Fmodules%2Fgui%2Fdetailedevents%2Fevents.c;h=690588f1d9d3794b6a9077d801c4f77a898c14b0;hb=203eb6f75ad579321e4fc8dc308eb0b2d66d86d9;hp=49186d5aa4391de5f089f95544617219f5365f19;hpb=6206a9930806589f81625c427d23ae013acf2cb8;p=lttv.git diff --git a/ltt/branches/poly/lttv/modules/gui/detailedevents/events.c b/ltt/branches/poly/lttv/modules/gui/detailedevents/events.c index 49186d5a..690588f1 100644 --- a/ltt/branches/poly/lttv/modules/gui/detailedevents/events.c +++ b/ltt/branches/poly/lttv/modules/gui/detailedevents/events.c @@ -63,6 +63,7 @@ #include #include #include +#include #include "hGuiEventsInsert.xpm" @@ -124,12 +125,16 @@ typedef struct _EventViewerData { gint num_visible_events; LttvTracesetContextPosition *currently_selected_position; + gboolean update_cursor; /* Speed optimisation : do not update cursor when + unnecessary */ LttvTracesetContextPosition *first_event; /* Time of the first event shown */ LttvTracesetContextPosition *last_event; /* Time of the first event shown */ LttvFilter *main_win_filter; + gint background_info_waiting; + } EventViewerData ; /** hook functions for update time interval, current time ... */ @@ -139,6 +144,8 @@ gboolean update_current_position(void * hook_data, void * call_data); gboolean traceset_changed(void * hook_data, void * call_data); gboolean filter_changed(void * hook_data, void * call_data); +static void request_background_data(EventViewerData *event_viewer_data); + //! Event Viewer's constructor hook GtkWidget *h_gui_events(Tab *tab); //! Event Viewer's constructor @@ -167,6 +174,7 @@ static void tree_v_cursor_changed_cb (GtkWidget *widget, gpointer data); static void tree_v_move_cursor_cb (GtkWidget *widget, GtkMovementStep arg1, gint arg2, gpointer data); +static gint redraw_notify(void *hook_data, void *call_data); static void get_events(double time, EventViewerData *event_viewer_data); @@ -240,6 +248,9 @@ gui_events(Tab *tab) traceset_changed,event_viewer_data); lttvwindow_register_filter_notify(tab, filter_changed, event_viewer_data); + lttvwindow_register_redraw_notify(tab, + redraw_notify, event_viewer_data); + event_viewer_data->scroll_win = gtk_scrolled_window_new (NULL, NULL); gtk_widget_show (event_viewer_data->scroll_win); @@ -256,6 +267,8 @@ gui_events(Tab *tab) event_viewer_data->main_win_filter = NULL; + event_viewer_data->update_cursor = TRUE; + /* Create a model for storing the data list */ event_viewer_data->store_m = gtk_list_store_new ( N_COLUMNS, /* Total number of columns */ @@ -421,6 +434,8 @@ gui_events(Tab *tab) gtk_box_pack_start(GTK_BOX(event_viewer_data->hbox_v), event_viewer_data->scroll_win, TRUE, TRUE, 0); + gtk_container_set_border_width(GTK_CONTAINER(event_viewer_data->hbox_v), 1); + /* Create vertical scrollbar and pack it */ event_viewer_data->vscroll_vc = gtk_vscrollbar_new(NULL); gtk_range_set_update_policy (GTK_RANGE(event_viewer_data->vscroll_vc), @@ -479,12 +494,89 @@ gui_events(Tab *tab) event_viewer_data, (GDestroyNotify)gui_events_free); + event_viewer_data->background_info_waiting = 0; + + request_background_data(event_viewer_data); + return event_viewer_data; } +static gint background_ready(void *hook_data, void *call_data) +{ + EventViewerData *event_viewer_data = (EventViewerData *)hook_data; + LttvTrace *trace = (LttvTrace*)call_data; + + event_viewer_data->background_info_waiting--; + + if(event_viewer_data->background_info_waiting == 0) { + g_message("event viewer : background computation data ready."); + + redraw_notify(event_viewer_data, NULL); + } + + return 0; +} + + +static void request_background_data(EventViewerData *event_viewer_data) +{ + LttvTracesetContext * tsc = + lttvwindow_get_traceset_context(event_viewer_data->tab); + gint num_traces = lttv_traceset_number(tsc->ts); + gint i; + LttvTrace *trace; + + LttvHooks *background_ready_hook = + lttv_hooks_new(); + lttv_hooks_add(background_ready_hook, background_ready, event_viewer_data, + LTTV_PRIO_DEFAULT); + event_viewer_data->background_info_waiting = 0; + + for(i=0;its, i); + + if(lttvwindowtraces_get_ready(g_quark_from_string("state"),trace)==FALSE) { + + if(lttvwindowtraces_get_in_progress(g_quark_from_string("state"), + trace) == FALSE) { + /* We first remove requests that could have been done for the same + * information. Happens when two viewers ask for it before servicing + * starts. + */ + lttvwindowtraces_background_request_remove(trace, "state"); + lttvwindowtraces_background_request_queue(trace, + "state"); + lttvwindowtraces_background_notify_queue(event_viewer_data, + trace, + ltt_time_infinite, + NULL, + background_ready_hook); + event_viewer_data->background_info_waiting++; + } else { /* in progress */ + + lttvwindowtraces_background_notify_current(event_viewer_data, + trace, + ltt_time_infinite, + NULL, + background_ready_hook); + event_viewer_data->background_info_waiting++; + } + } else { + /* Data ready. Be its nature, this viewer doesn't need to have + * its data ready hook called htere, because a background + * request is always linked with a redraw. + */ + } + + } + + lttv_hooks_destroy(background_ready_hook); + +} + static gboolean header_size_allocate(GtkWidget *widget, GtkAllocation *allocation, @@ -601,8 +693,10 @@ void tree_v_move_cursor_cb (GtkWidget *widget, /* Must get down one event and select the last one */ gtk_tree_selection_unselect_all(gtk_tree_view_get_selection( GTK_TREE_VIEW(event_viewer_data->tree_v))); + event_viewer_data->update_cursor = FALSE; gtk_adjustment_set_value(event_viewer_data->vadjust_c, gtk_adjustment_get_value(event_viewer_data->vadjust_c) + 1); + event_viewer_data->update_cursor = TRUE; path = gtk_tree_path_new_from_indices( event_viewer_data->pos->len - 1, -1); gtk_tree_view_set_cursor(GTK_TREE_VIEW(event_viewer_data->tree_v), @@ -629,8 +723,10 @@ void tree_v_move_cursor_cb (GtkWidget *widget, /* Must get up one event and select the first one */ gtk_tree_selection_unselect_all(gtk_tree_view_get_selection( GTK_TREE_VIEW(event_viewer_data->tree_v))); + event_viewer_data->update_cursor = FALSE; gtk_adjustment_set_value(event_viewer_data->vadjust_c, gtk_adjustment_get_value(event_viewer_data->vadjust_c) - 1); + event_viewer_data->update_cursor = TRUE; path = gtk_tree_path_new_from_indices( 0, -1); gtk_tree_view_set_cursor(GTK_TREE_VIEW(event_viewer_data->tree_v), @@ -654,8 +750,10 @@ void tree_v_move_cursor_cb (GtkWidget *widget, gtk_tree_selection_unselect_all(gtk_tree_view_get_selection( GTK_TREE_VIEW(event_viewer_data->tree_v))); + event_viewer_data->update_cursor = FALSE; gtk_adjustment_set_value(event_viewer_data->vadjust_c, gtk_adjustment_get_value(event_viewer_data->vadjust_c) + 2); + event_viewer_data->update_cursor = TRUE; path = gtk_tree_path_new_from_indices( event_viewer_data->pos->len - 1, -1); @@ -679,8 +777,10 @@ void tree_v_move_cursor_cb (GtkWidget *widget, gtk_tree_selection_unselect_all(gtk_tree_view_get_selection( GTK_TREE_VIEW(event_viewer_data->tree_v))); + event_viewer_data->update_cursor = FALSE; gtk_adjustment_set_value(event_viewer_data->vadjust_c, gtk_adjustment_get_value(event_viewer_data->vadjust_c) - 2); + event_viewer_data->update_cursor = TRUE; path = gtk_tree_path_new_from_indices( 0, -1); @@ -1031,13 +1131,13 @@ static void get_events(double new_value, EventViewerData *event_viewer_data) ScrollDirection direction; gint relative_position; - if(value < 0.0) { + if(value < -0.8) { if(value >= -1.0) direction = SCROLL_STEP_UP; else { if(value >= -2.0) direction = SCROLL_PAGE_UP; else direction = SCROLL_JUMP; } - } else if(value > 0.0) { + } else if(value > 0.8) { if(value <= 1.0) direction = SCROLL_STEP_DOWN; else { if(value <= 2.0) direction = SCROLL_PAGE_DOWN; @@ -1169,12 +1269,10 @@ static void get_events(double new_value, EventViewerData *event_viewer_data) /* Mathieu : - * I make the choice not to use the mainwindow lttvwindow API here : it will - * be faster to read the events ourself from lttv_process_traceset_middle, as - * we are already at the right read position. It would be costier to use the - * seek time closest for nothing, as we only have few events to read. + * I make the choice not to use the mainwindow lttvwindow API here : the idle + * loop might have a too low priority, and we want good update while + * scrolling. */ - /* FIXME : use seek time closest and middle to have a good state. */ lttv_process_traceset_begin(tsc, NULL, NULL, NULL, event_viewer_data->event_hooks, NULL); @@ -1259,15 +1357,17 @@ int event_hook(void *hook_data, void *call_data) g_string_free(desc, TRUE); - if(lttv_traceset_context_pos_pos_compare(pos, - event_viewer_data->currently_selected_position) == 0) { - GtkTreePath *path = gtk_tree_path_new_from_indices( - event_viewer_data->pos->len - 1, -1); - gtk_tree_view_set_cursor(GTK_TREE_VIEW(event_viewer_data->tree_v), - path, NULL, FALSE); - gtk_tree_path_free(path); + if(event_viewer_data->update_cursor) { + if(lttv_traceset_context_pos_pos_compare(pos, + event_viewer_data->currently_selected_position) == 0) { + GtkTreePath *path = gtk_tree_path_new_from_indices( + event_viewer_data->pos->len - 1, -1); + gtk_tree_view_set_cursor(GTK_TREE_VIEW(event_viewer_data->tree_v), + path, NULL, FALSE); + gtk_tree_path_free(path); + } } - + if(event_viewer_data->pos->len >= event_viewer_data->num_visible_events -1 ) return TRUE; else @@ -1424,8 +1524,12 @@ gboolean filter_changed(void * hook_data, void * call_data) } +gint redraw_notify(void *hook_data, void *call_data) +{ + EventViewerData *event_viewer_data = (EventViewerData*) hook_data; - + get_events(event_viewer_data->vadjust_c->value, event_viewer_data); +} void gui_events_free(EventViewerData *event_viewer_data) { @@ -1460,6 +1564,10 @@ void gui_events_free(EventViewerData *event_viewer_data) traceset_changed, event_viewer_data); lttvwindow_unregister_filter_notify(tab, filter_changed, event_viewer_data); + lttvwindow_unregister_redraw_notify(tab, + redraw_notify, event_viewer_data); + + lttvwindowtraces_background_notify_remove(event_viewer_data); g_event_viewer_data_list = g_slist_remove(g_event_viewer_data_list, event_viewer_data);