[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 16/17] target-i386: set custom 'tsc-frequency' w
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH 16/17] target-i386: set custom 'tsc-frequency' without intermediate x86_def_t |
Date: |
Mon, 14 Jan 2013 16:49:23 +0100 |
On Mon, 14 Jan 2013 13:20:44 -0200
Eduardo Habkost <address@hidden> wrote:
> On Fri, Jan 11, 2013 at 03:10:30AM +0100, Igor Mammedov wrote:
> > Signed-off-by: Igor Mammedov <address@hidden>
>
> Reviewed-by: Eduardo Habkost <address@hidden>
>
> Will this be converted to a simple qdict_put() of the raw string value,
> to be parsed using visit_type_freq() (or something equivalent), in the
> future? It would be interesting to keep the ability to use
> "tsc-frequency=1GHz" on device_add/-device.
Anthony was opposed to idea of special visitor.
This patch allow us to move forward and in easily convert to visitor
later if there will be agreement to adding new visitor.
On the other hand we could leave it as it done here and enforce users to
provide numeric value for "tsc-frequency" in via device_add interface
while maintaining compatibility for legacy 'tsc_freq'.
>
>
> > ---
> > v2:
> > make patch independent of visit_type_freq()
> > ---
> > target-i386/cpu.c | 5 ++++-
> > 1 files changed, 4 insertions(+), 1 deletions(-)
> >
> > diff --git a/target-i386/cpu.c b/target-i386/cpu.c
> > index ec27cf6..c3e1792 100644
> > --- a/target-i386/cpu.c
> > +++ b/target-i386/cpu.c
> > @@ -1371,6 +1371,7 @@ static int cpu_x86_parse_featurestr(x86_def_t
> > *x86_cpu_def, char *features, } else if (!strcmp(featurestr, "tsc_freq"))
> > { int64_t tsc_freq;
> > char *err;
> > + QString *s;
> >
> > tsc_freq = strtosz_suffix_unit(val, &err,
> > STRTOSZ_DEFSUFFIX_B,
> > 1000); @@ -1378,7 +1379,9 @@ static int
> > cpu_x86_parse_featurestr(x86_def_t *x86_cpu_def, char *features,
> > fprintf(stderr, "bad numerical value %s\n", val); goto error;
> > }
> > - x86_cpu_def->tsc_khz = tsc_freq / 1000;
> > + s = qstring_new();
> > + qstring_append_int(s, tsc_freq);
> > + qdict_put(*props, "tsc-frequency", s);
> > } else if (!strcmp(featurestr, "hv_spinlocks")) {
> > char *err;
> > numvalue = strtoul(val, &err, 0);
> > --
> > 1.7.1
> >
> >
>
- [Qemu-devel] [PATCH 11/17] target-i386: set custom 'level' without intermediate x86_def_t, (continued)
- [Qemu-devel] [PATCH 11/17] target-i386: set custom 'level' without intermediate x86_def_t, Igor Mammedov, 2013/01/10
- [Qemu-devel] [PATCH 12/17] target-i386: set custom 'model-id' without intermediate x86_def_t, Igor Mammedov, 2013/01/10
- [Qemu-devel] [PATCH 09/17] target-i386: print deprecated warning if xlevel < 0x80000000, Igor Mammedov, 2013/01/10
- [Qemu-devel] [PATCH 13/17] target-i386: set custom 'stepping' without intermediate x86_def_t, Igor Mammedov, 2013/01/10
- [Qemu-devel] [PATCH 17/17] target-i386: remove setting tsc-frequency from x86_def_t, Igor Mammedov, 2013/01/10
- [Qemu-devel] [PATCH 14/17] target-i386: set custom 'model' without intermediate x86_def_t, Igor Mammedov, 2013/01/10
- [Qemu-devel] [PATCH 16/17] target-i386: set custom 'tsc-frequency' without intermediate x86_def_t, Igor Mammedov, 2013/01/10
- [Qemu-devel] [PATCH 15/17] target-i386: set custom 'family' without intermediate x86_def_t, Igor Mammedov, 2013/01/10
- Re: [Qemu-devel] [PATCH qom-cpu 00/17] x86 CPU cleanup, part 3, Andreas Färber, 2013/01/14