From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrey Smetanin Subject: Re: [PATCH v1 0/5] KVM-UNIT-TESTS: Hyper-V SynIC timers test Date: Fri, 27 Nov 2015 14:31:33 +0300 Message-ID: <56583F15.8030900@virtuozzo.com> References: <1448555397-29150-1-git-send-email-asmetanin@virtuozzo.com> <56583BC4.9070108@redhat.com> Reply-To: asmetanin@virtuozzo.com Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Cc: "Denis V. Lunev" , Marcelo Tosatti , Roman Kagan , qemu-devel@nongnu.org To: Paolo Bonzini , Return-path: In-Reply-To: <56583BC4.9070108@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org Sender: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org List-Id: kvm.vger.kernel.org On 11/27/2015 02:17 PM, Paolo Bonzini wrote: > > > On 26/11/2015 17:29, Andrey Smetanin wrote: >> The test checks Hyper-V SynIC timers functionality. >> The test runs on every vCPU and performs start/stop >> of periodic/one-shot timers (with period=1ms) and checks >> validity of received expiration messages in appropriate >> ISR's. >> >> Signed-off-by: Andrey Smetanin >> Reviewed-by: Roman Kagan >> CC: Paolo Bonzini >> CC: Marcelo Tosatti >> CC: Roman Kagan >> CC: Denis V. Lunev >> CC: qemu-devel@nongnu.org >> >> Andrey Smetanin (5): >> lib/x86: Added Hyper-V MSR's availability bits into msr.h >> lib/x86: Added HV_X64_MSR_TIME_REF_COUNT value into msr.h >> lib/x86: Added Hyper-V SynIC timers MSR's values >> lib/x86: Make free_page() available to call >> x86: Hyper-V SynIC timers test > > In addition to my comments on 5/5, can you make instead a hyperv.h file > with all that you need in the two testcases? Yes, I'll do it. Thanks! > > Paolo > >> config/config-x86-common.mak | 4 +- >> lib/x86/msr.h | 19 ++ >> lib/x86/vm.h | 1 + >> x86/hyperv_stimer.c | 500 +++++++++++++++++++++++++++++++++++++++++++ >> x86/hyperv_synic.c | 2 +- >> x86/unittests.cfg | 5 + >> 6 files changed, 529 insertions(+), 2 deletions(-) >> create mode 100644 x86/hyperv_stimer.c >>