How about adding a counter to grub_dprintf to make it easy to instrument
GRUB and find which are the bottlenecks in boot time?
Sidenote: perhaps it'd be a good idea to conditionalize all grub_dprintf
calls with #ifdef DEBUG to obtain a smaller core.img. It's not hard to
ask a user to rebuild if dprintf is needed, and we can find non-ugly ways
to do this without massive #ifdefs all over the code.
--
Robert Millan
<GPLv2> I know my rights; I want my phone call!
<DRM> What good is a phone call… if you are unable to speak?
(as seen on /.)
diff -x ChangeLog -x configure -x config.h.in -x CVS -x '*~' -x '*.mk' -urp ../grub2/include/grub/time.h ./include/grub/time.h
--- ../grub2/include/grub/time.h 2007-10-22 22:02:16.000000000 +0200
+++ ./include/grub/time.h 2008-06-19 15:23:45.000000000 +0200
@@ -23,6 +23,8 @@
#include <grub/machine/time.h>
#include <grub/cpu/time.h>
+extern grub_uint32_t grub_start_time;
+
void EXPORT_FUNC(grub_millisleep) (grub_uint32_t ms);
void EXPORT_FUNC(grub_millisleep_generic) (grub_uint32_t ms);
diff -x ChangeLog -x configure -x config.h.in -x CVS -x '*~' -x '*.mk' -urp ../grub2/kern/main.c ./kern/main.c
--- ../grub2/kern/main.c 2008-02-10 18:05:10.000000000 +0100
+++ ./kern/main.c 2008-06-19 15:19:45.000000000 +0200
@@ -106,12 +106,15 @@ grub_load_normal_mode (void)
grub_print_error ();
}
+grub_uint32_t grub_start_time;
+
/* The main routine. */
void
grub_main (void)
{
/* First of all, initialize the machine. */
grub_machine_init ();
+ grub_start_time = grub_get_rtc ();
/* Hello. */
grub_setcolorstate (GRUB_TERM_COLOR_HIGHLIGHT);
diff -x ChangeLog -x configure -x config.h.in -x CVS -x '*~' -x '*.mk' -urp ../grub2/kern/misc.c ./kern/misc.c
--- ../grub2/kern/misc.c 2008-06-16 22:11:22.000000000 +0200
+++ ./kern/misc.c 2008-06-19 15:26:40.000000000 +0200
@@ -147,7 +147,15 @@ grub_real_dprintf (const char *file, con
if (grub_strword (debug, "all") || grub_strword (debug, condition))
{
- grub_printf ("%s:%d: ", file, line);
+ grub_printf (
+#ifndef GRUB_UTIL
+ "[%u] "
+#endif
+ "%s:%d: ",
+#ifndef GRUB_UTIL
+ grub_get_rtc () - grub_start_time,
+#endif
+ file, line);
va_start (args, fmt);
grub_vprintf (fmt, args);
va_end (args);
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
