>Here is the updated patch. It incorporates your suggestions. I have left >kern_memdesc as a linked list as opposed to array. Linked list is little >more versatile structure and the cost is minimal over an array. Unless >you feel strongly about using arrays instead of linked list, I would >prefer to leave it this way.
Khalid, Looking good (from my 2 minute scan, I'll take a better look later). One question ... why do we need "is_available_memory" and "is_usable_memory"? They look to be almost the same (except is_usable... doesn't consider EFI_LOADER_DATA to be usable). It doesn't look like is_available_memory needs to be in meminit.h [in last version of this patch it was used in more than one file, but now the only usage in in efi.c] I'll ponder the linked question ... if you really think the extra versatility is worthwhile, then I'll consider them ... but that will raise the issue of why aren't you using the standard Linux kernel list.h macros. -Tony - To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
