diff options
| author | Markus Armbruster <armbru@redhat.com> | 2025-04-07 10:26:43 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2025-04-24 09:34:53 +0200 |
| commit | abb55b1abaaa8521e93d8202dd67b9e70a4ca1a2 (patch) | |
| tree | 99445c2e5aa99af6b21ca3474dabada3227db57d /hw/ipmi/ipmi_bmc_sim.c | |
| parent | 8a2b516ba2855c4530388051de2b8d17bc780ea8 (diff) | |
| download | focaccia-qemu-abb55b1abaaa8521e93d8202dd67b9e70a4ca1a2.tar.gz focaccia-qemu-abb55b1abaaa8521e93d8202dd67b9e70a4ca1a2.zip | |
cleanup: Drop pointless label at end of function
Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Acked-by: Corey Minyard <cminyard@mvista.com> Message-ID: <20250407082643.2310002-4-armbru@redhat.com>
Diffstat (limited to 'hw/ipmi/ipmi_bmc_sim.c')
| -rw-r--r-- | hw/ipmi/ipmi_bmc_sim.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/hw/ipmi/ipmi_bmc_sim.c b/hw/ipmi/ipmi_bmc_sim.c index c6a85e8cd1..41fe6835a7 100644 --- a/hw/ipmi/ipmi_bmc_sim.c +++ b/hw/ipmi/ipmi_bmc_sim.c @@ -465,13 +465,12 @@ void ipmi_bmc_gen_event(IPMIBmc *b, uint8_t *evt, bool log) } if (ibs->msg_flags & IPMI_BMC_MSG_FLAG_EVT_BUF_FULL) { - goto out; + return; } memcpy(ibs->evtbuf, evt, 16); ibs->msg_flags |= IPMI_BMC_MSG_FLAG_EVT_BUF_FULL; k->set_atn(s, 1, attn_irq_enabled(ibs)); - out: } static void gen_event(IPMIBmcSim *ibs, unsigned int sens_num, uint8_t deassert, uint8_t evd1, uint8_t evd2, uint8_t evd3) @@ -996,7 +995,7 @@ static void get_msg(IPMIBmcSim *ibs, if (QTAILQ_EMPTY(&ibs->rcvbufs)) { rsp_buffer_set_error(rsp, 0x80); /* Queue empty */ - goto out; + return; } rsp_buffer_push(rsp, 0); /* Channel 0 */ msg = QTAILQ_FIRST(&ibs->rcvbufs); @@ -1011,8 +1010,6 @@ static void get_msg(IPMIBmcSim *ibs, ibs->msg_flags &= ~IPMI_BMC_MSG_FLAG_RCV_MSG_QUEUE; k->set_atn(s, attn_set(ibs), attn_irq_enabled(ibs)); } - -out: } static unsigned char |