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

[Paolo Bonzini <pbonzini@redhat.com>]

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