Thomas suggested adding __attribute__((format(printf, i, j))) to our *printf functions. So I did. And then fixed all the warnings... 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 | 13 +++++++------ 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, 85 insertions(+), 67 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