Re: [Xen-devel] [PATCH RFC v2] Add SUPPORT.md

2017-09-26 Thread Dario Faggioli
[Cc-list modified by removing someone and adding someone else] On Mon, 2017-09-25 at 16:10 -0700, Stefano Stabellini wrote: > On Mon, 11 Sep 2017, George Dunlap wrote: > > +### RTDS based Scheduler > > + > > +Status: Experimental > > + > > +A soft real-time CPU scheduler built to provide guara

Re: [Xen-devel] [PATCH 01/27 v10] xen/arm: vpl011: Define common ring buffer helper functions in console.h

2017-09-26 Thread Jan Beulich
>>> On 26.09.17 at 01:08, wrote: > Yes, after including the __STRICT_ANSI__ check the headers.chk check > passes. But I had to include string header file (after suggestion from > Stefano) for fixing the headers++.chk. I'd like to have a more detailed explanation here - since the header passed th

Re: [Xen-devel] pci-passthrough loses msi-x interrupts ability after domain destroy

2017-09-26 Thread Jan Beulich
>>> On 25.09.17 at 18:10, wrote: > I tried the various device reset patches posted on this discussion > (do_flr, Christopher's "more thorough" reset_device) but without luck. > > After reset, I could notice that lspci shows the device's Masked > state has been cleared but a newly started guest wi

[Xen-devel] [PATCH] libxc: remove stale error check for domain size in xc_sr_save_x86_hvm.c

2017-09-26 Thread Juergen Gross
Long ago domains to be saved were limited to 1TB size due to the migration stream v1 limitations which used a 32 bit value for the PFN and the frame type (4 bits) leaving only 28 bits for the PFN. Migration stream V2 uses a 64 bit value for this purpose, so there is no need to refuse saving (or mi

Re: [Xen-devel] Guest start issue on ARM (maybe related to Credit2) [Was: Re: [xen-unstable test] 113807: regressions - FAIL]

2017-09-26 Thread Dario Faggioli
On Mon, 2017-09-25 at 17:23 +0100, Julien Grall wrote: > On 09/25/2017 03:07 PM, Dario Faggioli wrote: > > I don't see much in the logs, TBH, but both `xl vcpu-list' and the > > 'r' > > debug key seem to suggest that vCPU 0 is running, while the other > > vCPUs > > have never run... like it was an

Re: [Xen-devel] Windows "heinsenbug" (WAS: Re: Notes Design Session: Making Releases Lessons Learned: Improving Our Release Process and Tooling)

2017-09-26 Thread Paul Durrant
> -Original Message- > From: Julien Grall [mailto:julien.gr...@arm.com] > Sent: 25 September 2017 22:32 > To: Paul Durrant ; Ian Jackson > ; Jan Beuli ch > Cc: Juergen Gross ; Wei Liu ; > ross.philip...@gmail.com; lars.kurth@gmail.com; xen- > de...@lists.xen.org; committ...@xenproject.

Re: [Xen-devel] [PATCH] libxc: remove stale error check for domain size in xc_sr_save_x86_hvm.c

2017-09-26 Thread Andrew Cooper
On 26/09/2017 08:33, Juergen Gross wrote: > Long ago domains to be saved were limited to 1TB size due to the > migration stream v1 limitations which used a 32 bit value for the > PFN and the frame type (4 bits) leaving only 28 bits for the PFN. > > Migration stream V2 uses a 64 bit value for this p

Re: [Xen-devel] [PATCH] x86/levelling: Avoid NULL pointer dereference

2017-09-26 Thread Jan Beulich
>>> On 25.09.17 at 18:28, wrote: > --- a/xen/arch/x86/cpu/intel.c > +++ b/xen/arch/x86/cpu/intel.c > @@ -156,7 +156,6 @@ static void intel_ctxt_switch_levelling(const struct vcpu > *next) > struct cpuidmasks *these_masks = &this_cpu(cpuidmasks); > const struct domain *nextd = next ? n

Re: [Xen-devel] [PATCH] x86/pv: Fix assertion failure in pv_emulate_privileged_op()

2017-09-26 Thread Jan Beulich
>>> On 25.09.17 at 19:19, wrote: > The ABI of {read,write}_msr() are required them to use x86_emul_hw_exception() s/are required/require/? > if they raise an exception with the emulator core. Perhaps better "if they report an exception to", as otherwise both parts of the sentence state the same

Re: [Xen-devel] [PATCH 01/27 v10] xen/arm: vpl011: Define common ring buffer helper functions in console.h

2017-09-26 Thread Bhupinder Thakur
Hi Jan, On 26 September 2017 at 12:45, Jan Beulich wrote: On 26.09.17 at 01:08, wrote: >> Yes, after including the __STRICT_ANSI__ check the headers.chk check >> passes. But I had to include string header file (after suggestion from >> Stefano) for fixing the headers++.chk. > > I'd like to

[Xen-devel] [linux-linus test] 113818: regressions - FAIL

2017-09-26 Thread osstest service owner
flight 113818 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/113818/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs. 113809 test-amd64-i3

Re: [Xen-devel] [PATCH v3 2/3] Introduce migration precopy policy

2017-09-26 Thread Wei Liu
On Mon, Sep 25, 2017 at 07:55:34PM +0100, Jennifer Herbert wrote: > This Patch allows a migration precopy policy to be specified. > > The precopy phase of the xc_domain_save() live migration algorithm has > historically been implemented to run until either a) (almost) no pages > are dirty or b) so

Re: [Xen-devel] [PATCH] libxc: remove stale error check for domain size in xc_sr_save_x86_hvm.c

2017-09-26 Thread Juergen Gross
On 26/09/17 09:41, Andrew Cooper wrote: > On 26/09/2017 08:33, Juergen Gross wrote: >> Long ago domains to be saved were limited to 1TB size due to the >> migration stream v1 limitations which used a 32 bit value for the >> PFN and the frame type (4 bits) leaving only 28 bits for the PFN. >> >> Mig

Re: [Xen-devel] [PATCH v4 04/15] x86: implement data structure and CPU init flow for MBA

2017-09-26 Thread Roger Pau Monné
On Sat, Sep 23, 2017 at 09:48:13AM +, Yi Sun wrote: > This patch implements main data structures of MBA. > > Like CAT features, MBA HW info has cos_max which means the max thrtl > register number, and thrtl_max which means the max throttle value > (delay value). It also has a flag to represent

Re: [Xen-devel] [PATCH v4 05/15] x86: implement get hw info flow for MBA

2017-09-26 Thread Roger Pau Monné
On Sat, Sep 23, 2017 at 09:48:14AM +, Yi Sun wrote: > This patch implements get HW info flow for MBA including its callback > function and sysctl interface. > > Signed-off-by: Yi Sun > --- > CC: Jan Beulich > CC: Andrew Cooper > CC: Wei Liu > CC: Roger Pau Monné > CC: Chao Peng > > v4:

Re: [Xen-devel] [PATCH v3 3/3] RFC: migration: defer precopy policy to libxl

2017-09-26 Thread Wei Liu
On Mon, Sep 25, 2017 at 07:55:35PM +0100, Jennifer Herbert wrote: > Provide an implementation of the old policy as a callback in > libxl and plumb it through the IPC machinery to libxc. > > This serves as an example for defining a libxl policy, > and provides no advantage over the default policy i

Re: [Xen-devel] [PATCH] libxc: remove stale error check for domain size in xc_sr_save_x86_hvm.c

2017-09-26 Thread Juergen Gross
On 26/09/17 09:41, Andrew Cooper wrote: > On 26/09/2017 08:33, Juergen Gross wrote: >> Long ago domains to be saved were limited to 1TB size due to the >> migration stream v1 limitations which used a 32 bit value for the >> PFN and the frame type (4 bits) leaving only 28 bits for the PFN. >> >> Mig

[Xen-devel] [ovmf test] 113820: all pass - PUSHED

2017-09-26 Thread osstest service owner
flight 113820 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/113820/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 8932679df5be046feba30fae80776c5815232a08 baseline version: ovmf baaa3cee1eafc044606ee

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

2017-09-26 Thread Juergen Gross
On 22/09/17 18:25, 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 hypercall. This hypercall > will also forcefully update the pvti which will set some of

[Xen-devel] [PATCH v2 1/2] xen/arm64: Add Support for Allwinner H5 (sun50i)

2017-09-26 Thread Awais Masood
This patch adds support for Allwinner H5/sun50i SoC. Makefile updated to enable ARM64 compilation for sunxi.c. sunxi.c updates include: - Addition of H5/sun50i dt compatibility string. - Handling of different Watchdog timer base addresses on sun7i and sun50i. Tested on Orange Pi PC2 Sig

[Xen-devel] [PATCH v2 2/2] xen/ns16550: Fix ISR lockup on Designware 8250 (H5)

2017-09-26 Thread Awais Masood
On Allwinner H5 (Orange Pi PC2) serial driver goes into an infinite loop when interrupts are enabled. The reason is a residual "busy detect" interrupt. Since the condition UART_IIR_NOINT will not be true unless this interrupt is cleared, the interrupt handler will remain locked up in this while loo

[Xen-devel] [PATCH v2 0/2] xen/arm64/ns16550: Support for Allwinner H5 SoC

2017-09-26 Thread Awais Masood
This patch series adds support for Allwinner H5 (ARM64/Cortex-A53) Builds upon existing support for A20/A31(sun7i) with updates to support different watchdog timer base addresses and device tree compatibility strings for sun50i. The ns16550 update is a fix for a hardware quirk that causes a locku

Re: [Xen-devel] [PATCH v4 07/15] x86: implement set value flow for MBA

2017-09-26 Thread Roger Pau Monné
On Sat, Sep 23, 2017 at 09:48:16AM +, Yi Sun wrote: > @@ -274,29 +277,6 @@ static enum psr_feat_type psr_type_to_feat_type(enum > psr_type type) > return feat_type; > } > > -static bool psr_check_cbm(unsigned int cbm_len, unsigned long cbm) > -{ > -unsigned int first_bit, zero_bit;

Re: [Xen-devel] [PATCH v3 3/3] RFC: migration: defer precopy policy to libxl

2017-09-26 Thread Jennifer Herbert
On 26/09/17 09:48, Wei Liu wrote: On Mon, Sep 25, 2017 at 07:55:35PM +0100, Jennifer Herbert wrote: Provide an implementation of the old policy as a callback in libxl and plumb it through the IPC machinery to libxc. This serves as an example for defining a libxl policy, and provides no advantag

[Xen-devel] [PATCH v2] Config.mk: update OVMF changeset

2017-09-26 Thread Anthony PERARD
Update to allow to build OVMF with GCC 7.2. Signed-off-by: Anthony PERARD --- Config.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Config.mk b/Config.mk index bba81bee7d..57d3e2bc44 100644 --- a/Config.mk +++ b/Config.mk @@ -272,7 +272,7 @@ QEMU_TRADITIONAL_URL ?= git:/

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; >> +struct pvclock_vcpu_time_info *pvti; >> +

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

2017-09-26 Thread Juergen Gross
On 26/09/17 11:57, Joao Martins wrote: > 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; >>> +

Re: [Xen-devel] [PATCH v2] Config.mk: update OVMF changeset

2017-09-26 Thread Wei Liu
On Tue, Sep 26, 2017 at 10:52:10AM +0100, Anthony PERARD wrote: > Update to allow to build OVMF with GCC 7.2. > > Signed-off-by: Anthony PERARD Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v4 08/15] tools: create general interfaces to support psr allocation features

2017-09-26 Thread Roger Pau Monné
On Sat, Sep 23, 2017 at 09:48:17AM +, Yi Sun wrote: > This patch creates general interfaces in libxl to support all psr > allocation features. > > Add 'LIBXL_HAVE_PSR_GENERIC' to indicate interface change. > > Please note, the functionality cannot work until later patches > are applied. > >

Re: [Xen-devel] [PATCH] x86/mm: Consolidate all Xen L4 slot writing into init_xen_l4_slots()

2017-09-26 Thread George Dunlap
On 09/01/2017 06:07 PM, Andrew Cooper wrote: > Having all of this logic together makes it easier to follow Xen's virtual > setup across the whole system. > > No practical changes to the resulting L4, although the logic has been > rearanged to avoid rewriting some slots. This changes the zap_ro_mp

Re: [Xen-devel] [PATCH v4 09/15] tools: implement the new libxc get hw info interface

2017-09-26 Thread Roger Pau Monné
On Sat, Sep 23, 2017 at 09:48:18AM +, Yi Sun wrote: > +enum xc_psr_feat_type { > +XC_PSR_CAT_L3, > +XC_PSR_CAT_L2, > +XC_PSR_MBA, > +}; > +typedef enum xc_psr_feat_type xc_psr_feat_type; > + > +struct xc_psr_hw_info { > +union { > +struct { > +uint32_t cos_ma

Re: [Xen-devel] [PATCH RFC v2] Add SUPPORT.md

2017-09-26 Thread George Dunlap
On 09/26/2017 12:10 AM, Stefano Stabellini wrote: > On Mon, 11 Sep 2017, George Dunlap wrote: >> +### RTDS based Scheduler >> + >> +Status: Experimental >> + >> +A soft real-time CPU scheduler built to provide guaranteed CPU capacity to >> guest VMs on SMP hosts >> + >> +### ARINC653 Scheduler

Re: [Xen-devel] [PATCH v4 10/15] tools: implement the new libxl get hw info interface

2017-09-26 Thread Roger Pau Monné
On Sat, Sep 23, 2017 at 09:48:19AM +, Yi Sun wrote: > This patch implements the new libxl get hw info interface, > 'libxl_psr_get_hw_info', which is suitable to all psr allocation > features. It also implements corresponding list free function, > 'libxl_psr_hw_info_list_free' and makes 'libxl_p

Re: [Xen-devel] [PATCH v7 08/12] x86/hvm/ioreq: maintain an array of ioreq servers rather than a list

2017-09-26 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 25 September 2017 16:17 > To: Paul Durrant > Cc: Andrew Cooper ; xen- > de...@lists.xenproject.org > Subject: Re: [PATCH v7 08/12] x86/hvm/ioreq: maintain an array of ioreq > servers rather than a list > > >>> On

[Xen-devel] [qemu-mainline baseline-only test] 72156: regressions - trouble: blocked/broken/fail/pass

2017-09-26 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 72156 qemu-mainline real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72156/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-win10-i386 10 window

Re: [Xen-devel] [PATCH v3 2/3] Introduce migration precopy policy

2017-09-26 Thread Andrew Cooper
On 25/09/17 19:55, Jennifer Herbert wrote: > +/* > + * A precopy_policy callback may not be running in the same address > + * space as libxc an so precopy_stats is passed by value. > + */ Please take a step back and thing about what is written here... As I've said repeatedly, the structure vs poi

Re: [Xen-devel] [PATCH v3 2/3] Introduce migration precopy policy

2017-09-26 Thread Andrew Cooper
On 25/09/17 19:55, Jennifer Herbert wrote: > @@ -46,7 +60,22 @@ struct save_callbacks { > */ > int (*suspend)(void* data); > > -/* Called after the guest's dirty pages have been > +/* > + * Called before and after every batch of page data sent during > + * the precopy p

Re: [Xen-devel] [PATCH v4 11/15] tools: implement the new xl get hw info interface

2017-09-26 Thread Roger Pau Monné
On Sat, Sep 23, 2017 at 09:48:20AM +, Yi Sun wrote: > This patch implements a new xl get HW info interface. A new argument > is added for psr-hwinfo command to get and show MBA HW info. > > Signed-off-by: Yi Sun Reviewed-by: Roger Pau Monné Thanks, Roger. _

Re: [Xen-devel] [PATCH v4 12/15] tools: rename 'xc_psr_cat_type' to 'xc_psr_type'

2017-09-26 Thread Roger Pau Monné
On Sat, Sep 23, 2017 at 09:48:21AM +, Yi Sun wrote: > This patch renames 'xc_psr_cat_type' to 'xc_psr_type' so that > the structure name is common for all allocation features. > > Signed-off-by: Yi Sun > Acked-by: Wei Liu > Reviewed-by: Chao Peng Reviewed-by: Roger Pau Monné Just a small

[Xen-devel] [xen-unstable-smoke test] 113830: tolerable all pass - PUSHED

2017-09-26 Thread osstest service owner
flight 113830 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/113830/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 13 migrate-support-checkfail never pass test-armhf-armhf-xl 1

Re: [Xen-devel] [PATCH v4 13/15] tools: implement new generic get value interface and MBA get value command

2017-09-26 Thread Roger Pau Monné
On Sat, Sep 23, 2017 at 09:48:22AM +, Yi Sun wrote: > This patch implements generic get value interfaces in libxc and libxl. > It also refactors the get value flow in xl to make it be suitable for all > allocation features. Based on that, a new MBA get value command is added in > xl. > > Sign

Re: [Xen-devel] [alsa-devel] [PATCH RESEND1 00/12] ALSA: vsnd: Add Xen para-virtualized frontend driver

2017-09-26 Thread Oleksandr Andrushchenko
Clemens, Sakamoto-san, could you please review the below if you by chance have a minute? Thank you, Oleksandr On 09/19/2017 11:57 AM, Oleksandr Andrushchenko wrote: Hi, all! We did some work on implementing the idea with feedback events from the backend to the frontend. Please see attached

Re: [Xen-devel] [PATCH v3 2/3] Introduce migration precopy policy

2017-09-26 Thread Jennifer Herbert
On 26/09/17 11:58, Andrew Cooper wrote: On 25/09/17 19:55, Jennifer Herbert wrote: +/* + * A precopy_policy callback may not be running in the same address + * space as libxc an so precopy_stats is passed by value. + */ Please take a step back and thing about what is written here... As I've sa

Re: [Xen-devel] [PATCH v4 14/15] tools: implement new generic set value interface and MBA set value command

2017-09-26 Thread Roger Pau Monné
On Sat, Sep 23, 2017 at 09:48:23AM +, Yi Sun wrote: > int libxl_psr_cat_get_cbm(libxl_ctx *ctx, uint32_t domid, > @@ -450,7 +425,32 @@ int libxl_psr_set_val(libxl_ctx *ctx, uint32_t domid, >libxl_psr_type type, libxl_bitmap *target_map, >uint64_t

Re: [Xen-devel] [PATCH 01/27 v10] xen/arm: vpl011: Define common ring buffer helper functions in console.h

2017-09-26 Thread Jan Beulich
>>> On 26.09.17 at 10:16, wrote: > On 26 September 2017 at 12:45, Jan Beulich wrote: > On 26.09.17 at 01:08, wrote: >>> Yes, after including the __STRICT_ANSI__ check the headers.chk check >>> passes. But I had to include string header file (after suggestion from >>> Stefano) for fixing the

Re: [Xen-devel] [PATCH v4 15/15] docs: add MBA description in docs

2017-09-26 Thread Roger Pau Monné
On Sat, Sep 23, 2017 at 09:48:24AM +, Yi Sun wrote: > This patch adds MBA description in related documents. > > Signed-off-by: Yi Sun > Acked-by: Wei Liu There's still a 'closed-loop' mention that I think I've already pointed out in the last review, but anyway: Reviewed-by: Roger Pau Monné

Re: [Xen-devel] [PATCH v7 08/12] x86/hvm/ioreq: maintain an array of ioreq servers rather than a list

2017-09-26 Thread Jan Beulich
>>> On 26.09.17 at 12:55, wrote: >> Sent: 25 September 2017 16:17 >> To: Paul Durrant >> >>> On 18.09.17 at 17:31, wrote: >> > @@ -785,29 +797,27 @@ int hvm_get_ioreq_server_info(struct domain >> *d, ioservid_t id, >> > >> > spin_lock_recursive(&d->arch.hvm_domain.ioreq_server.lock); >> > >

[Xen-devel] [libvirt test] 113821: tolerable all pass - PUSHED

2017-09-26 Thread osstest service owner
flight 113821 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/113821/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt-xsm 14 saverestore-support-checkfail like 113763 test-armhf-armhf-libvirt 14 saveresto

Re: [Xen-devel] [PATCH v4 00/15] Enable Memory Bandwidth Allocation in Xen

2017-09-26 Thread Roger Pau Monné
On Sat, Sep 23, 2017 at 09:48:09AM +, Yi Sun wrote: > Hi, all, > > We plan to bring a new PSR (Platform Shared Resource) feature called > Intel Memory Bandwidth Allocation (MBA) to Xen. > > Besides the MBA enabling, we change some interfaces to make them more > general but not only for CAT.

Re: [Xen-devel] [PATCH v3 2/3] Introduce migration precopy policy

2017-09-26 Thread Jennifer Herbert
On 26/09/17 12:02, Andrew Cooper wrote: On 25/09/17 19:55, Jennifer Herbert wrote: @@ -46,7 +60,22 @@ struct save_callbacks { */ int (*suspend)(void* data); -/* Called after the guest's dirty pages have been +/* + * Called before and after every batch of page data s

Re: [Xen-devel] [PATCH v4 05/15] x86: implement get hw info flow for MBA

2017-09-26 Thread Jan Beulich
>>> On 26.09.17 at 10:46, wrote: > On Sat, Sep 23, 2017 at 09:48:14AM +, Yi Sun wrote: >> --- a/xen/arch/x86/sysctl.c >> +++ b/xen/arch/x86/sysctl.c >> @@ -174,7 +174,7 @@ long arch_do_sysctl( >> case XEN_SYSCTL_psr_alloc: >> switch ( sysctl->u.psr_alloc.cmd ) >> { >> -

[Xen-devel] [xtf test] 113831: all pass - PUSHED

2017-09-26 Thread osstest service owner
flight 113831 xtf real [real] http://logs.test-lab.xenproject.org/osstest/logs/113831/ Perfect :-) All tests in this flight passed as required version targeted for testing: xtf 9d53bd3275bb408881d4382e7eff98c8d4bd9a40 baseline version: xtf 7001ab0503fe91e4962ab2

Re: [Xen-devel] [PATCH v2 2/2] xen/ns16550: Fix ISR lockup on Designware 8250 (H5)

2017-09-26 Thread Jan Beulich
>>> On 26.09.17 at 11:37, wrote: First of all please only send to one of xen-devel@lists.xen.org and xen-de...@lists.xenproject.org. > On Allwinner H5 (Orange Pi PC2) serial driver goes into an > infinite loop when interrupts are enabled. The reason is a > residual "busy detect" interrupt. Since

[Xen-devel] [PATCH v2] libxc: remove stale error check for domain size in xc_sr_save_x86_hvm.c

2017-09-26 Thread Juergen Gross
Long ago domains to be saved were limited to 1TB size due to the migration stream v1 limitations which used a 32 bit value for the PFN and the frame type (4 bits) leaving only 28 bits for the PFN. Migration stream V2 uses a 64 bit value for this purpose, so there is no need to refuse saving (or mi

Re: [Xen-devel] [PATCH v7 08/12] x86/hvm/ioreq: maintain an array of ioreq servers rather than a list

2017-09-26 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 26 September 2017 12:45 > To: Paul Durrant > Cc: Andrew Cooper ; xen- > de...@lists.xenproject.org > Subject: RE: [PATCH v7 08/12] x86/hvm/ioreq: maintain an array of ioreq > servers rather than a list > > >>> On

Re: [Xen-devel] [PATCH v7 02/12] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-09-26 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of > Paul Durrant > Sent: 25 September 2017 16:00 > To: 'Jan Beulich' > Cc: Andrew Cooper ; xen- > de...@lists.xenproject.org > Subject: Re: [Xen-devel] [PATCH v7 02/12] x86/mm: add > HYPERVISOR_memor

[Xen-devel] [ovmf baseline-only test] 72158: regressions - FAIL

2017-09-26 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 72158 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72158/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ovmf-amd64 18 guest-start/deb

Re: [Xen-devel] [PATCH v7 02/12] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-09-26 Thread Jan Beulich
>>> On 26.09.17 at 14:20, wrote: >> -Original Message- >> From: Xen-devel [mailto:xen-devel-boun...@lists.xen.org] On Behalf Of >> Paul Durrant >> Sent: 25 September 2017 16:00 >> To: 'Jan Beulich' >> Cc: Andrew Cooper ; xen- >> de...@lists.xenproject.org >> Subject: Re: [Xen-devel] [PA

Re: [Xen-devel] [PATCH v7 08/12] x86/hvm/ioreq: maintain an array of ioreq servers rather than a list

2017-09-26 Thread Jan Beulich
>>> On 26.09.17 at 14:12, wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: 26 September 2017 12:45 >> >>> On 26.09.17 at 12:55, wrote: >> >> Sent: 25 September 2017 16:17 >> >> To: Paul Durrant >> >> >>> On 18.09.17 at 17:31, wrote: >> >> > @@ -785,29 +797,27 @@ int hvm_get_ioreq

[Xen-devel] [distros-debian-snapshot test] 72157: tolerable trouble: blocked/broken/fail/pass

2017-09-26 Thread Platform Team regression test user
flight 72157 distros-debian-snapshot real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72157/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-armhf-daily-netboot-pygrub 1 build-check(1) blocked n/a build-arm64-pvops

Re: [Xen-devel] [PATCH v7 08/12] x86/hvm/ioreq: maintain an array of ioreq servers rather than a list

2017-09-26 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 26 September 2017 13:38 > To: Paul Durrant > Cc: Andrew Cooper ; xen- > de...@lists.xenproject.org > Subject: RE: [PATCH v7 08/12] x86/hvm/ioreq: maintain an array of ioreq > servers rather than a list > > >>> On

Re: [Xen-devel] [PATCH v7 02/12] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-09-26 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 26 September 2017 13:35 > To: Andrew Cooper ; Paul Durrant > > Cc: xen-de...@lists.xenproject.org > Subject: RE: [PATCH v7 02/12] x86/mm: add HYPERVISOR_memory_op to > acquire guest resources > > >>> On 26.09.17 a

[Xen-devel] [xen-unstable test] 113819: regressions - trouble: broken/fail/pass

2017-09-26 Thread osstest service owner
flight 113819 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/113819/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-xtf-amd64-amd64-5 broken test-xtf-amd64-amd64-3

Re: [Xen-devel] [PATCH v7 12/12] x86/hvm/ioreq: add a new mappable resource type...

2017-09-26 Thread Jan Beulich
>>> On 18.09.17 at 17:31, wrote: > @@ -762,7 +863,8 @@ int hvm_get_ioreq_server_info(struct domain *d, > ioservid_t id, > goto out; > } > > -*ioreq_gfn = gfn_x(s->ioreq.gfn); > +if ( ioreq_gfn ) > +*ioreq_gfn = gfn_x(s->ioreq.gfn); Ah, this is what actually wa

Re: [Xen-devel] [PATCH v7 08/12] x86/hvm/ioreq: maintain an array of ioreq servers rather than a list

2017-09-26 Thread Jan Beulich
>>> On 26.09.17 at 14:41, wrote: >> -Original Message- >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: 26 September 2017 13:38 >> To: Paul Durrant >> Cc: Andrew Cooper ; xen- >> de...@lists.xenproject.org >> Subject: RE: [PATCH v7 08/12] x86/hvm/ioreq: maintain an array of iore

Re: [Xen-devel] [PATCH v7 12/12] x86/hvm/ioreq: add a new mappable resource type...

2017-09-26 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 26 September 2017 13:59 > To: Paul Durrant > Cc: Andrew Cooper ; Ian Jackson > ; Stefano Stabellini ; xen- > de...@lists.xenproject.org; Konrad Rzeszutek Wilk > ; Tim (Xen.org) > Subject: Re: [PATCH v7 12/12] x86/

Re: [Xen-devel] [PATCH v7 12/12] x86/hvm/ioreq: add a new mappable resource type...

2017-09-26 Thread Jan Beulich
>>> On 26.09.17 at 15:05, wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: 26 September 2017 13:59 >> >>> On 18.09.17 at 17:31, wrote: >> > @@ -780,6 +882,33 @@ int hvm_get_ioreq_server_info(struct domain *d, >> ioservid_t id, >> > return rc; >> > } >> > >> > +mfn_t hvm_get_i

Re: [Xen-devel] [PATCH v7 08/12] x86/hvm/ioreq: maintain an array of ioreq servers rather than a list

2017-09-26 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 26 September 2017 14:04 > To: Paul Durrant > Cc: Andrew Cooper ; xen- > de...@lists.xenproject.org > Subject: RE: [PATCH v7 08/12] x86/hvm/ioreq: maintain an array of ioreq > servers rather than a list > > >>> On

Re: [Xen-devel] [PATCH v7 12/12] x86/hvm/ioreq: add a new mappable resource type...

2017-09-26 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 26 September 2017 14:11 > To: Paul Durrant > Cc: Andrew Cooper ; Ian Jackson > ; Stefano Stabellini ; xen- > de...@lists.xenproject.org; KonradRzeszutek Wilk > ; Tim (Xen.org) > Subject: RE: [PATCH v7 12/12] x86/h

Re: [Xen-devel] [xen-unstable test] 113819: regressions - trouble: broken/fail/pass

2017-09-26 Thread Andrew Cooper
On 26/09/17 13:53, osstest service owner wrote: > flight 113819 xen-unstable real [real] > http://logs.test-lab.xenproject.org/osstest/logs/113819/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: > test-xtf-amd64-amd64-5

[Xen-devel] Re : blkdev_issue_discard hung [solved ?]

2017-09-26 Thread Olivier Bonvalet
It seems to be a timeout problem. I can reproduce with a LVM device and block throttling : # ls -ld /dev/dm-3 brw-rw 1 root disk 253, 3 sept. 26 12:17 /dev/dm-3 # echo '253:3 500' > /sys/fs/cgroup/blkio/blkio.throttle.read_iops_device # echo '253:3 500' > /sys/fs/cgroup/blkio/blkio.throttle.

Re: [Xen-devel] [Qemu-devel] [PATCH v1 1/8] Replace all occurances of __FUNCTION__ with __func__

2017-09-26 Thread Eric Blake
On 09/25/2017 07:08 PM, Alistair Francis wrote: > Replace all occurs of __FUNCTION__ except for the check in checkpatch > with the non GCC specific __func__. > > One line in hcd-musb.c was manually tweaked to pass checkpatch. > > Signed-off-by: Alistair Francis > Cc: Gerd Hoffmann > Cc: Andrzej

[Xen-devel] [xen-unstable-smoke test] 113833: tolerable all pass - PUSHED

2017-09-26 Thread osstest service owner
flight 113833 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/113833/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 13 migrate-support-checkfail never pass test-armhf-armhf-xl 1

Re: [Xen-devel] [PATCH v13 1/3] x86emul: New return code for unimplemented instruction

2017-09-26 Thread Jan Beulich
>>> On 25.09.17 at 14:03, wrote: > Enforce the distinction between an instruction not implemented by the > emulator and the failure to emulate that instruction by defining a new > return code, X86EMUL_UNIMPLEMENTED. > > This value should only be returned by the core emulator only if it fails to >

Re: [Xen-devel] [xen-unstable test] 113819: regressions - trouble: broken/fail/pass

2017-09-26 Thread Ian Jackson
Andrew Cooper writes ("Re: [Xen-devel] [xen-unstable test] 113819: regressions - trouble: broken/fail/pass"): > > test-xtf-amd64-amd64-4 broken in > > 113816 > > test-xtf-amd64-amd64-5 63 leak-check/check fail REGR. vs. > > 113387 > > This is: > > Sep

Re: [Xen-devel] [PATCH v4 02/15] Rename PSR sysctl/domctl interfaces and xsm policy to make them be general

2017-09-26 Thread Wei Liu
On Sat, Sep 23, 2017 at 05:48:11PM +0800, Yi Sun wrote: > This patch renames PSR sysctl/domctl interfaces and related xsm policy to > make them be general for all resource allocation features but not only > for CAT. Then, we can resuse the interfaces for all allocation features. > > Basically, it

Re: [Xen-devel] [PATCH 27/27 v10] xen/arm: vpl011: Correct the logic for asserting/de-asserting SBSA UART TX interrupt

2017-09-26 Thread Dave Martin
On Fri, Sep 22, 2017 at 01:53:26PM +0530, Bhupinder Thakur wrote: > This patch fixes the issue observed when pl011 patches were tested on > the junos hardware by Andre/Julien. It was observed that when large output is > generated such as on running 'find /', output was getting truncated > intermit

[Xen-devel] [ovmf test] 113827: all pass - PUSHED

2017-09-26 Thread osstest service owner
flight 113827 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/113827/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 1b8eca8b1affc81357c9f685ac90e5de75ba4b87 baseline version: ovmf 8932679df5be046feba30

[Xen-devel] [PATCH 3/3 v4] xenfb: Add [feature|request]-raw-pointer

2017-09-26 Thread Owen Smith
Writes "feature-raw-pointer" during init to indicate the backend can pass raw unscaled values for absolute axes to the frontend. Frontends set "request-raw-pointer" to indicate the backend should not attempt to scale absolute values to console size. "request-raw-pointer" is only valid if "request-a

[Xen-devel] [PATCH 2/3 v4] xenfb: Use Input Handlers directly

2017-09-26 Thread Owen Smith
Avoid the unneccessary calls through the input-legacy.c file by using the qemu_input_handler_*() calls directly. This did require reworking the event and sync handlers to use the reverse mapping from qcode to linux using qemu_input_qcode_to_linux(). Removes the scancode2linux mapping, and supportin

[Xen-devel] [PATCH 1/3 v4] ui/input: add qemu_input_qcode_to_linux

2017-09-26 Thread Owen Smith
Build a reverse mapping on first call that converts qcodes to linux KEY_* identifiers. Signed-off-by: Owen Smith --- include/ui/input.h | 1 + ui/input-keymap.c | 16 2 files changed, 17 insertions(+) diff --git a/include/ui/input.h b/include/ui/input.h index c488585..c3c1664

[Xen-devel] [PATCH 0/3 v4] xenfb: Enablement for Windows PV HID frontend

2017-09-26 Thread Owen Smith
Improve the input device model in xenfb, by updating the Qemu input handlers and adding a feature to allow for raw (unscaled) absolute coordinates to be represented. By using a reverse mapping call, the Linux input.h #defines are not pulled into xenfb, and so should remove the compiler warnings re

Re: [Xen-devel] [PATCH 1/3] xen: timers: don't miss a timer event because of stop_timer()

2017-09-26 Thread Jan Beulich
>>> On 15.09.17 at 20:01, wrote: > --- a/xen/common/timer.c > +++ b/xen/common/timer.c > @@ -217,7 +217,7 @@ static inline void activate_timer(struct timer *timer) > timer->status = TIMER_STATUS_invalid; > list_del(&timer->inactive); > > -if ( add_entry(timer) ) > +if ( add_ent

Re: [Xen-devel] [PATCH 2/3] xen: RCU: make the period of the idle timer configurable.

2017-09-26 Thread Jan Beulich
>>> On 15.09.17 at 20:01, wrote: > --- a/xen/common/rcupdate.c > +++ b/xen/common/rcupdate.c > @@ -110,10 +110,35 @@ struct rcu_data { > * About how far in the future the timer should be programmed each time, > * it's hard to tell (guess!!). Since this mimics Linux's periodic timer > * tick,

[Xen-devel] Xen 4.10 Development update: code freeze date

2017-09-26 Thread Julien Grall
Hi all, The hard code freeze date will be extended by a week and half. I still expect Xen 4.10 released on time. The new schedule is: * Code freeze: October 11th, 2017 * RC1: TBD * Release: December 2nd, 2017 Note that the last posting date has already past. All new series posted

Re: [Xen-devel] [PATCH 3/3] xen: RCU: make the period of the idle timer adaptive.

2017-09-26 Thread Jan Beulich
>>> On 15.09.17 at 20:01, wrote: > @@ -495,8 +507,18 @@ void rcu_idle_timer_stop() > > static void rcu_idle_timer_handler(void* data) > { > -/* Nothing, really... Just count the number of times we fire */ > perfc_incr(rcu_idle_timer); > + > +if ( !cpumask_empty(&rcu_ctrlblk.cpumas

Re: [Xen-devel] [PATCH 27/27 v10] xen/arm: vpl011: Correct the logic for asserting/de-asserting SBSA UART TX interrupt

2017-09-26 Thread Julien Grall
Hi Dave, On 09/26/2017 03:38 PM, Dave Martin wrote: On Fri, Sep 22, 2017 at 01:53:26PM +0530, Bhupinder Thakur wrote: This patch fixes the issue observed when pl011 patches were tested on the junos hardware by Andre/Julien. It was observed that when large output is generated such as on running

[Xen-devel] [linux-linus test] 113823: tolerable FAIL - PUSHED

2017-09-26 Thread osstest service owner
flight 113823 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/113823/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 17 guest-stop fail REGR. vs. 113809 test-amd64-amd64-xl-qemut-wi

Re: [Xen-devel] [PATCH v1 03/14] x86/vvmx: use np2m_flush_base() for INVEPT_SINGLE_CONTEXT

2017-09-26 Thread George Dunlap
On 09/04/2017 09:14 AM, Sergey Dyasli wrote: > nvmx_handle_invept() updates current's np2m just to flush it. Instead, > use the new np2m_flush_base() directly for this purpose. This one and the previous one look good, but it seems like it would be better to have them as a single patch. -George

Re: [Xen-devel] [PATCH v1 04/14] x86/np2m: remove np2m_base from p2m_get_nestedp2m()

2017-09-26 Thread George Dunlap
On 09/04/2017 09:14 AM, Sergey Dyasli wrote: > Remove np2m_base parameter as it should always match the value of > np2m_base in VMCX12. > > Signed-off-by: Sergey Dyasli Reviewed-by: George Dunlap ___ Xen-devel mailing list Xen-devel@lists.xen.org htt

Re: [Xen-devel] [PATCH 27/27 v10] xen/arm: vpl011: Correct the logic for asserting/de-asserting SBSA UART TX interrupt

2017-09-26 Thread Dave Martin
On Tue, Sep 26, 2017 at 04:50:44PM +0100, Julien Grall wrote: > Hi Dave, > > On 09/26/2017 03:38 PM, Dave Martin wrote: > >On Fri, Sep 22, 2017 at 01:53:26PM +0530, Bhupinder Thakur wrote: > >>This patch fixes the issue observed when pl011 patches were tested on > >>the junos hardware by Andre/Jul

[Xen-devel] [ovmf test] 113838: all pass - PUSHED

2017-09-26 Thread osstest service owner
flight 113838 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/113838/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 67a6cf6dcaaab8ddd4e7dc6a7fe929b752461102 baseline version: ovmf 1b8eca8b1affc81357c9f

Re: [Xen-devel] Guest start issue on ARM (maybe related to Credit2) [Was: Re: [xen-unstable test] 113807: regressions - FAIL]

2017-09-26 Thread Julien Grall
Hi Dario, On 09/26/2017 08:33 AM, Dario Faggioli wrote: > On Mon, 2017-09-25 at 17:23 +0100, Julien Grall wrote: >> On 09/25/2017 03:07 PM, Dario Faggioli wrote: >>> I don't see much in the logs, TBH, but both `xl vcpu-list' and the >>> 'r' >>> debug key seem to suggest that vCPU 0 is running, whi

[Xen-devel] [ovmf baseline-only test] 72164: all pass

2017-09-26 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 72164 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72164/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 1b8eca8b1affc81357c9f685ac90e5de75ba4b87 baseline v

Re: [Xen-devel] [PATCH 2/3] xen: RCU: make the period of the idle timer configurable.

2017-09-26 Thread Dario Faggioli
On Tue, 2017-09-26 at 09:14 -0600, Jan Beulich wrote: > > > > On 15.09.17 at 20:01, wrote: > > --- a/xen/common/rcupdate.c > > +++ b/xen/common/rcupdate.c > > +int ret = 0; > > + > > +if ( MILLISECS(period) > IDLE_TIMER_PERIOD_MAX ) > > +{ > > +printk("WARNING: rcu_idle_timer_p

Re: [Xen-devel] [PATCH 3/3] xen: RCU: make the period of the idle timer adaptive.

2017-09-26 Thread Dario Faggioli
On Tue, 2017-09-26 at 09:24 -0600, Jan Beulich wrote: > > > > On 15.09.17 at 20:01, wrote: > > @@ -495,8 +507,18 @@ void rcu_idle_timer_stop() > >   > >  static void rcu_idle_timer_handler(void* data) > >  { > > -/* Nothing, really... Just count the number of times we fire > > */ > >  perf

Re: [Xen-devel] [PATCH 1/3] xen: timers: don't miss a timer event because of stop_timer()

2017-09-26 Thread Dario Faggioli
On Tue, 2017-09-26 at 08:59 -0600, Jan Beulich wrote: > > > > On 15.09.17 at 20:01, wrote: > > --- a/xen/common/timer.c > > +++ b/xen/common/timer.c > > @@ -217,7 +217,7 @@ static inline void activate_timer(struct timer > > *timer) > >  timer->status = TIMER_STATUS_invalid; > >  list_del(&

[Xen-devel] [xen-unstable test] 113834: regressions - FAIL

2017-09-26 Thread osstest service owner
flight 113834 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/113834/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemut-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs. 113387 test-armhf-a

Re: [Xen-devel] Guest start issue on ARM (maybe related to Credit2) [Was: Re: [xen-unstable test] 113807: regressions - FAIL]

2017-09-26 Thread Dario Faggioli
On Tue, 2017-09-26 at 18:28 +0100, Julien Grall wrote: > On 09/26/2017 08:33 AM, Dario Faggioli wrote: > > > > > Here's the logs: > > http://logs.test-lab.xenproject.org/osstest/logs/113816/test-armhf- > > armhf-xl-rtds/info.html > > It does not seem to be similar, in the credit2 case the kernel

[Xen-devel] [PATCH] xen/arm: Fix the issue in cmp_mmio_handler used in find_mmio_handler

2017-09-26 Thread Bhupinder Thakur
This function returns true/false based on whether the key value is in the range (start, start+size). However, it should check against (start, start+size-1) because start+size falls outside the range. This resulted in returning a wrong mmio_handler for a given mmio address which happened to be star

  1   2   >