From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36541) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a2HFn-0000u2-2u for qemu-devel@nongnu.org; Fri, 27 Nov 2015 06:31:54 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a2HFk-0004fO-Cs for qemu-devel@nongnu.org; Fri, 27 Nov 2015 06:31:51 -0500 Received: from relay.parallels.com ([195.214.232.42]:57051) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a2HFk-0004fE-54 for qemu-devel@nongnu.org; Fri, 27 Nov 2015 06:31:48 -0500 References: <1448555397-29150-1-git-send-email-asmetanin@virtuozzo.com> <56583BC4.9070108@redhat.com> From: Andrey Smetanin Message-ID: <56583F15.8030900@virtuozzo.com> Date: Fri, 27 Nov 2015 14:31:33 +0300 MIME-Version: 1.0 In-Reply-To: <56583BC4.9070108@redhat.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v1 0/5] KVM-UNIT-TESTS: Hyper-V SynIC timers test Reply-To: asmetanin@virtuozzo.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini , kvm@vger.kernel.org Cc: "Denis V. Lunev" , Marcelo Tosatti , Roman Kagan , qemu-devel@nongnu.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 >>