Hi Toshi,

On 09/13/2013 03:22 AM, Toshi Kani wrote:
......
+               if (memblock_direction_bottom_up()) {
+                       addr = memblock_alloc_bottom_up(
+                                               MEMBLOCK_ALLOC_ACCESSIBLE,
+                                               limit, size, alignment);
+                       if (addr)
+                               goto success;
+               }

I am afraid that this version went to a wrong direction.  Allocating
from the bottom up needs to be an internal logic within the memblock
allocator.  It should not require the callers to be aware of the
direction and make a special request.


I think my v1 patch-set was trying to do so. Was it too complicated ?

So just move this logic to memblock_find_in_range_node(), is this OK ?

Thanks.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to