diff options
| author | Aaron Durbin <adurbin@chromium.org> | 2013-03-20 13:49:27 -0500 |
|---|---|---|
| committer | Ronald G. Minnich <rminnich@gmail.com> | 2013-03-20 21:48:32 +0100 |
| commit | eb06a4259b48128faed94b4ca3f8c64d3cd5a4c3 (patch) | |
| tree | bd0f9ac205315f4443c35ea0c016e1da298f3885 /src/lib/cbfs.c | |
| parent | 1cc4737c3bc7e0b1e560f3d049f4edbef9340177 (diff) | |
| download | coreboot-eb06a4259b48128faed94b4ca3f8c64d3cd5a4c3.tar.xz | |
x86: don't clear bss in ramstage entry
The cbfs stage loading routine already zeros out the full
memory region that a stage will be loaded. Therefore, it is
unnecessary to to clear the bss again after once ramstage starts.
Change-Id: Icc7021329dbf59bef948a41606f56746f21b507f
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/2865
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
Diffstat (limited to 'src/lib/cbfs.c')
| -rw-r--r-- | src/lib/cbfs.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/lib/cbfs.c b/src/lib/cbfs.c index abb95abc3f..c56f550e17 100644 --- a/src/lib/cbfs.c +++ b/src/lib/cbfs.c @@ -129,6 +129,7 @@ void * cbfs_load_stage(struct cbfs_media *media, const char *name) name, (uint32_t) stage->load, stage->memlen, stage->entry); + /* Stages rely the below clearing so that the bss is initialized. */ memset((void *) (uint32_t) stage->load, 0, stage->memlen); if (cbfs_decompress(stage->compression, |