[libvirt] [PATCH] ARMHF: Fixed Parser for /proc/cpuinfo needs to be adapted for your architecture
Eric Blake
eblake at redhat.com
Thu Sep 27 23:40:40 UTC 2012
On 07/26/2012 07:32 AM, Chuck Short wrote:
Eeek - I let two months go by without reviewing this. My apologies.
> Minimal CPU "parser" for armhf to avoid compile time warning.
>
> Signed-off-by: Chuck Short <chuck.short at canonical.com>
> ---
> src/nodeinfo.c | 21 +++++++++++++++++++++
> 1 file changed, 21 insertions(+)
ACK, and will push shortly once I validate that it passes tests.
>
> diff --git a/src/nodeinfo.c b/src/nodeinfo.c
> index 84a5d66..358fa3b 100644
> --- a/src/nodeinfo.c
> +++ b/src/nodeinfo.c
> @@ -397,6 +397,27 @@ int linuxNodeInfoCPUPopulate(FILE *cpuinfo,
> * and parsed in next iteration, because it is not in expected
> * format and thus lead to error. */
> }
> +# elif defined(__arm__)
> + char *buf = line;
> + if (STRPREFIX(buf, "BogoMIPS")) {
> + char *p;
> + unsigned int ui;
> +
> + buf += 8;
> + while (*buf && c_isspace(*buf))
> + buf++;
> +
> + if (*buf != ':' || !buf[1]) {
> + nodeReportError(VIR_ERR_INTERNAL_ERROR,
> + "%s", _("parsing cpu MHz from cpuinfo"));
> + goto cleanup;
> + }
> +
> + if (virStrToLong_ui(buf+1, &p, 10, &ui) == 0
> + /* Accept trailing fractional part. */
> + && (*p == '\0' || *p == '.' || c_isspace(*p)))
> + nodeinfo->mhz = ui;
> + }
> # elif defined(__s390__) || \
> defined(__s390x__)
> /* s390x has no realistic value for CPU speed,
> --
> 1.7.10.4
>
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
>
>
--
Eric Blake eblake at redhat.com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 617 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120927/79090e8a/attachment-0001.sig>
More information about the libvir-list
mailing list