On 05/14/2010 01:30 AM, Benjamin Herrenschmidt wrote: > On Thu, 2010-05-13 at 23:44 -0700, Yinghai wrote: > >> On 05/13/2010 07:32 PM, Benjamin Herrenschmidt wrote: >> >>> On Thu, 2010-05-13 at 17:19 -0700, Yinghai Lu wrote: >>> >>>> Some areas from firmware could be reserved several times from different callers. >>>> >>>> If these area are overlapped, We may have overlapped entries in lmb.reserved. >>>> >>>> Try to free the area at first, before rerserve them again. >>>> >>> I have already told you to make this a property of lmb_reserve() instead >>> of adding that function with a terrible name. >>> >> make every lmb_reserve() call lmb_free at first? >> > Either that, or make it check for collisions first, and if there's > one, call free and try again. A little bit more work but I plan toq > make it smarter at some stage, ie, directly adjust surrounding ranges > instead which is not -that- hard to do. > > later, after this patchset. this patchset is hanging around too long already. YH -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html