[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PV
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR |
Date: |
Fri, 25 Aug 2017 14:16:44 +1000 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Thu, Aug 24, 2017 at 10:21:50AM +0200, Igor Mammedov wrote:
> Signed-off-by: Igor Mammedov <address@hidden>
> ---
> target/ppc/translate_init.c | 27 +++++++++++----------------
> 1 file changed, 11 insertions(+), 16 deletions(-)
>
> diff --git a/target/ppc/translate_init.c b/target/ppc/translate_init.c
> index f1a559d..ca9f1e3 100644
> --- a/target/ppc/translate_init.c
> +++ b/target/ppc/translate_init.c
> @@ -34,6 +34,7 @@
> #include "hw/ppc/ppc.h"
> #include "mmu-book3s-v3.h"
> #include "sysemu/qtest.h"
> +#include "qemu/cutils.h"
>
> //#define PPC_DUMP_CPU
> //#define PPC_DEBUG_SPR
> @@ -10203,22 +10204,16 @@ static ObjectClass *ppc_cpu_class_by_name(const
> char *name)
> char *cpu_model, *typename;
> ObjectClass *oc;
> const char *p;
> - int i, len;
> -
> - /* Check if the given name is a PVR */
> - len = strlen(name);
> - if (len == 10 && name[0] == '0' && name[1] == 'x') {
> - p = name + 2;
> - goto check_pvr;
> - } else if (len == 8) {
> - p = name;
> - check_pvr:
> - for (i = 0; i < 8; i++) {
> - if (!qemu_isxdigit(*p++))
> - break;
> - }
> - if (i == 8) {
> - return OBJECT_CLASS(ppc_cpu_class_by_pvr(strtoul(name, NULL,
> 16)));
> + unsigned long pvr;
> +
> + /* Lookup by PVR if cpu_model is valid 8 digit hex number
> + * (excl: 0x prefix if present)
> + */
> + if (!qemu_strtoul(name, &p, 16, &pvr)) {
> + int len = p - name;
> + len = (len == 10) && (name[1] == 'x') ? len - 2 : len;
> + if (len == 8) {
> + return OBJECT_CLASS(ppc_cpu_class_by_pvr(pvr));
This doesn't look quite right. A hex string of a PVR followed by
other stuff isn't a valid option, so if p (endptr) doesn't point to
the end of the string, then we shouldn't be using this path (from the
looks of qemu_strtoul() we can simply omit the endptr parameter to get
that behaviour). I'm not sure what the messing around with len is
for, either. If it's a valid hex string we can try this, I don't see
that we need further checks.
> }
> }
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- [Qemu-ppc] [PATCH for-2.11 0/6] ppc: cpu_model handling cleanups, Igor Mammedov, 2017/08/24
- [Qemu-ppc] [PATCH for-2.11 3/6] ppc: make cpu alias point only to real cpu models, Igor Mammedov, 2017/08/24
- [Qemu-ppc] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, Igor Mammedov, 2017/08/24
- Re: [Qemu-ppc] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR,
David Gibson <=
- Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, Igor Mammedov, 2017/08/25
- Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, David Gibson, 2017/08/25
- Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, Igor Mammedov, 2017/08/25
- Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, Igor Mammedov, 2017/08/30
- Re: [Qemu-ppc] [Qemu-devel] [PATCH for-2.11 5/6] ppc: simplify cpu model lookup by PVR, David Gibson, 2017/08/31
[Qemu-ppc] [PATCH for-2.11 4/6] ppc: replace inter-function cyclic dependency/recurssion with 2 simple lookups, Igor Mammedov, 2017/08/24