On 03/03/2016 13:33, Andrew Jones wrote: > 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(-) > Applied, thanks. Paolo -- 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