On Thu, Mar 03, 2016 at 10:57:54AM +0100, Thomas Huth wrote:
> 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:
Will fix for v3.
Thanks,
drew
>
> 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
--
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
