[Xen-devel] [xen-4.9-testing test] 114070: regressions - FAIL

2017-10-06 Thread osstest service owner
flight 114070 xen-4.9-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/114070/ 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. 113367 test-amd6

[Xen-devel] [xen-unstable-smoke test] 114087: regressions - FAIL

2017-10-06 Thread osstest service owner
flight 114087 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/114087/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt 12 guest-start fail REGR. vs. 113972 Tests which

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

2017-10-06 Thread osstest service owner
flight 114069 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/114069/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt 14 saverestore-support-checkfail like 113982 test-amd64-i386-xl-qemut-win7-amd64 1

Re: [Xen-devel] [RFC PATCH v3 00/24] ARM: Add Xen NUMA support

2017-10-06 Thread vkilari
Hi Julien, Sorry for not being active on this topic for some time. I was on vacation and have left cavium. Note: My email has been changed After going through the comments and fixing some of them, I see that clean up and code movement patches are ~20+. So I would like to split this series in

Re: [Xen-devel] [PATCH] MAINTAINERS: update entries to new email address.

2017-10-06 Thread Juergen Gross
On 05/10/17 16:28, Dario Faggioli wrote: > Replace, in the 'M:' fields of the components I co-maintain > ('CPU POOLS', 'SCHEDULING' and 'RTDS SCHEDULER'), the Citrix > email, to which I don't have access any longer, with my > personal email. > > Signed-off-by: Dario Faggioli Acked-by: Juergen Gr

[Xen-devel] [xen-unstable-smoke test] 114085: regressions - FAIL

2017-10-06 Thread osstest service owner
flight 114085 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/114085/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt 12 guest-start fail REGR. vs. 113972 Tests which

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

2017-10-06 Thread Stefano Stabellini
On Thu, 14 Sep 2017, 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 by the p2m lock.

[Xen-devel] [PATCH v5 13/13] xen: introduce a Kconfig option to enable the pvcalls frontend

2017-10-06 Thread Stefano Stabellini
Also add pvcalls-front to the Makefile. Signed-off-by: Stefano Stabellini CC: boris.ostrov...@oracle.com CC: jgr...@suse.com --- drivers/xen/Kconfig | 9 + drivers/xen/Makefile | 1 + 2 files changed, 10 insertions(+) diff --git a/drivers/xen/Kconfig b/drivers/xen/Kconfig index 4545561

[Xen-devel] [PATCH v5 05/13] xen/pvcalls: implement connect command

2017-10-06 Thread Stefano Stabellini
Send PVCALLS_CONNECT to the backend. Allocate a new ring and evtchn for the active socket. Introduce fields in struct sock_mapping to keep track of active sockets. Introduce a waitqueue to allow the frontend to wait on data coming from the backend on the active socket (recvmsg command). Two mutex

[Xen-devel] [PATCH v5 02/13] xen/pvcalls: implement frontend disconnect

2017-10-06 Thread Stefano Stabellini
Introduce a data structure named pvcalls_bedata. It contains pointers to the command ring, the event channel, a list of active sockets and a list of passive sockets. Lists accesses are protected by a spin_lock. Introduce a waitqueue to allow waiting for a response on commands sent to the backend.

[Xen-devel] [PATCH v5 12/13] xen/pvcalls: implement release command

2017-10-06 Thread Stefano Stabellini
Send PVCALLS_RELEASE to the backend and wait for a reply. Take both in_mutex and out_mutex to avoid concurrent accesses. Then, free the socket. For passive sockets, check whether we have already pre-allocated an active socket for the purpose of being accepted. If so, free that as well. Signed-off

[Xen-devel] [PATCH v5 11/13] xen/pvcalls: implement poll command

2017-10-06 Thread Stefano Stabellini
For active sockets, check the indexes and use the inflight_conn_req waitqueue to wait. For passive sockets if an accept is outstanding (PVCALLS_FLAG_ACCEPT_INFLIGHT), check if it has been answered by looking at bedata->rsp[req_id]. If so, return POLLIN. Otherwise use the inflight_accept_req waitq

[Xen-devel] [PATCH v5 08/13] xen/pvcalls: implement accept command

2017-10-06 Thread Stefano Stabellini
Introduce a waitqueue to allow only one outstanding accept command at any given time and to implement polling on the passive socket. Introduce a flags field to keep track of in-flight accept and poll commands. Send PVCALLS_ACCEPT to the backend. Allocate a new active socket. Make sure that only on

[Xen-devel] [PATCH v5 01/13] xen/pvcalls: introduce the pvcalls xenbus frontend

2017-10-06 Thread Stefano Stabellini
Introduce a xenbus frontend for the pvcalls protocol, as defined by https://xenbits.xen.org/docs/unstable/misc/pvcalls.html. This patch only adds the stubs, the code will be added by the following patches. Signed-off-by: Stefano Stabellini Reviewed-by: Boris Ostrovsky CC: boris.ostrov...@oracle

[Xen-devel] [PATCH v5 03/13] xen/pvcalls: connect to the backend

2017-10-06 Thread Stefano Stabellini
Implement the probe function for the pvcalls frontend. Read the supported versions, max-page-order and function-calls nodes from xenstore. Only one frontend<->backend connection is supported at any given time for a guest. Store the active frontend device to a static pointer. Introduce a stub func

[Xen-devel] [PATCH v5 04/13] xen/pvcalls: implement socket command and handle events

2017-10-06 Thread Stefano Stabellini
Send a PVCALLS_SOCKET command to the backend, use the masked req_prod_pvt as req_id. This way, req_id is guaranteed to be between 0 and PVCALLS_NR_REQ_PER_RING. We already have a slot in the rsp array ready for the response, and there cannot be two outstanding responses with the same req_id. Wait

[Xen-devel] [PATCH v5 06/13] xen/pvcalls: implement bind command

2017-10-06 Thread Stefano Stabellini
Send PVCALLS_BIND to the backend. Introduce a new structure, part of struct sock_mapping, to store information specific to passive sockets. Introduce a status field to keep track of the status of the passive socket. Signed-off-by: Stefano Stabellini CC: boris.ostrov...@oracle.com CC: jgr...@suse

[Xen-devel] [PATCH v5 09/13] xen/pvcalls: implement sendmsg

2017-10-06 Thread Stefano Stabellini
Send data to an active socket by copying data to the "out" ring. Take the active socket out_mutex so that only one function can access the ring at any given time. If not enough room is available on the ring, rather than returning immediately or sleep-waiting, spin for up to 5000 cycles. This small

[Xen-devel] [PATCH v5 07/13] xen/pvcalls: implement listen command

2017-10-06 Thread Stefano Stabellini
Send PVCALLS_LISTEN to the backend. Signed-off-by: Stefano Stabellini Reviewed-by: Boris Ostrovsky CC: boris.ostrov...@oracle.com CC: jgr...@suse.com --- drivers/xen/pvcalls-front.c | 57 + drivers/xen/pvcalls-front.h | 1 + 2 files changed, 58 inser

[Xen-devel] [PATCH v5 10/13] xen/pvcalls: implement recvmsg

2017-10-06 Thread Stefano Stabellini
Implement recvmsg by copying data from the "in" ring. If not enough data is available and the recvmsg call is blocking, then wait on the inflight_conn_req waitqueue. Take the active socket in_mutex so that only one function can access the ring at any given time. Signed-off-by: Stefano Stabellini

[Xen-devel] [PATCH v5 00/13] introduce the Xen PV Calls frontend:wq

2017-10-06 Thread Stefano Stabellini
Hi all, this series introduces the frontend for the newly introduced PV Calls procotol. PV Calls is a paravirtualized protocol that allows the implementation of a set of POSIX functions in a different domain. The PV Calls frontend sends POSIX function calls to the backend, which implements them a

[Xen-devel] [PATCH] ARM: sunxi: support more Allwinner SoCs

2017-10-06 Thread Andre Przywara
So far we only supported the Allwinner A20 SoC. Add support for most of the other virtualization capable Allwinner SoCs by: - supporting the watchdog in newer (sun8i) SoCs - getting the watchdog address from DT - adding compatible strings for other 32-bit SoCs - adding compatible strings for 64-bit

Re: [Xen-devel] [PATCH v4 02/13] xen/pvcalls: implement frontend disconnect

2017-10-06 Thread Stefano Stabellini
On Wed, 20 Sep 2017, Boris Ostrovsky wrote: > > static int pvcalls_front_remove(struct xenbus_device *dev) > > { > > + struct pvcalls_bedata *bedata; > > + struct sock_mapping *map = NULL, *n; > > + > > + bedata = dev_get_drvdata(&pvcalls_front_dev->dev); > > + dev_set_drvdata(&dev->dev,

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

2017-10-06 Thread osstest service owner
flight 114064 linux-next real [real] http://logs.test-lab.xenproject.org/osstest/logs/114064/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-pvops 6 kernel-build fail REGR. vs. 113902 Tests which did not

Re: [Xen-devel] [PATCH v4 12/13] xen/pvcalls: implement release command

2017-10-06 Thread Stefano Stabellini
On Fri, 22 Sep 2017, Boris Ostrovsky wrote: > > +static void pvcalls_front_free_map(struct pvcalls_bedata *bedata, > > + struct sock_mapping *map) > > I just noticed: pvcalls_front_free_map() is referenced by patches 2 and 8. I'll create an empty stub earlier. > > +

Re: [Xen-devel] [PATCH v4 11/13] xen/pvcalls: implement poll command

2017-10-06 Thread Stefano Stabellini
On Fri, 22 Sep 2017, Boris Ostrovsky wrote: > > +static unsigned int pvcalls_front_poll_passive(struct file *file, > > + struct pvcalls_bedata *bedata, > > + struct sock_mapping *map, > > +

Re: [Xen-devel] [PATCH v4 11/13] xen/pvcalls: implement poll command

2017-10-06 Thread Stefano Stabellini
On Tue, 19 Sep 2017, Andrea Parri wrote: > Hi Stefano, > > On Fri, Sep 15, 2017 at 04:00:38PM -0700, Stefano Stabellini wrote: > > For active sockets, check the indexes and use the inflight_conn_req > > waitqueue to wait. > > > > For passive sockets if an accept is outstanding > > (PVCALLS_FLAG_A

[Xen-devel] [qemu-upstream-4.8-testing baseline-only test] 72209: regressions - trouble: blocked/broken/fail/pass

2017-10-06 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 72209 qemu-upstream-4.8-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72209/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-pair6 host-pin

[Xen-devel] [xen-unstable-smoke test] 114084: regressions - FAIL

2017-10-06 Thread osstest service owner
flight 114084 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/114084/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt 12 guest-start fail REGR. vs. 113972 Tests which

Re: [Xen-devel] [PATCH] Changing my email address

2017-10-06 Thread Dario Faggioli
On Thu, 2017-10-05 at 13:19 -0400, Meng Xu wrote: > On Thu, Oct 5, 2017 at 10:28 AM, Dario Faggioli > wrote: > > > > Soon I won't have access to dario.faggi...@citrix.com email > > address. > > It's sad to hear this. :( > Things change. It's life. :-D > > Therefore, replace it, in my entries i

Re: [Xen-devel] [PATCH v4 08/13] xen/pvcalls: implement accept command

2017-10-06 Thread Stefano Stabellini
On Thu, 21 Sep 2017, Boris Ostrovsky wrote: > > +int pvcalls_front_accept(struct socket *sock, struct socket *newsock, int > > flags) > > +{ > > + struct pvcalls_bedata *bedata; > > + struct sock_mapping *map; > > + struct sock_mapping *map2 = NULL; > > + struct xen_pvcalls_request *req; >

Re: [Xen-devel] [PATCH v4 09/13] xen/pvcalls: implement sendmsg

2017-10-06 Thread Stefano Stabellini
On Fri, 22 Sep 2017, Boris Ostrovsky wrote: > > +static bool pvcalls_front_write_todo(struct sock_mapping *map) > > +{ > > + struct pvcalls_data_intf *intf = map->active.ring; > > + RING_IDX cons, prod, size = XEN_FLEX_RING_SIZE(PVCALLS_RING_ORDER); > > + int32_t error; > > + > > + cons = i

[Xen-devel] [xen-unstable-smoke test] 114082: regressions - FAIL

2017-10-06 Thread osstest service owner
flight 114082 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/114082/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt 12 guest-start fail REGR. vs. 113972 Tests which

Re: [Xen-devel] [PATCH v4 10/13] xen/pvcalls: implement recvmsg

2017-10-06 Thread Stefano Stabellini
On Fri, 22 Sep 2017, Boris Ostrovsky wrote: > > +static bool pvcalls_front_read_todo(struct sock_mapping *map) > > +{ > > + struct pvcalls_data_intf *intf = map->active.ring; > > + RING_IDX cons, prod; > > + int32_t error; > > + > > + cons = intf->in_cons; > > + prod = intf->in_prod; > >

Re: [Xen-devel] [PATCH v4 04/13] xen/pvcalls: implement socket command and handle events

2017-10-06 Thread Boris Ostrovsky
On 10/06/2017 02:38 PM, Stefano Stabellini wrote: > On Thu, 21 Sep 2017, Boris Ostrovsky wrote: >>> + >>> +static inline int get_request(struct pvcalls_bedata *bedata, int *req_id) >>> +{ >>> + *req_id = bedata->ring.req_prod_pvt & (RING_SIZE(&bedata->ring) - 1); >>> + if (RING_FULL(&bedata->ri

Re: [Xen-devel] [PATCH v4 06/13] xen/pvcalls: implement bind command

2017-10-06 Thread Stefano Stabellini
On Thu, 21 Sep 2017, Boris Ostrovsky wrote: > On 09/15/2017 07:00 PM, Stefano Stabellini wrote: > > Send PVCALLS_BIND to the backend. Introduce a new structure, part of > > struct sock_mapping, to store information specific to passive sockets. > > > > Introduce a status field to keep track of the

Re: [Xen-devel] [PATCH v4 02/13] xen/pvcalls: implement frontend disconnect

2017-10-06 Thread Boris Ostrovsky
On 10/06/2017 04:29 PM, Stefano Stabellini wrote: > On Fri, 6 Oct 2017, Boris Ostrovsky wrote: >> On 10/06/2017 01:51 PM, Stefano Stabellini wrote: >>> On Wed, 20 Sep 2017, Boris Ostrovsky wrote: > + > +struct pvcalls_bedata { > + struct xen_pvcalls_front_ring ring; > + grant_ref_t

Re: [Xen-devel] [PATCH v4 02/13] xen/pvcalls: implement frontend disconnect

2017-10-06 Thread Stefano Stabellini
On Fri, 6 Oct 2017, Boris Ostrovsky wrote: > On 10/06/2017 01:51 PM, Stefano Stabellini wrote: > > On Wed, 20 Sep 2017, Boris Ostrovsky wrote: > >>> + > >>> +struct pvcalls_bedata { > >>> + struct xen_pvcalls_front_ring ring; > >>> + grant_ref_t ref; > >>> + int irq; > >>> + > >>> + struct list_hea

Re: [Xen-devel] [PATCH v4 02/13] xen/pvcalls: implement frontend disconnect

2017-10-06 Thread Boris Ostrovsky
On 10/06/2017 01:51 PM, Stefano Stabellini wrote: > On Wed, 20 Sep 2017, Boris Ostrovsky wrote: >>> + >>> +struct pvcalls_bedata { >>> + struct xen_pvcalls_front_ring ring; >>> + grant_ref_t ref; >>> + int irq; >>> + >>> + struct list_head socket_mappings; >>> + struct list_head socketpas

Re: [Xen-devel] [Qemu-devel] [PATCH v3 0/8] xen: xen-domid-restrict improvements

2017-10-06 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 150731-30835-1-git-send-email-ian.jack...@eu.citrix.com Subject: [Qemu-devel] [PATCH v3 0/8] xen: xen-domid-restrict improvements === TEST SCRIPT BEGIN === #!/bin/bash B

[Xen-devel] [PATCH] tools/libxc: Fix domid parameter types

2017-10-06 Thread Andrew Cooper
Mixed throughout libxc are uint32_t, int, and domid_t for domid parameters. With a signed type, and an explicitly 16-bit type, it is exceedingly difficult to construct an INVALID_DOMID constant which works with all of them. (The main problem being that domid_t gets unconditionally zero extended wh

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

2017-10-06 Thread osstest service owner
flight 114060 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/114060/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt broken test-armhf-armhf-li

Re: [Xen-devel] [PATCH v4 04/13] xen/pvcalls: implement socket command and handle events

2017-10-06 Thread Stefano Stabellini
On Thu, 21 Sep 2017, Boris Ostrovsky wrote: > > + > > +static inline int get_request(struct pvcalls_bedata *bedata, int *req_id) > > +{ > > + *req_id = bedata->ring.req_prod_pvt & (RING_SIZE(&bedata->ring) - 1); > > + if (RING_FULL(&bedata->ring) || > > + READ_ONCE(bedata->rsp[*req_id].re

[Xen-devel] [PATCH 1/8] xen: link against xentoolcore

2017-10-06 Thread Ian Jackson
From: Anthony PERARD Xen libraries 4.10 will include a new xentoolcore library, without which xendevicemodel et al will not work. Signed-off-by: Ian Jackson --- configure | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/configure b/configure index fd7e3a5..6f691df 10

[Xen-devel] [PATCH 3/8] xen: defer call to xen_restrict until just before os_setup_post

2017-10-06 Thread Ian Jackson
We need to restrict *all* the control fds that qemu opens. Looking in /proc/PID/fd shows there are many; their allocation seems scattered throughout Xen support code in qemu. We must postpone the restrict call until roughly the same time as qemu changes its uid, chroots (if applicable), and so on

[Xen-devel] [PATCH 5/8] xen: move xc_interface compatibility fallback further up the file

2017-10-06 Thread Ian Jackson
We are going to want to use the dummy xendevicemodel_handle type in new stub functions in the CONFIG_XEN_CTRL_INTERFACE_VERSION < 41000 section. So we need to provide that definition, or (as applicable) include the appropriate header, earlier in the file. (Ideally the newer compatibility layers w

[Xen-devel] [PATCH 8/8] RFC configure: do_compiler: Dump some extra info under bash

2017-10-06 Thread Ian Jackson
This makes it much easier to find a particular thing in config.log. The information may be lacking in other shells, resulting in harmless empty output. (This is why we don't use the proper ${FUNCNAME[*]} array syntax - other shells will choke on that.) The extra output is only printed if configu

[Xen-devel] [PATCH v3 0/8] xen: xen-domid-restrict improvements

2017-10-06 Thread Ian Jackson
I have been working on trying to get qemu, when running as a Xen device model, to _actually_ not have power equivalent to root. I think I have achieved this, with some limitations (which are discussed in my series against xen.git. However, there are changes to qemu needed. In particular * The

[Xen-devel] [PATCH 4/8] xen: destroy_hvm_domain: Move reason into a variable

2017-10-06 Thread Ian Jackson
We are going to want to reuse this. No functional change. Signed-off-by: Ian Jackson Reviewed-by: Anthony PERARD --- hw/i386/xen/xen-hvm.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/hw/i386/xen/xen-hvm.c b/hw/i386/xen/xen-hvm.c index 7b60ec6..83420cd 100644 --- a/

[Xen-devel] [PATCH 6/8] xen: destroy_hvm_domain: Try xendevicemodel_shutdown

2017-10-06 Thread Ian Jackson
xc_interface_open etc. is not going to work if we have dropped privilege, but xendevicemodel_shutdown will if everything is new enough. xendevicemodel_shutdown is only availabe in Xen 4.10 and later, so provide a stub for earlier versions. Signed-off-by: Ian Jackson --- v2: Add compatibility stu

[Xen-devel] [PATCH 7/8] os-posix: Provide new -runasid option

2017-10-06 Thread Ian Jackson
This allows the caller to specify a uid and gid to use, even if there is no corresponding password entry. This will be useful in certain Xen configurations. Signed-off-by: Ian Jackson --- v3: Error messages fixed. Thanks to Peter Maydell and Ross Lagerwall. v2: Coding style fixes. --- os-posix

[Xen-devel] [PATCH 2/8] xen: restrict: use xentoolcore_restrict_all

2017-10-06 Thread Ian Jackson
And insist that it works. Drop individual use of xendevicemodel_restrict and xenforeignmemory_restrict. These are not actually effective in this version of qemu, because qemu has a large number of fds open onto various Xen control devices. The restriction arrangements are still not right, becaus

Re: [Xen-devel] [PATCH for-4.10 5/5] tools/dombuilder: Prevent failures of xc_dom_gnttab_init()

2017-10-06 Thread Andrew Cooper
On 06/10/17 18:39, Wei Liu wrote: > On Thu, Oct 05, 2017 at 07:23:44PM +0100, Andrew Cooper wrote: >> Recent changes in grant table configuration have caused calls to >> xc_dom_gnttab_init() to fail if not proceeded with a call to >> xc_domain_set_gnttab_limits(). This is backwards from the point

[Xen-devel] [xen-unstable-smoke test] 114079: regressions - FAIL

2017-10-06 Thread osstest service owner
flight 114079 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/114079/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt 12 guest-start fail REGR. vs. 113972 Tests which

Re: [Xen-devel] [PATCH for-4.10 5/5] tools/dombuilder: Prevent failures of xc_dom_gnttab_init()

2017-10-06 Thread Andrew Cooper
On 06/10/17 11:30, Roger Pau Monné wrote: > On Thu, Oct 05, 2017 at 06:23:44PM +, Andrew Cooper wrote: >> Recent changes in grant table configuration have caused calls to >> xc_dom_gnttab_init() to fail if not proceeded with a call to >> xc_domain_set_gnttab_limits(). This is backwards from th

Re: [Xen-devel] [PATCH v4 05/13] xen/pvcalls: implement connect command

2017-10-06 Thread Stefano Stabellini
On Fri, 6 Oct 2017, Stefano Stabellini wrote: > On Thu, 21 Sep 2017, Boris Ostrovsky wrote: > > On 09/15/2017 07:00 PM, Stefano Stabellini wrote: > > > Send PVCALLS_CONNECT to the backend. Allocate a new ring and evtchn for > > > the active socket. > > > > > > Introduce fields in struct sock_mappi

Re: [Xen-devel] [PATCH v4 02/13] xen/pvcalls: implement frontend disconnect

2017-10-06 Thread Stefano Stabellini
On Wed, 20 Sep 2017, Boris Ostrovsky wrote: > > + > > +struct pvcalls_bedata { > > + struct xen_pvcalls_front_ring ring; > > + grant_ref_t ref; > > + int irq; > > + > > + struct list_head socket_mappings; > > + struct list_head socketpass_mappings; > > + spinlock_t socket_lock; > > + >

[Xen-devel] [qemu-upstream-4.8-testing test] 114058: tolerable FAIL - PUSHED

2017-10-06 Thread osstest service owner
flight 114058 qemu-upstream-4.8-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/114058/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-armhf-armhf-xl-credit2 16 guest-start/debian.repeat fail in 114038 pass in 114058 test-armhf

Re: [Xen-devel] [PATCH v4 05/13] xen/pvcalls: implement connect command

2017-10-06 Thread Stefano Stabellini
On Thu, 21 Sep 2017, Boris Ostrovsky wrote: > On 09/15/2017 07:00 PM, Stefano Stabellini wrote: > > Send PVCALLS_CONNECT to the backend. Allocate a new ring and evtchn for > > the active socket. > > > > Introduce fields in struct sock_mapping to keep track of active sockets. > > Introduce a waitqu

Re: [Xen-devel] [PATCH 04/24] xentoolcore, _restrict_all: Introduce new library and implementation

2017-10-06 Thread Ross Lagerwall
On 10/04/2017 04:57 PM, Ian Jackson wrote: diff --git a/tools/libs/toolcore/Makefile b/tools/libs/toolcore/Makefile new file mode 100644 index 000..f86f839 --- /dev/null +++ b/tools/libs/toolcore/Makefile @@ -0,0 +1,101 @@ +XEN_ROOT = $(CURDIR)/../../.. +include $(XEN_ROOT)/tools/Rules.mk + +

[Xen-devel] [PATCH v2] xen, tools: console.h shouldn't require string.h by default

2017-10-06 Thread Wei Liu
Unilaterally making string.h a prerequisite for console.h is going to break build for a lot of consumers of console.h. Define a macro for the new flex ring. Consumers which want to use it should define the macro. Partially revert af8d9356417cb617b635c5ace782388ebfe86e3a. Signed-off-by: Wei Liu

Re: [Xen-devel] [PATCH for-4.10 5/5] tools/dombuilder: Prevent failures of xc_dom_gnttab_init()

2017-10-06 Thread Wei Liu
On Thu, Oct 05, 2017 at 07:23:44PM +0100, Andrew Cooper wrote: > Recent changes in grant table configuration have caused calls to > xc_dom_gnttab_init() to fail if not proceeded with a call to > xc_domain_set_gnttab_limits(). This is backwards from the point of view of > 3rd party dombuilder users

Re: [Xen-devel] [PATCH for-4.10 3/5] tools/dombuilder: Switch to using gfn terminology for console and xenstore rings

2017-10-06 Thread Roger Pau Monné
On Fri, Oct 06, 2017 at 02:48:48PM +, Julien Grall wrote: > Hi Roger, > > On 06/10/17 11:36, Roger Pau Monné wrote: > > On Fri, Oct 06, 2017 at 10:03:39AM +, Andrew Cooper wrote: > > > On 06/10/17 10:57, Roger Pau Monné wrote: > > > > On Thu, Oct 05, 2017 at 06:23:41PM +, Andrew Cooper

Re: [Xen-devel] [PATCH for-4.10 4/5] tools/dombuilder: Fix asymetry when setting up console and xenstore rings

2017-10-06 Thread Wei Liu
On Thu, Oct 05, 2017 at 07:23:42PM +0100, Andrew Cooper wrote: > libxl always uses xc_dom_gnttab_init(), which internally calls > xc_dom_gnttab{_hvm,}_seed() to set up the grants point at the console and > xenstore rings. For HVM guests, libxl then asks Xen for the information set > up previously,

Re: [Xen-devel] [PATCH for-4.10 3/5] tools/dombuilder: Switch to using gfn terminology for console and xenstore rings

2017-10-06 Thread Wei Liu
On Thu, Oct 05, 2017 at 07:23:41PM +0100, Andrew Cooper wrote: > The sole use of xc_dom_translated() and xc_dom_p2m() outside of the domain > builder is for libxl_dom() to translate the console and xenstore pfns back > into useful values. PV guest pfns are only interesting to the domain builder, >

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

2017-10-06 Thread Roger Pau Monné
On Sat, Sep 30, 2017 at 01:39:25AM +, Yi Sun wrote: > This patch implements new generic set value interfaces in libxc and libxl. > These interfaces are suitable for all allocation features. It also adds a > new MBA set value command in xl. > > Signed-off-by: Yi Sun Reviewed-by: Roger Pau Mon

Re: [Xen-devel] [PATCH 0/7] xen: Convert __page_to_mfn and __mfn_to_page to use typesafe MFN

2017-10-06 Thread Tim Deegan
At 19:15 +0100 on 04 Oct (1507144519), Julien Grall wrote: > Hi all, > > Most of the users of page_to_mfn and mfn_to_page are either overriding > the macros to make them work with mfn_t or use mfn_x/_mfn becaue the rest > of the function use mfn_t. > > So I think it is time to make __page_to_mfn

Re: [Xen-devel] [PATCH] x86: Make use of pagetable_get_mfn() where appropriate

2017-10-06 Thread Tim Deegan
At 19:36 +0100 on 28 Sep (1506627400), Andrew Cooper wrote: > ... instead of the opencoded _mfn(pagetable_get_pfn(...)) construct. > > Fix two overly long lines; no functional change. > > Signed-off-by: Andrew Cooper Acked-by: Tim Deegan ___ Xen-dev

Re: [Xen-devel] [PATCH v5 10/16] tools: implement the new libxc get hw info interface

2017-10-06 Thread Roger Pau Monné
On Sat, Sep 30, 2017 at 01:39:20AM +, Yi Sun wrote: > This patch implements a new libxc get hw info interface and corresponding > data structures. It also changes libxl_psr.c to call this new interface. > > Signed-off-by: Yi Sun Reviewed-by: Roger Pau Monné Provided that my comment and Wei

Re: [Xen-devel] [PATCH for-4.10 2/5] tools/dombuilder: Remove clear_page() from xc_dom_boot.c

2017-10-06 Thread Wei Liu
On Thu, Oct 05, 2017 at 07:23:40PM +0100, Andrew Cooper wrote: > pfn 0 is a legitimate (albeit unlikely) frame to use for translated domains, > so skipping it is wrong. (This behaviour appears to exists simply to cover > the fact that zero is the default value of an uninitialised field in dom.) >

Re: [Xen-devel] [PATCH v5 14/16] tools: implement new generic get value interface and MBA get value command

2017-10-06 Thread Roger Pau Monné
On Sat, Sep 30, 2017 at 01:39:24AM +, 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] [RFC PATCH v3 00/24] ARM: Add Xen NUMA support

2017-10-06 Thread Julien Grall
On 06/10/17 18:09, vkil...@codeaurora.org wrote: Hi Julien, Hello, Sorry for not being active on this topic for some time. I was on vacation and have left cavium. Note: My email has been changed After going through the comments and fixing some of them, I see that clean up and code m

Re: [Xen-devel] [PATCH v5 09/16] tools: create general interfaces to support psr allocation features

2017-10-06 Thread Roger Pau Monné
On Sat, Sep 30, 2017 at 01:39:19AM +, 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 v5 08/16] x86: implement set value flow for MBA

2017-10-06 Thread Roger Pau Monné
On Sat, Sep 30, 2017 at 01:39:18AM +, Yi Sun wrote: > This patch implements set value flow for MBA including its callback > function and domctl interface. > > It also changes the memebers in 'cos_write_info' to transfer the > feature array, feature properties array and value array. Then, we >

Re: [Xen-devel] [PATCH v5 06/16] x86: implement get hw info flow for MBA

2017-10-06 Thread Roger Pau Monné
On Sat, Sep 30, 2017 at 01:39:16AM +, Yi Sun wrote: > This patch implements get HW info flow for MBA including its callback > function and sysctl interface. > > Signed-off-by: Yi Sun Reviewed-by: Roger Pau Monné Just one nit below. > --- > CC: Jan Beulich > CC: Andrew Cooper > CC: Wei L

Re: [Xen-devel] [PATCH v5 05/16] x86: implement data structure and CPU init flow for MBA

2017-10-06 Thread Roger Pau Monné
On Sat, Sep 30, 2017 at 01:39:15AM +, 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 v2 01/13] x86emul/fuzz: add rudimentary limit checking

2017-10-06 Thread George Dunlap
On 10/06/2017 04:54 PM, Jan Beulich wrote: On 06.10.17 at 17:21, wrote: >> On Mon, Sep 25, 2017 at 3:26 PM, George Dunlap >> wrote: >>> From: Jan Beulich >>> >>> fuzz_insn_fetch() is the only data access helper where it is possible >>> to see offsets larger than 4Gb in 16- or 32-bit modes,

Re: [Xen-devel] [PATCH v5 04/16] x86: a few optimizations to psr codes

2017-10-06 Thread Roger Pau Monné
On Sat, Sep 30, 2017 at 01:39:14AM +, Yi Sun wrote: > This patch refines psr codes: > 1. Change type of 'cat_init_feature' to 'bool' to remove the pointless >returning of error code. > 2. Move printk in 'cat_init_feature' to reduce a return path. > 3. Define a local variable 'ebx' in 'psr_c

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

2017-10-06 Thread Roger Pau Monné
On Sat, Sep 30, 2017 at 01:39:12AM +, 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

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

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

Re: [Xen-devel] [PATCH v2 25/24] xl: Document VGA problems arising from lack of physmap dmop

2017-10-06 Thread Paul Durrant
> -Original Message- > From: Ian Jackson [mailto:ian.jack...@eu.citrix.com] > Sent: 06 October 2017 16:02 > To: Paul Durrant > Cc: Wei Liu ; xen-de...@lists.xensource.com; Ross > Lagerwall > Subject: RE: [PATCH v2 25/24] xl: Document VGA problems arising from lack > of physmap dmop > > P

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

2017-10-06 Thread osstest service owner
flight 114055 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/114055/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-pvops 6 kernel-build fail REGR. vs. 114024 Tests which are fa

Re: [Xen-devel] [PATCH 11/13] x86/paravirt: Add paravirt alternatives infrastructure

2017-10-06 Thread Josh Poimboeuf
On Fri, Oct 06, 2017 at 11:29:52AM -0400, Boris Ostrovsky wrote: > >>> + > >>> void __init_or_module apply_paravirt(struct paravirt_patch_site *start, > >>>struct paravirt_patch_site *end) > >>> { > >>> diff --git a/arch/x86/kernel/cpu/hypervisor.c > >>> b/arch/x8

Re: [Xen-devel] [PATCH v3] x86/altp2m: Added xc_altp2m_set_mem_access_multi()

2017-10-06 Thread Razvan Cojocaru
On 10/06/2017 06:34 PM, Jan Beulich wrote: On 05.10.17 at 17:42, wrote: >> @@ -4451,6 +4453,7 @@ static int do_altp2m_op( >> case HVMOP_altp2m_destroy_p2m: >> case HVMOP_altp2m_switch_p2m: >> case HVMOP_altp2m_set_mem_access: >> +case HVMOP_altp2m_set_mem_access_multi: > >

Re: [Xen-devel] [PATCH v2 01/13] x86emul/fuzz: add rudimentary limit checking

2017-10-06 Thread Jan Beulich
>>> On 06.10.17 at 17:21, wrote: > On Mon, Sep 25, 2017 at 3:26 PM, George Dunlap > wrote: >> From: Jan Beulich >> >> fuzz_insn_fetch() is the only data access helper where it is possible >> to see offsets larger than 4Gb in 16- or 32-bit modes, as we leave the >> incoming rIP untouched in the

[Xen-devel] [xen-unstable-smoke test] 114077: regressions - FAIL

2017-10-06 Thread osstest service owner
flight 114077 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/114077/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt 12 guest-start fail REGR. vs. 113972 Tests which

Re: [Xen-devel] [PATCH v3] x86/altp2m: Added xc_altp2m_set_mem_access_multi()

2017-10-06 Thread Jan Beulich
>>> On 05.10.17 at 17:42, wrote: > @@ -4451,6 +4453,7 @@ static int do_altp2m_op( > case HVMOP_altp2m_destroy_p2m: > case HVMOP_altp2m_switch_p2m: > case HVMOP_altp2m_set_mem_access: > +case HVMOP_altp2m_set_mem_access_multi: Was it agreed that this, just like others (many wron

Re: [Xen-devel] [PATCH 11/13] x86/paravirt: Add paravirt alternatives infrastructure

2017-10-06 Thread Boris Ostrovsky
On 10/06/2017 10:32 AM, Josh Poimboeuf wrote: > On Thu, Oct 05, 2017 at 04:35:03PM -0400, Boris Ostrovsky wrote: >>> #ifdef CONFIG_PARAVIRT >>> +/* >>> + * Paravirt alternatives are applied much earlier than normal alternatives. >>> + * They are only applied when running on a hypervisor. They rep

Re: [Xen-devel] [PATCH v4 01/27] linkage: new macros for assembler symbols

2017-10-06 Thread Josh Poimboeuf
On Mon, Oct 02, 2017 at 11:12:20AM +0200, Jiri Slaby wrote: >SYM_CODE_INNER_LABEL -- only for labels in the middle of code >SYM_CODE_INNER_LABEL_NOALIGN -- only for labels in the middle of code Why are the inner labels aligned by default? Seems like unaligned would be the most common case

Re: [Xen-devel] [PATCH v2 01/13] x86emul/fuzz: add rudimentary limit checking

2017-10-06 Thread George Dunlap
On Mon, Sep 25, 2017 at 3:26 PM, George Dunlap wrote: > From: Jan Beulich > > fuzz_insn_fetch() is the only data access helper where it is possible > to see offsets larger than 4Gb in 16- or 32-bit modes, as we leave the > incoming rIP untouched in the emulator itself. The check is needed here >

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

2017-10-06 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 72198 xen-unstable real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/72198/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-examine 10 examine-serial/

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

2017-10-06 Thread Jan Beulich
>>> On 02.10.17 at 14:23, wrote: > +static void *hvmemul_map_linear_addr( > +unsigned long linear, unsigned int bytes, uint32_t pfec, > +struct hvm_emulate_ctxt *hvmemul_ctxt) > +{ > +struct vcpu *curr = current; > +void *err, *mapping; > + > +/* First and final gfns which need

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

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

Re: [Xen-devel] [PATCH v2 25/24] xl: Document VGA problems arising from lack of physmap dmop

2017-10-06 Thread Ian Jackson
Paul Durrant writes ("RE: [PATCH v2 25/24] xl: Document VGA problems arising from lack of physmap dmop"): > I suspect what it happening is that the VRAM is not moved into the BAR, but > the emulation in QEMU instead handles the BAR writes and then updates the > VRAM in its current location. Is

Re: [Xen-devel] [PATCH v5 01/16] docs: create Memory Bandwidth Allocation (MBA) feature document

2017-10-06 Thread Roger Pau Monné
On Sat, Sep 30, 2017 at 01:39:11AM +, Yi Sun wrote: > This patch creates MBA feature document in doc/features/. It describes > key points to implement MBA which is described in details in Intel SDM > "Introduction to Memory Bandwidth Allocation". > > Signed-off-by: Yi Sun Reviewed-by: Roger

Re: [Xen-devel] [PATCH v2 25/24] xl: Document VGA problems arising from lack of physmap dmop

2017-10-06 Thread Wei Liu
On Fri, Oct 06, 2017 at 03:57:09PM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH v2 25/24] xl: Document VGA problems arising from > lack of physmap dmop"): > > On Fri, Oct 06, 2017 at 03:33:52PM +0100, Ian Jackson wrote: > > > [ extra patch, following test report from Ross ] > ... > > >

Re: [Xen-devel] [PATCH v2 25/24] xl: Document VGA problems arising from lack of physmap dmop

2017-10-06 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH v2 25/24] xl: Document VGA problems arising from lack of physmap dmop"): > On Fri, Oct 06, 2017 at 03:33:52PM +0100, Ian Jackson wrote: > > [ extra patch, following test report from Ross ] ... > > +Domains with stdvga graphics cards to not work. > > +Domains with cirrus

Re: [Xen-devel] [PATCH v2 25/24] xl: Document VGA problems arising from lack of physmap dmop

2017-10-06 Thread Paul Durrant
> -Original Message- > From: Wei Liu [mailto:wei.l...@citrix.com] > Sent: 06 October 2017 15:51 > To: Ian Jackson > Cc: xen-de...@lists.xensource.com; Ross Lagerwall > ; Wei Liu ; Paul Durrant > > Subject: Re: [PATCH v2 25/24] xl: Document VGA problems arising from lack > of physmap dmop

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

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

Re: [Xen-devel] [PATCH v2 25/24] xl: Document VGA problems arising from lack of physmap dmop

2017-10-06 Thread Ross Lagerwall
On 10/06/2017 03:51 PM, Wei Liu wrote: On Fri, Oct 06, 2017 at 03:33:52PM +0100, Ian Jackson wrote: [ extra patch, following test report from Ross ] Ross reports that stdvga guests do not work, and cirrus guests are slow, because qemu tries to do xc_domain_add_to_physmap. We will need another

  1   2   >