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@lists.libvirt.org
To unsubscribe send an email to devel-le...@lists.libvirt.org

Reply via email to