Re: [Xen-devel] [PATCH v3 10/22] libxl: add PVH support to domain creation

2017-09-27 Thread Roger Pau Monné
On Wed, Sep 27, 2017 at 02:28:11PM +, Ian Jackson wrote: > Roger Pau Monne writes ("[PATCH v3 10/22] libxl: add PVH support to domain > creation"): > > Remove the device model "none" support from domain creation and > > introduce support for PVH. > > This is a backwards-incompatible change,

[Xen-devel] [PATCH] xen-hvm: use errno in error message

2017-09-27 Thread Wei Liu
The error code is encoding in errno, not rc. Signed-off-by: Wei Liu --- Cc: Stefano Stabellini Cc: Anthony PERARD --- hw/i386/xen/xen-hvm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [Xen-devel] [PATCH] x86/pvh: fix PVHv2 Dom0 memory calculation

2017-09-27 Thread Boris Ostrovsky
On 09/27/2017 11:10 AM, Roger Pau Monné wrote: > On Wed, Sep 27, 2017 at 02:26:37PM +, Jan Beulich wrote: > On 27.09.17 at 16:16, wrote: >>> --- a/xen/arch/x86/dom0_build.c >>> +++ b/xen/arch/x86/dom0_build.c >>> @@ -263,8 +263,7 @@ unsigned long __init

Re: [Xen-devel] [PATCH] xen-hvm: use errno in error message

2017-09-27 Thread Wei Liu
On Wed, Sep 27, 2017 at 05:10:09PM +0100, Wei Liu wrote: > The error code is encoding in errno, not rc. > "is encoded", sorry for the typo... ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH] xen-hvm: use errno in error message

2017-09-27 Thread Anthony PERARD
On Wed, Sep 27, 2017 at 05:10:09PM +0100, Wei Liu wrote: > The error code is encoding in errno, not rc. ^ encoded I think. Otherwise, Reviewed-by: Anthony PERARD Thanks, > > Signed-off-by: Wei Liu > --- > Cc: Stefano

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

2017-09-27 Thread Konrad Rzeszutek Wilk
On Wed, Sep 27, 2017 at 02:46:23PM +0100, 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 that there's already a text notice at the > top of the header file, hence

Re: [Xen-devel] [PATCH v3 10/22] libxl: add PVH support to domain creation

2017-09-27 Thread Roger Pau Monné
On Wed, Sep 27, 2017 at 02:28:11PM +, Ian Jackson wrote: > Roger Pau Monne writes ("[PATCH v3 10/22] libxl: add PVH support to domain > creation"): > > Remove the device model "none" support from domain creation and > > introduce support for PVH. > > This is a backwards-incompatible change,

Re: [Xen-devel] [PATCH] x86/pvh: fix PVHv2 Dom0 memory calculation

2017-09-27 Thread Roger Pau Monné
On Wed, Sep 27, 2017 at 02:26:37PM +, Jan Beulich wrote: > >>> On 27.09.17 at 16:16, wrote: > > --- a/xen/arch/x86/dom0_build.c > > +++ b/xen/arch/x86/dom0_build.c > > @@ -263,8 +263,7 @@ unsigned long __init dom0_compute_nr_pages( > > avail -= max_pdx >> s;

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 =

Re: [Xen-devel] RT-Xen on ARM

2017-09-27 Thread Meng Xu
On Wed, Sep 27, 2017 at 10:37 AM, Andrii Anisov wrote: > Hello, > > > On 27.09.17 16:57, Meng Xu wrote: >> >> The command is: >> java -jar carts.jar inputfile outputfile > > From the next example, I would say the command is: > java -jar carts.jar inputfile

[Xen-devel] [PATCH v2 4/5] pci: Add INTERFACE_CONVENTIONAL_PCI_DEVICE to Conventional PCI devices

2017-09-27 Thread Eduardo Habkost
Add INTERFACE_CONVENTIONAL_PCI_DEVICE to all direct subtypes of TYPE_PCI_DEVICE, except: 1) The ones that already have INTERFACE_PCIE_DEVICE set: * base-xhci * e1000e * nvme * pvscsi * vfio-pci * virtio-pci * vmxnet3 2) base-pci-bridge Not all PCI bridges are Conventional PCI devices, so

Re: [Xen-devel] [PATCH] xen-pciback: relax BAR sizing write value check

2017-09-27 Thread Boris Ostrovsky
On 09/26/2017 02:43 AM, Juergen Gross wrote: > On 25/09/17 10:01, Jan Beulich wrote: >> Just like done in d2bd05d88d ("xen-pciback: return proper values during >> BAR sizing") for the ROM BAR, ordinary ones also shouldn't compare the >> written value directly against ~0, but consider the r/o bits

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

2017-09-27 Thread Boris Ostrovsky
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; >>> + struct pvclock_vsyscall_time_info *ti; >>> + struct pvclock_vcpu_time_info *pvti; >>> +

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

2017-09-27 Thread osstest service owner
flight 113855 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/113855/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-i386-xl-qemut-win7-amd64 16 guest-localmigrate/x10 fail in 113845 pass in 113855

[Xen-devel] [linux-next test] 113857: regressions - FAIL

2017-09-27 Thread osstest service owner
flight 113857 linux-next real [real] http://logs.test-lab.xenproject.org/osstest/logs/113857/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs. 113823 Tests which

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

2017-09-27 Thread osstest service owner
flight 113865 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/113865/ 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

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

2017-09-27 Thread osstest service owner
flight 113859 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/113859/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 36d083ef0018b78ea12db1a5f2e8df5336541fea baseline version: ovmf

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

2017-09-27 Thread Boris Ostrovsky
On 09/27/2017 05:43 AM, Juergen Gross wrote: > On 27/09/17 11:41, Zhenzhong Duan wrote: >> 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

[Xen-devel] [linux-3.18 test] 113856: regressions - FAIL

2017-09-27 Thread osstest service owner
flight 113856 linux-3.18 real [real] http://logs.test-lab.xenproject.org/osstest/logs/113856/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemut-debianhvm-amd64 16 guest-localmigrate/x10 fail REGR. vs. 113503

Re: [Xen-devel] [Qemu-devel] [PATCH v1 3/8] hw: Replace fprintf(stderr, "*\n" with error_report()

2017-09-27 Thread Alistair Francis
On Mon, Sep 25, 2017 at 8:51 PM, Thomas Huth wrote: > On 26.09.2017 02:08, Alistair Francis wrote: >> Replace a large number of the fprintf(stderr, "*\n" calls with >> error_report(). The functions were renamed with these commands and then >> compiler issues where manually

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; + struct pvclock_vsyscall_time_info

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

2017-09-27 Thread Boris Ostrovsky
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: > +static void xen_setup_vsyscall_time_info(void) > +{ > + struct

[Xen-devel] [linux-4.9 test] 113860: regressions - FAIL

2017-09-27 Thread osstest service owner
flight 113860 linux-4.9 real [real] http://logs.test-lab.xenproject.org/osstest/logs/113860/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-xsm 7 xen-boot fail REGR. vs. 113736

[Xen-devel] Building xtf on older distros

2017-09-27 Thread Glenn Enright
Hi there When building the xen testing framework on CentOS6, I see the below issue. I see similar issues with CentOS7 and other distros with older tool chains. The only one that built cleanly for me was Ubuntu 1604. Some workarounds (disabling the -W options) let it get a bit further but I

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

2017-09-27 Thread Yi Sun
On 17-09-26 15:19:19, Wei Liu wrote: > 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

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

2017-09-27 Thread Yi Sun
On 17-09-26 12:48:29, Roger Pau Monn� wrote: > 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

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

2017-09-27 Thread Yi Sun
On 17-09-26 10:39:31, Roger Pau Monn� wrote: > 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,

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

2017-09-27 Thread Yi Sun
On 17-09-26 12:39:21, Roger Pau Monn� wrote: > 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,

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: >> +static void xen_setup_vsyscall_time_info(void)

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

2017-09-27 Thread Julien Grall
Hi Dario, On 09/26/2017 09:51 PM, Dario Faggioli wrote: > 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

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

2017-09-27 Thread Julien Grall
On 09/28/2017 12:51 AM, Julien Grall wrote: Hi Dario, On 09/26/2017 09:51 PM, Dario Faggioli wrote: 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-

Re: [Xen-devel] [PATCH v10 01/11] xen: add function for obtaining highest possible memory address

2017-09-27 Thread Julien Grall
Hi Juergen, On 09/25/2017 11:00 AM, Juergen Gross wrote: Add a function for obtaining the highest possible physical memory address of the system. This value is influenced by: - hypervisor configuration (CONFIG_BIGMEM) - processor capability (max. addressable physical memory) - memory map at

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

2017-09-27 Thread Alistair Francis
On Tue, Sep 26, 2017 at 6:32 AM, Eric Blake wrote: > 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

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

2017-09-27 Thread Peter Maydell
On 26 September 2017 at 06:32, Eric Blake wrote: > On 09/25/2017 07:08 PM, Alistair Francis wrote: >> diff --git a/hw/arm/nseries.c b/hw/arm/nseries.c >> index 58005b6619..32687afced 100644 >> --- a/hw/arm/nseries.c >> +++ b/hw/arm/nseries.c >> @@ -463,7 +463,7 @@ static

Re: [Xen-devel] [PATCH v10 08/11] xen/arm: move arch specific grant table bits into grant_table.c

2017-09-27 Thread Julien Grall
Hi Juergen, On 09/25/2017 11:00 AM, Juergen Gross wrote: Instead of attaching the ARM specific grant table data to the domain structure add it to struct grant_table. Add the needed arch functions to the asm-*/grant_table.h includes. Signed-off-by: Juergen Gross Reviewed-by:

Re: [Xen-devel] [PATCH v10 09/11] xen: make grant resource limits per domain

2017-09-27 Thread Julien Grall
Hi Juergen, On 09/25/2017 11:00 AM, Juergen Gross wrote: diff --git a/xen/include/asm-arm/grant_table.h b/xen/include/asm-arm/grant_table.h index 30db2d1616..bcdeada9ca 100644 --- a/xen/include/asm-arm/grant_table.h +++ b/xen/include/asm-arm/grant_table.h @@ -2,9 +2,11 @@ #define

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

2017-09-27 Thread Julien Grall
Hi Bhupinder, Good catch! On 09/26/2017 11:45 PM, Bhupinder Thakur wrote: 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

Re: [Xen-devel] [PATCH] xen/arm: p2m: Read *_mapped_gfn with the p2m lock taken

2017-09-27 Thread Julien Grall
Hi, Ping? Cheers, On 09/14/2017 04:39 PM, Julien Grall wrote: *_mapped_gfn are currently read before acquiring the lock. However, they may be modified by the p2m code before the lock was acquired. This means we will use the wrong values. Fix it by moving the read inside the section protected

Re: [Xen-devel] [PATCH v2 4/5] pci: Add INTERFACE_CONVENTIONAL_PCI_DEVICE to Conventional PCI devices

2017-09-27 Thread David Gibson
On Wed, Sep 27, 2017 at 04:56:34PM -0300, Eduardo Habkost wrote: > Add INTERFACE_CONVENTIONAL_PCI_DEVICE to all direct subtypes of > TYPE_PCI_DEVICE, except: > > 1) The ones that already have INTERFACE_PCIE_DEVICE set: > > * base-xhci > * e1000e > * nvme > * pvscsi > * vfio-pci > * virtio-pci >

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

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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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 17/27 v11] xen/arm: vpl011: Add a new handle_console_tty function in xenconsole

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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-27 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-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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 09/27 v11] xen/arm: vpl011: Rename the console structure field conspath to xspath

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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

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

2017-09-27 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] [distros-debian-squeeze test] 72166: tolerable trouble: broken/fail/pass

2017-09-27 Thread Platform Team regression test user
flight 72166 distros-debian-squeeze real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72166/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: build-arm64-pvops 2 hosts-allocate broken like 72130 build-arm64

[Xen-devel] [PATCH v5] x86/hvm: Implement hvmemul_write() using real mappings

2017-09-27 Thread Alexandru Isaila
From: Andrew Cooper An access which crosses a page boundary is performed atomically by x86 hardware, albeit with a severe performance penalty. An important corner case is when a straddled access hits two pages which differ in whether a translation exists, or in net

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

2017-09-27 Thread Jan Beulich
>>> On 26.09.17 at 20:11, wrote: > On Tue, 2017-09-26 at 08:59 -0600, Jan Beulich wrote: >> > > > On 15.09.17 at 20:01, wrote: >> > @@ -326,7 +326,17 @@ void stop_timer(struct timer *timer) >> > return; >> > >> > if (

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

2017-09-27 Thread Jan Beulich
>>> On 26.09.17 at 19:48, wrote: > 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 (

Re: [Xen-devel] [PATCH v5] x86/hvm: Implement hvmemul_write() using real mappings

2017-09-27 Thread Andrew Cooper
On 27/09/2017 09:04, Alexandru Isaila wrote: > From: Andrew Cooper > > An access which crosses a page boundary is performed atomically by x86 > hardware, albeit with a severe performance penalty. An important corner case > is when a straddled access hits two pages

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

2017-09-27 Thread osstest service owner
flight 113845 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/113845/ 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. 113823 Tests which

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

2017-09-27 Thread Juergen Gross
On 27/09/17 05:03, Zhenzhong Duan wrote: > 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

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

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

Re: [Xen-devel] [PATCH v5] x86/hvm: Implement hvmemul_write() using real mappings

2017-09-27 Thread Jan Beulich
>>> On 27.09.17 at 10:38, wrote: > On 27/09/2017 09:04, Alexandru Isaila wrote: >> --- a/xen/arch/x86/hvm/emulate.c >> +++ b/xen/arch/x86/hvm/emulate.c >> @@ -498,6 +498,156 @@ static int hvmemul_do_mmio_addr(paddr_t mmio_gpa, >> } >> >> /* >> + * Map the frame(s)

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

2017-09-27 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

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

2017-09-27 Thread Juergen Gross
On 27/09/17 11:41, Zhenzhong Duan wrote: > 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

[Xen-devel] [OSSTEST PATCH] ts-leak-check: Treat listing failure as fail, not broken

2017-09-27 Thread Ian Jackson
This can easily occur if the test host crashes, due to a bug. Signed-off-by: Ian Jackson --- ts-leak-check | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ts-leak-check b/ts-leak-check index 7dbabfe..678d069 100755 --- a/ts-leak-check +++

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

2017-09-27 Thread Dario Faggioli
On Wed, 2017-09-27 at 02:20 -0600, Jan Beulich wrote: > > > > On 26.09.17 at 20:11, wrote: > > it means there is an event that *is* in progress right now (i.e., > > we're > > stopping the timer on the path that goes from the interrupt that > > raised > > TIMER_SOFTIRQ,

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

2017-09-27 Thread Jan Beulich
>>> On 27.09.17 at 12:18, wrote: > On Wed, 2017-09-27 at 02:20 -0600, Jan Beulich wrote: >> In the end what I think I'm missing is a clear description of an >> actual >> case where the current behavior causes breakage (plus of course >> an explanation why the new

[Xen-devel] [xen-unstable-coverity test] 113858: all pass - PUSHED

2017-09-27 Thread osstest service owner
flight 113858 xen-unstable-coverity real [real] http://logs.test-lab.xenproject.org/osstest/logs/113858/ Perfect :-) All tests in this flight passed as required version targeted for testing: xen a8ea6e2688118a3e19e29b39e316faa5f96ab9d1 baseline version: xen

[Xen-devel] [OSSTEST PATCH] ts-kernel-build: enable ntfs and fuse support

2017-09-27 Thread Wei Liu
They will be useful for extracting files from Windows and other OSes from Dom0 while debugging. Signed-off-by: Wei Liu --- ts-kernel-build | 5 + 1 file changed, 5 insertions(+) diff --git a/ts-kernel-build b/ts-kernel-build index 22a027a..5f152a3 100755 ---

Re: [Xen-devel] [PATCH v7 01/12] x86/mm: allow a privileged PV domain to map guest mfns

2017-09-27 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 25 September 2017 14:03 > To: Paul Durrant > Cc: Andrew Cooper ; xen- > de...@lists.xenproject.org > Subject: Re: [PATCH v7 01/12] x86/mm: allow a privileged PV

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

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

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

2017-09-27 Thread Andrew Cooper
On 26/09/17 13:49, Paul Durrant wrote: >> -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:

[Xen-devel] [PATCH 0/2] Coverity issues for credit2 CAP series

2017-09-27 Thread Wei Liu
Wei Liu (2): xen/credit2: add missing unlock python/libxc: extend the call to get/set cap for credit2 tools/python/xen/lowlevel/xc/xc.c | 17 ++--- xen/common/sched_credit2.c| 1 + 2 files changed, 11 insertions(+), 7 deletions(-) -- 2.11.0

[Xen-devel] [PATCH 2/2] python/libxc: extend the call to get/set cap for credit2

2017-09-27 Thread Wei Liu
Commit 68817024 ("xen: credit2: allow to set and get utilization cap") added a new parameter. Implement it for the python binding as well. Coverity-ID: 1418532 Signed-off-by: Wei Liu --- Cc: George Dunlap Cc: Dario Faggioli

[Xen-devel] [PATCH 1/2] xen/credit2: add missing unlock

2017-09-27 Thread Wei Liu
Coverity-ID: 1418531 Signed-off-by: Wei Liu --- Cc: George Dunlap Cc: Dario Faggioli --- xen/common/sched_credit2.c | 1 + 1 file changed, 1 insertion(+) diff --git a/xen/common/sched_credit2.c

Re: [Xen-devel] [PATCH v5] x86/hvm: Implement hvmemul_write() using real mappings

2017-09-27 Thread Alexandru Stefan ISAILA
On Mi, 2017-09-27 at 09:38 +0100, Andrew Cooper wrote: > On 27/09/2017 09:04, Alexandru Isaila wrote: > > > > From: Andrew Cooper > > > > > > -return X86EMUL_EXCEPTION; > > -case HVMTRANS_bad_gfn_to_mfn: > > -return hvmemul_linear_mmio_write(addr,

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

2017-09-27 Thread Joao Martins
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 interface so far is kvm. This commit adds a setter function for the pvti page and moves

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

2017-09-27 Thread Joao Martins
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 the necessary flags for vdso. Afterwards we

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

2017-09-27 Thread Joao Martins
Hey, This take 3 for vdso for Xen. PVCLOCK_TSC_STABLE_BIT can be set starting Xen 4.8 which is required for vdso time related calls. In order to have it on, you need to have the hypervisor clocksource be TSC e.g. with the following boot params "clocksource=tsc tsc=stable:socket". Series is

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

2017-09-27 Thread Joao Martins
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 that there's already a text notice at the top of the header file, hence this commit simply enforces it more explicitly and have both peers

Re: [Xen-devel] [PATCH 1/2] xen/credit2: add missing unlock

2017-09-27 Thread Dario Faggioli
On Wed, 2017-09-27 at 12:46 +0100, Wei Liu wrote: > Coverity-ID: 1418531 > > Signed-off-by: Wei Liu > --- > diff --git a/xen/common/sched_credit2.c b/xen/common/sched_credit2.c > index d33c881c3d..18f39cafe4 100644 > --- a/xen/common/sched_credit2.c > +++

Re: [Xen-devel] [PATCH v5] x86/hvm: Implement hvmemul_write() using real mappings

2017-09-27 Thread Andrew Cooper
On 27/09/17 12:48, Alexandru Stefan ISAILA wrote: > On Mi, 2017-09-27 at 09:38 +0100, Andrew Cooper wrote: >> On 27/09/2017 09:04, Alexandru Isaila wrote: >>> From: Andrew Cooper >>> >>> >>> -return X86EMUL_EXCEPTION; >>> -case HVMTRANS_bad_gfn_to_mfn: >>> -

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

2017-09-27 Thread Juergen Gross
On 27/09/17 14:00, 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

Re: [Xen-devel] [PATCH 2/2] python/libxc: extend the call to get/set cap for credit2

2017-09-27 Thread Dario Faggioli
On Wed, 2017-09-27 at 12:46 +0100, Wei Liu wrote: > Commit 68817024 ("xen: credit2: allow to set and get utilization > cap") > added a new parameter. Implement it for the python binding as well. > > Coverity-ID: 1418532 > Right. Sorry, I tend to forget about these more often that I would want

  1   2   >