Re: [libvirt] [PATCH] tests: Only format the CPU frequency if it's known
On Mon, Jan 08, 2018 at 15:10:29 +0100, Andrea Bolognani wrote:
Instead of formatting 'MHz: 0', which can be confusing, skip
the
field altogether. This behavior is consistent with that of 'virsh
nodeinfo'.
Suggested-by: John Ferlan <jferlan(a)redhat.com>
Signed-off-by: Andrea Bolognani <abologna(a)redhat.com>
---
.../linux-aarch64-f21-mustang.expected | 2 +-
.../linux-aarch64-rhel74-moonshot.expected | 2 +-
.../linux-aarch64-rhelsa-3.19.0-mustang.expected | 2 +-
.../linux-armv6l-raspberrypi.expected | 2 +-
tests/virhostcputest.c | 28 +++++++++++++++++-----
5 files changed, 26 insertions(+), 10 deletions(-)
Well, these are tests, so confusion really should not be a problem.
Formatting all the values unconditionally has a benefit that you don't
have to look at the code to see when some are formatted, but rather know
the raw value instead.
I'd suggest to not push this.
Attachments:
- signature.asc (application/pgp-signature — 833 bytes)