Re: [Xen-devel] [PATCH net-next v1] xen-netback: make copy batch size configurable

2017-11-13 Thread Joao Martins
On Mon, Nov 13, 2017 at 04:39:09PM +, Paul Durrant wrote: > > -Original Message- > > From: Joao Martins [mailto:joao.m.mart...@oracle.com] > > Sent: 13 November 2017 16:34 > > To: Paul Durrant > > Cc: net...@vger.kernel.org; Wei Liu ; xen- > > de

Re: [Xen-devel] [PATCH net-next v1] xen-netback: make copy batch size configurable

2017-11-13 Thread Joao Martins
On Mon, Nov 13, 2017 at 11:58:03AM +, Paul Durrant wrote: > On Mon, Nov 13, 2017 at 11:54:00AM +0000, Joao Martins wrote: > > On 11/13/2017 10:33 AM, Paul Durrant wrote: > > > On 11/10/2017 19:35 PM, Joao Martins wrote: [snip] > > >> diff --git a/drivers/net/

Re: [Xen-devel] [PATCH net-next v1] xen-netback: make copy batch size configurable

2017-11-13 Thread Joao Martins
On 11/13/2017 10:33 AM, Paul Durrant wrote: >> -Original Message- >> From: Joao Martins [mailto:joao.m.mart...@oracle.com] >> Sent: 10 November 2017 19:35 >> To: net...@vger.kernel.org >> Cc: Joao Martins ; Wei Liu >> ; Paul Durrant ; xen- >> de.

[Xen-devel] [PATCH net-next v1] xen-netback: make copy batch size configurable

2017-11-10 Thread Joao Martins
er batches (i.e. for better throughput with big packets) as it was prior to the above mentioned commit. Signed-off-by: Joao Martins --- drivers/net/xen-netback/common.h| 6 -- drivers/net/xen-netback/interface.c | 25 - drivers/net/xen-netback/netback.c | 5

[Xen-devel] [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 Reviewed-by: Juergen Gross Reviewed-by: Boris Ostrovsky --- Changes since v5: * Move xen_setup_vsyscall_time_info within the PVCLOCK_TSC_STABLE_BIT clause added in predecessor patch. Changes since v4: * Remove pvclock_set_flags since predecessor

[Xen-devel] [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 Acked-by: Andy Lutomirski Acked-by: Paolo Bonzini Acked-by: Thomas Gleixner --- Changes since v7: * Add Paolo Acked-by (Comments from Thomas Gleixner) * Rename getter to pvclock_get_pvti_cpu0_va and fixup its callsites (vdso/vma

[Xen-devel] [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 Reviewed-by: Boris Ostrovsky --- Changes since v5

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

2017-11-08 Thread Joao Martins
. Signed-off-by: Joao Martins Acked-by: Radim Krčmář --- 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 2b1b212c219e..e04d7b2ecb3a 100644 --- a/drivers/ptp/ptp_kvm.c

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

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

[Xen-devel] [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

Re: [Xen-devel] [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

Re: [Xen-devel] [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: [Xen-devel] [PATCH RFC 3/8] libxl: add backend_features to libxl_device_disk

2017-11-07 Thread Joao Martins
On 11/07/2017 11:28 AM, Oleksandr Grytsov wrote: > On Thu, Nov 2, 2017 at 8:06 PM, Joao Martins <mailto:joao.m.mart...@oracle.com>> wrote: > > The function libxl__device_generic_add will have an additional > argument whereby it adds a second set of entries visible

Re: [Xen-devel] [PATCH RFC 2/8] public/io/netif: add directory for backend parameters

2017-11-06 Thread Joao Martins
On Mon, Nov 06, 2017 at 10:33:59AM +, Paul Durrant wrote: > > -Original Message- > > From: Joao Martins [mailto:joao.m.mart...@oracle.com] > > Sent: 02 November 2017 18:06 > > To: Xen Development List > > Cc: Joao Martins ; Konrad Rzeszutek Wil

[Xen-devel] [PATCH RFC 8/8] xen-netback: frontend feature control

2017-11-02 Thread Joao Martins
for 'feature-no-csum-offload' to allow toolstack to control per device checksum offloading. Signed-off-by: Joao Martins --- drivers/net/xen-netback/xenbus.c | 122 +++ 1 file changed, 99 insertions(+), 23 deletions(-) diff --git a/drivers/net/xen-netb

[Xen-devel] [PATCH RFC 7/8] xen-blkback: frontend feature control

2017-11-02 Thread Joao Martins
ed and all values set there are used in place of the module parameters being used. Note, however that feature-barrier, feature-flush-support and feature-discard aren't probed because first two are physical device dependent and feature-discard already has tunables to adjust. Signed-off-b

[Xen-devel] [PATCH RFC 0/8] libxl, xl, public/io: PV backends feature control

2017-11-02 Thread Joao Martins
ly specific to how it's done and could possibly be implemented in other PV interfaces. But there wasn't a protocol agnostic file to put all this, so I went ahead and did for the two individual io types (block and netif) I am most interested in. Any comments appreciated :) Thanks! Joa

[Xen-devel] [PATCH RFC 5/8] libxlu: parse disk backend features parameters

2017-11-02 Thread Joao Martins
ire-multi-queue-max-queues=1" ] ... Which would set the number of queues to 2 as opposed to e.g. the global blkback defined xen_blkback.max_queues parameter. Signed-off-by: Joao Martins --- tools/libxl/libxlu_disk_l.l | 42 ++ 1 file changed, 42 inser

[Xen-devel] [PATCH RFC 6/8] xl: parse vif backend features parameters

2017-11-02 Thread Joao Martins
multi-queue-max-queues=2"] ... Which would set the number of queues to 2 as opposed to e.g. the global netback defined xen_netback.max_queues parameter. Signed-off-by: Joao Martins --- tools/xl/xl_parse.c | 37 + tools/xl/xl_parse.h | 2 ++ 2 files ch

[Xen-devel] [PATCH RFC 3/8] libxl: add backend_features to libxl_device_disk

2017-11-02 Thread Joao Martins
a key value store storing: = xl|libxl are stateless with respect to feature names therefore is up to the admin to carefully select those. If backend isn't supported therefore the features won't be overwritten. Signed-off-by: Joao Martins --- tools/libxl/libxl.h | 8 +

[Xen-devel] [PATCH RFC 2/8] public/io/netif: add directory for backend parameters

2017-11-02 Thread Joao Martins
The proposed directory provides a mechanism for tools to control the maximum feature set of the device being provisioned by backend. The parameters/features include offloading features, number of queues etc. Signed-off-by: Joao Martins --- xen/include/public/io/netif.h | 16 1

[Xen-devel] [PATCH RFC 4/8] libxl: add backend_features to libxl_device_nic

2017-11-02 Thread Joao Martins
ned-off-by: Joao Martins --- tools/libxl/libxl.h | 8 tools/libxl/libxl_9pfs.c | 2 +- tools/libxl/libxl_console.c | 2 +- tools/libxl/libxl_device.c | 14 -- tools/libxl/libxl_internal.h | 2 +- tools/libxl/libxl_nic.c | 13 - t

[Xen-devel] [PATCH RFC 1/8] public/io/blkif: add directory for backend parameters

2017-11-02 Thread Joao Martins
The proposed directory provides a mechanism for tools to control the maximum feature set of the device being provisioned by backends. Examples include max ring page order, persistent grants, number of queues etc. Signed-off-by: Joao Martins --- xen/include/public/io/blkif.h | 14

Re: [Xen-devel] [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

[Xen-devel] [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 Reviewed-by: Juergen Gross Reviewed-by: Boris Ostrovsky --- Changes since v6: * Add Boris RoB Changes since v5: * Move xen_setup_vsyscall_time_info within the PVCLOCK_TSC_STABLE_BIT clause added in predecessor patch. Changes since v4: * Remove

[Xen-devel] [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 Acked-by: Andy Lutomirski --- Changes since v1: * Rebased: the only conflict was that I had move the export pvclock_pvti_cpu0_va()

[Xen-devel] [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

[Xen-devel] [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 Reviewed-by: Boris Ostrovsky --- Changes since v5

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

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

[Xen-devel] [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 --- 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/drivers/ptp/ptp_kvm.c +++ b/drivers/ptp

Re: [Xen-devel] [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

[Xen-devel] [PATCH v5 1/2] public/io/netif.h: add gref mapping control messages

2017-10-03 Thread Joao Martins
added are: - Add grefs to be mapped by backend - Remove grefs mappings (If they are not in use) - Get maximum amount of grefs kept mapped. Signed-off-by: Joao Martins Reviewed-by: Paul Durrant Reviewed-by: Konrad Rzeszutek Wilk --- v5: * Added RoB from Paul and Konrad v4: * Declare

[Xen-devel] [PATCH v5 0/2] netif: staging grants for I/O requests

2017-10-03 Thread Joao Martins
/jpemartins/linux.git xen-net-stg-gnts-v3 Thanks! Joao Martins (2): public/io/netif.h: add gref mapping control messages docs/misc: add netif staging grants design document docs/misc/netif-staging-grants.pandoc | 587 ++ xen/include/public/io/netif.h | 123

[Xen-devel] [PATCH v5 2/2] docs/misc: add netif staging grants design document

2017-10-03 Thread Joao Martins
Add a document outlining how the guest can map a set of grants on the backend through the control ring. Signed-off-by: Joao Martins Reviewed-by: Konrad Rzeszutek Wilk --- New in v5 --- docs/misc/netif-staging-grants.pandoc | 587 ++ 1 file changed, 587

[Xen-devel] [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

[Xen-devel] [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

[Xen-devel] [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_va()

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

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

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

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

Re: [Xen-devel] [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 > PVCLOCK_TS

[Xen-devel] [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_va()

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

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

[Xen-devel] [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 co

[Xen-devel] [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

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

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

Re: [Xen-devel] [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: [Xen-devel] [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: >>>>>>

Re: [Xen-devel] [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: [Xen-devel] [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 = &__this_cpu_read(

Re: [Xen-devel] [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

[Xen-devel] [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 --- 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 occurrences. * Change last VCLOCK_PVCLOCK

[Xen-devel] [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 Acked-by: Andy Lutomirski --- Changes since v1: * Rebased: the only conflict was that I had move the export pvclock_pvti_cpu0_va()

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

2017-09-27 Thread Joao Martins
peers noticed when such changes happen. Signed-off-by: Joao Martins Acked-by: Juergen Gross --- 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 would deserve its own place in MAINTAINERS file. I

[Xen-devel] [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

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

2017-09-27 Thread Joao Martins
peers noticed when such changes happen. Signed-off-by: Joao Martins Acked-by: Juergen Gross --- 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 would deserve its own place in MAINTAINERS file. I

[Xen-devel] [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

[Xen-devel] [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 --- 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 adding the needed one. (IOW adding

[Xen-devel] [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 Acked-by: Andy Lutomirski --- Changes since v1: * Rebased: the only conflict was that I had move the export pvclock_pvti_cpu0_va()

Re: [Xen-devel] [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

[Xen-devel] [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_va()

[Xen-devel] [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

[Xen-devel] [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

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

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

[Xen-devel] [PATCH v4 0/1] netif: staging grants for I/O requests

2017-09-19 Thread Joao Martins
Hey, This is v4 taking into consideration all comments received from v3 (changelog in the first patch). The specification is right after the diffstat. Reference implementation also here (on top of net-next): https://github.com/jpemartins/linux.git xen-net-stg-gnts-v3 Cheers, Joao Martins (1

[Xen-devel] [PATCH v4 1/1] public/io/netif.h: add gref mapping control messages

2017-09-19 Thread Joao Martins
added are: - Add grefs to be mapped by backend - Remove grefs mappings (If they are not in use) - Get maximum amount of grefs kept mapped. Signed-off-by: Joao Martins --- v4: * Declare xen_netif_gref parameters are input or output. * Clarify status field and that it doesn't require to b

Re: [Xen-devel] Feature control on PV devices

2017-09-19 Thread Joao Martins
On 09/18/2017 08:59 PM, Konrad Rzeszutek Wilk wrote: > On Thu, Sep 14, 2017 at 05:08:18PM +0100, Joao Martins wrote: >> [ Realized that I didn't CC the maintainers, >> so doing that now, +Linux folks +PV interfaces czar >> Sorry for the noise! ] >> >> On 0

Re: [Xen-devel] [PATCH v3 1/1] public/io/netif.h: add gref mapping control messages

2017-09-18 Thread Joao Martins
On Mon, Sep 18, 2017 at 12:11:04PM +, Paul Durrant wrote: > > -Original Message- > > From: Joao Martins [mailto:joao.m.mart...@oracle.com] > > Sent: 18 September 2017 12:56 > > To: Paul Durrant > > Cc: Xen-devel ; Wei Liu ; > > Konrad Rzeszutek

Re: [Xen-devel] [PATCH v3 1/1] public/io/netif.h: add gref mapping control messages

2017-09-18 Thread Joao Martins
On Mon, Sep 18, 2017 at 09:53:18AM +, Paul Durrant wrote: > > -Original Message- > > From: Joao Martins [mailto:joao.m.mart...@oracle.com] > > Sent: 13 September 2017 19:11 > > To: Xen-devel > > Cc: Wei Liu ; Paul Durrant ; > > Konrad Rzeszutek

Re: [Xen-devel] [PATCH v3 0/1] netif: staging grants for I/O requests

2017-09-18 Thread Joao Martins
On Mon, Sep 18, 2017 at 09:45:06AM +, Paul Durrant wrote: > > -Original Message- > > From: Joao Martins [mailto:joao.m.mart...@oracle.com] > > Sent: 13 September 2017 19:11 > > To: Xen-devel > > Cc: Wei Liu ; Paul Durrant ; > > Konrad Rzeszutek

Re: [Xen-devel] Feature control on PV devices

2017-09-15 Thread Joao Martins
On 09/15/2017 12:34 PM, Juergen Gross wrote: > On 15/09/17 13:19, Wei Liu wrote: >> On Thu, Sep 14, 2017 at 05:18:44PM +0100, Joao Martins wrote: >>> On 09/14/2017 05:10 PM, Wei Liu wrote: >>>> On Thu, Sep 07, 2017 at 05:53:54PM +0100, Joao Martins wrote: >&

Re: [Xen-devel] Feature control on PV devices

2017-09-14 Thread Joao Martins
On 09/14/2017 05:10 PM, Wei Liu wrote: > On Thu, Sep 07, 2017 at 05:53:54PM +0100, Joao Martins wrote: >> Hey! >> >> We wanted to brought up this small proposal regarding the lack of >> parameterization on PV devices on Xen. >> >> Currently users don&#x

Re: [Xen-devel] Feature control on PV devices

2017-09-14 Thread Joao Martins
[ Realized that I didn't CC the maintainers, so doing that now, +Linux folks +PV interfaces czar Sorry for the noise! ] On 09/08/2017 09:49 AM, Joao Martins wrote: > [Forgot two important details regarding Xenbus states] > On 09/07/2017 05:53 PM, Joao Martins wrote: >> Hey! &

[Xen-devel] [PATCH v3 0/1] netif: staging grants for I/O requests

2017-09-13 Thread Joao Martins
oking up on our mappings table. Thoughts, or perhaps people may prefer the one already described in the series? Cheers, Joao Martins (1): public/io/netif.h: add gref mapping control messages xen/include/public/io/netif.h | 115 ++ 1 file changed, 1

[Xen-devel] [PATCH v3 1/1] public/io/netif.h: add gref mapping control messages

2017-09-13 Thread Joao Martins
added are: - Add grefs to be mapped by backend - Remove grefs mappings (If they are not in use) - Get maximum amount of grefs kept mapped. Signed-off-by: Joao Martins --- v3: * Use DEL for unmapping grefs instead of PUT * Rname from xen_netif_gref_alloc to xen_netif_gref * Add 'status&#

Re: [Xen-devel] Feature control on PV devices

2017-09-08 Thread Joao Martins
[Forgot two important details regarding Xenbus states] On 09/07/2017 05:53 PM, Joao Martins wrote: > Hey! > > We wanted to brought up this small proposal regarding the lack of > parameterization on PV devices on Xen. > > Currently users don't have a way for enforce and

[Xen-devel] Feature control on PV devices

2017-09-07 Thread Joao Martins
Hey! We wanted to brought up this small proposal regarding the lack of parameterization on PV devices on Xen. Currently users don't have a way for enforce and control what features/queues/etc the backend provides. So far there's only global parameters on backends, and specs do not mention anythin

Re: [Xen-devel] [PATCH v2 1/1] public/io/netif.h: add gref mapping control messages

2017-09-06 Thread Joao Martins
On 09/06/2017 02:49 PM, Paul Durrant wrote: >> -Original Message- >> From: Joao Martins [mailto:joao.m.mart...@oracle.com] >> Sent: 01 September 2017 15:51 >> To: Xen-devel >> Cc: Wei Liu ; Paul Durrant ; >> Konrad Rzeszutek Wilk ; Joao Martins >

[Xen-devel] [PATCH v2 1/1] public/io/netif.h: add gref mapping control messages

2017-09-01 Thread Joao Martins
added are: - Add grefs to be mapped by backend - Remove grefs mappings (If they are not in use) - Get maximum amount of grefs kept mapped. Signed-off-by: Joao Martins --- xen/include/public/io/netif.h | 114 ++ 1 file changed, 114 insertions(+) diff

[Xen-devel] [PATCH v2 0/1] netif: staging grants for I/O requests

2017-09-01 Thread Joao Martins
ll be less effort involved into making netback work with something like zerogrant. Cheers, Joao Joao Martins (1): public/io/netif.h: add gref mapping control messages xen/include/public/io/netif.h | 114 ++ 1 file changed, 114 insertions(+) --- % Staging

Re: [Xen-devel] DESIGN v2: CPUID part 3

2017-08-02 Thread Joao Martins
On 08/01/2017 07:34 PM, Andrew Cooper wrote: > On 31/07/2017 20:49, Konrad Rzeszutek Wilk wrote: >> On Wed, Jul 05, 2017 at 02:22:00PM +0100, Joao Martins wrote: >>> On 07/05/2017 12:16 PM, Andrew Cooper wrote: >>>> On 05/07/17 10:46, Joao Martins wrote: >>>&

Re: [Xen-devel] PV drivers and zero copying

2017-07-31 Thread Joao Martins
On 07/31/2017 12:41 PM, Oleksandr Andrushchenko wrote: > Hi, Joao! > > On 07/31/2017 02:03 PM, Joao Martins wrote: >> Hey Oleksandr, >> >> On 07/31/2017 09:34 AM, Oleksandr Andrushchenko wrote: >>> Hi, all! >>>

Re: [Xen-devel] PV drivers and zero copying

2017-07-31 Thread Joao Martins
On 07/31/2017 11:37 AM, Oleksandr Andrushchenko wrote: > On 07/31/2017 01:04 PM, Julien Grall wrote: >> On 31/07/17 10:52, Oleksandr Andrushchenko wrote: >>> On 07/31/2017 12:47 PM, Julien Grall wrote: On 31/07/17 10:46, Oleksandr Andrushchenko wrote: Do you have any example of hardware?

Re: [Xen-devel] PV drivers and zero copying

2017-07-31 Thread Joao Martins
Hey Oleksandr, On 07/31/2017 09:34 AM, Oleksandr Andrushchenko wrote: > Hi, all! > [snip] > > Comparison for display use-case > === > > 1 Number of grant references used > 1-1 grant references: nr_pages > 1-2 GNTTABOP_transfer: nr_pages > 1-3 XENMEM_exchange: not an

Re: [Xen-devel] DESIGN v2: CPUID part 3

2017-07-05 Thread Joao Martins
On 07/05/2017 12:16 PM, Andrew Cooper wrote: > On 05/07/17 10:46, Joao Martins wrote: >> Hey Andrew, >> >> On 07/04/2017 03:55 PM, Andrew Cooper wrote: >>> Presented herewith is the a plan for the final part of CPUID work, which >>> primarily covers better Xe

Re: [Xen-devel] DESIGN v2: CPUID part 3

2017-07-05 Thread Joao Martins
On 07/05/2017 10:46 AM, Joao Martins wrote: > Hey Andrew, > > On 07/04/2017 03:55 PM, Andrew Cooper wrote: >> Presented herewith is the a plan for the final part of CPUID work, which >> primarily covers better Xen/Toolstack interaction for configuring the guests >> CPU

Re: [Xen-devel] DESIGN v2: CPUID part 3

2017-07-05 Thread Joao Martins
Hey Andrew, On 07/04/2017 03:55 PM, Andrew Cooper wrote: > Presented herewith is the a plan for the final part of CPUID work, which > primarily covers better Xen/Toolstack interaction for configuring the guests > CPUID policy. > Really nice write up, a few comments below. > A PDF version of this

Re: [Xen-devel] [dpdk-dev] [PATCH] maintainers: claim responsability for xen

2017-02-20 Thread Joao Martins
On 02/20/2017 09:56 AM, Jan Blunck wrote: > On Fri, Feb 17, 2017 at 5:07 PM, Konrad Rzeszutek Wilk > wrote: >> On Thu, Feb 16, 2017 at 10:51:44PM +0100, Vincent JARDIN wrote: >>> Le 16/02/2017 à 14:36, Konrad Rzeszutek Wilk a écrit : > Is it time now to officially remove Dom0 support? So

Re: [Xen-devel] [dpdk-dev] [PATCH] maintainers: claim responsability for xen

2017-02-20 Thread Joao Martins
used? OK, see attached - I should note that its a WIP as Konrad noted, but once the staging grants work is finished, the code would be improved to have it in better shape (as well as in feature parity) for a proper RFC [and adhering to the project coding style]. Joao >From 3bced1452e1e619e7f470

Re: [Xen-devel] [PATCH] x86/time: tsc_check_writability() may need to be run a second time

2017-02-10 Thread Joao Martins
On 02/10/2017 11:17 AM, Andrew Cooper wrote: > On 10/02/17 11:11, Joao Martins wrote: >> On 02/10/2017 11:03 AM, Jan Beulich wrote: >>> While we shouldn't remove its current invocation, we need to re-run it >>> for the case that the X86_FEATURE_TSC_RELIABLE featu

Re: [Xen-devel] [PATCH] x86/time: tsc_check_writability() may need to be run a second time

2017-02-10 Thread Joao Martins
e written. > > Signed-off-by: Jan Beulich FWIW, Reviewed-by: Joao Martins ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [libvirt] [PATCH 2/2] libxl: fix dom0 maximum memory setting

2017-02-08 Thread Joao Martins
On 02/08/2017 04:06 PM, Jim Fehlig wrote: > Joao Martins wrote: >> On 02/02/2017 10:31 PM, Jim Fehlig wrote: >>> When the libxl driver is initialized, it creates a virDomainDef >>> object for dom0 and adds it to the list of domains. Total memory >>> for dom0 wa

Re: [Xen-devel] [libvirt] [PATCH 2/2] libxl: fix dom0 maximum memory setting

2017-02-08 Thread Joao Martins
On 02/02/2017 10:31 PM, Jim Fehlig wrote: > When the libxl driver is initialized, it creates a virDomainDef > object for dom0 and adds it to the list of domains. Total memory > for dom0 was being set from the max_memkb field of libxl_dominfo > struct retrieved from libxl, but this field can be set

Re: [Xen-devel] [PATCH v3] x86/time: correctly honor late clearing of TSC related feature flags

2017-02-07 Thread Joao Martins
ally causing noticeably (even if only temporary) strange >>> behavior. As we're anyway expecting only sufficiently similar CPUs to >>> appear during hotplug, this shouldn't be introducing new limitations. >>> >>> Reported-by: Joao Martins >>> Sig

Re: [Xen-devel] [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 > wrote: >> This file defines an ABI shared between guest and hypervisor(s) >> (KVM, Xen) and as such there should be an correspondent entry in >> MAINTAINERS file. Notice tha

Re: [Xen-devel] [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 > wrote: >> Right now there is only a pvclock_pvti_cpu0_va() which is defined >> on kvmclock since: >> >> commit dac16fba6fc5 >> ("x86/vdso: Get pvclock dat

Re: [Xen-devel] [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

[Xen-devel] [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 --- 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 supported. (New) * Set seco

[Xen-devel] [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 --- Changes since RFC: (Comments from Andy Lutomirski) * Add WARN_ON(vclock_was_used(VCLOCK_PVCLOCK)) to pvclock_set_pv

[Xen-devel] [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

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

2017-01-25 Thread Joao Martins
peers noticed when such changes happen. Signed-off-by: Joao Martins --- 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 could instead add a "P

  1   2   3   4   >