From b33a2b05afe2b309da5a608f3d17ffcf112c120a Mon Sep 17 00:00:00 2001 From: Furquan Shaikh Date: Thu, 26 Sep 2019 23:51:46 -0700 Subject: fmap: Add get_fmap_flash_offset() CB:35377 changed the behavior of find_fmap_directory() to return pointer to CBMEM_ID_FMAP if fmap is cached in cbmem. lb_boot_media_params() calls find_fmap_directory to add offset of fmap in flash to coreboot table. However, because of the change in behavior of find_fmap_directory(), it ended up adding 0 as the offset. This change adds a new function get_fmap_flash_offset() which returns the offset of fmap in flash. Ideally, all payloads should move to using the FMAP from CBMEM. However, in order to maintain compatibility with payloads which are not updated, ensure that fmap_offset is updated correctly. Since find_fmap_directory() is no longer used outside fmap.c, this change also removes it from fmap.h and limits scope to fmap.c. In a follow up patch, we need to push a change to libpayload to expose the fmap cache pointer to lib_sysinfo. BUG=b:141723751 Change-Id: I7ff6e8199143d1a992a83d7de1e3b44813b733f4 Signed-off-by: Furquan Shaikh Reviewed-on: https://review.coreboot.org/c/coreboot/+/35639 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Rudolph Reviewed-by: Aaron Durbin Reviewed-by: Shelley Chen --- src/include/fmap.h | 7 ++++--- src/lib/coreboot_table.c | 5 +---- src/lib/fmap.c | 7 ++++++- 3 files changed, 11 insertions(+), 8 deletions(-) (limited to 'src') diff --git a/src/include/fmap.h b/src/include/fmap.h index ab7e5ab895..649ecc0000 100644 --- a/src/include/fmap.h +++ b/src/include/fmap.h @@ -19,9 +19,6 @@ #include #include -/* Locate the fmap directory. Return 0 on success, < 0 on error. */ -int find_fmap_directory(struct region_device *fmrd); - /* Locate the named area in the fmap and fill in a region device representing * that area. The region is a sub-region of the readonly boot media. Return * 0 on success, < 0 on error. */ @@ -48,4 +45,8 @@ ssize_t fmap_read_area(const char *name, void *buffer, size_t size); /* Write provided buffer into fmap area. * Return size written on success, < 0 on error. */ ssize_t fmap_overwrite_area(const char *name, const void *buffer, size_t size); + +/* Get offset of FMAP in flash. */ +uint64_t get_fmap_flash_offset(void); + #endif diff --git a/src/lib/coreboot_table.c b/src/lib/coreboot_table.c index 9c5942fa9d..d3576e6a32 100644 --- a/src/lib/coreboot_table.c +++ b/src/lib/coreboot_table.c @@ -256,7 +256,6 @@ static void lb_boot_media_params(struct lb_header *header) struct lb_boot_media_params *bmp; struct cbfs_props props; const struct region_device *boot_dev; - struct region_device fmrd; boot_device_init(); @@ -275,9 +274,7 @@ static void lb_boot_media_params(struct lb_header *header) bmp->cbfs_size = props.size; bmp->boot_media_size = region_device_sz(boot_dev); - bmp->fmap_offset = ~(uint64_t)0; - if (find_fmap_directory(&fmrd) == 0) - bmp->fmap_offset = region_device_offset(&fmrd); + bmp->fmap_offset = get_fmap_flash_offset(); } static void lb_ram_code(struct lb_header *header) diff --git a/src/lib/fmap.c b/src/lib/fmap.c index f0074186da..a427102210 100644 --- a/src/lib/fmap.c +++ b/src/lib/fmap.c @@ -31,7 +31,12 @@ static int fmap_print_once CAR_GLOBAL; static struct mem_region_device fmap_cache CAR_GLOBAL; -int find_fmap_directory(struct region_device *fmrd) +uint64_t get_fmap_flash_offset(void) +{ + return FMAP_OFFSET; +} + +static int find_fmap_directory(struct region_device *fmrd) { const struct region_device *boot; struct fmap *fmap; -- cgit v1.2.3