From 779b537f2bcbdb7ac8775b391afb12cb563825a0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?S=C3=B8ren=20Sandmann?= Date: Wed, 11 Jan 2006 22:53:08 +0000 Subject: [PATCH] Remove debug spew. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Wed Jan 11 17:51:54 2006 Søren Sandmann * collector.c: Remove debug spew. * TODO: updates --- ChangeLog | 6 ++++++ TODO | 24 ++++++++++++++---------- collector.c | 3 +-- 3 files changed, 21 insertions(+), 12 deletions(-) diff --git a/ChangeLog b/ChangeLog index 6fbcc5ed..e1bedcf6 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +Wed Jan 11 17:51:54 2006 Søren Sandmann + + * collector.c: Remove debug spew. + + * TODO: updates + Mon Jan 9 09:58:25 2006 Soeren Sandmann * module/sysprof-module.c (timer_notify): Remove START_OF_STACK diff --git a/TODO b/TODO index 00f5cf1f..3b57ab00 100644 --- a/TODO +++ b/TODO @@ -16,18 +16,11 @@ Before 1.0.2: * See if we can reproduce this: Press start without kernel module loaded, then load kernel module and press start again. Segmentation fault. +* Test on x86-64. + Before 1.2: -* - Run a.out generated normally with gcc. - - Run sysprof - - hit profile - - Kill a.out - - strip a.out - - run a.out - - hit start - - hit profile - At this point we should not get any symbols, but we do. There is some - sort of bad caching going on. +* Add spew infrastructure to make remote debugging easier. * Make it compile and work on x86-64 @@ -427,6 +420,17 @@ Later: DONE: +* - Run a.out generated normally with gcc. + - Run sysprof + - hit profile + - Kill a.out + - strip a.out + - run a.out + - hit start + - hit profile + At this point we should not get any symbols, but we do. There is some + sort of bad caching going on. + * support more than one reader of the samples properly - Don't generate them if noone cares diff --git a/collector.c b/collector.c index 5cea2799..2adb2a43 100644 --- a/collector.c +++ b/collector.c @@ -182,10 +182,9 @@ open_fd (Collector *collector, while (fd < 0 && g_timer_elapsed (timer, NULL) < 0.5) { + /* Give udev time to create the device */ usleep (100000); - g_print ("open\n"); - fd = open (SYSPROF_FILE, O_RDONLY); }