From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752484Ab3E1Sb6 (ORCPT ); Tue, 28 May 2013 14:31:58 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:40746 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751637Ab3E1Sb5 (ORCPT ); Tue, 28 May 2013 14:31:57 -0400 Date: Tue, 28 May 2013 14:31:37 -0400 From: Konrad Rzeszutek Wilk To: David Vrabel Cc: John Stultz , Jan Beulich , Thomas Gleixner , "xen-devel@lists.xen.org" , "linux-kernel@vger.kernel.org" Subject: Re: [Xen-devel] [PATCH 2/3] timekeeping: sync persistent clock and RTC on system time step changes Message-ID: <20130528183137.GA23213@phenom.dumpdata.com> References: <1368467768-2316-1-git-send-email-david.vrabel@citrix.com> <1368467768-2316-3-git-send-email-david.vrabel@citrix.com> <5191880F.3030308@linaro.org> <5192082D.4020400@citrix.com> <5192711B.1070607@linaro.org> <5193606502000078000D64E2@nat28.tlf.novell.com> <5193CFA2.8010008@linaro.org> <51A4F6C6.8080808@citrix.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <51A4F6C6.8080808@citrix.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-Source-IP: ucsinet22.oracle.com [156.151.31.94] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, May 28, 2013 at 07:26:14PM +0100, David Vrabel wrote: > On 15/05/13 19:10, John Stultz wrote: > > > > Ok, so really, as soon as the Dom0 time is set by NTP, all guests will > > see the right time? That makes more sense, and means the window for > > these sorts of issues is reasonably quite small. > > It's a small window but it's occurring in our automated test system. > > > David: So I'm less inclined to merge this individual change, but if you > > still feel strongly about it, let me know and we can circle around on it > > after you've addressed the specific issues I pointed out earlier. > > This patch was the actual bug fix but I've reworked it to use the > pvclock_gtod notifier chain as this seemed to be what KVM hosts were > using to maintain a clock for guests. Please review the new series, thanks. Looks good. John if you are OK I am thinking to push this to Linus shortly as it is fixing a bug. > > David