On Thu, Dec 14, 2023 at 02:08:48PM +0800, xianglai li wrote:
> +++ b/src/util/virhostcpu.c
> @@ -579,7 +579,7 @@ virHostCPUParsePhysAddrSize(FILE *cpuinfo, unsigned int *addrsz)
> char *str;
> char *endptr;
>
> - if (!(str = STRSKIP(line, "address sizes")))
> + if (!(str = STRCASESKIP(line, "address sizes")))
> continue;
So is the case different on loongarch than it is on other
architectures? Weird.
> +++ b/src/util/virsysinfo.c
> @@ -1241,14 +1241,15 @@ virSysinfoRead(void)
> {
> #if defined(__powerpc__)
> return virSysinfoReadPPC();
> -#elif defined(__arm__) || defined(__aarch64__)
> +#elif defined(__arm__) || defined(__aarch64__) || defined(__loongarch__)
> return virSysinfoReadARM();
This is definitely not right: we shouldn't be calling the
Arm-specific function on loongarch.
> #elif defined(__s390__) || defined(__s390x__)
> return virSysinfoReadS390();
> #elif !defined(WIN32) && \
> (defined(__x86_64__) || \
> defined(__i386__) || \
> - defined(__amd64__))
> + defined(__amd64__) || \
> + defined(__loongarch__))
> return virSysinfoReadDMI();
Does loongarch actually have DMI support?
--
Andrea Bolognani / Red Hat / Virtualization
_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx
