On Tue, 2009-07-21 at 16:02 -0400, Pavel Roskin wrote:
> By the way, kern/dl.c have some unused functions (grub_dl_unload_all).

I was wrong, I didn't check assembler files.

> Also, it looks like the memory for the module information (grub_dl_t) is
> allocated twice - first in grub_dl_load_core(), then in grub_dl_add() as
> part of grub_dl_list_t.

I was wrong here too.  I'm too accustomed to Linux style where typedefs
are discouraged.  The list keeps a pointer, not a copy.

I'll apply your patch if it passes testing.

-- 
Regards,
Pavel Roskin


_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to