On Wed, May 13, 2015 at 01:55:43PM +0200, Michal Privoznik wrote: > Due to a kernel commit (b4b8f770e), cpuinfo format has changed on > ARMs. Firstly, 'Processor: ...' may not be reported, it's > replaced by 'model name: ...'. Secondly, the "Processor" string > may occur in CPU name, e.g. 'ARMv7 Processor rev 5 (v7l)'. > Therefore, we must firstly look for 'model name' and then for > 'Processor' if not found. > Moreover, lines in the cpuinfo file are shuffled, so we better > not manipulate the pointer to start of internal buffer as we may > lost some info. > > Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> > --- > > diff to v1: > - Jan's comments worked in > - added new test case (yes, there really is a space at EOL in cpuinfo on my ARM) > > src/util/virsysinfo.c | 9 +++---- > tests/sysinfodata/arm-rpi2cpuinfo.data | 43 ++++++++++++++++++++++++++++++++ > tests/sysinfodata/arm-rpi2sysinfo.expect | 18 +++++++++++++ > tests/sysinfotest.c | 22 ++++++++++++---- > 4 files changed, 82 insertions(+), 10 deletions(-) > create mode 100644 tests/sysinfodata/arm-rpi2cpuinfo.data > create mode 100644 tests/sysinfodata/arm-rpi2sysinfo.expect > ACK Jan
Attachment:
signature.asc
Description: Digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list