From 4676ec52c279d44eee44ceef40d0107b09bedce6 Mon Sep 17 00:00:00 2001 From: Julius Werner Date: Wed, 10 Mar 2021 16:52:14 -0800 Subject: [PATCH] cbfs: Make `mdata` argument to cbfs_allocator_t const Right before CB:49334 was submitted, I changed the signature of cbfs_allocator_t function pointers to include another argument passing in the already loaded CBFS metadata (to allow for the rare edge case of allocators needing to read CBFS attributes). This interface is not meant to be able to modify the passed-in metadata, so to clarify that and prevent potential errors, we should declare the argument const. Signed-off-by: Julius Werner Change-Id: I7e3756490b9ad7ded91268c61797cef36c4118ee Reviewed-on: https://review.coreboot.org/c/coreboot/+/52081 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin Reviewed-by: Furquan Shaikh --- src/include/cbfs.h | 6 +++--- src/lib/cbfs.c | 4 ++-- src/lib/rmodule.c | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/include/cbfs.h b/src/include/cbfs.h index b89a13ddd3..37bac308d8 100644 --- a/src/include/cbfs.h +++ b/src/include/cbfs.h @@ -67,7 +67,7 @@ * attributes). Must return a pointer to space of the requested size where the file data should * be loaded, or NULL to make the operation fail. */ -typedef void *(*cbfs_allocator_t)(void *arg, size_t size, union cbfs_mdata *mdata); +typedef void *(*cbfs_allocator_t)(void *arg, size_t size, const union cbfs_mdata *mdata); static inline size_t cbfs_load(const char *name, void *buf, size_t size); static inline size_t cbfs_ro_load(const char *name, void *buf, size_t size); @@ -183,9 +183,9 @@ struct _cbfs_default_allocator_arg { void *buf; size_t buf_size; }; -void *_cbfs_default_allocator(void *arg, size_t size, union cbfs_mdata *unused); +void *_cbfs_default_allocator(void *arg, size_t size, const union cbfs_mdata *unused); -void *_cbfs_cbmem_allocator(void *arg, size_t size, union cbfs_mdata *unused); +void *_cbfs_cbmem_allocator(void *arg, size_t size, const union cbfs_mdata *unused); /********************************************************************************************** * INLINE IMPLEMENTATIONS * diff --git a/src/lib/cbfs.c b/src/lib/cbfs.c index fbf4531862..65bb721495 100644 --- a/src/lib/cbfs.c +++ b/src/lib/cbfs.c @@ -369,7 +369,7 @@ void *_cbfs_alloc(const char *name, cbfs_allocator_t allocator, void *arg, return loc; } -void *_cbfs_default_allocator(void *arg, size_t size, union cbfs_mdata *unused) +void *_cbfs_default_allocator(void *arg, size_t size, const union cbfs_mdata *unused) { struct _cbfs_default_allocator_arg *darg = arg; if (size > darg->buf_size) @@ -377,7 +377,7 @@ void *_cbfs_default_allocator(void *arg, size_t size, union cbfs_mdata *unused) return darg->buf; } -void *_cbfs_cbmem_allocator(void *arg, size_t size, union cbfs_mdata *unused) +void *_cbfs_cbmem_allocator(void *arg, size_t size, const union cbfs_mdata *unused) { return cbmem_add((uintptr_t)arg, size); } diff --git a/src/lib/rmodule.c b/src/lib/rmodule.c index ac9eb0b306..31bf14190f 100644 --- a/src/lib/rmodule.c +++ b/src/lib/rmodule.c @@ -192,7 +192,7 @@ int rmodule_load(void *base, struct rmodule *module) } static void *rmodule_cbfs_allocator(void *rsl_arg, size_t unused, - union cbfs_mdata *mdata) + const union cbfs_mdata *mdata) { struct rmod_stage_load *rsl = rsl_arg;