[kvm-unit-tests PATCH v3 0/4] add fmt warnings to printf, and fix bad uses

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thomas suggested adding __attribute__((format(printf, i, j))) to
our *printf functions. So I did. And then fixed all the warnings...

v3:
 * another lu in desc.c [Thomas]
 * handle phys_addr_t := u32 in alloc.c [Thomas]
v2:
 * don't use the temporary _f() macro the attributes [Paolo]
 * changes from Thomas' review comments
    - desc.c: use %lu instead of %ld
    - msr.c: don't screw up the int:int output of a longlong
    - taskswitch2.c: remove useless (ulong*) casts


Andrew Jones (4):
  lib: *printf: warn on format/arg mismatch
  lib/alloc: fix format warnings, add/use PRIx64
  arm64: fix printf format warnings
  x86: fix printf format warnings

 lib/alloc.c               | 25 ++++++++++++++-----------
 lib/arm64/processor.c     |  8 ++++----
 lib/libcflat.h            | 26 +++++++++++++++++++++-----
 lib/x86/desc.c            |  6 +++---
 lib/x86/vm.c              |  6 +++---
 x86/access.c              |  4 ++--
 x86/eventinj.c            |  8 ++++----
 x86/kvmclock.c            |  2 +-
 x86/kvmclock_test.c       |  6 +++---
 x86/msr.c                 |  3 ++-
 x86/s3.c                  |  6 +++---
 x86/svm.c                 | 12 ++++++------
 x86/taskswitch.c          |  2 +-
 x86/taskswitch2.c         |  4 ++--
 x86/tsc.c                 |  4 ++--
 x86/tscdeadline_latency.c |  2 +-
 x86/vmx.c                 | 18 +++++++++---------
 x86/vmx_tests.c           | 20 ++++++++++----------
 x86/xsave.c               |  2 +-
 19 files changed, 92 insertions(+), 72 deletions(-)

-- 
2.4.3

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux