[Xen-devel] [distros-debian-jessie test] 44240: trouble: blocked/broken

2016-03-11 Thread Platform Team regression test user
flight 44240 distros-debian-jessie real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/44240/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 3 host-install(3) broken REG

Re: [Xen-devel] [PATCH] xen: Restore p2m_access_t enum order to allow bitmask semantics

2016-03-11 Thread Malcolm Crossley
On 10/03/16 20:48, Tamas K Lengyel wrote: > > > On Wed, Mar 9, 2016 at 5:30 PM, George Dunlap > wrote: > > On 08/03/16 15:30, Malcolm Crossley wrote: > > Nested hap code assumed implict bitmask semantics of the p2m_access_t > > enum prior to C/S 4c63

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

2016-03-11 Thread osstest service owner
flight 85870 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/85870/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-rumpuserxen6 xen-build fail REGR. vs. 59254 build-amd64-rumpuserx

Re: [Xen-devel] [PATCH v11 10/27] tools/libxl: add back channel support to write stream

2016-03-11 Thread Wen Congyang
On 03/05/2016 01:00 AM, Ian Jackson wrote: > Changlong Xie writes ("[PATCH v11 10/27] tools/libxl: add back channel > support to write stream"): >> From: Wen Congyang >> >> Add back channel support to write stream. If the write stream is >> a back channel stream, this means the write stream is us

Re: [Xen-devel] [PATCH] flask: change default state to enforcing

2016-03-11 Thread Jan Beulich
>>> On 10.03.16 at 19:30, wrote: > This change will cause the boot to fail if you do not specify an XSM > policy during boot; if you need to load a policy from dom0, use the > "flask=late" boot parameter. And what mode is the system in until that happens? From the command line doc, I understand i

[Xen-devel] Request to revert superpage adjustments

2016-03-11 Thread Andrew Cooper
Sadly, c/s cf393624 "x86: use 2M superpages for text/data/bss mappings" exposes a bug in all Syslinux variants, including ISOlinux and PXELinux, causing a failure to boot. Xen currently requires its bootloader to decompress it, and place it at the 1MB physical boundary. The alignment adjustments

[Xen-devel] issue: xen pv guest text mode key press by TigerVNC

2016-03-11 Thread Chun Yan Liu
Hi, List, Currently after switch from TightVNC to TigerVNC, found a key-press issue in XEN pv guest, after investigation by myself, didn't find root cause yet, could anyone help to have a look or have some idea? Following is a description of the issue and some investigation result: In Xen pv gue

[Xen-devel] [PATCH] xen/events/fifo: Add missing hotplug notifier transition

2016-03-11 Thread Anna-Maria Gleixner
The evtchn_fifo_cpu_notification() hotplug callback lacks handling of the CPU_UP_CANCELED case. That means, if CPU_UP_PREPARE fails, the handle of the fifo events is not dropped. Add handling for CPU_UP_CANCELED transition to drop the fifo events handle. Cc: Konrad Rzeszutek Wilk Cc: Boris Ostro

[Xen-devel] [qemu-mainline test] 85884: tolerable FAIL - PUSHED

2016-03-11 Thread osstest service owner
flight 85884 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/85884/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-xl-rtds 11 guest-start fail like 85694 test-amd64-i386-xl-qemuu-win

Re: [Xen-devel] [V4] x86/xsaves: fix overwriting between non-lazy/lazy xsaves

2016-03-11 Thread Jan Beulich
>>> On 11.03.16 at 07:45, wrote: > On Thu, Mar 10, 2016 at 02:30:34AM -0700, Jan Beulich wrote: >> > --- a/xen/arch/x86/xstate.c >> > +++ b/xen/arch/x86/xstate.c >> > @@ -165,7 +165,7 @@ void expand_xsave_states(struct vcpu *v, void *dest, >> > unsigned int size) >> > u64 xstate_bv = xsave->

Re: [Xen-devel] [PATCH v3 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization

2016-03-11 Thread Dario Faggioli
On Fri, 2016-03-11 at 06:54 +, Xu, Quan wrote: > On March 11, 2016 11:25am, wrote: > > > > On Wed, Mar 9, 2016 at 8:17 AM, Quan Xu wrote: > > > > > > pcidevs_lock should be held with interrupt enabled. However there > > > remains an exception in AMD IOMMU code, where the lock is > > > acqui

Re: [Xen-devel] [PATCH] xen/events/fifo: Add missing hotplug notifier transition

2016-03-11 Thread David Vrabel
On 11/03/16 09:00, Anna-Maria Gleixner wrote: > The evtchn_fifo_cpu_notification() hotplug callback lacks handling of > the CPU_UP_CANCELED case. That means, if CPU_UP_PREPARE fails, the > handle of the fifo events is not dropped. > > Add handling for CPU_UP_CANCELED transition to drop the fifo ev

[Xen-devel] pygrub detects false keypress on console

2016-03-11 Thread Steven Haigh
Hi all, Testing on Xen 4.6.1 - and I've noticed that if I start an EL7 instance and auto-attach to the console, that pygrub seems to see a key press and waits. Sometimes, I have to press ^] to make it actually boot. The guest is EL7 with grub2 config file. Started with: # xl create /etc/xen/c

Re: [Xen-devel] [PATCH 2/4] ns16550: enable Pericom controller support

2016-03-11 Thread Jan Beulich
>>> On 11.03.16 at 03:31, wrote: > On Wed, Mar 09, 2016 at 10:01:08AM -0700, Jan Beulich wrote: >> >>> On 09.03.16 at 17:52, wrote: >> > On Tue, Mar 08, 2016 at 01:48:05AM -0700, Jan Beulich wrote: >> >> >>> On 07.03.16 at 23:04, wrote: >> >> > Hmm, if it was some other PCI based serial card lik

Re: [Xen-devel] [PATCH] xen/events/fifo: Add missing hotplug notifier transition

2016-03-11 Thread Anna-Maria Gleixner
On Fri, 11 Mar 2016, David Vrabel wrote: > On 11/03/16 09:00, Anna-Maria Gleixner wrote: > > The evtchn_fifo_cpu_notification() hotplug callback lacks handling of > > the CPU_UP_CANCELED case. That means, if CPU_UP_PREPARE fails, the > > handle of the fifo events is not dropped. > > > > Add handl

Re: [Xen-devel] [PATCH] xen: arm: zero EL2 pagetable pages before use

2016-03-11 Thread Jan Beulich
>>> On 10.03.16 at 23:00, wrote: First of all - please correctly Cc maintainers (there were two recent changes for ARM). > --- a/xen/arch/arm/mm.c > +++ b/xen/arch/arm/mm.c > @@ -730,6 +730,7 @@ void __init setup_xenheap_mappings(unsigned long base_mfn, > else > { >

Re: [Xen-devel] [PATCH v2] xen kconfig: don't "select INPUT_XEN_KBDDEV_FRONTEND"

2016-03-11 Thread Tomi Valkeinen
On 16/02/16 17:03, Arnd Bergmann wrote: > The Xen framebuffer driver selects the xen keyboard driver, so the latter > will be built-in if XEN_FBDEV_FRONTEND=y. However, when CONFIG_INPUT > is a loadable module, this configuration cannot work. On mainline kernels, > the symbol will be enabled but no

[Xen-devel] [PATCH 4/6] hwmon: use call_sync_on_phys_cpu() for dell-smm i8k

2016-03-11 Thread Juergen Gross
Use the call_sync_on_phys_cpu() function to call system management mode on cpu 0. Signed-off-by: Juergen Gross --- drivers/hwmon/dell-smm-hwmon.c | 27 --- 1 file changed, 12 insertions(+), 15 deletions(-) diff --git a/drivers/hwmon/dell-smm-hwmon.c b/drivers/hwmon/dell-

[Xen-devel] [PATCH 6/6] xen: add xen_pin_vcpu() to support calling functions on a dedicated pcpu

2016-03-11 Thread Juergen Gross
Some hardware models (e.g. Dell Studio 1555 laptops) require calls to the firmware to be issued on cpu 0 only. As Dom0 might have to use these calls, add xen_pin_vcpu() to achieve this functionality. In case either the domain doesn't have the privilege to make the related hypercall or the hypervis

[Xen-devel] [PATCH 1/6] xen: sync xen header

2016-03-11 Thread Juergen Gross
Import the actual version of include/xen/interface/sched.h from Xen. Signed-off-by: Juergen Gross --- include/xen/interface/sched.h | 100 ++ 1 file changed, 82 insertions(+), 18 deletions(-) diff --git a/include/xen/interface/sched.h b/include/xen/interf

[Xen-devel] [PATCH 5/6] virt, sched: add cpu pinning to call_sync_on_phys_cpu()

2016-03-11 Thread Juergen Gross
Add generic virtualization support for pinning the current vcpu to a specified physical cpu. As this operation isn't performance critical (a very limited set of operations like BIOS calls and SMIs is expected to need this) just add a hypervisor specific indirection. Such a pinning should last as s

[Xen-devel] [PATCH 3/6] dcdbas: make use of call_sync_on_phys_cpu()

2016-03-11 Thread Juergen Gross
Use call_sync_on_phys_cpu() to raise SMI on cpu 0. Signed-off-by: Juergen Gross --- drivers/firmware/dcdbas.c | 46 -- 1 file changed, 20 insertions(+), 26 deletions(-) diff --git a/drivers/firmware/dcdbas.c b/drivers/firmware/dcdbas.c index 829eec8..

[Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Juergen Gross
On some hardware models (e.g. Dell Studio 1555 laptop) some hardware related functions (e.g. SMIs) are to be executed on physical cpu 0 only. Instead of open coding such a functionality multiple times in the kernel add a service function for this purpose. This will enable the possibility to take sp

[Xen-devel] [PATCH 0/6] Support calling functions on dedicated physical cpu

2016-03-11 Thread Juergen Gross
Some hardware (e.g. Dell Studio laptops) require special functions to be called on physical cpu 0 in order to avoid occasional hangs. When running as dom0 under Xen this could be achieved only via special boot parameters (vcpu pinning) limiting the hypervisor in it's scheduling decisions. This pat

[Xen-devel] [xen-4.4-testing test] 85888: regressions - FAIL

2016-03-11 Thread osstest service owner
flight 85888 xen-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/85888/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-multivcpu 15 guest-start/debian.repeat fail REGR. vs. 85031 Regressions whic

Re: [Xen-devel] Request to revert superpage adjustments

2016-03-11 Thread Jan Beulich
>>> On 11.03.16 at 10:22, wrote: > Sadly, c/s cf393624 "x86: use 2M superpages for text/data/bss mappings" > exposes a bug in all Syslinux variants, including ISOlinux and > PXELinux, causing a failure to boot. > > Xen currently requires its bootloader to decompress it, and place it at > the 1MB

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Peter Zijlstra
On Fri, Mar 11, 2016 at 12:59:30PM +0100, Juergen Gross wrote: > +int call_sync_on_phys_cpu(unsigned cpu, int (*func)(void *), void *par) > +{ > + cpumask_var_t old_mask; > + int ret; > + > + if (cpu >= nr_cpu_ids) > + return -EINVAL; > + > + if (!alloc_cpumask_var(&old_

Re: [Xen-devel] [PATCH 0/6] Support calling functions on dedicated physical cpu

2016-03-11 Thread Peter Zijlstra
On Fri, Mar 11, 2016 at 12:59:28PM +0100, Juergen Gross wrote: > Some hardware (e.g. Dell Studio laptops) require special functions to > be called on physical cpu 0 in order to avoid occasional hangs. When > running as dom0 under Xen this could be achieved only via special boot > parameters (vcpu p

Re: [Xen-devel] [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions

2016-03-11 Thread Vladimir 'phcoder' Serbinenko
On Wednesday, March 9, 2016, Daniel Kiper wrote: > On Wed, Mar 02, 2016 at 05:51:36PM +0100, Daniel Kiper wrote: > > Hi, > > > > This patch series: > > - enables EFI boot services usage in loaded images > > by multiboot2 protocol, > > - add support for multiboot2 protocol compatible > >

Re: [Xen-devel] [PATCH 0/6] Support calling functions on dedicated physical cpu

2016-03-11 Thread Pali Rohár
On Friday 11 March 2016 13:25:14 Peter Zijlstra wrote: > On Fri, Mar 11, 2016 at 12:59:28PM +0100, Juergen Gross wrote: > > Some hardware (e.g. Dell Studio laptops) require special functions to > > be called on physical cpu 0 in order to avoid occasional hangs. When > > running as dom0 under Xen th

Re: [Xen-devel] [PATCH v3 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization

2016-03-11 Thread Xu, Quan
On March 11, 2016 6:36pm, wrote: > On Fri, 2016-03-11 at 06:54 +, Xu, Quan wrote: > > On March 11, 2016 11:25am, wrote: > > > > > > On Wed, Mar 9, 2016 at 8:17 AM, Quan Xu wrote: > > > > > > > > pcidevs_lock should be held with interrupt enabled. However there > > > > remains an exception in

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Peter Zijlstra
On Fri, Mar 11, 2016 at 01:19:50PM +0100, Peter Zijlstra wrote: > On Fri, Mar 11, 2016 at 12:59:30PM +0100, Juergen Gross wrote: > > +int call_sync_on_phys_cpu(unsigned cpu, int (*func)(void *), void *par) > > +{ > > + cpumask_var_t old_mask; > > + int ret; > > + > > + if (cpu >= nr_cpu_ids)

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Juergen Gross
On 11/03/16 13:19, Peter Zijlstra wrote: > On Fri, Mar 11, 2016 at 12:59:30PM +0100, Juergen Gross wrote: >> +int call_sync_on_phys_cpu(unsigned cpu, int (*func)(void *), void *par) >> +{ >> +cpumask_var_t old_mask; >> +int ret; >> + >> +if (cpu >= nr_cpu_ids) >> +return -EI

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Peter Zijlstra
On Fri, Mar 11, 2016 at 01:43:53PM +0100, Juergen Gross wrote: > On 11/03/16 13:19, Peter Zijlstra wrote: > > On Fri, Mar 11, 2016 at 12:59:30PM +0100, Juergen Gross wrote: > >> +int call_sync_on_phys_cpu(unsigned cpu, int (*func)(void *), void *par) > >> +{ > >> + cpumask_var_t old_mask; > >> +

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Juergen Gross
On 11/03/16 13:42, Peter Zijlstra wrote: > On Fri, Mar 11, 2016 at 01:19:50PM +0100, Peter Zijlstra wrote: >> On Fri, Mar 11, 2016 at 12:59:30PM +0100, Juergen Gross wrote: >>> +int call_sync_on_phys_cpu(unsigned cpu, int (*func)(void *), void *par) >>> +{ >>> + cpumask_var_t old_mask; >>> + in

Re: [Xen-devel] [PATCH] xen: arm: zero EL2 pagetable pages before use

2016-03-11 Thread Andrew Cooper
On 11/03/16 11:29, Jan Beulich wrote: On 10.03.16 at 23:00, wrote: > First of all - please correctly Cc maintainers (there were two recent > changes for ARM). > >> --- a/xen/arch/arm/mm.c >> +++ b/xen/arch/arm/mm.c >> @@ -730,6 +730,7 @@ void __init setup_xenheap_mappings(unsigned long >> ba

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Peter Zijlstra
On Fri, Mar 11, 2016 at 01:48:12PM +0100, Juergen Gross wrote: > On 11/03/16 13:42, Peter Zijlstra wrote: > > how about something like: > > > > struct xen_callback_struct { > > struct work_struct work; > > struct completion done; int (*func)(void*); >

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Juergen Gross
On 11/03/16 13:57, Peter Zijlstra wrote: > On Fri, Mar 11, 2016 at 01:48:12PM +0100, Juergen Gross wrote: >> On 11/03/16 13:42, Peter Zijlstra wrote: >>> how about something like: >>> >>> struct xen_callback_struct { >>> struct work_struct work; >>> struct completion done; >

Re: [Xen-devel] [PATCH] xen: arm: zero EL2 pagetable pages before use

2016-03-11 Thread Jan Beulich
>>> On 11.03.16 at 13:56, wrote: > On 11/03/16 11:29, Jan Beulich wrote: > On 10.03.16 at 23:00, wrote: >>> @@ -771,6 +772,7 @@ void __init setup_frametable_mappings(paddr_t ps, >>> paddr_t pe) >>> nr_second = frametable_size >> SECOND_SHIFT; >>> second_base = alloc_boot_pages(nr_s

Re: [Xen-devel] [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions

2016-03-11 Thread Daniel Kiper
On Fri, Mar 11, 2016 at 01:27:32PM +0100, Vladimir 'phcoder' Serbinenko wrote: > On Wednesday, March 9, 2016, Daniel Kiper wrote: > > > On Wed, Mar 02, 2016 at 05:51:36PM +0100, Daniel Kiper wrote: > > > Hi, > > > > > > This patch series: > > > - enables EFI boot services usage in loaded images

Re: [Xen-devel] [PATCH] xen: sched: Add Meng as RTDS maintainer.

2016-03-11 Thread Xu, Quan
On March 10, 2016 11:56pm, Wei Liu wrote: > On Thu, Mar 10, 2016 at 10:08:04AM -0500, Meng Xu wrote: > > On Thu, Mar 10, 2016 at 9:42 AM, Dario Faggioli > > wrote: > > > Meng Xu is one of the maintainers of the RT-Xen project, which is > > > from where the RTDS scheduler comes. He also is the main

Re: [Xen-devel] [PATCH 0/6] Support calling functions on dedicated physical cpu

2016-03-11 Thread One Thousand Gnomes
On Fri, 11 Mar 2016 13:25:14 +0100 Peter Zijlstra wrote: > On Fri, Mar 11, 2016 at 12:59:28PM +0100, Juergen Gross wrote: > > Some hardware (e.g. Dell Studio laptops) require special functions to > > be called on physical cpu 0 in order to avoid occasional hangs. When > > running as dom0 under Xe

Re: [Xen-devel] [PATCH 0/6] Support calling functions on dedicated physical cpu

2016-03-11 Thread Peter Zijlstra
On Fri, Mar 11, 2016 at 01:15:04PM +, One Thousand Gnomes wrote: > On Fri, 11 Mar 2016 13:25:14 +0100 > Peter Zijlstra wrote: > > > On Fri, Mar 11, 2016 at 12:59:28PM +0100, Juergen Gross wrote: > > > Some hardware (e.g. Dell Studio laptops) require special functions to > > > be called on phy

Re: [Xen-devel] Request to revert superpage adjustments

2016-03-11 Thread Andrew Cooper
On 11/03/16 12:06, Jan Beulich wrote: On 11.03.16 at 10:22, wrote: >> Sadly, c/s cf393624 "x86: use 2M superpages for text/data/bss mappings" >> exposes a bug in all Syslinux variants, including ISOlinux and >> PXELinux, causing a failure to boot. >> >> Xen currently requires its bootloader

Re: [Xen-devel] [GRUB2 PATCH v3 1/4] i386/relocator: Add grub_relocator64_efi relocator

2016-03-11 Thread Daniel Kiper
On Thu, Mar 10, 2016 at 09:23:23PM +0100, Vladimir 'phcoder' Serbinenko wrote: > On Wednesday, March 2, 2016, Daniel Kiper wrote: > > > Add grub_relocator64_efi relocator. It will be used on EFI 64-bit platforms > > when multiboot2 compatible image requests MULTIBOOT_TAG_TYPE_EFI_BS. > > Relocator

Re: [Xen-devel] [PATCH] xen: arm: zero EL2 pagetable pages before use

2016-03-11 Thread Andrew Cooper
On 11/03/16 13:13, Jan Beulich wrote: On 11.03.16 at 13:56, wrote: >> On 11/03/16 11:29, Jan Beulich wrote: >> On 10.03.16 at 23:00, wrote: @@ -771,6 +772,7 @@ void __init setup_frametable_mappings(paddr_t ps, paddr_t pe) nr_second = frametable_size >> SECOND_SHIFT;

Re: [Xen-devel] [GRUB2 PATCH v3 2/4] multiboot2: Add tags used to pass ImageHandle to loaded image

2016-03-11 Thread Daniel Kiper
On Thu, Mar 10, 2016 at 09:26:06PM +0100, Vladimir 'phcoder' Serbinenko wrote: > On Wednesday, March 2, 2016, Daniel Kiper wrote: > > > Add tags used to pass ImageHandle to loaded image if requested. > > It is used by at least ExitBootServices() function. > > > > Signed-off-by: Daniel Kiper > > >

Re: [Xen-devel] [GRUB2 PATCH v3 3/4] multiboot2: Do not pass memory maps to image if EFI boot services are enabled

2016-03-11 Thread Daniel Kiper
On Thu, Mar 10, 2016 at 09:28:25PM +0100, Vladimir 'phcoder' Serbinenko wrote: > On Wednesday, March 2, 2016, Daniel Kiper wrote: > > > Do not pass memory maps to image if it asked for EFI boot services. > > Main reason for not providing maps is because they will likely be > > invalid. We do a few

Re: [Xen-devel] [PATCH] xen: sched: Add Meng as RTDS maintainer.

2016-03-11 Thread Meng Xu
Hi Wei and Quan, On Fri, Mar 11, 2016 at 8:14 AM, Xu, Quan wrote: > On March 10, 2016 11:56pm, Wei Liu wrote: >> On Thu, Mar 10, 2016 at 10:08:04AM -0500, Meng Xu wrote: >> > On Thu, Mar 10, 2016 at 9:42 AM, Dario Faggioli >> > wrote: >> > > Meng Xu is one of the maintainers of the RT-Xen projec

Re: [Xen-devel] [PATCH v3 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization

2016-03-11 Thread Dario Faggioli
On Fri, 2016-03-11 at 12:36 +, Xu, Quan wrote: > On March 11, 2016 6:36pm, wrote: > > On Fri, 2016-03-11 at 06:54 +, Xu, Quan wrote: > > >  > > So, now, if we know for sure that a lock is _never_ever_ever_ taken > > from > > interrupt context, can we mix spin_lock() and spin_lock_irq() on

Re: [Xen-devel] Request to revert superpage adjustments

2016-03-11 Thread Jan Beulich
>>> On 11.03.16 at 14:21, wrote: > On 11/03/16 12:06, Jan Beulich wrote: > On 11.03.16 at 10:22, wrote: >>> Sadly, c/s cf393624 "x86: use 2M superpages for text/data/bss mappings" >>> exposes a bug in all Syslinux variants, including ISOlinux and >>> PXELinux, causing a failure to boot. >>>

Re: [Xen-devel] Request to revert superpage adjustments

2016-03-11 Thread Konrad Rzeszutek Wilk
> >> Nor would > >> that deal with avoiding reserved regions not too far above 1Mb, > >> since at best the main payload can be relocatable (but certainly > >> not the binary seen by the boot loader, as at least multiboot1 > >> doesn't support anything like that). > > > > The only reason Xen sits

Re: [Xen-devel] [PATCH v3 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization

2016-03-11 Thread Meng Xu
Hi Quan and Dario, On Fri, Mar 11, 2016 at 5:35 AM, Dario Faggioli wrote: > On Fri, 2016-03-11 at 06:54 +, Xu, Quan wrote: >> On March 11, 2016 11:25am, wrote: >> > >> > On Wed, Mar 9, 2016 at 8:17 AM, Quan Xu wrote: >> > > >> > > pcidevs_lock should be held with interrupt enabled. However

Re: [Xen-devel] [PATCH v3 2/3] tools: change checkpointed_stream's type from int to xc_migration_stream_t

2016-03-11 Thread Wei Liu
On Fri, Mar 11, 2016 at 01:39:03PM +0800, Wen Congyang wrote: > checkpointed_stream is also renamed to stream_type > > Signed-off-by: Wen Congyang Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v3 1/3] libxc: move migration_stream's definition to xenguest.h

2016-03-11 Thread Wei Liu
On Fri, Mar 11, 2016 at 01:39:02PM +0800, Wen Congyang wrote: > xc_domain_save() and xc_domain_restore's parameter will use this type, > so it should be public. > > Signed-off-by: Wen Congyang Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@li

Re: [Xen-devel] [PATCH v3 3/3] libxl: rename checkpointed_stream to stream_type

2016-03-11 Thread Wei Liu
On Fri, Mar 11, 2016 at 01:39:04PM +0800, Wen Congyang wrote: > Signed-off-by: Wen Congyang Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] Request to revert superpage adjustments

2016-03-11 Thread Andrew Cooper
On 11/03/16 14:13, Jan Beulich wrote: On 11.03.16 at 14:21, wrote: >> On 11/03/16 12:06, Jan Beulich wrote: >> On 11.03.16 at 10:22, wrote: Sadly, c/s cf393624 "x86: use 2M superpages for text/data/bss mappings" exposes a bug in all Syslinux variants, including ISOlinux and >>>

Re: [Xen-devel] help

2016-03-11 Thread Konrad Rzeszutek Wilk
On Thu, Mar 10, 2016 at 09:04:22PM +0100, Safa Hamza wrote: > hello > i'm trying to run xen on omap5 following > this > http://wiki.xenproject.org/wiki/Xen_ARM_with_Virtualization_Extensions/OMAP5432_uEVM > > the execution stops at this point > > **

[Xen-devel] [PATCH] vmx: Restore debug registers when injecting #DB traps

2016-03-11 Thread Ross Lagerwall
Commit a929bee0e652 ("x86/vmx: Fix injection of #DB traps following XSA-156") prevents an infinite loop in certain #DB traps. However, it changed the behavior to not call hvm_hw_inject_trap() for #DB and #AC traps which which means that the debug registers are not restored correctly and nullified c

Re: [Xen-devel] [PATCH] flask: change default state to enforcing

2016-03-11 Thread Konrad Rzeszutek Wilk
On Fri, Mar 11, 2016 at 02:07:11AM -0700, Jan Beulich wrote: > >>> On 10.03.16 at 19:30, wrote: > > This change will cause the boot to fail if you do not specify an XSM > > policy during boot; if you need to load a policy from dom0, use the > > "flask=late" boot parameter. > > And what mode is th

Re: [Xen-devel] [PATCH v3 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization

2016-03-11 Thread Meng Xu
> >> In fact, the deadlock arises because IRQs interrupt asynchronously what the >> CPU is doing, and that can happen when the CPU has taken the lock already. >> But >> if the 'asynchronous' part goes away, we really don't care whether a lock is >> take >> at time t1 with IRQ enabled, and at time

[Xen-devel] [PULL 17/53] msi_supported -> msi_nonbroken

2016-03-11 Thread Michael S. Tsirkin
Rename controller flag to make it clearer what it means. Add some documentation as well. Signed-off-by: Michael S. Tsirkin --- include/hw/pci/msi.h | 2 +- hw/i386/kvm/apic.c | 2 +- hw/i386/xen/xen_apic.c | 2 +- hw/intc/apic.c | 2 +- hw/intc/a

Re: [Xen-devel] help

2016-03-11 Thread Konrad Rzeszutek Wilk
On Fri, Mar 11, 2016 at 10:20:01AM -0500, Konrad Rzeszutek Wilk wrote: > On Fri, Mar 11, 2016 at 04:05:58PM +0100, Safa Hamza wrote: And please do not drop Xen-devel. Adding it back on. > > i did like u said but nothing change .. > > > > No you didn't. See below: > > U-Boot# setenv dom0_bootarg

[Xen-devel] [qemu-mainline baseline-only test] 44241: regressions - FAIL

2016-03-11 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 44241 qemu-mainline real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/44241/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-pvops 5 kernel-build

Re: [Xen-devel] [GRUB2 PATCH v3 2/4] multiboot2: Add tags used to pass ImageHandle to loaded image

2016-03-11 Thread Vladimir 'phcoder' Serbinenko
On Friday, March 11, 2016, Daniel Kiper wrote: > On Thu, Mar 10, 2016 at 09:26:06PM +0100, Vladimir 'phcoder' Serbinenko > wrote: > > On Wednesday, March 2, 2016, Daniel Kiper > wrote: > > > > > Add tags used to pass ImageHandle to loaded image if requested. > > > It is used by at least ExitBoot

Re: [Xen-devel] [PATCH] flask: change default state to enforcing

2016-03-11 Thread Daniel De Graaf
On 03/11/2016 04:07 AM, Jan Beulich wrote: On 10.03.16 at 19:30, wrote: This change will cause the boot to fail if you do not specify an XSM policy during boot; if you need to load a policy from dom0, use the "flask=late" boot parameter. And what mode is the system in until that happens? From

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

2016-03-11 Thread osstest service owner
flight 85893 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/85893/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-amd64-xl-multivcpu 3 host-install(3) broken in 85820 pass in 85893 test-armhf-armhf-libvirt-qcow2

Re: [Xen-devel] [GRUB2 PATCH v3 1/4] i386/relocator: Add grub_relocator64_efi relocator

2016-03-11 Thread Vladimir 'phcoder' Serbinenko
On Friday, March 11, 2016, Daniel Kiper wrote: > On Thu, Mar 10, 2016 at 09:23:23PM +0100, Vladimir 'phcoder' Serbinenko > wrote: > > On Wednesday, March 2, 2016, Daniel Kiper > wrote: > > > > > Add grub_relocator64_efi relocator. It will be used on EFI 64-bit > platforms > > > when multiboot2 c

Re: [Xen-devel] [PATCH] vmx: Restore debug registers when injecting #DB traps

2016-03-11 Thread Jan Beulich
>>> On 11.03.16 at 15:51, wrote: > --- a/xen/arch/x86/hvm/vmx/vmx.c > +++ b/xen/arch/x86/hvm/vmx/vmx.c > @@ -3091,24 +3091,23 @@ static int vmx_handle_eoi_write(void) > * It is the callers responsibility to ensure that this function is only used > * in the context of an appropriate vmexit. >

Re: [Xen-devel] [PATCH] flask: change default state to enforcing

2016-03-11 Thread Jan Beulich
>>> On 11.03.16 at 16:39, wrote: > On 03/11/2016 04:07 AM, Jan Beulich wrote: > On 10.03.16 at 19:30, wrote: >>> This change will cause the boot to fail if you do not specify an XSM >>> policy during boot; if you need to load a policy from dom0, use the >>> "flask=late" boot parameter. >> >>

Re: [Xen-devel] [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions

2016-03-11 Thread Vladimir 'phcoder' Serbinenko
On Friday, March 11, 2016, Daniel Kiper wrote: > On Fri, Mar 11, 2016 at 01:27:32PM +0100, Vladimir 'phcoder' Serbinenko > wrote: > > On Wednesday, March 9, 2016, Daniel Kiper > wrote: > > > > > On Wed, Mar 02, 2016 at 05:51:36PM +0100, Daniel Kiper wrote: > > > > Hi, > > > > > > > > This patch

Re: [Xen-devel] [GRUB2 PATCH v3 1/4] i386/relocator: Add grub_relocator64_efi relocator

2016-03-11 Thread Vladimir 'phcoder' Serbinenko
On Friday, March 11, 2016, Daniel Kiper wrote: > On Thu, Mar 10, 2016 at 09:23:23PM +0100, Vladimir 'phcoder' Serbinenko > wrote: > > On Wednesday, March 2, 2016, Daniel Kiper > wrote: > > > > > Add grub_relocator64_efi relocator. It will be used on EFI 64-bit > platforms > > > when multiboot2 c

Re: [Xen-devel] [PATCH v3 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization

2016-03-11 Thread Dario Faggioli
On Fri, 2016-03-11 at 09:49 -0500, Meng Xu wrote: > > Yes. > > Consistency may be helpful to avoid some easy-to-avoid lock errors. > > Moreover, without my fix, I think it would not lead dead lock, as > > the pcidevs_lock is not being taken > > In IRQ context. Right? > I think without your fix, the

Re: [Xen-devel] [PATCH] flask: change default state to enforcing

2016-03-11 Thread Daniel De Graaf
On 03/11/2016 10:43 AM, Jan Beulich wrote: On 11.03.16 at 16:39, wrote: On 03/11/2016 04:07 AM, Jan Beulich wrote: On 10.03.16 at 19:30, wrote: This change will cause the boot to fail if you do not specify an XSM policy during boot; if you need to load a policy from dom0, use the "flask=late

[Xen-devel] [PATCH] x86emul: special case far branch validation outside of long mode

2016-03-11 Thread Jan Beulich
In that case (with the new value being held in, or now in one case cast to, a 32-bit variable) there's no need to go through the long mode part of the checks. Primarily this was meant to hopefully address Coverity ID 1355278, but since the change produces smaller code as well I think we should use

Re: [Xen-devel] help

2016-03-11 Thread Konrad Rzeszutek Wilk
On Fri, Mar 11, 2016 at 04:47:47PM +0100, Safa Hamza wrote: > now i did just like u said ... a new error appears Adding XEn-devel back. Please reply all. > ** > U-Boot# fdt addr $dtb_addr_r > U-Boot# fdt resize > U-Boot#

Re: [Xen-devel] [GRUB2 PATCH v3 4/4] multiboot2: Add support for relocatable images

2016-03-11 Thread Daniel Kiper
On Thu, Mar 10, 2016 at 09:41:26PM +0100, Vladimir 'phcoder' Serbinenko wrote: > On Wednesday, March 2, 2016, Daniel Kiper wrote: > > > Currently multiboot2 protocol loads image exactly at address specified in > > ELF or multiboot2 header. This solution works quite well on legacy BIOS > > platform

Re: [Xen-devel] help

2016-03-11 Thread Safa Hamza
i did just like u said ... a new error appears ** U-Boot# fdt addr $dtb_addr_r U-Boot# fdt resize U-Boot# fdt set /chosen xen,xen-bootargs \"$xen_bootargs\" U-Boot# fdt resize U-Boot# fdt set /chosen xen,dom0-bootargs \"$d

Re: [Xen-devel] help

2016-03-11 Thread Wei Liu
On Fri, Mar 11, 2016 at 11:02:26AM -0500, Konrad Rzeszutek Wilk wrote: > On Fri, Mar 11, 2016 at 04:47:47PM +0100, Safa Hamza wrote: > > now i did just like u said ... a new error appears > > Adding XEn-devel back. Please reply all. > > > *

Re: [Xen-devel] [PATCH v3 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization

2016-03-11 Thread Dario Faggioli
On Fri, 2016-03-11 at 09:41 -0500, Meng Xu wrote: > Thank you very much for your explanation and education! They are > really helpful! :-D > > Let me summarize: ;-) > > > >  | spin_lock | > spin_lock_irq | spin_lock_irqsave > > > > Can run in i

Re: [Xen-devel] [GRUB2 PATCH v3 4/4] multiboot2: Add support for relocatable images

2016-03-11 Thread Vladimir 'phcoder' Serbinenko
> > > > > + if (relocatable) > > > +{ > > > + if (base_addr > min_addr) > > > + grub_multiboot_payload_eip += base_addr - min_addr; > > > + else > > > + grub_multiboot_payload_eip -= min_addr - base_addr; > > > +} > > > + > > > > > Why is it relative to min_addr? Soun

Re: [Xen-devel] [PATCH v3 3/3] libxl: rename checkpointed_stream to stream_type

2016-03-11 Thread Wei Liu
On Fri, Mar 11, 2016 at 01:39:04PM +0800, Wen Congyang wrote: > Signed-off-by: Wen Congyang > --- > tools/libxl/libxl.c | 4 ++-- > tools/libxl/libxl.h | 8 > tools/libxl/libxl_create.c | 4 ++-- > tools/libxl/libxl_dom_save.c | 6 +++--- > tools/l

Re: [Xen-devel] [GRUB2 PATCH v3 4/4] multiboot2: Add support for relocatable images

2016-03-11 Thread Daniel Kiper
On Thu, Mar 10, 2016 at 09:44:31PM +0100, Vladimir 'phcoder' Serbinenko wrote: > On Wednesday, March 2, 2016, Daniel Kiper wrote: > > > Currently multiboot2 protocol loads image exactly at address specified in > > ELF or multiboot2 header. This solution works quite well on legacy BIOS > > platform

Re: [Xen-devel] [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions

2016-03-11 Thread Daniel Kiper
On Fri, Mar 11, 2016 at 04:44:00PM +0100, Vladimir 'phcoder' Serbinenko wrote: [...] > multiboot2 spec is a rolling update. OK. So, I think that we should do this in that way: - apply this patch series after polishing; I hope that it will happen next week, - update multiboot2 doc, - ad

[Xen-devel] [PATCH v2] vmx: Restore debug registers when injecting #DB traps

2016-03-11 Thread Ross Lagerwall
Commit a929bee0e652 ("x86/vmx: Fix injection of #DB traps following XSA-156") prevents an infinite loop in certain #DB traps. However, it changed the behavior to not call hvm_hw_inject_trap() for #DB and #AC traps which which means that the debug registers are not restored correctly and nullified c

Re: [Xen-devel] [GRUB2 PATCH v3 1/4] i386/relocator: Add grub_relocator64_efi relocator

2016-03-11 Thread Daniel Kiper
On Fri, Mar 11, 2016 at 04:42:27PM +0100, Vladimir 'phcoder' Serbinenko wrote: > On Friday, March 11, 2016, Daniel Kiper wrote: > > > On Thu, Mar 10, 2016 at 09:23:23PM +0100, Vladimir 'phcoder' Serbinenko > > wrote: > > > On Wednesday, March 2, 2016, Daniel Kiper > > wrote: > > > > > > > Add gru

Re: [Xen-devel] [PATCH v2 1/2] x86/msr: Carry on after a non-"safe" MSR access fails without !panic_on_oops

2016-03-11 Thread Andy Lutomirski
On Thu, Oct 1, 2015 at 12:15 AM, Ingo Molnar wrote: > > * Andy Lutomirski wrote: > >> > These could still be open coded in an inlined fashion, like the scheduler >> > usage. >> >> We could have a raw_rdmsr for those. >> >> OTOH, I'm still not 100% convinced that this warn-but-don't-die behavior

Re: [Xen-devel] Interested to participate in Outreachy Program

2016-03-11 Thread sabiya kazi
Hi Doug, Can you tell me what steps I should follow for this project? Regards, Sabiya On Mar 5, 2016 11:59 PM, "sabiya kazi" wrote: > > Hi Doug, > > I would like to work on the project *Rust Bindings for libxl *as a part > of Outreachy Program 2016. > > I have completed my B.E in Computer enginee

Re: [Xen-devel] help

2016-03-11 Thread Wei Liu
Add back xen-devel On Fri, Mar 11, 2016 at 05:23:22PM +0100, Safa Hamza wrote: > ok .. can u tell me how compile xen with debug symbols !! i have xen-syms > after compiling xen with "make dist-xen XEN_TARGET_ARCH=arm32 > CROSS_COMPILE=arm-linux-gnueabihf- CONFIG_EARLY_PRINTK=omap5432" is this >

Re: [Xen-devel] [PATCH v3 1/2] IOMMU/spinlock: Fix a bug found in AMD IOMMU initialization

2016-03-11 Thread Meng Xu
On Fri, Mar 11, 2016 at 10:55 AM, Dario Faggioli wrote: > On Fri, 2016-03-11 at 09:49 -0500, Meng Xu wrote: >> > Yes. >> > Consistency may be helpful to avoid some easy-to-avoid lock errors. >> > Moreover, without my fix, I think it would not lead dead lock, as >> > the pcidevs_lock is not being t

[Xen-devel] [PATCH 0/3] x86: instruction emulator improvements

2016-03-11 Thread Jan Beulich
1: x86: rename XMM* features to SSE* 2: x86emul: check host features alongside guest ones where needed 3: x86emul: support MOVBE and CRC32 Signed-off-by: Jan Beulich ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions

2016-03-11 Thread Vladimir 'phcoder' Serbinenko
On Friday, March 11, 2016, Daniel Kiper wrote: > On Fri, Mar 11, 2016 at 04:44:00PM +0100, Vladimir 'phcoder' Serbinenko > wrote: > > [...] > > > multiboot2 spec is a rolling update. > > OK. So, I think that we should do this in that way: > - apply this patch series after polishing; > I hop

[Xen-devel] [PATCH 1/3] x86: rename XMM* features to SSE*

2016-03-11 Thread Jan Beulich
The latter are their canonical names, used already in the instruction emulator. Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper --- a/xen/arch/x86/cpu/common.c +++ b/xen/arch/x86/cpu/common.c @@ -205,12 +205,12 @@ static void __init early_cpu_detect(void c->x86_model += ((

[Xen-devel] [PATCH 2/3] x86emul: check host features alongside guest ones where needed

2016-03-11 Thread Jan Beulich
Signed-off-by: Jan Beulich --- a/xen/arch/x86/x86_emulate/x86_emulate.c +++ b/xen/arch/x86/x86_emulate/x86_emulate.c @@ -1093,6 +1093,22 @@ static bool_t vcpu_has( #define vcpu_must_have_cx16() vcpu_must_have(0x0001, ECX, 13) #define vcpu_must_have_avx() vcpu_must_have(0x0001, ECX, 28)

[Xen-devel] [PATCH 3/3] x86emul: support MOVBE and CRC32

2016-03-11 Thread Jan Beulich
The former in an attempt to at least gradually support all simple data movement instructions. The latter just because it shares the opcode with the former. Signed-off-by: Jan Beulich --- a/tools/tests/x86_emulator/test_x86_emulator.c +++ b/tools/tests/x86_emulator/test_x86_emulator.c @@ -78,7 +7

Re: [Xen-devel] [PATCH 2/3] x86emul: check host features alongside guest ones where needed

2016-03-11 Thread Andrew Cooper
On 11/03/16 17:34, Jan Beulich wrote: > Signed-off-by: Jan Beulich > > --- a/xen/arch/x86/x86_emulate/x86_emulate.c > +++ b/xen/arch/x86/x86_emulate/x86_emulate.c > @@ -1093,6 +1093,22 @@ static bool_t vcpu_has( > #define vcpu_must_have_cx16() vcpu_must_have(0x0001, ECX, 13) > #define vcpu_m

Re: [Xen-devel] [GRUB2 PATCH v3 0/4] multiboot2: Add two extensions

2016-03-11 Thread Daniel Kiper
On Fri, Mar 11, 2016 at 06:33:15PM +0100, Vladimir 'phcoder' Serbinenko wrote: > On Friday, March 11, 2016, Daniel Kiper wrote: > > > On Fri, Mar 11, 2016 at 04:44:00PM +0100, Vladimir 'phcoder' Serbinenko > > wrote: > > > > [...] > > > > > multiboot2 spec is a rolling update. > > > > OK. So, I th

Re: [Xen-devel] Request to revert superpage adjustments

2016-03-11 Thread Daniel Kiper
On Fri, Mar 11, 2016 at 09:35:19AM -0500, Konrad Rzeszutek Wilk wrote: > > >> Nor would > > >> that deal with avoiding reserved regions not too far above 1Mb, > > >> since at best the main payload can be relocatable (but certainly > > >> not the binary seen by the boot loader, as at least multiboo

[Xen-devel] [PATCH v3 5/5] x86/msr: Set the return value to zero when native_rdmsr_safe fails

2016-03-11 Thread Andy Lutomirski
This will cause unchecked native_rdmsr_safe failures to return deterministic results. Signed-off-by: Andy Lutomirski --- arch/x86/include/asm/msr.h | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/arch/x86/include/asm/msr.h b/arch/x86/include/asm/msr.h index 13da359881d7..

[Xen-devel] [PATCH v3 0/5] Improve non-"safe" MSR access failure handling

2016-03-11 Thread Andy Lutomirski
Setting CONFIG_PARAVIRT=y has an unintended side effect: it silently turns all rdmsr and wrmsr operations into the safe variants without any checks that the operations actually succeed. With CONFIG_PARAVIRT=n, unchecked MSR failures OOPS and probably cause boot to fail if they happen before init s

  1   2   >