Re: [Xen-devel] HEADS UP: Imported Xen 4.7 and blkback changes

2016-06-09 Thread Roger Pau Monné
On Wed, Jun 08, 2016 at 10:18:09PM +, Marcin Cieslak wrote: > On Fri, 3 Jun 2016, Roger Pau Monné wrote: > > > Hello, > > > > First of all, this message is only relevant to those that use FreeBSD as > > Dom0 (host), not as a DomU (guest), so don't panic. > > > > I've imported the latest Xen

Re: [Xen-devel] HEADS UP: Imported Xen 4.7 and blkback changes - domU respawning on_crash

2016-06-09 Thread Roger Pau Monné
On Wed, Jun 08, 2016 at 10:35:22PM +, Marcin Cieslak wrote: > On Wed, 8 Jun 2016, Marcin Cieslak wrote: > > > On Fri, 3 Jun 2016, Roger Pau Monné wrote: > > > > > Hello, > > > > > > First of all, this message is only relevant to those that use FreeBSD as > > > Dom0 (host), not as a DomU (gu

Re: [Xen-devel] HEADS UP: Imported Xen 4.7: no blkback

2016-06-09 Thread Roger Pau Monné
On Thu, Jun 09, 2016 at 12:16:59AM +, Marcin Cieslak wrote: > On Fri, 3 Jun 2016, Roger Pau Monné wrote: > > > One of the more relevant changes in 4.7 regarding FreeBSD is the support > > for > > block hotplug scripts. This means that we now have the option to use > > backends different tha

Re: [Xen-devel] BUG: NOW() seems to (sometimes) go backwards!

2016-06-09 Thread Jan Beulich
>>> On 08.06.16 at 22:36, wrote: > On 06/08/2016 02:43 PM, Dario Faggioli wrote: >> The code you're referring to should be this: >> >> /* If we have constant-rate TSCs then scale factor can be shared. */ >> if ( boot_cpu_has(X86_FEATURE_CONSTANT_TSC) ) >> { >> /* If TSCs are n

Re: [Xen-devel] [PATCH RFC 19/20] acpi: Set HW_REDUCED_ACPI in FADT if IOAPIC is not supported

2016-06-09 Thread Roger Pau Monné
On Wed, Jun 08, 2016 at 06:04:01PM -0400, Boris Ostrovsky wrote: > On 06/07/2016 11:41 AM, Jan Beulich wrote: > On 07.06.16 at 17:17, wrote: > >> On 06/07/2016 10:12 AM, Jan Beulich wrote: > >> On 07.06.16 at 16:02, wrote: > On 06/07/2016 02:06 AM, Jan Beulich wrote: > On 0

Re: [Xen-devel] [PATCH RFC 19/20] acpi: Set HW_REDUCED_ACPI in FADT if IOAPIC is not supported

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 10:13, wrote: > On Wed, Jun 08, 2016 at 06:04:01PM -0400, Boris Ostrovsky wrote: >> On 06/07/2016 11:41 AM, Jan Beulich wrote: >> On 07.06.16 at 17:17, wrote: >> >> On 06/07/2016 10:12 AM, Jan Beulich wrote: >> >> On 07.06.16 at 16:02, wrote: >> On 06/07/2016 02:

Re: [Xen-devel] HEADS UP: Imported Xen 4.7: no blkback

2016-06-09 Thread Wei Liu
On Thu, Jun 09, 2016 at 10:03:43AM +0200, Roger Pau Monné wrote: > On Thu, Jun 09, 2016 at 12:16:59AM +, Marcin Cieslak wrote: > > On Fri, 3 Jun 2016, Roger Pau Monné wrote: > > > > > One of the more relevant changes in 4.7 regarding FreeBSD is the support > > > for > > > block hotplug scrip

[Xen-devel] [qemu-upstream-4.5-testing test] 95429: regressions - trouble: broken/fail/pass

2016-06-09 Thread osstest service owner
flight 95429 qemu-upstream-4.5-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/95429/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt-qcow2 3 host-install(3) broken REGR. vs. 93922 test-ar

[Xen-devel] [xen-unstable test] 95414: trouble: blocked/broken/fail/pass

2016-06-09 Thread osstest service owner
flight 95414 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/95414/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemut-stubdom-debianhvm-amd64-xsm 3 host-install(3) broken

[Xen-devel] [PATCH] xen/xsm: Annotate xsm_initcall() data as const

2016-06-09 Thread Andrew Cooper
Additionally, link it adjacently to the other constant init data. Signed-off-by: Andrew Cooper --- CC: Daniel De Graaf CC: Jan Beulich CC: Stefano Stabellini CC: Julien Grall --- xen/arch/arm/xen.lds.S | 9 - xen/arch/x86/xen.lds.S | 9 - xen/include/xsm/xsm.h | 2 +- 3 file

[Xen-devel] [PATCH] x86/boot: copy/clear sections more efficiently

2016-06-09 Thread Andrew Cooper
Both the trampoline copy and BSS initialise can be performed more efficiently by using 4-byte variants of the string operations. On Intel systems with ERMSB (efficient rep movsb), this is no practical difference. On all other systems, this is 4 times more efficient. Signed-off-by: Andrew Cooper

[Xen-devel] [PATCH] xen/init: Annotate all command line parameter infrastructure as const

2016-06-09 Thread Andrew Cooper
There is no reason for any of it to be modified. Additionally, link .init.setup beside the other constant .init data. While editing this area, correct the types used in the extern declarations for __setup_start and __setup_end to match the types the linker actually produces. No functional change

[Xen-devel] [PATCH] xen/vsprintf: Avoid returning NULL from number()

2016-06-09 Thread Andrew Cooper
In practice this is an unused codepath, as every caller of number() passes an explicit base of 8, 10 or 16. For all other uses, number() returns a pointer between the str and end parameters, as do the other similar helper functions. However, the fact that there is a NULL return path causes Coveri

Re: [Xen-devel] [PATCH] xen/init: Annotate all command line parameter infrastructure as const

2016-06-09 Thread Andrew Cooper
CC'ing Stefano's correct address. (This patch pre-dates quite a lot of recent changes). On 09/06/16 10:58, Andrew Cooper wrote: > There is no reason for any of it to be modified. Additionally, link > .init.setup beside the other constant .init data. > > While editing this area, correct the types

[Xen-devel] [PATCH 0/2] Validate host VMX MSRs before accessing them

2016-06-09 Thread Euan Harris
nvmx_msr_read_intercept() does not check the prerequisites before accessing MSR_IA32_VMX_PROCBASED_CTLS2, MSR_IA32_VMX_EPT_VPID_CAP, MSR_IA32_VMX_VMFUNC on the host. Accessing these MSRs from a nested VMX guest running on a host which does not support them will cause Xen to crash with a GPF. Eua

[Xen-devel] [PATCH 1/2] nested vmx: Fix comment typos in nvmx_msr_read_intercept()

2016-06-09 Thread Euan Harris
Signed-off-by: Euan Harris --- xen/arch/x86/hvm/vmx/vvmx.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/xen/arch/x86/hvm/vmx/vvmx.c b/xen/arch/x86/hvm/vmx/vvmx.c index faa8b69..d9493ff 100644 --- a/xen/arch/x86/hvm/vmx/vvmx.c +++ b/xen/arch/x86/hvm/vmx/vvmx.c @

[Xen-devel] [PATCH 2/2] nested vmx: Validate host VMX MSRs before accessing them

2016-06-09 Thread Euan Harris
Some VMX MSRs may not exist on certain processor models, or may be disabled because of configuration settings. It is only safe to access these MSRs if configuration flags in other MSRs are set. These prerequisites are listed in the Intel 64 and IA-32 Architectures Software Developer’s Manual, Vo

Re: [Xen-devel] [PATCH V8 2/3] drivers/pl011: Use combination of UARTRIS and UARTMSC instead of UARTMIS

2016-06-09 Thread Julien Grall
Hello Shanker, On 08/06/16 14:28, Shanker Donthineni wrote: The Masked interrupt status register (UARTMIS) is not described in ARM SBSA 2.x document. Anding of two registers UARTMSC and UARTRIS values gives the same information as register UARTMIS. UARTRIS, UARTMSC and UARTMIS definitions are f

Re: [Xen-devel] [PATCH V8 3/3] arm/acpi: Add Server Base System Architecture UART support

2016-06-09 Thread Julien Grall
Hello Shanker, On 08/06/16 14:28, Shanker Donthineni wrote: The ARM Server Base System Architecture describes a generic UART interface. It doesn't support clock control registers, modem control, DMA and hardware flow control features. So, extend the driver probe() to handle SBSA interface and sk

Re: [Xen-devel] BUG: NOW() seems to (sometimes) go backwards!

2016-06-09 Thread George Dunlap
On 07/06/16 16:54, Dario Faggioli wrote: > Hi, > > I've been fighting with this during the past few days, while testing > and benchmarking some of the modification I'm doing to Credit2. > > In summary, what I see is NOW() going backwards. The issues shows up > when I call NOW() on different pCPUs

Re: [Xen-devel] BUG: NOW() seems to (sometimes) go backwards!

2016-06-09 Thread Joao Martins
Yet when the scaling values get set only once at boot, monotonic (cross-CPU) TSC means monotonic (cross-CPU) returns from NOW(). >>> Yep. And at this point, this is what needs to be verified, I guess... >> I think get_s_time_fixed doesn't guarantee monotonicity across CPUs being it >

Re: [Xen-devel] [PATCH v7 03/11] IOMMU: propagate IOMMU Device-TLB flush error up to IOMMU unmapping (top level ones)

2016-06-09 Thread Julien Grall
Hello, On 08/06/16 09:58, Xu, Quan wrote: From: Quan Xu Signed-off-by: Quan Xu Acked-by: Kevin Tian CC: Stefano Stabellini CC: Julien Grall For the ARM bits: Acked-by: Julien Grall Regards, -- Julien Grall ___ Xen-devel mailing list Xen-d

Re: [Xen-devel] [PATCH v7 04/11] IOMMU: propagate IOMMU Device-TLB flush error up to IOMMU mapping (top level ones)

2016-06-09 Thread Julien Grall
Hello, On 08/06/16 09:58, Xu, Quan wrote: From: Quan Xu Signed-off-by: Quan Xu Acked-by: Kevin Tian CC: Suravee Suthikulpanit CC: Stefano Stabellini CC: Julien Grall CC: Kevin Tian CC: Feng Wu CC: Jan Beulich v7: drop the amd_iommu_hwdom_init() fix, which has been added in patch

Re: [Xen-devel] [PATCH] xen: typo: use ' as apostrophe in grant_table.h

2016-06-09 Thread George Dunlap
On Wed, Feb 24, 2016 at 5:19 PM, Dario Faggioli wrote: > If grep 2.23 is installed, build fails like this: > ... > mkdir -p compat > grep -v 'DEFINE_XEN_GUEST_HANDLE(long)' public/grant_table.h | \ > python /home/SOURCES/xen/xen/xen.git/xen/tools/compat-build-source.py > >compat/grant_table.c.new

Re: [Xen-devel] BUG: NOW() seems to (sometimes) go backwards!

2016-06-09 Thread Joao Martins
On 06/09/2016 11:24 AM, Joao Martins wrote: > Yet when the scaling values get set only once at boot, monotonic > (cross-CPU) TSC means monotonic (cross-CPU) returns from NOW(). > Yep. And at this point, this is what needs to be verified, I guess... >>> I think get_s_time_fixed do

Re: [Xen-devel] [PATCH] x86/shadow: sh_pagetable_dying() cleanup

2016-06-09 Thread Tim Deegan
At 07:17 -0600 on 08 Jun (1465370220), Jan Beulich wrote: > Don't call shadow_hash_lookup() at all when get_gfn_query_unlocked() > didn't return a valid MFN. > > Also no need for local variables used only once, the more with scopes > much wider than their actual use. > > Signed-off-by: Jan Beulic

Re: [Xen-devel] [PATCH] tools: fix libxengnttab dependencies

2016-06-09 Thread Ian Jackson
Jan Beulich writes ("[PATCH] tools: fix libxengnttab dependencies"): > Without this some ld versions warn about not being able to find > libxentoollog.so.1 when linking libxenvchan. > > Signed-off-by: Jan Beulich Acked-by: Ian Jackson I think this is a 4.7 candidate. Wei ? Ian.

Re: [Xen-devel] [PATCH] tools: fix libxengnttab dependencies

2016-06-09 Thread Wei Liu
On Thu, Jun 09, 2016 at 12:00:13PM +0100, Ian Jackson wrote: > Jan Beulich writes ("[PATCH] tools: fix libxengnttab dependencies"): > > Without this some ld versions warn about not being able to find > > libxentoollog.so.1 when linking libxenvchan. > > > > Signed-off-by: Jan Beulich > > Acked-by

Re: [Xen-devel] [PATCH v7 05/11] IOMMU/MMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (top level ones)

2016-06-09 Thread Julien Grall
Hello, On 08/06/16 09:58, Xu, Quan wrote: From: Quan Xu Signed-off-by: Quan Xu Reviewed-by: Jan Beulich CC: Stefano Stabellini CC: Julien Grall CC: Jan Beulich CC: Kevin Tian --- xen/arch/arm/p2m.c | 4 +++- xen/common/memory.c | 12 ++-- xe

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

2016-06-09 Thread osstest service owner
flight 95418 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/95418/ 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] Xen 4.7 crash

2016-06-09 Thread Ian Jackson
Aaron Cornelius writes ("Re: [Xen-devel] Xen 4.7 crash"): > I am not that familiar with the xenstored code, but as far as I can tell > the grant mapping will be held by the xenstore until the xs_release() > function is called (which is not called by libxl, and I do not > explicitly call it in my

Re: [Xen-devel] [PATCH] public/errno: sort entries numerically

2016-06-09 Thread Andrew Cooper
On 08/06/16 14:11, Jan Beulich wrote: > Signed-off-by: Jan Beulich Oops - that appears to be my fault. I can count (most of the time). Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 1/2] x86/HVM: constify hvm_virtual_to_linear_addr()'s segment register parameter

2016-06-09 Thread Andrew Cooper
On 08/06/16 14:42, Jan Beulich wrote: > ... to clarify to callers that they don't need to fear the pointed to > data changing (which will be made use of subsequently). > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing lis

Re: [Xen-devel] BUG: NOW() seems to (sometimes) go backwards!

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 12:24, wrote: > Yet when the scaling values get set only once at boot, monotonic > (cross-CPU) TSC means monotonic (cross-CPU) returns from NOW(). > Yep. And at this point, this is what needs to be verified, I guess... >>> I think get_s_time_fixed doesn't guaran

Re: [Xen-devel] BUG: NOW() seems to (sometimes) go backwards!

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 13:25, wrote: On 09.06.16 at 12:24, wrote: >> Yet when the scaling values get set only once at boot, monotonic >> (cross-CPU) TSC means monotonic (cross-CPU) returns from NOW(). >> > Yep. And at this point, this is what needs to be verified, I guess...

Re: [Xen-devel] [PATCH 2/2] x86/HVM: re-order operations in hvm_ud_intercept()

2016-06-09 Thread Andrew Cooper
On 08/06/16 14:43, Jan Beulich wrote: > Don't fetch CS explicitly, leverage the fact that hvm_emulate_prepare() > already does (and that hvm_virtual_to_linear_addr() doesn't alter it). > > At once increase the length passed to hvm_virtual_to_linear_addr() by > one: There definitely needs to be at l

Re: [Xen-devel] [PATCH] x86/PV: drop pointless conditional from pv_cpuid()'s state leaf logic

2016-06-09 Thread Andrew Cooper
On 08/06/16 14:24, Jan Beulich wrote: > In the control/hardware domain case without it we simply re-read the > same value that was put into b near the top of the function. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing

Re: [Xen-devel] [PATCH] x86: drop hvm/iommu.h

2016-06-09 Thread Andrew Cooper
On 08/06/16 14:12, Jan Beulich wrote: > As a follow-up to commit af07377007 ("IOMMU/x86: per-domain control > structure is not HVM-specific"), fold hvm/iommu.h into iommu.h. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel maili

Re: [Xen-devel] [PATCH] xen: clean up AFLAGS management

2016-06-09 Thread Andrew Cooper
On 08/06/16 14:11, Jan Beulich wrote: > No need to force inclusion of xen/config.h - AFLAGS incorporates > CFLAGS, which already does this. > > Also no need to use nested $(filter-out ...) - one of them can deal > with any number of patterns. > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Coo

Re: [Xen-devel] [PATCH v7 05/11] IOMMU/MMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (top level ones)

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 13:12, wrote: > On 08/06/16 09:58, Xu, Quan wrote: >> From: Quan Xu >> >> Signed-off-by: Quan Xu >> Reviewed-by: Jan Beulich >> >> CC: Stefano Stabellini >> CC: Julien Grall >> CC: Jan Beulich >> CC: Kevin Tian >> --- >> xen/arch/arm/p2m.c | 4 +++- >>

Re: [Xen-devel] [PATCH 1/2] x86/HVM: latch linear->phys translation results

2016-06-09 Thread Andrew Cooper
On 08/06/16 14:09, Jan Beulich wrote: > ... to avoid re-doing the same translation later again (in a retry, for > example). This doesn't help very often according to my testing, but > it's pretty cheap to have, and will be of further use subsequently. > > Signed-off-by: Jan Beulich > > --- a/xen/a

[Xen-devel] [PATCH] x86/time: use correct (local) time stamp in constant-TSC calibration fast path

2016-06-09 Thread Jan Beulich
This looks like a copy and paste mistake in commit 1b6a99892d ("x86: Simpler time handling when TSC is constant across all power saving states"), responsible for occasional many-microsecond cross-CPU skew of what NOW() returns. Also improve the correlation between local TSC and stime stamps obtain

Re: [Xen-devel] [PATCH] x86/XSTATE: clarify XRSTOR() macro

2016-06-09 Thread Andrew Cooper
On 08/06/16 13:50, Jan Beulich wrote: > Make obvious that xcomp_bv is expected to be clear when we get here > with XSTATE_COMPACTION_ENABLED not set. > > Signed-off-by: Jan Beulich Reviewed-by Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.x

Re: [Xen-devel] [PATCH v7 05/11] IOMMU/MMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (top level ones)

2016-06-09 Thread Julien Grall
On 09/06/16 12:45, Jan Beulich wrote: On 09.06.16 at 13:12, wrote: On 08/06/16 09:58, Xu, Quan wrote: From: Quan Xu Signed-off-by: Quan Xu Reviewed-by: Jan Beulich CC: Stefano Stabellini CC: Julien Grall CC: Jan Beulich CC: Kevin Tian --- xen/arch/arm/p2m.c | 4

Re: [Xen-devel] [PATCH v7 05/11] IOMMU/MMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (top level ones)

2016-06-09 Thread Julien Grall
On 09/06/16 13:03, Julien Grall wrote: On 09/06/16 12:45, Jan Beulich wrote: On 09.06.16 at 13:12, wrote: On 08/06/16 09:58, Xu, Quan wrote: From: Quan Xu Signed-off-by: Quan Xu Reviewed-by: Jan Beulich CC: Stefano Stabellini CC: Julien Grall CC: Jan Beulich CC: Kevin Tian --- x

Re: [Xen-devel] [PATCH] x86/time: use correct (local) time stamp in constant-TSC calibration fast path

2016-06-09 Thread Andrew Cooper
On 09/06/16 13:01, Jan Beulich wrote: > This looks like a copy and paste mistake in commit 1b6a99892d ("x86: > Simpler time handling when TSC is constant across all power saving > states"), responsible for occasional many-microsecond cross-CPU skew of > what NOW() returns. > > Also improve the corr

Re: [Xen-devel] [PATCH] x86/time: use correct (local) time stamp in constant-TSC calibration fast path

2016-06-09 Thread Joao Martins
On 06/09/2016 01:01 PM, Jan Beulich wrote: > This looks like a copy and paste mistake in commit 1b6a99892d ("x86: > Simpler time handling when TSC is constant across all power saving > states"), responsible for occasional many-microsecond cross-CPU skew of > what NOW() returns. > > Also improve

Re: [Xen-devel] [PATCH] x86/time: use correct (local) time stamp in constant-TSC calibration fast path

2016-06-09 Thread Wei Liu
On Thu, Jun 09, 2016 at 06:01:18AM -0600, Jan Beulich wrote: > This looks like a copy and paste mistake in commit 1b6a99892d ("x86: > Simpler time handling when TSC is constant across all power saving > states"), responsible for occasional many-microsecond cross-CPU skew of > what NOW() returns. >

Re: [Xen-devel] [PATCH 1/2] x86/HVM: latch linear->phys translation results

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 13:54, wrote: > On 08/06/16 14:09, Jan Beulich wrote: >> ... to avoid re-doing the same translation later again (in a retry, for >> example). This doesn't help very often according to my testing, but >> it's pretty cheap to have, and will be of further use subsequently. >> >> Si

Re: [Xen-devel] [PATCH v7 05/11] IOMMU/MMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (top level ones)

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 14:03, wrote: > > On 09/06/16 12:45, Jan Beulich wrote: > On 09.06.16 at 13:12, wrote: >>> On 08/06/16 09:58, Xu, Quan wrote: From: Quan Xu Signed-off-by: Quan Xu Reviewed-by: Jan Beulich CC: Stefano Stabellini CC: Julien Grall

Re: [Xen-devel] [PATCH v7 05/11] IOMMU/MMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (top level ones)

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 14:08, wrote: > > On 09/06/16 13:03, Julien Grall wrote: >> On 09/06/16 12:45, Jan Beulich wrote: >> On 09.06.16 at 13:12, wrote: On 08/06/16 09:58, Xu, Quan wrote: > From: Quan Xu > > Signed-off-by: Quan Xu > Reviewed-by: Jan Beulich > > C

Re: [Xen-devel] [PATCH v7 05/11] IOMMU/MMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (top level ones)

2016-06-09 Thread Julien Grall
On 09/06/16 13:15, Jan Beulich wrote: On 09.06.16 at 14:08, wrote: On 09/06/16 13:03, Julien Grall wrote: On 09/06/16 12:45, Jan Beulich wrote: On 09.06.16 at 13:12, wrote: On 08/06/16 09:58, Xu, Quan wrote: From: Quan Xu Signed-off-by: Quan Xu Reviewed-by: Jan Beulich CC: Stefano

[Xen-devel] [PATCH] Run autogen.sh

2016-06-09 Thread Wei Liu
Signed-off-by: Wei Liu --- configure | 18 +- docs/configure| 18 +- stubdom/configure | 18 +- tools/configure | 18 +- 4 files changed, 36 insertions(+), 36 deletions(-) diff --git a/configure b/configure index 8b071

Re: [Xen-devel] [PATCH 2/2] x86/HVM: re-order operations in hvm_ud_intercept()

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 13:34, wrote: > On 08/06/16 14:43, Jan Beulich wrote: >> Don't fetch CS explicitly, leverage the fact that hvm_emulate_prepare() >> already does (and that hvm_virtual_to_linear_addr() doesn't alter it). >> >> At once increase the length passed to hvm_virtual_to_linear_addr() by

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

2016-06-09 Thread osstest service owner
flight 95433 qemu-upstream-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/95433/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-stop fail like 94048 test-amd64-i386-

Re: [Xen-devel] [PATCH v7 05/11] IOMMU/MMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (top level ones)

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 14:24, wrote: > On 09/06/16 13:15, Jan Beulich wrote: > On 09.06.16 at 14:08, wrote: >> >>> >>> On 09/06/16 13:03, Julien Grall wrote: On 09/06/16 12:45, Jan Beulich wrote: On 09.06.16 at 13:12, wrote: >> On 08/06/16 09:58, Xu, Quan wrote: >>> From: Q

Re: [Xen-devel] [PATCH] Run autogen.sh

2016-06-09 Thread Wei Liu
On Thu, Jun 09, 2016 at 01:28:05PM +0100, Wei Liu wrote: > Signed-off-by: Wei Liu > --- > configure | 18 +- > docs/configure| 18 +- > stubdom/configure | 18 +- > tools/configure | 18 +- > 4 files changed, 36 inserti

Re: [Xen-devel] [PATCH v7 05/11] IOMMU/MMU: propagate IOMMU Device-TLB flush error up to iommu_iotlb_flush{, _all} (top level ones)

2016-06-09 Thread Julien Grall
Hi Jan, On 09/06/16 13:32, Jan Beulich wrote: On 09.06.16 at 14:24, wrote: So the behavior of iommu_iotlb_flush is up to the IOMMU driver. Whilst the behavior of iommu_{,un}map_page are defined by the common code. I'm certainly up for moving the logic up to the generic IOMMU layer, if that's

Re: [Xen-devel] [PATCH] xen/init: Annotate all command line parameter infrastructure as const

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 11:58, wrote: > --- a/xen/arch/arm/xen.lds.S > +++ b/xen/arch/arm/xen.lds.S > @@ -135,6 +135,12 @@ SECTIONS > *(.init.rodata) > *(.init.rodata.rel) > *(.init.rodata.str*) > + > + . = ALIGN(POINTER_ALIGN); > + __setup_start = .; > + *(.in

Re: [Xen-devel] [PATCH] xen/xsm: Annotate xsm_initcall() data as const

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 11:58, wrote: > --- a/xen/include/xsm/xsm.h > +++ b/xen/include/xsm/xsm.h > @@ -51,7 +51,7 @@ typedef void (*xsm_initcall_t)(void); > extern xsm_initcall_t __xsm_initcall_start[], __xsm_initcall_end[]; > > #define xsm_initcall(fn) \ > -static xsm_initcall_t __initcall_##

Re: [Xen-devel] [PATCH] x86/boot: copy/clear sections more efficiently

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 11:58, wrote: > --- a/xen/arch/x86/boot/head.S > +++ b/xen/arch/x86/boot/head.S > @@ -128,7 +128,8 @@ __start: > mov $sym_phys(__bss_end),%ecx > sub %edi,%ecx > xor %eax,%eax > -rep stosb > +shr $2,%ecx > +r

Re: [Xen-devel] [PATCH] xen/vsprintf: Avoid returning NULL from number()

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 12:00, wrote: > In practice this is an unused codepath, as every caller of number() passes an > explicit base of 8, 10 or 16. For all other uses, number() returns a > pointer > between the str and end parameters, as do the other similar helper > functions. > > However, the f

Re: [Xen-devel] [PATCH 2/2] nested vmx: Validate host VMX MSRs before accessing them

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 12:14, wrote: > --- a/xen/arch/x86/hvm/vmx/vvmx.c > +++ b/xen/arch/x86/hvm/vmx/vvmx.c > @@ -1820,11 +1820,20 @@ int nvmx_msr_read_intercept(unsigned int msr, u64 > *msr_content) > return 0; > > /* > - * Those MSRs are available only when bit 55 of > - *

[Xen-devel] [PATCH 06/11] libxc: honour XEN_LOG_DIR in xc_dom_core.c

2016-06-09 Thread Wei Liu
Signed-off-by: Wei Liu --- Cc: Ian Jackson --- tools/libxc/xc_dom_core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/libxc/xc_dom_core.c b/tools/libxc/xc_dom_core.c index 55c779d..ebada89 100644 --- a/tools/libxc/xc_dom_core.c +++ b/tools/libxc/xc_dom_core.c @@ -3

[Xen-devel] [PATCH 01/11] Config.mk: add XEN_LOG_DIR to BUILD_MAKE_VARS

2016-06-09 Thread Wei Liu
... so that it can be turned into shell environment and exported to header files. 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: Wei Liu --- Config.mk | 2 +- 1 file change

Re: [Xen-devel] [PATCH] x86/time: use correct (local) time stamp in constant-TSC calibration fast path

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 14:11, wrote: > On 06/09/2016 01:01 PM, Jan Beulich wrote: >> This looks like a copy and paste mistake in commit 1b6a99892d ("x86: >> Simpler time handling when TSC is constant across all power saving >> states"), responsible for occasional many-microsecond cross-CPU skew of >>

[Xen-devel] [PATCH 00/11] Honour XEN_{LOG, RUN}_DIR in various places

2016-06-09 Thread Wei Liu
Wei Liu (11): Config.mk: add XEN_LOG_DIR to BUILD_MAKE_VARS docs: use XEN_LOG_DIR in log file location tools: install XEN_{LOG,RUN}_DIR xenconsoled: honour XEN_LOG_DIR and remove hard-coded path xenbackendd: honour XEN_{RUN,LOG}_DIR libxc: honour XEN_LOG_DIR in xc_dom_core.c hotplug/F

[Xen-devel] [PATCH 08/11] hotplug/Linux: honour XEN_LOG_DIR

2016-06-09 Thread Wei Liu
Signed-off-by: Wei Liu --- Cc: Ian Jackson Cc: Roger Pau Monné --- tools/hotplug/Linux/init.d/sysconfig.xencommons.in | 2 +- tools/hotplug/Linux/init.d/xencommons.in | 2 +- tools/hotplug/Linux/xen-hotplug-common.sh.in | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) d

[Xen-devel] [PATCH 09/11] hotplug/NetBSD: honour XEN_{LOG, RUN}_DIR

2016-06-09 Thread Wei Liu
Signed-off-by: Wei Liu --- Cc: Ian Jackson Cc: Roger Pau Monné --- tools/hotplug/NetBSD/rc.d/xencommons.in | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/tools/hotplug/NetBSD/rc.d/xencommons.in b/tools/hotplug/NetBSD/rc.d/xencommons.in index d7552cd..886a2e2

[Xen-devel] [PATCH 02/11] docs: use XEN_LOG_DIR in log file location

2016-06-09 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: Wei Liu --- docs/misc/hvm-emulated-unplug.markdown | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/mi

[Xen-devel] [PATCH 04/11] xenconsoled: honour XEN_LOG_DIR and remove hard-coded path

2016-06-09 Thread Wei Liu
Make a _paths.h for xenconsoled as well and use that to generate a default path for log file directory. Signed-off-by: Wei Liu --- Cc: Ian Jackson --- .gitignore | 1 + tools/console/Makefile | 5 + tools/console/daemon/main.c | 3 ++- 3 files changed, 8 insertions(+),

[Xen-devel] [PATCH 05/11] xenbackendd: honour XEN_{RUN,LOG}_DIR

2016-06-09 Thread Wei Liu
Also added a gitignore entry for xenbackendd binary while I was there. Signed-off-by: Wei Liu --- Cc: Ian Jackson --- .gitignore | 2 ++ tools/xenbackendd/Makefile | 6 +- tools/xenbackendd/xenbackendd.c | 6 -- 3 files changed, 11 insertions(+), 3 deletions(-)

[Xen-devel] [PATCH 03/11] tools: install XEN_{LOG,RUN}_DIR

2016-06-09 Thread Wei Liu
Signed-off-by: Wei Liu --- Cc: Ian Jackson --- tools/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/Makefile b/tools/Makefile index 6440dec..d43a229 100644 --- a/tools/Makefile +++ b/tools/Makefile @@ -61,7 +61,8 @@ build all: subdirs-all .PHONY: install i

[Xen-devel] [PATCH 07/11] hotplug/FreeBSD: honour XEN_{LOG, RUN}_DIR

2016-06-09 Thread Wei Liu
Signed-off-by: Wei Liu --- Cc: Ian Jackson Cc: Roger Pau Monné --- tools/hotplug/FreeBSD/rc.d/xencommons.in | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tools/hotplug/FreeBSD/rc.d/xencommons.in b/tools/hotplug/FreeBSD/rc.d/xencommons.in index d453c6b..2f9e1ec

Re: [Xen-devel] Question about device tree block

2016-06-09 Thread Julien Grall
On 09/06/16 00:40, 조현권 wrote: Hi i have a question about device tree block area Hello, In the function setup_mm xen copy original device tree block to the end of xen heap space Original device tree block area seems useless during domain0 creation but xen discard it with kernel modules afte

Re: [Xen-devel] [PATCH] Run autogen.sh

2016-06-09 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH] Run autogen.sh"): > On Thu, Jun 09, 2016 at 01:28:05PM +0100, Wei Liu wrote: > > Signed-off-by: Wei Liu > > --- > > configure | 18 +- > > docs/configure| 18 +- > > stubdom/configure | 18 +- > > tools/confi

Re: [Xen-devel] [PATCH] Run autogen.sh

2016-06-09 Thread Andrew Cooper
On 09/06/16 14:02, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH] Run autogen.sh"): >> On Thu, Jun 09, 2016 at 01:28:05PM +0100, Wei Liu wrote: >>> Signed-off-by: Wei Liu >>> --- >>> configure | 18 +- >>> docs/configure| 18 +- >>> stubdom/configure

Re: [Xen-devel] [PATCH 03/11] tools: install XEN_{LOG,RUN}_DIR

2016-06-09 Thread Andrew Cooper
On 09/06/16 13:57, Wei Liu wrote: > Signed-off-by: Wei Liu > --- > Cc: Ian Jackson > --- > tools/Makefile | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tools/Makefile b/tools/Makefile > index 6440dec..d43a229 100644 > --- a/tools/Makefile > +++ b/tools/Makefile > @@

Re: [Xen-devel] [PATCH 00/11] Honour XEN_{LOG, RUN}_DIR in various places

2016-06-09 Thread Andrew Cooper
On 09/06/16 13:57, Wei Liu wrote: > Wei Liu (11): > Config.mk: add XEN_LOG_DIR to BUILD_MAKE_VARS > docs: use XEN_LOG_DIR in log file location > tools: install XEN_{LOG,RUN}_DIR > xenconsoled: honour XEN_LOG_DIR and remove hard-coded path > xenbackendd: honour XEN_{RUN,LOG}_DIR > libxc:

Re: [Xen-devel] [PATCH 2/2] nested vmx: Validate host VMX MSRs before accessing them

2016-06-09 Thread Euan Harris
On Thu, Jun 09, 2016 at 06:47:55AM -0600, Jan Beulich wrote: > > /* > > - * Those MSRs are available only when bit 55 of > > - * MSR_IA32_VMX_BASIC is set. > > + * These MSRs are only available when flags in other MSRs are set. > > + * These prerequisites are listed in the Inte

[Xen-devel] [PATCH 11/11] oxenstored: honour XEN_{LOG, RUN}_DIR in oxenstored.conf

2016-06-09 Thread Wei Liu
Generate oxenstored.conf with configure. This involves modifying tools/configure.ac and rerun autogen.sh. Signed-off-by: Wei Liu --- Cc: Ian Jackson Cc: David Scott There are two hard-coded paths in logging.ml, but I'm not sure if generate an ocaml _Path module is the right thing to do. --- .

Re: [Xen-devel] [PATCH 0/5] x86: mwait-idle updates

2016-06-09 Thread Andrew Cooper
On 08/06/16 14:47, Jan Beulich wrote: > The first three are Linux imports, while the 4th is an adjustment genuine > to our clone, and the 5th is an adjustment #3, the Linux equivalent of > which I didn't get any feedback on so far. > > 1: add SKX support > 2: add KBL support > 3: add BXT support T

Re: [Xen-devel] [PATCH for-4.7] Revert "libxl: No emulated disk driver for xvdX disk"

2016-06-09 Thread Olaf Hering
On Wed, Jun 08, Konrad Rzeszutek Wilk wrote: > On Wed, Jun 08, 2016 at 06:17:55PM +0200, Olaf Hering wrote: > > On Wed, Jun 08, George Dunlap wrote: > > > > > CC'ing Olaf and Konrad for their information. :-) > > > > I'm fine with the revert. Another case is pvgrub2, which also uses the 'dev' p

[Xen-devel] [PATCH 10/11] libxl: honour XEN_LOG_DIR

2016-06-09 Thread Wei Liu
Signed-off-by: Wei Liu --- Cc: Ian Jackson --- tools/libxl/libxl_utils.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/tools/libxl/libxl_utils.c b/tools/libxl/libxl_utils.c index 4ca6bcb..6108d4b 100644 --- a/tools/libxl/libxl_utils.c +++ b/tools/libxl/libxl_u

Re: [Xen-devel] [PATCH 0/5] x86: mwait-idle updates

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 15:20, wrote: > On 08/06/16 14:47, Jan Beulich wrote: >> The first three are Linux imports, while the 4th is an adjustment genuine >> to our clone, and the 5th is an adjustment #3, the Linux equivalent of >> which I didn't get any feedback on so far. >> >> 1: add SKX support >>

Re: [Xen-devel] [PATCH v3 09/16] xen/arm: Introduce alternative runtime patching

2016-06-09 Thread Julien Grall
Hi, On 07/06/16 17:06, Julien Grall wrote: +#else /* !CONFIG_ALTERNATIVE */ + +static inline void apply_alternatives_all(void) +{ +} + +int apply_alternatives(void *start, size_t lenght) I have just notice a compilation issue on ARM32 with this patch which I did not spot during build testing.

[Xen-devel] [GIT PULL] (stable/for-jens-4.7) Xen block fixes for v4.7

2016-06-09 Thread Konrad Rzeszutek Wilk
Hey Jens, Please git pull in your 'for-4.7/drivers' the following branch (based of your 'for-4.7/drivers): git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git stable/for-jens-4.7 which has two fixes for a guest migrating from host that has multi-queue to one without it (and vice-versa

Re: [Xen-devel] [PATCH] Run autogen.sh

2016-06-09 Thread Olaf Hering
On Thu, Jun 09, Andrew Cooper wrote: > Yes - your patch changing the version number from 4.7 to 4.8 and opening > unstable again for general contribution. This also means pkgconfig(xenlight) should report itself as 4.8. As suggested by Roger the SONAME in tools/libxl should be bumped at some poin

Re: [Xen-devel] [PATCH] xen/init: Annotate all command line parameter infrastructure as const

2016-06-09 Thread Andrew Cooper
On 09/06/16 13:39, Jan Beulich wrote: On 09.06.16 at 11:58, wrote: >> --- a/xen/arch/arm/xen.lds.S >> +++ b/xen/arch/arm/xen.lds.S >> @@ -135,6 +135,12 @@ SECTIONS >> *(.init.rodata) >> *(.init.rodata.rel) >> *(.init.rodata.str*) >> + >> + . = ALIGN(POINTER_ALIGN

[Xen-devel] [PATCH] xen/arm: gic-v2: Fix ARM32 build after "xen/arm: gicv2: Export GICv2m..."

2016-06-09 Thread Julien Grall
Commit "xen/arm: gicv2: Export GICv2m register frames to DOM0 by device tree" breaks compilation on ARM32. This is because paddr_t field are printed using %lx. Fix it by using PRIpaddr which will provide the correct modifiers. Signed-off-by: Julien Grall --- Stefano, the patch "xen/arm: gic

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

2016-06-09 Thread osstest service owner
flight 95475 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/95475/ 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 03/11] tools: install XEN_{LOG,RUN}_DIR

2016-06-09 Thread Wei Liu
On Thu, Jun 09, 2016 at 02:13:23PM +0100, Andrew Cooper wrote: > On 09/06/16 13:57, Wei Liu wrote: > > Signed-off-by: Wei Liu > > --- > > Cc: Ian Jackson > > --- > > tools/Makefile | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/tools/Makefile b/tools/Makefile >

Re: [Xen-devel] [PATCH 2/2] x86/HVM: re-order operations in hvm_ud_intercept()

2016-06-09 Thread Andrew Cooper
On 09/06/16 13:31, Jan Beulich wrote: On 09.06.16 at 13:34, wrote: >> On 08/06/16 14:43, Jan Beulich wrote: >>> Don't fetch CS explicitly, leverage the fact that hvm_emulate_prepare() >>> already does (and that hvm_virtual_to_linear_addr() doesn't alter it). >>> >>> At once increase the lengt

Re: [Xen-devel] [PATCH RFC 19/20] acpi: Set HW_REDUCED_ACPI in FADT if IOAPIC is not supported

2016-06-09 Thread Boris Ostrovsky
On 06/09/2016 04:41 AM, Jan Beulich wrote: On 09.06.16 at 10:13, wrote: >> On Wed, Jun 08, 2016 at 06:04:01PM -0400, Boris Ostrovsky wrote: >>> On 06/07/2016 11:41 AM, Jan Beulich wrote: >>> On 07.06.16 at 17:17, wrote: > On 06/07/2016 10:12 AM, Jan Beulich wrote: > On 07.06.

Re: [Xen-devel] [PATCH] Run autogen.sh

2016-06-09 Thread Wei Liu
On Thu, Jun 09, 2016 at 02:02:00PM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH] Run autogen.sh"): > > On Thu, Jun 09, 2016 at 01:28:05PM +0100, Wei Liu wrote: > > > Signed-off-by: Wei Liu > > > --- > > > configure | 18 +- > > > docs/configure| 18 +

Re: [Xen-devel] [PATCH 2/2] x86/HVM: re-order operations in hvm_ud_intercept()

2016-06-09 Thread Jan Beulich
>>> On 09.06.16 at 16:06, wrote: > On 09/06/16 13:31, Jan Beulich wrote: > On 09.06.16 at 13:34, wrote: >>> On 08/06/16 14:43, Jan Beulich wrote: Don't fetch CS explicitly, leverage the fact that hvm_emulate_prepare() already does (and that hvm_virtual_to_linear_addr() doesn't alter

Re: [Xen-devel] [PATCH] APEI: pull a signedness check ahead for Coverity's sake

2016-06-09 Thread Andrew Cooper
On 08/06/16 12:37, Jan Beulich wrote: > On 64-bit architectures (which is all we care about right now in ACPI > code), the value coming from a __u32 field makes "len" positive anyway, > but since from an abstract pov the tool is right, let's just re-order > things. > > Coverity ID: 1204965 > > Sign

Re: [Xen-devel] [PATCH] Run autogen.sh

2016-06-09 Thread Wei Liu
On Thu, Jun 09, 2016 at 03:34:02PM +0200, Olaf Hering wrote: > On Thu, Jun 09, Andrew Cooper wrote: > > > Yes - your patch changing the version number from 4.7 to 4.8 and opening > > unstable again for general contribution. > > This also means pkgconfig(xenlight) should report itself as 4.8. > As

Re: [Xen-devel] [PATCH] xen/init: Annotate all command line parameter infrastructure as const

2016-06-09 Thread Julien Grall
(CC the correct email for Stefano) On 09/06/16 15:19, Julien Grall wrote: Hi Andrew, On 09/06/16 14:45, Andrew Cooper wrote: On 09/06/16 13:39, Jan Beulich wrote: On 09.06.16 at 11:58, wrote: --- a/xen/arch/arm/xen.lds.S +++ b/xen/arch/arm/xen.lds.S @@ -135,6 +135,12 @@ SECTIONS *(

[Xen-devel] [PATCH for 4.7] libxenvchan: Change license of header from Lesser GPL v2.1 to BSD

2016-06-09 Thread Konrad Rzeszutek Wilk
As the xen/COPYING file says: "A few files are licensed under both GPL and a weaker BSD-style license. This includes all files within the subdirectory include/public, as described in include/public/COPYING. All such files include the non-GPL license text as a source-code comment. Although the licen

  1   2   3   >