On Mon, Aug 17, 2009 at 04:40:27PM +0530, Syed Mohammed, Khasim wrote: > This patch seem to be working, I don't see any dump like before. Great. > Can you please advice on how to get this patch into kernel? I mean, > will you be handling it or we need to re-submit them with your signed off? Yes, I'll be sending it. However, I believe the patch I sent contains one error: > > +static int __init meminfo_cmp(const void *_a, const void *_b) > > +{ > > + const struct membank *a = _a, *b = _b; > > + long cmp = bank_pfn_start(b) - bank_pfn_start(a); I think this should be bank_pfn_start(a) - bank_pfn_start(b). When you tested this unmodified patch, did the "Memory:" line list the memory bank sizes in reverse order? -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html