Re: [kvm-unit-tests PATCH v2 0/2] x86: apic: add LVTT test

[Andrew Jones <drjones@redhat.com>]

On Wed, Sep 21, 2016 at 01:09:40PM +0800, Peter Xu wrote:
> 1st patch fixes not eoi()ing in existing deadline timer isr.
> 
> 2nd patch adds lvtt test. Please review. Thanks!
> 
> v2
> - adding eoi() in LVT timer handler, otherwise continuous interrupt
>   will be ignored (found this when testing together with deadline
>   timer)
> - better comments for "initial" [Drew]
> - check lvtt_counter==1 in report() [Drew]
> - setting x86/apic timeout to 30s [Drew]
> 
> Peter Xu (2):
>   x86: apic: eoi() for deadline timer isr
>   x86: apic: add LVT timer test
> 
>  x86/apic.c        | 44 ++++++++++++++++++++++++++++++++++++++++++++
>  x86/unittests.cfg |  1 +
>  2 files changed, 45 insertions(+)
> 
> -- 
> 2.7.4

Both patches look good to me.

Thanks,
drew