Re: [Xen-devel] [PATCH v4 3/5] xen: Put EFI machinery in place

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 20.05.14 at 13:29, <daniel.kiper@xxxxxxxxxx> wrote:
> On Tue, May 20, 2014 at 10:47:00AM +0100, David Vrabel wrote:
>> On 16/05/14 21:41, Daniel Kiper wrote:
>> > @@ -1714,6 +1725,21 @@ asmlinkage __visible void __init 
>> > xen_start_kernel(void)
>> >
>> >    xen_setup_runstate_info(0);
>> >
>> > +  efi_systab_xen = xen_efi_probe();
>> > +
>> > +  if (efi_systab_xen) {
>> > +          strncpy((char *)&boot_params.efi_info.efi_loader_signature, 
>> > "Xen",
>> > +                          
>> > sizeof(boot_params.efi_info.efi_loader_signature));
>> > +          boot_params.efi_info.efi_systab = 
>> > (__u32)((__u64)efi_systab_xen);
>> > +          boot_params.efi_info.efi_systab_hi = 
>> > (__u32)((__u64)efi_systab_xen >> 32);
>> > +
>> > +          x86_platform.get_wallclock = efi_get_time;
>>
>> x86_platform.get_wallclock should always be xen_get_wallclock().
> 
> Hmmm... Make sens... Jan, why did you replace x86_platform.get_wallclock
> with efi_get_time() in your implementation?

On the basis that (for Dom0 only) this is the equivalent of (and
actually also falls back to) mach_get_cmos_time(). If on Dom0
.get_wallclock doesn't get set to mach_get_cmos_time() on
pv-ops, then that line above should also be dropped.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel