mirror of
https://github.com/izzy2lost/xemu.git
synced 2026-07-06 00:20:22 -07:00
cleanup: Drop pointless return at end of function
A few functions now end with a label. The next commit will clean them
up.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-ID: <20250407082643.2310002-3-armbru@redhat.com>
[Straightforward conflict with commit 988ad4cceb (hw/loongarch/virt:
Fix cpuslot::cpu set at last in virt_cpu_plug()) resolved]
This commit is contained in:
@@ -665,7 +665,6 @@ static inline void tb_add_jump(TranslationBlock *tb, int n,
|
||||
|
||||
out_unlock_next:
|
||||
qemu_spin_unlock(&tb_next->jmp_lock);
|
||||
return;
|
||||
}
|
||||
|
||||
static inline bool cpu_handle_halt(CPUState *cpu)
|
||||
|
||||
@@ -972,8 +972,6 @@ static void qemu_gluster_reopen_commit(BDRVReopenState *state)
|
||||
|
||||
g_free(state->opaque);
|
||||
state->opaque = NULL;
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
@@ -993,8 +991,6 @@ static void qemu_gluster_reopen_abort(BDRVReopenState *state)
|
||||
|
||||
g_free(state->opaque);
|
||||
state->opaque = NULL;
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
#ifdef CONFIG_GLUSTERFS_ZEROFILL
|
||||
|
||||
@@ -254,7 +254,6 @@ static void qemu_rbd_parse_filename(const char *filename, QDict *options,
|
||||
done:
|
||||
g_free(buf);
|
||||
qobject_unref(keypairs);
|
||||
return;
|
||||
}
|
||||
|
||||
static int qemu_rbd_set_auth(rados_t cluster, BlockdevOptionsRbd *opts,
|
||||
|
||||
@@ -176,7 +176,6 @@ static void replication_child_perm(BlockDriverState *bs, BdrvChild *c,
|
||||
*nshared = BLK_PERM_CONSISTENT_READ
|
||||
| BLK_PERM_WRITE
|
||||
| BLK_PERM_WRITE_UNCHANGED;
|
||||
return;
|
||||
}
|
||||
|
||||
static int64_t coroutine_fn GRAPH_RDLOCK
|
||||
|
||||
@@ -908,7 +908,6 @@ unlock:
|
||||
qemu_mutex_unlock(&tg->lock);
|
||||
qapi_free_ThrottleLimits(argp);
|
||||
error_propagate(errp, local_err);
|
||||
return;
|
||||
}
|
||||
|
||||
static void throttle_group_get_limits(Object *obj, Visitor *v,
|
||||
|
||||
@@ -441,7 +441,6 @@ void queue_signal(CPUArchState *env, int sig, int si_type,
|
||||
ts->sync_signal.pending = sig;
|
||||
/* Signal that a new signal is pending. */
|
||||
qatomic_set(&ts->signal_pending, 1);
|
||||
return;
|
||||
}
|
||||
|
||||
static int fatal_signal(int sig)
|
||||
|
||||
@@ -476,8 +476,6 @@ out_free:
|
||||
g_free(saved_ctx);
|
||||
out_cr3:
|
||||
first_x86_cpu->env.cr[3] = saved_cr3;
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
#else /* !TARGET_X86_64 */
|
||||
|
||||
@@ -73,8 +73,6 @@ static void event_loop_base_set_param(Object *obj, Visitor *v,
|
||||
if (bc->update_params) {
|
||||
bc->update_params(base, errp);
|
||||
}
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
static void event_loop_base_complete(UserCreatable *uc, Error **errp)
|
||||
|
||||
@@ -10,47 +10,39 @@ void acpi_switch_to_modern_cphp(AcpiCpuHotplug *gpe_cpu,
|
||||
CPUHotplugState *cpuhp_state,
|
||||
uint16_t io_port)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void legacy_acpi_cpu_hotplug_init(MemoryRegion *parent, Object *owner,
|
||||
AcpiCpuHotplug *gpe_cpu, uint16_t base)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void cpu_hotplug_hw_init(MemoryRegion *as, Object *owner,
|
||||
CPUHotplugState *state, hwaddr base_addr)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_cpu_ospm_status(CPUHotplugState *cpu_st, ACPIOSTInfoList ***list)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_cpu_plug_cb(HotplugHandler *hotplug_dev,
|
||||
CPUHotplugState *cpu_st, DeviceState *dev, Error **errp)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void legacy_acpi_cpu_plug_cb(HotplugHandler *hotplug_dev,
|
||||
AcpiCpuHotplug *g, DeviceState *dev, Error **errp)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_cpu_unplug_cb(CPUHotplugState *cpu_st,
|
||||
DeviceState *dev, Error **errp)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_cpu_unplug_request_cb(HotplugHandler *hotplug_dev,
|
||||
CPUHotplugState *cpu_st,
|
||||
DeviceState *dev, Error **errp)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
@@ -7,29 +7,24 @@ const VMStateDescription vmstate_memory_hotplug;
|
||||
void acpi_memory_hotplug_init(MemoryRegion *as, Object *owner,
|
||||
MemHotplugState *state, hwaddr io_base)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_memory_ospm_status(MemHotplugState *mem_st, ACPIOSTInfoList ***list)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_memory_plug_cb(HotplugHandler *hotplug_dev, MemHotplugState *mem_st,
|
||||
DeviceState *dev, Error **errp)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_memory_unplug_cb(MemHotplugState *mem_st,
|
||||
DeviceState *dev, Error **errp)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_memory_unplug_request_cb(HotplugHandler *hotplug_dev,
|
||||
MemHotplugState *mem_st,
|
||||
DeviceState *dev, Error **errp)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
@@ -4,5 +4,4 @@
|
||||
|
||||
void nvdimm_acpi_plug_cb(HotplugHandler *hotplug_dev, DeviceState *dev)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
@@ -7,37 +7,31 @@ const VMStateDescription vmstate_acpi_pcihp_pci_status;
|
||||
void acpi_pcihp_init(Object *owner, AcpiPciHpState *s, PCIBus *root_bus,
|
||||
MemoryRegion *address_space_io, uint16_t io_base)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_pcihp_device_plug_cb(HotplugHandler *hotplug_dev, AcpiPciHpState *s,
|
||||
DeviceState *dev, Error **errp)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_pcihp_device_pre_plug_cb(HotplugHandler *hotplug_dev,
|
||||
DeviceState *dev, Error **errp)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_pcihp_device_unplug_cb(HotplugHandler *hotplug_dev, AcpiPciHpState *s,
|
||||
DeviceState *dev, Error **errp)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_pcihp_device_unplug_request_cb(HotplugHandler *hotplug_dev,
|
||||
AcpiPciHpState *s, DeviceState *dev,
|
||||
Error **errp)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
void acpi_pcihp_reset(AcpiPciHpState *s)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
bool acpi_pcihp_is_hotpluggbale_bus(AcpiPciHpState *s, BusState *bus)
|
||||
|
||||
@@ -462,7 +462,6 @@ static uint64_t exynos4210_chipid_and_omr_read(void *opaque, hwaddr offset,
|
||||
static void exynos4210_chipid_and_omr_write(void *opaque, hwaddr offset,
|
||||
uint64_t value, unsigned size)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
static const MemoryRegionOps exynos4210_chipid_and_omr_ops = {
|
||||
|
||||
@@ -712,7 +712,6 @@ static void combine_tlb(SMMUTLBEntry *tlbe, SMMUTLBEntry *tlbe_s2,
|
||||
tlbe->entry.iova = iova & ~tlbe->entry.addr_mask;
|
||||
/* parent_perm has s2 perm while perm keeps s1 perm. */
|
||||
tlbe->parent_perm = tlbe_s2->entry.perm;
|
||||
return;
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
@@ -14,7 +14,6 @@
|
||||
void arch_handle_ioreq(XenIOState *state, ioreq_t *req)
|
||||
{
|
||||
hw_error("Invalid ioreq type 0x%x\n", req->type);
|
||||
return;
|
||||
}
|
||||
|
||||
void arch_xen_set_memory(XenIOState *state, MemoryRegionSection *section,
|
||||
|
||||
@@ -406,7 +406,6 @@ static void asc_fifo_write(void *opaque, hwaddr addr, uint64_t value,
|
||||
} else {
|
||||
fs->fifo[addr] = value;
|
||||
}
|
||||
return;
|
||||
}
|
||||
|
||||
static const MemoryRegionOps asc_fifo_ops = {
|
||||
|
||||
@@ -793,7 +793,6 @@ separator_error:
|
||||
error_setg(errp, "reserved region fields must be separated with ':'");
|
||||
out:
|
||||
g_free(str);
|
||||
return;
|
||||
}
|
||||
|
||||
const PropertyInfo qdev_prop_reserved_region = {
|
||||
|
||||
@@ -67,8 +67,6 @@ static void cxl_fixed_memory_window_config(CXLState *cxl_state,
|
||||
|
||||
cxl_state->fixed_windows = g_list_append(cxl_state->fixed_windows,
|
||||
g_steal_pointer(&fw));
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
void cxl_fmws_link_targets(CXLState *cxl_state, Error **errp)
|
||||
|
||||
@@ -383,7 +383,6 @@ static void macfb_sense_write(MacfbState *s, uint32_t val)
|
||||
s->regs[DAFB_MODE_SENSE >> 2] = val;
|
||||
|
||||
trace_macfb_sense_write(val);
|
||||
return;
|
||||
}
|
||||
|
||||
static void macfb_update_mode(MacfbState *s)
|
||||
|
||||
@@ -729,7 +729,6 @@ static uint64_t tcx_dummy_readl(void *opaque, hwaddr addr,
|
||||
static void tcx_dummy_writel(void *opaque, hwaddr addr,
|
||||
uint64_t val, unsigned size)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
static const MemoryRegionOps tcx_dummy_ops = {
|
||||
|
||||
Some files were not shown because too many files have changed in this diff Show More
Reference in New Issue
Block a user