[Xen-devel] [PATCH v4 14/14] docs: document xenstore domain xenstore paths

2016-01-18 Thread Juergen Gross
Add the xenstore domain specific xenstore paths to docs/misc/xenstore-paths.markdown Signed-off-by: Juergen Gross Acked-by: Ian Campbell --- V4: fixed typo Signed-off-by: Juergen Gross --- docs/misc/xenstore-paths.markdown | 10 ++ 1 file changed, 10 insertions(+) diff --git a/docs/m

[Xen-devel] [PATCH v4 00/14] xenstore: make it easier to run xenstore in a domain

2016-01-18 Thread Juergen Gross
Xen supports to run xenstore in a dedicated domain. It is, however, a setup which isn't easy to configure. Today flask is required for full functionality and the resulting xenstore domain is not configurable in the same way as the xenstore daemon. This patch series enables running a xenstore domai

[Xen-devel] [PATCH v4 09/14] xenstore: make use of the "xenstore domain" flag

2016-01-18 Thread Juergen Gross
Create the xenstore domain with the xenstore flag specified. This enables us to test whether such a domain is already running before we create it. As there ought to be only one xenstore in the system we don't need to start another one. Signed-off-by: Juergen Gross Acked-by: Ian Campbell --- V3:

[Xen-devel] [PATCH v4 11/14] tools: split up xen-init-dom0.c

2016-01-18 Thread Juergen Gross
Split up tools/helpers/xen-init-dom0.c in order to prepare reusing generation of the json configuration by init-xenstore-domain.c. Signed-off-by: Juergen Gross Acked-by: Ian Campbell --- tools/helpers/Makefile| 2 +- tools/helpers/init-dom-json.c | 59 ++

[Xen-devel] [PATCH v4 04/14] xenstore: adjust coding style of init-xenstore-domain.c

2016-01-18 Thread Juergen Gross
Adjust the coding style of init-xenstore-domain.c to the general Xen coding style. Signed-off-by: Juergen Gross Acked-by: Ian Campbell --- tools/helpers/init-xenstore-domain.c | 221 --- 1 file changed, 126 insertions(+), 95 deletions(-) diff --git a/tools/helpe

[Xen-devel] [PATCH v4 08/14] xenstore: modify init-xenstore-domain parameter syntax

2016-01-18 Thread Juergen Gross
init-xenstore-domain takes only positional parameters today. Change this to a more flexible parameter syntax allowing to specify additional options or to omit some. Today the supported usage is: init-xenstore-domain [] Modify this to: init-xenstore-domain --kernel

[Xen-devel] [PATCH v4 03/14] xenstore: move init-xenstore-domain to tools/helpers

2016-01-18 Thread Juergen Gross
The program init-xenstore-domain to start a xenstore domain instead of the xenstored daemon is built, but not installed. Change that. Move the program to a new tools subdirectory "helpers" to be able to use libxl in a later patch. Otherwise a dependency loop will be created. Signed-off-by: Juerge

[Xen-devel] [PATCH v4 10/14] xenstore: add init-xenstore-domain parameter to specify cmdline

2016-01-18 Thread Juergen Gross
Add a parameter to init-xenstore-domain for support of arbitrary parameters for the xenstore domain. Signed-off-by: Juergen Gross Acked-by: Ian Campbell Acked-by: Daniel De Graaf --- tools/helpers/init-xenstore-domain.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) dif

[Xen-devel] [PATCH v4 02/14] libxl: provide a flag in dominfo to avoid stopping it

2016-01-18 Thread Juergen Gross
Add a "never_stop" flag to dominfo as indicator for the toolstack that this domain is to be kept running. For now it is being set for xenstore domain, but there might be other domains in the future. Signed-off-by: Juergen Gross Acked-by: Ian Campbell --- V3: flag in dominfo instead function to g

[Xen-devel] [PATCH v4 07/14] xenstore: add error messages to init-xenstore-domain

2016-01-18 Thread Juergen Gross
Additional add some diagnostic messages to the program to have an idea why it failed. Signed-off-by: Juergen Gross Acked-by: Ian Campbell Acked-by: Daniel De Graaf --- tools/helpers/init-xenstore-domain.c | 66 ++-- 1 file changed, 64 insertions(+), 2 deletions(

[Xen-devel] [PATCH v4 12/14] xenstore: write xenstore domain data to xenstore

2016-01-18 Thread Juergen Gross
After starting the xenstore domain write the basic data (domid, name and memory values) to the xenstore. This makes the domain appear correctly in xl list. Create a stub json object in order to make e.g. xl list -l happy. Add a new option to init-xenstore-domain to be able to specify the domain's

[Xen-devel] [PATCH v4 05/14] libxl: move xen-init-dom0 to tools/helpers

2016-01-18 Thread Juergen Gross
Move xen-init-dom0 from tools/libxl to tools/helpers, as it is just a helper program. Signed-off-by: Juergen Gross Acked-by: Ian Campbell --- V3: adjust .gitignore as requested by Ian Campbell use libxentoollog instead of libxc as requested by Ian Campbell Signed-off-by: Juergen Gross ---

[Xen-devel] [PATCH v4 01/14] libxc: support new xenstore domain flag in libxc

2016-01-18 Thread Juergen Gross
Support the xenstore domain flag for obtaining domain info. Signed-off-by: Juergen Gross Acked-by: Ian Campbell --- V3: changed name of flag from "xs_domain" to "xenstore" as requested by Ian Campbell Signed-off-by: Juergen Gross --- tools/libxc/include/xenctrl.h | 2 +- tools/libxc/xc_do

[Xen-devel] [PATCH v4 06/14] xenstore: destroy xenstore domain in case of error after creating it

2016-01-18 Thread Juergen Gross
When creating a xenstore domain via init-xenstore-domain destroy it in case of an error occurred after calling xc_domain_create(). Signed-off-by: Juergen Gross Acked-by: Daniel De Graaf Acked-by: Ian Campbell --- tools/helpers/init-xenstore-domain.c | 4 +++- 1 file changed, 3 insertions(+), 1

[Xen-devel] [PATCH v4 13/14] tools: don't stop xenstore domain when stopping dom0

2016-01-18 Thread Juergen Gross
When restarting or shutting down dom0 the xendomains script tries to stop all other domains. Don't do this for the xenstore domain, as it might survive a dom0 reboot in the future. The same applies to xl shutdown --all. Here the xenstore domain is flagged as "never stop". Signed-off-by: Juergen G

Re: [Xen-devel] [PATCH v3] x86/p2m: use large pages for MMIO mappings

2016-01-18 Thread Jan Beulich
>>> On 15.01.16 at 15:55, wrote: > On Fri, 2016-01-15 at 07:39 -0700, Jan Beulich wrote: >> I don't think I agree - there are two models. The meaning of >> -E2BIG for the caller to retry with a smaller amount doesn't exist in >> the new model anymore, and hence libxc wouldn't need to deal >> with

Re: [Xen-devel] [v3,11/41] mips: reuse asm-generic/barrier.h

2016-01-18 Thread Herbert Xu
Paul E. McKenney wrote: > > You could use SYNC_ACQUIRE() to implement read_barrier_depends() and > smp_read_barrier_depends(), but SYNC_RMB probably does not suffice. > The reason for this is that smp_read_barrier_depends() must order the > pointer load against any subsequent read or write through

Re: [Xen-devel] [PATCH v10 6/7] vmx: VT-d posted-interrupt core logic handling

2016-01-18 Thread Jan Beulich
>>> On 18.01.16 at 02:20, wrote: > Hi Jan, have you had a chance to look at this remaining one so far? No, not yet, sorry. Jan ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v10 6/7] vmx: VT-d posted-interrupt core logic handling

2016-01-18 Thread Wu, Feng
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Monday, January 18, 2016 4:41 PM > To: Wu, Feng > Cc: andrew.coop...@citrix.com; dario.faggi...@citrix.com; > george.dun...@eu.citrix.com; Tian, Kevin ; xen- > de...@lists.xen.org; k...@xen.org > Subject: RE: RE:

Re: [Xen-devel] [PATCH 4/4] hvmloader: add support to load extra ACPI tables from qemu

2016-01-18 Thread Jan Beulich
>>> On 18.01.16 at 01:52, wrote: > On 01/15/16 10:10, Jan Beulich wrote: >> >>> On 29.12.15 at 12:31, wrote: >> > NVDIMM devices are detected and configured by software through >> > ACPI. Currently, QEMU maintains ACPI tables of vNVDIMM devices. This >> > patch extends the existing mechanism in h

[Xen-devel] [distros-debian-sid test] 38654: trouble: blocked/broken

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

Re: [Xen-devel] [PATCH v3 2/3] XENVER_build_id: Provide ld-embedded build-ids (v8)

2016-01-18 Thread Jan Beulich
>>> On 15.01.16 at 22:49, wrote: >> > --- a/Config.mk >>> +++ b/Config.mk >>> @@ -126,6 +126,17 @@ endef >>> check-$(gcc) = $(call cc-ver-check,CC,0x040100,"Xen requires at least > gcc-4.1") >>> $(eval $(check-y)) >>> >>> +ld-ver = $(shell if [ $$((`$(1) --version | head -1 | sed 's/[^0-9]/ /g'

Re: [Xen-devel] [xen-unstable test] 77945: regressions - FAIL [and 2 more messages]

2016-01-18 Thread Andrew Cooper
On 18/01/2016 07:49, Jan Beulich wrote: On 15.01.16 at 18:42, wrote: >> On Fri, 2016-01-15 at 17:24 +, Andrew Cooper wrote: >>> On 15/01/16 17:15, Jan Beulich wrote: >>> On 15.01.16 at 18:06, wrote: > On Thu, 2016-01-14 at 16:27 +, Ian Jackson wrote: >> * I don't have a

Re: [Xen-devel] [PATCH v10 6/7] vmx: VT-d posted-interrupt core logic handling

2016-01-18 Thread Jan Beulich
>>> On 18.01.16 at 09:45, wrote: > It has been pending for such a long time, and Dario and Kevin both reviewed > it, it is pending on your comments, could you tell when you will look at it? I can't really, and I don't think I've seen e.g. Dario give his Reviewed-by. This is a complicated topic, w

Re: [Xen-devel] [PATCH v3 4/4] x86/PV: enable the emulated PIT

2016-01-18 Thread Andrew Cooper
On 18/01/2016 07:43, Jan Beulich wrote: On 15.01.16 at 18:45, wrote: >> Changes since v2: >> - Change 'if ( (a && b) || (!a && c) )' into 'if ( a ? b : c )'. > Thanks, but after some more thinking about it I'm afraid there are > a few more aspects to consider here: > >> --- a/xen/arch/x86/do

Re: [Xen-devel] [BUG] Xen Remus, DRBD Protocol D and old Linux kernels

2016-01-18 Thread Ian Campbell
Just adding the Remus maintainers to the CC. On Fri, 2016-01-15 at 15:26 -0300, Pato Sáinz wrote: > Good evening, people. > > More than a bug, it's an issue, a big one IMO. > > Even though information on the Remus project and its implementation is kind of > scarce on the internet and is usually

Re: [Xen-devel] [xen-unstable test] 77945: regressions - FAIL [and 2 more messages]

2016-01-18 Thread Ian Campbell
On Mon, 2016-01-18 at 00:49 -0700, Jan Beulich wrote: > > > > On 15.01.16 at 18:42, wrote: > > On Fri, 2016-01-15 at 17:24 +, Andrew Cooper wrote: > > > On 15/01/16 17:15, Jan Beulich wrote: > > > > > > > On 15.01.16 at 18:06, wrote: > > > > > On Thu, 2016-01-14 at 16:27 +, Ian Jackson wr

Re: [Xen-devel] Nested virtualization off VMware vSphere 6.0 with EL6 guests crashes on Xen 4.6

2016-01-18 Thread Jan Beulich
>>> On 15.01.16 at 22:39, wrote: > On Tue, Jan 12, 2016 at 02:22:03AM -0700, Jan Beulich wrote: >> Since we can (I hope) pretty much exclude a paging type, the >> ASSERT() must have triggered because of vapic_pg being NULL. >> That might be verifiable without extra printk()s, just by checking >> t

Re: [Xen-devel] [PATCH v3 4/4] x86/PV: enable the emulated PIT

2016-01-18 Thread Jan Beulich
>>> On 18.01.16 at 10:29, wrote: > On 18/01/2016 07:43, Jan Beulich wrote: > On 15.01.16 at 18:45, wrote: >>> Changes since v2: >>> - Change 'if ( (a && b) || (!a && c) )' into 'if ( a ? b : c )'. >> Thanks, but after some more thinking about it I'm afraid there are >> a few more aspects to

Re: [Xen-devel] [xen-unstable test] 77945: regressions - FAIL [and 2 more messages]

2016-01-18 Thread Jan Beulich
>>> On 18.01.16 at 10:41, wrote: > On Mon, 2016-01-18 at 00:49 -0700, Jan Beulich wrote: >> > > > On 15.01.16 at 18:42, wrote: >> > On Fri, 2016-01-15 at 17:24 +, Andrew Cooper wrote: >> > > On 15/01/16 17:15, Jan Beulich wrote: >> > > > > > > On 15.01.16 at 18:06, wrote: >> > > > > On Thu,

Re: [Xen-devel] [PATCH v3 4/4] x86/PV: enable the emulated PIT

2016-01-18 Thread Roger Pau Monné
El 18/01/16 a les 8.43, Jan Beulich ha escrit: On 15.01.16 at 18:45, wrote: >> Changes since v2: >> - Change 'if ( (a && b) || (!a && c) )' into 'if ( a ? b : c )'. > > Thanks, but after some more thinking about it I'm afraid there are > a few more aspects to consider here: > >> --- a/xen/

Re: [Xen-devel] [PATCH v3 4/4] x86/PV: enable the emulated PIT

2016-01-18 Thread Jan Beulich
>>> On 18.01.16 at 10:50, wrote: > El 18/01/16 a les 8.43, Jan Beulich ha escrit: > On 15.01.16 at 18:45, wrote: >>> Changes since v2: >>> - Change 'if ( (a && b) || (!a && c) )' into 'if ( a ? b : c )'. >> >> Thanks, but after some more thinking about it I'm afraid there are >> a few more

Re: [Xen-devel] [PATCH v7 0/2] public/io/netif.h: support for toeplitz hashing

2016-01-18 Thread Paul Durrant
> -Original Message- > From: Paul Durrant [mailto:paul.durr...@citrix.com] > Sent: 12 January 2016 09:59 > To: xen-de...@lists.xenproject.org > Cc: Paul Durrant > Subject: [PATCH v7 0/2] public/io/netif.h: support for toeplitz hashing > > This series documents changes needed to support toe

Re: [Xen-devel] Uniform commands for booting xen

2016-01-18 Thread Ian Campbell
On Mon, 2016-01-11 at 15:06 +0100, Vladimir 'φ-coder/phcoder' Serbinenko wrote: > On 13.11.2015 10:50, Ian Campbell wrote: > > On Fri, 2015-11-13 at 12:04 +0300, Andrei Borzenkov wrote: > > > > How do you express modules other than kernel+initrd in that > > > > scheme, without grub needing to be aw

Re: [Xen-devel] [BUG] Assertion '(sp == 0) || (peoi[sp-1].vector < vector)' failed at irq.c:1163

2016-01-18 Thread Jan Beulich
>>> On 18.01.16 at 00:07, wrote: > There we go :-/ . Log attached from boot to assertion-failure with > loglvl=all guest_loglvl=all . Some of the log output might be a bit > cryptic, they are notes to myself from local boot-scripts, basically > firing up my router/name-server/dhcp-server and waiti

Re: [Xen-devel] [PATCH v2 01/16] Xen: ACPI: Hide UART used by Xen

2016-01-18 Thread Mark Rutland
On Fri, Jan 15, 2016 at 02:55:14PM +0800, Shannon Zhao wrote: > From: Shannon Zhao > > ACPI 6.0 introduces a new table STAO to list the devices which are used > by Xen and can't be used by Dom0. On Xen virtual platforms, the physical > UART is used by Xen. So here it hides UART from Dom0. > > Si

Re: [Xen-devel] [BUG] Assertion '(sp == 0) || (peoi[sp-1].vector < vector)' failed at irq.c:1163

2016-01-18 Thread Andrew Cooper
On 18/01/16 10:31, Jan Beulich wrote: On 18.01.16 at 00:07, wrote: >> There we go :-/ . Log attached from boot to assertion-failure with >> loglvl=all guest_loglvl=all . Some of the log output might be a bit >> cryptic, they are notes to myself from local boot-scripts, basically >> firing up

Re: [Xen-devel] [PATCH v4 09/10] arm/uart: Rename dt-uart.c to arm-uart.c

2016-01-18 Thread Jan Beulich
>>> On 16.01.16 at 06:01, wrote: > From: Shannon Zhao > > Since we will add ACPI initialization for UART in this file later, > rename it with a generic name. > > Signed-off-by: Shannon Zhao > --- > v4: split the original patch to renaming this and adding ACPI parts. > --- > MAINTAINERS

Re: [Xen-devel] [PATCH v3 4/4] x86/PV: enable the emulated PIT

2016-01-18 Thread Andrew Cooper
On 18/01/16 09:44, Jan Beulich wrote: On 18.01.16 at 10:29, wrote: >> On 18/01/2016 07:43, Jan Beulich wrote: >> On 15.01.16 at 18:45, wrote: Changes since v2: - Change 'if ( (a && b) || (!a && c) )' into 'if ( a ? b : c )'. >>> Thanks, but after some more thinking about it I'

Re: [Xen-devel] [PATCH v2 11/16] ARM64: ACPI: Check if it runs on Xen to enable or disable ACPI

2016-01-18 Thread Mark Rutland
On Fri, Jan 15, 2016 at 02:55:24PM +0800, Shannon Zhao wrote: > From: Shannon Zhao > > When it's a Xen domain0 booting with ACPI, it will supply a /chosen and > a /hypervisor node in DT. So check if it needs to enable ACPI. > > Signed-off-by: Shannon Zhao > --- > arch/arm64/kernel/acpi.c | 12

Re: [Xen-devel] [PATCH v4 04/10] x86/hvm: Collect information of TSC scaling ratio

2016-01-18 Thread Egger, Christoph
On 17/01/16 22:58, Haozhong Zhang wrote: > Both VMX TSC scaling and SVM TSC ratio use the 64-bit TSC scaling ratio, > but the number of fractional bits of the ratio is different between VMX > and SVM. This patch adds the architecture code to collect the number of > fractional bits and other related

Re: [Xen-devel] Questions about the use of idle_vcpu[]

2016-01-18 Thread George Dunlap
On Fri, Jan 15, 2016 at 1:04 AM, Tianyang Chen wrote: > Hello, > > I'm confused by the use of idle_vcpu[] in Xen schedulers. In credit, credit2 > and rtds schedulers, they are used in conjunction with variable > "tasklet_work_scheduled" like this: > > if ( tasklet_work_scheduled ) > { > snext

Re: [Xen-devel] [PATCH v2 12/16] ARM: Xen: Document UEFI support on Xen ARM virtual platforms

2016-01-18 Thread Mark Rutland
On Fri, Jan 15, 2016 at 02:55:25PM +0800, Shannon Zhao wrote: > From: Shannon Zhao > > Add a "uefi" node under /hypervisor node in FDT, then Linux kernel could > scan this to get the UEFI information. > > Signed-off-by: Shannon Zhao > --- > Documentation/devicetree/bindings/arm/xen.txt | 42 >

Re: [Xen-devel] [BUG] Assertion '(sp == 0) || (peoi[sp-1].vector < vector)' failed at irq.c:1163

2016-01-18 Thread Jan Beulich
>>> On 18.01.16 at 11:35, wrote: > On 18/01/16 10:31, Jan Beulich wrote: > On 18.01.16 at 00:07, wrote: >>> There we go :-/ . Log attached from boot to assertion-failure with >>> loglvl=all guest_loglvl=all . Some of the log output might be a bit >>> cryptic, they are notes to myself from loc

Re: [Xen-devel] [PATCH v2 13/16] ARM: XEN: Set EFI_PARAVIRT if Xen supports EFI

2016-01-18 Thread Mark Rutland
On Fri, Jan 15, 2016 at 02:55:26PM +0800, Shannon Zhao wrote: > From: Shannon Zhao > > Check if there is "uefi" node in the DT. If so, set EFI_PARAVIRT flag. > > Signed-off-by: Shannon Zhao > --- > arch/arm/xen/enlighten.c | 23 +++ > arch/arm64/kernel/efi.c | 5 + >

Re: [Xen-devel] Questions about the use of idle_vcpu[]

2016-01-18 Thread Dario Faggioli
On Mon, 2016-01-18 at 10:47 +, George Dunlap wrote: > On Fri, Jan 15, 2016 at 1:04 AM, Tianyang Chen > > > If an idle vcpu is picked, the ret.time is set accordingly in both > > credit > > and credit2 by checking whether snext is idle. if so, credit > > returns -1 and > > credit2 returns 2ms.

[Xen-devel] [ovmf test] 78398: regressions - FAIL

2016-01-18 Thread osstest service owner
flight 78398 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/78398/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-ovmf-amd64 15 guest-localmigrate/x10 fail REGR. vs. 65543 test-amd64-amd64-xl-qem

Re: [Xen-devel] [PATCH v2 14/16] Xen: EFI: Parse DT parameters for Xen specific UEFI

2016-01-18 Thread Mark Rutland
On Fri, Jan 15, 2016 at 02:55:27PM +0800, Shannon Zhao wrote: > From: Shannon Zhao > > Add a new function to parse DT parameters for Xen specific UEFI just > like the way for normal UEFI. Then it could reuse the existing codes. > > Signed-off-by: Shannon Zhao > --- > drivers/firmware/efi/efi.c

Re: [Xen-devel] [PATCH v3 4/4] x86/PV: enable the emulated PIT

2016-01-18 Thread Jan Beulich
>>> On 18.01.16 at 11:41, wrote: > On 18/01/16 09:44, Jan Beulich wrote: > On 18.01.16 at 10:29, wrote: >>> On 18/01/2016 07:43, Jan Beulich wrote: >>> On 15.01.16 at 18:45, wrote: > Changes since v2: > - Change 'if ( (a && b) || (!a && c) )' into 'if ( a ? b : c )'. Thanks

Re: [Xen-devel] [PATCH v2 16/16] ARM64: XEN: Initialize Xen specific UEFI runtime services

2016-01-18 Thread Mark Rutland
On Fri, Jan 15, 2016 at 02:55:29PM +0800, Shannon Zhao wrote: > From: Shannon Zhao > > When running on Xen hypervisor, runtime services are supported through > hypercall. So call Xen specific function to initialize runtime services. > > Signed-off-by: Shannon Zhao > --- > arch/arm/xen/enlighte

Re: [Xen-devel] [PATCH] xen/gntdev: Don't allocate struct gntdev_copy_batch on stack

2016-01-18 Thread David Vrabel
On 15/01/16 19:43, Boris Ostrovsky wrote: > struct gntdev_copy_batch is over 1300 bytes in size, we shouldn't > put it on stack. > > Some compilers (e.g. 5.2.1) complain: > drivers/xen/gntdev.c: In function ‘gntdev_ioctl_grant_copy.isra.5’: > drivers/xen/gntdev.c:949:1: warning: the frame size o

Re: [Xen-devel] [xen-unstable test] 77945: regressions - FAIL [and 2 more messages]

2016-01-18 Thread Ian Campbell
On Mon, 2016-01-18 at 02:47 -0700, Jan Beulich wrote: > Ugly. Could we live with that until #1 and #2 get put in place? #1 is trivial (see below). #2 is, as noted in my original mail, something which while it logically belongs between #1 and #3 could be deferred. > Otherwise it looks very much

Re: [Xen-devel] [RFC/WIP] xen: clk: introudce pvclk for device passthrough

2016-01-18 Thread George Dunlap
On Sat, Jan 16, 2016 at 5:22 AM, Peng Fan wrote: > This patch was just a initial patch, not sure whether this way > is ok from you side for handlding clk when doing platform device > passhthrough. Any comments are appreciated, and your comments may > give me a better direction. Hey Peng, Just sp

Re: [Xen-devel] [RFC/WIP] xen: clk: introudce pvclk for device passthrough

2016-01-18 Thread David Vrabel
On 16/01/16 05:22, Peng Fan wrote: > This patch was just a initial patch, not sure whether this way > is ok from you side for handlding clk when doing platform device > passhthrough. Any comments are appreciated, and your comments may > give me a better direction. There's no documentation on the i

Re: [Xen-devel] [xen-unstable test] 77945: regressions - FAIL [and 2 more messages]

2016-01-18 Thread Ian Jackson
Ian Campbell writes ("Re: [Xen-devel] [xen-unstable test] 77945: regressions - FAIL [and 2 more messages]"): > On Mon, 2016-01-18 at 02:47 -0700, Jan Beulich wrote: > > Ugly. Could we live with that until #1 and #2 get put in place? > > #1 is trivial (see below). Acked-by: Ian Jackson > #2 is,

Re: [Xen-devel] [xen-unstable test] 77945: regressions - FAIL [and 2 more messages]

2016-01-18 Thread Ian Campbell
On Mon, 2016-01-18 at 11:28 +, Ian Jackson wrote: > Ian Campbell writes ("Re: [Xen-devel] [xen-unstable test] 77945: > regressions - FAIL [and 2 more messages]"): > > On Mon, 2016-01-18 at 02:47 -0700, Jan Beulich wrote: > > > Ugly. Could we live with that until #1 and #2 get put in place? > >

Re: [Xen-devel] [PATCH v2 01/16] Xen: ACPI: Hide UART used by Xen

2016-01-18 Thread Stefano Stabellini
On Mon, 18 Jan 2016, Mark Rutland wrote: > On Fri, Jan 15, 2016 at 02:55:14PM +0800, Shannon Zhao wrote: > > From: Shannon Zhao > > > > ACPI 6.0 introduces a new table STAO to list the devices which are used > > by Xen and can't be used by Dom0. On Xen virtual platforms, the physical > > UART is

Re: [Xen-devel] [PATCH v2 07/16] Xen: ARM: Add support for mapping AMBA device mmio

2016-01-18 Thread Stefano Stabellini
On Fri, 15 Jan 2016, Shannon Zhao wrote: > From: Shannon Zhao > > Add a bus_notifier for AMBA bus device in order to map the device > mmio regions when DOM0 booting with ACPI. > > Signed-off-by: Shannon Zhao Reviewed-by: Stefano Stabellini > drivers/xen/arm-device.c | 43 ++

Re: [Xen-devel] [xen-unstable test] 77945: regressions - FAIL [and 2 more messages]

2016-01-18 Thread Ian Campbell
On Mon, 2016-01-18 at 11:36 +, Ian Campbell wrote: > On Mon, 2016-01-18 at 11:28 +, Ian Jackson wrote: > > Ian Campbell writes ("Re: [Xen-devel] [xen-unstable test] 77945: > > regressions - FAIL [and 2 more messages]"): > > > On Mon, 2016-01-18 at 02:47 -0700, Jan Beulich wrote: > > > > Ugl

Re: [Xen-devel] [PATCH] tools: make FLASK utils build unconditional

2016-01-18 Thread Ian Campbell
On Fri, 2016-01-15 at 19:48 +, Andrew Cooper wrote: > On 15/01/16 17:39, Doug Goldstein wrote: > > The flask utilities only have dependencies on libxc so there's no > > downside to always building it. Distros and projects based on Xen can > > put these in a different package to not install them

[Xen-devel] [xen-unstable test] 78395: regressions - FAIL

2016-01-18 Thread osstest service owner
flight 78395 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/78395/ 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 5 xen-install fail REGR. vs. 77892 test-amd64

Re: [Xen-devel] Questions about the use of idle_vcpu[]

2016-01-18 Thread George Dunlap
On Mon, Jan 18, 2016 at 11:00 AM, Dario Faggioli wrote: >> It looks like credit2's behavior will probably prevent the processor >> from going into deeper power-saving states, and rtds' behavior might >> cause it to essentially busy-wait. >> > RTDS behavior is broken in many respect, including this

Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ

2016-01-18 Thread Stefano Stabellini
On Fri, 15 Jan 2016, Shannon Zhao wrote: > From: Shannon Zhao > > Add a new delivery type: > val[63:56] == 3: val[15:8] is flag: val[7:0] is a PPI. > To the flag, bit 0 stands the interrupt mode is edge(1) or level(0) and > bit 1 stands the interrupt polarity is active low(1) or high(0). > > Sig

Re: [Xen-devel] [RFC/WIP] xen: clk: introudce pvclk for device passthrough

2016-01-18 Thread Ian Campbell
On Mon, 2016-01-18 at 11:24 +, David Vrabel wrote: > On 16/01/16 05:22, Peng Fan wrote: > > This patch was just a initial patch, not sure whether this way > > is ok from you side for handlding clk when doing platform device > > passhthrough. Any comments are appreciated, and your comments may >

Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ

2016-01-18 Thread Andrew Cooper
On 18/01/16 12:38, Stefano Stabellini wrote: > On Fri, 15 Jan 2016, Shannon Zhao wrote: >> From: Shannon Zhao >> >> Add a new delivery type: >> val[63:56] == 3: val[15:8] is flag: val[7:0] is a PPI. >> To the flag, bit 0 stands the interrupt mode is edge(1) or level(0) and >> bit 1 stands the inte

Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ

2016-01-18 Thread Stefano Stabellini
On Mon, 18 Jan 2016, Andrew Cooper wrote: > On 18/01/16 12:38, Stefano Stabellini wrote: > > On Fri, 15 Jan 2016, Shannon Zhao wrote: > >> From: Shannon Zhao > >> > >> Add a new delivery type: > >> val[63:56] == 3: val[15:8] is flag: val[7:0] is a PPI. > >> To the flag, bit 0 stands the interrupt

Re: [Xen-devel] Questions about the use of idle_vcpu[]

2016-01-18 Thread Dario Faggioli
On Mon, 2016-01-18 at 12:37 +, George Dunlap wrote: > On Mon, Jan 18, 2016 at 11:00 AM, Dario Faggioli > wrote: > > >  > > Credit2, AFAICR, could also avoid _always_ re-setting the timer, > > but it > > does need to do that at least a few times, even when idle is > > selected, > > because of t

Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ

2016-01-18 Thread Andrew Cooper
On 18/01/16 12:46, Stefano Stabellini wrote: > On Mon, 18 Jan 2016, Andrew Cooper wrote: >> On 18/01/16 12:38, Stefano Stabellini wrote: >>> On Fri, 15 Jan 2016, Shannon Zhao wrote: From: Shannon Zhao Add a new delivery type: val[63:56] == 3: val[15:8] is flag: val[7:0] is a PP

[Xen-devel] description of gnttab_max_frames command line option.

2016-01-18 Thread Ian Campbell
docs/misc/xen-command-line.markdown says: ### gnttab\_max\_frames > `= ` Specify the maximum number of frames per grant table operation. Is this true? I thought the limit was on the number of pages which the guest may use as grant table memory, rather than the maximum number of opera

Re: [Xen-devel] [xen-unstable test] 78395: regressions - FAIL

2016-01-18 Thread Jan Beulich
>>> On 18.01.16 at 13:10, wrote: > test-amd64-i386-xl-qemut-win7-amd64 9 windows-installfail REGR. vs. > 77892 > test-amd64-i386-xl-qemuu-win7-amd64 9 windows-installfail REGR. vs. > 77892 Hmm, between all those XSM-related failures I didn't notice that we've acquired an apparent re

Re: [Xen-devel] description of gnttab_max_frames command line option.

2016-01-18 Thread Jan Beulich
>>> On 18.01.16 at 13:53, wrote: > docs/misc/xen-command-line.markdown says: > > ### gnttab\_max\_frames > > `= ` > > Specify the maximum number of frames per grant table operation. > > Is this true? I thought the limit was on the number of pages which the > guest may use as grant t

Re: [Xen-devel] description of gnttab_max_frames command line option.

2016-01-18 Thread Ian Campbell
On Mon, 2016-01-18 at 06:02 -0700, Jan Beulich wrote: > > > > On 18.01.16 at 13:53, wrote: > > [...] > Indeed [...] > Yes indeed. Thanks, I'll send a patch for both. Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-de

[Xen-devel] [PATCH] docs: correct descriptions of gnttab_max_{, maptrack}_frames

2016-01-18 Thread Ian Campbell
gnttab_max_frames incorrectly referred to numbers of grant tab operations and gnttab_max_maptrack_frames was confusingly worded. Add the default for gnttab_max_frames while here (it's currently the same on all arches since no arch uses the available arch override). Signed-off-by: Ian Campbell --

Re: [Xen-devel] [PATCH v4 04/10] acpi: Don't do traditional BIOS table scan for ARM64

2016-01-18 Thread Jan Beulich
>>> On 16.01.16 at 06:01, wrote: > --- a/xen/drivers/acpi/tables/tbxfroot.c > +++ b/xen/drivers/acpi/tables/tbxfroot.c > @@ -49,6 +49,12 @@ > #define _COMPONENT ACPI_TABLES > ACPI_MODULE_NAME("tbxfroot") > > +#ifdef CONFIG_ARM > +acpi_status __init acpi_find_root_pointer(acpi_native_u

Re: [Xen-devel] [PATCH v4 05/10] acpi: Refactor acpi_os_map_memory to be architecturally independent

2016-01-18 Thread Jan Beulich
>>> On 16.01.16 at 06:01, wrote: > --- a/xen/drivers/acpi/osl.c > +++ b/xen/drivers/acpi/osl.c > @@ -86,17 +86,7 @@ acpi_physical_address __init acpi_os_get_root_pointer(void) > void __iomem * > acpi_os_map_memory(acpi_physical_address phys, acpi_size size) > { > - if (system_state >= SYS_S

Re: [Xen-devel] [PATCH v4 01/10] x86/hvm: Scale host TSC when setting/getting guest TSC

2016-01-18 Thread Jan Beulich
>>> On 17.01.16 at 22:58, wrote: > The existing hvm_[set|get]_guest_tsc_fixed() calculate the guest TSC by > adding the TSC offset to the host TSC. When the TSC scaling is enabled, > the host TSC should be scaled first. This patch adds the scaling logic > to those two functions. > > Reviewed-by:

Re: [Xen-devel] [PATCH v4 02/10] x86/time.c: Scale host TSC in pvclock properly

2016-01-18 Thread Jan Beulich
>>> On 17.01.16 at 22:58, wrote: > This patch makes the pvclock return the scaled host TSC and > corresponding scaling parameters to HVM domains if guest TSC is not > emulated and TSC scaling is enabled. > > Signed-off-by: Haozhong Zhang This doesn't look to be dependent on patch 1, and I would

Re: [Xen-devel] [PATCH] docs: correct descriptions of gnttab_max_{, maptrack}_frames

2016-01-18 Thread Jan Beulich
>>> On 18.01.16 at 14:19, wrote: > gnttab_max_frames incorrectly referred to numbers of grant tab > operations and gnttab_max_maptrack_frames was confusingly worded. > > Add the default for gnttab_max_frames while here (it's currently the > same on all arches since no arch uses the available arch

Re: [Xen-devel] [PATCH] docs: correct descriptions of gnttab_max_{, maptrack}_frames

2016-01-18 Thread Andrew Cooper
On 18/01/16 13:19, Ian Campbell wrote: > gnttab_max_frames incorrectly referred to numbers of grant tab > operations and gnttab_max_maptrack_frames was confusingly worded. > > Add the default for gnttab_max_frames while here (it's currently the > same on all arches since no arch uses the available

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

2016-01-18 Thread osstest service owner
flight 78403 linux-mingo-tip-master real [real] http://logs.test-lab.xenproject.org/osstest/logs/78403/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-rumpuserxen-i386 10 guest-start fail REGR. vs. 60684 Tests whic

[Xen-devel] Load calculation refresh in credit2 (was in Re: Questions about the use of idle_vcpu[])

2016-01-18 Thread George Dunlap
[Changing the title to align with the current topic] On 18/01/16 12:47, Dario Faggioli wrote: > On Mon, 2016-01-18 at 12:37 +, George Dunlap wrote: >> On Mon, Jan 18, 2016 at 11:00 AM, Dario Faggioli >> wrote: >>> Credit2, AFAICR, could also avoid _always_ re-setting the timer, >>> but

[Xen-devel] [PATCH OSSTEST v2 10/12] Qualify TftpDiVersion with the suite.

2016-01-18 Thread Ian Campbell
This allows the version to differ e.g. between Wheezy and Jessie. Update production-config* to set TftpDiVersion_jessie instead of just TftpDiVersion, also add TftpDiVersion_wheezy using the version replaced in commit f610ea162836 "Switch to Debian 8.0 (jessie) as OS for test hosts". In mfi-commo

[Xen-devel] [PATCH OSSTEST v2 05/12] mfi-common: Always add debian_suite to debian_runvars

2016-01-18 Thread Ian Campbell
This adds an explicit debian_suite to some jobs which didn't already have one, meaning that those jobs will remain the same when cloned for a bisect and run in a tree where $c{DebianGuestSuite} has changed since the original construction. No expected semantic change. Signed-off-by: Ian Campbell

[Xen-devel] [PATCH OSSTEST v2 03/12] mfi-common: Rename $suite_runvars as $hostos_runvars

2016-01-18 Thread Ian Campbell
Later in the series more runvars to control the host install will be added. Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- mfi-common | 27 ++- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git a/mfi-common b/mfi-common index b739c7d..ccf8a5b 100644 --

[Xen-devel] [PATCH OSSTEST v2 09/12] mfi-common: Set di_version for build & test host install

2016-01-18 Thread Ian Campbell
This means that bisections will use the same version, even if production-config changed in the mean time. Signed-off-by: Ian Campbell --- v2: s/diversion/di_version/ Moved to same model of setting $def... as used for the suites. Dropped ack due to change to $def... --- make-flight | 4 +

[Xen-devel] [PATCH OSSTEST v2 00/12] Specify host and guest os version in runvars

2016-01-18 Thread Ian Campbell
The following series makes it possible to specify the OS used on the host and in guests (e.g. DebianSuite, d-i version, ISO image version) via runvars and causes the production configurations to always fully specify these rather than relying on test scripts picking up the defaults from the configur

[Xen-devel] [PATCH OSSTEST v2 01/12] Debian: Abolish $suite and $xopts{Suite} from preseed_* interfaces.

2016-01-18 Thread Ian Campbell
Generating a preseed for a suite which does not match the ->{Suite} of the underlying guest or host object does not seem useful, so remove this option and use ->{Suite} instead. For guests ->{Suite} is set by debian_guest_suite() (which is called from preseed_guest_create(), although it is often a

[Xen-devel] [PATCH OSSTEST v2 12/12] make-flight: Use older Debian for host and guest OS with older Xen

2016-01-18 Thread Ian Campbell
Sometimes when updating osstest to use a newer version of Debian as a baseline we find that the new compiler or other tools pickup latent errors in older code bases for which the fixes are invasive or otherwise inappropriate for a stable branch. This is the case with Debian Jessie and Xen 4.3 and

[Xen-devel] [PATCH OSSTEST v2 07/12] ts-debian-di-install: Allow Di Version to come from runvars

2016-01-18 Thread Ian Campbell
and following the lead of the suite arrange for a version selected from the defaults to be written back to the runvars. Signed-off-by: Ian Campbell --- v2: s/diversion/di_version/ --- Osstest/Debian.pm| 15 ++- ts-debian-di-install | 3 ++- 2 files changed, 16 insertions(+), 2 d

[Xen-devel] [PATCH OSSTEST v2 02/12] target_var: Support fallback to all_(guest|host)_$vn

2016-01-18 Thread Ian Campbell
Having to set {ident}_foo for all idents used in a job (e.g host vs src_host+dst_host) in make-flight would be a little fiddly. Instead follow the lead of all_hostflags and consult all_host_$vn. I have no immediate use for all_guest_$vn, but support it for consistency. Signed-off-by: Ian Campbel

[Xen-devel] [PATCH OSSTEST v2 06/12] ts-host-install: Support DiVersion coming from runvars

2016-01-18 Thread Ian Campbell
To do so initialise $ho->{DiVersion} in select host and use it in ts-host-install. Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- Osstest/TestSupport.pm | 1 + ts-host-install| 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/Osstest/TestSupport.pm b/Osstest/Te

[Xen-devel] [PATCH OSSTEST v2 04/12] mfi-common: always add host suite to hostos_runvars

2016-01-18 Thread Ian Campbell
This avoids situations where production-config* has changed DebianSuite but the bisector is still picking up baselines etc from before the change and reusing their runvars (without suite) with an inconsistent config. Switch selecthost() to use target_var when querying the suite. This means it will

[Xen-devel] [PATCH OSSTEST v2 11/12] mfi-common: usual_debianhvm_image: derive version from $guestsuite

2016-01-18 Thread Ian Campbell
This more likely matches the callers intention. Move the setting into production-config* alongside the Suite and TftpDiVersion settings. Continue to support $DEBIAN_IMAGE_VERSION as an override. The value for Wheezy is from what was replaced in 610ea1628363 "Switch to Debian 8.0 (jessie) as OS for

[Xen-devel] [PATCH OSSTEST v2 08/12] make-flight: Set di_version runvar on d-i based test jobs.

2016-01-18 Thread Ian Campbell
Note that make-distros-flight does not want this, since it uses d-i fetched from the web not the version in our config. Signed-off-by: Ian Campbell Acked-by: Ian Jackson --- v2: s/diversion/di_version/ --- make-flight | 4 1 file changed, 4 insertions(+) diff --git a/make-flight b/make-fl

Re: [Xen-devel] [PATCH v2 09/16] arm/xen: Get event-channel irq through HVM_PARAM when booting with ACPI

2016-01-18 Thread Stefano Stabellini
On Fri, 15 Jan 2016, Shannon Zhao wrote: > From: Shannon Zhao > > When booting with ACPI, it could get the event-channel irq through > HVM_PARAM_CALLBACK_IRQ. > > Signed-off-by: Shannon Zhao > --- > arch/arm/xen/enlighten.c | 30 +- > 1 file changed, 29 insertions(+

[Xen-devel] [PATCH] x86/HVM: add padding to hvm_hw_cpu

2016-01-18 Thread Roger Pau Monne
So that the size of the structure is the same on 32 and 64bit. Signed-off-by: Roger Pau Monné --- Cc: Ian Campbell Cc: Ian Jackson Cc: Jan Beulich Cc: Andrew Cooper --- This should fix the issues seen on OSSTest when using a 32bit toolstack on a 64bit hypervisor to create a Windows 7 HVM gues

Re: [Xen-devel] [PATCH] x86/HVM: add padding to hvm_hw_cpu

2016-01-18 Thread Andrew Cooper
On 18/01/16 14:39, Roger Pau Monne wrote: > So that the size of the structure is the same on 32 and 64bit. > > Signed-off-by: Roger Pau Monné > --- > Cc: Ian Campbell > Cc: Ian Jackson > Cc: Jan Beulich > Cc: Andrew Cooper > --- > This should fix the issues seen on OSSTest when using a 32bit t

Re: [Xen-devel] [PATCH v7 1/2] public/io/netif.h: clarifications to wire formats

2016-01-18 Thread Ian Campbell
On Tue, 2016-01-12 at 09:58 +, Paul Durrant wrote: > My previous patch 03809ae7 "document transmit and receive wire formats > separately" improved documentation of the receive and transmit wire > formats but further clarifications were requested. > > This patch adds those clarifications. > >

Re: [Xen-devel] [PATCH v2 10/16] ARM: XEN: Move xen_early_init() before efi_init()

2016-01-18 Thread Stefano Stabellini
On Fri, 15 Jan 2016, Shannon Zhao wrote: > From: Shannon Zhao > > Move xen_early_init() before efi_init(), then when calling efi_init() > could initialize Xen specific UEFI. > > Check if it runs on Xen hypervisor through the flat dts. > > Signed-off-by: Shannon Zhao > --- > arch/arm/xen/enlig

Re: [Xen-devel] [PATCH v2 11/16] ARM64: ACPI: Check if it runs on Xen to enable or disable ACPI

2016-01-18 Thread Stefano Stabellini
On Mon, 18 Jan 2016, Mark Rutland wrote: > On Fri, Jan 15, 2016 at 02:55:24PM +0800, Shannon Zhao wrote: > > From: Shannon Zhao > > > > When it's a Xen domain0 booting with ACPI, it will supply a /chosen and > > a /hypervisor node in DT. So check if it needs to enable ACPI. > > > > Signed-off-by

  1   2   3   >