Re: [Xen-devel] [PATCH 1/7] SUPPORT.md: Syntax: Fix some bullet lists

2018-04-11 Thread George Dunlap
On Tue, Apr 10, 2018 at 6:22 PM, Ian Jackson wrote: > Continuations of bullet list items must be indented by exactly 4 > spaces (according to pandoc_markdown(5) on Debian jessie). > > This is most easily achieved by making the bullet list items have two > spaces before

Re: [Xen-devel] [PATCH 2/7] SUPPORT.md: Syntax: Fix a typo "States"

2018-04-11 Thread George Dunlap
On Tue, Apr 10, 2018 at 6:22 PM, Ian Jackson wrote: > Signed-off-by: Ian Jackson Acked-by: George Dunlap > --- > SUPPORT.md | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/SUPPORT.md

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Dario Faggioli
On Wed, 2018-04-11 at 12:00 +0200, Olaf Hering wrote: > On Wed, Apr 11, Dario Faggioli wrote: > > > Olaf, can you give it a try? It should be fine to run it on top of > > the > > last debug patch (the one that produced this crash). > > Yes, with both changes it did >4k iterations already.

Re: [Xen-devel] Weird altp2m behaviour when switching early to a new view

2018-04-11 Thread Razvan Cojocaru
> After much debugging, it turns out that the > "p2m_is_ram(p2mt)" test in hvm_hap_nested_page_fault() fails if I switch > to the new altp2m view fast enough, and that in turn disables the > logdirty processing gated on it Actually as it turns out the exit doesn't happen at all anymore so

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Dario Faggioli
On Wed, 2018-04-11 at 08:23 +0200, Olaf Hering wrote: > It turned out that I had a typo all the time in my template, it used > 'cpu=' rather than 'cpus='. On this system none of this works: > #pus="node:${node}" > cpus="nodes:${node}" > #pus="nodes:${node},^node:0" >

Re: [Xen-devel] [PATCH v4] x86/setup: disallow any src/dst overlaps when relocating Xen image

2018-04-11 Thread Jan Beulich
>>> On 11.04.18 at 10:54, wrote: > On Tue, Apr 03, 2018 at 05:54:14PM +0200, Daniel Kiper wrote: >> Commit 0d31d16 (x86/setup: do not relocate Xen over current Xen image >> placement) disallowed src/dst images overlaps when relocating Xen image. >> Though it deliberately

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

2018-04-11 Thread osstest service owner
flight 122171 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/122171/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 13 migrate-support-checkfail never pass test-arm64-arm64-xl-xsm

Re: [Xen-devel] [PATCH 3/7] SUPPORT.md: Syntax: Provide a title rather than a spurious empty section

2018-04-11 Thread George Dunlap
On Wed, Apr 11, 2018 at 11:37 AM, Ian Jackson wrote: > George Dunlap writes ("Re: [Xen-devel] [PATCH 3/7] SUPPORT.md: Syntax: > Provide a title rather than a spurious empty section"): >> On Tue, Apr 10, 2018 at 6:22 PM, Ian Jackson >> wrote:

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Juergen Gross
On 11/04/18 09:31, Dario Faggioli wrote: > On Wed, 2018-04-11 at 00:59 +0200, Dario Faggioli wrote: >> [Adding Andrew, not because I expect anything, but just because >> we've chatted about this issue on IRC :-) ] >> > Except, I did not add it. :-P > > Anyway... > >> On Tue, 2018-04-10 at 22:37

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

2018-04-11 Thread osstest service owner
flight 122173 xen-unstable-coverity real [real] http://logs.test-lab.xenproject.org/osstest/logs/122173/ Perfect :-) All tests in this flight passed as required version targeted for testing: xen 50f8ba84a50ebf80dd22067a04062dbaaf2621ff baseline version: xen

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Dario Faggioli
On Wed, 2018-04-11 at 10:48 +0200, Olaf Hering wrote: > On Wed, Apr 11, Dario Faggioli wrote: > > So, now, when you say 'does not work', do you mean 'domain creation > > is > > aborted with errors' or 'domain is created, but memory is not where > > it > > should be'. > > domU can not be created

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread George Dunlap
On 04/11/2018 11:17 AM, Dario Faggioli wrote: > On Wed, 2018-04-11 at 12:00 +0200, Olaf Hering wrote: >> On Wed, Apr 11, Dario Faggioli wrote: >> >>> Olaf, can you give it a try? It should be fine to run it on top of >>> the >>> last debug patch (the one that produced this crash). >> >> Yes, with

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Dario Faggioli
On Wed, 2018-04-11 at 11:37 +0100, George Dunlap wrote: > On 04/10/2018 11:59 PM, Dario Faggioli wrote: > > > > So, basically, the race is between context_saved() and > > vcpu_set_affinity(). Basically, vcpu_set_affinity() sets the > > VPF_migrating pause flags on a vcpu in a runqueue, with the

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Dario Faggioli
On Wed, 2018-04-11 at 00:59 +0200, Dario Faggioli wrote: > [Adding Andrew, not because I expect anything, but just because > we've chatted about this issue on IRC :-) ] > Except, I did not add it. :-P Anyway... > On Tue, 2018-04-10 at 22:37 +0200, Olaf Hering wrote: > > On Tue, Apr 10, Dario

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

2018-04-11 Thread osstest service owner
flight 122155 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/122155/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-amd64-xl-qcow2 15 guest-localmigrate fail in 122129 pass in 122155 test-armhf-armhf-xl-multivcpu

Re: [Xen-devel] [PATCH v4] x86/setup: disallow any src/dst overlaps when relocating Xen image

2018-04-11 Thread Daniel Kiper
On Tue, Apr 03, 2018 at 05:54:14PM +0200, Daniel Kiper wrote: > Commit 0d31d16 (x86/setup: do not relocate Xen over current Xen image > placement) disallowed src/dst images overlaps when relocating Xen image. > Though it deliberately allowed destination region between __image_base__ > and

Re: [Xen-devel] [PATCH] x86/xen: zero MSR_IA32_SPEC_CTRL before suspend

2018-04-11 Thread Jan Beulich
>>> On 11.04.18 at 09:08, wrote: > On 14/03/18 09:48, Jan Beulich wrote: > On 26.02.18 at 15:08, wrote: >>> @@ -35,6 +40,9 @@ void xen_arch_post_suspend(int cancelled) >>> >>> static void xen_vcpu_notify_restore(void *data) >>> { >>> + if

Re: [Xen-devel] [PATCH] x86/xen: zero MSR_IA32_SPEC_CTRL before suspend

2018-04-11 Thread Juergen Gross
On 14/03/18 09:48, Jan Beulich wrote: On 26.02.18 at 15:08, wrote: >> @@ -35,6 +40,9 @@ void xen_arch_post_suspend(int cancelled) >> >> static void xen_vcpu_notify_restore(void *data) >> { >> +if (xen_pv_domain() && boot_cpu_has(X86_FEATURE_SPEC_CTRL)) >> +

Re: [Xen-devel] [PATCH for-4.11] x86/VT-x: Fix determination of EFER.LMA in vmcs_dump_vcpu()

2018-04-11 Thread Andrew Cooper
On 11/04/2018 03:15, Tian, Kevin wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: Tuesday, April 10, 2018 4:44 PM >> > On 09.04.18 at 19:56, wrote: >>> --- a/xen/arch/x86/hvm/vmx/vmcs.c >>> +++ b/xen/arch/x86/hvm/vmx/vmcs.c >>> @@ -1788,7 +1788,10 @@

Re: [Xen-devel] [PATCH v2] x86/xen/efi: Initialize UEFI secure boot state during dom0 boot

2018-04-11 Thread Daniel Kiper
On Wed, Apr 04, 2018 at 12:38:24PM +0200, Daniel Kiper wrote: > On Tue, Apr 03, 2018 at 10:00:52AM -0700, James Bottomley wrote: > > On Tue, 2018-04-03 at 18:07 +0200, Daniel Kiper wrote: > > > On Tue, Apr 03, 2018 at 08:44:41AM -0700, James Bottomley wrote: > > [...] > > > > > This looks like a

Re: [Xen-devel] [PATCH for-4.11] x86/VT-x: Fix determination of EFER.LMA in vmcs_dump_vcpu()

2018-04-11 Thread Jan Beulich
>>> On 11.04.18 at 10:38, wrote: > On 11/04/2018 03:15, Tian, Kevin wrote: >>> From: Jan Beulich [mailto:jbeul...@suse.com] >>> Sent: Tuesday, April 10, 2018 4:44 PM >>> >> On 09.04.18 at 19:56, wrote: ---

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Dario Faggioli
On Wed, 2018-04-11 at 09:39 +0200, Juergen Gross wrote: > On 11/04/18 09:31, Dario Faggioli wrote: > > > On Tue, 2018-04-10 at 22:37 +0200, Olaf Hering wrote: > > > > On Tue, Apr 10, Dario Faggioli wrote: > > > > > > > > BUG_ON(__vcpu_on_runq(CSCHED_VCPU(vc))); > > > > > > > > ... patch

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Olaf Hering
On Wed, Apr 11, Dario Faggioli wrote: > So, now, when you say 'does not work', do you mean 'domain creation is > aborted with errors' or 'domain is created, but memory is not where it > should be'. domU can not be created due to "libxl__set_vcpuaffinity: setting vcpu affinity: Invalid argument".

Re: [Xen-devel] [PATCH v4] xen/privcmd: add IOCTL_PRIVCMD_MMAP_RESOURCE

2018-04-11 Thread Julien Grall
Hi, On 10/04/18 08:58, Paul Durrant wrote: +static long privcmd_ioctl_mmap_resource(struct file *file, void __user *udata) +{ + struct privcmd_data *data = file->private_data; + struct mm_struct *mm = current->mm; + struct vm_area_struct *vma; + struct

Re: [Xen-devel] [PATCH 3/7] SUPPORT.md: Syntax: Provide a title rather than a spurious empty section

2018-04-11 Thread George Dunlap
On Tue, Apr 10, 2018 at 6:22 PM, Ian Jackson wrote: > Signed-off-by: Ian Jackson > --- > SUPPORT.md | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/SUPPORT.md b/SUPPORT.md > index e447069..264b23f 100644 > ---

Re: [Xen-devel] [PATCH v4] x86/setup: disallow any src/dst overlaps when relocating Xen image

2018-04-11 Thread Daniel Kiper
On Wed, Apr 11, 2018 at 02:58:01AM -0600, Jan Beulich wrote: > >>> On 11.04.18 at 10:54, wrote: > > On Tue, Apr 03, 2018 at 05:54:14PM +0200, Daniel Kiper wrote: > >> Commit 0d31d16 (x86/setup: do not relocate Xen over current Xen image > >> placement) disallowed src/dst

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

2018-04-11 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 74574 qemu-mainline real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/74574/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64

Re: [Xen-devel] [PATCH v4] xen/privcmd: add IOCTL_PRIVCMD_MMAP_RESOURCE

2018-04-11 Thread Paul Durrant
> -Original Message- > From: Julien Grall [mailto:julien.gr...@arm.com] > Sent: 11 April 2018 10:46 > To: Paul Durrant ; xen-devel@lists.xenproject.org; > linux-arm-ker...@lists.infradead.org; linux-ker...@vger.kernel.org > Cc: Juergen Gross ;

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Olaf Hering
On Wed, Apr 11, Dario Faggioli wrote: > Olaf, can you give it a try? It should be fine to run it on top of the > last debug patch (the one that produced this crash). Yes, with both changes it did >4k iterations already. Thanks. Olaf signature.asc Description: PGP signature

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Dario Faggioli
On Wed, 2018-04-11 at 10:48 +0200, Olaf Hering wrote: > On Wed, Apr 11, Dario Faggioli wrote: > > So, now, when you say 'does not work', do you mean 'domain creation > > is > > aborted with errors' or 'domain is created, but memory is not where > > it > > should be'. > > domU can not be created

Re: [Xen-devel] [PATCH 1/7] SUPPORT.md: Syntax: Fix some bullet lists

2018-04-11 Thread Ian Jackson
George Dunlap writes ("Re: [Xen-devel] [PATCH 1/7] SUPPORT.md: Syntax: Fix some bullet lists"): > You forgot to CC "THE REST". Rather, I decided not to. There are no meaningful changes here, just build system and formatting syntax changes. Feel free to CC them if you think they will want to

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread George Dunlap
On 04/10/2018 11:59 PM, Dario Faggioli wrote: > [Adding Andrew, not because I expect anything, but just because we've > chatted about this issue on IRC :-) ] > > On Tue, 2018-04-10 at 22:37 +0200, Olaf Hering wrote: >> On Tue, Apr 10, Dario Faggioli wrote: >> >>

Re: [Xen-devel] [PATCH 3/7] SUPPORT.md: Syntax: Provide a title rather than a spurious empty section

2018-04-11 Thread Ian Jackson
George Dunlap writes ("Re: [Xen-devel] [PATCH 3/7] SUPPORT.md: Syntax: Provide a title rather than a spurious empty section"): > On Tue, Apr 10, 2018 at 6:22 PM, Ian Jackson > wrote: > > -# Support statement for this release > > +% Support statement for this release >

Re: [Xen-devel] [PATCH] x86/xen: zero MSR_IA32_SPEC_CTRL before suspend

2018-04-11 Thread Jan Beulich
>>> On 11.04.18 at 13:53, wrote: > * Jan Beulich wrote: > >> Additionally, x86 maintainers: is there a particular reason this (or >> any functionally equivalent patch) isn't upstream yet? As indicated >> before, I had not been able to find any discussion,

Re: [Xen-devel] Resume from suspend to RAM broken when using early microcode updates

2018-04-11 Thread Jan Beulich
>>> On 11.04.18 at 13:51, wrote: > On 11/04/18 12:48, Simon Gaiser wrote: >> Hi, >> >> when I use early microcode loading with the microcode update with the >> BTI mitigations, resuming from suspend to RAM is broken. >> >> Based on added logging to enter_state() (from

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Jan Beulich
>>> On 11.04.18 at 13:02, wrote: > On 04/11/2018 11:17 AM, Dario Faggioli wrote: >> On Wed, 2018-04-11 at 12:00 +0200, Olaf Hering wrote: >>> On Wed, Apr 11, Dario Faggioli wrote: >>> Olaf, can you give it a try? It should be fine to run it on top of the

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

2018-04-11 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 74576 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/74576/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 13d909f89a3cee1c1f6b851a4cda7bd1a44e90ae baseline

[Xen-devel] [PATCH 1/7] xen/arm: Added handling of the trapped access to OSLSR register

2018-04-11 Thread Mirela Simonovic
Linux/dom0 accesses OSLSR register when saving CPU context during the suspend procedure. Xen traps access to this register, but has no handling for it. Consequently, Xen injects undef exception to linux, causing it to crash. This patch adds handling of the trapped access to OSLSR as ro/raz.

[Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-11 Thread Mirela Simonovic
This patch set contains fixes required to enable CPU hotplug for secondary CPUs. CPU hotplug of secondary CPUs will be used for suspend to RAM support for ARM. With these patches calling disable_nonboot_cpus() from the boot CPU will cause all secondary CPUs to be stopped. When a CPU is stopped it

[Xen-devel] [PATCH 5/7] xen/arm: Remove __initdata and __init to enable CPU hotplug

2018-04-11 Thread Mirela Simonovic
CPU up flow is currently used during the initial boot to start secondary CPUs. However, the same flow should be used for CPU hotplug, e.g. when hotplugging secondary CPUs within the resume procedure (resume from the suspend to RAM). Therefore, prefixes __initdata and __init had to be removed from

[Xen-devel] [PATCH 6/7] xen/arm: Setup virtual paging for secondary CPUs in non-boot scenario

2018-04-11 Thread Mirela Simonovic
In existing code the paging for secondary CPUs is setup only in boot flow. The setup is triggered from start_xen function after all CPUs are brought online. In other words, the initialization of VTCR_EL2 register is done out of the cpu_up/start_secondary control flow. However, the cpu_up flow

[Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-11 Thread Mirela Simonovic
Freeing percpu area is done when a non-boot CPU is disabled upon suspend. This use to be scheduled for execution after a period of time, what caused the following racing issues. If CPU is enabled after it is disabled and before the freeing of percpu area is performed, Xen would crash upon

[Xen-devel] [PATCH 3/7] xen/arm/psci: Implement CPU_OFF PSCI call (physical interface)

2018-04-11 Thread Mirela Simonovic
This patch adds the PSCI CPU_OFF call to the EL3 in order to trigger powering down of the calling CPU when the CPU is stopped. If CPU_OFF call fails for some reason, e.g. EL3 does not implement the PSCI CPU_OFF function, the calling CPU will loop in the infinite while/wfi, as it was looping before

[Xen-devel] [PATCH 7/7] xen/arm: Restore IRQ affinity after hotplugging a CPU

2018-04-11 Thread Mirela Simonovic
Secondary pCPUs will be offlined on system suspend and hotplugged on resume. When offlining secondary CPUs all interrupts targeted to those CPUs will be routed to the boot CPU. The boot CPU is responsible for finalizing suspend procedure. All wake-up interrupts are therefore targeted to the boot

[Xen-devel] [PATCH 2/7] xen/arm/vgic-v2: Ignore write to GICD_ISACTIVERn registers

2018-04-11 Thread Mirela Simonovic
Guests attempt to write into these registers on resume (for example Linux). Without this patch a data abort exception will be raised to the guest. This patch handles the write access by ignoring it. This should be fine for now because reading these registers is already handled as 'read as zero'.

Re: [Xen-devel] Resume from suspend to RAM broken when using early microcode updates

2018-04-11 Thread Jan Beulich
>>> On 11.04.18 at 14:01, wrote: > Andrew Cooper: >> On 11/04/18 12:48, Simon Gaiser wrote: >>> Hi, >>> >>> when I use early microcode loading with the microcode update with the >>> BTI mitigations, resuming from suspend to RAM is broken. >>> >>> Based on added

Re: [Xen-devel] Resume from suspend to RAM broken when using early microcode updates

2018-04-11 Thread Andrew Cooper
On 11/04/18 12:48, Simon Gaiser wrote: > Hi, > > when I use early microcode loading with the microcode update with the > BTI mitigations, resuming from suspend to RAM is broken. > > Based on added logging to enter_state() (from power.c) it doesn't > survive the local_irq_restore(flags) call (at

[Xen-devel] [PATCH v5] xen/privcmd: add IOCTL_PRIVCMD_MMAP_RESOURCE

2018-04-11 Thread Paul Durrant
My recent Xen patch series introduces a new HYPERVISOR_memory_op to support direct priv-mapping of certain guest resources (such as ioreq pages, used by emulators) by a tools domain, rather than having to access such resources via the guest P2M. This patch adds the necessary infrastructure to the

[Xen-devel] [RFC PATCH 1/3] xen: Introduce vcpu_sleep_nosync_locked()

2018-04-11 Thread George Dunlap
There are a lot of places which release a lock before calling vcpu_sleep_nosync(), which then just grabs the lock again. This is not only a waste of time, but leads to more code duplication (since you have to copy-and-paste recipes rather than calling a unified function), which in turn leads to

[Xen-devel] [RFC PATCH 3/3] sched/credit: Avoid doing cpu_pick calculation twice when deciding to move

2018-04-11 Thread George Dunlap
In vcpu_acct(), we call _csched_cpu_pick() in order to decide whether to consider migrating; but then we throw that result away and do it again in context_saved() if we decide we do need to move. Instead, just initiate the migration and let the vcpu_migrate_finish() in context_saved() determine

[Xen-devel] [RFC PATCH 2/3] xen: Refactor migration

2018-04-11 Thread George Dunlap
The current sequence to initiate vcpu migration is inefficent and error-prone: - The initiator sets VPF_migraging with the lock held, then drops the lock and calls vcpu_sleep_nosync(), which immediately grabs the lock again - A number of places unnecessarily check for v->pause_flags in

[Xen-devel] [RFC PATCH 0/3] vcpu migration improvements

2018-04-11 Thread George Dunlap
Some compile-tested-only sketches of what I'm talking about. Let me know what you think. ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] Resume from suspend to RAM broken when using early microcode updates

2018-04-11 Thread Jan Beulich
>>> On 11.04.18 at 14:46, wrote: > Jan Beulich: > On 11.04.18 at 14:11, wrote: >> On 11.04.18 at 14:01, wrote: Andrew Cooper: > On 11/04/18 12:48, Simon Gaiser wrote: >> Hi, >> >> when I

[Xen-devel] Resume from suspend to RAM broken when using early microcode updates

2018-04-11 Thread Simon Gaiser
Hi, when I use early microcode loading with the microcode update with the BTI mitigations, resuming from suspend to RAM is broken. Based on added logging to enter_state() (from power.c) it doesn't survive the local_irq_restore(flags) call (at least a printk() after the call doesn't output

Re: [Xen-devel] Resume from suspend to RAM broken when using early microcode updates

2018-04-11 Thread Simon Gaiser
Andrew Cooper: > On 11/04/18 12:48, Simon Gaiser wrote: >> Hi, >> >> when I use early microcode loading with the microcode update with the >> BTI mitigations, resuming from suspend to RAM is broken. >> >> Based on added logging to enter_state() (from power.c) it doesn't >> survive the

Re: [Xen-devel] Resume from suspend to RAM broken when using early microcode updates

2018-04-11 Thread Andrew Cooper
On 11/04/18 13:01, Simon Gaiser wrote: > Andrew Cooper: >> On 11/04/18 12:48, Simon Gaiser wrote: >>> Hi, >>> >>> when I use early microcode loading with the microcode update with the >>> BTI mitigations, resuming from suspend to RAM is broken. >>> >>> Based on added logging to enter_state() (from

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

2018-04-11 Thread osstest service owner
flight 122161 xen-4.8-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/122161/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-xtf-amd64-amd64-5 50 xtf/test-hvm64-lbr-tsx-vmentry fail like 121291

Re: [Xen-devel] Resume from suspend to RAM broken when using early microcode updates

2018-04-11 Thread Simon Gaiser
Jan Beulich: On 11.04.18 at 14:11, wrote: > On 11.04.18 at 14:01, wrote: >>> Andrew Cooper: On 11/04/18 12:48, Simon Gaiser wrote: > Hi, > > when I use early microcode loading with the microcode update with the > BTI

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

2018-04-11 Thread osstest service owner
flight 122174 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/122174/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 13 migrate-support-checkfail never pass test-arm64-arm64-xl-xsm

Re: [Xen-devel] [PATCH] x86/xen: zero MSR_IA32_SPEC_CTRL before suspend

2018-04-11 Thread Ingo Molnar
* Jan Beulich wrote: > Additionally, x86 maintainers: is there a particular reason this (or > any functionally equivalent patch) isn't upstream yet? As indicated > before, I had not been able to find any discussion, and hence I > see no reason why this is a patch we

Re: [Xen-devel] Resume from suspend to RAM broken when using early microcode updates

2018-04-11 Thread Jan Beulich
>>> On 11.04.18 at 14:11, wrote: On 11.04.18 at 14:01, wrote: >> Andrew Cooper: >>> On 11/04/18 12:48, Simon Gaiser wrote: Hi, when I use early microcode loading with the microcode update with the BTI mitigations, resuming

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Olaf Hering
On Wed, Apr 11, Dario Faggioli wrote: > If you're interested in figuring out, I'd like to see: > - full output of `xl info -n' > - output of `xl debug-key u' > - xl vcpu-list > - xl list -n Logs for this .cfg attached: name='fv_sles12sp1.0' vif=[ 'mac=00:18:3e:58:00:c1,bridge=br0' ] memory=

Re: [Xen-devel] [PATCH v2 07/17] arm64: vgic-v3: Add ICV_EOIR1_EL1 handler

2018-04-11 Thread Julien Grall
Hi, On 04/06/2018 09:37 AM, Manish Jaggi wrote: On 04/05/2018 03:10 PM, Julien Grall wrote: Hi, On 02/04/18 12:17, Manish Jaggi wrote: On 04/02/2018 04:33 PM, Manish Jaggi wrote: On 03/27/2018 03:48 PM, Marc Zyngier wrote: On 27/03/18 10:07, Manish Jaggi wrote: This patch is ported

Re: [Xen-devel] [PATCH 4/7] xen/arm: When CPU dies, free percpu area immediatelly

2018-04-11 Thread Julien Grall
Hi, On 11/04/18 14:19, Mirela Simonovic wrote: Freeing percpu area is done when a non-boot CPU is disabled upon suspend. This use to be scheduled for execution after a period of time, what caused the following racing issues. If CPU is enabled after it is disabled and before the freeing of

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Olaf Hering
On Wed, Apr 11, Olaf Hering wrote: > On Wed, Apr 11, Dario Faggioli wrote: > > > Olaf, can you give it a try? It should be fine to run it on top of the > > last debug patch (the one that produced this crash). > > Yes, with both changes it did >4k iterations already. Thanks. That was with

Re: [Xen-devel] [PATCH 3/9] SUPPORT.md: Syntax: Provide a title rather than a spurious empty section

2018-04-11 Thread George Dunlap
On 04/11/2018 04:35 PM, Ian Jackson wrote: > This commits (more or less) this file to be processed with pandoc, > rather than other markdown processors. There is, unfortunately, no > widely-accepted way to declare a title for the document. > > I tested feeding the document to markdown(1) on

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-11 Thread Mirela Simonovic
Hi Julien, On Wed, Apr 11, 2018 at 6:02 PM, Julien Grall wrote: > Hi, > > On 11/04/18 16:58, Mirela Simonovic wrote: > >> On 04/11/2018 05:07 PM, Julien Grall wrote: >> >>> On 11/04/18 14:19, Mirela Simonovic wrote: >>> >> Migrating interrupts when turning off a CPU

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Jan Beulich
>>> On 11.04.18 at 17:03, wrote: > On Wed, Apr 11, Olaf Hering wrote: > >> On Wed, Apr 11, Dario Faggioli wrote: >> >> > Olaf, can you give it a try? It should be fine to run it on top of the >> > last debug patch (the one that produced this crash). >> >> Yes, with both changes

[Xen-devel] [PATCH 4/9] docs/gen-html-index: Extract titles from HTML documents

2018-04-11 Thread Ian Jackson
Signed-off-by: Ian Jackson Release-acked-by: Juergen Gross --- docs/gen-html-index | 13 + 1 file changed, 13 insertions(+) diff --git a/docs/gen-html-index b/docs/gen-html-index index e9792bf..5b43b42 100644 --- a/docs/gen-html-index +++

Re: [Xen-devel] [PATCH 7/7] xen/arm: Restore IRQ affinity after hotplugging a CPU

2018-04-11 Thread Julien Grall
Hi, Please CC relevant maintainers for the code you modify. You can use scripts/get_maintainers.pl for that. I have CCed them for you this time. Cheers, On 11/04/18 14:19, Mirela Simonovic wrote: Secondary pCPUs will be offlined on system suspend and hotplugged on resume. When offlining

Re: [Xen-devel] Resume from suspend to RAM broken when using early microcode updates

2018-04-11 Thread Andrew Cooper
On 11/04/18 15:49, Konrad Rzeszutek Wilk wrote: > On Wed, Apr 11, 2018 at 01:12:51PM +0100, Andrew Cooper wrote: >> On 11/04/18 13:01, Simon Gaiser wrote: >>> Andrew Cooper: On 11/04/18 12:48, Simon Gaiser wrote: > Hi, > > when I use early microcode loading with the microcode

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Olaf Hering
On Wed, Apr 11, Olaf Hering wrote: > On Wed, Apr 11, Olaf Hering wrote: > > On Wed, Apr 11, Dario Faggioli wrote: > > > Olaf, can you give it a try? It should be fine to run it on top of the > > > last debug patch (the one that produced this crash). > > Yes, with both changes it did >4k

Re: [Xen-devel] Resume from suspend to RAM broken when using early microcode updates

2018-04-11 Thread Konrad Rzeszutek Wilk
On Wed, Apr 11, 2018 at 01:12:51PM +0100, Andrew Cooper wrote: > On 11/04/18 13:01, Simon Gaiser wrote: > > Andrew Cooper: > >> On 11/04/18 12:48, Simon Gaiser wrote: > >>> Hi, > >>> > >>> when I use early microcode loading with the microcode update with the > >>> BTI mitigations, resuming from

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-11 Thread Mirela Simonovic
Hi Julien, Thank you for the feedback. On 04/11/2018 05:07 PM, Julien Grall wrote: Hi Mirela, Thank you for sending the series. On 11/04/18 14:19, Mirela Simonovic wrote: This patch set contains fixes required to enable CPU hotplug for secondary CPUs. CPU hotplug of secondary CPUs will be

Re: [Xen-devel] [PATCH 1/7] xen/arm: Added handling of the trapped access to OSLSR register

2018-04-11 Thread Julien Grall
Hi, You seem to have used a wrong address for me. Title: This patch is only adding the arm64 side. Please make it clear in it. Cheers, On 04/11/2018 02:19 PM, Mirela Simonovic wrote: Linux/dom0 accesses OSLSR register when saving CPU context during the suspend procedure. Xen traps access to

Re: [Xen-devel] [PATCH 3/7] xen/arm/psci: Implement CPU_OFF PSCI call (physical interface)

2018-04-11 Thread Julien Grall
Hi, On 11/04/18 14:19, Mirela Simonovic wrote: This patch adds the PSCI CPU_OFF call to the EL3 in order to trigger powering down of the calling CPU when the CPU is stopped. If CPU_OFF call fails for some reason, e.g. EL3 does not implement the PSCI CPU_OFF function, the calling CPU will loop

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-11 Thread Julien Grall
Hi, I forgot to mention that the title of most of the patch gives the impression you fixes hotplug for both Arm32 and Arm64. After a deeper look, it is only arm64. Please make clear over commit message and cover letter. Cheers, On 11/04/18 14:19, Mirela Simonovic wrote: This patch set

Re: [Xen-devel] Resume from suspend to RAM broken when using early microcode updates

2018-04-11 Thread Jan Beulich
>>> On 11.04.18 at 17:05, wrote: > On 11/04/18 15:49, Konrad Rzeszutek Wilk wrote: >> On Wed, Apr 11, 2018 at 01:12:51PM +0100, Andrew Cooper wrote: >>> On 11/04/18 13:01, Simon Gaiser wrote: Andrew Cooper: > On 11/04/18 12:48, Simon Gaiser wrote: >> Hi,

Re: [Xen-devel] [PATCH 9/9] docs: Provide support-matrix-generate, to generate a support matrix in HTML

2018-04-11 Thread Juergen Gross
On 11/04/18 17:35, Ian Jackson wrote: > This archaeology script: > - figures out what the current and previous Xen versions were > - looks for appropriate git branches for them > - finds SUPPORT.md for each one > - feeds its findings to parse-support-md > > We do not intend to integrate this

Re: [Xen-devel] [PATCH 8/9] docs: Provide parse-support-md

2018-04-11 Thread Juergen Gross
On 11/04/18 17:35, Ian Jackson wrote: > This utility reads json format pandoc output, from parsing one or more > SUPPORT.md files, and generates an HTML table element containing the > principal version and feature information. > > This is rather hairier than I anticipated when I started out;

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Olaf Hering
Am Wed, 11 Apr 2018 09:38:59 -0600 schrieb "Jan Beulich" : > And till now I had assumed we've taken care of them with earlier > fixes (all 4.7 reports were with old packages, like 4.7.2 based > ones). Can you repro this with a debug hypervisor (so we can > both trust the stack

Re: [Xen-devel] [PATCH 3/9] SUPPORT.md: Syntax: Provide a title rather than a spurious empty section

2018-04-11 Thread Ian Jackson
George Dunlap writes ("Re: [PATCH 3/9] SUPPORT.md: Syntax: Provide a title rather than a spurious empty section"): > On 04/11/2018 04:35 PM, Ian Jackson wrote: > > I tested feeding the document to markdown(1) on Debian jessie and it > > reproduced the % line as if it were simple text. I guess

Re: [Xen-devel] Xen and safety certification, Minutes of the meeting on Apr 4th

2018-04-11 Thread Artem Mygaiev
Hi Stefano On 06.04.18 23:47, Stefano Stabellini wrote: On Fri, 6 Apr 2018, Artem Mygaiev wrote: 2) Create a subset of functions that need to go through certifications Next step: create a small Kconfig. We could use the Renesas Rcar as reference. We need a discussion about the features we

Re: [Xen-devel] [PATCH 2/7] xen/arm/vgic-v2: Ignore write to GICD_ISACTIVERn registers

2018-04-11 Thread Julien Grall
Hi, On 11/04/18 14:19, Mirela Simonovic wrote: Guests attempt to write into these registers on resume (for example Linux). Without this patch a data abort exception will be raised to the guest. This patch handles the write access by ignoring it. This should be fine for now because reading these

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-11 Thread Julien Grall
Hi Mirela, Thank you for sending the series. On 11/04/18 14:19, Mirela Simonovic wrote: This patch set contains fixes required to enable CPU hotplug for secondary CPUs. CPU hotplug of secondary CPUs will be used for suspend to RAM support for ARM. With these patches calling

Re: [Xen-devel] [PATCH 6/7] xen/arm: Setup virtual paging for secondary CPUs in non-boot scenario

2018-04-11 Thread Julien Grall
Hi, On 11/04/18 14:19, Mirela Simonovic wrote: In existing code the paging for secondary CPUs is setup only in boot flow. The setup is triggered from start_xen function after all CPUs are brought online. In other words, the initialization of VTCR_EL2 register is done out of the

[Xen-devel] [PATCH 8/9] docs: Provide parse-support-md

2018-04-11 Thread Ian Jackson
This utility reads json format pandoc output, from parsing one or more SUPPORT.md files, and generates an HTML table element containing the principal version and feature information. This is rather hairier than I anticipated when I started out; hence the 400-odd-line Perl script. Machinery to

[Xen-devel] [PATCH 2/9] SUPPORT.md: Syntax: Fix a typo "States"

2018-04-11 Thread Ian Jackson
Signed-off-by: Ian Jackson Release-acked-by: Juergen Gross Acked-by: George Dunlap --- SUPPORT.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/SUPPORT.md b/SUPPORT.md index 1c5220b..e447069 100644 ---

[Xen-devel] [PATCH 7/9] docs/Makefile: Format SUPPORT.md into the toplevel

2018-04-11 Thread Ian Jackson
Signed-off-by: Ian Jackson Release-acked-by: Juergen Gross --- docs/Makefile | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/docs/Makefile b/docs/Makefile index d82463f..b300bb6 100644 --- a/docs/Makefile +++ b/docs/Makefile @@

[Xen-devel] [PATCH 3/9] SUPPORT.md: Syntax: Provide a title rather than a spurious empty section

2018-04-11 Thread Ian Jackson
This commits (more or less) this file to be processed with pandoc, rather than other markdown processors. There is, unfortunately, no widely-accepted way to declare a title for the document. I tested feeding the document to markdown(1) on Debian jessie and it reproduced the % line as if it were

[Xen-devel] [PATCH 6/9] docs/Makefile: Introduce GENERATE_PANDOC_RULE_RAW

2018-04-11 Thread Ian Jackson
We are going to want to format SUPPORT.md which does not match the filename patterns in docs/. So provide a way to make an ad-hoc rule using pandoc with the standard options. No functional change in this patch. Signed-off-by: Ian Jackson Release-acked-by: Juergen

[Xen-devel] [PATCH 9/9] docs: Provide support-matrix-generate, to generate a support matrix in HTML

2018-04-11 Thread Ian Jackson
This archaeology script: - figures out what the current and previous Xen versions were - looks for appropriate git branches for them - finds SUPPORT.md for each one - feeds its findings to parse-support-md We do not intend to integrate this into docs/Makefile, because it relies on the git

[Xen-devel] [PATCH 1/9] SUPPORT.md: Syntax: Fix some bullet lists

2018-04-11 Thread Ian Jackson
Continuations of bullet list items must be indented by exactly 4 spaces (according to pandoc_markdown(5) on Debian jessie). This is most easily achieved by making the bullet list items have two spaces before the `*'. Signed-off-by: Ian Jackson Release-acked-by:

[Xen-devel] [PATCH for-4.11 v2 0/9] Provide support matrix generator

2018-04-11 Thread Ian Jackson
This series provides code to generate a feature support matrix, to replace the one on the wiki. You can see an example of the output here: http://xenbits.xen.org/people/iwj/2018/support-matrix-example-v2/t.html There is also an accompanying SUPPORT.html to make the links for 4.11 work. Patches

[Xen-devel] [PATCH 5/9] docs/gen-html-index: Support documents at the toplevel

2018-04-11 Thread Ian Jackson
There are none yet. Signed-off-by: Ian Jackson Release-acked-by: Juergen Gross --- docs/gen-html-index | 4 1 file changed, 4 insertions(+) diff --git a/docs/gen-html-index b/docs/gen-html-index index 5b43b42..8258e2b 100644 ---

Re: [Xen-devel] [PATCH 0/7] xen/arm: CPU hotplug fixes

2018-04-11 Thread Julien Grall
Hi, On 11/04/18 16:58, Mirela Simonovic wrote: On 04/11/2018 05:07 PM, Julien Grall wrote: On 11/04/18 14:19, Mirela Simonovic wrote: Migrating interrupts when turning off a CPU already works. However, when a CPU is turned back on there is no interrupt migration back to the hotplugged CPU -

[Xen-devel] [distros-debian-squeeze test] 74577: trouble: broken/fail/pass

2018-04-11 Thread Platform Team regression test user
flight 74577 distros-debian-squeeze real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/74577/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64 broken

Re: [Xen-devel] [PATCH for-4.11 v2 0/9] Provide support matrix generator

2018-04-11 Thread Lars Kurth
On 11/04/2018, 16:36, "Ian Jackson" wrote: This series provides code to generate a feature support matrix, to replace the one on the wiki. You can see an example of the output here:

Re: [Xen-devel] X86 Community Call: Wed March 14, 15:00 - 16:00 UTC - Meeting Minutes

2018-04-11 Thread Razvan Cojocaru
On 04/11/2018 08:10 PM, Lars Kurth wrote: > ### [PATCH RFC 00/14] EPT-Based Sub-page Write Protection Support > > ### (​Zhang Yi) > > RFC posted by Zhang Yi Oct 19, 2017 > https://marc.info/?l=xen-devel= > https://xen.markmail.org/thread/m75h6b2aiwk5h7fx > > > No acks, reviews only by

Re: [Xen-devel] crash in csched_load_balance after xl vcpu-pin

2018-04-11 Thread Dario Faggioli
On Wed, 2018-04-11 at 17:27 +0200, Olaf Hering wrote: > On Wed, Apr 11, Olaf Hering wrote: > > > That was with sched=credit2, sorry for that. > > Now with just that second patch ... > > Still BUG in csched_load_balance. > > (XEN) Xen BUG at sched_credit.c:1694 > (XEN) [

  1   2   >