Re: [Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-15 Thread Jan Beulich
>>> On 15.12.15 at 15:50, wrote: > On Tue, Dec 15, 2015 at 12:19:45AM -0700, Jan Beulich wrote: >> >>> On 08.12.15 at 12:02, wrote: >> > On Wed, 2015-12-02 at 02:33 -0700, Jan Beulich wrote: >> >> Then we should see about adding support for "efi=no-time". >> > >> > And based on what I'm reading

Re: [Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-14 Thread Jan Beulich
>>> On 08.12.15 at 12:02, wrote: > On Wed, 2015-12-02 at 02:33 -0700, Jan Beulich wrote: >> Then we should see about adding support for "efi=no-time". > > And based on what I'm reading in this thread about the reliability of the > time RS in the field it seems to me we should make it the default

Re: [Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-14 Thread Jan Beulich
>>> On 14.12.15 at 17:53, wrote: > On Tue, Dec 08, 2015 at 11:02:16AM +, Ian Campbell wrote: >> In reality only Microsoft (and perhaps to a lesser extent Apple) have any >> sort of ability to force things in this way (even so it is alleged in this >> thread that even Windows avoids the GetTime

Re: [Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-14 Thread Konrad Rzeszutek Wilk
On Tue, Dec 08, 2015 at 11:02:16AM +, Ian Campbell wrote: > On Wed, 2015-12-02 at 02:33 -0700, Jan Beulich wrote: > > > > > On 01.12.15 at 20:26, wrote: > > > On 01/12/15 17:26, Jan Beulich wrote: > > > > > > > On 01.12.15 at 17:57, wrote: > > > > > When EFI is used, don't use EFI's GetTime()

Re: [Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-08 Thread Ian Campbell
On Wed, 2015-12-02 at 02:33 -0700, Jan Beulich wrote: > > > > On 01.12.15 at 20:26, wrote: > > On 01/12/15 17:26, Jan Beulich wrote: > > > > > > On 01.12.15 at 17:57, wrote: > > > > When EFI is used, don't use EFI's GetTime() to get the time, > > > > because it > > > > is broken on many platforms

Re: [Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-02 Thread Jan Beulich
>>> On 01.12.15 at 20:26, wrote: > On 01/12/15 17:26, Jan Beulich wrote: > On 01.12.15 at 17:57, wrote: >>> When EFI is used, don't use EFI's GetTime() to get the time, because it >>> is broken on many platforms. From Linux commit 7efe665903d0 ("rtc: >>> Disable EFI rtc for x86"): >>> "Disabl

Re: [Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-01 Thread Andrew Cooper
On 01/12/15 17:26, Jan Beulich wrote: On 01.12.15 at 17:57, wrote: >> When EFI is used, don't use EFI's GetTime() to get the time, because it >> is broken on many platforms. From Linux commit 7efe665903d0 ("rtc: >> Disable EFI rtc for x86"): >> "Disable it explicitly for x86 so that we don't

Re: [Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-01 Thread Jan Beulich
>>> On 01.12.15 at 17:57, wrote: > When EFI is used, don't use EFI's GetTime() to get the time, because it > is broken on many platforms. From Linux commit 7efe665903d0 ("rtc: > Disable EFI rtc for x86"): > "Disable it explicitly for x86 so that we don't give users false > hope that this driver wi

Re: [Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-01 Thread David Vrabel
On 01/12/15 16:57, Ross Lagerwall wrote: > When EFI is used, don't use EFI's GetTime() to get the time, because it > is broken on many platforms. [...] > --- a/xen/arch/x86/time.c > +++ b/xen/arch/x86/time.c > @@ -679,20 +679,28 @@ static void __get_cmos_time(struct rtc_time *rtc) > rtc->y

[Xen-devel] [PATCH] x86/time: Don't use EFI's GetTime call by default

2015-12-01 Thread Ross Lagerwall
When EFI is used, don't use EFI's GetTime() to get the time, because it is broken on many platforms. From Linux commit 7efe665903d0 ("rtc: Disable EFI rtc for x86"): "Disable it explicitly for x86 so that we don't give users false hope that this driver will work - it won't, and your machine is like