[Xen-devel] [PATCH 26/27 v11] xen/arm: vpl011: Fix the slow early console SBSA UART output

2017-09-26 Thread Bhupinder Thakur
The early console output uses pl011_early_write() to write data. This function waits for BUSY bit to get cleared before writing the next byte. In the SBSA UART emulation logic, the BUSY bit was set as soon one byte was written in the FIFO and it remained set until the FIFO was emptied. This meant

[Xen-devel] [PATCH 21/27 v11] xen/arm: vpl011: Add support for multiple consoles in xenconsole

2017-09-26 Thread Bhupinder Thakur
This patch adds the support for multiple consoles and introduces the iterator functions to operate on multiple consoles. The functions called by the iterators check that they are operating on valid I/O parameters. This ensures that if a particular console is not initialized then the functions will

[Xen-devel] [PATCH 24/27 v11] xen/arm: vpl011: Add a pl011 uart DT node in the guest device tree

2017-09-26 Thread Bhupinder Thakur
The SBSA UART node format is as specified in Documentation/devicetree/bindings/serial/arm_sbsa_uart.txt and given below: ARM SBSA defined generic UART -- This UART uses a subset of the PL011 registers and consequently lives in the PL011 driver. It's baudrate and other c

[Xen-devel] [PATCH 11/27 v11] xen/arm: vpl011: Add a new console_init function in xenconsole

2017-09-26 Thread Bhupinder Thakur
This patch introduces a new console_init function. This function initializes the console structure. Signed-off-by: Bhupinder Thakur Acked-by: Wei Liu --- CC: Ian Jackson CC: Wei Liu CC: Stefano Stabellini CC: Julien Grall Changes since v5: - Split this change in a separate patch. tools/co

[Xen-devel] [PATCH 25/27 v11] xen/arm: vpl011: Update documentation for vuart console support

2017-09-26 Thread Bhupinder Thakur
1. Update documentation for a new vuart option added. 2. Update documentation about SPI irq reserved for vuart. Signed-off-by: Bhupinder Thakur Reviewed-by: Stefano Stabellini Acked-by: Wei Liu --- CC: Ian Jackson CC: Wei Liu CC: Andrew Cooper CC: George Dunlap CC: Jan Beulich CC: Konrad R

[Xen-devel] [PATCH 23/27 v11] xen/arm: vpl011: Add a new vuart console type to xenconsole client

2017-09-26 Thread Bhupinder Thakur
Add a new console type VUART to connect to guest's emualated vuart console. Signed-off-by: Bhupinder Thakur Reviewed-by: Stefano Stabellini Acked-by: Wei Liu --- CC: Ian Jackson CC: Wei Liu CC: Stefano Stabellini CC: Julien Grall Changes since v4: - Removed the vuart compile time flag so t

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

2017-09-26 Thread Bhupinder Thakur
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 intermittently due to OUT ring buffer getting full. This issue was due to the fac

[Xen-devel] [PATCH 12/27 v11] xen/arm: vpl011: Add a new buffer_available function in xenconsole

2017-09-26 Thread Bhupinder Thakur
This patch introduces a new buffer_available function to check if more data is allowed to be buffered. Signed-off-by: Bhupinder Thakur Reviewed-by: Stefano Stabellini Acked-by: Wei Liu --- CC: Ian Jackson CC: Wei Liu CC: Stefano Stabellini CC: Julien Grall Changes since v5: - Split this ch

[Xen-devel] [PATCH 05/27 v11] xen/arm: vpl011: Rearrange xen header includes in alphabetical order in domctl.c

2017-09-26 Thread Bhupinder Thakur
Rearrange xen header includes in alphabetical order in domctl.c. Signed-off-by: Bhupinder Thakur Reviewed-by: Stefano Stabellini Reviewed-by: Julien Grall --- CC: Stefano Stabellini CC: Julien Grall Changes since v5: - Corrected include of in alphabetical order. xen/arch/arm/domctl.c | 6

[Xen-devel] [PATCH 15/27 v11] xen/arm: vpl011: Add a new console_evtchn_unmask function in xenconsole

2017-09-26 Thread Bhupinder Thakur
This patch introduces a new console_evtchn_unmask function. This function unmasks the console event channel if it is masked for some timeout period. One optimization that has been done is to merge the two for loops. One for loop was used to iterate through all domains and unmask the domain event

[Xen-devel] [PATCH 22/27 v11] xen/arm: vpl011: Add support for vuart console in xenconsole

2017-09-26 Thread Bhupinder Thakur
This patch finally adds the support for vuart console. It adds two new fields in the console initialization: - optional - use_gnttab optional flag tells whether the console is optional. use_gnttab tells whether the ring buffer should be allocated using grant table. The VUART console is enabled

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

2017-09-26 Thread Bhupinder Thakur
DEFINE_XEN_FLEX_RING(xencons) defines common helper functions such as xencons_queued() to tell the current size of the ring buffer, xencons_mask() to mask off the index, which are useful helper functions. pl011 emulation code will use these helper functions. io/console.h includes io/ring.h which d

[Xen-devel] [PATCH 10/27 v11] xen/arm: vpl011: Modify xenconsole functions to take console structure as input

2017-09-26 Thread Bhupinder Thakur
Xenconsole functions take domain structure as input. These functions shall be modified to take console structure as input since these functions typically perform console specific operations. Also the console specific functions starting with prefix "domain_" shall be modified to "console_" to ind

[Xen-devel] [PATCH 08/27 v11] xen/arm: vpl011: Modify xenconsole to define and use a new console structure

2017-09-26 Thread Bhupinder Thakur
Xenconsole uses a domain structure which contains console specific fields. This patch defines a new console structure, which would be used by the xenconsole functions to perform console specific operations like reading/writing data from/to the console ring buffer or reading/writing data from/to co

[Xen-devel] [PATCH 09/27 v11] xen/arm: vpl011: Rename the console structure field conspath to xspath

2017-09-26 Thread Bhupinder Thakur
The console->conspath name is changed to console->xspath as it is clear from the name that it is referring to xenstore path. Signed-off-by: Bhupinder Thakur Reviewed-by: Stefano Stabellini Acked-by: Wei Liu --- CC: Ian Jackson CC: Wei Liu CC: Stefano Stabellini CC: Julien Grall Changes sin

[Xen-devel] [PATCH 13/27 v11] xen/arm: vpl011: Add a new maybe_add_console_evtchn_fd function in xenconsole

2017-09-26 Thread Bhupinder Thakur
This patch introduces a new maybe_add_console_evtchn_fd function. This function adds the console event channel FD to list of polled FDs. Signed-off-by: Bhupinder Thakur Acked-by: Wei Liu --- CC: Ian Jackson CC: Wei Liu CC: Stefano Stabellini CC: Julien Grall Changes since v6: - Renamed add_

[Xen-devel] [PATCH 16/27 v11] xen/arm: vpl011: Add a new handle_console_ring function in xenconsole

2017-09-26 Thread Bhupinder Thakur
This patch introduces a new handle_console_ring function. This function reads the data from the ring buffer on receiving an event. The initialization of event channel poll fd to -1 is moved inside the handle_console_ring function as they are related. There should be no change in the behavior as th

[Xen-devel] [PATCH 20/27 v11] xen/arm: vpl011: Add a new console_close_evtchn function in xenconsole

2017-09-26 Thread Bhupinder Thakur
This patch introduces a console_close_evtchn function. This function closes the console event channel. Signed-off-by: Bhupinder Thakur Reviewed-by: Stefano Stabellini Acked-by: Wei Liu --- CC: Ian Jackson CC: Wei Liu CC: Stefano Stabellini CC: Julien Grall Changes since v5: - Split this ch

[Xen-devel] [PATCH 03/27 v11] xen/arm: vpl011: Allocate a new GFN in the toolstack for vuart

2017-09-26 Thread Bhupinder Thakur
Allocate a new gfn to be used as a ring buffer between xenconsole and Xen for sending/receiving pl011 console data. Signed-off-by: Bhupinder Thakur Acked-by: Stefano Stabellini Acked-by: Wei Liu --- CC: Ian Jackson CC: Wei Liu CC: Stefano Stabellini CC: Julien Grall Changes since v4: - Rem

[Xen-devel] [PATCH 04/27 v11] xen/arm: vpl011: Add support for vuart in libxl

2017-09-26 Thread Bhupinder Thakur
An option is provided in libxl to enable/disable SBSA vuart while creating a guest domain. Libxl now supports a generic vuart console and SBSA uart is a specific type. In future support can be added for multiple vuart of different types. User can enable SBSA vuart by adding the following line in

[Xen-devel] [PATCH 17/27 v11] xen/arm: vpl011: Add a new handle_console_tty function in xenconsole

2017-09-26 Thread Bhupinder Thakur
This patch introduces a new handle_console_tty function. This function performs read/write from/to console tty. Signed-off-by: Bhupinder Thakur Reviewed-by: Stefano Stabellini Acked-by: Wei Liu --- CC: Ian Jackson CC: Wei Liu CC: Stefano Stabellini CC: Julien Grall Changes since v5: - Spli

[Xen-devel] [PATCH 06/27 v11] xen/arm: vpl011: Add a new domctl API to initialize vpl011

2017-09-26 Thread Bhupinder Thakur
Add a new domctl API to initialize vpl011. It takes the GFN and console backend domid as input and returns an event channel to be used for sending and receiving events from Xen. Xen will communicate with xenconsole using GFN as the ring buffer and the event channel to transmit and receive pl011 da

[Xen-devel] [PATCH 02/27 v11] xen/arm: vpl011: Add SBSA UART emulation in Xen

2017-09-26 Thread Bhupinder Thakur
Add emulation code to emulate read/write access to pl011 registers and pl011 interrupts: - Emulate DR read/write by reading and writing from/to the IN and OUT ring buffers and raising an event to the backend when there is data in the OUT ring buffer and injecting an interrupt

[Xen-devel] [PATCH 18/27 v11] xen/arm: vpl011: Add a new console_cleanup function in xenconsole

2017-09-26 Thread Bhupinder Thakur
This patch introduces a new console_cleanup function. This function frees up the console resources. Signed-off-by: Bhupinder Thakur Reviewed-by: Stefano Stabellini Acked-by: Wei Liu --- CC: Ian Jackson CC: Wei Liu CC: Stefano Stabellini CC: Julien Grall Changes since v6: - Removed a null p

[Xen-devel] [PATCH 00/27 v11] SBSA UART emulation support in Xen

2017-09-26 Thread Bhupinder Thakur
SBSA UART emulation for guests in Xen == Linaro has published VM System specification for ARM Processors, which provides a set of guidelines for both guest OS and hypervisor implementations, such that building OS images according to these guidelines guarantees t

[Xen-devel] [PATCH 14/27 v11] xen/arm: vpl011: Add a new maybe_add_console_tty_fd function in xenconsole

2017-09-26 Thread Bhupinder Thakur
This patch introduces a new maybe_add_console_tty_fd function. This function adds the tty fd to the list of polled fds. Signed-off-by: Bhupinder Thakur Reviewed-by: Stefano Stabellini Acked-by: Wei Liu --- CC: Ian Jackson CC: Wei Liu CC: Stefano Stabellini CC: Julien Grall Changes since v6

[Xen-devel] [PATCH 07/27 v11] xen/arm: vpl011: Add a new vuart node in the xenstore

2017-09-26 Thread Bhupinder Thakur
Add a new vuart console node to xenstore. This node is added at /local/domain/$DOMID/vuart/0. The node contains information such as the ring-ref, event channel, buffer limit and type of console. Xenconsole reads the node information to setup the ring buffer and event channel for sending/receivin

[Xen-devel] [PATCH 19/27 v11] xen/arm: vpl011: Add a new console_open_log function in xenconsole

2017-09-26 Thread Bhupinder Thakur
This patch introduces a console_open_log console_cleanup function. This function opens the console log file. Signed-off-by: Bhupinder Thakur Reviewed-by: Stefano Stabellini Acked-by: Wei Liu --- CC: Ian Jackson CC: Wei Liu CC: Stefano Stabellini CC: Julien Grall Changes since v5: - Split t

[Xen-devel] [qemu-mainline test] 113844: trouble: broken/fail/pass

2017-09-26 Thread osstest service owner
flight 113844 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/113844/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-multivcpu broken test-armhf-armhf-x

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

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

[Xen-devel] [PATCH] Call xen_cleanhighmap() with 4MB aligned for page tables mapping

2017-09-26 Thread Zhenzhong Duan
When bootup a PVM guest with large memory(Ex.240GB), XEN provided initial mapping overlaps with kernel module virtual space. When mapping in this space is cleared by xen_cleanhighmap(), in certain case there could be an 2MB mapping left. This is due to XEN initialize 4MB aligned mapping but xen_cle

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

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

[Xen-devel] [qemu-mainline test] 113839: regressions - FAIL

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

[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

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] [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] [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(&

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 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

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

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 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

[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 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

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

[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 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,

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

[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

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

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

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

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

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

[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 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

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

[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 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

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

[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 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

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

[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 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] [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 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 ) >> { >> -

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 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.

[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 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); >> > >

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

[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 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

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 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 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

[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 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

  1   2   >