Re: [Xen-devel] [PATCH for-next v3 01/22] x86/traps: move privilege instruction emulation code

2017-05-30 Thread Jan Beulich
>>> On 30.05.17 at 19:30, wrote: > On 30/05/17 18:27, Wei Liu wrote: >> On Mon, May 29, 2017 at 09:14:07AM -0600, Jan Beulich wrote: >> On 18.05.17 at 19:28, wrote: From 58df816b937dc7a3598de01f053a6030e631057e Mon Sep 17 00:00:00 2001

Re: [Xen-devel] Schedule is live for the Xen Project’s Annual Conference - Discounted Registration closes by end of May

2017-05-30 Thread Dietmar Hahn
Hi Lars, ich habe eine Frage bzgl. der Online-Registrierung. Damit meine Firma die Kosten übernimmt, muss auf der Rechnung letztendlich die Firmenadresse stehen. Gibt es am Ende der Registrierung eine Rechnung die auf die Firma lautet? Danke! Dietmar. Am Donnerstag, 25. Mai 2017, 16:09:40

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

2017-05-30 Thread osstest service owner
flight 109877 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/109877/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 4fc8277133fb011d028b4e0a42444ab6f552d0b9 baseline version: ovmf

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

2017-05-30 Thread osstest service owner
flight 109851 linux-4.9 real [real] http://logs.test-lab.xenproject.org/osstest/logs/109851/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-credit2 6 xen-boot fail REGR. vs. 107358

Re: [Xen-devel] [PATCH v11 04/23] x86: refactor psr: L3 CAT: implement main data structures, CPU init and free flows.

2017-05-30 Thread Yi Sun
On 17-05-30 07:05:18, Jan Beulich wrote: > >>> On 03.05.17 at 10:44, wrote: > > +/* > > + * PSR features are managed per socket. Below structure defines the members > > + * used to manage these features. > > + * ref_lock - A lock to protect cos_ref. > > + * features -

Re: [Xen-devel] Proposal to allow setting up shared memory areas between VMs from xl config file

2017-05-30 Thread Zhongze Liu
Hi Wei Liu, Thanks for your review and approval. Then I'll revise my proposal according to these conclusions. Cheers, Zhongze Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH] xen: avoid type warning in xchg_xen_ulong

2017-05-30 Thread Stefano Stabellini
On Mon, 29 May 2017, Arnd Bergmann wrote: > The improved type-checking version of container_of() triggers a warning for > xchg_xen_ulong, pointing out that 'xen_ulong_t' is unsigned, but atomic64_t > contains a signed value: > > drivers/xen/events/events_2l.c: In function

Re: [Xen-devel] [PATCH v10 26/32] ARM: vITS: handle MOVI command

2017-05-30 Thread Stefano Stabellini
On Fri, 26 May 2017, Andre Przywara wrote: > The MOVI command moves the interrupt affinity from one redistributor > (read: VCPU) to another. > For now migration of "live" LPIs is not yet implemented, but we store > the changed affinity in our virtual ITTE and the pending_irq. > > Signed-off-by:

Re: [Xen-devel] [PATCH v10 28/32] ARM: vITS: handle INV command

2017-05-30 Thread Stefano Stabellini
On Fri, 26 May 2017, Andre Przywara wrote: > The INV command instructs the ITS to update the configuration data for > a given LPI by re-reading its entry from the property table. > We don't need to care so much about the priority value, but enabling > or disabling an LPI has some effect: We remove

[Xen-devel] [libvirt test] 109846: regressions - FAIL

2017-05-30 Thread osstest service owner
flight 109846 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/109846/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt15 guest-start/debian.repeat fail REGR. vs. 109793 Tests which did not

Re: [Xen-devel] [PATCH v10 11/32] ARM: GICv3: forward pending LPIs to guests

2017-05-30 Thread Stefano Stabellini
On Tue, 30 May 2017, Julien Grall wrote: > Hi Andre, > > On 26/05/17 18:35, Andre Przywara wrote: > > Upon receiving an LPI on the host, we need to find the right VCPU and > > virtual IRQ number to get this IRQ injected. > > Iterate our two-level LPI table to find the domain ID and the virtual >

Re: [Xen-devel] [PATCH v10 03/32] ARM: vGIC: move irq_to_pending() calls under the VGIC VCPU lock

2017-05-30 Thread Stefano Stabellini
On Tue, 30 May 2017, Julien Grall wrote: > Hi Andre, > > On 26/05/17 18:35, Andre Przywara wrote: > > So far irq_to_pending() is just a convenience function to lookup > > statically allocated arrays. This will change with LPIs, which are > > more dynamic, so the memory for their struct

Re: [Xen-devel] [PATCH v10 01/32] ARM: vGIC: avoid rank lock when reading priority

2017-05-30 Thread Stefano Stabellini
On Tue, 30 May 2017, Julien Grall wrote: > Hi Andre, > > On 26/05/17 18:35, Andre Przywara wrote: > > When reading the priority value of a virtual interrupt, we were taking > > the respective rank lock so far. > > However for forwarded interrupts (Dom0 only so far) this may lead to a > > deadlock

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

2017-05-30 Thread osstest service owner
flight 109845 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/109845/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-win7-amd64 15 guest-localmigrate/x10 fail REGR. vs. 106776

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

2017-05-30 Thread osstest service owner
flight 109864 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/109864/ 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 test-amd64-amd64-libvirt 12

Re: [Xen-devel] [PATCH v2] xen: don't print error message in case of missing Xenstore entry

2017-05-30 Thread Boris Ostrovsky
On 05/30/2017 02:52 PM, Juergen Gross wrote: > When registering for the Xenstore watch of the node control/sysrq the > handler will be called at once. Don't issue an error message if the > Xenstore node isn't there, as it will be created only when an event > is being triggered. > > Signed-off-by:

[Xen-devel] [xen-unstable test] 109841: tolerable FAIL

2017-05-30 Thread osstest service owner
flight 109841 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/109841/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-arm64-arm64-xl-credit2 6 xen-boot fail in 109828 pass in 109841 test-armhf-armhf-libvirt 5

[Xen-devel] [PATCH v2] xen: don't print error message in case of missing Xenstore entry

2017-05-30 Thread Juergen Gross
When registering for the Xenstore watch of the node control/sysrq the handler will be called at once. Don't issue an error message if the Xenstore node isn't there, as it will be created only when an event is being triggered. Signed-off-by: Juergen Gross ---

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

2017-05-30 Thread osstest service owner
flight 109860 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/109860/ 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 test-amd64-amd64-libvirt 12

Re: [Xen-devel] [PATCH for-next v3 13/22] x86/traps: move toggle_guest_mode

2017-05-30 Thread Andrew Cooper
On 29/05/17 17:05, Jan Beulich wrote: On 18.05.17 at 19:09, wrote: >> Move from x86_64/traps.c to pv/traps.c. > This again doesn't necessarily fit into traps.c (but it's an option). > Perhaps we want some misc.c or util.c? Or pv/domain.c, as this does logically fit with

Re: [Xen-devel] xen-evtchn: Bind dyn evtchn:qemu-dm interrupt to next online CPU

2017-05-30 Thread Boris Ostrovsky
On 05/30/2017 11:17 AM, Anoob Soman wrote: > On 16/05/17 20:02, Boris Ostrovsky wrote: > > Hi Boris, > > Sorry for the delay, I was out traveling. > >>> rc = evtchn_bind_to_user(u, bind_interdomain.local_port); >>> -if (rc == 0) >>> +if (rc == 0) { >>> rc =

Re: [Xen-devel] xl create failure on arm64 with XEN 4.9rc6

2017-05-30 Thread Feng Kan
Yap, worked like a charm. Much thanks. Tested on Centos 7 arm64 system with 4.12rc2 64K page size kernel (dom0 and domU) & Xen 4.9rc6 Tested-by: Feng Kan On Sun, May 28, 2017 at 10:12 AM, Julien Grall wrote: > Hi, > > On 05/26/2017 11:22 PM, Feng Kan wrote:

Re: [Xen-devel] [PATCH for-next v3 06/22] x86/traps: move PV hypercall handlers to pv/traps.c

2017-05-30 Thread Andrew Cooper
On 29/05/17 16:40, Jan Beulich wrote: On 18.05.17 at 19:09, wrote: >> The following handlers are moved: >> 1. do_set_trap_table > This one makes sense to move to pv/traps.c, but ... > >> 2. do_set_debugreg >> 3. do_get_debugreg >> 4. do_fpu_taskswitch > ... none of these

Re: [Xen-devel] Notes on stubdoms and latency on ARM

2017-05-30 Thread Julien Grall
On 30/05/17 18:29, Stefano Stabellini wrote: On Fri, 26 May 2017, Volodymyr Babchuk wrote: The other issue with stubdoms is context switch times. Volodymyr showed that minios has much higher context switch times compared to EL0 apps. It is probably due to GIC context switch, that is skipped

Re: [Xen-devel] [PATCH for-next v3 01/22] x86/traps: move privilege instruction emulation code

2017-05-30 Thread Andrew Cooper
On 30/05/17 18:27, Wei Liu wrote: > On Mon, May 29, 2017 at 09:14:07AM -0600, Jan Beulich wrote: > On 18.05.17 at 19:28, wrote: >>> From 58df816b937dc7a3598de01f053a6030e631057e Mon Sep 17 00:00:00 2001 >>> From: Wei Liu >>> Date: Thu, 18 May 2017

Re: [Xen-devel] Notes on stubdoms and latency on ARM

2017-05-30 Thread Stefano Stabellini
On Fri, 26 May 2017, Volodymyr Babchuk wrote: > >> > The other issue with stubdoms is context switch times. Volodymyr showed > >> > that minios has much higher context switch times compared to EL0 apps. > >> > It is probably due to GIC context switch, that is skipped for EL0 apps. > >> > Maybe we

Re: [Xen-devel] [PATCH for-next v3 01/22] x86/traps: move privilege instruction emulation code

2017-05-30 Thread Wei Liu
On Mon, May 29, 2017 at 09:14:07AM -0600, Jan Beulich wrote: > >>> On 18.05.17 at 19:28, wrote: > > From 58df816b937dc7a3598de01f053a6030e631057e Mon Sep 17 00:00:00 2001 > > From: Wei Liu > > Date: Thu, 18 May 2017 16:18:56 +0100 > > Subject: [PATCH]

Re: [Xen-devel] [PATCH] xen: don't print error message in case of missing Xenstore entry

2017-05-30 Thread Juergen Gross
On 30/05/17 19:08, Boris Ostrovsky wrote: > On 05/30/2017 11:03 AM, Juergen Gross wrote: >> On 30/05/17 15:25, Boris Ostrovsky wrote: >>> On 05/29/2017 05:13 AM, Juergen Gross wrote: When registering for the Xenstore watch of the node control/sysrq the handler will be called at once.

Re: [Xen-devel] [PATCH] xen: don't print error message in case of missing Xenstore entry

2017-05-30 Thread Boris Ostrovsky
On 05/30/2017 11:03 AM, Juergen Gross wrote: > On 30/05/17 15:25, Boris Ostrovsky wrote: >> On 05/29/2017 05:13 AM, Juergen Gross wrote: >>> When registering for the Xenstore watch of the node control/sysrq the >>> handler will be called at once. Don't issue an error message if the >>> Xenstore

Re: [Xen-devel] Commit moratorium for branching Xen 4.9

2017-05-30 Thread Julien Grall
On 25/05/17 16:54, Julien Grall wrote: Hi, The patches fixing the 2 outstanding regressions have been merged. I would like to make another attempt to branch with the RC (4.9 RC7). I don't want to branch when master != staging, so please avoid committing new patches to staging now to let

Re: [Xen-devel] [PATCH 2/2] xen/input: add multi-touch support

2017-05-30 Thread Dmitry Torokhov
On Tue, May 30, 2017 at 03:50:20PM +0300, Oleksandr Andrushchenko wrote: > Hi, Dmitry! > > On 05/30/2017 08:51 AM, Dmitry Torokhov wrote: > >On Fri, Apr 21, 2017 at 09:40:36AM +0300, Oleksandr Andrushchenko wrote: > >>Hi, Dmitry! > >> > >>On 04/21/2017 05:10 AM, Dmitry Torokhov wrote: > >>>Hi

[Xen-devel] Xen 4.9 branched, staging open for new patches

2017-05-30 Thread Julien Grall
Hi all We just finished branching 4.9. The staging branch is now open for committing new patches. As always please be cautious about what patches are accepted into staging until 4.9 is released. For bug fixes please commit them to staging first then backport them to staging-4.9. Thanks, --

Re: [Xen-devel] Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])

2017-05-30 Thread Ian Jackson
Julien Grall writes ("Re: Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])"): > On 30/05/17 17:13, Ian Jackson wrote: ... > > I see. Can you do that please ? It's blocking moving our testing to > > a non-ancient

[Xen-devel] [PATCH] xen: stop_machine: fill fn_result only in case of error.

2017-05-30 Thread gregory . herrero
From: Gregory Herrero Since fn_result member is shared across all cpus, it must be filled only if an error happens. Assume CPU1 detects an error and set fn_result to -1, then CPU2 doesn't detect an error and set fn_result to 0. The error detected by CPU1 will be

Re: [Xen-devel] Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])

2017-05-30 Thread Julien Grall
Hi Ian, On 30/05/17 17:13, Ian Jackson wrote: Julien Grall writes ("Re: Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])"): There are two missing patches in Linux 4.9 to be able to boot on the Arndale: -

Re: [Xen-devel] Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])

2017-05-30 Thread Ian Jackson
Julien Grall writes ("Re: Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])"): > There are two missing patches in Linux 4.9 to be able to boot on the > Arndale: > - c827283586a4 "ARM: 8636/1: Cleanup

Re: [Xen-devel] Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])

2017-05-30 Thread Julien Grall
Hi Ian, On 30/05/17 16:47, Ian Jackson wrote: Boris Ostrovsky writes ("Re: Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])"): On 05/30/2017 10:28 AM, Ian Jackson wrote: osstest service owner writes ("[osstest test]

Re: [Xen-devel] Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])

2017-05-30 Thread Ian Jackson
Boris Ostrovsky writes ("Re: Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])"): > On 05/30/2017 10:28 AM, Ian Jackson wrote: > > osstest service owner writes ("[osstest test] 109837: regressions - FAIL"): > >> Tests

Re: [Xen-devel] [RFC PATCH V2 11/26] x86/hvm: Introduce a emulated VTD for HVM

2017-05-30 Thread Jan Beulich
>>> On 30.05.17 at 17:36, wrote: > On Thu, May 18, 2017 at 01:34:41AM -0400, Lan Tianyu wrote: >> --- a/xen/arch/x86/hvm/Makefile >> +++ b/xen/arch/x86/hvm/Makefile >> @@ -22,6 +22,7 @@ obj-y += rtc.o >> obj-y += save.o >> obj-y += stdvga.o >> obj-y += vioapic.o >> +obj-y

Re: [Xen-devel] staging aarch64 build fails in xc_dom_arm.c

2017-05-30 Thread Andrew Cooper
On 30/05/17 16:40, Olaf Hering wrote: > On Tue, May 30, Andrew Cooper wrote: > >> Does >> +domctl.u.address_size.size = 0; >> help? > Likely yes. I just scanned the buildlogs and found this failure. > I do not have a ARM toolchain around to double check. > Not sure why the marco, or its usage,

Re: [Xen-devel] [RFC PATCH V2 1/26] VIOMMU: Add vIOMMU helper functions to create, destroy and query capabilities

2017-05-30 Thread Jan Beulich
>>> On 30.05.17 at 17:36, wrote: > On Thu, May 18, 2017 at 01:34:31AM -0400, Lan Tianyu wrote: >> --- a/xen/common/Kconfig >> +++ b/xen/common/Kconfig >> @@ -73,6 +73,17 @@ config TMEM >> >>If unsure, say Y. >> >> +config VIOMMU >> +def_bool y >> +prompt

Re: [Xen-devel] staging aarch64 build fails in xc_dom_arm.c

2017-05-30 Thread Olaf Hering
On Tue, May 30, Andrew Cooper wrote: > Does > +domctl.u.address_size.size = 0; > help? Likely yes. I just scanned the buildlogs and found this failure. I do not have a ARM toolchain around to double check. Not sure why the marco, or its usage, is not like 'struct xen_domctl domctl = {}'

Re: [Xen-devel] [RFC PATCH V2 4/26] VIOMMU: Add get irq info callback to convert irq remapping request

2017-05-30 Thread Wei Liu
On Thu, May 18, 2017 at 01:34:34AM -0400, Lan Tianyu wrote: > This patch is to add get_irq_info callback for platform implementation > to convert irq remapping request to irq info (E,G vector, dest, dest_mode > and so on). > > Signed-off-by: Lan Tianyu > --- >

Re: [Xen-devel] [RFC PATCH V2 2/26] DOMCTL: Introduce new DOMCTL commands for vIOMMU support

2017-05-30 Thread Wei Liu
On Thu, May 18, 2017 at 01:34:32AM -0400, Lan Tianyu wrote: > This patch is to introduce create, destroy and query capabilities > command for vIOMMU. vIOMMU layer will deal with requests and call > arch vIOMMU ops. > > Signed-off-by: Lan Tianyu > --- > xen/common/domctl.c

Re: [Xen-devel] [RFC PATCH V2 10/26] libxl: create vIOMMU during domain construction

2017-05-30 Thread Wei Liu
On Thu, May 18, 2017 at 01:34:40AM -0400, Lan Tianyu wrote: > From: Chao Gao > > If guest is configured to have a vIOMMU, create it during domain construction. > > Signed-off-by: Chao Gao > Signed-off-by: Lan Tianyu > --- >

Re: [Xen-devel] [RFC PATCH V2 12/26] X86/vvtd: Add MMIO handler for VVTD

2017-05-30 Thread Wei Liu
On Thu, May 18, 2017 at 01:34:42AM -0400, Lan Tianyu wrote: > From: Chao Gao > > This patch adds VVTD MMIO handler to deal with MMIO access. > > Signed-off-by: Chao Gao > Signed-off-by: Lan Tianyu > --- > xen/arch/x86/hvm/vvtd.c |

Re: [Xen-devel] [RFC PATCH V2 21/26] Tools/libxc: Add a new interface to bind remapping format msi with pirq

2017-05-30 Thread Wei Liu
On Thu, May 18, 2017 at 01:34:51AM -0400, Lan Tianyu wrote: > From: Chao Gao > > Introduce a new binding relationship and provide a new interface to > manage the new relationship. > > Signed-off-by: Chao Gao > Signed-off-by: Lan Tianyu

Re: [Xen-devel] [RFC PATCH V2 11/26] x86/hvm: Introduce a emulated VTD for HVM

2017-05-30 Thread Wei Liu
On Thu, May 18, 2017 at 01:34:41AM -0400, Lan Tianyu wrote: > From: Chao Gao > > This patch adds create/destroy/query function for the emulated VTD > and adapts it to the common VIOMMU abstraction. > > Signed-off-by: Chao Gao > Signed-off-by: Lan Tianyu

Re: [Xen-devel] [RFC PATCH V2 6/26] Tools/libxc: Add viommu operations in libxc

2017-05-30 Thread Wei Liu
On Thu, May 18, 2017 at 01:34:36AM -0400, Lan Tianyu wrote: > From: Chao Gao > > This patch is to add XEN_DOMCTL_viommu_op hypercall. This hypercall > comprise three sub-command: > - query capabilities of one specific type vIOMMU emulated by Xen > - create vIOMMU in Xen

Re: [Xen-devel] [RFC PATCH V2 3/26] VIOMMU: Add irq request callback to deal with irq remapping

2017-05-30 Thread Wei Liu
On Thu, May 18, 2017 at 01:34:33AM -0400, Lan Tianyu wrote: > This patch is to add irq request callback for platform implementation > to deal with irq remapping request. > > Signed-off-by: Lan Tianyu > --- > xen/common/viommu.c | 15 + >

Re: [Xen-devel] [PATCH v11 12/23] x86: refactor psr: L3 CAT: set value: implement write msr flow.

2017-05-30 Thread Jan Beulich
>>> On 03.05.17 at 10:44, wrote: > +struct cos_write_info > +{ > +unsigned int cos; > +struct feat_node *feature; > +uint32_t *val; > +enum psr_feat_type feat_type; > +}; > + > +static void do_write_psr_msrs(void *data) > +{ > +struct cos_write_info

Re: [Xen-devel] [RFC PATCH V2 1/26] VIOMMU: Add vIOMMU helper functions to create, destroy and query capabilities

2017-05-30 Thread Wei Liu
On Thu, May 18, 2017 at 01:34:31AM -0400, Lan Tianyu wrote: > This patch is to introduct an abstract layer for arch vIOMMU implementation > to deal with requests from dom0. Arch vIOMMU code needs to provide callback > to perform create, destroy and query capabilities operation. > > Signed-off-by:

Re: [Xen-devel] staging aarch64 build fails in xc_dom_arm.c

2017-05-30 Thread Andrew Cooper
On 30/05/17 16:27, Olaf Hering wrote: > With gcc7 the aarch64 build in staging fails: > > xc_dom_arm.c: In function 'meminit': > xc_dom_arm.c:229:31: error: 'domctl.u.address_size.size' may be used > uninitialized in this function [-Werror=maybe-uninitialized] > if (

[Xen-devel] staging aarch64 build fails in xc_dom_arm.c

2017-05-30 Thread Olaf Hering
With gcc7 the aarch64 build in staging fails: xc_dom_arm.c: In function 'meminit': xc_dom_arm.c:229:31: error: 'domctl.u.address_size.size' may be used uninitialized in this function [-Werror=maybe-uninitialized] if ( domctl.u.address_size.size == 0 )

Re: [Xen-devel] [PATCH v11 10/23] x86: refactor psr: L3 CAT: set value: implement cos finding flow.

2017-05-30 Thread Jan Beulich
>>> On 03.05.17 at 10:44, wrote: > static int find_cos(const uint32_t val[], unsigned int array_len, > enum psr_feat_type feat_type, > const struct psr_socket_info *info) > { > +unsigned int cos, i; > +const unsigned

Re: [Xen-devel] xen-evtchn: Bind dyn evtchn:qemu-dm interrupt to next online CPU

2017-05-30 Thread Anoob Soman
On 16/05/17 20:02, Boris Ostrovsky wrote: Hi Boris, Sorry for the delay, I was out traveling. rc = evtchn_bind_to_user(u, bind_interdomain.local_port); -if (rc == 0) +if (rc == 0) { rc = bind_interdomain.local_port; +selected_cpu =

Re: [Xen-devel] [PATCH v11 09/23] x86: refactor psr: L3 CAT: set value: assemble features value array.

2017-05-30 Thread Jan Beulich
>>> On 03.05.17 at 10:44, wrote: > Only can one COS ID be used by one domain at one time. That means all enabled > features' COS registers at this COS ID are valid for this domain at that > time. > > When user updates a feature's value, we need make sure all other

Re: [Xen-devel] [for-4.9] Re: HVM guest performance regression

2017-05-30 Thread Jan Beulich
>>> On 30.05.17 at 16:57, wrote: > On 30/05/17 12:43, Jan Beulich wrote: > On 30.05.17 at 12:33, wrote: >>> On 30/05/17 09:24, Jan Beulich wrote: But if the OS allocated large pages internally for relevant data structures, those obviously won't

Re: [Xen-devel] [PATCH] xen: don't print error message in case of missing Xenstore entry

2017-05-30 Thread Juergen Gross
On 30/05/17 15:25, Boris Ostrovsky wrote: > On 05/29/2017 05:13 AM, Juergen Gross wrote: >> When registering for the Xenstore watch of the node control/sysrq the >> handler will be called at once. Don't issue an error message if the >> Xenstore node isn't there, as it will be created only when an

Re: [Xen-devel] [for-4.9] Re: HVM guest performance regression

2017-05-30 Thread Juergen Gross
On 30/05/17 12:43, Jan Beulich wrote: On 30.05.17 at 12:33, wrote: >> On 30/05/17 09:24, Jan Beulich wrote: >> On 29.05.17 at 21:05, wrote: Creating the domains with xl -vvv create ... showed the numbers of superpages and normal

Re: [Xen-devel] Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])

2017-05-30 Thread Boris Ostrovsky
On 05/30/2017 10:28 AM, Ian Jackson wrote: > Ian Jackson writes ("[OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by > default"): >> I ran a special report[1] to see what to expect and: > ... > > It seems I ran or read this wrong. In fact, my change is stuck in the > osstest self push gate

Re: [Xen-devel] [PATCH v2] x86/vmx: Fix vmentry failure because of invalid LER on Broadwell

2017-05-30 Thread Jan Beulich
>>> On 30.05.17 at 16:05, wrote: > --- a/xen/include/asm-x86/x86_64/page.h > +++ b/xen/include/asm-x86/x86_64/page.h > @@ -28,6 +28,9 @@ > #define PADDR_MASK ((1UL << PADDR_BITS)-1) > #define VADDR_MASK ((1UL << VADDR_BITS)-1) > > +#define

Re: [Xen-devel] [PATCH] tools/firmware: pass EXTRAVERSION to seabios build

2017-05-30 Thread Olaf Hering
On Tue, May 30, Wei Liu wrote: > subdirs-seabios-dir: EXTRAVERSION=XXX > Limit it to the one that needs the environment variable -- seabios or > ipxe. Ok, I will try it. Last time I looked environment did not work. Olaf signature.asc Description: PGP signature

Re: [Xen-devel] WARNING: CPU: 0 PID: 0 at net/sched/sch_generic.c:316 dev_watchdog+0x217/0x220

2017-05-30 Thread Steven Haigh
On 31/05/17 00:18, Boris Ostrovsky wrote: > On 05/30/2017 06:27 AM, Steven Haigh wrote: >> Just wanted to give this a nudge to try and get some suggestions on >> where to go / what to do about this. >> >> On 28/05/17 09:44, Steven Haigh wrote: >>> The last couple of days running on kernel 4.9.29

Re: [Xen-devel] [PATCH] tools/firmware: pass EXTRAVERSION to seabios build

2017-05-30 Thread Wei Liu
On Tue, May 30, 2017 at 04:24:18PM +0200, Olaf Hering wrote: > On Tue, May 30, Wei Liu wrote: > > > In that case, can you confine such hackery to be seabios only? > > Is it worth the hassle? It seems only ipxe would recognize the > EXTRAVERSION. And how would I actually limit it to seabios?

Re: [Xen-devel] [PATCH v11 08/23] x86: refactor psr: L3 CAT: set value: implement framework.

2017-05-30 Thread Jan Beulich
>>> On 03.05.17 at 10:44, wrote: > --- a/xen/arch/x86/psr.c > +++ b/xen/arch/x86/psr.c > @@ -118,11 +118,13 @@ static const struct feat_props { > * COS ID. Every entry of cos_ref corresponds to one COS ID. > */ > struct psr_socket_info { > -bool

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

2017-05-30 Thread osstest service owner
flight 109855 xtf real [real] http://logs.test-lab.xenproject.org/osstest/logs/109855/ Perfect :-) All tests in this flight passed as required version targeted for testing: xtf d1a237abf651f181e6b11fbd7358abf07684c7c8 baseline version: xtf

Re: [Xen-devel] [PATCH] xen: preserve native TSC speed during migration between identical hosts

2017-05-30 Thread Konrad Rzeszutek Wilk
On Tue, May 30, 2017 at 11:27:05AM +0200, Olaf Hering wrote: > On Wed, May 24, Konrad Rzeszutek Wilk wrote: > > > Is there some form of tests that they can run to verify and test > > that this is safe? Or perhaps this is something that is based on the kernel > > versions? Like 4.11 are safe, but

[Xen-devel] Nested virt broken in Linux 4.9 (was Re: [OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default [and 1 more messages])

2017-05-30 Thread Ian Jackson
Ian Jackson writes ("[OSSTEST PATCH 2/3] ap-common: Switch to Linux 4.9 by default"): > I ran a special report[1] to see what to expect and: ... It seems I ran or read this wrong. In fact, my change is stuck in the osstest self push gate because: osstest service owner writes ("[osstest test]

Re: [Xen-devel] [PATCH] tools/firmware: pass EXTRAVERSION to seabios build

2017-05-30 Thread Olaf Hering
On Tue, May 30, Wei Liu wrote: > In that case, can you confine such hackery to be seabios only? Is it worth the hassle? It seems only ipxe would recognize the EXTRAVERSION. And how would I actually limit it to seabios? Something like "make $(filter-out subdir-seabios-dir, subdirs-$@)"? Olaf

Re: [Xen-devel] [PATCH v2] x86/vmx: Fix vmentry failure because of invalid LER on Broadwell

2017-05-30 Thread Andrew Cooper
On 30/05/17 15:05, Ross Lagerwall wrote: > Occasionally, on certain Broadwell CPUs MSR_IA32_LASTINTTOIP has been > observed to have the top three bits corrupted as though the MSR is using > the LBR_FORMAT_EIP_FLAGS_TSX format. This is incorrect and causes a > vmentry failure -- the MSR should

Re: [Xen-devel] [PATCH 1/1] xl man page cleanup and fixes

2017-05-30 Thread Julien Grall
Hello, Your Signed-off-by is missing in the commit message. Cheers, On 29/05/17 19:01, Armando Vega wrote: --- docs/man/xl.pod.1.in | 362 +-- 1 file changed, 179 insertions(+), 183 deletions(-) diff --git a/docs/man/xl.pod.1.in

Re: [Xen-devel] WARNING: CPU: 0 PID: 0 at net/sched/sch_generic.c:316 dev_watchdog+0x217/0x220

2017-05-30 Thread Boris Ostrovsky
On 05/30/2017 06:27 AM, Steven Haigh wrote: > Just wanted to give this a nudge to try and get some suggestions on > where to go / what to do about this. > > On 28/05/17 09:44, Steven Haigh wrote: >> The last couple of days running on kernel 4.9.29 and 4.9.30 with Xen >> 4.9.0-rc6 I've had a number

Re: [Xen-devel] [PATCH 0/1] xl man page cleanup and fixes proposal

2017-05-30 Thread Ian Jackson
Armando Vega writes ("[PATCH 0/1] xl man page cleanup and fixes proposal"): > If you accept this patch I would gladly donate my time to fixing the > rest of the man pages, as I did notice the same problems in some of > them. I'd love more patches like that one. > I have a personal fork of this

Re: [Xen-devel] [PATCH 1/1] xl man page cleanup and fixes

2017-05-30 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 1/1] xl man page cleanup and fixes"): > Hi Thanks for the patch. > > I welcome the improvement to our documentation. > > FAOD I will leave this patch to Ian because he's a native English > speaker. Thanks. I read the first half of the patch and it was good. Good

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

2017-05-30 Thread osstest service owner
flight 109839 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/109839/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt 7 host-ping-check-xen fail REGR. vs. 109656

[Xen-devel] [PATCH v2] x86/vmx: Fix vmentry failure because of invalid LER on Broadwell

2017-05-30 Thread Ross Lagerwall
Occasionally, on certain Broadwell CPUs MSR_IA32_LASTINTTOIP has been observed to have the top three bits corrupted as though the MSR is using the LBR_FORMAT_EIP_FLAGS_TSX format. This is incorrect and causes a vmentry failure -- the MSR should contain an offset into the current code segment. This

Re: [Xen-devel] [PATCH v11 07/23] x86: refactor psr: L3 CAT: implement get value flow.

2017-05-30 Thread Jan Beulich
>>> On 03.05.17 at 10:44, wrote: > --- a/xen/arch/x86/psr.c > +++ b/xen/arch/x86/psr.c > @@ -476,23 +476,34 @@ static struct psr_socket_info *get_socket_info(unsigned > int socket) > return socket_info + socket; > } > > +static struct feat_node

Re: [Xen-devel] [PATCH v11 06/23] x86: refactor psr: L3 CAT: implement get hw info flow.

2017-05-30 Thread Jan Beulich
>>> On 03.05.17 at 10:44, wrote: > +int psr_get_info(unsigned int socket, enum cbm_type type, > + uint32_t data[], unsigned int array_len) > +{ > +const struct psr_socket_info *info = get_socket_info(socket); > +const struct feat_node *feat; > +

Re: [Xen-devel] [PATCH v11 05/23] x86: refactor psr: L3 CAT: implement Domain init/free and schedule flows.

2017-05-30 Thread Jan Beulich
>>> On 03.05.17 at 10:44, wrote: > +static unsigned int get_max_cos_max(const struct psr_socket_info *info) > +{ > +unsigned int cos_max = 0, i; > + > +for ( i = 0; i < PSR_SOCKET_FEAT_NUM; i++ ) > +{ > +const struct feat_node *feat =

Re: [Xen-devel] [PATCH] xen: don't print error message in case of missing Xenstore entry

2017-05-30 Thread Boris Ostrovsky
On 05/29/2017 05:13 AM, Juergen Gross wrote: > When registering for the Xenstore watch of the node control/sysrq the > handler will be called at once. Don't issue an error message if the > Xenstore node isn't there, as it will be created only when an event > is being triggered. > > Signed-off-by:

Re: [Xen-devel] [PATCH v11 04/23] x86: refactor psr: L3 CAT: implement main data structures, CPU init and free flows.

2017-05-30 Thread Jan Beulich
>>> On 03.05.17 at 10:44, wrote: > +/* > + * PSR features are managed per socket. Below structure defines the members > + * used to manage these features. > + * ref_lock - A lock to protect cos_ref. > + * features - A feature node array used to manage all features

Re: [Xen-devel] [PATCH v10 16/32] ARM: vGIC: advertise LPI support

2017-05-30 Thread Julien Grall
Hi Andre, On 26/05/17 18:35, Andre Przywara wrote: To let a guest know about the availability of virtual LPIs, set the respective bits in the virtual GIC registers and let a guest control the LPI enable bit. Only report the LPI capability if the host has initialized at least one ITS. I am not

Re: [Xen-devel] [PATCH 2/2] xen/input: add multi-touch support

2017-05-30 Thread Oleksandr Andrushchenko
Hi, Dmitry! On 05/30/2017 08:51 AM, Dmitry Torokhov wrote: On Fri, Apr 21, 2017 at 09:40:36AM +0300, Oleksandr Andrushchenko wrote: Hi, Dmitry! On 04/21/2017 05:10 AM, Dmitry Torokhov wrote: Hi Oleksandr, On Thu, Apr 13, 2017 at 02:38:04PM +0300, Oleksandr Andrushchenko wrote: From:

Re: [Xen-devel] [PATCH] tools/firmware: pass EXTRAVERSION to seabios build

2017-05-30 Thread Wei Liu
On Tue, May 30, 2017 at 02:25:11PM +0200, Olaf Hering wrote: > On Tue, May 30, Wei Liu wrote: > > > What I meant was: this is passing EXTRAVERSION to all subdir targets, > > which seems unnecessary to me. And you already specified a version > > string for seabios. > > True, but

Re: [Xen-devel] [PATCH 1/2] xen/input: use string constants from PV protocol

2017-05-30 Thread Oleksandr Andrushchenko
Hi, Dmitry! On 04/21/2017 05:11 AM, Dmitry Torokhov wrote: On Thu, Apr 13, 2017 at 02:38:03PM +0300, Oleksandr Andrushchenko wrote: From: Oleksandr Andrushchenko Xen input para-virtual protocol defines string constants used by both back and frontend. Use

Re: [Xen-devel] Guidance for PVHv2 usage?

2017-05-30 Thread PGNet Dev
On 5/30/17 12:29 AM, Roger Pau Monné wrote: ... > dom0=pvh is what should be used once it's finished. Right now there's > no PVHv2 Dom0 support in Linux, and the Xen side is not finished. If > you manage to get this booting, you are going to hit a panic in Xen > code [0]. ...> I think

Re: [Xen-devel] [PATCH] tools/firmware: pass EXTRAVERSION to seabios build

2017-05-30 Thread Olaf Hering
On Tue, May 30, Wei Liu wrote: > What I meant was: this is passing EXTRAVERSION to all subdir targets, > which seems unnecessary to me. And you already specified a version > string for seabios. True, but scripts/buildversion.py insists on --extra 'whatever'. Olaf signature.asc Description:

Re: [Xen-devel] [PATCH 06/12 v3] xen/arm: vpl011: Add a new vuart node in the xenstore

2017-05-30 Thread Wei Liu
On Mon, May 22, 2017 at 04:33:15PM +0530, Bhupinder Thakur wrote: > Hi Wei, > > > > [...] > >> >> @@ -151,13 +154,19 @@ retry_transaction: > >> >> if (rc) goto out; > >> >> > >> >> if (!libxl_only) { > >> >> -rc = libxl__xs_write_checked(gc, t, > >> >>

Re: [Xen-devel] [PATCH v10 12/32] ARM: GICv3: enable ITS and LPIs on the host

2017-05-30 Thread Julien Grall
Hi Andre, On 26/05/17 18:35, Andre Przywara wrote: Now that the host part of the ITS code is in place, we can enable the ITS and also LPIs on each redistributor to get the show rolling. At this point there would be no LPIs mapped, as guests don't know about the ITS yet. In that case, why

Re: [Xen-devel] [PATCH v10 11/32] ARM: GICv3: forward pending LPIs to guests

2017-05-30 Thread Julien Grall
Hi Andre, On 26/05/17 18:35, Andre Przywara wrote: Upon receiving an LPI on the host, we need to find the right VCPU and virtual IRQ number to get this IRQ injected. Iterate our two-level LPI table to find the domain ID and the virtual LPI number quickly when the host takes an LPI. We then look

Re: [Xen-devel] [PATCH 05/12 v3] xen/arm: vpl011: Add new domctl APIs to initialize/de-initialize vpl011

2017-05-30 Thread Wei Liu
On Mon, May 22, 2017 at 04:05:44PM +0530, Bhupinder Thakur wrote: > Hi Wei, > > > >> --- a/tools/libxl/libxl_dom.c > >> +++ b/tools/libxl/libxl_dom.c > >> @@ -688,6 +688,15 @@ static int libxl__build_dom(libxl__gc *gc, uint32_t > >> domid, > >> goto out; > >> } > >> > >> +if (

Re: [Xen-devel] [PATCH] tools/firmware: pass EXTRAVERSION to seabios build

2017-05-30 Thread Wei Liu
On Tue, May 30, 2017 at 12:33:15PM +0100, Wei Liu wrote: > On Mon, May 29, 2017 at 09:57:58AM +0200, Olaf Hering wrote: > > On Fri, May 26, Ian Jackson wrote: > > > > > This seems like a real problem which should be improved. But maybe we > > > should use Xen's EXTRAVERSION by default ? > > > >

Re: [Xen-devel] [PATCH v10 06/32] ARM: GIC: Add checks for NULL pointer pending_irq's

2017-05-30 Thread Julien Grall
Hi Andre, On 26/05/17 18:35, Andre Przywara wrote: For LPIs the struct pending_irq's are dynamically allocated and the pointers will be stored in a radix tree. Since an LPI can be "unmapped" at any time, teach the VGIC how to deal with irq_to_pending() returning a NULL pointer. We just do

Re: [Xen-devel] [PATCH] tools/firmware: pass EXTRAVERSION to seabios build

2017-05-30 Thread Wei Liu
On Mon, May 29, 2017 at 09:57:58AM +0200, Olaf Hering wrote: > On Fri, May 26, Ian Jackson wrote: > > > This seems like a real problem which should be improved. But maybe we > > should use Xen's EXTRAVERSION by default ? > > After thinking about it, why does the tools build not just enforce a >

Re: [Xen-devel] [PATCH v10 05/32] ARM: vGIC: introduce gic_remove_irq()

2017-05-30 Thread Julien Grall
Hi Andre, On 26/05/17 18:35, Andre Przywara wrote: To avoid code duplication in a later patch, introduce a generic function to remove a virtual IRQ from the VGIC. Call that function instead of the open-coded version in vgic_migrate_irq(). Signed-off-by: Andre Przywara

Re: [Xen-devel] [PATCH v10 04/32] ARM: vGIC: rework gic_remove_from_queues()

2017-05-30 Thread Julien Grall
Hi Andre, On 26/05/17 18:35, Andre Przywara wrote: The function name gic_remove_from_queues() was a bit of a misnomer, since it just removes an IRQ from the pending queue, not both queues. Rename the function to make this more clear, also give it a pointer to a struct pending_irq directly and

Re: [Xen-devel] [PATCH v10 03/32] ARM: vGIC: move irq_to_pending() calls under the VGIC VCPU lock

2017-05-30 Thread Julien Grall
Hi Andre, On 26/05/17 18:35, Andre Przywara wrote: So far irq_to_pending() is just a convenience function to lookup statically allocated arrays. This will change with LPIs, which are more dynamic, so the memory for their struct pending_irq might go away. The proper answer to the issue of

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

2017-05-30 Thread Platform Team regression test user
flight 71457 distros-debian-snapshot real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/71457/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-i386-daily-netboot-pygrub 10 guest-start fail REGR. vs. 71408

Re: [Xen-devel] [PATCH v10 02/32] ARM: GICv3: setup number of LPI bits for a GICv3 guest

2017-05-30 Thread Julien Grall
Hi Andre, On 26/05/17 18:35, Andre Przywara wrote: The host supports a certain number of LPI identifiers, as stored in the GICD_TYPER register. Store this number from the hardware register in vgic_v3_hw to allow injecting the very same number into a guest (Dom0). DomUs get the legacy number of

  1   2   >