On Mon, 2013-04-08 at 13:44 -0700, Andrew Morton wrote: > On Mon, 8 Apr 2013 11:09:53 -0600 Toshi Kani <toshi.kani@xxxxxx> wrote: > > > Memory hot-delete to a memory range present at boot causes an > > error message in __release_region(), such as: > > > > Trying to free nonexistent resource <0000000070000000-0000000077ffffff> > > > > Hot-delete operation still continues since __release_region() is > > a void function, but the target memory range is not freed from > > iomem_resource as the result. This also leads a failure in a > > subsequent hot-add operation to the same memory range since the > > address range is still in-use in iomem_resource. > > > > This problem happens because the granularity of memory resource ranges > > may be different between boot and hot-delete. > > So we don't need this new code if CONFIG_MEMORY_HOTPLUG=n? If so, can > we please arrange for it to not be present if the user doesn't need it? Good point! Yes, since the new function is intended for memory hot-delete and is only called from __remove_pages() in mm/memory_hotplug.c, it should be added as #ifdef CONFIG_MEMORY_HOTPLUG in PATCH 2/3. I will make the change, and send an updated patch to PATCH 2/3. Thanks, -Toshi -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>