Re: [Xen-devel] [PATCH v4 03/16] libxl/arm: Generate static ACPI DSDT table

2016-08-30 Thread Shannon Zhao
On 2016/8/30 1:46, Julien Grall wrote: > Hi Shannon, > > On 16/08/2016 06:25, Shannon Zhao wrote: >> From: Shannon Zhao >> >> It uses static DSDT table like the way x86 uses. Currently the DSDT >> table only contains processor device objects and it generates the >> maximal objects which so far

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

2016-08-30 Thread osstest service owner
flight 100670 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/100670/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 965268ea6df485d78b982d00270bd4ce7f673820 baseline version: ovmf 5f53a7aa59d4df1fe4326

[Xen-devel] [PATCH v3 0/6] VMX: Properly handle pi descriptor and per-cpu blocking list

2016-08-30 Thread Feng Wu
The current VT-d PI related code may operate incorrectly in the following scenarios: 1. When the last assigned device is dettached from the domain, all the PI related hooks are removed then, however, the vCPU can be blocked, switched to another pCPU, etc, all without the aware of PI. After the next

[Xen-devel] [PATCH v3 6/6] VMX: Fixup PI descritpor when cpu is offline

2016-08-30 Thread Feng Wu
When cpu is offline, we need to move all the vcpus in its blocking list to another online cpu, this patch handles it. Signed-off-by: Feng Wu --- xen/arch/x86/hvm/vmx/vmcs.c | 1 + xen/arch/x86/hvm/vmx/vmx.c| 54 +++ xen/include/asm-x86/hvm/vmx/v

[Xen-devel] [PATCH v3 2/6] VMX: Properly handle pi when all the assigned devices are removed

2016-08-30 Thread Feng Wu
This patch handles some concern cases when the last assigned device is removed from the domain. In this case we should carefully handle pi descriptor and the per-cpu blocking list, to make sure: - all the PI descriptor are in the right state when next time a devices is assigned to the domain again.

[Xen-devel] [PATCH v3 3/6] VMX: Cleanup PI per-cpu blocking list when vcpu is destroyed

2016-08-30 Thread Feng Wu
We should remove the vCPU from the per-cpu blocking list if it is going to be destroyed. Signed-off-by: Feng Wu --- xen/arch/x86/hvm/vmx/vmx.c | 1 + 1 file changed, 1 insertion(+) diff --git a/xen/arch/x86/hvm/vmx/vmx.c b/xen/arch/x86/hvm/vmx/vmx.c index b869728..37fa2f1 100644 --- a/xen/arch/

[Xen-devel] [PATCH v3 1/6] VMX: Statically assign two PI hooks

2016-08-30 Thread Feng Wu
PI hooks: vmx_pi_switch_from() and vmx_pi_switch_to() are needed even all the assigned devices were dettached from the domain. We change the state of SN bit in these two functions, and evaluate this bit in vmx_deliver_posted_intr() when trying to deliver the interrupt in posted way via software. Th

[Xen-devel] [PATCH v3 5/6] VT-d: No need to set irq affinity for posted format IRTE

2016-08-30 Thread Feng Wu
We don't set the affinity for posted format IRTE, since the destination of these interrupts is vCPU and the vCPU affinity is set during vCPU scheduling. Signed-off-by: Feng Wu --- xen/drivers/passthrough/vtd/intremap.c | 62 +++--- 1 file changed, 34 insertions(+), 28

[Xen-devel] [PATCH v3 4/6] Pause/Unpause the domain before/after assigning PI hooks

2016-08-30 Thread Feng Wu
Pausing the domain can make sure the vCPU is not running and hence calling the hooks simultaneously when deassigning the PI hooks. This makes sure that all the appropriate state of PI descriptor is actually set up for all vCPus before leaving this function. Signed-off-by: Feng Wu --- xen/arch/x8

[Xen-devel] [seabios baseline-only test] 67610: tolerable FAIL

2016-08-30 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 67610 seabios real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67610/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-stop

Re: [Xen-devel] [RFC 12/22] xen/arm: p2m: Introduce p2m_get_entry and use it to implement __p2m_lookupo

2016-08-30 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > Currently, for a given GFN, the function __p2m_lookup will only return > the associated MFN and the p2m type of the mapping. > > In some case we need the order of the mapping and the memaccess > permission. Rather than providing separate function for this

[Xen-devel] [qemu-mainline test] 100664: tolerable FAIL - PUSHED

2016-08-30 Thread osstest service owner
flight 100664 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/100664/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemuu-win7-amd64 16 guest-stop fail like 100660 test-amd64-amd64-xl-rtds

[Xen-devel] [seabios test] 100665: tolerable FAIL - PUSHED

2016-08-30 Thread osstest service owner
flight 100665 seabios real [real] http://logs.test-lab.xenproject.org/osstest/logs/100665/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemuu-win7-amd64 16 guest-stop fail like 100577 test-amd64-amd64-xl-qemuu-win7-a

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

2016-08-30 Thread osstest service owner
flight 100663 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/100663/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt-qemuu-debianhvm-amd64-xsm 9 debian-hvm-install fail REGR. vs. 100657 Re

Re: [Xen-devel] [PATCH v2] xen: use native disk xenbus protocol if possible

2016-08-30 Thread Stefano Stabellini
On Tue, 30 Aug 2016, Juergen Gross wrote: > On 05/07/16 16:55, Stefano Stabellini wrote: > > On Wed, 29 Jun 2016, Anthony PERARD wrote: > >> On Wed, Jun 29, 2016 at 05:50:48PM +0200, Juergen Gross wrote: > >>> The qdisk implementation is using the native xenbus protocol only in > >>> case of no pro

Re: [Xen-devel] [PATCH 4/4] xen: add a gcov Kconfig option

2016-08-30 Thread Doug Goldstein
> On Aug 30, 2016, at 8:06 AM, Wei Liu wrote: > > On Tue, Aug 30, 2016 at 06:56:36AM -0600, Jan Beulich wrote: > On 30.08.16 at 14:01, wrote: >>> Signed-off-by: Wei Liu >> >> Acked-by: Jan Beulich >> >> with one adjustment (see below). Thanks for doing this Wei. I agree with Jan's revi

Re: [Xen-devel] Device model operation hypercall (DMOP, re qemu depriv)

2016-08-30 Thread Stefano Stabellini
On Tue, 30 Aug 2016, Ian Jackson wrote: > Jan Beulich writes ("Re: Device model operation hypercall (DMOP, re qemu > depriv)"): > > Well, in a way. And then not: Initially I had thought no issue would > > arise, until I came to realize the kernel memory corruption potential. > > Question is whethe

[Xen-devel] [qemu-mainline baseline-only test] 67609: tolerable FAIL

2016-08-30 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 67609 qemu-mainline real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67609/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-qemuu-nested-intel 16 debian-hvm-insta

Re: [Xen-devel] [PATCH 3/3] mini-os: support "make config" for out-of-tree users

2016-08-30 Thread Samuel Thibault
Hello, Juergen Gross, on Tue 30 Aug 2016 13:51:23 +0200, wrote: > @@ -51,7 +51,7 @@ endif > > libc = $(stubdom) > > -XEN_INTERFACE_VERSION := 0x00030205 > +XEN_INTERFACE_VERSION ?= 0x00030205 Why making it overridable? AIUI changing the version would change the xen headers API, and thus the

Re: [Xen-devel] [PATCH v4 05/16] libxl/arm: Construct ACPI RSDP table

2016-08-30 Thread Stefano Stabellini
On Tue, 30 Aug 2016, Julien Grall wrote: > Hi Shannon, > > On 30/08/16 02:21, Shannon Zhao wrote: > > > > > > On 2016/8/30 2:05, Julien Grall wrote: > > > Hi Shannon, > > > > > > On 25/08/2016 04:05, Shannon Zhao wrote: > > > > > > > > > > > > On 2016/8/24 20:52, Wei Liu wrote: > > > > > On T

Re: [Xen-devel] [Minios-devel] [PATCH 2/3] mini-os: provide irq on/off/save/restore functions for Mini-OS apps

2016-08-30 Thread Samuel Thibault
Juergen Gross, on Tue 30 Aug 2016 16:08:45 +0200, wrote: > On 30/08/16 15:54, Wei Liu wrote: > > On Tue, Aug 30, 2016 at 01:51:22PM +0200, Juergen Gross wrote: > >> Provide non-inline variants of the local_irq_*() functions for Mini-OS > >> apps which should not depend on Mini-OS configuration. > >

Re: [Xen-devel] [PATCH 1/3] mini-os: support newer xen interface

2016-08-30 Thread Samuel Thibault
Wei Liu, on Tue 30 Aug 2016 14:53:10 +0100, wrote: > On Tue, Aug 30, 2016 at 01:51:21PM +0200, Juergen Gross wrote: > > Mini-OS is currently setting __XEN_INTERFACE_VERSION__ to a rather > > ancient version. > > > > To be able to use a more recent variant garnt_entry_t must be changed > > to grant

Re: [Xen-devel] [PATCH v4 08/16] kbuild: enable option to force compile force-obj-y and force-lib-y

2016-08-30 Thread Luis R. Rodriguez
On Tue, Aug 23, 2016 at 01:59:10AM +0200, Luis R. Rodriguez wrote: > On Fri, Aug 19, 2016 at 03:10:33PM -0700, Kees Cook wrote: > > On Fri, Aug 19, 2016 at 2:32 PM, wrote: > > > diff --git a/init/Kconfig b/init/Kconfig > > > index cac3f096050d..ef09e83b9196 100644 > > > --- a/init/Kconfig > > > +

Re: [Xen-devel] [PATCH v4 14/16] kprobes: move kprobe declarations to asm-generic/kprobes.h

2016-08-30 Thread Luis R. Rodriguez
On Mon, Aug 29, 2016 at 11:04:18PM +0900, Masami Hiramatsu wrote: > On Tue, 23 Aug 2016 18:31:05 +0200 > "Luis R. Rodriguez" wrote: > > > On Tue, Aug 23, 2016 at 12:11:40AM +0900, Masami Hiramatsu wrote: > > > On Fri, 19 Aug 2016 14:34:12 -0700 > > > mcg...@kernel.org wrote: > > > > > > > From:

Re: [Xen-devel] [PATCH v5 14/16] x86/boot: implement early command line parser in C

2016-08-30 Thread Daniel Kiper
On Thu, Aug 25, 2016 at 07:27:21AM -0600, Jan Beulich wrote: > >>> On 20.08.16 at 00:43, wrote: > > +#define NULL ((void *)0) > > + > > +#define __packed __attribute__((__packed__)) > > +#define __stdcall __attribute__((__stdcall__)) > > + > > +#define max(x,y) ({ \ > > +const typ

Re: [Xen-devel] [PATCH v5 13/16] x86: add multiboot2 protocol support for EFI platforms

2016-08-30 Thread Daniel Kiper
On Thu, Aug 25, 2016 at 06:59:54AM -0600, Jan Beulich wrote: > >>> On 20.08.16 at 00:43, wrote: > > @@ -100,19 +107,45 @@ multiboot2_header_start: > > gdt_boot_descr: > > .word 6*8-1 > > .long sym_phys(trampoline_gdt) > > +.long 0 /* Needed for 64-bit lgdt */ > > +

Re: [Xen-devel] Moving Xen Project ACPI tables to xenbits

2016-08-30 Thread Julien Grall
(CC xen-devel) Hi, I failed to CC xen-devel on my first ML. Sorry for that. On 30/08/2016 19:28, Ian Jackson wrote: Julien Grall writes ("Moving Xen Project ACPI tables to xenbits"): Some ACPI tables (STAO, XENV) are currently maintained by the Xen Project and referenced on the ACPI webpage [

Re: [Xen-devel] [PATCH v5 10/16] efi: create efi_enabled()

2016-08-30 Thread Daniel Kiper
On Thu, Aug 25, 2016 at 06:16:35AM -0600, Jan Beulich wrote: > >>> On 20.08.16 at 00:43, wrote: > > --- a/xen/arch/x86/efi/stub.c > > +++ b/xen/arch/x86/efi/stub.c > > @@ -4,9 +4,10 @@ > > #include > > #include > > > > -#ifndef efi_enabled > > -const bool_t efi_enabled = 0; > > -#endif > > +bo

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

2016-08-30 Thread osstest service owner
flight 100662 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/100662/ 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 1

Re: [Xen-devel] [PATCH v4 05/16] libxl/arm: Construct ACPI RSDP table

2016-08-30 Thread Julien Grall
Hi Shannon, On 30/08/16 02:21, Shannon Zhao wrote: On 2016/8/30 2:05, Julien Grall wrote: Hi Shannon, On 25/08/2016 04:05, Shannon Zhao wrote: On 2016/8/24 20:52, Wei Liu wrote: On Tue, Aug 16, 2016 at 06:25:02PM +0800, Shannon Zhao wrote: From: Shannon Zhao Construct ACPI RSDP table

Re: [Xen-devel] [PATCH 3/3] x86/EFI: don't accept 64-bit base relocations on page tables

2016-08-30 Thread Andrew Cooper
On 19/08/16 13:57, Jan Beulich wrote: On 19.08.16 at 14:39, wrote: >> On 19/08/16 08:52, Jan Beulich wrote: >>> Page tables get pre-populated with physical addresses which, due to >>> living inside the Xen image, will never exceed 32 bits in width. That >>> in turn results in the tool generat

[Xen-devel] [qemu-mainline test] 100660: tolerable FAIL - PUSHED

2016-08-30 Thread osstest service owner
flight 100660 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/100660/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemuu-win7-amd64 16 guest-stop fail like 100611 test-amd64-amd64-xl-rtds

[Xen-devel] Ping: [PATCH 3/3] x86/EFI: don't accept 64-bit base relocations on page tables

2016-08-30 Thread Jan Beulich
>>> On 19.08.16 at 09:52, wrote: > Page tables get pre-populated with physical addresses which, due to > living inside the Xen image, will never exceed 32 bits in width. That > in turn results in the tool generating the relocations to produce > 32-bit relocations for them instead of the 64-bit one

Re: [Xen-devel] [PATCH v2] xen: use native disk xenbus protocol if possible

2016-08-30 Thread Juergen Gross
On 05/07/16 16:55, Stefano Stabellini wrote: > On Wed, 29 Jun 2016, Anthony PERARD wrote: >> On Wed, Jun 29, 2016 at 05:50:48PM +0200, Juergen Gross wrote: >>> The qdisk implementation is using the native xenbus protocol only in >>> case of no protocol specified at all. As using the explicit 32- or

Re: [Xen-devel] [OSSTEST PATCH 0/7] Apropos of XTF: Improve failed step selection

2016-08-30 Thread Andrew Cooper
On 30/08/16 17:00, Wei Liu wrote: > On Thu, Aug 11, 2016 at 05:17:56PM +0100, Ian Jackson wrote: >> Running XTF in osstest is likely to produce failures where multiple >> steps fail interestingly. We would like to prefer to report, and >> bisect, earlier steps. >> >> This series does that. >> >> W

Re: [Xen-devel] [OSSTEST PATCH 0/7] Apropos of XTF: Improve failed step selection

2016-08-30 Thread Ian Jackson
Wei Liu writes ("Re: [OSSTEST PATCH 0/7] Apropos of XTF: Improve failed step selection"): > The rebased patch is as followed, please check. Acked-by: Ian Jackson Thanks, Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xe

Re: [Xen-devel] Questions about Using Perf at Dom0

2016-08-30 Thread Boris Ostrovsky
On 08/30/2016 11:22 AM, Sanghyun Hong wrote: >> As I said, this works for me for Xen guests. I thought perf kvm might >> check whether there is a KVM guest but it's not. The error message that >> you see is generated at the end of sampling if no samples are found. > > Oh, I see. I tried the same co

Re: [Xen-devel] [OSSTEST PATCH 0/7] Apropos of XTF: Improve failed step selection

2016-08-30 Thread Wei Liu
On Thu, Aug 11, 2016 at 05:17:56PM +0100, Ian Jackson wrote: > Running XTF in osstest is likely to produce failures where multiple > steps fail interestingly. We would like to prefer to report, and > bisect, earlier steps. > > This series does that. > > Wei, NB, this has a conflict with the XTF

Re: [Xen-devel] [PATCH v2] make use of .startof.() and .sizeof.() assembler expressions

2016-08-30 Thread Julien Grall
On 30/08/16 16:15, Jan Beulich wrote: On 30.08.16 at 15:18, wrote: On 30/08/16 14:09, Jan Beulich wrote: On 30.08.16 at 14:50, wrote: If you want to add it, it should be a separate patch and we should go further by removing the section in the linker script. While I don't think this needs

Re: [Xen-devel] Questions about Using Perf at Dom0

2016-08-30 Thread Sanghyun Hong
As I said, this works for me for Xen guests. I thought perf kvm might check whether there is a KVM guest but it's not. The error message that you see is generated at the end of sampling if no samples are found. Oh, I see. I tried the same command and check the contents of the perf.data.kvm file,

Re: [Xen-devel] [PATCH v5 02/16] x86/boot: remove multiboot1_header_end from symbol table

2016-08-30 Thread Jan Beulich
>>> On 30.08.16 at 16:27, wrote: > On Thu, Aug 25, 2016 at 05:21:24AM -0600, Jan Beulich wrote: >> >>> On 20.08.16 at 00:43, wrote: >> > Its visibility is not needed and just pollute symbol table. >> > >> > Suggested-by: Jan Beulich >> > Signed-off-by: Daniel Kiper >> >> With Andrew effectively

[Xen-devel] [PATCH RESEND] xenbus: Use proc_create_mount_point() to create /proc/xen

2016-08-30 Thread Seth Forshee
Mounting proc in user namespace containers fails if the xenbus filesystem is mounted on /proc/xen because this directory fails the "permanently empty" test. proc_create_mount_point() exists specifically to create such mountpoints in proc but is currently proc-internal. Export this interface to modu

Re: [Xen-devel] [PATCH v2] make use of .startof.() and .sizeof.() assembler expressions

2016-08-30 Thread Jan Beulich
>>> On 30.08.16 at 15:18, wrote: > On 30/08/16 14:09, Jan Beulich wrote: > On 30.08.16 at 14:50, wrote: >>> If you want to add it, it should be a separate patch and we should go >>> further by removing the section in the linker script. >> >> While I don't think this needs to be broken out, it

Re: [Xen-devel] [PATCH v5 09/16] x86: add multiboot2 protocol support

2016-08-30 Thread Jan Beulich
>>> On 30.08.16 at 16:41, wrote: > On Thu, Aug 25, 2016 at 05:50:04AM -0600, Jan Beulich wrote: >> >>> On 20.08.16 at 00:43, wrote: >> > +case MULTIBOOT2_TAG_TYPE_MMAP: >> > +if ( get_mb2_data(tag, mmap, entry_size) < sizeof(*mmap_src) ) >> > +break; >> > + >>

Re: [Xen-devel] [PATCH] xenbus: Use proc_create_mount_point() to create /proc/xen

2016-08-30 Thread David Vrabel
On 30/08/16 16:10, Seth Forshee wrote: > On Tue, Aug 30, 2016 at 04:00:03PM +0100, David Vrabel wrote: >> On 29/08/16 16:03, Seth Forshee wrote: >>> Mounting proc in user namespace containers fails if the xenbus >>> filesystem is mounted on /proc/xen because this directory fails >>> the "permanentl

Re: [Xen-devel] [PATCH v5 06/16] x86/boot/reloc: create generic alloc and copy functions

2016-08-30 Thread Jan Beulich
>>> On 30.08.16 at 16:32, wrote: > On Thu, Aug 25, 2016 at 05:34:31AM -0600, Jan Beulich wrote: >> >>> On 20.08.16 at 00:43, wrote: >> > Create generic alloc and copy functions. We need >> > separate tools for memory allocation and copy to >> > provide multiboot2 protocol support. >> > >> > Signe

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

2016-08-30 Thread Jan Beulich
>>> On 30.08.16 at 16:15, wrote: > On Mon, Aug 22, 2016 at 04:10:04AM -0600, Jan Beulich wrote: >> >>> On 20.08.16 at 00:43, wrote: >> > The final goal is xen.efi binary file which could be loaded by EFI >> > loader, multiboot (v1) protocol (only on legacy BIOS platforms) and >> > multiboot2 prot

Re: [Xen-devel] [PATCH] xenbus: Use proc_create_mount_point() to create /proc/xen

2016-08-30 Thread Seth Forshee
On Tue, Aug 30, 2016 at 04:00:03PM +0100, David Vrabel wrote: > On 29/08/16 16:03, Seth Forshee wrote: > > Mounting proc in user namespace containers fails if the xenbus > > filesystem is mounted on /proc/xen because this directory fails > > the "permanently empty" test. proc_create_mount_point() e

Re: [Xen-devel] [PATCH] xenbus: Use proc_create_mount_point() to create /proc/xen

2016-08-30 Thread Seth Forshee
On Tue, Aug 30, 2016 at 04:48:08PM +0200, Juergen Gross wrote: > On 29/08/16 17:03, Seth Forshee wrote: > > Mounting proc in user namespace containers fails if the xenbus > > filesystem is mounted on /proc/xen because this directory fails > > the "permanently empty" test. proc_create_mount_point()

Re: [Xen-devel] [PATCH] xenbus: Use proc_create_mount_point() to create /proc/xen

2016-08-30 Thread David Vrabel
On 29/08/16 16:03, Seth Forshee wrote: > Mounting proc in user namespace containers fails if the xenbus > filesystem is mounted on /proc/xen because this directory fails > the "permanently empty" test. proc_create_mount_point() exists > specifically to create such mountpoints in proc but is current

[Xen-devel] [PATCH v2] stubdom: support Mini-OS config for Mini-OS apps

2016-08-30 Thread Juergen Gross
Mini-OS apps need to be compiled with the appropriate config settings of Mini-OS, as there are various dependencies on those settings in header files included by the apps. Enhance stubdom Makefile to set the appropriate CPPFLAGS when calling the apps' make. Signed-off-by: Juergen Gross --- V2: a

Re: [Xen-devel] [PATCH] xenbus: Use proc_create_mount_point() to create /proc/xen

2016-08-30 Thread Juergen Gross
On 29/08/16 17:03, Seth Forshee wrote: > Mounting proc in user namespace containers fails if the xenbus > filesystem is mounted on /proc/xen because this directory fails > the "permanently empty" test. proc_create_mount_point() exists > specifically to create such mountpoints in proc but is current

Re: [Xen-devel] [PATCH v5 09/16] x86: add multiboot2 protocol support

2016-08-30 Thread Daniel Kiper
On Thu, Aug 25, 2016 at 05:50:04AM -0600, Jan Beulich wrote: > >>> On 20.08.16 at 00:43, wrote: > > +case MULTIBOOT2_TAG_TYPE_MMAP: > > +if ( get_mb2_data(tag, mmap, entry_size) < sizeof(*mmap_src) ) > > +break; > > + > > +mbi_out->flags |= MBI_MEMMA

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

2016-08-30 Thread osstest service owner
flight 100661 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/100661/ 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 1

Re: [Xen-devel] Questions about Using Perf at Dom0

2016-08-30 Thread Boris Ostrovsky
On 08/29/2016 08:12 PM, Sanghyun Hong wrote: > Hi Boris, > > /[Appreciate for your support, and I have one last question]/ > > Since we’re using the */Xen/* hypervisor (not the KVM that is > type-II), I think we cannot use the /*perf k*/*/vm/* command. Here are > the outputs: > > > # pe

Re: [Xen-devel] [PATCH v5 06/16] x86/boot/reloc: create generic alloc and copy functions

2016-08-30 Thread Daniel Kiper
On Thu, Aug 25, 2016 at 05:34:31AM -0600, Jan Beulich wrote: > >>> On 20.08.16 at 00:43, wrote: > > Create generic alloc and copy functions. We need > > separate tools for memory allocation and copy to > > provide multiboot2 protocol support. > > > > Signed-off-by: Daniel Kiper > > The amount of

Re: [Xen-devel] [PATCH v5 02/16] x86/boot: remove multiboot1_header_end from symbol table

2016-08-30 Thread Daniel Kiper
On Thu, Aug 25, 2016 at 05:21:24AM -0600, Jan Beulich wrote: > >>> On 20.08.16 at 00:43, wrote: > > Its visibility is not needed and just pollute symbol table. > > > > Suggested-by: Jan Beulich > > Signed-off-by: Daniel Kiper > > With Andrew effectively having NAK-ed v4 of this patch, I don't se

Re: [Xen-devel] [PATCH] memory: fix compat handling of XENMEM_access_op

2016-08-30 Thread Razvan Cojocaru
On 08/30/2016 01:05 PM, Jan Beulich wrote: On 30.08.16 at 11:46, wrote: >> On 08/30/2016 12:15 PM, Jan Beulich wrote: >>> Within compat_memory_op() this needs to be placed in the first switch() >>> statement, or it ends up being dead code (as that first switch() has a >>> default case chainin

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

2016-08-30 Thread Daniel Kiper
On Mon, Aug 22, 2016 at 04:10:04AM -0600, Jan Beulich wrote: > >>> On 20.08.16 at 00:43, wrote: > > The final goal is xen.efi binary file which could be loaded by EFI > > loader, multiboot (v1) protocol (only on legacy BIOS platforms) and > > multiboot2 protocol. This way we will have: > > - sma

Re: [Xen-devel] [PATCH v3 5/6] x86/time: implement PVCLOCK_TSC_STABLE_BIT

2016-08-30 Thread Joao Martins
On 08/30/2016 01:45 PM, Jan Beulich wrote: On 30.08.16 at 14:26, wrote: >> On 08/29/2016 11:06 AM, Jan Beulich wrote: >> On 26.08.16 at 17:44, wrote: On 08/25/2016 11:37 AM, Jan Beulich wrote: On 24.08.16 at 14:43, wrote: >> This patch proposes relying on host TSC sync

Re: [Xen-devel] [Minios-devel] [PATCH 2/3] mini-os: provide irq on/off/save/restore functions for Mini-OS apps

2016-08-30 Thread Juergen Gross
On 30/08/16 15:54, Wei Liu wrote: > On Tue, Aug 30, 2016 at 01:51:22PM +0200, Juergen Gross wrote: >> Provide non-inline variants of the local_irq_*() functions for Mini-OS >> apps which should not depend on Mini-OS configuration. >> > > I think it would be worth pointing out which apps need to ac

Re: [Xen-devel] [PATCH] stubdom: support Mini-OS config for Mini-OS apps

2016-08-30 Thread Juergen Gross
On 30/08/16 15:52, Wei Liu wrote: > On Tue, Aug 30, 2016 at 01:52:40PM +0200, Juergen Gross wrote: >> Mini-OS apps need to be compiled with the appropriate config settings >> of Mini-OS, as there are various dependencies on those settings in >> header files included by the apps. >> >> Enhance stubd

Re: [Xen-devel] [PATCH v3 2/6] x86/time: implement tsc as clocksource

2016-08-30 Thread Joao Martins
On 08/30/2016 01:30 PM, Jan Beulich wrote: On 30.08.16 at 14:08, wrote: >> On 08/29/2016 10:36 AM, Jan Beulich wrote: >> On 26.08.16 at 17:11, wrote: On 08/25/2016 11:06 AM, Jan Beulich wrote: On 24.08.16 at 14:43, wrote: >> - Change init_tsctimer to skip all the te

Re: [Xen-devel] [Minios-devel] [PATCH 3/3] mini-os: support "make config" for out-of-tree users

2016-08-30 Thread Wei Liu
On Tue, Aug 30, 2016 at 01:51:23PM +0200, Juergen Gross wrote: > Mini-OS applications being compiled using Mini-OS headers without > being integrated in the make environment of Mini-OS need a way to set > CONFIG_* defines according to their Mini-OS configuration. > > Add a new make target "config"

Re: [Xen-devel] [Minios-devel] [PATCH 2/3] mini-os: provide irq on/off/save/restore functions for Mini-OS apps

2016-08-30 Thread Wei Liu
On Tue, Aug 30, 2016 at 01:51:22PM +0200, Juergen Gross wrote: > Provide non-inline variants of the local_irq_*() functions for Mini-OS > apps which should not depend on Mini-OS configuration. > I think it would be worth pointing out which apps need to access such low level functionalities in com

Re: [Xen-devel] [PATCH 1/3] mini-os: support newer xen interface

2016-08-30 Thread Wei Liu
On Tue, Aug 30, 2016 at 01:51:21PM +0200, Juergen Gross wrote: > Mini-OS is currently setting __XEN_INTERFACE_VERSION__ to a rather > ancient version. > > To be able to use a more recent variant garnt_entry_t must be changed > to grant_entry_v1_t. In balloon.c we omit initializing elements of > st

Re: [Xen-devel] [PATCH] stubdom: support Mini-OS config for Mini-OS apps

2016-08-30 Thread Wei Liu
On Tue, Aug 30, 2016 at 01:52:40PM +0200, Juergen Gross wrote: > Mini-OS apps need to be compiled with the appropriate config settings > of Mini-OS, as there are various dependencies on those settings in > header files included by the apps. > > Enhance stubdom Makefile to set the appropriate CPPFL

Re: [Xen-devel] [PATCH] libxl: update flex output files

2016-08-30 Thread Wei Liu
On Tue, Aug 30, 2016 at 01:51:33PM +0100, Ian Jackson wrote: > Wei Liu writes ("[PATCH] libxl: update flex output files"): > > Libxl ships output files from flex (libxlu_*_l.{c,h}). We use the flex > > shipped in Debian to generate those files. Debian just patched their > > flex (DSA 3653-1) to fix

Re: [Xen-devel] [PATCH 3/4] xen: fix gcov compilation

2016-08-30 Thread Wei Liu
On Tue, Aug 30, 2016 at 01:57:06PM +0100, Ian Jackson wrote: > Wei Liu writes ("[PATCH 3/4] xen: fix gcov compilation"): > > Currently enabling gcov in hypervisor won't build because although > > 26c9d03d ("gcov: Adding support for coverage information") claimed that > > %.init.o files were exclude

Re: [Xen-devel] [PATCH 2/4] arm64: use "b" to branch to start_xen

2016-08-30 Thread Konrad Rzeszutek Wilk
On Tue, Aug 30, 2016 at 01:01:30PM +0100, Wei Liu wrote: > The cbz instruction has range limitation. When compiled with gcov > support the object is larger so cbz can't handle that anymore. The error > message is like: > > aarch64-linux-gnu-ld-EL -T xen.lds -N prelink.o \ > /local/work/xe

Re: [Xen-devel] [PATCH v2 1/2] tools: remove blktap2 related code and documentation

2016-08-30 Thread Wei Liu
On Tue, Aug 30, 2016 at 01:54:17PM +0100, Ian Jackson wrote: > Wei Liu writes ("[PATCH v2 1/2] tools: remove blktap2 related code and > documentation"): > > Blktap2 is effectively dead code for a few years. > > > > Notable changes in this patch: > > > > 0. Unhook blktap2 from build system > > 1.

Re: [Xen-devel] [PATCH v2] make use of .startof.() and .sizeof.() assembler expressions

2016-08-30 Thread Julien Grall
Hi Jan, On 30/08/16 14:09, Jan Beulich wrote: On 30.08.16 at 14:50, wrote: On 24/08/16 08:44, Jan Beulich wrote: --- a/xen/arch/arm/device.c +++ b/xen/arch/arm/device.c @@ -21,8 +21,8 @@ #include #include -extern const struct device_desc _sdevice[], _edevice[]; -extern const struct acpi_

Re: [Xen-devel] [PATCH v2] make use of .startof.() and .sizeof.() assembler expressions

2016-08-30 Thread Jan Beulich
>>> On 30.08.16 at 14:50, wrote: > On 24/08/16 08:44, Jan Beulich wrote: >> --- a/xen/arch/arm/device.c >> +++ b/xen/arch/arm/device.c >> @@ -21,8 +21,8 @@ >> #include >> #include >> >> -extern const struct device_desc _sdevice[], _edevice[]; >> -extern const struct acpi_device_desc _asdevice[

Re: [Xen-devel] [PATCH 4/4] xen: add a gcov Kconfig option

2016-08-30 Thread Wei Liu
On Tue, Aug 30, 2016 at 06:56:36AM -0600, Jan Beulich wrote: > >>> On 30.08.16 at 14:01, wrote: > > Signed-off-by: Wei Liu > > Acked-by: Jan Beulich > > with one adjustment (see below). > > > Cc: Konrad Rzeszutek Wilk > > Cc: Stefano Stabellini > > Cc: Tim Deegan > > Cc: Doug Goldstein >

Re: [Xen-devel] [PATCH 3/4] xen: fix gcov compilation

2016-08-30 Thread Jan Beulich
>>> On 30.08.16 at 14:57, wrote: > Wei Liu writes ("[PATCH 3/4] xen: fix gcov compilation"): >> Currently enabling gcov in hypervisor won't build because although >> 26c9d03d ("gcov: Adding support for coverage information") claimed that >> %.init.o files were excluded from applying compilation op

[Xen-devel] [ovmf baseline-only test] 67608: all pass

2016-08-30 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 67608 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67608/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 5f53a7aa59d4df1fe4326af18a9240d4dfebc129 baseline v

Re: [Xen-devel] [PATCH] libxc: correct max_pfn calculation for saving domain

2016-08-30 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH] libxc: correct max_pfn calculation for saving domain"): > Pushed. > > Ian, please backport this to Xen 4.7. Noted, thanks. Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH] libxl: update flex output files

2016-08-30 Thread Ian Jackson
Wei Liu writes ("[PATCH] libxl: update flex output files"): > Libxl ships output files from flex (libxlu_*_l.{c,h}). We use the flex > shipped in Debian to generate those files. Debian just patched their > flex (DSA 3653-1) to fix CVE-2016-6354, which is a buffer overrun bug. > > Note that libxl i

Re: [Xen-devel] [PATCH 3/4] xen: fix gcov compilation

2016-08-30 Thread Ian Jackson
Wei Liu writes ("[PATCH 3/4] xen: fix gcov compilation"): > Currently enabling gcov in hypervisor won't build because although > 26c9d03d ("gcov: Adding support for coverage information") claimed that > %.init.o files were excluded from applying compilation options, it was > in fact not true. > >

Re: [Xen-devel] [PATCH 4/4] xen: add a gcov Kconfig option

2016-08-30 Thread Jan Beulich
>>> On 30.08.16 at 14:01, wrote: > Signed-off-by: Wei Liu Acked-by: Jan Beulich with one adjustment (see below). > Cc: Konrad Rzeszutek Wilk > Cc: Stefano Stabellini > Cc: Tim Deegan > Cc: Doug Goldstein > > Would like to have it default to DEBUG, but we also need to check > compiler to

Re: [Xen-devel] [PATCH v2 1/2] tools: remove blktap2 related code and documentation

2016-08-30 Thread Ian Jackson
Wei Liu writes ("[PATCH v2 1/2] tools: remove blktap2 related code and documentation"): > Blktap2 is effectively dead code for a few years. > > Notable changes in this patch: > > 0. Unhook blktap2 from build system > 1. Now libxl no longer supports TAP disk backend, appropriate assertions >a

Re: [Xen-devel] [PATCH] tools: delete gtraceview and gtracestat

2016-08-30 Thread Ian Jackson
Wei Liu writes ("[PATCH] tools: delete gtraceview and gtracestat"): > There has not been any substantial update to them since 2011. My quick > check shows that they don't work. > > Just delete them. It would be easy to resurrect them from git log should > people still need them. I'm not sure what

Re: [Xen-devel] [PATCH v2] make use of .startof.() and .sizeof.() assembler expressions

2016-08-30 Thread Julien Grall
Hi Jan, On 24/08/16 08:44, Jan Beulich wrote: --- a/xen/arch/arm/device.c +++ b/xen/arch/arm/device.c @@ -21,8 +21,8 @@ #include #include -extern const struct device_desc _sdevice[], _edevice[]; -extern const struct acpi_device_desc _asdevice[], _aedevice[]; +extern const struct device_desc

Re: [Xen-devel] [PATCH v3 5/6] x86/time: implement PVCLOCK_TSC_STABLE_BIT

2016-08-30 Thread Jan Beulich
>>> On 30.08.16 at 14:26, wrote: > On 08/29/2016 11:06 AM, Jan Beulich wrote: > On 26.08.16 at 17:44, wrote: >>> On 08/25/2016 11:37 AM, Jan Beulich wrote: >>> On 24.08.16 at 14:43, wrote: > This patch proposes relying on host TSC synchronization and > passthrough to the guest, w

Re: [Xen-devel] [PATCH] libxc: correct max_pfn calculation for saving domain

2016-08-30 Thread Wei Liu
On Fri, Aug 26, 2016 at 02:09:53PM +0100, Wei Liu wrote: > On Fri, Aug 26, 2016 at 01:58:55PM +0200, Juergen Gross wrote: > > Commit 91e204d37f44913913776d0a89279721694f8b32 ("libxc: try to find > > last used pfn when migrating") introduced a bug for the case of a > > domain supporting the virtual

Re: [Xen-devel] Regression between Xen 4.6.0 and 4.7.0, Direct kernel boot on a qemu-xen and seabios HVM guest doesn't work anymore.

2016-08-30 Thread Wei Liu
Could you please use xl -vvv create to create the guest and collect the output? Wei. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] find_next{,_zero}_bit() inconsistencies

2016-08-30 Thread Julien Grall
Hi Jan, On 30/08/16 11:08, Jan Beulich wrote: On 30.08.16 at 11:26, wrote: On 29/08/16 12:59, Jan Beulich wrote: Hello, in the context of https://lists.xenproject.org/archives/html/xen-devel/2016-08/msg03068.html I once again came across the different behavior our various implementations of

Re: [Xen-devel] [PATCH v3 3/6] x86/time: streamline platform time init on plt_update()

2016-08-30 Thread Jan Beulich
>>> On 30.08.16 at 14:10, wrote: > What would be a preferred period - probably capping it to a day/hour/minutes? > Or > keeping the current calculated value? Maximum right now in current platform > timers > are few minutes depending on the HPET frequency. Ideally I would see you just use the ca

Re: [Xen-devel] [PATCH v3 2/6] x86/time: implement tsc as clocksource

2016-08-30 Thread Jan Beulich
>>> On 30.08.16 at 14:08, wrote: > On 08/29/2016 10:36 AM, Jan Beulich wrote: > On 26.08.16 at 17:11, wrote: >>> On 08/25/2016 11:06 AM, Jan Beulich wrote: >>> On 24.08.16 at 14:43, wrote: > - Change init_tsctimer to skip all the tests and assume it's called > only on reliable

Re: [Xen-devel] [PATCH] x86/PV: remove read_descriptor()'s regs parameter

2016-08-30 Thread Andrew Cooper
On 30/08/16 13:18, Jan Beulich wrote: > As of commit a35dc6ccbb ("x86: remove the use of vm86_mode()") it is > unused. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xe

Re: [Xen-devel] [PATCH v3 5/6] x86/time: implement PVCLOCK_TSC_STABLE_BIT

2016-08-30 Thread Joao Martins
On 08/29/2016 11:06 AM, Jan Beulich wrote: On 26.08.16 at 17:44, wrote: >> On 08/25/2016 11:37 AM, Jan Beulich wrote: >> On 24.08.16 at 14:43, wrote: This patch proposes relying on host TSC synchronization and passthrough to the guest, when running on a TSC-safe platform. On >>

[Xen-devel] [PATCH] x86/PV: remove read_descriptor()'s regs parameter

2016-08-30 Thread Jan Beulich
As of commit a35dc6ccbb ("x86: remove the use of vm86_mode()") it is unused. Signed-off-by: Jan Beulich --- a/xen/arch/x86/traps.c +++ b/xen/arch/x86/traps.c @@ -1851,7 +1851,6 @@ long do_fpu_taskswitch(int set) static int read_descriptor(unsigned int sel, const st

Re: [Xen-devel] [PATCH v5 4/4] x86/ioreq server: Reset outstanding p2m_ioreq_server entries when an ioreq server unmaps.

2016-08-30 Thread Yu Zhang
On 8/16/2016 9:35 PM, George Dunlap wrote: On 12/07/16 10:02, Yu Zhang wrote: This patch resets p2m_ioreq_server entries back to p2m_ram_rw, after an ioreq server has unmapped. The resync is done both asynchronously with the current p2m_change_entry_type_global() interface, and synchronously b

Re: [Xen-devel] [PATCH v3 3/6] x86/time: streamline platform time init on plt_update()

2016-08-30 Thread Joao Martins
On 08/29/2016 10:41 AM, Jan Beulich wrote: On 26.08.16 at 17:12, wrote: >> On 08/25/2016 11:13 AM, Jan Beulich wrote: >> On 24.08.16 at 14:43, wrote: And use to initialize platform time solely for clocksource=tsc, as opposed to initializing platform overflow timer, which would

Re: [Xen-devel] [PATCH v3 2/6] x86/time: implement tsc as clocksource

2016-08-30 Thread Joao Martins
On 08/29/2016 10:36 AM, Jan Beulich wrote: On 26.08.16 at 17:11, wrote: >> On 08/25/2016 11:06 AM, Jan Beulich wrote: >> On 24.08.16 at 14:43, wrote: This patch introduces support for using TSC as platform time source which is the highest resolution time and most performant to

[Xen-devel] [PATCH 1/4] arm: acpi/boot.c is only used during initialisation

2016-08-30 Thread Wei Liu
That file should contain code and data used during initialisation only. Mark it as such in build system and correctly annotate enabled_cpus. This is required to enable gcov support on ARM. Signed-off-by: Wei Liu --- Cc: Stefano Stabellini Cc: Julien Grall --- xen/arch/arm/acpi/Makefile | 2 +-

[Xen-devel] [PATCH 4/4] xen: add a gcov Kconfig option

2016-08-30 Thread Wei Liu
Signed-off-by: Wei Liu --- Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich Cc: Konrad Rzeszutek Wilk Cc: Stefano Stabellini Cc: Tim Deegan Cc: Doug Goldstein Would like to have it default to DEBUG, but we also need to check compiler to be gcc. I couldn't figure out how

[Xen-devel] [PATCH 2/4] arm64: use "b" to branch to start_xen

2016-08-30 Thread Wei Liu
The cbz instruction has range limitation. When compiled with gcov support the object is larger so cbz can't handle that anymore. The error message is like: aarch64-linux-gnu-ld-EL -T xen.lds -N prelink.o \ /local/work/xen.git/xen/common/symbols-dummy.o -o /local/work/xen.git/xen/.xen-sym

[Xen-devel] [PATCH 0/4] Fix gcov build

2016-08-30 Thread Wei Liu
A small series to fix gcov build for both x86 and arm. Wei Liu (4): arm: acpi/boot.c is only used during initialisation arm64: use "b" to branch to start_xen xen: fix gcov compilation xen: add a gcov Kconfig option Config.mk | 3 --- xen/Kconfig.debug | 5 +

[Xen-devel] [PATCH 3/4] xen: fix gcov compilation

2016-08-30 Thread Wei Liu
Currently enabling gcov in hypervisor won't build because although 26c9d03d ("gcov: Adding support for coverage information") claimed that %.init.o files were excluded from applying compilation options, it was in fact not true. Fix that by filtering out the options correctly. Due to the dependency

  1   2   >