Re: [Xen-devel] [PATCH] libxc: try to find last used pfn when migrating

2015-11-30 Thread Juergen Gross
On 27/11/15 18:16, Andrew Cooper wrote: > On 27/11/15 15:53, Juergen Gross wrote: >> On 27/11/15 16:33, Wei Liu wrote: >>> On Fri, Nov 27, 2015 at 03:50:53PM +0100, Juergen Gross wrote: For migration the last used pfn of a guest is needed to size the logdirty bitmap and as an upper bound

[Xen-devel] [RFC PATCH] libxl: add screendump API

2015-11-30 Thread Chunyan Liu
Currently libvirt kvm can support domain screenshot but libxl cannot. This patch is trying to add screendump API in libxl by calling qmp 'screendump' command, so to support screenshot for domains. Signed-off-by: Chunyan Liu --- tools/libxl/libxl.c | 36 ++

Re: [Xen-devel] HVM domains crash after upgrade from XEN 4.5.1 to 4.5.2

2015-11-30 Thread Jan Beulich
>>> On 27.11.15 at 23:51, wrote: > Am 24.11.15 um 11:43 schrieb Jan Beulich: >> Taking a random object out of that log, I can't see any non-standard >> option passed to the compiler, so I have to assume this is its default >> behavior (i.e. determined at build time, or established by extra >> patc

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

2015-11-30 Thread osstest service owner
flight 65237 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/65237/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-ovmf-amd64 9 debian-hvm-install fail REGR. vs. 64579 test-amd64-i386-xl-

Re: [Xen-devel] [PATCH] libxc: try to find last used pfn when migrating

2015-11-30 Thread Andrew Cooper
On 30/11/15 08:17, Juergen Gross wrote: > On 27/11/15 18:16, Andrew Cooper wrote: >> On 27/11/15 15:53, Juergen Gross wrote: >>> On 27/11/15 16:33, Wei Liu wrote: On Fri, Nov 27, 2015 at 03:50:53PM +0100, Juergen Gross wrote: > For migration the last used pfn of a guest is needed to size t

[Xen-devel] [PATCH] mwait_idle: Skylake Client Support

2015-11-30 Thread Jan Beulich
Skylake Client CPU idle Power states (C-states) are similar to the previous generation, Broadwell. However, Skylake does get its own table with updated worst-case latency and average energy-break-even residency values. Signed-off-by: Len Brown [Linux commit 493f133f47750aa5566fafa9403617e3f0506f8

Re: [Xen-devel] [PATCH] mwait_idle: Skylake Client Support

2015-11-30 Thread Andrew Cooper
On 30/11/15 09:49, Jan Beulich wrote: > Skylake Client CPU idle Power states (C-states) > are similar to the previous generation, Broadwell. > However, Skylake does get its own table with updated > worst-case latency and average energy-break-even residency values. > > Signed-off-by: Len Brown > [L

Re: [Xen-devel] [PATCH XEN v5 13/23] tools: Refactor foreign memory mapping into libxenforeignmemory

2015-11-30 Thread Ian Campbell
On Sun, 2015-11-29 at 09:54 +, Paul Durrant wrote: > > -Original Message- > [snip] > > > C99 was 16 years ago now, I'm struggling to think of a reason not to > > > move > > > the baseline for tools stuff at least to that. > > > > > > https://en.wikipedia.org/wiki/Visual_C%2B%2B might b

Re: [Xen-devel] [PATCH] x86/PV: hide features dependent on XSAVE when booted with "no-xsave"

2015-11-30 Thread Jan Beulich
>>> On 27.11.15 at 16:05, wrote: > On 27/11/15 11:05, Jan Beulich wrote: >> ... or when the guest has the XSAVE feature hidden by CPUID policy. >> Not doing so is at best confusing to guests. >> >> Signed-off-by: Jan Beulich > > These changes here are an improvement (so I don't object to taking

Re: [Xen-devel] [PATCH v4 3/4] xen/hvm: introduce a flags field in the CPU save record

2015-11-30 Thread Jan Beulich
>>> On 27.11.15 at 17:15, wrote: > El 26/11/15 a les 15.32, Jan Beulich ha escrit: > On 25.11.15 at 16:18, wrote: >>> --- a/xen/arch/x86/hvm/hvm.c >>> +++ b/xen/arch/x86/hvm/hvm.c >>> @@ -2085,16 +2091,17 @@ static int hvm_load_cpu_ctxt(struct domain *d, > hvm_domain_context_t *h) >>> s

Re: [Xen-devel] [PATCH 0/6] xen: Drop unused or empty annotations

2015-11-30 Thread Jan Beulich
>>> On 27.11.15 at 20:58, wrote: > No resulting change on any architecture. > > Andrew Cooper (6): > xen: Drop unused fastcall annotation > xen: Drop empty __cpuinitdata annotation > xen: Drop empty __cpuinit annotation > xen: Drop empty __devinitdata annotation > xen: Drop empty __devi

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Wei Liu
On Thu, Nov 26, 2015 at 08:35:02AM +0100, Juergen Gross wrote: > On 25/11/15 17:12, Boris Ostrovsky wrote: > > On 11/12/2015 08:43 AM, Juergen Gross wrote: > >> In case the kernel of a new pv-domU indicates it is supporting an > >> unmapped initrd, don't waste precious virtual space for the initrd,

Re: [Xen-devel] [PATCH] libxc: use correct return type for do_memory_op()

2015-11-30 Thread Wei Liu
On Fri, Nov 27, 2015 at 10:00:51AM +0100, Juergen Gross wrote: > Currently do_memory_op() is returning int, while the hypervisor is > returning long. This will lead to wrong return informations as soon as > e.g. a pfn larger than about 2 billion (8 TB) is returned. > > Use the correct long return

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Juergen Gross
On 30/11/15 11:20, Wei Liu wrote: > On Thu, Nov 26, 2015 at 08:35:02AM +0100, Juergen Gross wrote: >> On 25/11/15 17:12, Boris Ostrovsky wrote: >>> On 11/12/2015 08:43 AM, Juergen Gross wrote: In case the kernel of a new pv-domU indicates it is supporting an unmapped initrd, don't waste p

Re: [Xen-devel] xen can only detect one core of multiple cores cpu

2015-11-30 Thread Jan Beulich
>>> On 28.11.15 at 20:24, wrote: > On 28/11/15 17:23, quizyjones wrote: >> I'm using a Intel E5-2603 v3 @ 1.60GHz CPU of 6 cores. However, the >> dom0 can only find one core. here are some information that may helps >> in analyzing. > > From `xl dmesg` > > (XEN) ACPI Error (tbxfroot-0218): A va

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Wei Liu
On Mon, Nov 30, 2015 at 11:23:34AM +0100, Juergen Gross wrote: > On 30/11/15 11:20, Wei Liu wrote: > > On Thu, Nov 26, 2015 at 08:35:02AM +0100, Juergen Gross wrote: > >> On 25/11/15 17:12, Boris Ostrovsky wrote: > >>> On 11/12/2015 08:43 AM, Juergen Gross wrote: > In case the kernel of a new

Re: [Xen-devel] boot xen use legacy bios

2015-11-30 Thread Jan Beulich
>>> On 29.11.15 at 12:26, wrote: > I got the problem of multiple cores CPU cannot be fully used when using UEFI > mode. It is suggested that I should be boot the xen in legacy mode. How is it > done? For BIOSes that don't have a Legacy Support Module, booting in non-EFI mode just won't work. S

Re: [Xen-devel] Page Fault

2015-11-30 Thread Jan Beulich
>>> On 29.11.15 at 19:19, wrote: > Inside the page fault handler for shadow page tables (sh_page_fault > function in multi.c) where is the code for swapping in a page from disk? There is no swapping in from disk in that code. You probably think of memory-paging, which has nothing to do with shado

Re: [Xen-devel] xen can only detect one core of multiple cores cpu

2015-11-30 Thread quizy_jo...@outlook.com
>>> On 28.11.15 at 20:24, wrote: > On 28/11/15 17:23, quizyjones wrote: >> I'm using a Intel E5-2603 v3 @ 1.60GHz CPU of 6 cores. However, the >> dom0 can only find one core. here are some information that may helps >> in analyzing. > > From `xl dmesg` > > (XEN) ACPI Error (tbxfroot-0218): A va

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 11:23 +0100, Juergen Gross wrote: > On 30/11/15 11:20, Wei Liu wrote: > > On Thu, Nov 26, 2015 at 08:35:02AM +0100, Juergen Gross wrote: > > >   > > >  /* initrd parameters as specified in start_info page */ > > > -unsigned long initrd_start; > > > -unsigned long i

Re: [Xen-devel] [PATCH v9 6/9] libxc/xen: introduce a start info structure for HVMlite guests

2015-11-30 Thread Jan Beulich
>>> On 27.11.15 at 14:43, wrote: > This structure contains the physical address of the command line, as well as > the physical address of the list of loaded modules. The physical address of > this structure is passed to the guest at boot time in the %ebx register. > > Signed-off-by: Roger Pau Mon

Re: [Xen-devel] [PATCH] x86/PV: hide features dependent on XSAVE when booted with "no-xsave"

2015-11-30 Thread Andrew Cooper
On 30/11/15 10:01, Jan Beulich wrote: On 27.11.15 at 16:05, wrote: >> On 27/11/15 11:05, Jan Beulich wrote: >>> ... or when the guest has the XSAVE feature hidden by CPUID policy. >>> Not doing so is at best confusing to guests. >>> >>> Signed-off-by: Jan Beulich >> These changes here are an

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Juergen Gross
On 30/11/15 11:34, Ian Campbell wrote: > On Mon, 2015-11-30 at 11:23 +0100, Juergen Gross wrote: >> On 30/11/15 11:20, Wei Liu wrote: >>> On Thu, Nov 26, 2015 at 08:35:02AM +0100, Juergen Gross wrote: /* initrd parameters as specified in start_info page */ -unsigned long i

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 11:47 +0100, Juergen Gross wrote: > On 30/11/15 11:34, Ian Campbell wrote: > > On Mon, 2015-11-30 at 11:23 +0100, Juergen Gross wrote: > > > On 30/11/15 11:20, Wei Liu wrote: > > > > On Thu, Nov 26, 2015 at 08:35:02AM +0100, Juergen Gross wrote: > > > > >   > > > > >  /* i

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 10:51 +, Ian Campbell wrote: > On Mon, 2015-11-30 at 11:47 +0100, Juergen Gross wrote: > > On 30/11/15 11:34, Ian Campbell wrote: > > > On Mon, 2015-11-30 at 11:23 +0100, Juergen Gross wrote: > > > > On 30/11/15 11:20, Wei Liu wrote: > > > > > On Thu, Nov 26, 2015 at 08:35

[Xen-devel] Xen Security Advisory 162 (CVE-2015-7504) - heap buffer overflow vulnerability in pcnet emulator

2015-11-30 Thread Xen . org security team
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Xen Security Advisory CVE-2015-7504 / XSA-162 version 2 heap buffer overflow vulnerability in pcnet emulator UPDATES IN VERSION 2 Public release. Correct cut and paste referenc

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Juergen Gross
On 30/11/15 11:51, Ian Campbell wrote: > On Mon, 2015-11-30 at 11:47 +0100, Juergen Gross wrote: >> On 30/11/15 11:34, Ian Campbell wrote: >>> On Mon, 2015-11-30 at 11:23 +0100, Juergen Gross wrote: On 30/11/15 11:20, Wei Liu wrote: > On Thu, Nov 26, 2015 at 08:35:02AM +0100, Juergen Gross

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Juergen Gross
On 30/11/15 11:52, Ian Campbell wrote: > On Mon, 2015-11-30 at 10:51 +, Ian Campbell wrote: >> On Mon, 2015-11-30 at 11:47 +0100, Juergen Gross wrote: >>> On 30/11/15 11:34, Ian Campbell wrote: On Mon, 2015-11-30 at 11:23 +0100, Juergen Gross wrote: > On 30/11/15 11:20, Wei Liu wrote:

[Xen-devel] [PATCH] x86_emulate: Always truncate %eip in 32bit mode

2015-11-30 Thread Andrew Cooper
_regs.eip needs to be truncated after having size added to it, or emulating an instruction which crosses the 4GB boundary causes _regs.eip to become invalid, and fail vmentry checks when returning back to the guest. The comment /* real hardware doesn't truncate */ seems to appear in c/s ddef8e16 "

Re: [Xen-devel] [PATCH] x86/PV: hide features dependent on XSAVE when booted with "no-xsave"

2015-11-30 Thread Jan Beulich
>>> On 30.11.15 at 11:46, wrote: > On 30/11/15 10:01, Jan Beulich wrote: > On 27.11.15 at 16:05, wrote: >>> On 27/11/15 11:05, Jan Beulich wrote: ... or when the guest has the XSAVE feature hidden by CPUID policy. Not doing so is at best confusing to guests. Signed-off-by:

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

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

Re: [Xen-devel] [PATCH] x86/PV: hide features dependent on XSAVE when booted with "no-xsave"

2015-11-30 Thread Andrew Cooper
On 30/11/15 11:08, Jan Beulich wrote: On 30.11.15 at 11:46, wrote: >> On 30/11/15 10:01, Jan Beulich wrote: >> On 27.11.15 at 16:05, wrote: On 27/11/15 11:05, Jan Beulich wrote: > ... or when the guest has the XSAVE feature hidden by CPUID policy. > Not doing so is at best c

Re: [Xen-devel] [PATCH] x86_emulate: Always truncate %eip in 32bit mode

2015-11-30 Thread Jan Beulich
>>> On 30.11.15 at 12:07, wrote: > _regs.eip needs to be truncated after having size added to it, or emulating an > instruction which crosses the 4GB boundary causes _regs.eip to become invalid, > and fail vmentry checks when returning back to the guest. > > The comment /* real hardware doesn't t

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 12:03 +0100, Juergen Gross wrote: > On 30/11/15 11:52, Ian Campbell wrote: > > On Mon, 2015-11-30 at 10:51 +, Ian Campbell wrote: > > > On Mon, 2015-11-30 at 11:47 +0100, Juergen Gross wrote: > > > > On 30/11/15 11:34, Ian Campbell wrote: > > > > > On Mon, 2015-11-30 at 11

Re: [Xen-devel] [PATCH] x86/PV: hide features dependent on XSAVE when booted with "no-xsave"

2015-11-30 Thread Jan Beulich
>>> On 30.11.15 at 12:10, wrote: > On 30/11/15 11:08, Jan Beulich wrote: > On 30.11.15 at 11:46, wrote: >>> On 30/11/15 10:01, Jan Beulich wrote: >>> On 27.11.15 at 16:05, wrote: > On 27/11/15 11:05, Jan Beulich wrote: >> ... or when the guest has the XSAVE feature hidden by CPUI

[Xen-devel] [PATCH OSSTEST] target_fetchurl: Handle undefined $c{HttpProxy}

2015-11-30 Thread Ian Campbell
Avoiding a usage of a potentially undefined variable. Signed-off-by: Ian Campbell --- Osstest/TestSupport.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm index 18b03b4..28ac572 100644 --- a/Osstest/TestSupport.pm +++ b/Osstes

[Xen-devel] [PATCH OSSTEST] ts-debian-di-install: Don't set runvars for netboot kernel+ramdisk as outputs

2015-11-30 Thread Ian Campbell
Currently these runvars are either URLs provided by the definition (e.g. make-flight) or output controller-relative paths created by the execution (in the case where they aren't from the definition). This wierd dual-semantics is confusing and wrong, and in particular is broken if the test step is

Re: [Xen-devel] [PATCH 3/6] xen: Drop empty __cpuinit annotation

2015-11-30 Thread Julien Grall
Hi Andrew, On 27/11/15 19:58, Andrew Cooper wrote: > Signed-off-by: Andrew Cooper It's worth to note that on ARM we don't yet support CPU hotplug so it may be possible to define __cpuinit as __init and save few KB. Regards, -- Julien Grall ___ Xen-

Re: [Xen-devel] [PATCH v9 5/9] xen/x86: allow HVM guests to use hypercalls to bring up vCPUs

2015-11-30 Thread Julien Grall
Hi Roger, On 27/11/15 13:43, Roger Pau Monne wrote: > diff --git a/xen/common/domain.c b/xen/common/domain.c > index f56b7ff..3690eec 100644 > --- a/xen/common/domain.c > +++ b/xen/common/domain.c > @@ -1208,11 +1208,34 @@ void unmap_vcpu_info(struct vcpu *v) > put_page_and_type(mfn_to_page(m

Re: [Xen-devel] [PATCH v2 01/11] xen/arm: vgic-v2: Implement correctly ICFGR{0, 1} read-only

2015-11-30 Thread Julien Grall
Hi Ian, On 25/11/15 12:29, Ian Campbell wrote: > On Tue, 2015-11-24 at 17:14 +, Ian Campbell wrote: >> @@ -507,10 +507,12 @@ static int vgic_v2_distr_mmio_write(struct vcpu >>> *v, >>> mmio_info_t *info, >>> >>> case GICD_ICFGR: /* SGIs */ >>> goto write_ignore_32; >>> -cas

Re: [Xen-devel] [PATCH v2 02/11] xen/arm: vgic-v3: Don't try to emulate IROUTER which doesn't exist in the spec

2015-11-30 Thread Julien Grall
Hi Ian, On 24/11/15 17:17, Ian Campbell wrote: > On Wed, 2015-11-18 at 17:27 +, Julien Grall wrote: > > Subject: ... which do not exist in the ... > >> The range of valid IROUTER are n = 32 - 1019 (see 8.9.13 in IHI 0069A) >> which correspond to the offset 0x6100-0x7FD8. >> >> Other offset a

Re: [Xen-devel] [PATCH] libxc: try to find last used pfn when migrating

2015-11-30 Thread Juergen Gross
On 30/11/15 10:47, Andrew Cooper wrote: > On 30/11/15 08:17, Juergen Gross wrote: >> On 27/11/15 18:16, Andrew Cooper wrote: >>> On 27/11/15 15:53, Juergen Gross wrote: On 27/11/15 16:33, Wei Liu wrote: > On Fri, Nov 27, 2015 at 03:50:53PM +0100, Juergen Gross wrote: >> For migration t

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Juergen Gross
On 30/11/15 12:23, Ian Campbell wrote: > On Mon, 2015-11-30 at 12:03 +0100, Juergen Gross wrote: >> On 30/11/15 11:52, Ian Campbell wrote: >>> On Mon, 2015-11-30 at 10:51 +, Ian Campbell wrote: On Mon, 2015-11-30 at 11:47 +0100, Juergen Gross wrote: > On 30/11/15 11:34, Ian Campbell wr

Re: [Xen-devel] unhandled word causes Xen crash with recent Linux kernels, was: Re: [PATCH v2 05/11] xen/arm: vgic: Properly emulate the full register

2015-11-30 Thread Julien Grall
Hi Stefano, On 25/11/15 12:15, Stefano Stabellini wrote: > Hi Shannon, > > On Wed, 25 Nov 2015, Shannon Zhao wrote: >> Upstream Linux kernel applies below patch which will write >> GICD_ICACTIVER. But since Xen doesn't support it, so it will cause Dom0 >> initializes GIC failed. >> >> 0eece2b2284

Re: [Xen-devel] unhandled word causes Xen crash with recent Linux kernels, was: Re: [PATCH v2 05/11] xen/arm: vgic: Properly emulate the full register

2015-11-30 Thread Julien Grall
Hi Ian, On 25/11/15 12:26, Ian Campbell wrote: > On Wed, 2015-11-25 at 12:15 +, Stefano Stabellini wrote: >> On Wed, 25 Nov 2015, Shannon Zhao wrote: >>> Upstream Linux kernel applies below patch which will write >>> GICD_ICACTIVER. But since Xen doesn't support it, so it will cause Dom0 >>> i

Re: [Xen-devel] [PATCH] xen/arm: implement GICD_ICACTIVER read/write

2015-11-30 Thread Julien Grall
Hi Stefano, On 25/11/15 16:40, Stefano Stabellini wrote: > Implement GICD_ICACTIVER and GICD_ISACTIVER reads by looking for the > GIC_IRQ_GUEST_ACTIVE bit in the relevant struct pending_irq. However > given that the pending to active transaction for irqs in LRs in done in > hardware, the GIC_IRQ_G

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 13:20 +0100, Juergen Gross wrote: > On 30/11/15 12:23, Ian Campbell wrote: > > On Mon, 2015-11-30 at 12:03 +0100, Juergen Gross wrote: > > > On 30/11/15 11:52, Ian Campbell wrote: > > > > On Mon, 2015-11-30 at 10:51 +, Ian Campbell wrote: > > > > > On Mon, 2015-11-30 at 11

Re: [Xen-devel] unhandled word causes Xen crash with recent Linux kernels, was: Re: [PATCH v2 05/11] xen/arm: vgic: Properly emulate the full register

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 12:22 +, Julien Grall wrote: > Hi Ian, > > On 25/11/15 12:26, Ian Campbell wrote: > > On Wed, 2015-11-25 at 12:15 +, Stefano Stabellini wrote: > > > On Wed, 25 Nov 2015, Shannon Zhao wrote: > > > > Upstream Linux kernel applies below patch which will write > > > > GIC

Re: [Xen-devel] [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported

2015-11-30 Thread Juergen Gross
On 30/11/15 13:35, Ian Campbell wrote: > On Mon, 2015-11-30 at 13:20 +0100, Juergen Gross wrote: >> On 30/11/15 12:23, Ian Campbell wrote: >>> On Mon, 2015-11-30 at 12:03 +0100, Juergen Gross wrote: On 30/11/15 11:52, Ian Campbell wrote: > On Mon, 2015-11-30 at 10:51 +, Ian Campbell wr

[Xen-devel] pvgrub "Error 9: Unknown boot failure" booting Debian Jessie kernel (Was: Re: [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported)

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 13:59 +0100, Juergen Gross wrote: > On 30/11/15 13:35, Ian Campbell wrote: > > FYI attempting to upgrade osstest to use Debian Jessie in the guest > > seems > > to have exposed another issue here. > > > > http://logs.test-lab.xenproject.org/osstest/logs/65172/test-amd64-amd64

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

2015-11-30 Thread osstest service owner
flight 65255 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/65255/ 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 v6 4/6] xen/arm: vgic: Optimize the way to store the target vCPU in the rank

2015-11-30 Thread Julien Grall
Hi Ian, On 25/11/15 11:37, Ian Campbell wrote: > On Wed, 2015-11-18 at 16:42 +, Julien Grall wrote: >> Xen is currently directly storing the value of GICD_ITARGETSR register >> (for GICv2) and GICD_IROUTER (for GICv3) in the rank. This makes the >> emulation of the registers access very simple

Re: [Xen-devel] [PATCH] x86/PV: hide features dependent on XSAVE when booted with "no-xsave"

2015-11-30 Thread Andrew Cooper
On 30/11/15 11:30, Jan Beulich wrote: On 30.11.15 at 12:10, wrote: >> On 30/11/15 11:08, Jan Beulich wrote: >> On 30.11.15 at 11:46, wrote: On 30/11/15 10:01, Jan Beulich wrote: On 27.11.15 at 16:05, wrote: >> On 27/11/15 11:05, Jan Beulich wrote: >>> ... or when t

[Xen-devel] [OSSTEST PATCH] cs-bisection-step: Limit size of revision log included in reports

2015-11-30 Thread Ian Jackson
There is a limit in cr-daily-branch, but none in cs-bisection-step. adhoc-revtuple-generator could usefully have this built in but that's not so simple, so do it again here. We already slurp the whole thing into core so from a resource usage point of view we might as well do the length check here

Re: [Xen-devel] pvgrub "Error 9: Unknown boot failure" booting Debian Jessie kernel (Was: Re: [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported)

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 13:16 +, Ian Campbell wrote: > On Mon, 2015-11-30 at 13:59 +0100, Juergen Gross wrote: > > On 30/11/15 13:35, Ian Campbell wrote: > > > FYI attempting to upgrade osstest to use Debian Jessie in the guest > > > seems > > > to have exposed another issue here. > > > > > > ht

Re: [Xen-devel] [PATCH v6 4/6] xen/arm: vgic: Optimize the way to store the target vCPU in the rank

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 13:32 +, Julien Grall wrote: > Hi Ian, > > On 25/11/15 11:37, Ian Campbell wrote: > > On Wed, 2015-11-18 at 16:42 +, Julien Grall wrote: > > > Xen is currently directly storing the value of GICD_ITARGETSR > > > register > > > (for GICv2) and GICD_IROUTER (for GICv3) i

Re: [Xen-devel] [OSSTEST PATCH] cs-bisection-step: Limit size of revision log included in reports

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 13:41 +, Ian Jackson wrote: > There is a limit in cr-daily-branch, but none in cs-bisection-step. > > adhoc-revtuple-generator could usefully have this built in but that's > not so simple, so do it again here.  We already slurp the whole thing > into core so from a resour

Re: [Xen-devel] [PATCH OSSTEST] target_fetchurl: Handle undefined $c{HttpProxy}

2015-11-30 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] target_fetchurl: Handle undefined $c{HttpProxy}"): > Avoiding a usage of a potentially undefined variable. Acked-by: Ian Jackson ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH OSSTEST] ts-debian-di-install: Don't set runvars for netboot kernel+ramdisk as outputs

2015-11-30 Thread Ian Jackson
Ian Campbell writes ("[PATCH OSSTEST] ts-debian-di-install: Don't set runvars for netboot kernel+ramdisk as outputs"): > Currently these runvars are either URLs provided by the definition > (e.g. make-flight) or output controller-relative paths created by the > execution (in the case where they ar

Re: [Xen-devel] [PATCHv6] 01/28] build: import Kbuild/Kconfig from Linux 4.2

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:51, wrote: > --- /dev/null > +++ b/xen/include/linux/kconfig.h > @@ -0,0 +1,54 @@ > +#ifndef __LINUX_KCONFIG_H > +#define __LINUX_KCONFIG_H Neither placement in the source tree nor guard variable should say "Linux". > --- /dev/null > +++ b/xen/scripts/Makefile.host > @@ -0,

Re: [Xen-devel] [PATCH] xen/arm: implement GICD_ICACTIVER read/write

2015-11-30 Thread Julien Grall
Hi Stefano, On 25/11/15 16:40, Stefano Stabellini wrote: > Implement GICD_ICACTIVER and GICD_ISACTIVER reads by looking for the > GIC_IRQ_GUEST_ACTIVE bit in the relevant struct pending_irq. However > given that the pending to active transaction for irqs in LRs in done in > hardware, the GIC_IRQ_G

Re: [Xen-devel] [OSSTEST PATCH] cs-bisection-step: Limit size of revision log included in reports

2015-11-30 Thread Ian Jackson
Ian Campbell writes ("Re: [OSSTEST PATCH] cs-bisection-step: Limit size of revision log included in reports"): > Acked-by: Ian Campbell > > > +(Revision log too long, ommitted.) > > My MUA's spell checker thinks it is "omitted". Thanks. I have ommittedd the spurrious lettter. Ian. _

Re: [Xen-devel] [PATCH v6 4/6] xen/arm: vgic: Optimize the way to store the target vCPU in the rank

2015-11-30 Thread Julien Grall
On 30/11/15 13:55, Ian Campbell wrote: > On Mon, 2015-11-30 at 13:32 +, Julien Grall wrote: >> Hi Ian, >> >> On 25/11/15 11:37, Ian Campbell wrote: >>> On Wed, 2015-11-18 at 16:42 +, Julien Grall wrote: Xen is currently directly storing the value of GICD_ITARGETSR register (f

[Xen-devel] [xen-4.6-testing test] 65241: regressions - FAIL

2015-11-30 Thread osstest service owner
flight 65241 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/65241/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-stubdom-debianhvm-amd64-xsm 16 guest-localmigrate/x10 fail in 65210 RE

Re: [Xen-devel] [PATCHv2 2/3] mm: don't free pages until mm locks are released

2015-11-30 Thread David Vrabel
On 13/11/15 18:49, David Vrabel wrote: > If a page is freed without translations being invalidated, and the page is > subsequently allocated to another domain, a guest with a cached > translation will still be able to access the page. > > Currently translations are invalidated before releasing the

Re: [Xen-devel] pvgrub "Error 9: Unknown boot failure" booting Debian Jessie kernel (Was: Re: [PATCH v5 6/9] libxc: create unmapped initrd in domain builder if supported)

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 13:41 +, Ian Campbell wrote: > On Mon, 2015-11-30 at 13:16 +, Ian Campbell wrote: > > On Mon, 2015-11-30 at 13:59 +0100, Juergen Gross wrote: > > > On 30/11/15 13:35, Ian Campbell wrote: > > > > FYI attempting to upgrade osstest to use Debian Jessie in the guest > > >

Re: [Xen-devel] [PATCH] xen/grant-table: constify gnttab_ops structure

2015-11-30 Thread David Vrabel
On 28/11/15 14:28, Julia Lawall wrote: > The gnttab_ops structure is never modified, so declare it as const. > > Done with the help of Coccinelle. Applied to for-linus-4.5, thanks. David ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.

[Xen-devel] [ovmf test] 65244: all pass - PUSHED

2015-11-30 Thread osstest service owner
flight 65244 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/65244/ Perfect :-) All tests in this flight passed version targeted for testing: ovmf 81438fe8d0fc63f9bc9fcee0113baf6bd395f29c baseline version: ovmf ec613395d114ed6f7c13249a199d1e9cc00

Re: [Xen-devel] [PATCH 1/3] xen/gntdev: constify mmu_notifier_ops structures

2015-11-30 Thread David Vrabel
On 29/11/15 22:02, Julia Lawall wrote: > This mmu_notifier_ops structure is never modified, so declare it as > const, like the other mmu_notifier_ops structures. > > Done with the help of Coccinelle. Applied to for-linus-4.5, thanks. David ___ Xen-dev

[Xen-devel] Beginner Information: Final 2015 Xen Project Document Day is Wednesday Dec 2

2015-11-30 Thread Russ Pavlicek
OUR THEME OF THE MONTH: Beginner Information We've seen an increase of interest from potential new users recently and we need to make sure that people can successfully get started with 4.6 using our current set of Wiki pages. We have been slowly replacing the "xm" references with "xl", but we cou

Re: [Xen-devel] [PATCH 4/4] xen/arm: p2m: Remove translation table when it's empty

2015-11-30 Thread Ian Campbell
On Mon, 2015-11-30 at 14:26 +, Julien Grall wrote: > + > > > +p2m->stats.mappings[level - 1]--; > > > +update_reference_mapping(pages[level - 1], > > > old_entry, *entry); > > > + > > > +/* > > > + * We can't free the page now beca

Re: [Xen-devel] [PATCH 4/4] xen/arm: p2m: Remove translation table when it's empty

2015-11-30 Thread Julien Grall
Hi Ian, On 25/11/15 12:40, Ian Campbell wrote: > On Wed, 2015-11-18 at 15:49 +, Julien Grall wrote: >> Currently, the translation table is left in place even if no entries is >> inuse. Because of how the p2m code has been implemented, replacing a >> translation table by a block (i.e superpage)

Re: [Xen-devel] [PATCHv6] 02/28] build: build Kconfig and config rules

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:51, wrote: > --- a/.gitignore > +++ b/.gitignore > @@ -217,6 +217,11 @@ tools/xentrace/tbctl > tools/xentrace/xenctx > tools/xentrace/xentrace > xen/.banner > +xen/.config > +xen/.config.old > +xen/defconfig > +xen/**/*.cmd > +xen/**/modules.order The last two seem rather

Re: [Xen-devel] [PATCHv6] 03/28] build: use generated Kconfig options for Xen

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:51, wrote: > --- a/xen/Makefile > +++ b/xen/Makefile > @@ -26,6 +26,9 @@ default: build > .PHONY: dist > dist: install > > +.PHONY: build > +build:: $(BASEDIR)/include/config/auto.conf > + > .PHONY: build install uninstall clean distclean cscope TAGS tags MAP gtags I do

Re: [Xen-devel] [PATCH v3 07/62] arm/acpi: Add arch_acpi_os_map_memory helper function for ARM

2015-11-30 Thread Julien Grall
Hi, On 23/11/15 11:37, Stefano Stabellini wrote: > On Tue, 17 Nov 2015, shannon.z...@linaro.org wrote: >> From: Shannon Zhao > could you please add a couple of lines to the commit message mentioning > why __va(phys) is an acceptable implementation of arch_acpi_os_map_memory? FWIW, I already ask

Re: [Xen-devel] [PATCHv6] 04/28] build: convert HAS_PASSTHROUGH use to Kconfig

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:51, wrote: > --- a/xen/arch/x86/Kconfig > +++ b/xen/arch/x86/Kconfig > @@ -3,6 +3,7 @@ config X86_64 > > config X86 > def_bool y > + select HAS_PASSTHROUGH > select HAS_GDBSX Please get these sorted alphabetically from the beginning. > --- /dev/null > +++

Re: [Xen-devel] [PATCHv6] 07/28] build: convert HAS_NS16550 use to Kconfig

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:51, wrote: > --- /dev/null > +++ b/xen/drivers/char/Kconfig > @@ -0,0 +1,5 @@ > + > +config UART_NS16550 > + bool "16550-series UART support" Iirc we agreed not to have any user visible prompts for the moment; I think Ian was quite explicit about this. The 32- vs 64-bit

Re: [Xen-devel] [PATCHv6] 08/28] build: convert HAS_IOPORTS use to Kconfig

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:51, wrote: > --- a/xen/drivers/char/Kconfig > +++ b/xen/drivers/char/Kconfig > @@ -3,3 +3,8 @@ config UART_NS16550 > bool "16550-series UART support" > help > This selects the 16550-series UART support. For most systems, say Y. > + > +# Select HAS_IOPORTS

Re: [Xen-devel] [PATCH v3 20/62] arm/acpi: Add ACPI support for SMP initialization

2015-11-30 Thread Julien Grall
Hi Shannon, On 17/11/15 09:40, shannon.z...@linaro.org wrote: > diff --git a/xen/arch/arm/psci.c b/xen/arch/arm/psci.c > index d800cb6..dede0e1 100644 > --- a/xen/arch/arm/psci.c > +++ b/xen/arch/arm/psci.c > @@ -22,6 +22,7 @@ > #include > #include > #include > +#include > > /* > * Whi

Re: [Xen-devel] [PATCHv6] 08/28] build: convert HAS_IOPORTS use to Kconfig

2015-11-30 Thread Jan Beulich
>>> On 30.11.15 at 15:56, wrote: On 24.11.15 at 18:51, wrote: >> --- a/xen/drivers/char/Kconfig >> +++ b/xen/drivers/char/Kconfig >> @@ -3,3 +3,8 @@ config UART_NS16550 >> bool "16550-series UART support" >> help >>This selects the 16550-series UART support. For most system

Re: [Xen-devel] [PATCH v3 22/62] arm/gic-v3: Refactor gicv3_init into generic and dt specific parts

2015-11-30 Thread Julien Grall
On 24/11/15 10:51, Stefano Stabellini wrote: > On Tue, 17 Nov 2015, shannon.z...@linaro.org wrote: >> From: Shannon Zhao >> >> Refactor gic-v3 related functions into dt and generic parts. This will be >> helpful when adding acpi support for gic-v3. >> >> Signed-off-by: Shannon Zhao >> --- >> xen

Re: [Xen-devel] [PATCHv6] 10/28] build: convert HAS_VIDEO use to Kconfig

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:52, wrote: > --- /dev/null > +++ b/xen/drivers/video/Kconfig > @@ -0,0 +1,13 @@ > + > +# Select HAS_VIDEO if video is supported > +config HAS_VIDEO > + bool > + > +# Select HAS_VGA if VGA is supported > +config HAS_VGA > + bool > + depends on HAS_VIDEO As said in

Re: [Xen-devel] [PATCHv6] 11/28] build: convert HAS_VGA use to Kconfig

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:52, wrote: > Use the Kconfig generated CONFIG_HAS_VGA defines in the code base. This should be folded with the previous patch, or the VGA parts done there be moved here. Jan ___ Xen-devel mailing list Xen-devel@lists.xen.org htt

Re: [Xen-devel] [PATCHv6] 13/28] build: convert HAS_GDBSX use to Kconfig

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:52, wrote: > --- a/xen/common/Kconfig > +++ b/xen/common/Kconfig > @@ -5,4 +5,8 @@ menu "Common Features" > config HAS_DEVICE_TREE > bool > > +# Select HAS_GDBSX if GDBSX is supported > +config HAS_GDBSX > + bool Wasn't there a select of this option way earlier

Re: [Xen-devel] [PATCHv6] 15/28] build: convert HAS_KEXEC use to Kconfig

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:52, wrote: > --- a/xen/Rules.mk > +++ b/xen/Rules.mk > @@ -68,7 +68,7 @@ ifneq ($(max_phys_irqs),) > CFLAGS-y+= -DMAX_PHYS_IRQS=$(max_phys_irqs) > endif > > -CONFIG_KEXEC-$(HAS_KEXEC) := $(kexec) > +CONFIG_KEXEC-$(CONFIG_HAS_KEXEC) := $(kexec) > CONFIG_KE

Re: [Xen-devel] [PATCHv6] 22/28] build: convert HAS_EHCI use to Kconfig

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:52, wrote: > --- a/xen/arch/x86/Kconfig > +++ b/xen/arch/x86/Kconfig > @@ -11,6 +11,7 @@ config X86 > select HAS_CPUFREQ > select HAS_PCI > select UART_NS16550 > + select UART_EHCI I don't think this is a reasonable name. We're talking about EHCI debug

Re: [Xen-devel] [PATCHv6] 24/28] build: convert HAS_MEM_PAGING use to Kconfig

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:52, wrote: > --- a/config/x86_32.mk > +++ b/config/x86_32.mk > @@ -6,7 +6,6 @@ CONFIG_MIGRATE := y > CONFIG_XCUTILS := y > > HAS_MEM_ACCESS := y > -HAS_MEM_PAGING := y Considering that the previous patch supposedly eliminated HAS_MEM_ACCESS, the context here looks suspic

Re: [Xen-devel] [PATCHv6] 27/28] build: convert CONFIG_COMPAT to Kconfig

2015-11-30 Thread Jan Beulich
>>> On 24.11.15 at 18:52, wrote: > --- > xen/arch/x86/Kconfig | 7 +++ > 1 file changed, 7 insertions(+) This can't be complete - you ought to be deleting the current definition. > --- a/xen/arch/x86/Kconfig > +++ b/xen/arch/x86/Kconfig > @@ -25,6 +25,13 @@ config ARCH_DEFCONFIG > > menu

Re: [Xen-devel] [PATCH] x86/PV: hide features dependent on XSAVE when booted with "no-xsave"

2015-11-30 Thread Jan Beulich
>>> On 30.11.15 at 14:36, wrote: > On 30/11/15 11:30, Jan Beulich wrote: >> It's not well defined whether YMM register presence >> correlates to AVX, or is simply flagged by the respective XSTATE >> CPUID bit (or a mixture of both). > > It is indeed not well defined, which is what makes this area

Re: [Xen-devel] [PATCH v3 47/62] arm/p2m: Add helper functions to map memory regions

2015-11-30 Thread Julien Grall
Hi Shannon, On 17/11/15 09:40, shannon.z...@linaro.org wrote: > From: Parth Dixit > > Create a helper function for mapping with cached attributes. You are using those helpers to map the ACPI table in the guest. Do we really need them to be mapped read-write? Regards, -- Julien Grall ___

Re: [Xen-devel] [PATCH v3 49/62] arm/acpi: Map rest tables for Dom0

2015-11-30 Thread Julien Grall
Hi, On 27/11/15 12:16, Stefano Stabellini wrote: > On Tue, 17 Nov 2015, shannon.z...@linaro.org wrote: >> From: Shannon Zhao >> >> Map other reused tables for Dom0. > > "Map all other tables to Dom0 using 1:1 mappings." > > >> Signed-off-by: Shannon Zhao >> --- >> xen/arch/arm/domain_build.c

Re: [Xen-devel] [PATCH v3 47/62] arm/p2m: Add helper functions to map memory regions

2015-11-30 Thread Julien Grall
Hi, On 27/11/15 12:04, Stefano Stabellini wrote: > On Tue, 17 Nov 2015, shannon.z...@linaro.org wrote: >> From: Parth Dixit >> >> Create a helper function for mapping with cached attributes. >> >> Signed-off-by: Parth Dixit >> Signed-off-by: Shannon Zhao > > You might be able to use the existi

Re: [Xen-devel] [PATCH v3 38/62] arm/acpi: Add placeholder for efi and acpi load address

2015-11-30 Thread Julien Grall
Hi Stefano, On 26/11/15 16:04, Stefano Stabellini wrote: > On Wed, 18 Nov 2015, Julien Grall wrote: >> On 18/11/15 03:01, Shannon Zhao wrote: >>> "All above tables will be mapped to Dom0 non-RAM space. Since when >>> booting through ACPI it doesn't need the grant table region(see below >>> section

Re: [Xen-devel] [PATCH v3 40/62] arm/acpi: Estimate memory required for acpi/efi tables

2015-11-30 Thread Julien Grall
Hi Shannon, On 17/11/15 09:40, shannon.z...@linaro.org wrote: > diff --git a/xen/common/efi/boot.c b/xen/common/efi/boot.c > index 53c7452..78d8ae9 100644 > --- a/xen/common/efi/boot.c > +++ b/xen/common/efi/boot.c > @@ -13,6 +13,7 @@ > #include > #include > #include > +#include > #if EFI_

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

2015-11-30 Thread osstest service owner
flight 65243 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/65243/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-libvirt 5 libvirt-build fail REGR. vs. 63340 test-amd64-amd64-libvirt-

Re: [Xen-devel] [PATCH v3 56/62] arm/acpi: Deny MMIO access of UART

2015-11-30 Thread Julien Grall
Hi Shannon, On 17/11/15 13:10, Shannon Zhao wrote: > Regarding SMMU, since it doesn't support now, so I didn't add it but it > could be added later. Then please add a TODO, otherwise it's a call to forget it. Regards, -- Julien Grall ___ Xen-devel m

Re: [Xen-devel] [PATCH v3 58/62] xen/acpi: Fix event-channel interrupt when booting with ACPI

2015-11-30 Thread Julien Grall
On 27/11/15 15:12, Stefano Stabellini wrote: >> +} >> +else >> +{ >> +int type = 3; >> +int flag = 2; /* Active-low level-sensitive */ >> +d->arch.hvm_domain.params[HVM_PARAM_CALLBACK_IRQ] = (u64)type << 56 >> +

Re: [Xen-devel] boot xen use legacy bios

2015-11-30 Thread quizyjones
Thanks for your reply. I've already changed to boot with the xen.efi directly without grub2. Now I can see all the cpu cores, but I can't use the mouse or keyboard. What's more, the memory seems to be problematic too. 'xl dmesg' shows : (XEN) Unknown cachability for MFNs 0x8-0x8(XEN) Unk

Re: [Xen-devel] [PATCHv6] 01/28] build: import Kbuild/Kconfig from Linux 4.2

2015-11-30 Thread Ian Jackson
Doug Goldstein writes ("Re: [PATCHv6] 01/28] build: import Kbuild/Kconfig from Linux 4.2"): > The whole point here was to bring in the kconfig bits from Linux 4.2 > untouched for traceability for where the code came from. By doing it > this way it allows Xen to rebase kconfig support to a newer ve

Re: [Xen-devel] [PATCHv6] 01/28] build: import Kbuild/Kconfig from Linux 4.2

2015-11-30 Thread Doug Goldstein
On 11/30/15 7:59 AM, Jan Beulich wrote: On 24.11.15 at 18:51, wrote: >> --- /dev/null >> +++ b/xen/include/linux/kconfig.h >> @@ -0,0 +1,54 @@ >> +#ifndef __LINUX_KCONFIG_H >> +#define __LINUX_KCONFIG_H > > Neither placement in the source tree nor guard variable should say > "Linux". This f

  1   2   >