Re: [PATCH] x86/timers: Move simple_udelay_calibration past init_hypervisor_platform

Jan Kiszka
2017-05-28T16:41:59Z

On 2017-05-27 01:04, Andy Shevchenko wrote:
> +Cc Mathias
>
> On Wed, May 24, 2017 at 9:04 PM, Jan Kiszka <jan.kiszka@siemens.com> wrote:
>> This ensures that adjustments to x86_platform done by the hypervisor
>> setup is already respected by this simple calibration.
>>
>> The current user of this, introduced by 1b5aeebf3a92 ("x86/earlyprintk:
>> Add support for earlyprintk via USB3 debug port"), comes much later
>> into play.
>>
>
> Can you elaborate a bit what is the issue you are experience?

I stumbled over this while rebasing the patches to boot Linux over the
Jailhouse hypervisor (as non-root cell). Those come with hypervisor
setup code that changes relevant callbacks because Jailhouse does not
virtualize most of the platform devices and simply denies access.

While these patches are still out-of-tree (will propose later this
year), the issue seen here is conceptual: e.g., calibration is pointless
if the hypervisor has a PV channel to report the required parameters -
like vmware.

Jan

--
Siemens AG, Corporate Technology, CT RDA ITP SES-DE
Corporate Competence Center Embedded Linux