From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1753081AbdJSNlJ (ORCPT <rfc822;w@1wt.eu>);
        Thu, 19 Oct 2017 09:41:09 -0400
Received: from userp1040.oracle.com ([156.151.31.81]:49080 "EHLO
        userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1752118AbdJSNlH (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Thu, 19 Oct 2017 09:41:07 -0400
Subject: Re: [PATCH v6 1/4] x86/pvclock: add setter for pvclock_pvti_cpu0_va
To: kvm@vger.kernel.org, Paolo Bonzini <pbonzini@redhat.com>,
        Radim Krcmar <rkrcmar@redhat.com>
References: <20171003115531.22546-1-joao.m.martins@oracle.com>
 <20171003115531.22546-2-joao.m.martins@oracle.com>
 <7e9a38a6-63ae-6ed7-7c6c-6fac19789268@oracle.com>
Cc: linux-kernel@vger.kernel.org, Thomas Gleixner <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>, "H. Peter Anvin" <hpa@zytor.com>,
        x86@kernel.org, Andy Lutomirski <luto@amacapital.net>,
        xen-devel <xen-devel@lists.xenproject.org>,
        Juergen Gross <JGross@suse.com>,
        Boris Ostrovsky <boris.ostrovsky@oracle.com>
From: Joao Martins <joao.m.martins@oracle.com>
Message-ID: <fb896697-8f78-d7d1-ca52-4fdb22f3ac6c@oracle.com>
Date: Thu, 19 Oct 2017 14:40:15 +0100
MIME-Version: 1.0
In-Reply-To: <7e9a38a6-63ae-6ed7-7c6c-6fac19789268@oracle.com>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
X-Source-IP: aserv0021.oracle.com [141.146.126.233]
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 10/17/2017 04:34 PM, Joao Martins wrote:
> On 10/03/2017 12:55 PM, Joao Martins wrote:
>> Right now there is only a pvclock_pvti_cpu0_va() which is defined
>> on kvmclock since:
>>
>> commit dac16fba6fc5
>> ("x86/vdso: Get pvclock data from the vvar VMA instead of the fixmap")
>>
>> The only user of this interface so far is kvm. This commit adds a
>> setter function for the pvti page and moves pvclock_pvti_cpu0_va
>> to pvclock, which is a more generic place to have it; and would
>> allow other PV clocksources to use it, such as Xen.
>>
>> Signed-off-by: Joao Martins <joao.m.martins@oracle.com>
>> Acked-by: Andy Lutomirski <luto@kernel.org>
> 
> Ping?
> 
> While the rest of series has been acked, I think that this patch (per
> maintainers file) still misses x86 and (or?) kvm ack/review.

I found out an issue with ptp_kvm modinit (if attempted to be loaded) under Xen
related to this series, so I resent with that fixed. Hopefully things can be
taken from there - Sorry for the noise.

Thanks,
Joao