diff options
| author | Michael Brown | 2024-03-17 18:49:05 +0100 |
|---|---|---|
| committer | Michael Brown | 2024-03-17 18:49:05 +0100 |
| commit | fa4bda617d5d2a714d0f8abfba451fd87f11a33e (patch) | |
| tree | 17c35528a843b638725934c32afc800d7a52590d | |
| parent | [snp] Allocate additional padding for receive buffers (diff) | |
| download | ipxe-fa4bda617d5d2a714d0f8abfba451fd87f11a33e.tar.gz ipxe-fa4bda617d5d2a714d0f8abfba451fd87f11a33e.tar.xz ipxe-fa4bda617d5d2a714d0f8abfba451fd87f11a33e.zip | |
[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 914fe5c1..9e1179cc 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 ); |