[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 4/7] give noaccel a name
From: |
Glauber Costa |
Subject: |
[Qemu-devel] Re: [PATCH 4/7] give noaccel a name |
Date: |
Tue, 14 Oct 2008 14:04:52 -0200 |
User-agent: |
Mutt/1.5.18 (2008-05-17) |
On Tue, Oct 14, 2008 at 06:00:36PM +0200, Avi Kivity wrote:
> Glauber Costa wrote:
> > otherwise, string comparisons to check for the current accelerator
> > may break.
> >
> > Signed-off-by: Glauber Costa <address@hidden>
> > ---
> > accel.c | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/accel.c b/accel.c
> > index f9618b2..bf3cb73 100644
> > --- a/accel.c
> > +++ b/accel.c
> > @@ -21,7 +21,7 @@ CPUState *noaccel_get_env(void)
> >
> > /* Accelerator wrapper for the no-accel (raw qemu) case */
> > QEMUAccel noaccel = {
> > - .name = NULL,
> > + .name = "noaccel",
> > .cpu_interrupt = accel_nop,
> > .init_env = accel_nop,
> > .get_env = noaccel_get_env,
> >
>
> Maybe "none" or "tcg" or "emulation"?
I think none is okay.
>
> --
> Do not meddle in the internals of kernels, for they are subtle and quick to
> panic.
>
- [Qemu-devel] [PATCH 0/7] Update qemu accel, Glauber Costa, 2008/10/13
- [Qemu-devel] [PATCH 1/7] protect accel.h from multiple inclusion, Glauber Costa, 2008/10/13
- [Qemu-devel] [PATCH 2/7] use accel_trace_io in calling sites., Glauber Costa, 2008/10/13
- [Qemu-devel] [PATCH 4/7] give noaccel a name, Glauber Costa, 2008/10/13
- [Qemu-devel] [PATCH 3/7] use break_loop accel, Glauber Costa, 2008/10/13
- [Qemu-devel] [PATCH 5/7] check wether kqemu is enabled in open code, Glauber Costa, 2008/10/13
- [Qemu-devel] [PATCH 6/7] provide an opaque for accelerator in cpu state, Glauber Costa, 2008/10/13
- [Qemu-devel] [PATCH 7/7] Revert "don't allocate ram before accel starts", Glauber Costa, 2008/10/13
- [Qemu-devel] Re: [PATCH 0/7] Update qemu accel, Anthony Liguori, 2008/10/14