Re: [Xen-devel] question about memory allocation for driver domain

2015-02-09 Thread Julien Grall
On 07/02/2015 01:15, Oleksandr Tyshchenko wrote: Hi Julien Hi Oleksandr, On Thu, Feb 5, 2015 at 6:36 PM, Oleksandr Tyshchenko Let me describe in detail about solution #3 before answer to your question. Maybe I missed something in the first mail. Also the Ian's answer clarified to me some po

Re: [Xen-devel] Inplace upgrading 4.4.x -> 4.5.0

2015-02-09 Thread Stefano Stabellini
Hello Steven, upgrades from Xen 4.4 to 4.5 are supposed to work out of the box. Please post more details and we'll try to help you figure out what's wrong. Cheers, Stefano On Sun, 8 Feb 2015, Steven Haigh wrote: > Hi all, > > I was under the impression that you should be able to do in-place >

Re: [Xen-devel] Inplace upgrading 4.4.x -> 4.5.0

2015-02-09 Thread Steven Haigh
Thanks Stefano, I wanted to make sure I was correct with this belief before I posted all the info. I'll gather as much info as I can in the next day or so and see what we can do. On 9/02/2015 7:35 PM, Stefano Stabellini wrote: > Hello Steven, > upgrades from Xen 4.4 to 4.5 are supposed to work o

Re: [Xen-devel] Inplace upgrading 4.4.x -> 4.5.0

2015-02-09 Thread Sander Eikelenboom
Monday, February 9, 2015, 9:35:33 AM, you wrote: > Hello Steven, > upgrades from Xen 4.4 to 4.5 are supposed to work out of the box. > Please post more details and we'll try to help you figure out what's > wrong. > Cheers, > Stefano > On Sun, 8 Feb 2015, Steven Haigh wrote: >> Hi all, >> >>

Re: [Xen-devel] [RFC 7/7] libxl: Wait for QEMU startup in stubdomain

2015-02-09 Thread Ian Campbell
On Fri, 2015-02-06 at 17:23 +, Stefano Stabellini wrote: > > > > One of the main issues outstanding from when Anthony originally posted > > his patches is how we want to go about building this? I honestly do > > not know how well the current dracut-based approach to building the > > root imag

Re: [Xen-devel] Inplace upgrading 4.4.x -> 4.5.0

2015-02-09 Thread Steven Haigh
On 9/02/2015 7:59 PM, Sander Eikelenboom wrote: > Monday, February 9, 2015, 9:35:33 AM, you wrote: > >> Hello Steven, >> upgrades from Xen 4.4 to 4.5 are supposed to work out of the box. > >> Please post more details and we'll try to help you figure out what's >> wrong. > >> Cheers, > >> Stefan

Re: [Xen-devel] [RFC 2/7] linux-stubdomain: Compile Linux

2015-02-09 Thread Ian Campbell
On Fri, 2015-02-06 at 17:45 +, Stefano Stabellini wrote: > That's a good point. I think git clone would be more inline with any > other external project that we use. However I'll let the other > maintainers decide on this. If anything it should be a clone of a push-gated Linux git tree on xe

Re: [Xen-devel] [RFC 4/7] libxl: Add "stubdomain_version" to domain_build_info.

2015-02-09 Thread Ian Campbell
On Tue, 2015-02-03 at 23:06 -0500, Eric Shelton wrote: > This enum gives the ability to select between a MiniOS-based QEMU > traditional stub domain and a Linux-based QEMU upstream stub domain. To > use the Linux-based stubdomain, the following two lines should be > included in the appropriate xl.

Re: [Xen-devel] [RFC 7/7] libxl: Wait for QEMU startup in stubdomain

2015-02-09 Thread Stefano Stabellini
On Mon, 9 Feb 2015, Ian Campbell wrote: > On Fri, 2015-02-06 at 17:23 +, Stefano Stabellini wrote: > > > > > > One of the main issues outstanding from when Anthony originally posted > > > his patches is how we want to go about building this? I honestly do > > > not know how well the current d

Re: [Xen-devel] Inplace upgrading 4.4.x -> 4.5.0

2015-02-09 Thread Andrew Cooper
On 09/02/2015 09:09, Steven Haigh wrote: > On 9/02/2015 7:59 PM, Sander Eikelenboom wrote: >> Monday, February 9, 2015, 9:35:33 AM, you wrote: >> >>> Hello Steven, >>> upgrades from Xen 4.4 to 4.5 are supposed to work out of the box. >>> Please post more details and we'll try to help you figure out

Re: [Xen-devel] Inplace upgrading 4.4.x -> 4.5.0

2015-02-09 Thread Ian Campbell
On Mon, 2015-02-09 at 20:09 +1100, Steven Haigh wrote: > On 9/02/2015 7:59 PM, Sander Eikelenboom wrote: > > Monday, February 9, 2015, 9:35:33 AM, you wrote: > > > >> Hello Steven, > >> upgrades from Xen 4.4 to 4.5 are supposed to work out of the box. > > > >> Please post more details and we'll t

Re: [Xen-devel] Booting dom0, various issues since v3.16 kernels

2015-02-09 Thread Stefan Bader
On 05.02.2015 15:47, Sander Eikelenboom wrote: > > Thursday, February 5, 2015, 3:22:49 PM, you wrote: > >> Hey David, > >> after just being in that pain, I thought I might as well give a summary to >> you/the list. Maybe helpful to not forget which piece should go to which >> stable... > >> So

Re: [Xen-devel] [PATCH] x86 spinlock: Fix memory corruption on completing completions

2015-02-09 Thread Raghavendra K T
On 02/09/2015 02:44 AM, Jeremy Fitzhardinge wrote: On 02/06/2015 06:49 AM, Raghavendra K T wrote: [...] Linus suggested that we should not do any writes to lock after unlock(), and we can move slowpath clearing to fastpath lock. Yep, that seems like a sound approach. Current approach seem

Re: [Xen-devel] Booting dom0, various issues since v3.16 kernels

2015-02-09 Thread Sander Eikelenboom
Monday, February 9, 2015, 10:31:15 AM, you wrote: > On 05.02.2015 15:47, Sander Eikelenboom wrote: >> >> Thursday, February 5, 2015, 3:22:49 PM, you wrote: >> >>> Hey David, >> >>> after just being in that pain, I thought I might as well give a summary to >>> you/the list. Maybe helpful to not

Re: [Xen-devel] Inplace upgrading 4.4.x -> 4.5.0

2015-02-09 Thread Steven Haigh
On 9/02/2015 8:16 PM, Ian Campbell wrote: > On Mon, 2015-02-09 at 20:09 +1100, Steven Haigh wrote: >> On 9/02/2015 7:59 PM, Sander Eikelenboom wrote: >>> Monday, February 9, 2015, 9:35:33 AM, you wrote: >>> Hello Steven, upgrades from Xen 4.4 to 4.5 are supposed to work out of the box. >>

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

2015-02-09 Thread xen . org
flight 34299 linux-linus real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/34299/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt 12 guest-start.2 fail REGR. vs. 34227 test-amd64-amd64-libv

Re: [Xen-devel] question about memory allocation for driver domain

2015-02-09 Thread Ian Campbell
On Mon, 2015-02-09 at 16:31 +0800, Julien Grall wrote: > It seems logical to me that destroy/create domd in a row working fine. > But this use-case is too simple :). > > Let's imagine we decide to start classical domains (i.e no 1:1 mapping) > before creating domd (the 1:1 domain). As the free m

[Xen-devel] [GIT PULL] xen: features and fixes for 3.20-rc0

2015-02-09 Thread David Vrabel
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Linus, Please git pull the following tag: git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git stable/for-linus-3.20-rc0-tag xen: features and fixes for 3.20-rc0 - - Reworked handling for foreign (grant mapped) pages to simplify the code,

Re: [Xen-devel] Guidelines for new PV protocol submission

2015-02-09 Thread Roger Pau Monné
El 05/02/15 a les 11.42, Ian Campbell ha escrit: > On Tue, 2015-01-20 at 13:47 +0100, Roger Pau Monné wrote: >> Hello, >> >> I should probably have done this earlier because I've been aware of this >> issue for a long time (since I've started dealing with the PV blk protocol). >> >> The current way

Re: [Xen-devel] [Qemu-devel] [v2][PATCH] libxl: add one machine property to support IGD GFX passthrough

2015-02-09 Thread Ian Campbell
On Mon, 2015-02-09 at 14:28 +0800, Chen, Tiejun wrote: > What about this? I've not read the code in detail,since I'm travelling but from a quick glance it looks to be implementing the sort of thing I meant, thanks. A couple of higher level comments: I'd suggest to put the code for reading the v

[Xen-devel] [PATCH OSSTEST 01/12] toolstack: save / restore check

2015-02-09 Thread Wei Liu
Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- Osstest/Toolstack/libvirt.pm | 9 + Osstest/Toolstack/xend.pm| 3 +++ Osstest/Toolstack/xl.pm | 9 + 3 files changed, 21 insertions(+) diff --git a/Osstest/Toolstack/libvirt.pm b/Osstest/Toolstack/libvirt.pm i

[Xen-devel] [PATCH OSSTEST 00/12] Libvirt save, restore and migration support

2015-02-09 Thread Wei Liu
This series implements libvirt save, restore and migration test in OSSTest. The first 5 patches are used to 1. separate save / restore support from migration support; 2. introduce notion of local / remote migration support. With the above changes we lay the ground to test libvirt's save / restore

[Xen-devel] [PATCH OSSTEST 07/12] ts-xen-build-prep: install ebtables

2015-02-09 Thread Wei Liu
Libvirt's test suite needs it. Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- ts-xen-build-prep | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ts-xen-build-prep b/ts-xen-build-prep index 3684c0a..1bf40db 100755 --- a/ts-xen-build-prep +++ b/ts-xen-build-prep @

[Xen-devel] [PATCH OSSTEST 04/12] toolstack: distinguish local and remote migration support

2015-02-09 Thread Wei Liu
Libvirt supports migrating a guest to remote host but not local host. Distinguish the concept of local migration support and remote migration support. Toolstack's migrate_check now takes an extra argument to indicate which mode we're interested in. In sg-run-job we still check for local migration

[Xen-devel] [PATCH OSSTEST 09/12] ts-debian-hvm-install: stub out libvirt + ovmf / rombios

2015-02-09 Thread Wei Liu
Libvirt's configuration converter doesn't know how to deal with BIOS selection. The end result is it always use the default one (seabios). Stub out ovmf and rombios to avoid false positive results. This restriction will be removed once libvirt's converter knows how to deal with BIOS selection. Si

[Xen-devel] [PATCH OSSTEST 08/12] ts-libvirt-build: run libvirt test suite

2015-02-09 Thread Wei Liu
We're interested in xlconfigtest. Since we're running test anyway run the whole suite. Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- ts-libvirt-build | 23 +-- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a/ts-libvirt-build b/ts-libvirt-build

[Xen-devel] [PATCH OSSTEST 06/12] toolstack/libvirt: guest migrate, save and restore support

2015-02-09 Thread Wei Liu
Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- Osstest/Toolstack/libvirt.pm | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/Osstest/Toolstack/libvirt.pm b/Osstest/Toolstack/libvirt.pm index a804ed5..d7d5154 100644 --- a/Osstest/Toolstack/libvirt.pm +

[Xen-devel] [PATCH OSSTEST 05/12] sg-run-job: remove save/restore dependency on local migration support

2015-02-09 Thread Wei Liu
Since we've introduced different checks for save / restore and local migration, it's possible to run save / restore tests without running local migration tests. Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- sg-run-job | 18 +- 1 file changed, 13 insertions(+), 5 d

[Xen-devel] [PATCH OSSTEST 03/12] osstest migrate support check catch -> variables

2015-02-09 Thread Wei Liu
From: Ian Jackson Signed-off-by: Ian Jackson --- sg-run-job | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/sg-run-job b/sg-run-job index 2cf810a..cc2ab3b 100755 --- a/sg-run-job +++ b/sg-run-job @@ -289,7 +289,9 @@ proc run-job/test-pair {} { } proc test-guest-migr {

[Xen-devel] [PATCH OSSTEST 02/12] Introduce ts-saverestore-support-check

2015-02-09 Thread Wei Liu
Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- ts-saverestore-support-check | 27 +++ 1 file changed, 27 insertions(+) create mode 100755 ts-saverestore-support-check diff --git a/ts-saverestore-support-check b/ts-saverestore-support-check new file mode 10

Re: [Xen-devel] Guidelines for new PV protocol submission

2015-02-09 Thread Ian Campbell
On Mon, 2015-02-09 at 12:01 +0100, Roger Pau Monné wrote: > El 05/02/15 a les 11.42, Ian Campbell ha escrit: > > On Tue, 2015-01-20 at 13:47 +0100, Roger Pau Monné wrote: > >> Hello, > >> > >> I should probably have done this earlier because I've been aware of this > >> issue for a long time (since

Re: [Xen-devel] question about memory allocation for driver domain

2015-02-09 Thread Julien Grall
On 09/02/2015 18:53, Ian Campbell wrote: On Mon, 2015-02-09 at 16:31 +0800, Julien Grall wrote: It seems logical to me that destroy/create domd in a row working fine. But this use-case is too simple :). Let's imagine we decide to start classical domains (i.e no 1:1 mapping) before creating do

[Xen-devel] [PATCH OSSTEST 11/12] make-flight, mfi-common: rename onetoolstack to pairtoolstack

2015-02-09 Thread Wei Liu
The name "onetoolstack" in confusing. Currently it's in fact referring to the toolstack used to test pair migration, so rename it to "pairtoolstack". No functional changes introduced. Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- make-flight | 2 +- mfi-common | 8 2 f

[Xen-devel] [PATCH OSSTEST 10/12] make-flight: debian hvm tests with libvirt

2015-02-09 Thread Wei Liu
Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- make-flight | 21 ++--- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/make-flight b/make-flight index 8ac3a87..dabcacd 100755 --- a/make-flight +++ b/make-flight @@ -92,7 +92,9 @@ job_create_test_filter

[Xen-devel] [PATCH] x86/nmi: Fix shootdown of pcpus running in VMX non-root mode

2015-02-09 Thread Andrew Cooper
In the case of a crash, nmi_shootdown_cpus() patches nmi_crash() into the IDT of each processor, such that the next NMI it receives will force it into the crash path. c/s 7dd3b06ff "vmx: fix handling of NMI VMEXIT" fixed one issue but inadvertently introduced another. The original use of self_nmi

[Xen-devel] [PATCH OSSTEST 12/12] make-flight, mfi-common: create live migration test for libvirt

2015-02-09 Thread Wei Liu
Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- make-flight | 13 +++-- mfi-common | 6 +- 2 files changed, 16 insertions(+), 3 deletions(-) diff --git a/make-flight b/make-flight index 23d51ea..80c9c39 100755 --- a/make-flight +++ b/make-flight @@ -419,11 +419,20 @@

Re: [Xen-devel] [PATCH] x86/nmi: Fix shootdown of pcpus running in VMX non-root mode

2015-02-09 Thread Tim Deegan
Hi, At 11:25 + on 09 Feb (1423477508), Andrew Cooper wrote: > In the case of a crash, nmi_shootdown_cpus() patches nmi_crash() into the IDT > of each processor, such that the next NMI it receives will force it into the > crash path. > > c/s 7dd3b06ff "vmx: fix handling of NMI VMEXIT" fixed on

Re: [Xen-devel] [PATCH] x86/nmi: Fix shootdown of pcpus running in VMX non-root mode

2015-02-09 Thread Andrew Cooper
On 09/02/15 11:43, Tim Deegan wrote: > Hi, > > At 11:25 + on 09 Feb (1423477508), Andrew Cooper wrote: >> In the case of a crash, nmi_shootdown_cpus() patches nmi_crash() into the IDT >> of each processor, such that the next NMI it receives will force it into the >> crash path. >> >> c/s 7dd3b0

Re: [Xen-devel] Guidelines for new PV protocol submission

2015-02-09 Thread Vitaly Chernooky
Hi All! On Tue, Jan 20, 2015 at 2:47 PM, Roger Pau Monné wrote: > Hello, > > I should probably have done this earlier because I've been aware of this > issue for a long time (since I've started dealing with the PV blk > protocol). > > The current way to describe PV protocols in Xen is very ineff

Re: [Xen-devel] question about memory allocation for driver domain

2015-02-09 Thread Oleksandr Tyshchenko
On Mon, Feb 9, 2015 at 10:31 AM, Julien Grall wrote: > > > On 07/02/2015 01:15, Oleksandr Tyshchenko wrote: >> >> Hi Julien > > > Hi Oleksandr, Hi Julien > >> On Thu, Feb 5, 2015 at 6:36 PM, Oleksandr Tyshchenko >> Let me describe in detail about solution #3 before answer to your >> question. May

Re: [Xen-devel] [PATCH] x86 spinlock: Fix memory corruption on completing completions

2015-02-09 Thread Peter Zijlstra
On Mon, Feb 09, 2015 at 03:04:22PM +0530, Raghavendra K T wrote: > So we have 3 choices, > 1. xadd > 2. continue with current approach. > 3. a read before unlock and also after that. For the truly paranoid we have probe_kernel_address(), suppose the lock was in module space and the module just got

Re: [Xen-devel] [RFC 7/7] libxl: Wait for QEMU startup in stubdomain

2015-02-09 Thread Anthony PERARD
On Mon, Feb 09, 2015 at 09:07:13AM +, Ian Campbell wrote: > On Fri, 2015-02-06 at 17:23 +, Stefano Stabellini wrote: > > > > > > One of the main issues outstanding from when Anthony originally posted > > > his patches is how we want to go about building this? I honestly do > > > not know

Re: [Xen-devel] [PATCHv1] xen-blkback: default to X86_32 ABI on x86

2015-02-09 Thread Roger Pau Monné
El 05/02/15 a les 18.09, David Vrabel ha escrit: > Prior to the existance of 64-bit backends using the X86_64 ABI, > frontends used the X86_32 ABI. These old frontends do not specify the > ABI and when used with a 64-bit backend do not work. > > On x86, default to the X86_32 ABI if one is not spe

Re: [Xen-devel] question about memory allocation for driver domain

2015-02-09 Thread Oleksandr Tyshchenko
Hi Ian On Mon, Feb 9, 2015 at 12:53 PM, Ian Campbell wrote: > On Mon, 2015-02-09 at 16:31 +0800, Julien Grall wrote: >> It seems logical to me that destroy/create domd in a row working fine. >> But this use-case is too simple :). >> >> Let's imagine we decide to start classical domains (i.e no 1:

Re: [Xen-devel] [PATCH] x86 spinlock: Fix memory corruption on completing completions

2015-02-09 Thread Raghavendra K T
On 02/09/2015 05:32 PM, Peter Zijlstra wrote: On Mon, Feb 09, 2015 at 03:04:22PM +0530, Raghavendra K T wrote: So we have 3 choices, 1. xadd 2. continue with current approach. 3. a read before unlock and also after that. For the truly paranoid we have probe_kernel_address(), suppose the lock w

[Xen-devel] [PATCH V2] x86 spinlock: Fix memory corruption on completing completions

2015-02-09 Thread Raghavendra K T
Paravirt spinlock clears slowpath flag after doing unlock. As explained by Linus currently it does: prev = *lock; add_smp(&lock->tickets.head, TICKET_LOCK_INC); /* add_smp() is a full mb() */ if (unlikely(lock->tickets.tail & TICKET_

Re: [Xen-devel] question about memory allocation for driver domain

2015-02-09 Thread Oleksandr Tyshchenko
On Mon, Feb 9, 2015 at 1:16 PM, Julien Grall wrote: > > > On 09/02/2015 18:53, Ian Campbell wrote: >> >> On Mon, 2015-02-09 at 16:31 +0800, Julien Grall wrote: >>> >>> It seems logical to me that destroy/create domd in a row working fine. >>> But this use-case is too simple :). >>> >>> Let's imagi

Re: [Xen-devel] [PATCH] x86/nmi: Fix shootdown of pcpus running in VMX non-root mode

2015-02-09 Thread Jan Beulich
>>> On 09.02.15 at 12:52, wrote: > On 09/02/15 11:43, Tim Deegan wrote: >> Hi, >> >> At 11:25 + on 09 Feb (1423477508), Andrew Cooper wrote: >>> In the case of a crash, nmi_shootdown_cpus() patches nmi_crash() into the > IDT >>> of each processor, such that the next NMI it receives will force

Re: [Xen-devel] Crash in acpi_ps_peek_opcode when booting kernel 3.19 as Xen dom0

2015-02-09 Thread Stefan Bader
On 05.02.2015 20:36, Konrad Rzeszutek Wilk wrote: > On Thu, Feb 05, 2015 at 03:33:02PM +0100, Stefan Bader wrote: >> While experimenting/testing various kernel versions I discovered that trying >> to >> boot a Haswell based hosts will always crash when booting as Xen dom0 >> (Xen-4.4.1). The same

Re: [Xen-devel] question about memory allocation for driver domain

2015-02-09 Thread Ian Campbell
On Mon, 2015-02-09 at 19:16 +0800, Julien Grall wrote: > >> The last problem but not the least is, depending on which backend you > >> are running in the 1:1 domain (such blkback), grant won't be mapped 1:1 > >> to the guest, so you will have to use swiotlb in order to use the right > >> DMA addres

Re: [Xen-devel] [PATCH] x86/nmi: Fix shootdown of pcpus running in VMX non-root mode

2015-02-09 Thread Andrew Cooper
On 09/02/15 12:56, Jan Beulich wrote: On 09.02.15 at 12:52, wrote: >> On 09/02/15 11:43, Tim Deegan wrote: >>> Hi, >>> >>> At 11:25 + on 09 Feb (1423477508), Andrew Cooper wrote: In the case of a crash, nmi_shootdown_cpus() patches nmi_crash() into the >> IDT of each processor,

Re: [Xen-devel] Guidelines for new PV protocol submission

2015-02-09 Thread Ian Campbell
On Mon, 2015-02-09 at 13:59 +0200, Vitaly Chernooky wrote: > Guys, what do you think about using an Interface Description Language > such as Google Protocol Buffers or something like? After a bit of googling and reading it seems like Google Protocol Buffers includes a specific wire encoding, a la

[Xen-devel] [PATCH 0/3] Misc patches for libxl_device_disk functions

2015-02-09 Thread Wei Liu
The first two patches are cleaning up and refactoring. The third patch fixes a bug discovered by Jim's libvirt test case. Wei. Wei Liu (3): libxl, xl: don't init/dispose when not necessary libxl: factor out libxl__disk_backend_from_xs_be libxl: libxl__device_from_disk should retrieve backe

[Xen-devel] [PATCH 2/3] libxl: factor out libxl__disk_backend_from_xs_be

2015-02-09 Thread Wei Liu
Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- tools/libxl/libxl.c | 31 +++ 1 file changed, 23 insertions(+), 8 deletions(-) diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c index ef48550..01a2d11 100644 --- a/tools/libxl/libxl.c +++ b/tools/libx

[Xen-devel] [PATCH 3/3] libxl: libxl__device_from_disk should retrieve backend from xenstore

2015-02-09 Thread Wei Liu
... if backend is not set by caller. Also change the function to use "goto" idiom while I was there. Signed-off-by: Wei Liu Cc: Ian Campbell Cc: Ian Jackson --- tools/libxl/libxl.c | 21 ++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/tools/libxl/libxl.c b

[Xen-devel] [PATCH 1/3] libxl, xl: don't init/dispose when not necessary

2015-02-09 Thread Wei Liu
Functions like libxl__device_disk_from_xs_be and libxl_vdev_to_device_disk should not touch the disk struct passed in. It's caller's responsibility to do that. This leads to some changes to the caller to prepare the disk struct. Note that libxl_vdev_to_device_disk is a public API so a macro is de

Re: [Xen-devel] [PATCH 1/3] libxl, xl: don't init/dispose when not necessary

2015-02-09 Thread Wei Liu
Er... The title should be "libxl, xl: don't init/dispose when not unnecessary" -- ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH] x86/nmi: Fix shootdown of pcpus running in VMX non-root mode

2015-02-09 Thread Jan Beulich
>>> On 09.02.15 at 14:13, wrote: > On 09/02/15 12:56, Jan Beulich wrote: >> So wouldn't it be possible to get rid of nmi_crash() and have >> nmi() call *nmi_handler instead of don_nmi (and nmi_handler >> would really just become an alias of exception_table[2]? > > nmi_crash() deliberately doesn't

Re: [Xen-devel] Crash in acpi_ps_peek_opcode when booting kernel 3.19 as Xen dom0

2015-02-09 Thread Stefan Bader
On 09.02.2015 14:07, Stefan Bader wrote: > On 05.02.2015 20:36, Konrad Rzeszutek Wilk wrote: >> On Thu, Feb 05, 2015 at 03:33:02PM +0100, Stefan Bader wrote: >>> While experimenting/testing various kernel versions I discovered that >>> trying to >>> boot a Haswell based hosts will always crash whe

Re: [Xen-devel] Xen Development for Dummies.

2015-02-09 Thread Ian Campbell
On Sat, 2015-02-07 at 09:48 +, Jason Long wrote: > Can anyone tell me why Citrix Developer choose Ocaml? They thought it was a good fit for the problem at hand. More than that (i.e. comparisons of the relative merits of various computer languages for beginners) I'm afraid is not on topic for t

Re: [Xen-devel] Guidelines for new PV protocol submission

2015-02-09 Thread Vitaly Chernooky
On Mon, Feb 9, 2015 at 3:15 PM, Ian Campbell wrote: > On Mon, 2015-02-09 at 13:59 +0200, Vitaly Chernooky wrote: > > > Guys, what do you think about using an Interface Description Language > > such as Google Protocol Buffers or something like? > > After a bit of googling and reading it seems like

Re: [Xen-devel] [OSSTEST PATCH 01/10] TestSupport: Provide target_editfile_cancel

2015-02-09 Thread Ian Campbell
On Fri, 2015-02-06 at 19:17 +, Ian Jackson wrote: > This allows a caller of target_editfile to cancel the edit from within > their supplied code block. > > Signed-off-by: Ian Jackson This contains more Perl exception handling voodoo than I'm really qualified for, but based on a bit of search

Re: [Xen-devel] [OSSTEST PATCH 02/10] TestSupport: Introduce next_unique_name

2015-02-09 Thread Ian Campbell
On Fri, 2015-02-06 at 19:17 +, Ian Jackson wrote: > No functional change. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [OSSTEST PATCH 03/10] TestSupport: Make next_unique_name count in decimal, not unary

2015-02-09 Thread Ian Campbell
On Fri, 2015-02-06 at 19:17 +, Ian Jackson wrote: > Ie, we add `+' rather than an ever-longer series of `+'s. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbell ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-deve

Re: [Xen-devel] Crash in acpi_ps_peek_opcode when booting kernel 3.19 as Xen dom0

2015-02-09 Thread Juergen Gross
On 02/09/2015 02:33 PM, Stefan Bader wrote: On 09.02.2015 14:07, Stefan Bader wrote: On 05.02.2015 20:36, Konrad Rzeszutek Wilk wrote: On Thu, Feb 05, 2015 at 03:33:02PM +0100, Stefan Bader wrote: While experimenting/testing various kernel versions I discovered that trying to boot a Haswell ba

Re: [Xen-devel] [OSSTEST PATCH 04/10] tcl/osstestlib.tcl: Provide lshift

2015-02-09 Thread Ian Campbell
On Fri, 2015-02-06 at 19:17 +, Ian Jackson wrote: > No callers yet. > > Signed-off-by: Ian Jackson > --- > tcl/osstestlib.tcl |7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/tcl/osstestlib.tcl b/tcl/osstestlib.tcl > index 1531c87..a0413c4 100644 > --- a/tcl/osstestlib.tcl

Re: [Xen-devel] [OSSTEST PATCH 05/10] sg-run-job: testid generation: Move spawn-step-begin

2015-02-09 Thread Ian Campbell
On Fri, 2015-02-06 at 19:17 +, Ian Jackson wrote: > Run spawn-step-begin until after the loop over $args, and after the > computation of the basic deftestid. > > No functional change: nothing in that loop looks at stepno. > > Signed-off-by: Ian Jackson TBH any Ack I gave here (and to the fo

Re: [Xen-devel] [RFC 7/7] libxl: Wait for QEMU startup in stubdomain

2015-02-09 Thread Eric Shelton
On Mon, Feb 9, 2015 at 7:08 AM, Anthony PERARD wrote: > On Mon, Feb 09, 2015 at 09:07:13AM +, Ian Campbell wrote: >> On Fri, 2015-02-06 at 17:23 +, Stefano Stabellini wrote: >> > > >> > > One of the main issues outstanding from when Anthony originally posted >> > > his patches is how we wa

Re: [Xen-devel] [OSSTEST PATCH 09/10] rump kernel tests: Cancel unneeded edits of guest config

2015-02-09 Thread Ian Campbell
On Fri, 2015-02-06 at 19:17 +, Ian Jackson wrote: > If the guest config is already set up to preserve, cancel the edit. > We are going to repeat this test, and this avoids creating many > identical copies of the same file in the log output. > > Signed-off-by: Ian Jackson Acked-by: Ian Campbe

Re: [Xen-devel] [OSSTEST PATCH 10/10] rump kernel tests: Repeat the xenstorels test 50 times

2015-02-09 Thread Ian Campbell
On Fri, 2015-02-06 at 19:17 +, Ian Jackson wrote: > Add a new step which uses repeat-ts to run > ts-rumpuserxen-demo-xenstorels many times. > > We have to run ts-guest-destroy-hard first each time, to destroy the > guest which might exist at each previous step. To help with ad-hoc > debugging

Re: [Xen-devel] [PATCH] x86/nmi: Fix shootdown of pcpus running in VMX non-root mode

2015-02-09 Thread Andrew Cooper
On 09/02/15 13:26, Jan Beulich wrote: On 09.02.15 at 14:13, wrote: >> On 09/02/15 12:56, Jan Beulich wrote: >>> So wouldn't it be possible to get rid of nmi_crash() and have >>> nmi() call *nmi_handler instead of don_nmi (and nmi_handler >>> would really just become an alias of exception_tabl

Re: [Xen-devel] [RFC 4/7] libxl: Add "stubdomain_version" to domain_build_info.

2015-02-09 Thread Eric Shelton
On Mon, Feb 9, 2015 at 4:11 AM, Ian Campbell wrote: > On Tue, 2015-02-03 at 23:06 -0500, Eric Shelton wrote: >> This enum gives the ability to select between a MiniOS-based QEMU >> traditional stub domain and a Linux-based QEMU upstream stub domain. To >> use the Linux-based stubdomain, the follo

Re: [Xen-devel] [PATCH 1/3] libxl, xl: don't init/dispose when not necessary

2015-02-09 Thread Wei Liu
On Mon, Feb 09, 2015 at 01:23:26PM +, Wei Liu wrote: > Er... The title should be > > "libxl, xl: don't init/dispose when not unnecessary" > -- Grrr... What was I thinking. The initial title was correct. Sorry for the noise and spam. __

Re: [Xen-devel] question about memory allocation for driver domain

2015-02-09 Thread Julien Grall
Hi Ian, On 09/02/2015 21:08, Ian Campbell wrote: On Mon, 2015-02-09 at 19:16 +0800, Julien Grall wrote: The last problem but not the least is, depending on which backend you are running in the 1:1 domain (such blkback), grant won't be mapped 1:1 to the guest, so you will have to use swiotlb in

[Xen-devel] dom0 kernel - irq nobody cared ... the continuing saga ..

2015-02-09 Thread Sander Eikelenboom
at are now xen-4.5). [ 1905.880200] irq 18: nobody cared (try booting with the "irqpoll" option) [ 1905.914838] CPU: 0 PID: 0 Comm: swapper/0 Not tainted 3.19.0-creanuc-20150209-doflr+ #1 [ 1905.935473] Hardware name: /D53427RKE, BIOS RKPPT10H.86A.0017.2013.0425.1251 04/2

[Xen-devel] [qemu-upstream-unstable test] 34319: regressions - FAIL

2015-02-09 Thread xen . org
flight 34319 qemu-upstream-unstable real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/34319/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-freebsd10-i386 11 guest-localmigrate fail REGR. vs. 33488 test-amd64

Re: [Xen-devel] [PATCH V2] x86 spinlock: Fix memory corruption on completing completions

2015-02-09 Thread Oleg Nesterov
On 02/09, Raghavendra K T wrote: > > +static inline void __ticket_check_and_clear_slowpath(arch_spinlock_t *lock) > +{ > + arch_spinlock_t old, new; > + __ticket_t diff; > + > + old.tickets = READ_ONCE(lock->tickets); > + diff = (old.tickets.tail & ~TICKET_SLOWPATH_FLAG) - old.ticke

Re: [Xen-devel] dom0 kernel - irq nobody cared ... the continuing saga ..

2015-02-09 Thread David Vrabel
On 09/02/15 15:03, Sander Eikelenboom wrote: > Hi Jan / David / Konrad, > > I was just testing a 3.19 kernel on my intel machine and again > ran into the sporadically appearing "irq nobody cared" on the dom0 kernel. > This occurs now for quite some kernel versions (running xen-unstable now, > but

Re: [Xen-devel] dom0 kernel - irq nobody cared ... the continuing saga ..

2015-02-09 Thread Sander Eikelenboom
Monday, February 9, 2015, 4:18:15 PM, you wrote: > On 09/02/15 15:03, Sander Eikelenboom wrote: >> Hi Jan / David / Konrad, >> >> I was just testing a 3.19 kernel on my intel machine and again >> ran into the sporadically appearing "irq nobody cared" on the dom0 kernel. >> This occurs now for qu

Re: [Xen-devel] [PATCH v3 12/13] xen/iommu: smmu: Add Xen specific code to be able to use the driver

2015-02-09 Thread Julien Grall
Hi Stefano, On 06/02/2015 21:20, Stefano Stabellini wrote: On Fri, 30 Jan 2015, Julien Grall wrote: -static int force_stage; -module_param_named(force_stage, force_stage, int, S_IRUGO | S_IWUSR); -MODULE_PARM_DESC(force_stage, - "Force SMMU mappings to be installed at a particular stage o

[Xen-devel] [PATCH 4/4] libxl: More probably detect reentry of destroyed ctx

2015-02-09 Thread Ian Jackson
In libxl_ctx_free: 1. Move the GC_INIT earlier, so that we can: 2. Take the lock around most of the work. This is technically unnecessary since calling any other libxl entrypoint on a ctx being passed to libxl_ctx_free risks undefined behaviour. But, taking the lock allows us to much m

[Xen-devel] [PATCH 1/4] libxl: event handling: Add an assert to egc_run_callbacks

2015-02-09 Thread Ian Jackson
Check that the ao is still live when we are about to running some of its callbacks. This reveals an existing bug in libxl which is exercised by libvirt, converting libvirtd: libxl_event.c:1792: libxl__ao_complete_check_progress_reports: Assertion `ao->in_initiator' failed. into libvirtd: li

[Xen-devel] [PATCH 2/4] libxl: event handling: Break out ao_work_outstanding

2015-02-09 Thread Ian Jackson
Break out the test in libxl__ao_complete_check_progress_reports, into ao_work_outstanding, which reports false if either (i) the ao is still ongoing or (ii) there is a progress report (perhaps on a different thread's callback queue) which has yet to be reported to the application. No functional ch

[Xen-devel] [PATCH 3/4] libxl: event handling: ao_inprogress does waits while reports outstanding

2015-02-09 Thread Ian Jackson
libxl__ao_inprogress needs to check (like libxl__ao_complete_check_progress_reports) that there are no oustanding progress callbacks. Otherwise it might happen that we would destroy the ao while another thread has an outstanding callback its egc report queue. The other thread would then, in its e

[Xen-devel] [rumpuserxen test] 34346: regressions - FAIL

2015-02-09 Thread xen . org
flight 34346 rumpuserxen real [real] http://www.chiark.greenend.org.uk/~xensrcts/logs/34346/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-rumpuserxen6 xen-build fail REGR. vs. 33866 build-amd64-rumpuserx

Re: [Xen-devel] [PATCH 1/4] libxl: event handling: Add an assert to egc_run_callbacks

2015-02-09 Thread Wei Liu
On Mon, Feb 09, 2015 at 03:51:09PM +, Ian Jackson wrote: > Check that the ao is still live when we are about to running some of > its callbacks. > > This reveals an existing bug in libxl which is exercised by libvirt, > converting >libvirtd: libxl_event.c:1792: libxl__ao_complete_check_pro

Re: [Xen-devel] [PATCH 3/4] libxl: event handling: ao_inprogress does waits while reports outstanding

2015-02-09 Thread Wei Liu
On Mon, Feb 09, 2015 at 03:51:11PM +, Ian Jackson wrote: > libxl__ao_inprogress needs to check (like > libxl__ao_complete_check_progress_reports) that there are no > oustanding progress callbacks. > > Otherwise it might happen that we would destroy the ao while another > thread has an outstand

Re: [Xen-devel] [PATCH 0/2] libxl: fix handling of fd and timer registrations

2015-02-09 Thread Jim Fehlig
Anthony PERARD wrote: > On Mon, Feb 02, 2015 at 05:00:34PM -0700, Jim Fehlig wrote: > >> This small series fixes some assertions we occasionally see in the >> libxl driver when running libvirt-TCK. The assertions were due to >> races between destroying per-domain libxl_ctx and receiving fd and

Re: [Xen-devel] [PATCH 2/4] libxl: event handling: Break out ao_work_outstanding

2015-02-09 Thread Wei Liu
On Mon, Feb 09, 2015 at 03:51:10PM +, Ian Jackson wrote: > Break out the test in libxl__ao_complete_check_progress_reports, into > ao_work_outstanding, which reports false if either (i) the ao is still > ongoing or (ii) there is a progress report (perhaps on a different > thread's callback queu

Re: [Xen-devel] [PATCH 4/4] libxl: More probably detect reentry of destroyed ctx

2015-02-09 Thread Wei Liu
On Mon, Feb 09, 2015 at 03:51:12PM +, Ian Jackson wrote: > In libxl_ctx_free: > > 1. Move the GC_INIT earlier, so that we can: > > 2. Take the lock around most of the work. This is technically >unnecessary since calling any other libxl entrypoint on a ctx being >passed to libxl_ctx_f

Re: [Xen-devel] dom0 kernel - irq nobody cared ... the continuing saga ..

2015-02-09 Thread Jan Beulich
>>> On 09.02.15 at 16:18, wrote: > On 09/02/15 15:03, Sander Eikelenboom wrote: >> Hi Jan / David / Konrad, >> >> I was just testing a 3.19 kernel on my intel machine and again >> ran into the sporadically appearing "irq nobody cared" on the dom0 kernel. >> This occurs now for quite some kernel v

Re: [Xen-devel] [PATCH V2] x86 spinlock: Fix memory corruption on completing completions

2015-02-09 Thread Raghavendra K T
Ccing Davidlohr, (sorry that I got confused with similar address in cc list). On 02/09/2015 08:44 PM, Oleg Nesterov wrote: On 02/09, Raghavendra K T wrote: +static inline void __ticket_check_and_clear_slowpath(arch_spinlock_t *lock) +{ + arch_spinlock_t old, new; + __ticket_t diff;

Re: [Xen-devel] dom0 kernel - irq nobody cared ... the continuing saga ..

2015-02-09 Thread Sander Eikelenboom
Monday, February 9, 2015, 5:36:28 PM, you wrote: On 09.02.15 at 16:18, wrote: >> On 09/02/15 15:03, Sander Eikelenboom wrote: >>> Hi Jan / David / Konrad, >>> >>> I was just testing a 3.19 kernel on my intel machine and again >>> ran into the sporadically appearing "irq nobody cared" on th

Re: [Xen-devel] [PATCH] modify the IO_TLB_SEGSIZE to io_tlb_segsize configurable as flexible requirement about SW-IOMMU.

2015-02-09 Thread Konrad Rzeszutek Wilk
On Mon, Feb 09, 2015 at 02:13:30AM +, Wang, Xiaoming wrote: > Dear Wilk: > > > -Original Message- > > From: Konrad Rzeszutek Wilk [mailto:konrad.w...@oracle.com] > > Sent: Saturday, February 7, 2015 2:12 AM > > To: Wang, Xiaoming > > Cc: r...@linux-mips.org; boris.ostrov...@oracle.com;

Re: [Xen-devel] [PATCH 4/4] libxl: More probably detect reentry of destroyed ctx

2015-02-09 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 4/4] libxl: More probably detect reentry of destroyed ctx"): > On Mon, Feb 09, 2015 at 03:51:12PM +, Ian Jackson wrote: > > In libxl_ctx_free: > > > > 1. Move the GC_INIT earlier, so that we can: > > Acked-by: Wei Liu Thanks. > > +assert(!ctx->osevent_in_ho

Re: [Xen-devel] [PATCH 0/5] multiboot2: Enable EFI boot services usage in loaded images

2015-02-09 Thread Daniel Kiper
Hi all, On Fri, Jan 30, 2015 at 06:59:23PM +0100, Daniel Kiper wrote: > Hi, > > This patch series enable EFI boot services usage > in loaded images by multiboot2 protocol. Guys, do you have any comments on this patch series? Should I fix something? Could you apply them to GRUB2 tree? Daniel PS

Re: [Xen-devel] [PATCH 00/18] x86: multiboot2 protocol support

2015-02-09 Thread Daniel Kiper
Hi all, On Fri, Jan 30, 2015 at 06:54:04PM +0100, Daniel Kiper wrote: > Hi, > > I am sending, long awaited, first version of multiboot2 protocol > support for legacy BIOS and EFI platforms. > > The final goal is xen.efi binary file which could be loaded by EFI > loader, multiboot (v1) protocol (on

[Xen-devel] [OSSTEST PATCH 1/2] sg-report-job-history: Separate out $cond in queries

2015-02-09 Thread Ian Jackson
We are going to want to reuse this, so separate out the computation of $cond and @params. Signed-off-by: Ian Jackson --- sg-report-job-history | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/sg-report-job-history b/sg-report-job-history index ee021b6..479e34

[Xen-devel] [OSSTEST PATCH 2/2] sg-report-job-history: Show which host test ran on

2015-02-09 Thread Ian Jackson
Add a column listing the host(s) used. We first find the relevant set of host runvars (with a SELECT DISTINCT) and then look up each var for each actual flight. We do the pattern-matching on runvar names in perl to avoid giving the postgresql optimiser a chance to turn this query into a full tabl

Re: [Xen-devel] [OSSTEST PATCH 01/10] TestSupport: Provide target_editfile_cancel

2015-02-09 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST PATCH 01/10] TestSupport: Provide target_editfile_cancel"): > On Fri, 2015-02-06 at 19:17 +, Ian Jackson wrote: > > This allows a caller of target_editfile to cancel the edit from within > > their supplied code block. > > > > Signed-off-by: Ian Jackson > >

  1   2   >