diff options
author | Michael Brown <mcb30@ipxe.org> | 2024-03-17 17:49:05 +0000 |
---|---|---|
committer | Michael Brown <mcb30@ipxe.org> | 2024-03-17 17:49:05 +0000 |
commit | fa4bda617d5d2a714d0f8abfba451fd87f11a33e (patch) | |
tree | 17c35528a843b638725934c32afc800d7a52590d | |
parent | bac967d51a8851ef274da6aa7a3a9711de0bb056 (diff) | |
download | ipxe-fa4bda617d5d2a714d0f8abfba451fd87f11a33e.tar.gz |
[build] Fix building on older versions of gcc
Older versions of gcc (observed with gcc 4.8.5 on CentOS 7) complain
about having the label "err_ioremap" at the end of a compound
statement in bios_mp_start_all(). The label is correctly placed,
since it immediately follows the iounmap() that would be required to
undo a successful ioremap() in the non-error case.
Fix by adding an explicit "return" immediately after the label.
Signed-off-by: Michael Brown <mcb30@ipxe.org>
-rw-r--r-- | src/arch/x86/interface/pcbios/bios_mp.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/arch/x86/interface/pcbios/bios_mp.c b/src/arch/x86/interface/pcbios/bios_mp.c index 914fe5c10..9e1179ccd 100644 --- a/src/arch/x86/interface/pcbios/bios_mp.c +++ b/src/arch/x86/interface/pcbios/bios_mp.c @@ -165,6 +165,7 @@ static void bios_mp_start_all ( mp_func_t func, void *opaque ) { /* No way to handle errors: caller must check that * multiprocessor function executed as expected. */ + return; } PROVIDE_MPAPI_INLINE ( pcbios, mp_address ); |