Re: [Xen-devel] [PATCH 01/13] x86/time.c: Use system time to calculate elapsed_nsec in tsc_get_info()

2015-10-08 Thread Jan Beulich
>>> On 28.09.15 at 09:13, wrote: > When the TSC mode of a domain is TSC_MODE_DEFAULT and no TSC emulation > is used, the existing tsc_get_info() calculates elapsed_nsec by scaling > the host TSC with a ratio between guest TSC rate and > nanoseconds. However, the result will be incorrect if the gue

[Xen-devel] [ovmf test] 62723: trouble: broken/pass

2015-10-08 Thread osstest service owner
flight 62723 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/62723/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-ovmf-amd64 3 host-install(3) broken REGR. vs. 62699 version

Re: [Xen-devel] [V5 2/4] x86/xsaves: enable xsaves/xrstors/xsavec in xen

2015-10-08 Thread Shuai Ruan
On Mon, Sep 28, 2015 at 10:03:12AM -0600, Jan Beulich wrote: > >>> On 21.09.15 at 13:33, wrote: > > @@ -954,8 +975,13 @@ long arch_do_domctl( > > v->arch.xcr0_accum = _xcr0_accum; > > if ( _xcr0_accum & XSTATE_NONLAZY ) > > v->arch.nonlazy_xst

[Xen-devel] [linux-4.1 test] 62719: regressions - trouble: broken/fail/pass

2015-10-08 Thread osstest service owner
flight 62719 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/62719/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemut-stubdom-debianhvm-amd64-xsm 13 guest-localmigrate fail REGR. vs. 62318 Tests

Re: [Xen-devel] [PATCH 3/7] xen: sched: better handle (not) inserting idle vCPUs in runqueues

2015-10-08 Thread Juergen Gross
On 10/08/2015 02:52 PM, Dario Faggioli wrote: Idle vCPUs should never really be explicitly inserted in any of the schedulers' runqueue. In fact, they are just put in execution immediately (either on boot, or when a pCPU is assigned to a cpupool), and it will be the first scheduling decision that

Re: [Xen-devel] [PATCH 7/7] xen: sched / cpupool: dump the actual value of NOW()

2015-10-08 Thread Juergen Gross
On 10/08/2015 02:53 PM, Dario Faggioli wrote: rather than its hexadecimal representation. This makes it easier to compare the actual system time with other times being printed out (e.g., deadlines in RTDS). Signed-off-by: Dario Faggioli Acked-by: Juergen Gross --- Cc: Juergen Gross Cc: Ge

Re: [Xen-devel] [PATCH] tools: remove unused wrappers for python

2015-10-08 Thread Juergen Gross
On 10/08/2015 06:32 PM, Zhigang Wang wrote: On 10/08/2015 11:27 AM, Ian Campbell wrote: On Thu, 2015-10-08 at 11:09 -0400, Zhigang Wang wrote: On 10/08/2015 10:38 AM, Ian Campbell wrote: On Tue, 2015-10-06 at 11:24 -0400, Zhigang Wang wrote: We use these extentions along with xend XMLRPC API/

[Xen-devel] [qemu-upstream-4.4-testing baseline-only test] 38139: tolerable FAIL

2015-10-08 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 38139 qemu-upstream-4.4-testing real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/38139/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-amd64-i386-pvgrub 10 guest-start

[Xen-devel] [linux-3.0 test] 62716: tolerable trouble: broken/fail/pass - PUSHED

2015-10-08 Thread osstest service owner
flight 62716 linux-3.0 real [real] http://logs.test-lab.xenproject.org/osstest/logs/62716/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-amd64-xl-qemut-winxpsp3 3 host-install(3) broken pass in 62685 test-amd64-amd64-xl-qemut-winxpsp3 6

[Xen-devel] [PATCH v1 2/4] xen-version: Add third parameter (len) to the do_version hypercall.

2015-10-08 Thread Konrad Rzeszutek Wilk
All existing commands ignore the parameter so this does not break the ABI. This paves the way for expanding the XENVER_ hypercall with variable size structures, such as "XENVER_build_id: Provide ld-embedded build-ids" Suggested-by: Andrew Cooper Signed-off-by: Konrad Rzeszutek Wilk --- xen/arch

[Xen-devel] [PATCH v1] Add build-id to XENVER hypercall.

2015-10-08 Thread Konrad Rzeszutek Wilk
Hey, Attached are the four patches that will add XENVER_build_id and add the proper bits in libxl/libxc. However they also change the behavior of the existing hypercall for XENVER_[compile_info|changeset|commandline] and make them dom0 accessible. This is if XSM is built in or not (though with XS

[Xen-devel] [PATCH v1 4/4] libxl: info: Display build_id of the hypervisor.

2015-10-08 Thread Konrad Rzeszutek Wilk
If the hypervisor is built with we will display it. Signed-off-by: Konrad Rzeszutek Wilk --- tools/libxl/libxl.c | 16 tools/libxl/libxl_types.idl | 1 + tools/libxl/xl_cmdimpl.c| 1 + 3 files changed, 18 insertions(+) diff --git a/tools/libxl/libxl.c b/tools/libx

[Xen-devel] [PATCH v1 3/4] XENVER_build_id: Provide ld-embedded build-ids

2015-10-08 Thread Konrad Rzeszutek Wilk
From: Martin Pohlack The mechanism to get this is via the XENVER_build_id and we add a new subsequent sub-command to retrieve the binary build-id. The hypercall allows an arbitrary size (the buffer and size is provided to the hypervisor). To make this work with libxc it requires expanding the hy

[Xen-devel] [PATCH v1 1/4] xsm/libxl/xen_version: Add XSM for some of the xen_version commands.

2015-10-08 Thread Konrad Rzeszutek Wilk
The XENVER_[compile_info|changeset|commandline] are now guarded by an XSM check. The rest: XENVER_[version|extraversion|capabilities| parameters|get_features|page_size|guest_handle] behave as before (no XSM check). We allow the initial domain to see these while the other guests are not permitted.

[Xen-devel] [PATCH 3.2 018/107] x86/ldt: Make modify_ldt synchronous

2015-10-08 Thread Ben Hutchings
3.2.72-rc1 review patch. If anyone has any objections, please let me know. -- From: Andy Lutomirski commit 37868fe113ff2ba814b3b4eb12df214df555f8dc upstream. modify_ldt() has questionable locking and does not synchronize threads. Improve it: redesign the locking and synchroni

[Xen-devel] [xen-unstable test] 62711: tolerable FAIL - PUSHED

2015-10-08 Thread osstest service owner
flight 62711 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/62711/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-xl-rtds 15 guest-start.2 fail REGR. vs. 62646 test-amd64-amd64-xl-qemut-stu

Re: [Xen-devel] [linux-3.14 bisection] complete test-amd64-i386-xl-qcow2

2015-10-08 Thread Ben Hutchings
On Wed, 2015-09-02 at 10:18 +0100, Ian Campbell wrote: > [resending to correct stable address, sorry folks] > > TL;DR: Any backport of 30b03d05e074 to earlier than commit 1401c00e59e > ("xen/gntdev: convert priv->lock to a mutex", which was added in v4.0) > needs $something doing to it, either s/m

Re: [Xen-devel] [PATCH 0/2] Bulk mem-share identical domains

2015-10-08 Thread Tamas K Lengyel
In case you miss it, there is now soft-reset support which dumps all > memory plus various states from one domain to another, and toolstack > will take care of QEMU and various userspace bits. This might be useful > to you? > > To be clear, this is just FYI, not suggesting we block this series. > >

[Xen-devel] [PATCH v2 1/2] x86/mem-sharing: Bulk mem-sharing entire domains

2015-10-08 Thread Tamas K Lengyel
Currently mem-sharing can be performed on a page-by-page base from the control domain. However, when completely deduplicating (cloning) a VM, this requires at least 3 hypercalls per page. As the user has to loop through all pages up to max_gpfn, this process is very slow and wasteful. This patch i

[Xen-devel] [PATCH v2 2/2] tests/mem-sharing: Add bulk option to memshrtool

2015-10-08 Thread Tamas K Lengyel
Add the bulk option to the test tool to perform complete deduplication between domains. Signed-off-by: Tamas K Lengyel Cc: Ian Jackson Cc: Stefano Stabellini Cc: Ian Campbell Cc: Wei Liu Acked-by: Wei Liu --- tools/tests/mem-sharing/memshrtool.c | 22 ++ 1 file changed,

Re: [Xen-devel] [PATCH 3/9] xen: sched: make locking for {insert, remove}_vcpu consistent

2015-10-08 Thread Dario Faggioli
On Thu, 2015-10-08 at 18:23 +0100, Andrew Cooper wrote: > On 08/10/15 17:46, George Dunlap wrote: > > On 08/10/15 16:20, Andrew Cooper wrote: > > > There are a lot of codepaths, but I cant see one which is > > > definitely > > > called with interrupts disables. (OTOH, I can see several where > >

Re: [Xen-devel] [PATCH 3/9] xen: sched: make locking for {insert, remove}_vcpu consistent

2015-10-08 Thread Dario Faggioli
On Thu, 2015-10-08 at 16:20 +0100, Andrew Cooper wrote: > On 08/10/15 15:58, George Dunlap wrote: > > Generic scheduling code is called from interrupt contexts -- > > namely, > > vcpu_wake() > > There are a lot of codepaths, but I cant see one which is definitely > called with interrupts disables

[Xen-devel] [linux-next test] 62710: regressions - FAIL

2015-10-08 Thread osstest service owner
flight 62710 linux-next real [real] http://logs.test-lab.xenproject.org/osstest/logs/62710/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-pvh-amd 6 xen-boot fail REGR. vs. 62663 test-amd64-amd64-xl-pv

Re: [Xen-devel] [PATCH 2/7] xen: sched: fix locking for insert_vcpu() in credit1 and RTDS

2015-10-08 Thread Dario Faggioli
On Thu, 2015-10-08 at 17:49 +0200, Dario Faggioli wrote: > On Thu, 2015-10-08 at 16:16 +0100, George Dunlap wrote: > > It seems to me that the locking here wasn't to protect insert_vcpu, > > but > > to prevent any scheduling events from happening on cpu until all > > the > > expected infrastructur

[Xen-devel] [PATCH] xen/arm: ctxt_switch: Document the erratum #852523 related to Cortex A57

2015-10-08 Thread Julien Grall
When restoring the system register state for an AArch32 guest at EL2, writes to DACR32_EL2 may not be correctly synchronised by Cortex-A57, which can lead to the guest effectively running into unexpected domain faults. Thankfully, we don't hit this erratum in Xen. Nonetheless, document the code to

Re: [Xen-devel] [PATCH] build: drop unused SUBARCH variable

2015-10-08 Thread Julien Grall
On 05/10/15 14:37, Ian Jackson wrote: > Julien Grall writes ("Re: [Xen-devel] [PATCH] build: drop unused SUBARCH > variable"): >> Jan tends to not notify when a patch has been committed. It's in the >> tree since last week: >> >> commit 063792541db41167db9467feadb700de64cfcd16 >> Author: Doug Gold

[Xen-devel] [PATCH 1/2] xen/arm: Add support of PSCI v1.0 for the host

2015-10-08 Thread Julien Grall
>From Xen point of view, PSCI v0.2 and PSCI v1.0 are very similar. All the PSCI calls used within Xen (PSCI_VERSION, CPU_ON, SYSTEM_OFF and SYSTEM_RESET) behaves exactly the same. While there is no compatible string to represent PSCI v1.0 in the DT, it's possible to detect it using the function PS

[Xen-devel] [PATCH 2/2] xen/arm: Replace XEN_PSCI_* by PSCI_VERSION(major, minor)

2015-10-08 Thread Julien Grall
It will avoid to introduce a new XEN_PSCI_* define every time we support a new version of PSCI in Xen. Also fix the coding style in modified place. Signed-off-by: Julien Grall --- xen/arch/arm/psci.c| 6 +++--- xen/arch/arm/vpsci.c | 15 --- xen/include/asm-arm/psci.h

[Xen-devel] [PATCH 0/2] xen/arm: Add support for PSCI v1.0

2015-10-08 Thread Julien Grall
Hi all, This small patch series allows Xen boot on platform where the firmware is supporting only PSCI 1.0. Regards, Julien Grall (2): xen/arm: Add support of PSCI v1.0 for the host xen/arm: Replace XEN_PSCI_* by PSCI_VERSION(major, minor) xen/arch/arm/psci.c| 15 --- x

Re: [Xen-devel] [PATCH v3 7/9] xen/arm: vgic: Optimize the way to store the target vCPU in the rank

2015-10-08 Thread Julien Grall
On 08/10/15 15:25, Ian Campbell wrote: >> If the concern is the behavior is changed, I'm happy to rework this code >> to keep exactly the same behavior. I.e any 32-bit write containing >> a 0 byte will be ignored. This is not optimal but at least I'm not >> opening the pandora box of fixing every s

[Xen-devel] [PATCH v4 1/4] xen/arm: vgic-v2: Report the correct GICC size to the guest

2015-10-08 Thread Julien Grall
The GICv2 DT node is usually used by the guest to know the address/size of the regions (GICD, GICC...) to map into their virtual memory. While the GICv2 spec requires the size of the GICC to be 8KB, we correctly do an 8KB stage-2 mapping but errornously report 256 in the device tree (based on GUES

[Xen-devel] [PATCH v4 2/4] xen/arm: gic: Check the size of the CPU and vCPU interface retrieved from DT

2015-10-08 Thread Julien Grall
The size of the CPU interface will used in a follow-up patch to map the region in Xen memory. Based on GICv2 spec, the CPU interface should at least be 8KB, although most of the platform we are supporting use incorrectly the GICv1 size (i.e 4KB) in their DT. Only warn and update the size to avoid

[Xen-devel] [PATCH v4 4/4] xen/arm: platform: Drop the quirks callback

2015-10-08 Thread Julien Grall
All the quirks has been replaced by proper detection. Lets drop the callback and hope that no one will need new quirks. At the same time, remove the definition platform_dom0_evtchn_ppi with is not used any more. Signed-off-by: Julien Grall Acked-by: Ian Campbell --- Changes in v2:

[Xen-devel] [PATCH v4 0/4] xen/arm: gic-v2: Detect automatically aliased GIC 400

2015-10-08 Thread Julien Grall
Hi all, Only patch #3 is related to the subject of the cover letter. The rest is clean up of code I looked while I was working on this series. Though, patch #1 is a new bug fix I noticed between v3 and v4. Sincerely yours, Julien Grall (4): xen/arm: vgic-v2: Report the correct GICC size to the

[Xen-devel] [PATCH v4 3/4] xen/arm: gic-v2: Automatically detect aliased GIC400

2015-10-08 Thread Julien Grall
We are currently using a per-platform quirk to know if the 2 4KB region of the GIC CPU interface are each aligned to 64KB. Although, it may be possible to have different layout on a same platform (depending on the firmware version). Rather than having a quirk it's possible to detect by reading the

[Xen-devel] [qemu-upstream-4.4-testing test] 62702: tolerable FAIL - PUSHED

2015-10-08 Thread osstest service owner
flight 62702 qemu-upstream-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/62702/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-localmigrate/x10 fail in 62644 pass in 62580 test-amd

Re: [Xen-devel] [PATCH 3/9] xen: sched: make locking for {insert, remove}_vcpu consistent

2015-10-08 Thread Andrew Cooper
On 08/10/15 17:46, George Dunlap wrote: > On 08/10/15 16:20, Andrew Cooper wrote: >> On 08/10/15 15:58, George Dunlap wrote: >>> On 29/09/15 18:31, Andrew Cooper wrote: On 29/09/15 17:55, Dario Faggioli wrote: > The insert_vcpu() scheduler hook is called with an > inconsistent locking

[Xen-devel] [RFC] Xen 4.6 Acknowledgements

2015-10-08 Thread Lars Kurth
Hi, because we are now branching and opening master before the release, I have to make some changes to how I acknowledge Xen release contributions. In the past, I took the time-stamps of the two RELEASE tags for a release and counted contributions to xen.git and osstest.git (I didn't count qem

Re: [Xen-devel] [PATCH] tools: remove unused wrappers for python

2015-10-08 Thread Wei Liu
On Thu, Oct 08, 2015 at 12:32:44PM -0400, Zhigang Wang wrote: > On 10/08/2015 11:27 AM, Ian Campbell wrote: > > On Thu, 2015-10-08 at 11:09 -0400, Zhigang Wang wrote: > >> On 10/08/2015 10:38 AM, Ian Campbell wrote: > >>> On Tue, 2015-10-06 at 11:24 -0400, Zhigang Wang wrote: > We use these ex

Re: [Xen-devel] [RFC OSSTEST] ap-fetch-*: Support $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder

2015-10-08 Thread Ian Jackson
Ian Campbell writes ("Re: [RFC OSSTEST] ap-fetch-*: Support $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder"): > On Thu, 2015-10-08 at 17:00 +0100, Ian Jackson wrote: > > Maybe [placeholders] should be the default, but it is sometimes > > useful to see the effect of changes to ap-fetch*

Re: [Xen-devel] [OSSTEST PATCH v3 1/3] ts-cpupools: new test script

2015-10-08 Thread Ian Campbell
On Sat, 2015-10-03 at 02:39 +0200, Dario Faggioli wrote: > Copyright (C) 2009-2014 Citrix Inc. Year. > +our $default_pool= "Pool-0"; > +our @schedulers= ("credit","credit2","rtds"); I think @schedulers probably ought to come from a runvar (comma-separated). Consider testing cpupools on 4.4 (whic

Re: [Xen-devel] [PATCH OSSTEST v3 3/3] Create a flight to test OpenStack with xen-unstable and libvirt

2015-10-08 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH OSSTEST v3 3/3] Create a flight to test OpenStack with xen-unstable and libvirt"): > On Tue, 2015-09-29 at 17:52 +0100, Ian Jackson wrote: > > All these revision_FOO=master are rather odd. > > I think the revision_nova one was supposed to be $REVISION_OPENSTACK_NO

Re: [Xen-devel] [PATCH v7 31/32] libxl: allow the creation of HVM domains without a device model.

2015-10-08 Thread Ian Campbell
On Thu, 2015-10-08 at 18:27 +0200, Roger Pau Monné wrote: > El 08/10/15 a les 16.36, Ian Campbell ha escrit: > > On Fri, 2015-10-02 at 17:49 +0200, Roger Pau Monne wrote: > > > diff --git a/tools/libxl/libxl_types.idl > > > b/tools/libxl/libxl_types.idl > > > index d6ef9a2..082fed8 100644 > > > ---

Re: [Xen-devel] [PATCH 3/9] xen: sched: make locking for {insert, remove}_vcpu consistent

2015-10-08 Thread George Dunlap
On 08/10/15 16:20, Andrew Cooper wrote: > On 08/10/15 15:58, George Dunlap wrote: >> On 29/09/15 18:31, Andrew Cooper wrote: >>> On 29/09/15 17:55, Dario Faggioli wrote: The insert_vcpu() scheduler hook is called with an inconsistent locking strategy. In fact, it is sometimes invoked

Re: [Xen-devel] Regression: qemu crash of hvm domUs with spice (backtrace included)

2015-10-08 Thread Andreas Kinzler
Is this still current? I made an interesting observation: I had no problems with SPICE and vanilla Xen 4.5.1 when using it on Gentoo with glibc 2.19/gcc 4.6.4. Segfaults started when I switched to glibc 2.20/gcc 4.9.3 - I did not change Xen source code at all. All this might be related to: htt

Re: [Xen-devel] Regression: qemu crash of hvm domUs with spice (backtrace included)

2015-10-08 Thread Andreas Kinzler
Is this still current? I made an interesting observation: I had no problems with SPICE and vanilla Xen 4.5.1 when using it on Gentoo with glibc 2.19/gcc 4.6.4. Segfaults started when I switched to glibc 2.20/gcc 4.9.3 - I did not change Xen source code at all. All this might be related to: htt

Re: [Xen-devel] [PATCH] tools: remove unused wrappers for python

2015-10-08 Thread Zhigang Wang
On 10/08/2015 11:27 AM, Ian Campbell wrote: > On Thu, 2015-10-08 at 11:09 -0400, Zhigang Wang wrote: >> On 10/08/2015 10:38 AM, Ian Campbell wrote: >>> On Tue, 2015-10-06 at 11:24 -0400, Zhigang Wang wrote: We use these extentions along with xend XMLRPC API/xm. Even when move to xl,

Re: [Xen-devel] [RFC OSSTEST] ap-fetch-*: Support $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder

2015-10-08 Thread Ian Campbell
On Thu, 2015-10-08 at 17:00 +0100, Ian Jackson wrote: > Ian Campbell writes ("[RFC OSSTEST] ap-fetch-*: Support > $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder"): > > And use this in standalone-generate-dump-flight-runvars. In general I > > don't think we are interested in the specific

Re: [Xen-devel] [PATCH v7 31/32] libxl: allow the creation of HVM domains without a device model.

2015-10-08 Thread Roger Pau Monné
El 08/10/15 a les 16.36, Ian Campbell ha escrit: > On Fri, 2015-10-02 at 17:49 +0200, Roger Pau Monne wrote: >> diff --git a/tools/libxl/libxl_types.idl b/tools/libxl/libxl_types.idl >> index d6ef9a2..082fed8 100644 >> --- a/tools/libxl/libxl_types.idl >> +++ b/tools/libxl/libxl_types.idl >> @@ -98

Re: [Xen-devel] [PATCH OSSTEST v2 0/8] Fixes for switching to Jessie as base OS for test hosts

2015-10-08 Thread Ian Campbell
On Tue, 2015-10-06 at 13:46 +0100, Ian Campbell wrote: > Compared with last time there are a few new patches to fix things I > noticed > while testing and the prerequisites for actually deploying (f/w fix on > arndale and git cache) are now addressed. It remains the case that the > first 7 patches

Re: [Xen-devel] [PATCH] tools/python: remove broken xl binding

2015-10-08 Thread Zhigang Wang
On 10/08/2015 11:28 AM, Ian Campbell wrote: > On Thu, 2015-10-08 at 10:58 -0400, Zhigang Wang wrote: >> On 10/08/2015 10:40 AM, Ian Campbell wrote: >>> On Tue, 2015-10-06 at 15:09 -0400, Konrad Rzeszutek Wilk wrote: On Tue, Oct 06, 2015 at 06:13:04PM +0100, Andrew Cooper wrote: > On 06/10/

Re: [Xen-devel] [PATCH v7 32/32] libxl: add support for migrating HVM guests without a device model

2015-10-08 Thread Roger Pau Monné
El 02/10/15 a les 20.56, Andrew Cooper ha escrit: > On 02/10/15 16:49, Roger Pau Monne wrote: >> Only some minor libxl changes are needed in order to be able to migrate HVM >> guests without a device model, no hypervisor changes are needed. >> >> This change prevents sending the emulator context if

Re: [Xen-devel] [PATCH 7/7] xen: sched / cpupool: dump the actual value of NOW()

2015-10-08 Thread Jan Beulich
>>> On 08.10.15 at 15:12, wrote: > On 08/10/15 13:53, Dario Faggioli wrote: >> rather than its hexadecimal representation. This makes >> it easier to compare the actual system time with other >> times being printed out (e.g., deadlines in RTDS). >> >> Signed-off-by: Dario Faggioli > > Reviewed-b

Re: [Xen-devel] [PATCH OSSTEST v3 3/3] Create a flight to test OpenStack with xen-unstable and libvirt

2015-10-08 Thread Ian Campbell
On Tue, 2015-09-29 at 17:52 +0100, Ian Jackson wrote: > Anthony PERARD writes ("[PATCH OSSTEST v3 3/3] Create a flight to test > OpenStack with xen-unstable and libvirt"): > > Signed-off-by: Anthony PERARD > > > > --- > > Change in V3: > > - Switch to "track" Nova tree instead of devstack. > >

Re: [Xen-devel] [PATCH OSSTEST v4 7/7] standalone: do not rotate empty log files.

2015-10-08 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST v4 7/7] standalone: do not rotate empty log files."): > By passing -n to savelog. > > In particular this prevents the creation of an empty $log.0 on first > use when $log doesn't exist. > > Signed-off-by: Ian Campbell Acked-by: Ian Jackson ___

Re: [Xen-devel] [PATCH OSSTEST v4 4/7] standalone: Correctly quote $@ where it is used.

2015-10-08 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST v4 4/7] standalone: Correctly quote $@ where it is used."): > Else arguments with spaces become multiple arguments. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-

Re: [Xen-devel] [PATCH OSSTEST v4 2/7] cs-adjust-flight: Add job-status to report job stats

2015-10-08 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST v4 2/7] cs-adjust-flight: Add job-status to report job stats"): > The return code of sg-run-job does not reflect the state of the job, > which is instead written to the database. For the benefit of running > tests in a loop until failure add a command to retrie

Re: [Xen-devel] [PATCH OSSTEST v4 5/7] standalone: Make it possible to pass options to run-test

2015-10-08 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST v4 5/7] standalone: Make it possible to pass options to run-test"): > Currently the remainder of the comnand line is passed after the host= > ident, which allows for other idents to be given, which isn't all that > useful in practice. Acked-by: Ian Jackson _

Re: [Xen-devel] [RFC OSSTEST] ap-fetch-*: Support $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder

2015-10-08 Thread Ian Jackson
Ian Campbell writes ("[RFC OSSTEST] ap-fetch-*: Support $AP_FETCH_PLACEHOLDERS envvar which outputs a placeholder"): > And use this in standalone-generate-dump-flight-runvars. In general I > don't think we are interested in the specific revision_* runvars when > using this tool and this is quicker

Re: [Xen-devel] [PATCH 6/7] xen: sched: fix an 'off by one \t' in credit2 debug dump

2015-10-08 Thread Dario Faggioli
On Thu, 2015-10-08 at 16:42 +0100, George Dunlap wrote: > On 08/10/15 13:53, Dario Faggioli wrote: > > Signed-off-by: Dario Faggioli > > Reviewed-by: Juergen Gross > > Oh, I didn't realize this one was going to have stuff from the > previous > series as well. :-) > Just this patch. Since I was

Re: [Xen-devel] [PATCH OSSTEST] Debian: Arrange to drop "quiet" from command line of installed system

2015-10-08 Thread Ian Campbell
On Thu, 2015-10-08 at 16:52 +0100, Ian Jackson wrote: > Ian Campbell writes ("[PATCH OSSTEST] Debian: Arrange to drop "quiet" > from command line of installed system"): > > By editing /etc/default/grub in a late command iff it exists. > > > > This will effect ts-debian-{hvm,di}-install as well as

[Xen-devel] [PATCH OSSTEST v4 4/7] standalone: Correctly quote $@ where it is used.

2015-10-08 Thread Ian Campbell
Else arguments with spaces become multiple arguments. Signed-off-by: Ian Campbell --- standalone | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/standalone b/standalone index 7c7fde9..25e23ca 100755 --- a/standalone +++ b/standalone @@ -197,7 +197,7 @@ with_logging() {

[Xen-devel] [PATCH OSSTEST v4 3/7] standalone: Check job status at end of run-job.

2015-10-08 Thread Ian Campbell
Check if the job passed and if not (so status is fail, broken, running etc) then return an error. This is convenient for scripting. Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- standalone | 5 + 1 file changed, 5 insertions(+) diff --git a/standalone b/standalone index 98a323f..7

[Xen-devel] [PATCH OSSTEST v4 7/7] standalone: do not rotate empty log files.

2015-10-08 Thread Ian Campbell
By passing -n to savelog. In particular this prevents the creation of an empty $log.0 on first use when $log doesn't exist. Signed-off-by: Ian Campbell --- v2: Put -n in correct place --- standalone | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/standalone b/standalone inde

[Xen-devel] [PATCH OSSTEST v4 6/7] standalone: Use fail() from mgi-common in most places

2015-10-08 Thread Ian Campbell
Functional change is simply to prepend "$0: ", to change the exit code for unknown operation and to slightly alter the error message when no arguments are given. A few "exit 0" and "exit $rc" remain. Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- v3: New patch --- standalone | 42 ++

[Xen-devel] [PATCH OSSTEST v4 5/7] standalone: Make it possible to pass options to run-test

2015-10-08 Thread Ian Campbell
Currently the remainder of the comnand line is passed after the host= ident, which allows for other idents to be given, which isn't all that useful in practice. Instead arrange that any additional options up to a "--" marker are passed before host= and anything after are passed after. Since the o

[Xen-devel] [PATCH OSSTEST v4 2/7] cs-adjust-flight: Add job-status to report job stats

2015-10-08 Thread Ian Campbell
The return code of sg-run-job does not reflect the state of the job, which is instead written to the database. For the benefit of running tests in a loop until failure add a command to retrieve the status to stdout. Add a get-job-status command to the standalone helper script. Signed-off-by: Ian

[Xen-devel] [PATCH OSSTEST v4 0/7] Misc standalone wrapper improvements

2015-10-08 Thread Ian Campbell
Fixed up quoting of $@ (generally in a new patch, then also in the bits of the old patches). Added "standalone: do not rotate empty log files." posted separately. Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

[Xen-devel] [PATCH OSSTEST v4 1/7] cs-adjust-flight: `branch' command ought to be `branch-set'

2015-10-08 Thread Ian Campbell
Also add a doc string and since this op is not a change adjust the doc comment accordingly. Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- v3: New patch. --- cs-adjust-flight | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/cs-adjust-flight b/cs-adjust-flight in

Re: [Xen-devel] [PATCH OSSTEST] Debian: Arrange to drop "quiet" from command line of installed system

2015-10-08 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] Debian: Arrange to drop "quiet" from command line of installed system"): > By editing /etc/default/grub in a late command iff it exists. > > This will effect ts-debian-{hvm,di}-install as well as ^^ affect > ts-host-install and he

Re: [Xen-devel] [PATCH OSSTEST] Debian: Supply a defined default for firmware hostprop when checking for uefi

2015-10-08 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] Debian: Supply a defined default for firmware hostprop when checking for uefi"): > Otherwise: > Use of uninitialized value in string eq at Osstest/Debian.pm line 410. Acked-by: Ian Jackson ___ Xen-devel mailing li

Re: [Xen-devel] [PATCH OSSTEST] mfi-common: Do not enable ovmf for xen-4.4-testing build-*-prev jobs

2015-10-08 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] mfi-common: Do not enable ovmf for xen-4.4-testing build-*-prev jobs"): > That is, jobs which are building xen-4.3-testing, where ovmf was not > yet supported. > > Full diff to standalone-generate-dump-flight-runvars is: Acked-by: Ian Jackson _

Re: [Xen-devel] [PATCH 2/7] xen: sched: fix locking for insert_vcpu() in credit1 and RTDS

2015-10-08 Thread Dario Faggioli
On Thu, 2015-10-08 at 16:16 +0100, George Dunlap wrote: > On 08/10/15 13:52, Dario Faggioli wrote: > > diff --git a/xen/common/schedule.c b/xen/common/schedule.c > > index c5f640f..9aa209d 100644 > > --- a/xen/common/schedule.c > > +++ b/xen/common/schedule.c > > @@ -1488,9 +1488,7 @@ void __init

Re: [Xen-devel] [PATCH 6/7] xen: sched: fix an 'off by one \t' in credit2 debug dump

2015-10-08 Thread George Dunlap
On 08/10/15 13:53, Dario Faggioli wrote: > Signed-off-by: Dario Faggioli > Reviewed-by: Juergen Gross Oh, I didn't realize this one was going to have stuff from the previous series as well. :-) Acked-by: George Dunlap > --- > Cc: George Dunlap > --- > xen/common/sched_credit2.c |2 +- >

Re: [Xen-devel] [PATCH 5/7] xen: sched: get rid of the per domain vCPU list in Credit2

2015-10-08 Thread George Dunlap
On 08/10/15 13:53, Dario Faggioli wrote: > As, curently, there is no reason for bothering having > it and keeping it updated. > > In fact, it is only used for dumping and changing > vCPUs parameters, but that can be achieved easily with > for_each_vcpu. > > While there, improve alignment of comme

Re: [Xen-devel] [PATCH 5/7] xen: sched: get rid of the per domain vCPU list in Credit2

2015-10-08 Thread Andrew Cooper
On 08/10/15 16:32, Dario Faggioli wrote: > On Thu, 2015-10-08 at 14:56 +0100, Andrew Cooper wrote: >> On 08/10/15 13:53, Dario Faggioli wrote: >>> @@ -1443,7 +1433,7 @@ csched2_dom_cntl( >>> >>> if ( op->u.credit2.weight != 0 ) >>> { >>> -struct list_head *iter; >>>

Re: [Xen-devel] [PATCH 3/7] xen: sched: better handle (not) inserting idle vCPUs in runqueues

2015-10-08 Thread Dario Faggioli
On Thu, 2015-10-08 at 16:27 +0100, George Dunlap wrote: > On 08/10/15 13:52, Dario Faggioli wrote: > > [...] > > The check for whether we'd be inserting an idle vCPU > > in a queue, now, happens, once and for all schedulers, > > in generic code, at vCPU initialization time, while > > we can just av

Re: [Xen-devel] [PATCH] libxc: remove superpages option for pv domains

2015-10-08 Thread Wei Liu
On Thu, Oct 08, 2015 at 05:23:47PM +0200, Juergen Gross wrote: > The pv domain builder currently supports the additional flag > "superpages" to build a pv domain with 2MB pages. This feature isn't > being used by any component other than the python xc bindings. > > Remove the flag and it's support

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

2015-10-08 Thread osstest service owner
flight 62731 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/62731/ 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] [PATCH v7 27/32] xen/x86: allow HVM guests to use hypercalls to bring up vCPUs

2015-10-08 Thread Andrew Cooper
On 08/10/15 16:21, Jan Beulich wrote: On 08.10.15 at 15:35, wrote: >> El 05/10/15 a les 12.28, Andrew Cooper ha escrit: >>> On 02/10/15 16:48, Roger Pau Monne wrote: +#define SEG(b, l, a)\ +(struct segment_register){ .sel = 0,

Re: [Xen-devel] [PATCH 5/7] xen: sched: get rid of the per domain vCPU list in Credit2

2015-10-08 Thread Dario Faggioli
On Thu, 2015-10-08 at 14:56 +0100, Andrew Cooper wrote: > On 08/10/15 13:53, Dario Faggioli wrote: > > @@ -1443,7 +1433,7 @@ csched2_dom_cntl( > > > > if ( op->u.credit2.weight != 0 ) > > { > > -struct list_head *iter; > > +struct vcpu *vc; > > Any chanc

Re: [Xen-devel] [PATCH 4/7] xen: sched: get rid of the per domain vCPU list in RTDS

2015-10-08 Thread George Dunlap
On 08/10/15 13:52, Dario Faggioli wrote: > As, curently, there is no reason for bothering having > it and keeping it updated. > > In fact, it is only used for dumping and changing > vCPUs parameters, but that can be achieved easily with > for_each_vcpu. > > Signed-off-by: Dario Faggioli With An

Re: [Xen-devel] [PATCH] tools/python: remove broken xl binding

2015-10-08 Thread Ian Campbell
On Thu, 2015-10-08 at 10:58 -0400, Zhigang Wang wrote: > On 10/08/2015 10:40 AM, Ian Campbell wrote: > > On Tue, 2015-10-06 at 15:09 -0400, Konrad Rzeszutek Wilk wrote: > > > On Tue, Oct 06, 2015 at 06:13:04PM +0100, Andrew Cooper wrote: > > > > On 06/10/15 17:57, Wei Liu wrote: > > > > > Various p

Re: [Xen-devel] [PATCH 3/7] xen: sched: better handle (not) inserting idle vCPUs in runqueues

2015-10-08 Thread George Dunlap
On 08/10/15 13:52, Dario Faggioli wrote: > Idle vCPUs should never really be explicitly inserted > in any of the schedulers' runqueue. In fact, they are > just put in execution immediately (either on boot, or > when a pCPU is assigned to a cpupool), and it will be > the first scheduling decision th

Re: [Xen-devel] [PATCH] tools: remove unused wrappers for python

2015-10-08 Thread Ian Campbell
On Thu, 2015-10-08 at 11:09 -0400, Zhigang Wang wrote: > On 10/08/2015 10:38 AM, Ian Campbell wrote: > > On Tue, 2015-10-06 at 11:24 -0400, Zhigang Wang wrote: > > > We use these extentions along with xend XMLRPC API/xm. Even when move > > > to > > > xl, this will give us a choice to reserve some l

[Xen-devel] [PATCH] libxc: remove superpages option for pv domains

2015-10-08 Thread Juergen Gross
The pv domain builder currently supports the additional flag "superpages" to build a pv domain with 2MB pages. This feature isn't being used by any component other than the python xc bindings. Remove the flag and it's support from the xc bindings and the domain builder Signed-off-by: Juergen Gros

Re: [Xen-devel] [PATCH v7 27/32] xen/x86: allow HVM guests to use hypercalls to bring up vCPUs

2015-10-08 Thread Jan Beulich
>>> On 08.10.15 at 15:35, wrote: > El 05/10/15 a les 12.28, Andrew Cooper ha escrit: >> On 02/10/15 16:48, Roger Pau Monne wrote: >>> +#define SEG(b, l, a)\ >>> +(struct segment_register){ .sel = 0, .base = (b), .limit = (l), \ >>> +

Re: [Xen-devel] [OSSTEST PATCH v3 0/3] Test case for cpupools

2015-10-08 Thread Ian Campbell
On Sat, 2015-10-03 at 02:39 +0200, Dario Faggioli wrote: > There are some host related considerations. In fact, this test case > requires > that an host with at least 2 pCPUs is used. v2 was failing the test, if that > was not the case. Now, I'm just skipping doing pretty much everything, but I'm >

Re: [Xen-devel] [PATCH 3/9] xen: sched: make locking for {insert, remove}_vcpu consistent

2015-10-08 Thread Andrew Cooper
On 08/10/15 15:58, George Dunlap wrote: > On 29/09/15 18:31, Andrew Cooper wrote: >> On 29/09/15 17:55, Dario Faggioli wrote: >>> The insert_vcpu() scheduler hook is called with an >>> inconsistent locking strategy. In fact, it is sometimes >>> invoked while holding the runqueue lock and sometimes

Re: [Xen-devel] [PATCH V7 3/7] libxl: add pvusb API

2015-10-08 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH V7 3/7] libxl: add pvusb API"): > Since this is part of a new "controller" abstraction we do in theory have > the freedom to do things differently, but it seems to me that having > something as basic as the list operation differ for devices vs. controller > would do

Re: [Xen-devel] [PATCH 2/7] xen: sched: fix locking for insert_vcpu() in credit1 and RTDS

2015-10-08 Thread George Dunlap
On 08/10/15 13:52, Dario Faggioli wrote: > The insert_vcpu scheduler hook is called with an inconsistent > locking strategy. In fact, it is sometimes invoked while > holding the runqueue lock and sometimes when that is not the > case. > > For instance, in case of schedule_cpu_switch() the lock is

Re: [Xen-devel] [PATCH] tools: remove unused wrappers for python

2015-10-08 Thread Zhigang Wang
On 10/08/2015 10:38 AM, Ian Campbell wrote: > On Tue, 2015-10-06 at 11:24 -0400, Zhigang Wang wrote: >> We use these extentions along with xend XMLRPC API/xm. Even when move to >> xl, this will give us a choice to reserve some logic. > > There are a lot of interfaces here, are you using all of the

Re: [Xen-devel] RFC: LTS and stable release scheme

2015-10-08 Thread Jan Beulich
>>> On 08.10.15 at 16:23, wrote: > On Thu, Oct 08, 2015 at 06:13:27AM -0600, Jan Beulich wrote: >> >>> On 08.10.15 at 13:10, wrote: >> > I fail to get the idea why this would be a problem. Maybe you're seeing >> > every backport as your sole responsibility? From Xen project's point of >> > view,

Re: [Xen-devel] [PATCH] tools/python: remove broken xl binding

2015-10-08 Thread Zhigang Wang
On 10/08/2015 10:40 AM, Ian Campbell wrote: > On Tue, 2015-10-06 at 15:09 -0400, Konrad Rzeszutek Wilk wrote: >> On Tue, Oct 06, 2015 at 06:13:04PM +0100, Andrew Cooper wrote: >>> On 06/10/15 17:57, Wei Liu wrote: Various people say this binding doesn't compile or doesn't work. Remove >>>

Re: [Xen-devel] [PATCH 3/9] xen: sched: make locking for {insert, remove}_vcpu consistent

2015-10-08 Thread George Dunlap
On 29/09/15 18:31, Andrew Cooper wrote: > On 29/09/15 17:55, Dario Faggioli wrote: >> The insert_vcpu() scheduler hook is called with an >> inconsistent locking strategy. In fact, it is sometimes >> invoked while holding the runqueue lock and sometimes >> when that is not the case. >> >> In other w

Re: [Xen-devel] [PATCH V7 3/7] libxl: add pvusb API

2015-10-08 Thread Ian Campbell
On Thu, 2015-10-08 at 15:41 +0100, Ian Jackson wrote: > > +libxl_device_usbctrl * > > +libxl_device_usbctrl_list(libxl_ctx *ctx, uint32_t domid, int *num) > > +{ > > This function should return an rc, and the list should come in an out > parameter. For better of worse libxl.h defines the general

Re: [Xen-devel] [PATCH V7 3/7] libxl: add pvusb API

2015-10-08 Thread Ian Jackson
Chunyan Liu writes ("[PATCH V7 3/7] libxl: add pvusb API"): > Add pvusb APIs, including: ... > +/* Utility to read backend xenstore keys */ > +#define READ_BACKEND(tgc, subpath)\ > +libxl__xs_read(tgc, XBT_NULL, GCSPRINTF("%s/" subpath, be_path)) > +

Re: [Xen-devel] [PATCH] tools/python: remove broken xl binding

2015-10-08 Thread Ian Campbell
On Tue, 2015-10-06 at 15:09 -0400, Konrad Rzeszutek Wilk wrote: > On Tue, Oct 06, 2015 at 06:13:04PM +0100, Andrew Cooper wrote: > > On 06/10/15 17:57, Wei Liu wrote: > > > Various people say this binding doesn't compile or doesn't work. > > > Remove > > > it for the benefit of xl feature developme

Re: [Xen-devel] [PATCH] tools: remove unused wrappers for python

2015-10-08 Thread Ian Campbell
On Tue, 2015-10-06 at 11:24 -0400, Zhigang Wang wrote: > We use these extentions along with xend XMLRPC API/xm. Even when move to > xl, this will give us a choice to reserve some logic. There are a lot of interfaces here, are you using all of them? If not then could you enumerate the ones you care

Re: [Xen-devel] [PATCH v7 31/32] libxl: allow the creation of HVM domains without a device model.

2015-10-08 Thread Ian Campbell
On Fri, 2015-10-02 at 17:49 +0200, Roger Pau Monne wrote: > diff --git a/tools/libxl/libxl_types.idl b/tools/libxl/libxl_types.idl > index d6ef9a2..082fed8 100644 > --- a/tools/libxl/libxl_types.idl > +++ b/tools/libxl/libxl_types.idl > @@ -98,6 +98,7 @@ libxl_device_model_version = > Enumeration(

  1   2   >