Re: [Xen-devel] [PATCH v3 01/17] libxl/arm: Factor out codes for generating DTB

2016-07-07 Thread Julien Grall
On 07/07/16 17:12, Boris Ostrovsky wrote: On 07/07/2016 12:06 PM, Julien Grall wrote: On 07/07/16 16:48, Boris Ostrovsky wrote: On 07/07/2016 11:41 AM, Wei Liu wrote: On Tue, Jul 05, 2016 at 11:12:31AM +0800, Shannon Zhao wrote: From: Shannon Zhao Factor out codes for generating DTB to

Re: [Xen-devel] xenstored memory leak

2016-07-07 Thread Wei Liu
On Wed, Jul 06, 2016 at 09:31:38AM +0200, Juergen Gross wrote: > While testing some patches for support of ballooning in Mini-OS by using > the xenstore domain I realized that each xl create/destroy pair would > increase memory consumption in Mini-OS by about 5kB. Wondering whether > this is a xens

Re: [Xen-devel] [PATCH v3 01/17] libxl/arm: Factor out codes for generating DTB

2016-07-07 Thread Wei Liu
On Thu, Jul 07, 2016 at 12:12:48PM -0400, Boris Ostrovsky wrote: > On 07/07/2016 12:06 PM, Julien Grall wrote: > > > > > > On 07/07/16 16:48, Boris Ostrovsky wrote: > >> On 07/07/2016 11:41 AM, Wei Liu wrote: > >>> On Tue, Jul 05, 2016 at 11:12:31AM +0800, Shannon Zhao wrote: > From: Shannon Z

Re: [Xen-devel] [PATCH 16/18] arm/altp2m: Extended libxl to activate altp2m on ARM.

2016-07-07 Thread Wei Liu
On Mon, Jul 04, 2016 at 01:45:45PM +0200, Sergej Proskurin wrote: > The current implementation allows to set the parameter HVM_PARAM_ALTP2M. > This parameter allows further usage of altp2m on ARM. For this, we > define an additional altp2m field for PV domains as part of the > libxl_domain_type str

Re: [Xen-devel] default XSM policy for PCI passthrough for unlabeled resources.

2016-07-07 Thread anshul makkar
On 07/07/16 16:36, Daniel De Graaf wrote: On 07/06/2016 12:19 PM, anshul makkar wrote: On 06/07/16 16:59, Daniel De Graaf wrote: On 07/06/2016 11:34 AM, anshul makkar wrote: Hi, It allows the resource to be added and removed by the source domain to target domain, but its use by target domain

Re: [Xen-devel] [PATCH v3 06/17] libxl/arm: Estimate the size of ACPI tables

2016-07-07 Thread Julien Grall
Hi Wei, On 07/07/16 17:07, Wei Liu wrote: On Tue, Jul 05, 2016 at 11:12:36AM +0800, Shannon Zhao wrote: int libxl__prepare_acpi(libxl__gc *gc, libxl_domain_build_info *info, libxl__domain_build_state *state, struct xc_dom_image *dom) {

Re: [Xen-devel] [PATCH 13/19] xen: credit2: make the code less experimental

2016-07-07 Thread Dario Faggioli
On Thu, 2016-07-07 at 16:17 +0100, George Dunlap wrote: > On Sat, Jun 18, 2016 at 12:12 AM, Dario Faggioli > wrote: > > @@ -680,8 +677,8 @@ __update_svc_load(const struct scheduler *ops, > >  delta = now - svc->load_last_update; > >  if ( unlikely(delta < 0) ) > >  { > > - 

Re: [Xen-devel] [PATCH v3 11/17] libxl/arm: Construct ACPI MADT table

2016-07-07 Thread Julien Grall
Hi Wei, On 07/07/16 17:11, Wei Liu wrote: On Tue, Jul 05, 2016 at 11:12:41AM +0800, Shannon Zhao wrote: From: Shannon Zhao According to the GIC version, construct the MADT table. Signed-off-by: Shannon Zhao --- tools/libxl/libxl_arm_acpi.c | 83

Re: [Xen-devel] [PATCH v1 19/20] libxl/pvhv2: Include APIC page in MMIO hole for PVHv2 guests

2016-07-07 Thread Wei Liu
On Tue, Jul 05, 2016 at 03:05:18PM -0400, Boris Ostrovsky wrote: > Signed-off-by: Boris Ostrovsky > --- > > New patch > > tools/libxl/Makefile|2 ++ > tools/libxl/libxl_dom.c | 22 ++ > 2 files changed, 20 insertions(+), 4 deletions(-) > > diff --git a/tools/libxl

Re: [Xen-devel] [PATCH 19/19] xen: credit2: use cpumask_first instead of cpumask_any when choosing cpu

2016-07-07 Thread Dario Faggioli
On Tue, 2016-06-21 at 11:42 +0100, David Vrabel wrote: > On 18/06/16 00:13, Dario Faggioli wrote: > > > > because it is cheaper, and there is no much point in > > randomizing which cpu gets selected anyway, as such > > choice will be overridden shortly after, in runq_tickle(). > > > > If we reall

Re: [Xen-devel] [PATCH v3 17/17] libxl/arm: Initialize domain param HVM_PARAM_CALLBACK_IRQ

2016-07-07 Thread Julien Grall
Hi Wei, On 07/07/16 17:15, Wei Liu wrote: On Tue, Jul 05, 2016 at 11:12:47AM +0800, Shannon Zhao wrote: From: Shannon Zhao The guest kernel will get the event channel interrupt information via domain param HVM_PARAM_CALLBACK_IRQ. Initialize it here. Signed-off-by: Shannon Zhao --- tools/l

Re: [Xen-devel] [PATCH v1 02/20] acpi/hvmloader: Move acpi_info initialization out of ACPI code

2016-07-07 Thread Ian Jackson
Boris Ostrovsky writes ("[PATCH v1 02/20] acpi/hvmloader: Move acpi_info initialization out of ACPI code"): > acpi_info can be initialized by hvmloader itself. Now ACPI code > doesn't need to use hvmloader-private variables/routines such as > uart_exists(), lpt_exists() etc. ... > Signed-off-by: B

Re: [Xen-devel] [PATCH v1 19/20] libxl/pvhv2: Include APIC page in MMIO hole for PVHv2 guests

2016-07-07 Thread Boris Ostrovsky
On 07/07/2016 12:47 PM, Wei Liu wrote: > On Tue, Jul 05, 2016 at 03:05:18PM -0400, Boris Ostrovsky wrote: >> >> @@ -1006,10 +1009,21 @@ int libxl__build_hvm(libxl__gc *gc, uint32_t domid, >> dom->target_pages = mem_size >> XC_PAGE_SHIFT; >> if (dom->mmio_size == 0 && device_model) >

Re: [Xen-devel] [PATCH v1 02/20] acpi/hvmloader: Move acpi_info initialization out of ACPI code

2016-07-07 Thread Boris Ostrovsky
On 07/07/2016 12:58 PM, Ian Jackson wrote: > > There are two serious problems with this. > > 1. You have dropped the copyright attribution to Intel and Xensource. > > 2. You have changed the licence to BSD-style, even though the original >was GPLv2-only. I meant this to be a GPLv2 license. An

[Xen-devel] [xen-unstable baseline-only test] 66529: trouble: blocked/broken/pass

2016-07-07 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 66529 xen-unstable running [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/66529/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-rumpus

[Xen-devel] [qemu-mainline baseline-only test] 66530: trouble: blocked/broken/pass

2016-07-07 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 66530 qemu-mainline running [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/66530/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-amd64-libvirt

Re: [Xen-devel] [PATCH v1 02/20] acpi/hvmloader: Move acpi_info initialization out of ACPI code

2016-07-07 Thread Wei Liu
On Thu, Jul 07, 2016 at 01:09:28PM -0400, Boris Ostrovsky wrote: > On 07/07/2016 12:58 PM, Ian Jackson wrote: > > > > There are two serious problems with this. > > > > 1. You have dropped the copyright attribution to Intel and Xensource. > > > > 2. You have changed the licence to BSD-style, even th

Re: [Xen-devel] [PATCH v1 19/20] libxl/pvhv2: Include APIC page in MMIO hole for PVHv2 guests

2016-07-07 Thread Wei Liu
On Thu, Jul 07, 2016 at 01:02:12PM -0400, Boris Ostrovsky wrote: > On 07/07/2016 12:47 PM, Wei Liu wrote: > > On Tue, Jul 05, 2016 at 03:05:18PM -0400, Boris Ostrovsky wrote: > >> > >> @@ -1006,10 +1009,21 @@ int libxl__build_hvm(libxl__gc *gc, uint32_t domid, > >> dom->target_pages = me

Re: [Xen-devel] xenbits "official" repo for XTF (was Re: [PATCH 0/2] xtf: add launcher (+1 bugfix)

2016-07-07 Thread Lars Kurth
Alright, it appears we are at an impasse here. Not hosting the code on xenbits as suggested by David, seems to be the worst solution and will benefit no-one. > If we can't get consensus on something like this, the sensible thing > to do would be to vote. Our governance docs don't really cope wit

Re: [Xen-devel] [PATCH 11/18] arm/altp2m: Make flush_tlb_domain ready for altp2m.

2016-07-07 Thread Julien Grall
On 05/07/16 21:21, Sergej Proskurin wrote: On 07/05/2016 05:37 PM, Julien Grall wrote: When altp2m is active, the hostp2m is not used. All changes are applied directly to the individual altp2m views. As far as I know, the only situations, where a mapping is removed from a specific p2m view is

Re: [Xen-devel] [PATCH v1 02/20] acpi/hvmloader: Move acpi_info initialization out of ACPI code

2016-07-07 Thread Boris Ostrovsky
On 07/07/2016 01:15 PM, Wei Liu wrote: > On Thu, Jul 07, 2016 at 01:09:28PM -0400, Boris Ostrovsky wrote: >> On 07/07/2016 12:58 PM, Ian Jackson wrote: >>> There are two serious problems with this. >>> >>> 1. You have dropped the copyright attribution to Intel and Xensource. >>> >>> 2. You have cha

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

2016-07-07 Thread osstest service owner
flight 96756 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/96756/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-armhf-armhf-libvirt-xsm 6 xen-boot fail in 96739 pass in 96756 test-amd64-amd64-xl-rtds 6 xe

Re: [Xen-devel] [PATCH] xen/arm64: Use the correct TLBs flush instruction to nuke stage-2 TLBs

2016-07-07 Thread Stefano Stabellini
On Thu, 7 Jul 2016, Julien Grall wrote: > The function flush_tlb is called to invalidate the TLBs for the current > domain when the stage-2 page tables are modified. > > On ARMv8, the instruction "tlbi vmalle1is" (resp. "tlbi vmalle1") will > invalidate stage 1 entries associated to the current VM

[Xen-devel] [OSSTEST PATCH 2/2] DO NOT APPLY make-flight-substep-test

2016-07-07 Thread Ian Jackson
This is a simple test case for substeps. Signed-off-by: Ian Jackson CC: Wei Liu --- make-flight-substep-test | 14 ++ sg-run-job | 7 +++ 2 files changed, 21 insertions(+) create mode 100755 make-flight-substep-test diff --git a/make-flight-substep-test b/make-f

[Xen-devel] [OSSTEST PATCH 1/2] Executive: Support substeps

2016-07-07 Thread Ian Jackson
ts-* scripts can now create `substeps'. For the purposes of archaeology etc., a substep is just like a step. But it does correspond to a single specific ts-* invocation. Instead, it is started and finished explicitly as required. The whole job implementation code needs to explicitly assign a un

[Xen-devel] [PATCH v2] xen/arm64: Use the correct TLBs flush instruction to nuke stage-2 TLBs

2016-07-07 Thread Julien Grall
The function flush_tlb is called to invalidate the TLBs for the current domain when the stage-2 page tables are modified. On ARMv8, the instruction "tlbi vmalle1is" (resp. "tlbi vmalle1") will invalidate stage 1 entries associated to the current VMID (see D4-1811 in ARM DDI 0487A.j). Given that a

Re: [Xen-devel] [PATCH v2] xen/arm64: Use the correct TLBs flush instruction to nuke stage-2 TLBs

2016-07-07 Thread Stefano Stabellini
On Thu, 7 Jul 2016, Julien Grall wrote: > The function flush_tlb is called to invalidate the TLBs for the current > domain when the stage-2 page tables are modified. > > On ARMv8, the instruction "tlbi vmalle1is" (resp. "tlbi vmalle1") will > invalidate stage 1 entries associated to the current VM

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

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

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

2016-07-07 Thread osstest service owner
flight 96767 seabios real [real] http://logs.test-lab.xenproject.org/osstest/logs/96767/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-stop fail like 96501 Tests which did not succeed, but a

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

2016-07-07 Thread osstest service owner
flight 96762 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/96762/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemuu-ovmf-amd64 17 guest-start/debianhvm.repeat fail REGR. vs. 94748 test-amd64-amd64-

Re: [Xen-devel] [PATCH v3 07/17] libxl/arm: Construct ACPI RSDP table

2016-07-07 Thread Boris Ostrovsky
On 07/04/2016 11:12 PM, Shannon Zhao wrote: > From: Shannon Zhao > > Construct ACPI RSDP table and add a helper to calculate the ACPI table > checksum. > > Signed-off-by: Shannon Zhao > --- > tools/libxl/libxl_arm_acpi.c | 35 +++ > 1 file changed, 35 insertions(+

Re: [Xen-devel] [PATCH v3 6/8] x86/vm-event: minor ASSERT fix, add 'unlikely'

2016-07-07 Thread Tamas K Lengyel
On Wed, Jul 6, 2016 at 9:54 AM, Corneliu ZUZU wrote: > Minor fixes: > > * vm_event_register_write_resume: ASSERT on non-NULL v->arch.vm_event instead > of > &v->arch.vm_event->write_data. > * add 'unlikely' in if > > Signed-off-by: Corneliu ZUZU Acked-by: Tamas K Lengyel ___

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

2016-07-07 Thread osstest service owner
flight 96765 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/96765/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl 15 guest-start/debian.repeat fail REGR. vs. 96732 Regressions which a

[Xen-devel] [PATCH] vmx/monitor: CPUID events

2016-07-07 Thread Tamas K Lengyel
This patch implements sending notification to a monitor subscriber when an x86/vmx guest executes the CPUID instruction. Signed-off-by: Tamas K Lengyel --- Cc: Ian Jackson Cc: Wei Liu Cc: Razvan Cojocaru Cc: Jan Beulich Cc: Andrew Cooper Cc: Jun Nakajima Cc: Kevin Tian --- tools/libxc/inc

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-07 Thread Michael Turquette
Quoting Dirk Behme (2016-07-07 00:32:34) > Hi Michael, > > On 06.07.2016 22:42, Michael Turquette wrote: > > Hi Julien, > > > > Quoting Julien Grall (2016-07-06 06:10:52) > >> On 06/07/16 02:34, Michael Turquette wrote: > >>> Hi! > >> > >> Hello Michael, > >> > >>> Quoting Dirk Behme (2016-06-30 0

Re: [Xen-devel] [PATCH v3 07/17] libxl/arm: Construct ACPI RSDP table

2016-07-07 Thread Shannon Zhao
On 2016/7/8 6:43, Boris Ostrovsky wrote: >> +calculate_checksum(rsdp, offsetof(struct acpi_table_rsdp, checksum), >> > + acpitables[RSDP].size); > Should this be extended_checksum? checksum is for ACPI v1 (and thus for > smaller size structure, with only RSDT pointer). A

Re: [Xen-devel] [PATCH v3 17/17] libxl/arm: Initialize domain param HVM_PARAM_CALLBACK_IRQ

2016-07-07 Thread Shannon Zhao
On 2016/7/8 0:57, Julien Grall wrote: > On 07/07/16 17:15, Wei Liu wrote: >> On Tue, Jul 05, 2016 at 11:12:47AM +0800, Shannon Zhao wrote: >>> From: Shannon Zhao >>> >>> The guest kernel will get the event channel interrupt information via >>> domain param HVM_PARAM_CALLBACK_IRQ. Initialize it h

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-07 Thread Dirk Behme
On 08.07.2016 04:50, Michael Turquette wrote: Quoting Dirk Behme (2016-07-07 00:32:34) Hi Michael, On 06.07.2016 22:42, Michael Turquette wrote: Hi Julien, Quoting Julien Grall (2016-07-06 06:10:52) On 06/07/16 02:34, Michael Turquette wrote: Hi! Hello Michael, Quoting Dirk Behme (2016-

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

2016-07-07 Thread osstest service owner
flight 96769 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/96769/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): build-amd64-rumpuserxen 6 xen-buildfail like 96756 build-i386-rumpuserxen

[Xen-devel] [PATCH v15 0/3] VT-d Device-TLB flush issue

2016-07-07 Thread Jan Beulich
From: Quan Xu these patches fix current timeout concern and also allow limited ATS support. these patches are the rest ones: 1. move the domain crash logic up to the generic IOMMU layer 2. If Device-TLB flush timed out, we hide the target ATS device immediately. By hiding the device, we make

[Xen-devel] [PATCH v15 1/3] IOMMU/ATS: use a struct pci_dev * instead of SBDF

2016-07-07 Thread Jan Beulich
From: Quan Xu Do away with struct pci_ats_dev; integrate the few bits of information in struct pci_dev (and as a result drop get_ats_device() altogether). Hook ATS devices onto a linked list off of each IOMMU instead of on a global one. Signed-off-by: Quan Xu Signed-off-by: Jan Beulich --- v1

[Xen-devel] [PATCH v15 2/3] IOMMU: add domain crash logic

2016-07-07 Thread Jan Beulich
From: Quan Xu Add domain crash logic to the generic IOMMU layer to benefit all platforms. No spamming of the log can occur. For DomU, we avoid logging any message for already dying domains. For Dom0, that'll still be more verbose than we'd really like, but it at least wouldn't outright flood the

[Xen-devel] [PATCH v15 3/3] VT-d: fix Device-TLB flush timeout issue

2016-07-07 Thread Jan Beulich
From: Quan Xu If Device-TLB flush timed out, we hide the target ATS device immediately. By hiding the device, we make sure it can't be assigned to any domain any longer (see device_assigned). Signed-off-by: Quan Xu Signed-off-by: Jan Beulich --- v15: Re-base on heavily changed patch 1. Call di

Re: [Xen-devel] [PATCH v2] xen/arm: register clocks used by the hypervisor

2016-07-07 Thread Dirk Behme
Hi, On 06.07.2016 03:34, Michael Turquette wrote: Hi! Quoting Dirk Behme (2016-06-30 03:32:32) Some clocks might be used by the Xen hypervisor and not by the Linux kernel. If these are not registered by the Linux kernel, they might be disabled by clk_disable_unused() as the kernel doesn't know

Re: [Xen-devel] [PATCH v15 0/3] VT-d Device-TLB flush issue

2016-07-07 Thread Jan Beulich
>>> On 08.07.16 at 08:39, wrote: > From: Quan Xu > > these patches fix current timeout concern and also allow limited ATS > support. > > these patches are the rest ones: > 1. move the domain crash logic up to the generic IOMMU layer > > 2. If Device-TLB flush timed out, we hide the target ATS

<    1   2   3