From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60870) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a2H1t-0000dx-Km for qemu-devel@nongnu.org; Fri, 27 Nov 2015 06:17:30 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a2H1q-000868-TJ for qemu-devel@nongnu.org; Fri, 27 Nov 2015 06:17:29 -0500 Received: from mail-wm0-x232.google.com ([2a00:1450:400c:c09::232]:36499) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a2H1q-000863-Mq for qemu-devel@nongnu.org; Fri, 27 Nov 2015 06:17:26 -0500 Received: by wmww144 with SMTP id w144so51594240wmw.1 for ; Fri, 27 Nov 2015 03:17:26 -0800 (PST) Sender: Paolo Bonzini References: <1448555397-29150-1-git-send-email-asmetanin@virtuozzo.com> From: Paolo Bonzini Message-ID: <56583BC4.9070108@redhat.com> Date: Fri, 27 Nov 2015 12:17:24 +0100 MIME-Version: 1.0 In-Reply-To: <1448555397-29150-1-git-send-email-asmetanin@virtuozzo.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH v1 0/5] KVM-UNIT-TESTS: Hyper-V SynIC timers test List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Andrey Smetanin , kvm@vger.kernel.org Cc: "Denis V. Lunev" , Marcelo Tosatti , Roman Kagan , qemu-devel@nongnu.org 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? 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 >