[coreboot-gerrit] Patch merged into coreboot/master: 8bbd04e libpayload: special case large memalign() requests

gerrit at coreboot.org gerrit at coreboot.org
Fri Mar 20 09:59:23 CET 2015


the following patch was just integrated into master:
commit 8bbd04ea8d416b4ec952bf08f572f126ab7b4176
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Thu Jan 22 08:59:03 2015 -0600

    libpayload: special case large memalign() requests
    
    For memalign() requests the current allocator keeps metadata
    about each chunk of aligned memory that copmrises the size
    requested. For large allocations relative to the alignment
    this can cause significant metadata overhead. Instead, consider
    all memalign() requests whose size meets or exceeds 1KiB or
    alignment that meets or exceeds 1KiB large requests.
    These requests are handled specially to only allocate
    the amount of memory required for the size and alignment
    constraints by not allocating any metadata as the whole region
    would be consumed by the request.
    
    BUG=None
    BRANCH=None
    TEST=Built and tested various scenarios. Noted the ability to
         free() and properly coalesce the heap as expected.
    
    Change-Id: Ia9cf5529ca859e490617af296cffd2705c2c6fd8
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 4e32fc57626dac6194c9fd0141df680b4a5417e8
    Original-Change-Id: Icdf022831b733e3bb84a2d2f3b499f4e25d89128
    Original-Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/242456
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
    Reviewed-on: http://review.coreboot.org/8729
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


See http://review.coreboot.org/8729 for details.

-gerrit



More information about the coreboot-gerrit mailing list