[coreboot-gerrit] Patch merged into coreboot/master: d0db3a9 vboot2: Reduce minimum required work buffer size

gerrit at coreboot.org gerrit at coreboot.org
Fri Apr 10 16:49:06 CEST 2015


the following patch was just integrated into master:
commit d0db3a90add3333ccdd70922967ff6f2d9315451
Author: Julius Werner <jwerner at chromium.org>
Date:   Mon Dec 1 13:11:52 2014 -0800

    vboot2: Reduce minimum required work buffer size
    
    Apparently our initial submission of 16K was a little too generous for
    the vboot2 work buffer, and I hear that we should also be well within
    bounds for 12K. This patch reduces the minimum asserted by memlayout so
    some of our low-mem boards can get a few more kilobytes back for
    discretionary spending. Also changes the required minimum alignment to 8
    since that's what the current vboot code aligns it to anyway, and add a
    warning comment to make it clearer that this is a dangerous number
    people should not be playing with lightly.
    
    BRANCH=None
    BUG=None
    TEST=Built and booted on Pinky.
    
    Original-Change-Id: Iae9c74050500a315c90f5d5517427d755ac1dfea
    Original-Signed-off-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/232613
    Original-Reviewed-by: Randall Spangler <rspangler at chromium.org>
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    
    (cherry picked from commit 64e972f10363451cd544fdf8642bd484463703bc)
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    
    Change-Id: I362b8c33cf79534bb76bd7acda44d467563fe133
    Reviewed-on: http://review.coreboot.org/9445
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>
    Tested-by: build bot (Jenkins)


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

-gerrit



More information about the coreboot-gerrit mailing list