From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752861AbdKIUUb (ORCPT ); Thu, 9 Nov 2017 15:20:31 -0500 Received: from aserp1040.oracle.com ([141.146.126.69]:35396 "EHLO aserp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752231AbdKIUUa (ORCPT ); Thu, 9 Nov 2017 15:20:30 -0500 Subject: Re: [PATCH v8 0/5] x86/xen: pvclock vdso support To: Joao Martins , linux-kernel@vger.kernel.org, xen-devel@lists.xenproject.org References: <20171108171958.23921-1-joao.m.martins@oracle.com> Cc: Juergen Gross , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org, Andy Lutomirski From: Boris Ostrovsky Message-ID: Date: Thu, 9 Nov 2017 15:21:36 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <20171108171958.23921-1-joao.m.martins@oracle.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Source-IP: aserv0021.oracle.com [141.146.126.233] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/08/2017 12:19 PM, Joao Martins wrote: > Hey, > > This is take 8 for vdso for Xen. PVCLOCK_TSC_STABLE_BIT can be set starting Xen > 4.8 which is required for vdso time related calls. In order to have it on, you > need to have the hypervisor clocksource be TSC e.g. with the following boot > params "clocksource=tsc tsc=stable:socket". > > Series is structured as following: > > Patch 1 probes for kvm guest in ptp_kvm in the event having > pvclock_pvti_cpu0_va() moved to common pvclock (on the next patch) > Patch 2 streamlines pvti page get/set in pvclock for both of its users > Patch 3,4 registers the pvti page on Xen and sets it in pvclock accordingly > Patch 5 adds a file to KVM/Xen maintainers for tracking pvclock ABI changes. > > All patches appear to be Acked by its respective maintainers. > > The difference to v7 is adding the Acks on patches 1 and 2 plus the adjustment > from Thomas to rename the getter function. (Changelog in individual patches) > > Thanks, > Joao > > Joao Martins (5): > ptp_kvm: probe for kvm guest availability > x86/pvclock: add setter for pvclock_pvti_cpu0_va > x86/xen/time: set pvclock flags on xen_time_init() > x86/xen/time: setup vcpu 0 time info page > MAINTAINERS: xen, kvm: track pvclock-abi.h changes > > MAINTAINERS | 2 + > arch/x86/entry/vdso/vma.c | 2 +- > arch/x86/include/asm/pvclock.h | 19 +++++---- > arch/x86/kernel/kvmclock.c | 7 +-- > arch/x86/kernel/pvclock.c | 14 ++++++ > arch/x86/xen/suspend.c | 4 ++ > arch/x86/xen/time.c | 97 ++++++++++++++++++++++++++++++++++++++++++ > arch/x86/xen/xen-ops.h | 2 + > drivers/ptp/ptp_kvm.c | 5 ++- > include/xen/interface/vcpu.h | 42 ++++++++++++++++++ > 10 files changed, 177 insertions(+), 17 deletions(-) > Applied to for-linus-4.15. -boris