On 02.03.2016 19:07, Andrew Jones wrote:
> Signed-off-by: Andrew Jones <drjones@xxxxxxxxxx>
> ---
> 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 +-
> 16 files changed, 53 insertions(+), 52 deletions(-)
>
> diff --git a/lib/x86/desc.c b/lib/x86/desc.c
> index 1a80c01283853..12bcb57a42d1d 100644
> --- a/lib/x86/desc.c
> +++ b/lib/x86/desc.c
> @@ -53,7 +53,7 @@ static void check_exception_table(struct ex_regs *regs)
> return;
> }
> }
> - printf("unhandled exception %d\n", regs->vector);
> + printf("unhandled exception %lu\n", regs->vector);
> exit(7);
> }
>
> @@ -75,9 +75,9 @@ void do_handle_exception(struct ex_regs *regs)
> exception_handlers[regs->vector](regs);
> return;
> }
> - printf("unhandled cpu exception %d\n", regs->vector);
> + printf("unhandled cpu exception %ld\n", regs->vector);
Nit: That could be %lu here, too, as far as I can see.
Anyway, the remaining parts of the patch look fine, and it's really a
very small nit, so:
Reviewed-by: Thomas Huth <thuth@xxxxxxxxxx>
--
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
