qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [patch 01/10] qemu: create helper for event notificati


From: Glauber Costa
Subject: Re: [Qemu-devel] [patch 01/10] qemu: create helper for event notification
Date: Wed, 25 Mar 2009 20:18:58 -0300

> +void qemu_notify_event(void)
> +{
> +    CPUState *env = cpu_single_env;
> +
> +    if (env) {
> +        cpu_exit(env);
> +#ifdef USE_KQEMU
> +        if (env->kqemu_enabled)
> +            kqemu_cpu_interrupt(env);
> +#endif
> +     }
>  }

Have you tested this with kqemu? It will fire kqemu interrupts in a
lot of circunstances
it didn't before. That said, I don't even think kqemu is that
important, but if it's in the tree,
we don't want to introduce regressions. I'd suggest passing a flag
indicating the type of
event we're notifying. It's easy to foresee other uses for that as well.



-- 
Glauber  Costa.
"Free as in Freedom"
http://glommer.net

"The less confident you are, the more serious you have to act."




reply via email to

[Prev in Thread] Current Thread [Next in Thread]