Re: [Xen-devel] [PATCH v2 1/2] hvm/vmx: save dr7 during vmx_vmcs_save

2016-02-11 Thread Razvan Cojocaru
On 02/12/2016 02:22 AM, Tamas K Lengyel wrote: > Sending the dr7 register during vm_events is useful for various applications, > but the current way the register value is gathered is incorrent. In this patch > we extend vmx_vmcs_save so that we get the correct value. > > Suggested-by: Andrew Coope

Re: [Xen-devel] [PATCH v2 2/2] vm_event: consolidate hvm_event_fill_regs and p2m_vm_event_fill_regs

2016-02-11 Thread Razvan Cojocaru
On 02/12/2016 02:22 AM, Tamas K Lengyel wrote: > Currently the registers saved in the request depend on which type of event > is filling in the registers. In this patch we consolidate the two versions > of register filling function as to return a fix set of registers irrespective > of the underlyin

Re: [Xen-devel] [PATCH v2 1/6] xen: factor out p2m list allocation into separate function

2016-02-11 Thread Juergen Gross
On 11/02/16 18:09, Daniel Kiper wrote: > On Thu, Feb 11, 2016 at 01:38:10PM +0100, Juergen Gross wrote: >> On 11/02/16 13:19, Daniel Kiper wrote: >>> On Thu, Feb 11, 2016 at 08:53:21AM +0100, Juergen Gross wrote: Do the p2m list allocation of the to be loaded kernel in a separate function

Re: [Xen-devel] [PATCH v2 3/6] xen: factor out allocation of page tables into separate function

2016-02-11 Thread Juergen Gross
On 11/02/16 18:14, Daniel Kiper wrote: > On Thu, Feb 11, 2016 at 01:53:51PM +0100, Juergen Gross wrote: >> On 11/02/16 13:27, Daniel Kiper wrote: >>> On Thu, Feb 11, 2016 at 08:53:23AM +0100, Juergen Gross wrote: Do the allocation of page tables in a separate function. This will allow to

Re: [Xen-devel] [PATCH v2 4/6] xen: add capability to load initrd outside of initial mapping

2016-02-11 Thread Juergen Gross
On 11/02/16 18:25, Daniel Kiper wrote: > On Thu, Feb 11, 2016 at 03:13:40PM +0100, Juergen Gross wrote: >> On 11/02/16 13:33, Daniel Kiper wrote: >>> On Thu, Feb 11, 2016 at 08:53:24AM +0100, Juergen Gross wrote: Modern pvops linux kernels support an initrd not covered by the initial mapp

Re: [Xen-devel] [PATCH v2 3/7] xen/vm-events: Move monitor_domctl to common-side.

2016-02-11 Thread Corneliu ZUZU
On 2/11/2016 5:44 PM, Tamas K Lengyel wrote: * the #ifdefs make it possible for that code to be put in common => that makes it *clear* that those code parts are NOT architecture specific and their implementation can be safely used for all architectures. The current practice has

[Xen-devel] [linux-next test] 81763: regressions - trouble: blocked/broken/fail/pass

2016-02-11 Thread osstest service owner
flight 81763 linux-next real [real] http://logs.test-lab.xenproject.org/osstest/logs/81763/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-debianhvm-amd64-xsm 15 guest-localmigrate/x10 fail REGR. vs. 81424 test-ar

[Xen-devel] [PATCH v3 3/5] build: remove .config from /boot when uninstalling.

2016-02-11 Thread Konrad Rzeszutek Wilk
c/s 361b4f9f0f0d4adc19df428e224a7b8fa62cd392 "build: save generated xen .config" forgot to remove the config file when uninstalling. CC: ian.campb...@citrix.com CC: jbeul...@suse.com CC: car...@cardoe.com CC: ian.jack...@eu.citrix.com Signed-off-by: Konrad Rzeszutek Wilk --- xen/Makefile | 1 +

[Xen-devel] [PATCH v3 2/5] arm/config: Declare ELFSIZE_[32|64] respectively.

2016-02-11 Thread Konrad Rzeszutek Wilk
Otherwise any code that tries to use Elf_* macros instead of Elf32_ or Elf_64 fails to compile. CC: ian.campb...@citrix.com CC: wei.l...@citrix.com CC: stefano.stabell...@citrix.com Signed-off-by: Konrad Rzeszutek Wilk --- xen/include/asm-arm/config.h | 2 ++ 1 file changed, 2 insertions(+) dif

[Xen-devel] [PATCH v3 5/5] mkelf32: Close those file descriptors in the error paths.

2016-02-11 Thread Konrad Rzeszutek Wilk
While we are operating here we may as well fix some of the file descriptor leaks. Signed-off-by: Konrad Rzeszutek Wilk --- xen/arch/x86/boot/mkelf32.c | 28 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/xen/arch/x86/boot/mkelf32.c b/xen/arch/x86/boo

[Xen-devel] [PATCH v3 4/5] mkelf32: Remove the 32-bit hypervisor support.

2016-02-11 Thread Konrad Rzeszutek Wilk
We do not compile 32-bit hypervisor anymore so the code for the ELFCLASS32 is effectively dead. Signed-off-by: Konrad Rzeszutek Wilk --- xen/arch/x86/boot/mkelf32.c | 88 +++-- 1 file changed, 28 insertions(+), 60 deletions(-) diff --git a/xen/arch/x86/bo

[Xen-devel] [PATCH v3] Requisite paches for xSplice v3 (not yet posted).

2016-02-11 Thread Konrad Rzeszutek Wilk
This mismatch of patches came about as I was redoing the xSplice patches based on reviews and splitting and/or doing some extra things. The first patch: [PATCH v3 1/5] hypervisor/arm/keyhandler: Declare struct Allows one to include the keyhandler.h (ARM one) file without having extra header fi

[Xen-devel] [PATCH v3 1/5] hypervisor/arm/keyhandler: Declare struct cpu_user_regs;

2016-02-11 Thread Konrad Rzeszutek Wilk
in the keyhandler.h file. Otherwise on ARM builds if we just use the keyhandler file - the compile will fail. CC: ian.campb...@citrix.com CC: wei.l...@citrix.com CC: stefano.stabell...@citrix.com Signed-off-by: Konrad Rzeszutek Wilk --- xen/include/xen/keyhandler.h | 1 + 1 file changed, 1 inser

[Xen-devel] [linux-linus test] 81734: regressions - trouble: blocked/broken/fail/pass

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

[Xen-devel] [PATCH v2 2/2] vm_event: consolidate hvm_event_fill_regs and p2m_vm_event_fill_regs

2016-02-11 Thread Tamas K Lengyel
Currently the registers saved in the request depend on which type of event is filling in the registers. In this patch we consolidate the two versions of register filling function as to return a fix set of registers irrespective of the underlying event. Signed-off-by: Tamas K Lengyel Cc: Razvan Co

[Xen-devel] [PATCH v2 1/2] hvm/vmx: save dr7 during vmx_vmcs_save

2016-02-11 Thread Tamas K Lengyel
Sending the dr7 register during vm_events is useful for various applications, but the current way the register value is gathered is incorrent. In this patch we extend vmx_vmcs_save so that we get the correct value. Suggested-by: Andrew Cooper Signed-off-by: Tamas K Lengyel Cc: Jun Nakajima Cc:

[Xen-devel] [qemu-mainline test] 81760: regressions - FAIL

2016-02-11 Thread osstest service owner
flight 81760 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/81760/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64 5 xen-build fail REGR. vs. 79947 build-i386

Re: [Xen-devel] [PATCH] vm_event: consolidate hvm_event_fill_regs and p2m_vm_event_fill_regs

2016-02-11 Thread Lengyel, Tamas
On Thu, Feb 11, 2016 at 3:30 PM, Andrew Cooper wrote: > On 11/02/2016 22:25, Lengyel, Tamas wrote: > > > > On Thu, Feb 11, 2016 at 2:58 PM, Andrew Cooper > wrote: > >> On 11/02/2016 21:49, Razvan Cojocaru wrote: >> > On 02/11/2016 11:35 PM, Andrew Cooper wrote: >> >> On 11/02/2016 21:05, Tamas K

Re: [Xen-devel] [PATCH] vm_event: consolidate hvm_event_fill_regs and p2m_vm_event_fill_regs

2016-02-11 Thread Andrew Cooper
On 11/02/2016 22:25, Lengyel, Tamas wrote: > > > On Thu, Feb 11, 2016 at 2:58 PM, Andrew Cooper > mailto:andrew.coop...@citrix.com>> wrote: > > On 11/02/2016 21:49, Razvan Cojocaru wrote: > > On 02/11/2016 11:35 PM, Andrew Cooper wrote: > >> On 11/02/2016 21:05, Tamas K Lengyel wrote: >

Re: [Xen-devel] [PATCH] vm_event: consolidate hvm_event_fill_regs and p2m_vm_event_fill_regs

2016-02-11 Thread Lengyel, Tamas
On Thu, Feb 11, 2016 at 2:58 PM, Andrew Cooper wrote: > On 11/02/2016 21:49, Razvan Cojocaru wrote: > > On 02/11/2016 11:35 PM, Andrew Cooper wrote: > >> On 11/02/2016 21:05, Tamas K Lengyel wrote: > >> > >>> diff --git a/xen/arch/x86/vm_event.c b/xen/arch/x86/vm_event.c > >>> index 08d678a..fa5d

Re: [Xen-devel] [PATCH v4 01/17] Xen: ACPI: Hide UART used by Xen

2016-02-11 Thread Rafael J. Wysocki
On Thursday, February 11, 2016 04:04:14 PM Stefano Stabellini wrote: > On Wed, 10 Feb 2016, Rafael J. Wysocki wrote: > > On Tuesday, February 09, 2016 11:19:02 AM Stefano Stabellini wrote: > > > On Mon, 8 Feb 2016, Rafael J. Wysocki wrote: > > > > On Monday, February 08, 2016 10:57:01 AM Stefano St

Re: [Xen-devel] [PATCH] x86: Fix build following c/s 623c720f "x86: use CLFLUSHOPT when available"

2016-02-11 Thread Doug Goldstein
On 2/11/16 1:25 PM, Andrew Cooper wrote: > CentOS 7 gets into trouble when compiling Xen citing: > > flushtlb.c: Assembler messages: > flushtlb.c:149: Error: value of 256 too large for field of 1 bytes at 1 > > The line number is wrong, and the error message not helpful. It turns out > that

Re: [Xen-devel] [PATCH] vm_event: consolidate hvm_event_fill_regs and p2m_vm_event_fill_regs

2016-02-11 Thread Andrew Cooper
On 11/02/2016 21:49, Razvan Cojocaru wrote: > On 02/11/2016 11:35 PM, Andrew Cooper wrote: >> On 11/02/2016 21:05, Tamas K Lengyel wrote: >> >>> diff --git a/xen/arch/x86/vm_event.c b/xen/arch/x86/vm_event.c >>> index 08d678a..fa5d154 100644 >>> --- a/xen/arch/x86/vm_event.c >>> +++ b/xen/arch/x86/

Re: [Xen-devel] [PATCH] vm_event: consolidate hvm_event_fill_regs and p2m_vm_event_fill_regs

2016-02-11 Thread Razvan Cojocaru
On 02/11/2016 11:35 PM, Andrew Cooper wrote: > On 11/02/2016 21:05, Tamas K Lengyel wrote: > >> diff --git a/xen/arch/x86/vm_event.c b/xen/arch/x86/vm_event.c >> index 08d678a..fa5d154 100644 >> --- a/xen/arch/x86/vm_event.c >> +++ b/xen/arch/x86/vm_event.c >> @@ -122,6 +122,64 @@ void vm_event_se

Re: [Xen-devel] [PATCH] vm_event: consolidate hvm_event_fill_regs and p2m_vm_event_fill_regs

2016-02-11 Thread Andrew Cooper
On 11/02/2016 21:05, Tamas K Lengyel wrote: > diff --git a/xen/arch/x86/vm_event.c b/xen/arch/x86/vm_event.c > index 08d678a..fa5d154 100644 > --- a/xen/arch/x86/vm_event.c > +++ b/xen/arch/x86/vm_event.c > @@ -122,6 +122,64 @@ void vm_event_set_registers(struct vcpu *v, > vm_event_response_t *rs

Re: [Xen-devel] [PATCH] vm_event: Record FS_BASE/GS_BASE during events

2016-02-11 Thread Razvan Cojocaru
On 02/11/2016 11:12 PM, Tamas K Lengyel wrote: > > > On Thu, Feb 11, 2016 at 2:11 PM, Tamas K Lengyel > mailto:tamas.k.leng...@gmail.com>> wrote: > > > > On Thu, Feb 11, 2016 at 1:59 PM, Razvan Cojocaru > mailto:rcojoc...@bitdefender.com>> wrote: > > On 02/11/2016 10:38 PM, Ta

Re: [Xen-devel] [PATCH] vm_event: Record FS_BASE/GS_BASE during events

2016-02-11 Thread Tamas K Lengyel
On Thu, Feb 11, 2016 at 2:11 PM, Tamas K Lengyel wrote: > > > On Thu, Feb 11, 2016 at 1:59 PM, Razvan Cojocaru < > rcojoc...@bitdefender.com> wrote: > >> On 02/11/2016 10:38 PM, Tamas K Lengyel wrote: >> > >> > >> > On Thu, Feb 11, 2016 at 1:13 PM, Razvan Cojocaru >> > mailto:rcojoc...@bitdefende

Re: [Xen-devel] [PATCH] vm_event: Record FS_BASE/GS_BASE during events

2016-02-11 Thread Tamas K Lengyel
On Thu, Feb 11, 2016 at 1:59 PM, Razvan Cojocaru wrote: > On 02/11/2016 10:38 PM, Tamas K Lengyel wrote: > > > > > > On Thu, Feb 11, 2016 at 1:13 PM, Razvan Cojocaru > > mailto:rcojoc...@bitdefender.com>> wrote: > > > > On 02/11/2016 10:04 PM, Andrew Cooper wrote: > > > On 11/02/16 20:00,

[Xen-devel] [PATCH 3/4] xen/pcifront: Report the errors better.

2016-02-11 Thread Konrad Rzeszutek Wilk
The messages should be different depending on the type of error. Signed-off-by: Konrad Rzeszutek Wilk --- arch/x86/include/asm/xen/pci.h | 4 ++-- arch/x86/pci/xen.c | 5 - 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/arch/x86/include/asm/xen/pci.h b/arch/x86/in

[Xen-devel] [PATCH 2/4] xen/pciback: Save the number of MSI-X entries to be copied later.

2016-02-11 Thread Konrad Rzeszutek Wilk
c/s 8135cf8b092723dbfcc611fe6fdcb3a36c9951c5 "xen/pciback: Save xen_pci_op commands before processing it" would copyback the processed values - which was great. However it missed the case that xen_pcibk_enable_msix - when completing would overwrite op->value (which had the number of MSI-X vectors

[Xen-devel] [PATCH 1/4] xen/pciback: Check PF instead of VF for PCI_COMMAND_MEMORY

2016-02-11 Thread Konrad Rzeszutek Wilk
c/s 408fb0e5aa7fda0059db282ff58c3b2a4278baa0 "xen/pciback: Don't allow MSI-X ops if PCI_COMMAND_MEMORY is not set." would check the device for PCI_COMMAND_MEMORY which is great. Except that VF devices are unique - for example they have no legacy interrupts, and also any writes to PCI_COMMAND_MEMORY

[Xen-devel] [PATCH 4/4] xen/pcifront: Fix mysterious crashes when NUMA locality information was extracted.

2016-02-11 Thread Konrad Rzeszutek Wilk
Occasionaly PV guests would crash with: pciback :00:00.1: Xen PCI mapped GSI0 to IRQ16 BUG: unable to handle kernel paging request at 000d1a8c0be0 .. snip.. ] find_next_bit+0xb/0x10 [] cpumask_next_and+0x22/0x40 [] pci_device_probe+0xb8/0x120 [] ? driver_sysfs_add+0x77/0xa0 [] dr

[Xen-devel] [PATCH] Fixes to Xen pcifront and pciback (v1)

2016-02-11 Thread Konrad Rzeszutek Wilk
Hey, These are patches that were developed for the Debian bug 810379 which san Tommi had openned. The issue around from the two XSA fixes - which introduced this regression. I am the person who developed them and my explanation for this regression oversight is that I tested for the 'exploit use-

[Xen-devel] [PATCH] vm_event: consolidate hvm_event_fill_regs and p2m_vm_event_fill_regs

2016-02-11 Thread Tamas K Lengyel
Currently the registers saved in the request depend on which type of event is filling in the registers. In this patch we consolidate the two versions of register filling function as to return a fix set of registers irrespective of the underlying event. Signed-off-by: Tamas K Lengyel Cc: Razvan Co

Re: [Xen-devel] [PATCH] vm_event: Record FS_BASE/GS_BASE during events

2016-02-11 Thread Razvan Cojocaru
On 02/11/2016 10:38 PM, Tamas K Lengyel wrote: > > > On Thu, Feb 11, 2016 at 1:13 PM, Razvan Cojocaru > mailto:rcojoc...@bitdefender.com>> wrote: > > On 02/11/2016 10:04 PM, Andrew Cooper wrote: > > On 11/02/16 20:00, Razvan Cojocaru wrote: > >> On 02/11/2016 09:55 PM, Andrew Cooper

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

2016-02-11 Thread osstest service owner
flight 82022 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/82022/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 12 migrate-support-checkfail never pass test-armhf-armhf-xl 12

Re: [Xen-devel] [RFC v1 4/8] x86/init: add linker table support

2016-02-11 Thread Luis R. Rodriguez
On Thu, Jan 21, 2016 at 4:42 PM, H. Peter Anvin wrote: > On 01/21/16 16:25, Luis R. Rodriguez wrote: >>> >>> Basically, if the hardware is enumerable using standard PC mechanisms (PCI, >>> ACPI) and doesn't need a special boot flow it should use type 0. >> >> I can extend the documentation as par

Re: [Xen-devel] [PATCH] vm_event: Record FS_BASE/GS_BASE during events

2016-02-11 Thread Tamas K Lengyel
On Thu, Feb 11, 2016 at 1:13 PM, Razvan Cojocaru wrote: > On 02/11/2016 10:04 PM, Andrew Cooper wrote: > > On 11/02/16 20:00, Razvan Cojocaru wrote: > >> On 02/11/2016 09:55 PM, Andrew Cooper wrote: > >>> On 11/02/16 19:54, Razvan Cojocaru wrote: > On 02/11/2016 09:51 PM, Tamas K Lengyel wro

Re: [Xen-devel] [PATCH] vm_event: Record FS_BASE/GS_BASE during events

2016-02-11 Thread Razvan Cojocaru
On 02/11/2016 10:04 PM, Andrew Cooper wrote: > On 11/02/16 20:00, Razvan Cojocaru wrote: >> On 02/11/2016 09:55 PM, Andrew Cooper wrote: >>> On 11/02/16 19:54, Razvan Cojocaru wrote: On 02/11/2016 09:51 PM, Tamas K Lengyel wrote: > While the public vm_event header specifies fs_base/gs_base

Re: [Xen-devel] [PATCH] vm_event: Record FS_BASE/GS_BASE during events

2016-02-11 Thread Andrew Cooper
On 11/02/16 20:00, Razvan Cojocaru wrote: > On 02/11/2016 09:55 PM, Andrew Cooper wrote: >> On 11/02/16 19:54, Razvan Cojocaru wrote: >>> On 02/11/2016 09:51 PM, Tamas K Lengyel wrote: While the public vm_event header specifies fs_base/gs_base as registers that should be recorded fo

Re: [Xen-devel] [PATCH] vm_event: Record FS_BASE/GS_BASE during events

2016-02-11 Thread Razvan Cojocaru
On 02/11/2016 09:55 PM, Andrew Cooper wrote: > On 11/02/16 19:54, Razvan Cojocaru wrote: >> On 02/11/2016 09:51 PM, Tamas K Lengyel wrote: >>> While the public vm_event header specifies fs_base/gs_base as registers that >>> should be recorded for each event, that hasn't actually been the case. In >

Re: [Xen-devel] [RFC v1 6/8] x86/init: use linker table for i386 early setup

2016-02-11 Thread Luis R. Rodriguez
On Wed, Jan 20, 2016 at 1:41 PM, Luis R. Rodriguez wrote: > On Wed, Jan 20, 2016 at 1:14 PM, Konrad Rzeszutek Wilk > wrote: >> On Tue, Dec 15, 2015 at 02:16:35PM -0800, Luis R. Rodriguez wrote: >>> From: "Luis R. Rodriguez" >>> >>> This also annotates this is only used for PC and >>> lguest hard

Re: [Xen-devel] [PATCH] vm_event: Record FS_BASE/GS_BASE during events

2016-02-11 Thread Andrew Cooper
On 11/02/16 19:54, Razvan Cojocaru wrote: > On 02/11/2016 09:51 PM, Tamas K Lengyel wrote: >> While the public vm_event header specifies fs_base/gs_base as registers that >> should be recorded for each event, that hasn't actually been the case. In >> this patch we remedy the issue. >> >> Signed-off

Re: [Xen-devel] [PATCH] vm_event: Record FS_BASE/GS_BASE during events

2016-02-11 Thread Razvan Cojocaru
On 02/11/2016 09:51 PM, Tamas K Lengyel wrote: > While the public vm_event header specifies fs_base/gs_base as registers that > should be recorded for each event, that hasn't actually been the case. In > this patch we remedy the issue. > > Signed-off-by: Tamas K Lengyel > Cc: Razvan Cojocaru > C

[Xen-devel] [PATCH] xen/common: Uniformally use __ varients for attribute names

2016-02-11 Thread Andrew Cooper
Otherwise, debug code such as "void __attribute__((noreturn)) foobar()" fails to compile when the noreturn itself gets expanded, resulting in __attribute__((__attribute__((noreturn. No function change. Signed-off-by: Andrew Cooper --- CC: Jan Beulich CC: Tim Deegan CC: Ian Campbell --- x

[Xen-devel] [PATCH] vm_event: Record FS_BASE/GS_BASE during events

2016-02-11 Thread Tamas K Lengyel
While the public vm_event header specifies fs_base/gs_base as registers that should be recorded for each event, that hasn't actually been the case. In this patch we remedy the issue. Signed-off-by: Tamas K Lengyel Cc: Razvan Cojocaru Cc: Keir Fraser Cc: Jan Beulich Cc: Andrew Cooper --- xen/

Re: [Xen-devel] [PATCH] x86: Fix build following c/s 623c720f "x86: use CLFLUSHOPT when available"

2016-02-11 Thread Andrew Cooper
On 11/02/16 19:25, Andrew Cooper wrote: > CentOS 7 gets into trouble when compiling Xen citing: > > flushtlb.c: Assembler messages: > flushtlb.c:149: Error: value of 256 too large for field of 1 bytes at 1 > > The line number is wrong, and the error message not helpful. It turns out > that the

[Xen-devel] [PATCH] xen/x86: Drop uefi_call_wrapper() and EFI_FUNCTION_WRAPPER

2016-02-11 Thread Andrew Cooper
Nothing in the codebase uses them, and clang falls into the #error case Signed-off-by: Andrew Cooper --- CC: Jan Beulich --- xen/include/asm-x86/x86_64/efibind.h | 11 --- 1 file changed, 11 deletions(-) diff --git a/xen/include/asm-x86/x86_64/efibind.h b/xen/include/asm-x86/x86_64/ef

[Xen-devel] [PATCH] x86: Fix build following c/s 623c720f "x86: use CLFLUSHOPT when available"

2016-02-11 Thread Andrew Cooper
CentOS 7 gets into trouble when compiling Xen citing: flushtlb.c: Assembler messages: flushtlb.c:149: Error: value of 256 too large for field of 1 bytes at 1 The line number is wrong, and the error message not helpful. It turns out that the intermediate generated assembly was # 139 "arch/

[Xen-devel] [linux-mingo-tip-master test] 81646: regressions - FAIL

2016-02-11 Thread osstest service owner
flight 81646 linux-mingo-tip-master real [real] http://logs.test-lab.xenproject.org/osstest/logs/81646/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-rumpuserxen6 xen-build fail REGR. vs. 60684 build-amd6

Re: [Xen-devel] [PATCH] x86, microcode: Remove unnecessary paravirt_enabled check

2016-02-11 Thread Borislav Petkov
On Thu, Feb 11, 2016 at 10:13:18AM -0500, Boris Ostrovsky wrote: > Commit a18a0f6850d4 ("x86, microcode: Don't initialize microcode code on > paravirt") added a paravirt test in microcode_init(), primarily to avoid > making mc_bp_resume()->load_ucode_ap()->check_loader_disabled_ap() calls > On 32-b

[Xen-devel] [xen-4.6-testing baseline-only test] 38738: regressions - FAIL

2016-02-11 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 38738 xen-4.6-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38738/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt-raw 9 debian-di-in

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

2016-02-11 Thread osstest service owner
flight 81996 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/81996/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf 4 host-build-prep fail REGR. vs. 81826 Tests which di

Re: [Xen-devel] [PATCH v2 5/6] xen: modify page table construction

2016-02-11 Thread Daniel Kiper
On Thu, Feb 11, 2016 at 03:35:45PM +0100, Juergen Gross wrote: > On 11/02/16 13:47, Daniel Kiper wrote: > > On Thu, Feb 11, 2016 at 08:53:25AM +0100, Juergen Gross wrote: > >> Modify the page table construction to allow multiple virtual regions > >> to be mapped. This is done as preparation for rem

Re: [Xen-devel] [PATCH v3] travis: add initial Travis CI script to do builds

2016-02-11 Thread Doug Goldstein
On 2/7/16 8:45 PM, Doug Goldstein wrote: > This is just suppose to do a simple compile test on Travis CI. Currently > due to linux86 (bcc/bin86/dev86) not being whitelisted the tools cannot > be built. > > Signed-off-by: Doug Goldstein ping? -- Doug Goldstein signature.asc Description: Open

Re: [Xen-devel] [PATCH 2/3] xen: sched: add wakeup flags to the scheduler interface

2016-02-11 Thread Dario Faggioli
On Thu, 2016-02-11 at 06:24 -0700, Jan Beulich wrote: > > > > On 11.02.16 at 12:38, wrote: > > --- a/xen/include/xen/sched-if.h > > +++ b/xen/include/xen/sched-if.h > > @@ -144,7 +144,8 @@ struct scheduler { > >  void (*remove_vcpu)(const struct scheduler *, > > struct vcpu *); > >

Re: [Xen-devel] [PATCH 3/3] xen: credit1: avoid boosting vCPUs being "just" migrated

2016-02-11 Thread Dario Faggioli
On Thu, 2016-02-11 at 06:30 -0700, Jan Beulich wrote: > > > > On 11.02.16 at 12:39, wrote: > > --- a/xen/common/sched_credit.c > > +++ b/xen/common/sched_credit.c > > @@ -1022,11 +1022,14 @@ csched_vcpu_wake(const struct scheduler > > *ops, struct vcpu *vc, unsigned wf) > >   * more CPU resour

Re: [Xen-devel] [PATCH v3] travis: add initial Travis CI script to do builds

2016-02-11 Thread Doug Goldstein
On 2/9/16 6:33 PM, Dario Faggioli wrote: > On Tue, 2016-02-09 at 17:09 +, Andrew Cooper wrote: >> On 08/02/16 02:45, Doug Goldstein wrote: >>> This is just suppose to do a simple compile test on Travis CI. >>> Currently >>> due to linux86 (bcc/bin86/dev86) not being whitelisted the tools >>> ca

Re: [Xen-devel] [PATCH v2 4/6] xen: add capability to load initrd outside of initial mapping

2016-02-11 Thread Daniel Kiper
On Thu, Feb 11, 2016 at 03:13:40PM +0100, Juergen Gross wrote: > On 11/02/16 13:33, Daniel Kiper wrote: > > On Thu, Feb 11, 2016 at 08:53:24AM +0100, Juergen Gross wrote: > >> Modern pvops linux kernels support an initrd not covered by the initial > >> mapping. This capability is flagged by an elf-

Re: [Xen-devel] [PATCH v2 3/6] xen: factor out allocation of page tables into separate function

2016-02-11 Thread Daniel Kiper
On Thu, Feb 11, 2016 at 01:53:51PM +0100, Juergen Gross wrote: > On 11/02/16 13:27, Daniel Kiper wrote: > > On Thu, Feb 11, 2016 at 08:53:23AM +0100, Juergen Gross wrote: > >> Do the allocation of page tables in a separate function. This will > >> allow to do the allocation at different times of th

Re: [Xen-devel] [PATCH v2] tools: probe for existence of qemu-xen stderr trace backend.

2016-02-11 Thread Ian Campbell
On Thu, 2016-02-11 at 16:30 +, Wei Liu wrote: > On Thu, Feb 11, 2016 at 12:11:21PM +, Ian Campbell wrote: > > QEMU upstream commit ed7f5f1d8db0 ("trace: convert stderr backend to > > log") renamed the "stderr" trace backend to "log", which breaks the > > xen build when pointed at a QEMU tre

Re: [Xen-devel] [PATCH v2 1/6] xen: factor out p2m list allocation into separate function

2016-02-11 Thread Daniel Kiper
On Thu, Feb 11, 2016 at 01:38:10PM +0100, Juergen Gross wrote: > On 11/02/16 13:19, Daniel Kiper wrote: > > On Thu, Feb 11, 2016 at 08:53:21AM +0100, Juergen Gross wrote: > >> Do the p2m list allocation of the to be loaded kernel in a separate > >> function. This will allow doing the p2m list alloc

Re: [Xen-devel] [PATCH] tools/console: correct make dependencies for _paths.h

2016-02-11 Thread Ian Campbell
On Thu, 2016-02-11 at 16:03 +, Ian Campbell wrote: > On Thu, 2016-02-11 at 15:38 +, Olaf Hering wrote: > > Correct dependencies for _paths.h to avoid build failure with make -j. > > Only main.c requires _paths.h. This fixes commit 8398ec70 ("xenconsole: > > Ensure exclusive access to consol

Re: [Xen-devel] [BUG] gdbsx crashes Xen

2016-02-11 Thread Andrew Cooper
On 11/02/16 16:25, Carl Patenaude Poulin wrote: > Hey folks, > > With gdbsx running, Running `gdb -ex "target remote localhost:" > crashes Xen and reboots the server. > > Example session: http://i.imgur.com/UXh3RCy.png > > Minimal reproduction: https://github.com/lilred/xen-crash-repro > > Top

Re: [Xen-devel] [PATCH v2] tools: probe for existence of qemu-xen stderr trace backend.

2016-02-11 Thread Wei Liu
On Thu, Feb 11, 2016 at 12:11:21PM +, Ian Campbell wrote: > QEMU upstream commit ed7f5f1d8db0 ("trace: convert stderr backend to > log") renamed the "stderr" trace backend to "log", which breaks the > xen build when pointed at a QEMU tree after that point: > > ./configure of QEMU fail with: >

[Xen-devel] [BUG] gdbsx crashes Xen

2016-02-11 Thread Carl Patenaude Poulin
Hey folks, With gdbsx running, Running `gdb -ex "target remote localhost:" crashes Xen and reboots the server. Example session: http://i.imgur.com/UXh3RCy.png Minimal reproduction: https://github.com/lilred/xen-crash-repro Top line of `xl dmesg`: Xen version 4.6.0 (r...@cs.mcgill.ca) (

Re: [Xen-devel] [PATCH v2] tools: probe for existence of qemu-xen stderr trace backend.

2016-02-11 Thread Stefano Stabellini
On Thu, 11 Feb 2016, Ian Campbell wrote: > QEMU upstream commit ed7f5f1d8db0 ("trace: convert stderr backend to > log") renamed the "stderr" trace backend to "log", which breaks the > xen build when pointed at a QEMU tree after that point: > > ./configure of QEMU fail with: > "ERROR: invalid trace

Re: [Xen-devel] [PATCH] tools/console: correct make dependencies for _paths.h

2016-02-11 Thread Ian Campbell
On Thu, 2016-02-11 at 15:38 +, Olaf Hering wrote: > Correct dependencies for _paths.h to avoid build failure with make -j. > Only main.c requires _paths.h. This fixes commit 8398ec70 ("xenconsole: > Ensure exclusive access to console using locks") > > Signed-off-by: Olaf Hering > Cc: Ian Jack

Re: [Xen-devel] [PATCH v4 01/17] Xen: ACPI: Hide UART used by Xen

2016-02-11 Thread Stefano Stabellini
On Wed, 10 Feb 2016, Rafael J. Wysocki wrote: > On Tuesday, February 09, 2016 11:19:02 AM Stefano Stabellini wrote: > > On Mon, 8 Feb 2016, Rafael J. Wysocki wrote: > > > On Monday, February 08, 2016 10:57:01 AM Stefano Stabellini wrote: > > > > On Sat, 6 Feb 2016, Rafael J. Wysocki wrote: > > > >

Re: [Xen-devel] [PATCH 1/2] xenmon: close qos_fd when finished with it in alloc_qos_data

2016-02-11 Thread Ian Campbell
On Thu, 2016-02-11 at 11:18 +, Wei Liu wrote: > On Wed, Feb 10, 2016 at 04:26:24PM +, Ian Campbell wrote: > > Otherwise the fd is leaked. NB the mmap'd memory in the global > > cpu_qos_data[n] is not affected by closing the underlying fd. > > > > Compile tested only. > > > > CID: 1055930

Re: [Xen-devel] [PATCH v2] tools: libxl: make it illegal to pass libxl__realloc(gc) a non-gc ptr

2016-02-11 Thread Ian Campbell
On Thu, 2016-02-11 at 11:37 +, Wei Liu wrote: > On Thu, Feb 11, 2016 at 09:23:54AM +, Ian Campbell wrote: > > That is, if gc is not NOGC and ptr is not NULL then ptr must be > > associated gc. > > > > "associated with gc"? > > Anyway, I get the idea. Yeah, apparently I was having langua

[Xen-devel] [PATCH] tools/console: correct make dependencies for _paths.h

2016-02-11 Thread Olaf Hering
Correct dependencies for _paths.h to avoid build failure with make -j. Only main.c requires _paths.h. This fixes commit 8398ec70 ("xenconsole: Ensure exclusive access to console using locks") Signed-off-by: Olaf Hering Cc: Ian Jackson Cc: Stefano Stabellini Cc: Ian Campbell Cc: Wei Liu --- S

Re: [Xen-devel] [PATCH] public/io/netif.h: fix typos

2016-02-11 Thread Ian Campbell
On Thu, 2016-02-11 at 09:53 +, Ian Campbell wrote: > On Wed, 2016-02-10 at 16:49 +, Paul Durrant wrote: > > Unfortunately my patch 162a81ab "document control ring and toeplitz > > hashing" contained a couple of typos. This patch fixes them. > > > > Signed-off-by: Paul Durrant > > Cc: Ian

Re: [Xen-devel] [PATCH v3 1/2] build: specify minimum versions of gcc and binutils

2016-02-11 Thread Ian Campbell
On Thu, 2016-02-11 at 06:32 -0700, Jan Beulich wrote: > > > > On 11.02.16 at 14:21, wrote: > > On Thu, 2016-02-11 at 12:31 +, Andrew Cooper wrote: > > > On 11/02/16 12:23, Ian Campbell wrote: > > > > From: Doug Goldstein > > > > > > > > To help people avoid having to figure out what versions

Re: [Xen-devel] [PATCH v2 3/7] xen/vm-events: Move monitor_domctl to common-side.

2016-02-11 Thread Tamas K Lengyel
> > * the #ifdefs make it possible for that code to be put in common => that > makes it *clear* that those code parts are NOT > architecture specific and their implementation can be safely used for all > architectures. > The current practice has been to put empty static inline functions into archi

[Xen-devel] [PATCH v8 0/5] libbxl: add support for pvscsi, iteration 8

2016-02-11 Thread Olaf Hering
Port vscsi=[] and scsi-{attach,detach,list} commands from xend to libxl. libvirt uses its existing SCSI support: http://lists.xenproject.org/archives/html/xen-devel/2015-04/msg02963.html targetcli/rtslib has to be aware of xen-scsiback (upstream unresponsive): http://article.gmane.org/gmane.linux

[Xen-devel] [PATCH v8 4/5] vscsiif.h: add some notes about xenstore layout

2016-02-11 Thread Olaf Hering
Signed-off-by: Olaf Hering Acked-by: Ian Campbell Cc: Ian Campbell Cc: Ian Jackson Cc: Jan Beulich Cc: Keir Fraser Cc: Tim Deegan --- xen/include/public/io/vscsiif.h | 68 + 1 file changed, 68 insertions(+) diff --git a/xen/include/public/io/vscsiif.

[Xen-devel] [PATCH v8 5/5] Scripts to create and delete xen-scsiback nodes in Linux target framework

2016-02-11 Thread Olaf Hering
Just to make them public, not meant for merging: The scripts used during development to create a bunch of SCSI devices in dom0 using the Linux target framework. targetcli3 and rtslib3 is used. A patch is required for python-rtslib: http://article.gmane.org/gmane.linux.scsi.target.devel/8146 Signe

[Xen-devel] [PATCH v8 1/5] vscsiif.h: fix WWN notation for p-dev property

2016-02-11 Thread Olaf Hering
The pvops kernel expects either "naa.WWN:LUN" or "h:c:t:l" in the p-dev property. Add the missing :LUN part to the comment. Signed-off-by: Olaf Hering Acked-by: Ian Campbell Cc: Ian Campbell Cc: Ian Jackson Cc: Jan Beulich Cc: Keir Fraser Cc: Tim Deegan --- xen/include/public/io/vscsiif.h

[Xen-devel] [PATCH v8 3/5] libxl: add support for vscsi

2016-02-11 Thread Olaf Hering
Port pvscsi support from xend to libxl: vscsi=['pdev,vdev{,options}'] xl scsi-attach xl scsi-detach xl scsi-list Signed-off-by: Olaf Hering Cc: Ian Jackson Cc: Stefano Stabellini Cc: Ian Campbell Cc: Wei Liu --- docs/man/xl.cfg.pod.5| 55 +++ docs/man/xl.pod.1

[Xen-devel] [PATCH v8 2/5] docs: add vscsi to xenstore-paths.markdown

2016-02-11 Thread Olaf Hering
Signed-off-by: Olaf Hering Acked-by: Ian Campbell Cc: Ian Campbell Cc: Ian Jackson Cc: Jan Beulich Cc: Keir Fraser Cc: Tim Deegan --- docs/misc/xenstore-paths.markdown | 10 ++ 1 file changed, 10 insertions(+) diff --git a/docs/misc/xenstore-paths.markdown b/docs/misc/xenstore-pat

Re: [Xen-devel] [PATCH] init-xenstore-domain: cleanup all resources on a single exit path

2016-02-11 Thread Ian Campbell
On Thu, 2016-02-11 at 11:27 +, Wei Liu wrote: > On Wed, Feb 10, 2016 at 04:56:22PM +, Ian Campbell wrote: > > Previously xs_fd would be left open, which is CID 1055993 (previously > > partially fixed by 3bca826aae5eb). > > > > Instead arrange for both success and error cases to cleanup eve

Re: [Xen-devel] [PATCH v2] libxc: fix leak in xc_offline_page error path

2016-02-11 Thread Ian Campbell
On Wed, 2016-02-10 at 11:51 +, Ian Jackson wrote: > Harmandeep Kaur writes ("[PATCH v2] libxc: fix leak in xc_offline_page > error path"): > > Avoid leaking the mapping of the m2p in one of the possible failure > > cases. > > > > Coverity CID 1351225 > > > > Signed-off-by: Harmandeep Kaur >

[Xen-devel] [PATCH] x86, microcode: Remove unnecessary paravirt_enabled check

2016-02-11 Thread Boris Ostrovsky
Commit a18a0f6850d4 ("x86, microcode: Don't initialize microcode code on paravirt") added a paravirt test in microcode_init(), primarily to avoid making mc_bp_resume()->load_ucode_ap()->check_loader_disabled_ap() calls On 32-bit kernels this callchain ends up using __pa_nodebug() macro which is inv

Re: [Xen-devel] [PATCH v7 3/5] libxl: add support for vscsi

2016-02-11 Thread Olaf Hering
On Thu, Feb 11, Olaf Hering wrote: > I'm in the process to move the counter to the backend path. ... which fails due to libxl__device_exists(). So, back to /libxl Olaf ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v7 3/5] libxl: add support for vscsi

2016-02-11 Thread Olaf Hering
On Fri, Feb 05, Olaf Hering wrote: > +static int libxl__vscsictrl_next_vscsidev_id(libxl__gc *gc, uint32_t domid, > + libxl_device_vscsictrl *vscsi, > + libxl_devid *vscsidev_id) > +{ > +path = GCSPRINTF("

Re: [Xen-devel] [PATCH v2 5/6] xen: modify page table construction

2016-02-11 Thread Juergen Gross
On 11/02/16 13:47, Daniel Kiper wrote: > On Thu, Feb 11, 2016 at 08:53:25AM +0100, Juergen Gross wrote: >> Modify the page table construction to allow multiple virtual regions >> to be mapped. This is done as preparation for removing the p2m list >> from the initial kernel mapping in order to suppo

[Xen-devel] [PATCH v3 2/2] x86/traps: Dump instruction stream in show_execution_state()

2016-02-11 Thread Andrew Cooper
For first pass triage of crashes, it is useful to have the instruction stream present, especially now that Xen binary patches itself. A sample output now looks like: (XEN) [ Xen-4.7-unstable x86_64 debug=y Not tainted ] (XEN) CPU:0 (XEN) RIP:e008:[] default_idle+0x76/0x7b (XEN)

Re: [Xen-devel] [PATCH v2 4/6] xen: add capability to load initrd outside of initial mapping

2016-02-11 Thread Juergen Gross
On 11/02/16 13:33, Daniel Kiper wrote: > On Thu, Feb 11, 2016 at 08:53:24AM +0100, Juergen Gross wrote: >> Modern pvops linux kernels support an initrd not covered by the initial >> mapping. This capability is flagged by an elf-note. >> >> In case the elf-note is set by the kernel don't place the i

Re: [Xen-devel] [PATCH v2 2/2] x86/traps: Dump instruction stream in show_execution_state()

2016-02-11 Thread Jan Beulich
>>> On 11.02.16 at 14:41, wrote: > Wherever %rip is pointing, the code under %rip is directly relevant to > the exact values of the registers and stack dump printed. > > It will be obvious from the numeric value of %rip whether it is bad > (also, whether symbol information is found), and making i

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

2016-02-11 Thread osstest service owner
flight 81641 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/81641/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-pvops3 host-install(3) broken in 81339 REGR. vs. 66399 build-amd64-rumpuserxen

Re: [Xen-devel] [PATCH v2 2/2] x86/traps: Dump instruction stream in show_execution_state()

2016-02-11 Thread Andrew Cooper
On 11/02/16 12:52, Jan Beulich wrote: On 11.02.16 at 13:12, wrote: >> On 11/02/16 11:16, Jan Beulich wrote: >> On 11.02.16 at 11:52, wrote: --- a/xen/arch/x86/traps.c +++ b/xen/arch/x86/traps.c @@ -114,6 +114,56 @@ boolean_param("ler", opt_ler); #define stack_words_p

Re: [Xen-devel] [PATCH v3 1/2] build: specify minimum versions of gcc and binutils

2016-02-11 Thread Jan Beulich
>>> On 11.02.16 at 14:21, wrote: > On Thu, 2016-02-11 at 12:31 +, Andrew Cooper wrote: >> On 11/02/16 12:23, Ian Campbell wrote: >> > From: Doug Goldstein >> > >> > To help people avoid having to figure out what versions of gcc and >> > binutils need to be supported document them explicitly.

Re: [Xen-devel] [PATCH 3/3] xen: credit1: avoid boosting vCPUs being "just" migrated

2016-02-11 Thread Jan Beulich
>>> On 11.02.16 at 12:39, wrote: > --- a/xen/common/sched_credit.c > +++ b/xen/common/sched_credit.c > @@ -1022,11 +1022,14 @@ csched_vcpu_wake(const struct scheduler *ops, struct > vcpu *vc, unsigned wf) > * more CPU resource intensive VCPUs without impacting overall > * system fair

Re: [Xen-devel] [PATCH 2/3] xen: sched: add wakeup flags to the scheduler interface

2016-02-11 Thread Jan Beulich
>>> On 11.02.16 at 12:38, wrote: > --- a/xen/include/xen/sched-if.h > +++ b/xen/include/xen/sched-if.h > @@ -144,7 +144,8 @@ struct scheduler { > void (*remove_vcpu)(const struct scheduler *, struct vcpu *); > > void (*sleep) (const struct scheduler *, stru

Re: [Xen-devel] [PATCH v3 1/2] build: specify minimum versions of gcc and binutils

2016-02-11 Thread Ian Campbell
On Thu, 2016-02-11 at 12:31 +, Andrew Cooper wrote: > On 11/02/16 12:23, Ian Campbell wrote: > > From: Doug Goldstein > > > > To help people avoid having to figure out what versions of gcc and > > binutils need to be supported document them explicitly. > > > > Signed-off-by: Doug Goldstein

Re: [Xen-devel] [PATCH v2] tools: probe for existence of qemu-xen stderr trace backend.

2016-02-11 Thread Paul Durrant
> -Original Message- > From: Ian Campbell [mailto:ian.campb...@citrix.com] > Sent: 11 February 2016 12:11 > To: Ian Jackson; Wei Liu; xen-devel@lists.xen.org; Stefano Stabellini > Cc: Ian Campbell; Paul Durrant; Anthony Perard > Subject: [PATCH v2] tools: probe for existence of qemu-xen std

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

2016-02-11 Thread osstest service owner
flight 81645 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/81645/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt-qcow2 4 host-ping-check-native fail REGR. vs. 80121 Tests which did not succe

Re: [Xen-devel] [PATCH v2 3/6] xen: factor out allocation of page tables into separate function

2016-02-11 Thread Juergen Gross
On 11/02/16 13:27, Daniel Kiper wrote: > On Thu, Feb 11, 2016 at 08:53:23AM +0100, Juergen Gross wrote: >> Do the allocation of page tables in a separate function. This will >> allow to do the allocation at different times of the boot preparations >> depending on the features the kernel is supporti

Re: [Xen-devel] [PATCH v2 2/2] x86/traps: Dump instruction stream in show_execution_state()

2016-02-11 Thread Jan Beulich
>>> On 11.02.16 at 13:12, wrote: > On 11/02/16 11:16, Jan Beulich wrote: > On 11.02.16 at 11:52, wrote: >>> --- a/xen/arch/x86/traps.c >>> +++ b/xen/arch/x86/traps.c >>> @@ -114,6 +114,56 @@ boolean_param("ler", opt_ler); >>> #define stack_words_per_line 4 >>> #define ESP_BEFORE_EXCEPTION(r

  1   2   >