>>> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-efi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
