Re: [Xen-devel] [PATCH] tools: Include due to glibc 2.25

2017-03-14 Thread Olaf Hering
On Tue, Mar 14, Anthony PERARD wrote: > +++ b/tools/libxl/libxl_internal.h > +#include There is a __linux__ section in that file. I sent a similar patch yesterday, but with a wrong change for blktap2. Olaf signature.asc Description: PGP signature __

Re: [Xen-devel] [PATCH] xen/arm: Register re-mapped Xen area as a temporary virtual region

2017-03-14 Thread Wei Chen
Hi Julien, On 2017/3/15 0:01, Julien Grall wrote: > Hello Wei, > > On 14/03/17 09:27, Wei Chen wrote: >> While we're doing apply_alternatives, we will generate new instructions > > I didn't spot this on the first review. apply_alternatives does not call > __apply_alternatives_multi_stop, so please

Re: [Xen-devel] [PATCH] xen/arm: Register re-mapped Xen area as a temporary virtual region

2017-03-14 Thread Wei Chen
Hi Konrad, On 2017/3/14 21:56, Konrad Rzeszutek Wilk wrote: > On Tue, Mar 14, 2017 at 01:32:31PM +, Julien Grall wrote: >> Hi Konrad, >> >> On 14/03/17 13:20, Konrad Rzeszutek Wilk wrote: >>> On Tue, Mar 14, 2017 at 05:27:17PM +0800, Wei Chen wrote: While we're doing apply_alternatives, w

Re: [Xen-devel] [PATCH] xen/arm: Register re-mapped Xen area as a temporary virtual region

2017-03-14 Thread Wei Chen
Hi Julien, On 2017/3/14 23:35, Julien Grall wrote: > Hello Wei, > > Title: I would add alternative to make clear this is touching alternative. > > On 14/03/17 09:27, Wei Chen wrote: >> While we're doing apply_alternatives, we will generate new instructions >> if required. The new instructions need

Re: [Xen-devel] [PATCH] xen/arm: Register re-mapped Xen area as a temporary virtual region

2017-03-14 Thread Wei Chen
Hi Julien, On 2017/3/14 21:32, Julien Grall wrote: > Hi Konrad, > > On 14/03/17 13:20, Konrad Rzeszutek Wilk wrote: >> On Tue, Mar 14, 2017 at 05:27:17PM +0800, Wei Chen wrote: >>> While we're doing apply_alternatives, we will generate new instructions >>> if required. The new instructions need to

Re: [Xen-devel] [PATCH v2 08/17] tools: provide pkg-config file for libxengnttab

2017-03-14 Thread Juergen Gross
On 14/03/17 21:06, Andrew Cooper wrote: > On 14/03/17 13:31, Juergen Gross wrote: >> In order to be able to use pkg-config for obtaining linker- and >> compiler-flags provide a xengnttab.pc and a xengntshr.pc file. >> >> Signed-off-by: Juergen Gross >> Acked-by: Wei Liu > > What is xengntshr.pc

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

2017-03-14 Thread osstest service owner
flight 106677 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/106677/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 5 xen-buildfail REGR. vs. 106664 Tests which

Re: [Xen-devel] [PATCH v2] boot allocator: Use arch helper for virt_to_mfn on DIRECTMAP

2017-03-14 Thread Vijay Kilari
On Tue, Mar 14, 2017 at 9:02 PM, Julien Grall wrote: > Hello Vijay, > > On 13/03/17 11:43, vijay.kil...@gmail.com wrote: >> >> From: Vijaya Kumar K >> >> On ARM, virt_to_mfn uses the hardware for address >> translation. So if the virtual address is not mapped translation >> fault is raised.On ARM

Re: [Xen-devel] [PATCH v3 4/7] xen/9pfs: connect to the backend

2017-03-14 Thread Juergen Gross
On 14/03/17 22:22, Stefano Stabellini wrote: > Hi Juergen, > > thank you for the review! > > On Tue, 14 Mar 2017, Juergen Gross wrote: >> On 14/03/17 00:50, Stefano Stabellini wrote: >>> Implement functions to handle the xenbus handshake. Upon connection, >>> allocate the rings according to the p

Re: [Xen-devel] [PATCH v7 3/3] x86: Make the GDT remapping read-only on 64-bit

2017-03-14 Thread H. Peter Anvin
,"Luis R . Rodriguez" ,Stanislaw Gruszka ,Peter Zijlstra ,Josh Poimboeuf ,Vitaly Kuznetsov ,Tim Chen ,Joerg Roedel ,=?UTF-8?B?UmFkaW0gS3LEjW3DocWZ?= ,the arch/x86 maintainers ,LKML ,linux-...@vger.kernel.org,kasan-dev ,Linux-MM ,Linux PM list ,linux-...@vger.kernel.org,xen-de...@lists.xenpr

[Xen-devel] [xen-4.6-testing test] 106663: tolerable FAIL - PUSHED

2017-03-14 Thread osstest service owner
flight 106663 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/106663/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-xl-rtds15 guest-start/debian.repeat fail REGR. vs. 106556 test-armhf-armhf-libvirt

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

2017-03-14 Thread osstest service owner
flight 106675 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/106675/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 5 xen-buildfail REGR. vs. 106664 Tests which

Re: [Xen-devel] [PATCH v8 08/24] x86: refactor psr: set value: implement framework.

2017-03-14 Thread Yi Sun
On 17-03-14 04:24:51, Jan Beulich wrote: > >>> On 14.03.17 at 10:21, wrote: > > On 17-03-14 00:29:09, Jan Beulich wrote: > >> >>> Yi Sun 03/14/17 3:42 AM >>> > >> >There are three scenarios. E.g. > >> >1. User calls domctl interface on Dom0 to set a COS ID 1 for Dom1 into its > >> >psr_cos_ids[].

[Xen-devel] [ovmf test] 106662: regressions - FAIL

2017-03-14 Thread osstest service owner
flight 106662 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/106662/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-ovmf-amd64 9 debian-hvm-install fail REGR. vs. 105963 test-amd64-i386-xl-qemuu-

[Xen-devel] [PATCH] tools/xl/xl_parse.c: Remove blank line.

2017-03-14 Thread Zhang Chen
Address Liu wei's comments. Signed-off-by: Zhang Chen --- tools/xl/xl_parse.c | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/xl/xl_parse.c b/tools/xl/xl_parse.c index 0787337..faa9c61 100644 --- a/tools/xl/xl_parse.c +++ b/tools/xl/xl_parse.c @@ -524,7 +524,6 @@ int parse_nic_config(li

Re: [Xen-devel] [PATCH V4 7/8] COLO-Proxy: Use socket to get checkpoint event.

2017-03-14 Thread Zhang Chen
On 03/14/2017 07:24 PM, Wei Liu wrote: On Mon, Mar 06, 2017 at 10:59:25AM +0800, Zhang Chen wrote: We use kernel colo proxy's way to get the checkpoint event from qemu colo-compare. Qemu colo-compare need add a API to support this(I will add this in qemu). Qemu side patch: https://lists.nong

Re: [Xen-devel] [PATCH V4 5/8] COLO-Proxy: Add secondary userspace colo-proxy start args

2017-03-14 Thread Zhang Chen
On 03/14/2017 07:39 PM, Wei Liu wrote: On Tue, Mar 14, 2017 at 11:24:12AM +, Wei Liu wrote: On Mon, Mar 06, 2017 at 10:59:23AM +0800, Zhang Chen wrote: Qemu need this args to start userspace colo-proxy. Signed-off-by: Zhang Chen Acked-by: Wei Liu diff --git a/tools/xl/xl_parse.c b/t

[Xen-devel] [xen-4.7-testing test] 106661: tolerable FAIL - PUSHED

2017-03-14 Thread osstest service owner
flight 106661 xen-4.7-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/106661/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-libvirt-xsm 13 saverestore-support-checkfail like 106540 test-amd64-i386-xl-qemuu

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

2017-03-14 Thread osstest service owner
flight 106673 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/106673/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 5 xen-buildfail REGR. vs. 106664 Tests which

Re: [Xen-devel] [PATCH v2 1/9] configure: change CONFIG_XEN_BACKEND to be a target property

2017-03-14 Thread Greg Kurz
On Tue, 14 Mar 2017 13:23:09 -0700 (PDT) Stefano Stabellini wrote: > On Tue, 14 Mar 2017, Stefano Stabellini wrote: > > On Tue, 14 Mar 2017, Paolo Bonzini wrote: > > > On 14/03/2017 00:55, Stefano Stabellini wrote: > > > > CONFIG_XEN_BACKEND is currently set when the host supports Xen, > > >

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

2017-03-14 Thread osstest service owner
flight 106660 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/106660/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl 11 guest-start fail REGR. vs. 59254 test-armhf-armhf-xl

Re: [Xen-devel] [PATCH 07/18] xen/arm: Introduce a command line parameter for SErrors/Aborts

2017-03-14 Thread Stefano Stabellini
On Mon, 13 Mar 2017, Wei Chen wrote: > In order to distinguish guest-generated SErrors from hypervisor-generated > SErrors. We have to place SError checking code in every EL1 -> EL2 paths. ^ remove . > That will be an overhead on entries caused by dsb/isb. > > But not all platforms want

Re: [Xen-devel] [PATCH 06/18] xen/arm: Introduce a virtual abort injection helper

2017-03-14 Thread Stefano Stabellini
On Mon, 13 Mar 2017, Wei Chen wrote: > When guest triggers async aborts, in most platform, such aborts > will be routed to hypervisor. But we don't want the hypervisor > to handle such aborts, so we have to route such aborts back to > the guest. > > This helper is using the HCR_EL2.VSE (HCR.VA for

Re: [Xen-devel] [PATCH 04/18] xen/arm: Save HCR_EL2 when a guest took the SError

2017-03-14 Thread Stefano Stabellini
On Mon, 13 Mar 2017, Wei Chen wrote: > The HCR_EL2.VSE (HCR.VA for aarch32) bit can be used to generate a > virtual abort to guest. The HCR_EL2.VSE bit has a peculiar feature > of getting cleared when the guest has taken the abort (this is the > only bit that behaves as such in HCR_EL2 register). >

Re: [Xen-devel] [PATCH 03/18] xen/arm: Avoid setting/clearing HCR_RW at every context switch

2017-03-14 Thread Stefano Stabellini
On Mon, 13 Mar 2017, Wei Chen wrote: > The HCR_EL2 flags for 64-bit and 32-bit domains are different. But > when we initialized the HCR_EL2 for vcpu0 of Dom0 and all vcpus of > DomU in vcpu_initialise, we didn't know the domain's address size > information. We had to use compatible flags to initial

Re: [Xen-devel] [PATCH 02/18] xen/arm: Restore HCR_EL2 register

2017-03-14 Thread Stefano Stabellini
On Mon, 13 Mar 2017, Wei Chen wrote: > Different domains may have different HCR_EL2 flags. For example, the > 64-bit domain needs HCR_RW flag but the 32-bit does not need it. So > we give each domain a default HCR_EL2 value and save it in the VCPU's > context. > > HCR_EL2 register has only one bit

Re: [Xen-devel] [PATCH 01/18] xen/arm: Introduce a helper to get default HCR_EL2 flags

2017-03-14 Thread Stefano Stabellini
On Mon, 13 Mar 2017, Wei Chen wrote: > We want to add HCR_EL2 register to Xen context switch. And each copy > of HCR_EL2 in vcpu structure will be initialized with the same set > of trap flags as the HCR_EL2 register. We introduce a helper here to > represent these flags to be reused easily. > > S

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

2017-03-14 Thread osstest service owner
flight 106672 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/106672/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 5 xen-buildfail REGR. vs. 106664 Tests which

Re: [Xen-devel] [PATCH 3/3] xen/arm: p2m: Perform local TLB invalidation on vCPU migration

2017-03-14 Thread Stefano Stabellini
On Tue, 14 Mar 2017, Julien Grall wrote: > Hi Stefano, > > On 03/14/2017 10:39 PM, Stefano Stabellini wrote: > > On Wed, 8 Mar 2017, Julien Grall wrote: > > > On 08/03/2017 19:48, Stefano Stabellini wrote: > > > > On Wed, 8 Mar 2017, Julien Grall wrote: > > I see your point, it is a good idea to u

Re: [Xen-devel] [PATCH 3/3] xen/arm: p2m: Perform local TLB invalidation on vCPU migration

2017-03-14 Thread Julien Grall
Hi Stefano, On 03/14/2017 10:39 PM, Stefano Stabellini wrote: On Wed, 8 Mar 2017, Julien Grall wrote: On 08/03/2017 19:48, Stefano Stabellini wrote: On Wed, 8 Mar 2017, Julien Grall wrote: I see your point, it is a good idea to use the remaining space in struct domain. The only issue is that

Re: [Xen-devel] [PATCH 3/3] xen/arm: p2m: Perform local TLB invalidation on vCPU migration

2017-03-14 Thread Stefano Stabellini
On Wed, 8 Mar 2017, Julien Grall wrote: > On 08/03/2017 19:48, Stefano Stabellini wrote: > > On Wed, 8 Mar 2017, Julien Grall wrote: > > > Hi Stefano, > > > > > > On 08/03/17 18:58, Stefano Stabellini wrote: > > > > On Wed, 8 Mar 2017, Julien Grall wrote: > > > > > The ARM architecture allows an O

[Xen-devel] [PATCH 2/3] Introduce the Xen 9pfs transport header

2017-03-14 Thread Stefano Stabellini
Define the ring according to the protocol specification, using the new DEFINE_XEN_FLEX_RING_AND_INTF macro. Signed-off-by: Stefano Stabellini CC: konrad.w...@oracle.com --- xen/include/public/io/9pfs.h | 42 ++ 1 file changed, 42 insertions(+) create mode

[Xen-devel] [PATCH 3/3] Introduce the pvcalls header

2017-03-14 Thread Stefano Stabellini
Define the ring and request and response structs according to the specification. Use the new DEFINE_XEN_FLEX_RING macro. Signed-off-by: Stefano Stabellini CC: konrad.w...@oracle.com --- xen/include/public/io/pvcalls.h | 138 1 file changed, 138 insertions

[Xen-devel] [PATCH 1/3] [RESEND v4] ring.h: introduce macros to handle monodirectional rings with multiple req sizes

2017-03-14 Thread Stefano Stabellini
This patch introduces macros, structs and functions to handle rings in the format described by docs/misc/pvcalls.markdown and docs/misc/9pfs.markdown. The index page (struct __name##_data_intf) contains the indexes and the grant refs to setup two rings. Indexes page +

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

2017-03-14 Thread osstest service owner
flight 106670 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/106670/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 5 xen-buildfail REGR. vs. 106664 Tests which

[Xen-devel] [xen-4.8-testing test] 106659: tolerable FAIL - PUSHED

2017-03-14 Thread osstest service owner
flight 106659 xen-4.8-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/106659/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemuu-win7-amd64 16 guest-stop fail like 106535 test-amd64-amd64-xl-qemu

Re: [Xen-devel] [PATCH v3 4/7] xen/9pfs: connect to the backend

2017-03-14 Thread Stefano Stabellini
Hi Juergen, thank you for the review! On Tue, 14 Mar 2017, Juergen Gross wrote: > On 14/03/17 00:50, Stefano Stabellini wrote: > > Implement functions to handle the xenbus handshake. Upon connection, > > allocate the rings according to the protocol specification. > > > > Initialize a work_struct

Re: [Xen-devel] [PATCH v7 3/3] x86: Make the GDT remapping read-only on 64-bit

2017-03-14 Thread Thomas Garnier
On Tue, Mar 14, 2017 at 2:04 PM, Pavel Machek wrote: > On Tue 2017-03-14 10:05:08, Thomas Garnier wrote: >> This patch makes the GDT remapped pages read-only to prevent corruption. >> This change is done only on 64-bit. >> >> The native_load_tr_desc function was adapted to correctly handle a >> re

Re: [Xen-devel] Future x86 emulator direction

2017-03-14 Thread Razvan Cojocaru
On 12/14/2016 09:37 AM, Razvan Cojocaru wrote: > On 12/14/2016 09:14 AM, Jan Beulich wrote: > On 13.12.16 at 23:02, wrote: >>> On 13/12/2016 21:55, Razvan Cojocaru wrote: On a somewhat related note, it's important to also figure out how best to avoid emulation races such as the LOCK

Re: [Xen-devel] [PATCH v7 3/3] x86: Make the GDT remapping read-only on 64-bit

2017-03-14 Thread Pavel Machek
On Tue 2017-03-14 10:05:08, Thomas Garnier wrote: > This patch makes the GDT remapped pages read-only to prevent corruption. > This change is done only on 64-bit. > > The native_load_tr_desc function was adapted to correctly handle a > read-only GDT. The LTR instruction always writes to the GDT TS

Re: [Xen-devel] [PATCH v3 6/7] xen/9pfs: receive responses

2017-03-14 Thread Stefano Stabellini
On Tue, 14 Mar 2017, Juergen Gross wrote: > On 14/03/17 00:50, Stefano Stabellini wrote: > > Upon receiving a notification from the backend, schedule the > > p9_xen_response work_struct. p9_xen_response checks if any responses are > > available, if so, it reads them one by one, calling p9_client_cb

Re: [Xen-devel] [PATCH v3 3/7] xen/9pfs: introduce Xen 9pfs transport driver

2017-03-14 Thread Stefano Stabellini
On Tue, 14 Mar 2017, Juergen Gross wrote: > On 14/03/17 00:50, Stefano Stabellini wrote: > > Introduce the Xen 9pfs transport driver: add struct xenbus_driver to > > register as a xenbus driver and add struct p9_trans_module to register > > as v9fs driver. > > > > All functions are empty stubs for

Re: [Xen-devel] [PATCH v2 1/9] configure: change CONFIG_XEN_BACKEND to be a target property

2017-03-14 Thread Stefano Stabellini
On Tue, 14 Mar 2017, Stefano Stabellini wrote: > On Tue, 14 Mar 2017, Paolo Bonzini wrote: > > On 14/03/2017 00:55, Stefano Stabellini wrote: > > > CONFIG_XEN_BACKEND is currently set when the host supports Xen, > > > regardless of the chosen targets. As a consequence, Xen backends can be > > > ena

Re: [Xen-devel] [PATCH v2 5/9] xen/9pfs: connect to the frontend

2017-03-14 Thread Stefano Stabellini
On Tue, 14 Mar 2017, Juergen Gross wrote: > On 14/03/17 00:55, Stefano Stabellini wrote: > > Write the limits of the backend to xenstore. Connect to the frontend. > > Upon connection, allocate the rings according to the protocol > > specification. > > > > Initialize a QEMUBH to schedule work upon

[Xen-devel] [xen-unstable test] 106652: tolerable FAIL - PUSHED

2017-03-14 Thread osstest service owner
flight 106652 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/106652/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-armhf-armhf-xl-cubietruck 15 guest-start/debian.repeat fail in 106642 pass in 106652 test-armhf-armhf-lib

Re: [Xen-devel] [PATCH v2 4/9] xen/9pfs: introduce Xen 9pfs backend

2017-03-14 Thread Stefano Stabellini
On Tue, 14 Mar 2017, Juergen Gross wrote: > On 14/03/17 00:55, Stefano Stabellini wrote: > > Introduce the Xen 9pfs backend: add struct XenDevOps to register as a > > Xen backend and add struct V9fsTransport to register as v9fs transport. > > > > All functions are empty stubs for now. > > > > Sig

Re: [Xen-devel] [PATCH v2 08/17] tools: provide pkg-config file for libxengnttab

2017-03-14 Thread Andrew Cooper
On 14/03/17 13:31, Juergen Gross wrote: > In order to be able to use pkg-config for obtaining linker- and > compiler-flags provide a xengnttab.pc and a xengntshr.pc file. > > Signed-off-by: Juergen Gross > Acked-by: Wei Liu What is xengntshr.pc actually for? (I am forward porting the XenServer

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

2017-03-14 Thread osstest service owner
flight 106667 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/106667/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 5 xen-buildfail REGR. vs. 106664 Tests which

Re: [Xen-devel] WTH is going on with memory hotplug sysf interface (was: Re: [RFC PATCH] mm, hotplug: get rid of auto_online_blocks)

2017-03-14 Thread Andrea Arcangeli
Hello, On Mon, Mar 13, 2017 at 10:21:45AM +0100, Michal Hocko wrote: > On Fri 10-03-17 13:00:37, Reza Arbab wrote: > > On Fri, Mar 10, 2017 at 04:53:33PM +0100, Michal Hocko wrote: > > >OK, so while I was playing with this setup some more I probably got why > > >this is done this way. All new memb

[Xen-devel] [linux-4.1 test] 106655: regressions - FAIL

2017-03-14 Thread osstest service owner
flight 106655 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/106655/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-pvh-intel 11 guest-start fail REGR. vs. 104301 Tests which are faili

Re: [Xen-devel] [PATCH v2 1/9] configure: change CONFIG_XEN_BACKEND to be a target property

2017-03-14 Thread Stefano Stabellini
On Tue, 14 Mar 2017, Paolo Bonzini wrote: > On 14/03/2017 00:55, Stefano Stabellini wrote: > > CONFIG_XEN_BACKEND is currently set when the host supports Xen, > > regardless of the chosen targets. As a consequence, Xen backends can be > > enabled even on targets that don't support Xen. > > > > Fix

Re: [Xen-devel] [PATCH v4] xen: don't save/restore the physmap on VM save/restore

2017-03-14 Thread Stefano Stabellini
On Tue, 14 Mar 2017, Igor Druzhinin wrote: > Saving/restoring the physmap to/from xenstore was introduced to > QEMU majorly in order to cover up the VRAM region restore issue. > The sequence of restore operations implies that we should know > the effective guest VRAM address *before* we have the VR

Re: [Xen-devel] [PATCH v3] xen: don't save/restore the physmap on VM save/restore

2017-03-14 Thread Stefano Stabellini
On Tue, 14 Mar 2017, Igor Druzhinin wrote: > On 13/03/17 21:15, Stefano Stabellini wrote: > > On Mon, 13 Mar 2017, Igor Druzhinin wrote: > >> Saving/restoring the physmap to/from xenstore was introduced to > >> QEMU majorly in order to cover up the VRAM region restore issue. > >> The sequence of re

[Xen-devel] [PATCH] tools: Include due to glibc 2.25

2017-03-14 Thread Anthony PERARD
To fix the follwing error with glibc 2.25: tap-ctl-allocate.c:109:13: error: In the GNU C Library, "makedev" is defined by . For historical compatibility, it is currently defined by as well, but we plan to remove this soon. To use "makedev", include directly. If you did not intend to use a sy

[Xen-devel] [PATCH v3 18/21] xen/balloon: decorate PV-only parts with #ifdef CONFIG_XEN_PV

2017-03-14 Thread Vitaly Kuznetsov
Balloon driver uses several PV-only concepts (xen_start_info, xen_extra_mem,..) and it seems the simpliest solution to make HVM-only build happy is to decorate these parts with #ifdefs. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- drivers/xen/balloon.c | 30 +++

[Xen-devel] [PATCH v3 14/21] x86/xen: split suspend.c for PV and PVHVM guests

2017-03-14 Thread Vitaly Kuznetsov
Slit the code in suspend.c into suspend_pv.c and suspend_hvm.c. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- Changes since v2: #include added to suspend_pv.c [kbuild test robot] --- arch/x86/xen/Makefile | 3 ++- arch/x86/xen/suspend.c | 54 --

[Xen-devel] [PATCH v3 19/21] xen: create xen_create/destroy_contiguous_region() stubs for PVHVM only builds

2017-03-14 Thread Vitaly Kuznetsov
xen_create_contiguous_region()/xen_create_contiguous_region() are PV-only, they both contain xen_feature(XENFEAT_auto_translated_physmap) check and bail in the very beginning. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- include/xen/xen-ops.h | 14 ++ 1 file change

[Xen-devel] [PATCH v3 16/21] x86/xen: define startup_xen for XEN PV only

2017-03-14 Thread Vitaly Kuznetsov
startup_xen references PV-only code, decorate it with #ifdef CONFIG_XEN_PV to make PV-free builds possible. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- arch/x86/xen/xen-head.S | 4 1 file changed, 4 insertions(+) diff --git a/arch/x86/xen/xen-head.S b/arch/x86/xen/xen-h

[Xen-devel] [PATCH v3 21/21] x86/xen: rename some PV-only functions in smp_pv.c

2017-03-14 Thread Vitaly Kuznetsov
After code split between PV and HVM some functions in xen_smp_ops have xen_pv_ prefix and some only xen_ which makes them look like they're common for both PV and HVM while they're not. Rename all the rest to have xen_pv_ prefix. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- ar

[Xen-devel] [PATCH v3 15/21] x86/xen: put setup.c, pmu.c and apic.c under CONFIG_XEN_PV

2017-03-14 Thread Vitaly Kuznetsov
xen_pmu_init/finish() functions are used in suspend.c and enlighten.c, add stubs for now. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- arch/x86/xen/Kconfig | 2 +- arch/x86/xen/Makefile | 6 +++--- arch/x86/xen/pmu.h| 5 + 3 files changed, 9 insertions(+), 4 deletions

[Xen-devel] [PATCH v3 10/21] x86/xen: split off smp_hvm.c

2017-03-14 Thread Vitaly Kuznetsov
Move PVHVM related code to smp_hvm.c. Drop 'static' qualifier from xen_smp_send_reschedule(), xen_smp_send_call_function_ipi(), xen_smp_send_call_function_single_ipi(), these functions will be moved to common smp code when smp_pv.c is split. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gr

[Xen-devel] [PATCH v3 20/21] x86/xen: enable PVHVM-only builds

2017-03-14 Thread Vitaly Kuznetsov
Now everything is in place and we can move PV-only code under CONFIG_XEN_PV. CONFIG_XEN_PV_SMP is created to support the change. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- arch/x86/xen/Kconfig | 4 arch/x86/xen/Makefile | 9 + 2 files changed, 9 insertions(+),

[Xen-devel] [PATCH v3 00/21] x86/xen: untangle PV and PVHVM guest support code

2017-03-14 Thread Vitaly Kuznetsov
Changes since v2: - Rebase to 4.11.0-rc1+ - XEN_HAVE_PVMMU moved to config XEN_PV [Juergen Gross] - .pin_vcpu kept for x86_hyper_xen_hvm to support PVH Dom0 in future [Juergen Gross] - 'extern' qualifiers dropped from newly introduced function prototypes in headers [Juergen Gross] - A couple o

[Xen-devel] [PATCH v3 12/21] x86/xen: split off mmu_hvm.c

2017-03-14 Thread Vitaly Kuznetsov
Move PVHVM related code to mmu_hvm.c. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- arch/x86/xen/Makefile | 2 +- arch/x86/xen/mmu.c | 74 -- arch/x86/xen/mmu_hvm.c | 79 ++ 3 file

[Xen-devel] [PATCH v3 06/21] x86/xen: split off enlighten_pv.c

2017-03-14 Thread Vitaly Kuznetsov
Basically, enlighten.c is renamed to enlighten_pv.c and some code moved out to common enlighten.c. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- Changes since v2: xen_pin_vcpu() left in enlighten.c --- arch/x86/xen/Makefile |4 +- arch/x86/xen/enlighten.c| 1647

[Xen-devel] [PATCH v3 11/21] x86/xen: split off smp_pv.c

2017-03-14 Thread Vitaly Kuznetsov
Basically, smp.c is renamed to smp_pv.c and some code moved out to common smp.c. struct xen_common_irq delcaration ended up in smp.h. Reviewed-by: Juergen Gross Signed-off-by: Vitaly Kuznetsov --- arch/x86/xen/Makefile | 2 +- arch/x86/xen/smp.c| 488 +-

[Xen-devel] [PATCH v3 17/21] x86/xen: create stubs for HVM-only builds in page.h

2017-03-14 Thread Vitaly Kuznetsov
__pfn_to_mfn() is only used from PV code (mmu_pv.c, p2m.c) and from page.h where all functions calling it check for xen_feature(XENFEAT_auto_translated_physmap) first so we can replace it with any stub to make build happy. set_foreign_p2m_mapping()/clear_foreign_p2m_mapping() are used from grant-t

[Xen-devel] [PATCH v3 03/21] x86/xen: add CONFIG_XEN_PV to Kconfig

2017-03-14 Thread Vitaly Kuznetsov
All code to support Xen PV will get under this new option. For the beginning, check for it in the common code. Signed-off-by: Vitaly Kuznetsov --- Changes since v2: select XEN_HAVE_PVMMU moved to config XEN_PV [Juergen Gross] --- arch/x86/kernel/cpu/hypervisor.c | 4 +++- arch/x86/kernel/pro

[Xen-devel] [PATCH v3 02/21] x86/xen: globalize have_vcpu_info_placement

2017-03-14 Thread Vitaly Kuznetsov
have_vcpu_info_placement applies to both PV and HVM and as we're going to split the code we need to make it global. Rename to xen_have_vcpu_info_placement. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- arch/x86/xen/enlighten.c | 12 ++-- arch/x86/xen/xen-ops.h | 2 +

[Xen-devel] [PATCH v3 09/21] x86/xen: split xen_cpu_die()

2017-03-14 Thread Vitaly Kuznetsov
Split xen_cpu_die() into xen_pv_cpu_die() and xen_hvm_cpu_die() to support further splitting of smp.c. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- arch/x86/xen/smp.c | 26 -- 1 file changed, 20 insertions(+), 6 deletions(-) diff --git a/arch/x86/xen/s

[Xen-devel] [PATCH v3 07/21] x86/xen: split xen_smp_intr_init()/xen_smp_intr_free()

2017-03-14 Thread Vitaly Kuznetsov
xen_smp_intr_init() and xen_smp_intr_free() have PV-specific code and as a praparatory change to splitting smp.c we need to split these fucntions. Create xen_smp_intr_init_pv()/xen_smp_intr_free_pv(). Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- Changes since v2: 'extern' q

Re: [Xen-devel] DomU UEFI configuration

2017-03-14 Thread Anthony PERARD
On Tue, Mar 14, 2017 at 02:06:09PM +, Marcos E. Matsunaga wrote: > Hi Anthony, > > > I am trying to run a DomU guest using OVMF. I have built xen staging with > --with-ovmf option. I am able to install a new guest but I am unable to boot > it (see the attached image). > > The xl.cfg file I a

[Xen-devel] [PATCH v3 08/21] x86/xen: split xen_smp_prepare_boot_cpu()

2017-03-14 Thread Vitaly Kuznetsov
Split xen_smp_prepare_boot_cpu() into xen_pv_smp_prepare_boot_cpu() and xen_hvm_smp_prepare_boot_cpu() to support further splitting of smp.c. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- arch/x86/xen/smp.c | 49 ++--- 1 file changed,

[Xen-devel] [PATCH v3 05/21] x86/xen: split off enlighten_hvm.c

2017-03-14 Thread Vitaly Kuznetsov
Move PVHVM related code to enlighten_hvm.c. Three functions: xen_cpuhp_setup(), xen_reboot(), xen_emergency_restart() are shared, drop static qualifier from them. These functions will go to common code once it is split from enlighten.c. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross

[Xen-devel] [PATCH v3 01/21] x86/xen: separate PV and HVM hypervisors

2017-03-14 Thread Vitaly Kuznetsov
As a preparation to splitting the code we need to untangle it: x86_hyper_xen -> x86_hyper_xen_hvm and x86_hyper_xen_pv xen_platform() -> xen_platform_hvm() and xen_platform_pv() xen_cpu_up_prepare() -> xen_cpu_up_prepare_pv() and xen_cpu_up_prepare_hvm() xen_cpu_dead() -> xen_cpu_dead_pv() and xen

[Xen-devel] [PATCH v3 04/21] x86/xen: split off enlighten_pvh.c

2017-03-14 Thread Vitaly Kuznetsov
Create enlighten_pvh.c by splitting off PVH related code from enlighten.c, put it under CONFIG_XEN_PVH. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross --- Changes since v2: #include added [kbuild test robot] --- arch/x86/xen/Makefile| 2 + arch/x86/xen/enlighten.c

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

2017-03-14 Thread osstest service owner
flight 10 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/10/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 5 xen-buildfail REGR. vs. 106664 Tests which

[Xen-devel] [PATCH v7 1/3] x86/mm: Adapt MODULES_END based on Fixmap section size

2017-03-14 Thread Thomas Garnier
This patch aligns MODULES_END to the beginning of the Fixmap section. It optimizes the space available for both sections. The address is pre-computed based on the number of pages required by the Fixmap section. It will allow GDT remapping in the Fixmap section. The current MODULES_END static addre

[Xen-devel] [PATCH v7 2/3] x86: Remap GDT tables in the Fixmap section

2017-03-14 Thread Thomas Garnier
Each processor holds a GDT in its per-cpu structure. The sgdt instruction gives the base address of the current GDT. This address can be used to bypass KASLR memory randomization. With another bug, an attacker could target other per-cpu structures or deduce the base of the main memory section (PAGE

[Xen-devel] [PATCH v7 3/3] x86: Make the GDT remapping read-only on 64-bit

2017-03-14 Thread Thomas Garnier
This patch makes the GDT remapped pages read-only to prevent corruption. This change is done only on 64-bit. The native_load_tr_desc function was adapted to correctly handle a read-only GDT. The LTR instruction always writes to the GDT TSS entry. This generates a page fault if the GDT is read-only

Re: [Xen-devel] help please...

2017-03-14 Thread Wei Liu
On Mon, Mar 13, 2017 at 10:36:50AM +0800, 刘瑞丰 wrote: > [image: 内嵌图片 1] > Please don't send pictures to mailing list. Not everyone can see them (some people use text based email clients) and they consume a lot of bandwidth combined. > it reports: > > checking for PyArg_ParseTuple in -lpython2.7.

[Xen-devel] [PATCH v4] xen: don't save/restore the physmap on VM save/restore

2017-03-14 Thread Igor Druzhinin
Saving/restoring the physmap to/from xenstore was introduced to QEMU majorly in order to cover up the VRAM region restore issue. The sequence of restore operations implies that we should know the effective guest VRAM address *before* we have the VRAM region restored (which happens later). Unfortuna

Re: [Xen-devel] [PATCH v3 2/4] x86: remove PVHv1 code

2017-03-14 Thread Wei Liu
On Tue, Mar 14, 2017 at 10:27:25AM +, Wei Liu wrote: > On Mon, Mar 13, 2017 at 04:50:12PM +, Wei Liu wrote: > > On Fri, Mar 03, 2017 at 12:25:06PM +, Roger Pau Monne wrote: > > > This removal applies to both the hypervisor and the toolstack side of > > > PVHv1. > > > > > > Note that o

Re: [Xen-devel] WTH is going on with memory hotplug sysf interface

2017-03-14 Thread Michal Hocko
On Tue 14-03-17 12:05:59, YASUAKI ISHIMATSU wrote: > > > On 03/13/2017 05:19 AM, Michal Hocko wrote: > >On Fri 10-03-17 12:39:27, Yasuaki Ishimatsu wrote: > >>On 03/10/2017 08:58 AM, Michal Hocko wrote: [...] > >>># echo online_movable > /sys/devices/system/memory/memory34/state > >>># grep . /sy

Re: [Xen-devel] WTH is going on with memory hotplug sysf interface

2017-03-14 Thread YASUAKI ISHIMATSU
On 03/13/2017 05:19 AM, Michal Hocko wrote: On Fri 10-03-17 12:39:27, Yasuaki Ishimatsu wrote: On 03/10/2017 08:58 AM, Michal Hocko wrote: [...] OK so I did with -m 2G,slots=4,maxmem=4G -numa node,mem=1G -numa node,mem=1G which generated [...] [0.00] ACPI: SRAT: Node 0 PXM 0 [mem 0x

Re: [Xen-devel] [PATCH 10/10] x86/cpuid: Always enable faulting for the control domain

2017-03-14 Thread Wei Liu
On Tue, Mar 14, 2017 at 09:13:08AM -0600, Jan Beulich wrote: > >>> On 14.03.17 at 16:06, wrote: > > On Mon, Mar 13, 2017 at 05:48:44AM -0600, Jan Beulich wrote: > >> >>> On 10.03.17 at 18:10, wrote: > >> > On 28/02/17 09:31, Jan Beulich wrote: > >> > On 27.02.17 at 16:10, wrote: > >> >>> On

[Xen-devel] 2017 Xen Project Developer and Design Summit: CfP open from now to April 14, Event is held July 11-13, 2017 in Budapest

2017-03-14 Thread Lars Kurth
Dear Community Members, I am excited to announce that registration and the call for proposals is open for Xen Project Developer and Design Summit 2017, which will be held in Budapest, Hungary from July 11-13, 2017. The Xen Project Developer and Design Summit combines the formats of Xen Project

Re: [Xen-devel] [PATCH] xen/arm: Register re-mapped Xen area as a temporary virtual region

2017-03-14 Thread Julien Grall
Hello Wei, On 14/03/17 09:27, Wei Chen wrote: While we're doing apply_alternatives, we will generate new instructions I didn't spot this on the first review. apply_alternatives does not call __apply_alternatives_multi_stop, so please clarify the problem. Cheers, -- Julien Grall __

[Xen-devel] [xen-unstable-smoke test] 106664: tolerable trouble: broken/fail/pass - PUSHED

2017-03-14 Thread osstest service owner
flight 106664 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/106664/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-arm64-arm64-xl-xsm 1 build-check(1) blocked n/a build-arm64 5 xen

Re: [Xen-devel] [PATCH] xen/arm: Register re-mapped Xen area as a temporary virtual region

2017-03-14 Thread Julien Grall
Hello Wei, Title: I would add alternative to make clear this is touching alternative. On 14/03/17 09:27, Wei Chen wrote: While we're doing apply_alternatives, we will generate new instructions if required. The new instructions need to update the Xen text section, but Xen text section is read-on

Re: [Xen-devel] [PATCH v2] boot allocator: Use arch helper for virt_to_mfn on DIRECTMAP

2017-03-14 Thread Julien Grall
Hello Vijay, On 13/03/17 11:43, vijay.kil...@gmail.com wrote: From: Vijaya Kumar K On ARM, virt_to_mfn uses the hardware for address translation. So if the virtual address is not mapped translation fault is raised.On ARM, DIRECTMAP_VIRT region is direct mapped. This is not true. As I said be

Re: [Xen-devel] [PATCH] tools: correct build in directory below tools

2017-03-14 Thread Wei Liu
On Tue, Mar 14, 2017 at 04:04:41PM +0100, Juergen Gross wrote: > Recent changes to create *.pc files introduced a bug when trying to > build a library from a directory below tools as PKG_CONFIG_DIR wouldn't > be set. Correct this by adding a default value to Rules.mk. > > Signed-off-by: Juergen Gr

Re: [Xen-devel] [PATCH 10/10] x86/cpuid: Always enable faulting for the control domain

2017-03-14 Thread Jan Beulich
>>> On 14.03.17 at 16:06, wrote: > On Mon, Mar 13, 2017 at 05:48:44AM -0600, Jan Beulich wrote: >> >>> On 10.03.17 at 18:10, wrote: >> > On 28/02/17 09:31, Jan Beulich wrote: >> > On 27.02.17 at 16:10, wrote: >> >>> On 22/02/17 10:10, Jan Beulich wrote: >> >>> On 22.02.17 at 11:00, wrot

Re: [Xen-devel] [PATCH] tools/libxl: correct distclean target

2017-03-14 Thread Wei Liu
On Tue, Mar 14, 2017 at 04:04:42PM +0100, Juergen Gross wrote: > Commit 3e5f1a63b53920763 ("tools: adapt xenlight.pc and xlutil.pc to > new pkg-config scheme") introduced an error for "make distclean" as > *.pc.in are deleted which are now files in git. > > Signed-off-by: Juergen Gross Acked-by:

Re: [Xen-devel] [PATCH 08/29] drivers, md: convert mddev.active from atomic_t to refcount_t

2017-03-14 Thread James Bottomley
On Tue, 2017-03-14 at 12:29 +, Reshetova, Elena wrote: > > Elena Reshetova writes: > > > > > refcount_t type and corresponding API should be > > > used instead of atomic_t when the variable is used as > > > a reference counter. This allows to avoid accidental > > > refcounter overflows that m

Re: [Xen-devel] [PATCH 10/10] x86/cpuid: Always enable faulting for the control domain

2017-03-14 Thread Wei Liu
On Mon, Mar 13, 2017 at 05:48:44AM -0600, Jan Beulich wrote: > >>> On 10.03.17 at 18:10, wrote: > > On 28/02/17 09:31, Jan Beulich wrote: > > On 27.02.17 at 16:10, wrote: > >>> On 22/02/17 10:10, Jan Beulich wrote: > >>> On 22.02.17 at 11:00, wrote: > > On 22/02/17 09:23, Jan Beulich

[Xen-devel] [PATCH] tools/libxl: correct distclean target

2017-03-14 Thread Juergen Gross
Commit 3e5f1a63b53920763 ("tools: adapt xenlight.pc and xlutil.pc to new pkg-config scheme") introduced an error for "make distclean" as *.pc.in are deleted which are now files in git. Signed-off-by: Juergen Gross --- tools/libxl/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/

[Xen-devel] [PATCH] tools: correct build in directory below tools

2017-03-14 Thread Juergen Gross
Recent changes to create *.pc files introduced a bug when trying to build a library from a directory below tools as PKG_CONFIG_DIR wouldn't be set. Correct this by adding a default value to Rules.mk. Signed-off-by: Juergen Gross --- tools/Rules.mk | 2 ++ 1 file changed, 2 insertions(+) diff --

Re: [Xen-devel] [PATCH] EFI: retrieve and expose Apple device properties

2017-03-14 Thread Andrew Cooper
On 14/03/17 14:42, Jan Beulich wrote: > Apple's EFI drivers supply device properties which are needed to > support Macs optimally. They contain vital information which cannot be > obtained any other way (e.g. Thunderbolt Device ROM). They're also used > to convey the current device state so that OS

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

2017-03-14 Thread osstest service owner
flight 106648 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/106648/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-debianhvm-amd64 15 guest-localmigrate/x10 fail in 106641 REGR. vs. 1065

  1   2   3   >