[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 04/11] qemu_flush_work for remote vcpu execut
From: |
Marcelo Tosatti |
Subject: |
Re: [Qemu-devel] [PATCH v2 04/11] qemu_flush_work for remote vcpu execution |
Date: |
Wed, 2 Dec 2009 11:27:53 -0200 |
User-agent: |
Mutt/1.5.19 (2009-01-05) |
On Tue, Dec 01, 2009 at 10:51:30AM -0200, Glauber Costa wrote:
> This function is similar to qemu-kvm's on_vcpu mechanism. Totally synchronous,
> and guarantees that a given function will be executed at the specified vcpu.
>
> This patch also convert usage within the breakpoints system
>
> Signed-off-by: Glauber Costa <address@hidden>
> ---
> @@ -3436,8 +3441,7 @@ static int tcg_has_work(void);
>
> static pthread_key_t current_env;
>
> -CPUState *qemu_get_current_env(void);
> -CPUState *qemu_get_current_env(void)
> +static CPUState *qemu_get_current_env(void)
> {
> return pthread_getspecific(current_env);
> }
> @@ -3474,8 +3478,10 @@ static int qemu_init_main_loop(void)
>
> static void qemu_wait_io_event(CPUState *env)
> {
> - while (!tcg_has_work())
> + while (!tcg_has_work()) {
This checks all cpus, while for KVM it should check only
the current cpu.
> + qemu_flush_work(env);
> qemu_cond_timedwait(env->halt_cond, &qemu_global_mutex, 1000);
> + }
KVM vcpu threads should block SIGUSR1, set the in-kernel signal mask
with KVM_SET_SIGNAL_MASK ioctl, and eat the signal in
qemu_wait_io_event (qemu_flush_work should run after eating
the signal). Similarly to qemu-kvm's kvm_main_loop_wait.
Otherwise a vcpu thread can lose a signal (say handle SIGUSR1 when not
holding qemu_global_mutex before kernel entry).
I think this the source of the problems patch 8 attempts to fix.
- [Qemu-devel] [PATCH v2 07/11] Don't call kvm cpu reset on initialization, (continued)
- [Qemu-devel] [PATCH v2 07/11] Don't call kvm cpu reset on initialization, Glauber Costa, 2009/12/01
- [Qemu-devel] [PATCH v2 08/11] use cpu_kick instead of direct signalling., Glauber Costa, 2009/12/01
- [Qemu-devel] [PATCH v2 09/11] Use per-cpu reset handlers., Glauber Costa, 2009/12/01
- [Qemu-devel] [PATCH v2 10/11] Use __thread where available., Glauber Costa, 2009/12/01
- [Qemu-devel] [PATCH v2 11/11] remove smp restriction from kvm, Glauber Costa, 2009/12/01
- Re: [Qemu-devel] [PATCH v2 05/11] tell kernel about all registers instead of just mp_state, Gleb Natapov, 2009/12/02
- Re: [Qemu-devel] [PATCH v2 05/11] tell kernel about all registers instead of just mp_state, Glauber Costa, 2009/12/02
- [Qemu-devel] Re: [PATCH v2 05/11] tell kernel about all registers instead of just mp_state, Jan Kiszka, 2009/12/02
- [Qemu-devel] Re: [PATCH v2 05/11] tell kernel about all registers instead of just mp_state, Gleb Natapov, 2009/12/02
- [Qemu-devel] Re: [PATCH v2 05/11] tell kernel about all registers instead of just mp_state, Jan Kiszka, 2009/12/02
- Re: [Qemu-devel] [PATCH v2 04/11] qemu_flush_work for remote vcpu execution,
Marcelo Tosatti <=
- Re: [Qemu-devel] [PATCH v2 04/11] qemu_flush_work for remote vcpu execution, Glauber Costa, 2009/12/02
- Re: [Qemu-devel] [PATCH v2 04/11] qemu_flush_work for remote vcpu execution, Marcelo Tosatti, 2009/12/02
- Re: [Qemu-devel] [PATCH v2 04/11] qemu_flush_work for remote vcpu execution, Glauber Costa, 2009/12/02
Re: [Qemu-devel] [PATCH v2 0/11], Gleb Natapov, 2009/12/02