On Mon, Sep 22, 2003 at 12:39:40PM -0700, David Mosberger wrote: > You know, you'd greatly increase the chance of my applying the patch > if the patch: > > - were broken up into reasonably small, self-contained pieces Um.. I can try to break it up. > - doesn't actually BREAK the existing ia64 platforms Does it? We've tested it on quite a few... > I'm sorry, but until some of these criteria are met, there is no way > I'm going to apply this patch. As for the breakage: as far as I can > see, the patch moves the bootmem initialization back to the old place, > which means that drivers/acpi/tables.c will fail to allocate memory > for the SDT again. Len Brown already took the change to fix tables.c. > Some other comments: > > - in acpi.c, it looks like there is no reason for min_hole_size anymore, > yet the patch doesn't clean up this variable Ok. > - MAX_RSVD_REGIONS et al got moved to pgtable.h; why in the world would > that be the right place? Just because "it compiled"?? I'll have to look at that again. I think I needed to move some of that stuff to a header because I created contig.c to seperate things a little. > - why is there both vmem_map and mem_map in the NUMA case? Because it compiled :/. I'll try to fix that up too. Thanks, Jesse - To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.htmlReceived on Mon Sep 22 16:32:14 2003
This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:17 EST