Re: [PATCH] xen/x86: Adjust stack pointer in xen_sysexit

2015-11-17 Thread Joao Martins
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 >>>

Re: [Xen-devel] [PATCH RFC 3/3] xen/Kconfig: add XEN_TIME_VSYSCALL option

2016-01-04 Thread Joao Martins
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

Re: [PATCH RFC 2/3] x86/xen/time: setup vcpu 0 time info page

2016-01-04 Thread Joao Martins
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; >> +

Re: [PATCH RFC 2/3] x86/xen/time: setup vcpu 0 time info page

2016-01-05 Thread Joao Martins
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

[PATCH RFC 3/3] xen/Kconfig: add XEN_TIME_VSYSCALL option

2015-12-28 Thread Joao Martins
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

[PATCH RFC 1/3] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2015-12-28 Thread Joao Martins
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 |

[PATCH RFC 2/3] x86/xen/time: setup vcpu 0 time info page

2015-12-28 Thread Joao Martins
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/

[PATCH RFC 0/3] x86/xen: pvclock vdso support

2015-12-28 Thread Joao Martins
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

Re: [PATCH RFC 1/3] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2015-12-29 Thread Joao Martins
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 >>

Re: [Xen-devel] [PATCH linux 2/8] xen: introduce xen_vcpu_id mapping

2016-07-07 Thread Joao Martins
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

Re: [PATCH v1 2/3] x86/xen/time: setup vcpu 0 time info page

2017-01-26 Thread Joao Martins
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

Re: [PATCH v1 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-01-26 Thread Joao Martins
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 >

Re: [PATCH v1 1/3] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-01-26 Thread Joao Martins
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 >>

[PATCH v1 2/3] x86/xen/time: setup vcpu 0 time info page

2017-01-25 Thread Joao Martins
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

[PATCH v1 1/3] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-01-25 Thread Joao Martins
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

[PATCH v1 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-01-25 Thread Joao Martins
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

[PATCH v1 0/3] x86/xen: pvclock vdso support

2017-01-25 Thread Joao Martins
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

[PATCH v2 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-09-22 Thread Joao Martins
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

[PATCH v2 1/3] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-09-22 Thread Joao Martins
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

[PATCH v2 2/3] x86/xen/time: setup vcpu 0 time info page

2017-09-22 Thread Joao Martins
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

[PATCH v2 0/3] x86/xen: pvclock vdso support

2017-09-22 Thread Joao Martins
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

Re: [PATCH v5 3/4] x86/xen/time: setup vcpu 0 time info page

2017-10-02 Thread Joao Martins
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 >

Re: [PATCH v6 1/4] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-10-17 Thread Joao Martins
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

Re: [PATCH v4 2/3] x86/xen/time: setup vcpu 0 time info page

2017-09-27 Thread Joao Martins
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 =

[PATCH v5 1/4] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-10-02 Thread Joao Martins
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

[PATCH v5 3/4] x86/xen/time: setup vcpu 0 time info page

2017-10-02 Thread Joao Martins
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

[PATCH v5 0/4] x86/xen: pvclock vdso support

2017-10-02 Thread Joao Martins
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|

[PATCH v5 4/4] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-10-02 Thread Joao Martins
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

[PATCH v5 2/4] x86/xen/time: set pvclock flags on xen_time_init()

2017-10-02 Thread Joao Martins
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 ---

[PATCH v3 1/3] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-09-27 Thread Joao Martins
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

[PATCH v3 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-09-27 Thread Joao Martins
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

[PATCH v3 2/3] x86/xen/time: setup vcpu 0 time info page

2017-09-27 Thread Joao Martins
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

[PATCH v3 0/3] x86/xen: pvclock vdso support

2017-09-27 Thread Joao Martins
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

[PATCH v4 1/3] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-09-27 Thread Joao Martins
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

[PATCH v4 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-09-27 Thread Joao Martins
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

[PATCH v4 0/3] x86/xen: pvclock vdso support

2017-09-27 Thread Joao Martins
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

[PATCH v4 2/3] x86/xen/time: setup vcpu 0 time info page

2017-09-27 Thread Joao Martins
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

Re: [PATCH v3 2/3] x86/xen/time: setup vcpu 0 time info page

2017-09-27 Thread Joao Martins
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

Re: [PATCH v4 2/3] x86/xen/time: setup vcpu 0 time info page

2017-09-27 Thread Joao Martins
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;

Re: [PATCH v4 2/3] x86/xen/time: setup vcpu 0 time info page

2017-09-28 Thread Joao Martins
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: >>>>>

Re: [PATCH v4 2/3] x86/xen/time: setup vcpu 0 time info page

2017-09-27 Thread Joao Martins
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: >>>>>>

[PATCH v6 1/4] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-10-03 Thread Joao Martins
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

[PATCH v6 2/4] x86/xen/time: set pvclock flags on xen_time_init()

2017-10-03 Thread Joao Martins
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:

[PATCH v6 4/4] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-10-03 Thread Joao Martins
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

[PATCH v6 3/4] x86/xen/time: setup vcpu 0 time info page

2017-10-03 Thread Joao Martins
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

[PATCH v6 0/4] x86/xen: pvclock vdso support

2017-10-03 Thread Joao Martins
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 +

Re: [PATCH v2 2/3] x86/xen/time: setup vcpu 0 time info page

2017-09-26 Thread Joao Martins
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

Re: [PATCH v7 2/5] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-11-08 Thread Joao Martins
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

[PATCH v8 4/5] x86/xen/time: setup vcpu 0 time info page

2017-11-08 Thread Joao Martins
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

[PATCH v8 3/5] x86/xen/time: set pvclock flags on xen_time_init()

2017-11-08 Thread Joao Martins
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:

[PATCH v8 1/5] ptp_kvm: probe for kvm guest availability

2017-11-08 Thread Joao Martins
. 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

[PATCH v8 2/5] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-11-08 Thread Joao Martins
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

[PATCH v8 0/5] x86/xen: pvclock vdso support

2017-11-08 Thread Joao Martins
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

[PATCH v8 5/5] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-11-08 Thread Joao Martins
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

Re: [PATCH v7 2/5] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-11-07 Thread Joao Martins
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

Re: [PATCH] [v3] x86: Convert x86_platform_ops to timespec64

2018-04-27 Thread Joao Martins
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 >

[PATCH v2] x86/xen: Combine PV features to be disabled in xen_nopv

2018-05-28 Thread Joao Martins
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

Re: [PATCH] [v3] x86: Convert x86_platform_ops to timespec64

2018-05-02 Thread Joao Martins
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

[PATCH v7 4/5] x86/xen/time: setup vcpu 0 time info page

2017-10-19 Thread Joao Martins
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

[PATCH v7 3/5] x86/xen/time: set pvclock flags on xen_time_init()

2017-10-19 Thread Joao Martins
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:

Re: [PATCH v6 1/4] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-10-19 Thread Joao Martins
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

[PATCH v7 5/5] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-10-19 Thread Joao Martins
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

[PATCH v7 0/5] x86/xen: pvclock vdso support

2017-10-19 Thread Joao Martins
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

[PATCH v7 2/5] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-10-19 Thread Joao Martins
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

[PATCH v7 1/5] ptp_kvm: probe for kvm guest availability

2017-10-19 Thread Joao Martins
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

[PATCH] xenbus: track user request id

2018-02-01 Thread Joao Martins
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

[PATCH v2] xenbus: track caller request id

2018-02-02 Thread Joao Martins
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

[PATCH v3] xenbus: track caller request id

2018-02-08 Thread Joao Martins
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

Re: [PATCH v1] xen: acpi: upload _PSD info for offline CPUs too

2018-03-07 Thread Joao Martins
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

[PATCH v1] xen: acpi: upload _PSD info for offline CPUs too

2018-03-06 Thread Joao Martins
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 !=

[PATCH v2] xen/acpi: upload _PSD info for non Dom0 CPUs too

2018-03-08 Thread Joao Martins
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

[PATCH v3] xen/acpi: upload _PSD info for non Dom0 CPUs too

2018-03-15 Thread Joao Martins
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

Re: [Xen-devel] [PATCH v2] xen/acpi: upload _PSD info for non Dom0 CPUs too

2018-03-09 Thread Joao Martins
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%

Re: [PATCH v3] xen/acpi: upload _PSD info for non Dom0 CPUs too

2018-03-16 Thread Joao Martins
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

Re: [PATCH v3] xen/acpi: upload _PSD info for non Dom0 CPUs too

2018-03-20 Thread Joao Martins
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

Re: [Xen-devel] [PATCH] xen: drop writing error messages to xenstore

2018-10-11 Thread Joao Martins
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

Re: [PATCH v3] xen/acpi: upload _PSD info for non Dom0 CPUs too

2018-03-20 Thread Joao Martins
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

[PATCH v2] xen/acpi: upload _PSD info for non Dom0 CPUs too

2018-03-08 Thread Joao Martins
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

Re: [Xen-devel] [PATCH v2] xen/acpi: upload _PSD info for non Dom0 CPUs too

2018-03-09 Thread Joao Martins
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%

[PATCH v3] xen/acpi: upload _PSD info for non Dom0 CPUs too

2018-03-15 Thread Joao Martins
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

Re: [PATCH v3] xen/acpi: upload _PSD info for non Dom0 CPUs too

2018-03-16 Thread Joao Martins
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

Re: [PATCH] [v3] x86: Convert x86_platform_ops to timespec64

2018-04-27 Thread Joao Martins
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 >

Re: [PATCH] [v3] x86: Convert x86_platform_ops to timespec64

2018-05-02 Thread Joao Martins
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

Re: [PATCH v6 1/4] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-10-17 Thread Joao Martins
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

[PATCH v5 1/4] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-10-02 Thread Joao Martins
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

[PATCH v5 3/4] x86/xen/time: setup vcpu 0 time info page

2017-10-02 Thread Joao Martins
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

[PATCH v5 0/4] x86/xen: pvclock vdso support

2017-10-02 Thread Joao Martins
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|

[PATCH v5 4/4] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-10-02 Thread Joao Martins
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

[PATCH v5 2/4] x86/xen/time: set pvclock flags on xen_time_init()

2017-10-02 Thread Joao Martins
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

Re: [PATCH v5 3/4] x86/xen/time: setup vcpu 0 time info page

2017-10-02 Thread Joao Martins
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 >

[PATCH v6 1/4] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-10-03 Thread Joao Martins
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

[PATCH v6 2/4] x86/xen/time: set pvclock flags on xen_time_init()

2017-10-03 Thread Joao Martins
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

[PATCH v6 4/4] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-10-03 Thread Joao Martins
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

[PATCH v6 0/4] x86/xen: pvclock vdso support

2017-10-03 Thread Joao Martins
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 +

[PATCH v6 3/4] x86/xen/time: setup vcpu 0 time info page

2017-10-03 Thread Joao Martins
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

Re: [PATCH v4 2/3] x86/xen/time: setup vcpu 0 time info page

2017-09-28 Thread Joao Martins
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: >>>>>

[PATCH v2 0/3] x86/xen: pvclock vdso support

2017-09-22 Thread Joao Martins
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

[PATCH v2 1/3] x86/pvclock: add setter for pvclock_pvti_cpu0_va

2017-09-22 Thread Joao Martins
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

[PATCH v2 2/3] x86/xen/time: setup vcpu 0 time info page

2017-09-22 Thread Joao Martins
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

[PATCH v2 3/3] MAINTAINERS: xen, kvm: track pvclock-abi.h changes

2017-09-22 Thread Joao Martins
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   2   3   4   >