+Sat Jul 9 23:20:39 2005 Soeren Sandmann <sandmann@redhat.com>
+
+ * binfile.c (bin_file_new): Cache BinFiles by filename.
+
+ * stackstash.c (stack_stash_free): Plug leak
+
+ * process.c (process_free_maps): Plug leak
+
+ * module/Makefile (install): Check that depmod exists before
+ running it.
+
Sun Jun 19 15:42:34 2005 Søren Sandmann <sandmann@redhat.com>
* module/sysprof-module.c (SAMPLES_PER_SECOND): Set to 200.
- Possibly a special "view details" mode, assuming that
the details of a function are not that interesting
together with a tree.
-- consider caching [filename => bin_file]
- rethink caller list, not terribly useful at the moment.
- Have kernel module report the file the address was found in
DONE:
+* consider caching [filename => bin_file]
+
* Check the kernel we are building against, if it is SMP or
less than 2.6.11, print a warning and suggest upgrading.
int n_symbols;
Symbol *symbols;
Symbol undefined;
+ int ref_count;
};
static bfd *
bf->symbols = (Symbol *)g_array_free (symbols, FALSE);
}
+static GHashTable *bin_files;
+
BinFile *
bin_file_new (const char *filename)
{
- BinFile *bf = g_new0 (BinFile, 1);
+ BinFile *bf;
- bf->filename = g_strdup (filename);
-
- read_symbols (bf);
+ if (!bin_files)
+ bin_files = g_hash_table_new (g_str_hash, g_str_equal);
- bf->undefined.name = g_strdup_printf ("In file %s", filename);
- bf->undefined.address = 0x0;
+ bf = g_hash_table_lookup (bin_files, filename);
+ if (bf)
+ {
+ bf->ref_count++;
+ }
+ else
+ {
+ bf = g_new0 (BinFile, 1);
+ bf->filename = g_strdup (filename);
+
+ read_symbols (bf);
+
+ bf->undefined.name = g_strdup_printf ("In file %s", filename);
+ bf->undefined.address = 0x0;
+ bf->ref_count = 1;
+ g_hash_table_insert (bin_files, bf->filename, bf);
+ }
+
return bf;
}
void
bin_file_free (BinFile *bf)
{
- int i;
-
- g_free (bf->filename);
+ if (--bf->ref_count == 0)
+ {
+ int i;
- for (i = 0; i < bf->n_symbols; ++i)
- g_free (bf->symbols[i].name);
- g_free (bf->symbols);
- g_free (bf->undefined.name);
- g_free (bf);
+ g_hash_table_remove (bin_files, bf->filename);
+
+ g_free (bf->filename);
+
+ for (i = 0; i < bf->n_symbols; ++i)
+ g_free (bf->symbols[i].name);
+ g_free (bf->symbols);
+ g_free (bf->undefined.name);
+ g_free (bf);
+ }
}
/**
install:
$(KMAKE) modules_install
- depmod
+ [ -e /sbin/depmod ] && /sbin/depmod
install-data:
install-exec:
for (list = process->maps; list != NULL; list = list->next)
{
Map *map = list->data;
+
+ if (map->filename)
+ g_free (map->filename);
if (map->bin_file)
bin_file_free (map->bin_file);
/* profile objects */
info.profile_objects = g_hash_table_new_full (g_str_hash, g_str_equal,
g_free, NULL);
-
+
stack_stash_foreach (stash, generate_object_table, &info);
stack_stash_foreach (stash, generate_call_tree, &info);
link_parents (info.profile->call_tree, NULL);
g_hash_table_foreach (info.profile->nodes_by_object, compute_object_total, NULL);
g_hash_table_destroy (info.profile_objects);
-
+
return info.profile;
}
{
stack_node_free (stash->root);
g_hash_table_destroy (stash->leaves_by_process);
+ g_free (stash);
}
g_idle_add (load_file, file_open_data);
}
-
+
gtk_main ();
return 0;