On 11/16/2015 09:04 PM, Andy Lutomirski wrote:
> On Mon, Nov 16, 2015 at 1:03 PM, Konrad Rzeszutek Wilk
> wrote:
>> On Mon, Nov 16, 2015 at 12:50:19PM -0800, Andy Lutomirski wrote:
>>> On Mon, Nov 16, 2015 at 12:48 PM, Boris Ostrovsky
>>>
On 01/04/2016 04:15 PM, Boris Ostrovsky wrote:
> On 01/04/2016 11:12 AM, David Vrabel wrote:
>> On 28/12/15 21:52, Joao Martins wrote:
>>> This option enables support for pvclock vsyscall/vdso
>>> support on Xen. Default is off, since Xen doesn't
>>> expose
On 01/04/2016 04:07 PM, Boris Ostrovsky wrote:
> On 12/28/2015 04:52 PM, Joao Martins wrote:
>> +
>> +static int xen_setup_vsyscall_time_info(int cpu)
>> +{
>> +struct pvclock_vsyscall_time_info *ti;
>> +struct vcpu_register_time_memory_area t;
>> +
On 01/04/2016 09:34 PM, Boris Ostrovsky wrote:
> On 01/04/2016 03:41 PM, Joao Martins wrote:
>>
>> On 01/04/2016 04:07 PM, Boris Ostrovsky wrote:
>>> On 12/28/2015 04:52 PM, Joao Martins wrote:
>>>
>>>> +
>>>> + size = PAGE_ALIG
This option enables support for pvclock vsyscall/vdso
support on Xen. Default is off, since Xen doesn't
expose yet the PVCLOCK_TSC_STABLE_BIT flag.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
arch/x86/xen/Kconfig | 5 +
1 file changed, 5 insertions(+)
diff --git a/ar
and moves pvclock_pvti_cpu0_va to pvclock, which
is a more generic place to have it; and would allow other PV clocksources
to use it, such as Xen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
arch/x86/include/asm/pvclock.h | 22 +-
arch/x86/kernel/kvmclock.c |
vcpu_time_info copy.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
arch/x86/xen/time.c | 66
include/xen/interface/vcpu.h | 28 +++
2 files changed, 94 insertions(+)
diff --git a/arch/x86/xen/time.c b/arch/x86/xen/
will probably be discussed in another RFC I sent[0].
Any comments or suggestions are welcome!
Thanks,
Joao
[0] http://lists.xenproject.org/archives/html/xen-devel/2015-12/msg02835.html
Joao Martins (3):
x86/pvclock: add setter for pvclock_pvti_cpu0_va
x86/xen/time: setup vcpu 0 time info page
On 12/28/2015 11:45 PM, Andy Lutomirski wrote:
> On Mon, Dec 28, 2015 at 1:52 PM, Joao Martins <joao.m.mart...@oracle.com>
> wrote:
>> Right now there is only a pvclock_pvti_cpu0_va() which is defined on
>> kvmclock since:
>>
>> commit dac16fba6fc5
>>
On 07/05/2016 04:44 PM, Jan Beulich wrote:
On 05.07.16 at 17:34, wrote:
>> On Thu, Jun 30, 2016 at 03:10:11AM -0600, Jan Beulich wrote:
>> On 29.06.16 at 18:27, wrote:
On 29/06/16 17:19, Vitaly Kuznetsov wrote:
> To explain
On 01/25/2017 07:26 PM, Boris Ostrovsky wrote:
> On 01/25/2017 12:33 PM, Joao Martins wrote:
>> In order to support pvclock vdso on xen we need to setup the time
>> info page for vcpu 0 and register the page with Xen using the
>> VCPUOP_register_vcpu_time_memory_area hype
On 01/26/2017 05:25 PM, Andy Lutomirski wrote:
> On Wed, Jan 25, 2017 at 9:33 AM, Joao Martins <joao.m.mart...@oracle.com>
> wrote:
>> This file defines an ABI shared between guest and hypervisor(s)
>> (KVM, Xen) and as such there should be an correspondent entry in
>
On 01/26/2017 05:25 PM, Andy Lutomirski wrote:
> On Wed, Jan 25, 2017 at 9:33 AM, Joao Martins <joao.m.mart...@oracle.com>
> wrote:
>> Right now there is only a pvclock_pvti_cpu0_va() which is defined
>> on kvmclock since:
>>
>> commit dac16fba6fc5
>>
vcpu_time_info struct.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
Changes since RFC:
(Comments from Boris and David)
* Remove Kconfig option
* Use get_zeroed_page/free/page
* Remove the hypercall availability check
* Unregister pvti with arg.addr.v = NULL if stable bit isn't sup
eric place to have it
and adds the correspondent setter routine for it. This allows other
pvclock-based clocksources to use it, such as Xen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
Changes since RFC:
(Comments from Andy Lutomirski)
* Add WARN_ON(vclock_was_use
noticed when such changes happen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
This was suggested by folks at xen-devel as we missed some of the
ABI additions (e.g. flags field in pvti, TSC stable bit) - so this
patch is to help preventing that from happening. Alternatively I
BI changes. Changelog since RFC is
included in individual patches.
Any comments/suggestions are welcome.
Thanks,
Joao
Joao Martins (3):
x86/pvclock: add setter for pvclock_pvti_cpu0_va
x86/xen/time: setup vcpu 0 time info page
MAINTAINERS: xen, kvm: track pvclock-abi.h changes
M
noticed when such changes happen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Juergen Gross <jgr...@suse.com>
---
Out of the two options (and provided I was given a choice) I choose the
originally posted because this is so far the only ABI shared between Xen/KVM.
When
and moves pvclock_pvti_cpu0_va
to pvclock, which is a more generic place to have it; and would
allow other PV clocksources to use it, such as Xen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Andy Lutomirski <l...@kernel.org>
---
Changes since v1:
* Rebased: the only confli
vcpu_time_info struct.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
Changes since v1:
* Check flags ahead to see if the primary clock can use
PVCLOCK_TSC_STABLE_BIT even if secondary registration fails.
(Comments from Boris)
* Remove addr, addr variables;
* Change first pr
BI changes.
Changelog since v1 is included in individual patches.
Any comments/suggestions are welcome.
Thanks,
Joao
Joao Martins (3):
x86/pvclock: add setter for pvclock_pvti_cpu0_va
x86/xen/time: setup vcpu 0 time info page
MAINTAINERS: xen, kvm: track pvclock-abi.h changes
M
On 10/02/2017 07:44 PM, Boris Ostrovsky wrote:
>
>> +
>> +static void xen_setup_vsyscall_time_info(void)
>> +{
>> +struct vcpu_register_time_memory_area t;
>> +struct pvclock_vsyscall_time_info *ti;
>> +int ret;
>
>
> In the previous version you'd return immediately if
>
On 10/03/2017 12:55 PM, Joao Martins wrote:
> Right now there is only a pvclock_pvti_cpu0_va() which is defined
> on kvmclock since:
>
> commit dac16fba6fc5
> ("x86/vdso: Get pvclock data from the vvar VMA instead of the fixmap")
>
> The only user of this inter
On 09/27/2017 03:40 PM, Boris Ostrovsky wrote:
>> +static void xen_setup_vsyscall_time_info(void)
>> +{
>> +struct vcpu_register_time_memory_area t;
>> +struct pvclock_vsyscall_time_info *ti;
>> +struct pvclock_vcpu_time_info *pvti;
>> +int ret;
>> +
>> +pvti =
and moves pvclock_pvti_cpu0_va
to pvclock, which is a more generic place to have it; and would
allow other PV clocksources to use it, such as Xen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Andy Lutomirski <l...@kernel.org>
---
Changes since v1:
* Rebased: the only confli
vcpu_time_info struct.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Reviewed-by: Juergen Gross <jgr...@suse.com>
---
Changes since v4:
* Remove pvclock_set_flags since predecessor patch will set in
xen_time_init. Consequently pvti local variable is not so useful
and doesn't make
Changelog is in individual patches.
Thanks,
Joao
Joao Martins (4):
x86/pvclock: add setter for pvclock_pvti_cpu0_va
x86/xen/time: set pvclock flags on xen_time_init()
x86/xen/time: setup vcpu 0 time info page
MAINTAINERS: xen, kvm: track pvclock-abi.h changes
MAINTAINERS|
noticed when such changes happen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Juergen Gross <jgr...@suse.com>
Reviewed-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com>
Acked-by: Paolo Bonzini <pbonz...@redhat.com>
---
Changes since v4:
* Add Paolo's
Specifically check for PVCLOCK_TSC_STABLE_BIT and if this bit is set,
then set it too on pvclock flags. This allows Xen clocksource to use it
and thus speeding up xen_clocksource_read() callers (i.e. sched_clock())
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
New in v5
---
and moves pvclock_pvti_cpu0_va
to pvclock, which is a more generic place to have it; and would
allow other PV clocksources to use it, such as Xen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Andy Lutomirski <l...@kernel.org>
---
Changes since v1:
* Rebased: the only confli
noticed when such changes happen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Juergen Gross <jgr...@suse.com>
---
In the end, I choose the originally posted because this is so far the
only ABI shared between Xen/KVM. Therefore whenever we have more things
shared it w
vcpu_time_info struct.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
Changes since v2:
(Comments from Juergen)
* Omit the blan after the cast on all 3 occurrences.
* Change last VCLOCK_PVCLOCK message to be more descriptive
* Sync the complete vcpu.h header instead of just
hes.
(only patch 2 changed in this version)
Thanks,
Joao
Joao Martins (3):
x86/pvclock: add setter for pvclock_pvti_cpu0_va
x86/xen/time: setup vcpu 0 time info page
MAINTAINERS: xen, kvm: track pvclock-abi.h changes
MAINTAINERS| 2 +
arch/x86/include/asm/pvclo
and moves pvclock_pvti_cpu0_va
to pvclock, which is a more generic place to have it; and would
allow other PV clocksources to use it, such as Xen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Andy Lutomirski <l...@kernel.org>
---
Changes since v1:
* Rebased: the only confli
noticed when such changes happen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Juergen Gross <jgr...@suse.com>
---
In the end, I choose the originally posted because this is so far the
only ABI shared between Xen/KVM. Therefore whenever we have more things
shared it w
hes.
(only patch 2 changed in this version)
Thanks,
Joao
Joao Martins (3):
x86/pvclock: add setter for pvclock_pvti_cpu0_va
x86/xen/time: setup vcpu 0 time info page
MAINTAINERS: xen, kvm: track pvclock-abi.h changes
MAINTAINERS| 2 +
arch/x86/include/asm/pvclo
vcpu_time_info struct.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
Changes since v3:
(Comments from Juergen)
* Remove _t added suffix from *GUEST_HANDLE* when sync vcpu.h
with the latest
Changes since v2:
(Comments from Juergen)
* Omit the blank after the cast on all 3 occur
On 09/27/2017 01:14 PM, Juergen Gross wrote:
> On 27/09/17 14:00, Joao Martins wrote:
[...]
>> diff --git a/include/xen/interface/vcpu.h b/include/xen/interface/vcpu.h
>> index 98188c87f5c1..b4a1eabcf1c4 100644
>> --- a/include/xen/interface/vcpu.h
>> +++ b/i
On 09/27/2017 09:22 PM, Boris Ostrovsky wrote:
> On 09/27/2017 11:26 AM, Joao Martins wrote:
>> On 09/27/2017 03:40 PM, Boris Ostrovsky wrote:
>>>> +static void xen_setup_vsyscall_time_info(void)
>>>> +{
>>>> + struct vcpu_register_time_memory_area t;
On 09/28/2017 12:46 AM, Joao Martins wrote:
> On 09/27/2017 11:44 PM, Boris Ostrovsky wrote:
>> On 09/27/2017 04:57 PM, Joao Martins wrote:
>>> On 09/27/2017 09:22 PM, Boris Ostrovsky wrote:
>>>> On 09/27/2017 11:26 AM, Joao Martins wrote:
>>>>>
On 09/27/2017 11:44 PM, Boris Ostrovsky wrote:
> On 09/27/2017 04:57 PM, Joao Martins wrote:
>> On 09/27/2017 09:22 PM, Boris Ostrovsky wrote:
>>> On 09/27/2017 11:26 AM, Joao Martins wrote:
>>>> On 09/27/2017 03:40 PM, Boris Ostrovsky wrote:
>>>>>>
and moves pvclock_pvti_cpu0_va
to pvclock, which is a more generic place to have it; and would
allow other PV clocksources to use it, such as Xen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Andy Lutomirski <l...@kernel.org>
---
Changes since v1:
* Rebased: the only confli
Specifically check for PVCLOCK_TSC_STABLE_BIT and if this bit is set,
then set it too on pvclock flags. This allows Xen clocksource to use it
and thus speeding up xen_clocksource_read() callers (i.e. sched_clock())
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Reviewed-by:
noticed when such changes happen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Juergen Gross <jgr...@suse.com>
Reviewed-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com>
Acked-by: Paolo Bonzini <pbonz...@redhat.com>
---
Changes since v4:
* Add Paolo's
vcpu_time_info struct.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Reviewed-by: Juergen Gross <jgr...@suse.com>
---
Changes since v5:
* Move xen_setup_vsyscall_time_info within the PVCLOCK_TSC_STABLE_BIT
clause added in the previous patch.
Changes since v4:
* Remove pvclock_set
Changelog is in individual patches.
Thanks,
Joao
Joao Martins (4):
x86/pvclock: add setter for pvclock_pvti_cpu0_va
x86/xen/time: set pvclock flags on xen_time_init()
x86/xen/time: setup vcpu 0 time info page
MAINTAINERS: xen, kvm: track pvclock-abi.h changes
MAINTAINERS| 2 +
On 09/26/2017 10:32 AM, Juergen Gross wrote:
> On 22/09/17 18:25, Joao Martins wrote:
[snip]
>> +static void xen_setup_vsyscall_time_info(void)
>> +{
>> +struct vcpu_register_time_memory_area t;
>> +struct pvclock_vsyscall_time_info *ti;
>> +st
On 11/08/2017 11:06 AM, Thomas Gleixner wrote:
> On Tue, 7 Nov 2017, Joao Martins wrote:
>> On 11/06/2017 04:09 PM, Paolo Bonzini wrote:
>>> On 19/10/2017 15:39, Joao Martins wrote:
>>>> Right now there is only a pvclock_pvti_cpu0_va() which is defined
>>&g
vcpu_time_info struct.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Reviewed-by: Juergen Gross <jgr...@suse.com>
Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracle.com>
---
Changes since v5:
* Move xen_setup_vsyscall_time_info within the PVCLOCK_TSC_STABLE_BIT
clause ad
Specifically check for PVCLOCK_TSC_STABLE_BIT and if this bit is set,
then set it too on pvclock flags. This allows Xen clocksource to use it
and thus speeding up xen_clocksource_read() callers (i.e. sched_clock())
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Reviewed-by:
.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Radim Krčmář <rkrc...@redhat.com>
---
Changes since v7:
* Add Radim's Acked-by
---
drivers/ptp/ptp_kvm.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/ptp/ptp_kvm.c b/drivers/ptp/ptp_kvm.c
index
the setter (pvclock_set_pvti_cpu0_va).
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Andy Lutomirski <l...@kernel.org>
Acked-by: Paolo Bonzini <pbonz...@redhat.com>
Acked-by: Thomas Gleixner <t...@linutronix.de>
---
Changes since v7:
* Add Paolo Acked-by
(Commen
individual patches)
Thanks,
Joao
Joao Martins (5):
ptp_kvm: probe for kvm guest availability
x86/pvclock: add setter for pvclock_pvti_cpu0_va
x86/xen/time: set pvclock flags on xen_time_init()
x86/xen/time: setup vcpu 0 time info page
MAINTAINERS: xen, kvm: track pvclock-abi.h changes
M
noticed when such changes happen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Juergen Gross <jgr...@suse.com>
Reviewed-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com>
Acked-by: Paolo Bonzini <pbonz...@redhat.com>
---
Changes since v4:
* Add Paolo's
On 11/06/2017 04:09 PM, Paolo Bonzini wrote:
> On 19/10/2017 15:39, Joao Martins wrote:
>> Right now there is only a pvclock_pvti_cpu0_va() which is defined
>> on kvmclock since:
>>
>> commit dac16fba6fc5
>> ("x86/vdso: Get pvclock data from the vvar VMA inste
On 04/27/2018 09:13 PM, Arnd Bergmann wrote:
> diff --git a/arch/x86/kernel/pvclock.c b/arch/x86/kernel/pvclock.c
> index 761f6af6efa5..637982efecd8 100644
> --- a/arch/x86/kernel/pvclock.c
> +++ b/arch/x86/kernel/pvclock.c
> @@ -123,28 +123,35 @@ u64 pvclock_clocksource_read(struct
>
Rzeszutek Wilk
Signed-off-by: Joao Martins
---
Changes since RFC:
(https://lists.xenproject.org/archives/html/xen-devel/2015-10/msg00898.html
Comments from Boris, and new changes)
- Improve subject and commit message
- Fix Documentation format and deprecate 'xen_nopvspin' instead of removing
On 04/28/2018 11:09 AM, Arnd Bergmann wrote:
> On Sat, Apr 28, 2018 at 12:21 AM, Joao Martins
> <joao.m.mart...@oracle.com> wrote:
>> On 04/27/2018 09:13 PM, Arnd Bergmann wrote:
>>> diff --git a/arch/x86/kernel/pvclock.c b/arch/x86/kernel/pvclock.c
>>> ind
vcpu_time_info struct.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Reviewed-by: Juergen Gross <jgr...@suse.com>
Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracle.com>
---
Changes since v6:
* Add Boris RoB
Changes since v5:
* Move xen_setup_vsyscal
Specifically check for PVCLOCK_TSC_STABLE_BIT and if this bit is set,
then set it too on pvclock flags. This allows Xen clocksource to use it
and thus speeding up xen_clocksource_read() callers (i.e. sched_clock())
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Reviewed-by:
On 10/17/2017 04:34 PM, Joao Martins wrote:
> On 10/03/2017 12:55 PM, Joao Martins wrote:
>> Right now there is only a pvclock_pvti_cpu0_va() which is defined
>> on kvmclock since:
>>
>> commit dac16fba6fc5
>> ("x86/vdso: Get pvclock data from the vvar VMA
noticed when such changes happen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Juergen Gross <jgr...@suse.com>
Reviewed-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com>
Acked-by: Paolo Bonzini <pbonz...@redhat.com>
---
Changes since v4:
* Add Paolo's
anks,
Joao
Joao Martins (5):
ptp_kvm: probe for kvm guest availability
x86/pvclock: add setter for pvclock_pvti_cpu0_va
x86/xen/time: set pvclock flags on xen_time_init()
x86/xen/time: setup vcpu 0 time info page
MAINTAINERS: xen, kvm: track pvclock-abi.h changes
M
and moves pvclock_pvti_cpu0_va
to pvclock, which is a more generic place to have it; and would
allow other PV clocksources to use it, such as Xen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Acked-by: Andy Lutomirski <l...@kernel.org>
---
Changes since v1:
* Rebased: the only confli
whether it is running in a KVM guest.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
New in v7;
---
drivers/ptp/ptp_kvm.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/ptp/ptp_kvm.c b/drivers/ptp/ptp_kvm.c
index 2b1b212c219e..e04d7b2ecb3a 100644
--- a/drive
nstore accesses")
Reported-by: Bhavesh Davda <bhavesh.da...@oracle.com>
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
Sending out for some feedback first, more to double check whether this indeed
constitutes a problem. If it does I'll need to include a
"Cc: <sta
Bhavesh Davda <bhavesh.da...@oracle.com>
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
Here's a link to a unit test (https://pastebin.com/2q51j2sR) where req_id of
reply and response are being asserted each request. Without this patch
the assert will fail (e.g. try it with `./xsw
Bhavesh Davda <bhavesh.da...@oracle.com>
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
Reviewed-by: Juergen Gross <jgr...@suse.com>
---
Here's a link to a unit test (https://pastebin.com/N0kqNBnM) where req_id of
reply and response are being asserted each request. Witho
On 03/07/2018 05:51 PM, Boris Ostrovsky wrote:
> On 03/06/2018 03:12 PM, Joao Martins wrote:
>> All uploaded PM data from offline CPUs takes the info from vCPU 0 and
>
> "offline" may not be the right term here. Maybe "non-dom0"?
>
Yeah, probably
and offline) and always uploading the correct P-State
dependency up to Xen.
Signed-off-by: Joao Martins <joao.m.mart...@oracle.com>
---
xen_processor_get_psd() is taken from acpi_processor_get_psd()
(drivers/acpi/processor_perflib.c) simply because only Xen Dom0 needs
to handle vcpus !=
procesors and always uploading the correct info to Xen. We export
acpi_processor_get_psd() to help with this purpose, but change signature
to not assume the existence of a struct acpi_processor given that ACPI
doesn't create an acpi_processor for non-dom0 CPUs.
Signed-off-by: Joao Martins <joao.m.m
processors and thus always uploading the correct info to Xen. We export
acpi_processor_get_psd() for that this purpose, but change signature
to not assume an existent of acpi_processor given that ACPI isn't creating
an acpi_processor for non-dom0 CPUs.
Signed-off-by: Joao Martins <joao.m.m
On 03/08/2018 10:33 PM, Boris Ostrovsky wrote:
> On 03/08/2018 05:57 AM, Joao Martins wrote:
>
>> @@ -372,6 +376,15 @@ read_acpi_id(acpi_handle handle, u32 lvl, void
>> *context, void **rv)
>>
>> pr_debug("ACPI CPU%u w/ PBLK:0x%
On 03/15/2018 03:45 PM, Boris Ostrovsky wrote:
> On 03/15/2018 10:22 AM, Joao Martins wrote:
>> All uploaded PM data from non-dom0 CPUs takes the info from vCPU 0 and
>> changing only the acpi_id. For processors which P-state coordination type
>> is HW_ALL (0xFD) it is OK to
On 03/20/2018 09:41 AM, Rafael J. Wysocki wrote:
> On Fri, Mar 16, 2018 at 2:57 PM, Joao Martins <joao.m.mart...@oracle.com>
> wrote:
>> On 03/15/2018 03:45 PM, Boris Ostrovsky wrote:
>>> On 03/15/2018 10:22 AM, Joao Martins wrote:
>>>> All uploaded PM data
On 10/11/2018 06:05 AM, Juergen Gross wrote:
> On 10/10/2018 18:57, Boris Ostrovsky wrote:
>> On 10/10/18 11:53 AM, Juergen Gross wrote:
>>> On 10/10/2018 17:09, Joao Martins wrote:
>>>> On 10/09/2018 05:09 PM, Juergen Gross wrote:
>>>>> xenbus_va
On 03/20/2018 09:41 AM, Rafael J. Wysocki wrote:
> On Fri, Mar 16, 2018 at 2:57 PM, Joao Martins
> wrote:
>> On 03/15/2018 03:45 PM, Boris Ostrovsky wrote:
>>> On 03/15/2018 10:22 AM, Joao Martins wrote:
>>>> All uploaded PM data from non-dom0 CPUs takes the i
procesors and always uploading the correct info to Xen. We export
acpi_processor_get_psd() to help with this purpose, but change signature
to not assume the existence of a struct acpi_processor given that ACPI
doesn't create an acpi_processor for non-dom0 CPUs.
Signed-off-by: Joao Martins
---
Changes
On 03/08/2018 10:33 PM, Boris Ostrovsky wrote:
> On 03/08/2018 05:57 AM, Joao Martins wrote:
>
>> @@ -372,6 +376,15 @@ read_acpi_id(acpi_handle handle, u32 lvl, void
>> *context, void **rv)
>>
>> pr_debug("ACPI CPU%u w/ PBLK:0x%
processors and thus always uploading the correct info to Xen. We export
acpi_processor_get_psd() for that this purpose, but change signature
to not assume an existent of acpi_processor given that ACPI isn't creating
an acpi_processor for non-dom0 CPUs.
Signed-off-by: Joao Martins
---
Changes since v2
On 03/15/2018 03:45 PM, Boris Ostrovsky wrote:
> On 03/15/2018 10:22 AM, Joao Martins wrote:
>> All uploaded PM data from non-dom0 CPUs takes the info from vCPU 0 and
>> changing only the acpi_id. For processors which P-state coordination type
>> is HW_ALL (0xFD) it is OK to
On 04/27/2018 09:13 PM, Arnd Bergmann wrote:
> diff --git a/arch/x86/kernel/pvclock.c b/arch/x86/kernel/pvclock.c
> index 761f6af6efa5..637982efecd8 100644
> --- a/arch/x86/kernel/pvclock.c
> +++ b/arch/x86/kernel/pvclock.c
> @@ -123,28 +123,35 @@ u64 pvclock_clocksource_read(struct
>
On 04/28/2018 11:09 AM, Arnd Bergmann wrote:
> On Sat, Apr 28, 2018 at 12:21 AM, Joao Martins
> wrote:
>> On 04/27/2018 09:13 PM, Arnd Bergmann wrote:
>>> diff --git a/arch/x86/kernel/pvclock.c b/arch/x86/kernel/pvclock.c
>>> index 761f6af6efa5..637982efecd8 10
On 10/03/2017 12:55 PM, Joao Martins wrote:
> Right now there is only a pvclock_pvti_cpu0_va() which is defined
> on kvmclock since:
>
> commit dac16fba6fc5
> ("x86/vdso: Get pvclock data from the vvar VMA instead of the fixmap")
>
> The only user of this inter
and moves pvclock_pvti_cpu0_va
to pvclock, which is a more generic place to have it; and would
allow other PV clocksources to use it, such as Xen.
Signed-off-by: Joao Martins
Acked-by: Andy Lutomirski
---
Changes since v1:
* Rebased: the only conflict was that I had move the export
pvclock_pvti_cpu0_v
vcpu_time_info struct.
Signed-off-by: Joao Martins
Reviewed-by: Juergen Gross
---
Changes since v4:
* Remove pvclock_set_flags since predecessor patch will set in
xen_time_init. Consequently pvti local variable is not so useful
and doesn't make things more clear - therefore remove it.
* Adjust comment
Changelog is in individual patches.
Thanks,
Joao
Joao Martins (4):
x86/pvclock: add setter for pvclock_pvti_cpu0_va
x86/xen/time: set pvclock flags on xen_time_init()
x86/xen/time: setup vcpu 0 time info page
MAINTAINERS: xen, kvm: track pvclock-abi.h changes
MAINTAINERS|
noticed when such changes happen.
Signed-off-by: Joao Martins
Acked-by: Juergen Gross
Reviewed-by: Konrad Rzeszutek Wilk
Acked-by: Paolo Bonzini
---
Changes since v4:
* Add Paolo's Acked-by
* Add Konrad's Reviewed-by
Changes since v1:
* Add Juergen's Gross Acked-by.
---
MAINTAINERS | 2 ++
1
Specifically check for PVCLOCK_TSC_STABLE_BIT and if this bit is set,
then set it too on pvclock flags. This allows Xen clocksource to use it
and thus speeding up xen_clocksource_read() callers (i.e. sched_clock())
Signed-off-by: Joao Martins
---
New in v5
---
arch/x86/xen/time.c | 9
On 10/02/2017 07:44 PM, Boris Ostrovsky wrote:
>
>> +
>> +static void xen_setup_vsyscall_time_info(void)
>> +{
>> +struct vcpu_register_time_memory_area t;
>> +struct pvclock_vsyscall_time_info *ti;
>> +int ret;
>
>
> In the previous version you'd return immediately if
>
and moves pvclock_pvti_cpu0_va
to pvclock, which is a more generic place to have it; and would
allow other PV clocksources to use it, such as Xen.
Signed-off-by: Joao Martins
Acked-by: Andy Lutomirski
---
Changes since v1:
* Rebased: the only conflict was that I had move the export
pvclock_pvti_cpu0_v
Specifically check for PVCLOCK_TSC_STABLE_BIT and if this bit is set,
then set it too on pvclock flags. This allows Xen clocksource to use it
and thus speeding up xen_clocksource_read() callers (i.e. sched_clock())
Signed-off-by: Joao Martins
Reviewed-by: Boris Ostrovsky
---
Changes since v5
noticed when such changes happen.
Signed-off-by: Joao Martins
Acked-by: Juergen Gross
Reviewed-by: Konrad Rzeszutek Wilk
Acked-by: Paolo Bonzini
---
Changes since v4:
* Add Paolo's Acked-by
* Add Konrad's Reviewed-by
Changes since v1:
* Add Juergen's Gross Acked-by.
---
MAINTAINERS | 2 ++
1
Changelog is in individual patches.
Thanks,
Joao
Joao Martins (4):
x86/pvclock: add setter for pvclock_pvti_cpu0_va
x86/xen/time: set pvclock flags on xen_time_init()
x86/xen/time: setup vcpu 0 time info page
MAINTAINERS: xen, kvm: track pvclock-abi.h changes
MAINTAINERS| 2 +
vcpu_time_info struct.
Signed-off-by: Joao Martins
Reviewed-by: Juergen Gross
---
Changes since v5:
* Move xen_setup_vsyscall_time_info within the PVCLOCK_TSC_STABLE_BIT
clause added in the previous patch.
Changes since v4:
* Remove pvclock_set_flags since predecessor patch will set in
xen_time_init
On 09/28/2017 12:46 AM, Joao Martins wrote:
> On 09/27/2017 11:44 PM, Boris Ostrovsky wrote:
>> On 09/27/2017 04:57 PM, Joao Martins wrote:
>>> On 09/27/2017 09:22 PM, Boris Ostrovsky wrote:
>>>> On 09/27/2017 11:26 AM, Joao Martins wrote:
>>>>>
BI changes.
Changelog since v1 is included in individual patches.
Any comments/suggestions are welcome.
Thanks,
Joao
Joao Martins (3):
x86/pvclock: add setter for pvclock_pvti_cpu0_va
x86/xen/time: setup vcpu 0 time info page
MAINTAINERS: xen, kvm: track pvclock-abi.h changes
M
and moves pvclock_pvti_cpu0_va
to pvclock, which is a more generic place to have it; and would
allow other PV clocksources to use it, such as Xen.
Signed-off-by: Joao Martins
Acked-by: Andy Lutomirski
---
Changes since v1:
* Rebased: the only conflict was that I had move the export
pvclock_pvti_cpu0_v
vcpu_time_info struct.
Signed-off-by: Joao Martins
---
Changes since v1:
* Check flags ahead to see if the primary clock can use
PVCLOCK_TSC_STABLE_BIT even if secondary registration fails.
(Comments from Boris)
* Remove addr, addr variables;
* Change first pr_debug to pr_warn;
* Change last
noticed when such changes happen.
Signed-off-by: Joao Martins
Acked-by: Juergen Gross
---
Out of the two options (and provided I was given a choice) I choose the
originally posted because this is so far the only ABI shared between Xen/KVM.
Whenever we have more things shared it would probably deserve
1 - 100 of 326 matches
Mail list logo