Skip to content

Commit

Permalink
qemu-common.h: Make qemu_init_vcpu() stub static inline
Browse files Browse the repository at this point in the history
Turn the *-user macro into a no-op inline function to avoid
unused-variable warnings and band-aiding #ifdef'ery.

This allows to drop an #ifdef for alpha and avoids more for unicore32
and other upcoming trivial realizefn implementations.

Suggested-by: Lluís Vilanova <vilanova@ac.upc.edu>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
Reviewed-by: Eduardo Habkost <ehabkost@redhat.com>
  • Loading branch information
afaerber committed Jan 8, 2013
1 parent e8beac0 commit 75a192a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 3 additions & 1 deletion include/qemu-common.h
Expand Up @@ -288,7 +288,9 @@ struct qemu_work_item {
};

#ifdef CONFIG_USER_ONLY
#define qemu_init_vcpu(env) do { } while (0)
static inline void qemu_init_vcpu(void *env)
{
}
#else
void qemu_init_vcpu(void *env);
#endif
Expand Down
2 changes: 0 additions & 2 deletions target-alpha/cpu.c
Expand Up @@ -26,11 +26,9 @@

static void alpha_cpu_realize(Object *obj, Error **errp)
{
#ifndef CONFIG_USER_ONLY
AlphaCPU *cpu = ALPHA_CPU(obj);

qemu_init_vcpu(&cpu->env);
#endif
}

/* Sort alphabetically by type name. */
Expand Down

0 comments on commit 75a192a

Please sign in to comment.