X-Git-Url: http://gitweb.hugovil.com/?a=blobdiff_plain;f=src%2Fgtk%2Fmemwin.c;h=c525fb1a42eccfe68736a66f1ef131798b4bb746;hb=d755f9638255472107ae445f4f82d8fcd3446b20;hp=58ce1d2e6b7f87fae080c4fe57d1b2202a78c3d3;hpb=1eb382f72510d50b3636fb88c4bfaf17183672b6;p=emu8051.git diff --git a/src/gtk/memwin.c b/src/gtk/memwin.c index 58ce1d2..c525fb1 100644 --- a/src/gtk/memwin.c +++ b/src/gtk/memwin.c @@ -24,7 +24,9 @@ #endif #include +#include #include /* For isprint */ +#include /* For crc32 routine */ #include "common.h" #include "memory.h" @@ -48,8 +50,17 @@ enum COL_DATA0, }; -static GtkWidget *memlist_internal; -static GtkWidget *memlist_external; +/* Contains informations for the two different memory window types. */ +struct memwin_infos_t { + GtkWidget *memlist; + int data_rows; + u_int32_t *crc; + int crc_init; +}; + +static struct memwin_infos_t memwin_infos_internal; +static struct memwin_infos_t memwin_infos_external; +static struct memwin_infos_t *memwin_infos; /* Creating a model */ static GtkListStore * @@ -155,7 +166,7 @@ memwin_init_columns(GtkWidget *listview, int memory_id) gtk_tree_view_column_set_sizing(column, GTK_TREE_VIEW_COLUMN_AUTOSIZE); gtk_tree_view_append_column(GTK_TREE_VIEW(listview), column); - for (i = COL_DATA0; i < (COL_DATA0 + cfg->bits_per_row); i++) { + for (i = COL_DATA0; i < (COL_DATA0 + cfg->bytes_per_row); i++) { char col_name[8]; /* Create new renderer for each editable cell. */ @@ -176,7 +187,7 @@ memwin_init_columns(GtkWidget *listview, int memory_id) GUINT_TO_POINTER(memory_id)); /* Use two digits only if DATA_ROWS > 10 */ - if (cfg->bits_per_row < 10) + if (cfg->bytes_per_row < 10) sprintf(col_name, "B%1d", i - COL_DATA0); else sprintf(col_name, "B%02d", i - COL_DATA0); @@ -197,21 +208,35 @@ memwin_init_columns(GtkWidget *listview, int memory_id) gtk_tree_view_append_column(GTK_TREE_VIEW(listview), column); } -static int -compute_data_rows(int memory_id) +static void +memwin_infos_select(int memory_id) { - int data_rows; - if (memory_id == INT_MEM_ID) { - data_rows = options.iram_size / cfg->bits_per_row; + log_debug("memory ID = INTERNAL"); + memwin_infos = &memwin_infos_internal; } else if (memory_id == EXT_MEM_ID) { - data_rows = options.xram_size / cfg->bits_per_row; + log_debug("memory ID = EXTERNAL"); + memwin_infos = &memwin_infos_external; } else { log_fail("Invalid memory type"); exit(1); } +} + +static void +compute_data_rows(int memory_id) +{ + if (memory_id == INT_MEM_ID) { + memwin_infos->data_rows = options.iram_size / cfg->bytes_per_row; + } else if (memory_id == EXT_MEM_ID) { + memwin_infos->data_rows = options.xram_size / cfg->bytes_per_row; + } - return data_rows; + if (memwin_infos->crc) + free(memwin_infos->crc); + + memwin_infos->crc = malloc(memwin_infos->data_rows * sizeof(u_int32_t)); + memwin_infos->crc_init = false; } GtkWidget * @@ -220,10 +245,10 @@ memwin_init(char *title, int memory_id) GtkWidget *frame; GtkWidget *scrollwin; GtkListStore *store; - GtkWidget *memlist; - int data_rows; - COL_ASCII = cfg->bits_per_row + 1; + log_debug("memwin_init"); + + COL_ASCII = cfg->bytes_per_row + 1; N_COLUMNS = COL_ASCII + 1; frame = gtk_frame_new(title); @@ -239,26 +264,18 @@ memwin_init(char *title, int memory_id) gtk_container_add(GTK_CONTAINER(frame), scrollwin); - data_rows = compute_data_rows(memory_id); + memwin_infos_select(memory_id); + compute_data_rows(memory_id); /* Creating a model */ - store = memwin_init_store(data_rows); + store = memwin_init_store(memwin_infos->data_rows); /* Creating the view component */ - memlist = gtk_tree_view_new_with_model(GTK_TREE_MODEL(store)); - gtk_tree_view_set_headers_visible(GTK_TREE_VIEW(memlist), TRUE); - gtk_container_add(GTK_CONTAINER(scrollwin), memlist); - - if (memory_id == INT_MEM_ID) { - memlist_internal = memlist; - } else if (memory_id == EXT_MEM_ID) { - memlist_external = memlist; - } else { - log_fail("Invalid memory type"); - exit(1); - } + memwin_infos->memlist = gtk_tree_view_new_with_model(GTK_TREE_MODEL(store)); + gtk_tree_view_set_headers_visible(GTK_TREE_VIEW(memwin_infos->memlist), TRUE); + gtk_container_add(GTK_CONTAINER(scrollwin), memwin_infos->memlist); - memwin_init_columns(memlist, memory_id); + memwin_init_columns(memwin_infos->memlist, memory_id); /* * The tree view has acquired its own reference to the model, so we can @@ -270,32 +287,51 @@ memwin_init(char *title, int memory_id) return frame; } +/* + * Use CRC to detect which rows have changed. This is only to + * improve performance when using stepping mode, as we then only update + * rows which have been modified. + */ +static int +memwin_row_changed(int memory_id, int row, unsigned int address) +{ + int row_changed; + u_int32_t crc_new = 0; + u_int8_t *buf8; + + buf8 = memory_getbuf(memory_id, address); + crc_new = crc32(0L, Z_NULL, 0); + crc_new = crc32(crc_new, buf8, cfg->bytes_per_row); + + if ((memwin_infos->crc_init != false) && + (crc_new == memwin_infos->crc[row])) { + row_changed = false; + } else { + memwin_infos->crc[row] = crc_new; + log_debug(" Row %02d value(s) change", row); + row_changed = true; + } + + return row_changed; +} + /* Dump internal or external memory. */ void memwin_refresh(int memory_id) { int row; - unsigned int Address; + unsigned int Address = 0; GtkListStore *store; - GtkWidget *memlist; - int data_rows; - Address = 0; + log_debug("memwin_refresh"); - data_rows = compute_data_rows(memory_id); + memwin_infos_select(memory_id); - if (memory_id == INT_MEM_ID) { - memlist = memlist_internal; - } else if (memory_id == EXT_MEM_ID) { - memlist = memlist_external; - } else { - log_fail("Invalid memory type"); - exit(1); - } - - store = GTK_LIST_STORE(gtk_tree_view_get_model(GTK_TREE_VIEW(memlist))); + store = GTK_LIST_STORE(gtk_tree_view_get_model( + GTK_TREE_VIEW(memwin_infos->memlist))); - for (row = 0; row < data_rows; row++) { + for (row = 0; row < memwin_infos->data_rows; + row++, Address += cfg->bytes_per_row) { int valid; GtkTreeIter iter; char str[4+1]; /* Maximum str len is for address column (4 digits) */ @@ -313,34 +349,38 @@ memwin_refresh(int memory_id) } if (!valid) { - printf("Invalid iter...\n"); + log_err("Tree model: invalid iter"); return; } - /* Display base address. */ - int2asciihex(Address, str, 4); - - gtk_list_store_set(store, &iter, COL_ADDRESS, str, -1); + /* Only update row if it has been modified. */ + if (memwin_row_changed(memory_id, row, Address)) { + /* Display base address. */ + int2asciihex(Address, str, 4); - for (col = 0; col < cfg->bits_per_row; col++) { - u_int8_t data; + gtk_list_store_set(store, &iter, COL_ADDRESS, str, -1); - data = memory_read8(memory_id, Address + col); + for (col = 0; col < cfg->bytes_per_row; col++) { + u_int8_t data; - /* Display hex data */ - int2asciihex(data, str, 2); + data = memory_read8(memory_id, Address + col); - gtk_list_store_set(store, &iter, col + 1, str, -1); + /* Display hex data */ + int2asciihex(data, str, 2); - /* Append to ASCII string (if applicable). */ - if (!isprint(data)) - data = '.'; - sprintf(&ascii_str[col], "%c", data); - } + gtk_list_store_set(store, &iter, col + 1, str, -1); - /* Display ASCII characters. */ - gtk_list_store_set(store, &iter, COL_ASCII, ascii_str, -1); + /* Append to ASCII string (if applicable). */ + if (!isprint(data)) + data = '.'; + sprintf(&ascii_str[col], "%c", data); + } - Address += cfg->bits_per_row; + /* Display ASCII characters. */ + gtk_list_store_set(store, &iter, COL_ASCII, ascii_str, -1); + } } + + /* At this point we know all rows crc have been initialized. */ + memwin_infos->crc_init = true; }