From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752186AbbKJMcr (ORCPT ); Tue, 10 Nov 2015 07:32:47 -0500 Received: from mout.kundenserver.de ([212.227.126.130]:62905 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751027AbbKJMcq (ORCPT ); Tue, 10 Nov 2015 07:32:46 -0500 From: Arnd Bergmann To: linux-arm-kernel@lists.infradead.org Cc: Stefano Stabellini , xen-devel@lists.xensource.com, Ian.Campbell@citrix.com, konrad.wilk@oracle.com, linux-kernel@vger.kernel.org, david.vrabel@citrix.com, boris.ostrovsky@oracle.com Subject: Re: [PATCH v2 4/7] xen: introduce XENPF_settime64 Date: Tue, 10 Nov 2015 13:32:10 +0100 Message-ID: <3704230.3XlU4asf6t@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <1447156675-7418-4-git-send-email-stefano.stabellini@eu.citrix.com> References: <1447156675-7418-4-git-send-email-stefano.stabellini@eu.citrix.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:1y5jL78cyvMrO/H7YeVjhmcXhRxnki8buhJnqMD3J7t/QpnTk0R qkAVJR1jcb9pKeupr/o+JsP04y5CxQJPrw+c4oiTQxhWdXaM308ppAvDPzdHLmfRvk3gI32 9Ke/LN4KUocLvtEhRGR/tjGU39974QYXP6BC4WKALn8phWz7vWLYvK4VpTsw4/VwioS0KKe Zfx3yn+why7jAFBsAHjtA== X-UI-Out-Filterresults: notjunk:1;V01:K0:fuyaQbbzVCY=:Hu5moBkUm5xMZL53S11/e8 krgVQFOfzC+fZHmlErbnAtaiK2MIKx7eMiISwkAPI29F82BmGl4TBOknYh3EisS0ToFRHFv6o j9xeE3zqcNfvJnHrCC2nU9IbIynpX5weJHsx1S08MUHYfHQpsNUOtwgnyMBucKsD6tUEEByZ7 P9xxikt1lbwQkp/244iwxdeIAI9DLwRaHVT2DDoWMoUX+dhD8z8JeagxYtacHWr8Ei2gXbdcS zxfOhV0EsFlC5Goyl+85W+BBXlQr3bgIedqe2Wj/YLTcSNTIv7cE4fQZYkeODVphPXSdgSRs8 X7G+bTsFOXxe6Mlzrjcm8N/eJHFvEI9C/AaXWLVUknB8Wylfb2J3AXZBxXOv3sElDjMQd4hZn rMAE2eiUa87I3LHBMy39Pptk5TWZBjfnl3+yxEW/ybCfs7axl6l7+uc6TLPBR1n1UiYCAlX8q ictLA59vlV7zC2bWN4x1/7VDqA5hD0GzXF5DWfudb3Yv81G9+ooImnL4uRrg1ykMUahtFAXZN o7a3EsZ7epK8Gm2jvvpYdzU6WL8718dRnptu8Ig+InVmUy2Ix1I3skkhadlyzeki6xJCe9UF2 Wmt/pWA3zqBl9ZMbHxAHueuoy1wMWAB6cncWo4thdQCocM8xAgJTt9gKdknAXwM/Fnn2htXOL dQOWcpeBN1UpFJbiaSmGRO6M9RUL8mF10MMhlxyOkjkLxRMuqMI9mL8jI3nLCmbcQzJRP6k8S gG3m4vqNxHOKzJIN Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday 10 November 2015 11:57:52 Stefano Stabellini wrote: > Rename the current XENPF_settime hypercall and related struct to > XENPF_settime32. > > Signed-off-by: Stefano Stabellini > CC: konrad.wilk@oracle.com > CC: david.vrabel@citrix.com > CC: boris.ostrovsky@oracle.com Looks good. > --- > arch/x86/xen/time.c | 8 ++++---- > include/xen/interface/platform.h | 18 ++++++++++++++---- > 2 files changed, 18 insertions(+), 8 deletions(-) > > diff --git a/arch/x86/xen/time.c b/arch/x86/xen/time.c > index 663c2ea..3bbd377 100644 > --- a/arch/x86/xen/time.c > +++ b/arch/x86/xen/time.c > @@ -134,10 +134,10 @@ static int xen_pvclock_gtod_notify(struct notifier_block *nb, > if (!was_set && timespec_compare(&now, &next_sync) < 0) > return NOTIFY_OK; > > - op.cmd = XENPF_settime; > - op.u.settime.secs = now.tv_sec; > - op.u.settime.nsecs = now.tv_nsec; > - op.u.settime.system_time = xen_clocksource_read(); > + op.cmd = XENPF_settime32; > + op.u.settime32.secs = now.tv_sec; > + op.u.settime32.nsecs = now.tv_nsec; > + op.u.settime32.system_time = xen_clocksource_read(); > > (void)HYPERVISOR_platform_op(&op); I suppose someone will have to get back to this eventually and change the code to try XENPF_settime64 first. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Tue, 10 Nov 2015 13:32:10 +0100 Subject: [PATCH v2 4/7] xen: introduce XENPF_settime64 In-Reply-To: <1447156675-7418-4-git-send-email-stefano.stabellini@eu.citrix.com> References: <1447156675-7418-4-git-send-email-stefano.stabellini@eu.citrix.com> Message-ID: <3704230.3XlU4asf6t@wuerfel> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tuesday 10 November 2015 11:57:52 Stefano Stabellini wrote: > Rename the current XENPF_settime hypercall and related struct to > XENPF_settime32. > > Signed-off-by: Stefano Stabellini > CC: konrad.wilk at oracle.com > CC: david.vrabel at citrix.com > CC: boris.ostrovsky at oracle.com Looks good. > --- > arch/x86/xen/time.c | 8 ++++---- > include/xen/interface/platform.h | 18 ++++++++++++++---- > 2 files changed, 18 insertions(+), 8 deletions(-) > > diff --git a/arch/x86/xen/time.c b/arch/x86/xen/time.c > index 663c2ea..3bbd377 100644 > --- a/arch/x86/xen/time.c > +++ b/arch/x86/xen/time.c > @@ -134,10 +134,10 @@ static int xen_pvclock_gtod_notify(struct notifier_block *nb, > if (!was_set && timespec_compare(&now, &next_sync) < 0) > return NOTIFY_OK; > > - op.cmd = XENPF_settime; > - op.u.settime.secs = now.tv_sec; > - op.u.settime.nsecs = now.tv_nsec; > - op.u.settime.system_time = xen_clocksource_read(); > + op.cmd = XENPF_settime32; > + op.u.settime32.secs = now.tv_sec; > + op.u.settime32.nsecs = now.tv_nsec; > + op.u.settime32.system_time = xen_clocksource_read(); > > (void)HYPERVISOR_platform_op(&op); I suppose someone will have to get back to this eventually and change the code to try XENPF_settime64 first. Arnd