[Xen-devel] [linux-4.1 test] 100587: regressions - FAIL

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

Re: [Xen-devel] [PATCH v1 7/7] tools: add userspace linker table sandbox

2016-08-22 Thread H. Peter Anvin
Vrabel ,Konrad Rzeszutek Wilk ,Michael Brown ,Juergen Gross ,Andrew Cooper ,Andy Shevchenko ,Paul Gortmaker

[Xen-devel] [PATCH -next] xen-netback: using kfree_rcu() to simplify the code

2016-08-22 Thread Wei Yongjun
From: Wei Yongjun The callback function of call_rcu() just calls a kfree(), so we can use kfree_rcu() instead of call_rcu() + callback function. Signed-off-by: Wei Yongjun --- drivers/net/xen-netback/hash.c | 13 ++--- 1 file changed, 2

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Yang Zhang
On 2016/8/22 22:57, Jan Beulich wrote: On 22.08.16 at 15:09, wrote: On 2016/8/22 20:04, Jan Beulich wrote: On 22.08.16 at 13:41, wrote: On August 22, 2016 6:36 PM, wrote: On 19.08.16 at 14:58, wrote:

[Xen-devel] [PATCH v5] x86/cpuid: AVX-512 Feature Detection

2016-08-22 Thread Luwei Kang
AVX512 is an extention of AVX2. Its spec can be found at: https://software.intel.com/sites/default/files/managed/b4/3a/319433-024.pdf This patch detects AVX512 features by CPUID. Signed-off-by: Luwei Kang --- [V5]: Modify the comment of dependency between AVX512 and AVX2.

Re: [Xen-devel] [RFC 11/22] xen/arm: p2m: Introduce p2m_get_root_pointer and use it in __p2m_lookup

2016-08-22 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > Mapping the root table is always done the same way. To avoid duplicating > the code in a later patch, move the code in a separate helper. > > Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini >

Re: [Xen-devel] [PATCH v4 04/16] generic-sections: add section core helpers

2016-08-22 Thread Nicholas Piggin
On Fri, 19 Aug 2016 14:34:02 -0700 mcg...@kernel.org wrote: > From: "Luis R. Rodriguez" > > Linux makes extensive use of custom ELF header sections, > documentation for these are well scatterred. Unify this > documentation in a central place and provide helpers to > build

Re: [Xen-devel] [RFC 09/22] xen/arm: p2m: Change the type of level_shifts from paddr_t to unsigned int

2016-08-22 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > The level shift can be encoded with 32-bit. So it is not necessary to > use paddr_t (i.e 64-bit). You might as well use 8 bit. > Signed-off-by: Julien Grall > --- > xen/arch/arm/p2m.c | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [Xen-devel] [RFC 08/22] xen/arm: p2m: Invalidate the TLBs when write unlocking the p2m

2016-08-22 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > Sometimes the invalidation of the TLBs can be deferred until the p2m is > unlocked. This is for instance the case when multiple mappings are > removed. In other case, such as shattering a superpage, an immediate > flush is required. > > Keep track

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

2016-08-22 Thread osstest service owner
flight 100585 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/100585/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-rtds 6 xen-boot fail REGR. vs. 100578 build-amd64-rumpuserxen

Re: [Xen-devel] [RFC 07/22] xen/arm: p2m: Rework p2m_put_l3_page

2016-08-22 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > Modify the prototype to directly pass the mfn and the type in > parameters. This will be useful later when we do not have the entry in > hand. > > Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini

Re: [Xen-devel] [RFC 06/22] xen/arm: traps: Check the P2M before injecting a data/instruction abort

2016-08-22 Thread Stefano Stabellini
On Thu, 28 Jul 2016, Julien Grall wrote: > A data/instruction abort may have occurred if another CPU was playing > with the stage-2 page table when following the break-before-make > sequence (see D4.7.1 in ARM DDI 0487A.j). Rather than injecting directly > the fault to the guest, we need to check

Re: [Xen-devel] [PATCHv3 0/2] libfs, xenfs: replace /proc/xen/xenbus with a symlink

2016-08-22 Thread Doug Goldstein
On 6/28/16 2:06 PM, David Vrabel wrote: > Using /proc/xen/xenbus can cause deadlocks on the atomic file position > mutex since this file should behave like a character device and not a > regular file. This is easiest to achive by making it a symlink to the > existing /dev/xen/xenbus device. > >

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Xuquan (Euler)
On August 22, 2016 11:12 PM, wrote: On 22.08.16 at 16:02, wrote: >> On August 22, 2016 8:04 PM, wrote: >> On 22.08.16 at 13:41, wrote: On August 22, 2016 6:36 PM, wrote: On

Re: [Xen-devel] [PATCH 1/2] xen/Kconfig: Drop redundant comments from Kconfig files

2016-08-22 Thread Doug Goldstein
On 8/19/16 11:54 AM, Andrew Cooper wrote: > Most of the comments are duplicated from the help text, and those without help > provide no useful additional input. > > Signed-off-by: Andrew Cooper > --- > CC: Doug Goldstein > CC: Jan Beulich

Re: [Xen-devel] [PATCH v1 7/7] tools: add userspace linker table sandbox

2016-08-22 Thread Luis R. Rodriguez
On Fri, Aug 19, 2016 at 03:31:47PM -0700, Kees Cook wrote: > On Fri, Aug 19, 2016 at 2:41 PM, wrote: > > From: "Luis R. Rodriguez" > > > > Add a userspace sandbox to allow easy experimentation and > > test extensions with linker tables, section ranges and

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

2016-08-22 Thread Luis R. Rodriguez
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 > > +++ b/init/Kconfig > > @@ -53,6 +53,28 @@ config

Re: [Xen-devel] [PATCH v4 07/16] tables.h: add linker table support

2016-08-22 Thread Luis R. Rodriguez
On Fri, Aug 19, 2016 at 03:02:07PM -0700, Kees Cook wrote: > On Fri, Aug 19, 2016 at 2:32 PM, wrote: > > diff --git a/arch/c6x/include/asm/tables.h b/arch/c6x/include/asm/tables.h > > new file mode 100644 > > index ..09a9e31c573a > > --- /dev/null > > +++

Re: [Xen-devel] [PATCH v4 06/16] ranges.h: add helpers to build and identify Linux section ranges

2016-08-22 Thread Luis R. Rodriguez
On Fri, Aug 19, 2016 at 02:55:43PM -0700, Kees Cook wrote: > On Fri, Aug 19, 2016 at 2:32 PM, wrote: > > From: "Luis R. Rodriguez" > > > > Section ranges are on one of the types of custom sections > > types used in Linux. This provides a series of helpers

[Xen-devel] [qemu-mainline baseline-only test] 67578: regressions - FAIL

2016-08-22 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 67578 qemu-mainline real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67578/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-libvirt-qcow2 9

Re: [Xen-devel] [PATCH v4 04/16] generic-sections: add section core helpers

2016-08-22 Thread Luis R. Rodriguez
On Fri, Aug 19, 2016 at 02:47:48PM -0700, Kees Cook wrote: > On Fri, Aug 19, 2016 at 2:32 PM, wrote: > > From: "Luis R. Rodriguez" > > > > +SECTION_RODATA > > +-- > > +.. kernel-doc:: include/asm-generic/section-core.h > > + :doc:

Re: [Xen-devel] [PATCH v4 00/16] linux: generalize sections, ranges and linker tables

2016-08-22 Thread Luis R. Rodriguez
On Fri, Aug 19, 2016 at 03:29:24PM -0700, Kees Cook wrote: > On Fri, Aug 19, 2016 at 2:32 PM, wrote: > > From: "Luis R. Rodriguez" > > > > This v4 addresses feedback from the previous v3 series [0], and also > > addresses a huge array of additional tests

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

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

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

2016-08-22 Thread osstest service owner
flight 100584 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/100584/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl-qemuu-win7-amd64 9 windows-install fail REGR. vs. 100581 Regressions

Re: [Xen-devel] [PATCH v1 9/9] livepatch: tests: Make them compile under ARM64

2016-08-22 Thread Konrad Rzeszutek Wilk
> > > @@ -24,7 +27,9 @@ const char *xen_hello_world(void) > > > */ > > > rc = __get_user(tmp, non_canonical_addr); > > > BUG_ON(rc != -EFAULT); > > > - > > > +#else > > > + asm(ALTERNATIVE("nop", "nop", 1)); > > > > Why the hardcoded 1 here? I am wondering if we should

[Xen-devel] [PATCH v05 56/72] include/uapi/xen/evtchn.h: include xen/privcmd.h

2016-08-22 Thread Mikko Rapeli
It has definition of domid_t. Fixes userspace compiler error when xen/privcmd.h is compiled alone: xen/evtchn.h:100:2: error: unknown type name ‘domid_t’ domid_t domid; ^~~ Signed-off-by: Mikko Rapeli --- include/uapi/xen/evtchn.h | 2 ++ 1 file changed, 2

Re: [Xen-devel] [PATCH v1 6/9] livepatch: Initial ARM64 support.

2016-08-22 Thread Konrad Rzeszutek Wilk
> > > -/* On ARM32,64 instructions are always 4 bytes long. */ > > > -#define PATCH_INSN_SIZE 4 > > > > Rather than moving again PATCH_INSN_SIZE in this patch. Can you directly > > move it in patch [1]? > > Sure. The patch [1] changed where it does not have anymore. And because of that (and

[Xen-devel] [PATCH v05 54/72] include/uapi/xen/privcmd.h: fix compilation in userspace

2016-08-22 Thread Mikko Rapeli
xen/interface/xen.h is not exported from kernel headers so remove the dependency and provide needed defines for domid_t and xen_pfn_t if they are not already defined by some other e.g. Xen specific headers. Suggested by Andrew Cooper on lkml message

Re: [Xen-devel] HVMOP_altp2m_vcpu_enable_notify code example usage

2016-08-22 Thread Dmitry Rockosov
Tamas, My experiment is using the same HVMOP - HVMOP_get_param. Usermode code is using it from https://xenbits.xen.org/gitweb/?p=xen.git;a=blob;f=xen/include/public/hvm/hvm_op.h;hb=2a99aa99fc84a45f505f84802af56b006d14c52e Kernelmode is using it from

[Xen-devel] [PATCH v05 55/72] include/uapi/xen/gntdev.h: include xen/privcmd.h and define grant_ref_t

2016-08-22 Thread Mikko Rapeli
Both are needed to compile wihtout compiler warnings in userspace. Fixes these userspace compile errors: xen/gntdev.h:151:4: error: unknown type name ‘grant_ref_t’ grant_ref_t ref; ^ xen/gntdev.h:153:4: error: unknown type name ‘domid_t’ domid_t domid; ^ Signed-off-by: Mikko

Re: [Xen-devel] HVMOP_altp2m_vcpu_enable_notify code example usage

2016-08-22 Thread Tamas K Lengyel
On Mon, Aug 22, 2016 at 12:42 PM, Dmitry Rockosov wrote: > Tamas, what do you think, why the same hypercall operations (get_param) are > executed by Xen differently? Does Xen know anything about caller domU CPL: > usermode or kernelmode? It's not about the guest's CPL - all

Re: [Xen-devel] HVMOP_altp2m_vcpu_enable_notify code example usage

2016-08-22 Thread Dmitry Rockosov
Tamas, what do you think, why the same hypercall operations (get_param) are executed by Xen differently? Does Xen know anything about caller domU CPL: usermode or kernelmode? Thank you for answers! Best Regards, Rockosov Dmitry 2016-08-22 21:28 GMT+03:00 Tamas K Lengyel

Re: [Xen-devel] HVMOP_altp2m_vcpu_enable_notify code example usage

2016-08-22 Thread Tamas K Lengyel
On Mon, Aug 22, 2016 at 12:12 PM, Dmitry Rockosov wrote: > The problem is in hypervisor checking definitely. I deeper debugged the > code, I met vmcall instruction execution. This means EFAULT is coming from > hypervisor. As it should, question is where exactly it gets denied

Re: [Xen-devel] [DRAFT v5] PV Calls protocol design document (former XenSock)

2016-08-22 Thread Stefano Stabellini
Thank you! On Sun, 21 Aug 2016, Christopher Clark wrote: > The PV Calls (formerly XenSock) protocol design has recently attracted > interest within the OpenXT software > development community, as it is a novel interdomain communication mechanism > and protocol. > > We have a longstanding and

Re: [Xen-devel] HVMOP_altp2m_vcpu_enable_notify code example usage

2016-08-22 Thread Dmitry Rockosov
The problem is in hypervisor checking definitely. I deeper debugged the code, I met vmcall instruction execution. This means EFAULT is coming from hypervisor. Best Regards, Rockosov Dmitry 2016-08-22 18:09 GMT+03:00 Dmitry Rockosov : > Hello Tamas, > > Thank you for the

Re: [Xen-devel] [PATCH v1 9/9] livepatch: tests: Make them compile under ARM64

2016-08-22 Thread Konrad Rzeszutek Wilk
> > > > +#else > > > > + asm(ALTERNATIVE("nop", "nop", 1)); > > > > > > Why the hardcoded 1 here? I am wondering if we should introduce a new > > > capability "LIVEPATCH_TEST" which is enabled by default. So we can test > > > that > > > the the alternative is working on all the platform.

Re: [Xen-devel] [PATCH v2] x86/PV: don't wrongly hide/expose CPUID.OSXSAVE from/to user mode

2016-08-22 Thread Andrew Cooper
On 19/08/16 19:07, Andrew Cooper wrote: > On 19/08/16 18:09, Andrew Cooper wrote: >> On 19/08/16 13:53, Jan Beulich wrote: >>> User mode code generally cannot be expected to invoke the PV-enabled >>> CPUID Xen supports, and prior to the CPUID levelling changes for 4.7 >>> (as well as even nowadays

Re: [Xen-devel] [PATCHv2 0/2] xen/privcmd: prevent page migration for hypercall buffers

2016-08-22 Thread David Vrabel
On 04/08/16 16:16, David Vrabel wrote: > Currently libxencall using mlocked buffers for hypercall buffers. > This pages are subject to compaction and page migration. A userspace > process may see a hypercall fail with -EFAULT if a page backing a > hypercall buffer is in the process of being

Re: [Xen-devel] [PATCHv2 2/2] xen/privcmd: add ioctls for locking/unlocking hypercall buffers

2016-08-22 Thread David Vrabel
On 04/08/16 17:02, Jan Beulich wrote: On 04.08.16 at 17:16, wrote: >> Using mlock() for hypercall buffers is not sufficient since mlocked >> pages are still subject to compaction and page migration. Page >> migration can be prevented by taking additional references

Re: [Xen-devel] [PATCH v1 7/9] livepatch: ARM64: Ignore mapping symbols: $[a, d, x, p]

2016-08-22 Thread Konrad Rzeszutek Wilk
On Wed, Aug 17, 2016 at 06:21:03AM -0600, Jan Beulich wrote: > >>> On 15.08.16 at 01:07, wrote: > > According to the code you mean $t instead of $p in the subject. Yes! Thanks for noticing that. > > > --- a/xen/arch/arm/livepatch.c > > +++ b/xen/arch/arm/livepatch.c > >

Re: [Xen-devel] [PATCH v1 2/9] x86/arm: Make 'make debug' work properly.

2016-08-22 Thread Konrad Rzeszutek Wilk
On Wed, Aug 17, 2016 at 06:00:22AM -0600, Jan Beulich wrote: > >>> On 15.08.16 at 01:07, wrote: > > On x86 it works great but on ARM 32,64 not so much. > > Considering the nature of the change ... > > > --- a/xen/Makefile > > +++ b/xen/Makefile > > @@ -101,7 +101,7 @@

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

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

Re: [Xen-devel] [PATCH net-next] xen-netfront: avoid packet loss when ethernet header crosses page boundary

2016-08-22 Thread David Vrabel
On 22/08/16 16:42, Vitaly Kuznetsov wrote: > > I see two ways to fix the issue: > - Change the 'wire' protocol between netfront and netback to start keeping > the original SKB structure. We'll have to add a flag indicating the fact > that the particular request is a part of the original

[Xen-devel] [PATCH net-next] xen-netfront: avoid packet loss when ethernet header crosses page boundary

2016-08-22 Thread Vitaly Kuznetsov
Small packet loss is reported on complex multi host network configurations including tunnels, NAT, ... My investigation led me to the following check in netback which drops packets: if (unlikely(txreq.size < ETH_HLEN)) { netdev_err(queue->vif->dev,

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

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

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

2016-08-22 Thread osstest service owner
flight 100581 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/100581/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-qemuu-win7-amd64 16 guest-stopfail like 100562

Re: [Xen-devel] [PATCH v8 05/13] libxl: Load guest BIOS from file

2016-08-22 Thread Wei Liu
On Mon, Aug 22, 2016 at 04:09:07PM +0100, Andrew Cooper wrote: > On 22/08/16 15:26, Wei Liu wrote: > > On Mon, Aug 22, 2016 at 02:26:11PM +0100, Andrew Cooper wrote: > >> On 22/08/16 14:13, Wei Liu wrote: > >>> On Fri, Aug 19, 2016 at 03:43:00PM +0100, Andrew Cooper wrote: > On 18/08/16

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

2016-08-22 Thread Masami Hiramatsu
On Fri, 19 Aug 2016 14:34:12 -0700 mcg...@kernel.org wrote: > From: "Luis R. Rodriguez" > > Often all is needed is these small helpers, instead of compiler.h > or a full kprobes.h. This is important for asm helpers, in fact even > some asm/kprobes.h make use of these

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Jan Beulich
>>> On 22.08.16 at 16:02, wrote: > On August 22, 2016 8:04 PM, wrote: > On 22.08.16 at 13:41, wrote: >>> On August 22, 2016 6:36 PM, wrote: >>> On 19.08.16 at 14:58, wrote: > From

Re: [Xen-devel] HVMOP_altp2m_vcpu_enable_notify code example usage

2016-08-22 Thread Dmitry Rockosov
Hello Tamas, Thank you for the answer! I installed the same xen source code (4.7.0) as on dom0 to domU, built dist-tools, installed xen-tools, updated rc.d configuration and rebooted. IOCTL channel to privcmd driver is working fine, but all requests to hypervisor like hvm_get_param,

Re: [Xen-devel] [PATCH v8 05/13] libxl: Load guest BIOS from file

2016-08-22 Thread Andrew Cooper
On 22/08/16 15:26, Wei Liu wrote: > On Mon, Aug 22, 2016 at 02:26:11PM +0100, Andrew Cooper wrote: >> On 22/08/16 14:13, Wei Liu wrote: >>> On Fri, Aug 19, 2016 at 03:43:00PM +0100, Andrew Cooper wrote: On 18/08/16 15:13, Wei Liu wrote: > From: Anthony PERARD

[Xen-devel] [PATCH 1/2] tools: only define {OVMF, SEABIOS}_PATH when they are enabled

2016-08-22 Thread Wei Liu
Signed-off-by: Wei Liu --- Rerun autogen.sh --- tools/configure| 8 tools/configure.ac | 16 ++-- 2 files changed, 18 insertions(+), 6 deletions(-) diff --git a/tools/configure b/tools/configure index 81ccf34..998090a 100755 --- a/tools/configure

[Xen-devel] [PATCH 2/2] libxl: only return {OVMF, SEABIOS}_PATH if available

2016-08-22 Thread Wei Liu
Signed-off-by: Wei Liu --- tools/libxl/libxl_paths.c | 8 1 file changed, 8 insertions(+) diff --git a/tools/libxl/libxl_paths.c b/tools/libxl/libxl_paths.c index 6972b90..0643c1b 100644 --- a/tools/libxl/libxl_paths.c +++ b/tools/libxl/libxl_paths.c @@ -37,12

[Xen-devel] [PATCH 0/2] Fix issue with {OVMF,SEABIOS}_PATH

2016-08-22 Thread Wei Liu
They shouldn't be available when the respective BIOS is disabled at build time. This should fix the issus that causes xtf fail to launch hvm guests. Wei Liu (2): tools: only define {OVMF,SEABIOS}_PATH when they are enabled libxl: only return {OVMF,SEABIOS}_PATH if available tools/configure

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Jan Beulich
>>> On 22.08.16 at 15:09, wrote: > On 2016/8/22 20:04, Jan Beulich wrote: > On 22.08.16 at 13:41, wrote: >>> On August 22, 2016 6:36 PM, wrote: >>> On 19.08.16 at 14:58, wrote: > From

Re: [Xen-devel] [PATCH v8 05/13] libxl: Load guest BIOS from file

2016-08-22 Thread Wei Liu
On Mon, Aug 22, 2016 at 02:26:11PM +0100, Andrew Cooper wrote: > On 22/08/16 14:13, Wei Liu wrote: > > On Fri, Aug 19, 2016 at 03:43:00PM +0100, Andrew Cooper wrote: > >> On 18/08/16 15:13, Wei Liu wrote: > >>> From: Anthony PERARD > >>> > >>> The path to the BIOS blob

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Yang Zhang
2016年8月22日星期一,Xuquan (Euler) 写道: > On August 22, 2016 8:04 PM, > wrote: > On 22.08.16 at 13:41, > wrote: > >> On August 22, 2016 6:36 PM, > wrote: > >> On 19.08.16 at

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Yang Zhang
2016年8月22日星期一,Xuquan (Euler) 写道: > On August 22, 2016 8:04 PM, > wrote: > On 22.08.16 at 13:41, > wrote: > >> On August 22, 2016 6:36 PM, > wrote: > >> On 19.08.16 at

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Xuquan (Euler)
On August 22, 2016 8:04 PM, wrote: On 22.08.16 at 13:41, wrote: >> On August 22, 2016 6:36 PM, wrote: >> On 19.08.16 at 14:58, wrote: From 9b2df963c13ad27e2cffbeddfa3267782ac3da2a Mon Sep 17 00:00:00

Re: [Xen-devel] Regression in kernel 4.7 xenstore

2016-08-22 Thread Christoph Moench-Tegeder
## Jan Beulich (jbeul...@suse.com): > See https://patchwork.kernel.org/patch/9281193/. Yes, that's it. Thanks, fix confirmed. Regards, Christoph -- Spare Space ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

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

2016-08-22 Thread osstest service owner
flight 100582 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/100582/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 72388f9c10126a718a7ac381dc6879d3337ccb8b baseline version: ovmf

Re: [Xen-devel] [PATCH v8 05/13] libxl: Load guest BIOS from file

2016-08-22 Thread Andrew Cooper
On 22/08/16 14:13, Wei Liu wrote: > On Fri, Aug 19, 2016 at 03:43:00PM +0100, Andrew Cooper wrote: >> On 18/08/16 15:13, Wei Liu wrote: >>> From: Anthony PERARD >>> >>> The path to the BIOS blob can be overriden by the xl's >>> bios_path_override option, or provided by

Re: [Xen-devel] [PATCH v2 0/2] hvmloader: fix two issues spotted by Coverity

2016-08-22 Thread Wei Liu
On Mon, Aug 22, 2016 at 01:47:51PM +0100, Wei Liu wrote: > Wei Liu (2): > hvmloader: correctly copy signature to info structures > hvmloader: use bound checking in get_module_entry > Pushed. ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Xuquan (Euler)
On August 22, 2016 9:10 PM, Yang Zhang wrote: >On 2016/8/22 20:04, Jan Beulich wrote: > On 22.08.16 at 13:41, wrote: >>> On August 22, 2016 6:36 PM, wrote: >>> On 19.08.16 at 14:58, wrote: > From

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Yang Zhang
On 2016/8/22 20:04, Jan Beulich wrote: On 22.08.16 at 13:41, wrote: On August 22, 2016 6:36 PM, wrote: On 19.08.16 at 14:58, wrote: From 9b2df963c13ad27e2cffbeddfa3267782ac3da2a Mon Sep 17 00:00:00 2001 From: Quan Xu

Re: [Xen-devel] [PATCH v2 1/2] hvmloader: correctly copy signature to info structures

2016-08-22 Thread Jan Beulich
>>> On 22.08.16 at 14:47, wrote: > The original code used sizeof(info->signature) as the size parameter for > memcpy, which was wrong. > > Fix that by using structure assignment. > > Signed-off-by: Wei Liu Acked-by: Jan Beulich

Re: [Xen-devel] [PATCH v2 2/2] hvmloader: use bound checking in get_module_entry

2016-08-22 Thread Jan Beulich
>>> On 22.08.16 at 14:47, wrote: > Coverity complains: > > overflow_before_widen: Potentially overflowing expression > info->nr_modules * 32U with type unsigned int (32 bits, unsigned) is > evaluated using 32-bit arithmetic, and then used in a context that > expects an

[Xen-devel] [PATCH v2 1/2] hvmloader: correctly copy signature to info structures

2016-08-22 Thread Wei Liu
The original code used sizeof(info->signature) as the size parameter for memcpy, which was wrong. Fix that by using structure assignment. Signed-off-by: Wei Liu --- tools/firmware/hvmloader/ovmf.c| 8 tools/firmware/hvmloader/seabios.c | 8 2 files

[Xen-devel] [PATCH v2 2/2] hvmloader: use bound checking in get_module_entry

2016-08-22 Thread Wei Liu
Coverity complains: overflow_before_widen: Potentially overflowing expression info->nr_modules * 32U with type unsigned int (32 bits, unsigned) is evaluated using 32-bit arithmetic, and then used in a context that expects an expression of type uint64_t (64 bits, unsigned). The overflow is

[Xen-devel] [PATCH v2 0/2] hvmloader: fix two issues spotted by Coverity

2016-08-22 Thread Wei Liu
Wei Liu (2): hvmloader: correctly copy signature to info structures hvmloader: use bound checking in get_module_entry tools/firmware/hvmloader/hvmloader.c | 4 ++-- tools/firmware/hvmloader/ovmf.c | 8 tools/firmware/hvmloader/seabios.c | 8 3 files changed, 10

Re: [Xen-devel] Xen 4.6.1 crash with altp2m enabledbydefault

2016-08-22 Thread Kevin.Mayer
Hi The reproduction should be pretty simple: Apply the patch to enable altp2m unconditionally: d->arch.hvm_domain.params[HVM_PARAM_HPET_ENABLED] = 1; d->arch.hvm_domain.params[HVM_PARAM_TRIPLE_FAULT_REASON] = SHUTDOWN_reboot; +d->arch.hvm_domain.params[HVM_PARAM_ALTP2M] = 1; +

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Jan Beulich
>>> On 22.08.16 at 13:41, wrote: > On August 22, 2016 6:36 PM, wrote: > On 19.08.16 at 14:58, wrote: >>> From 9b2df963c13ad27e2cffbeddfa3267782ac3da2a Mon Sep 17 00:00:00 2001 >>> From: Quan Xu >>> Date: Fri, 19

Re: [Xen-devel] [PATCH 2/2] hvmloader: cast to 64bit before multiplication in get_module_entry

2016-08-22 Thread Jan Beulich
>>> On 22.08.16 at 13:37, wrote: > On Fri, Aug 19, 2016 at 06:00:12AM -0600, Jan Beulich wrote: >> >>> On 19.08.16 at 12:09, wrote: >> > On 19/08/16 09:31, Jan Beulich wrote: >> > On 19.08.16 at 10:06, wrote: >> >>> ---

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Jan Beulich
>>> On 22.08.16 at 13:40, wrote: > On 2016/8/22 18:35, Jan Beulich wrote: > On 19.08.16 at 14:58, wrote: >>> From 9b2df963c13ad27e2cffbeddfa3267782ac3da2a Mon Sep 17 00:00:00 2001 >>> From: Quan Xu >>> Date: Fri, 19 Aug 2016

Re: [Xen-devel] Xen 4.6.1 crash with altp2m enabledbydefault

2016-08-22 Thread Andrew Cooper
On 19/08/16 11:01, kevin.ma...@gdata.de wrote: > Hi > > I took another look at Xen and a new crashdump. > The last successful __vmwrite should be in > static void vmx_vcpu_update_vmfunc_ve(struct vcpu *v) > [...] > __vmwrite(SECONDARY_VM_EXEC_CONTROL, >

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Xuquan (Euler)
On August 22, 2016 7:40 PM, Yang Zhang wrote: >On 2016/8/22 18:35, Jan Beulich wrote: > On 19.08.16 at 14:58, wrote: >>> From 9b2df963c13ad27e2cffbeddfa3267782ac3da2a Mon Sep 17 00:00:00 >>> 2001 >>> From: Quan Xu >>> Date: Fri, 19 Aug 2016 20:40:31

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Xuquan (Euler)
On August 22, 2016 6:36 PM, wrote: On 19.08.16 at 14:58, wrote: >> From 9b2df963c13ad27e2cffbeddfa3267782ac3da2a Mon Sep 17 00:00:00 2001 >> From: Quan Xu >> Date: Fri, 19 Aug 2016 20:40:31 +0800 >> Subject: [RFC PATCH] x86/apicv:

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Yang Zhang
On 2016/8/22 18:35, Jan Beulich wrote: On 19.08.16 at 14:58, wrote: From 9b2df963c13ad27e2cffbeddfa3267782ac3da2a Mon Sep 17 00:00:00 2001 From: Quan Xu Date: Fri, 19 Aug 2016 20:40:31 +0800 Subject: [RFC PATCH] x86/apicv: fix RTC periodic timer and

Re: [Xen-devel] check if a page is modified

2016-08-22 Thread sepanta s
I use xen-access to trace the paegs which are accessed to be wrtitten but when I print out the page content immediately after getting the event in xen-access, its content is unchanged . What should I do?? On Sat, Aug 20, 2016 at 5:13 PM, sepanta s wrote: > Hi, > How can I

Re: [Xen-devel] [PATCH 2/2] hvmloader: cast to 64bit before multiplication in get_module_entry

2016-08-22 Thread Wei Liu
On Fri, Aug 19, 2016 at 06:00:12AM -0600, Jan Beulich wrote: > >>> On 19.08.16 at 12:09, wrote: > > On 19/08/16 09:31, Jan Beulich wrote: > > On 19.08.16 at 10:06, wrote: > >>> --- a/tools/firmware/hvmloader/hvmloader.c > >>> +++

Re: [Xen-devel] [PATCH v4] x86/cpuid: AVX-512 Feature Detection

2016-08-22 Thread Jan Beulich
>>> On 22.08.16 at 13:22, wrote: >> > >> First of all - please don't top post. >> > >> >> > >>> What about remove the dependency between AVX2 and AVX512F >> > ( AVX2: >> > >> [AVX512F], ) ? >> > >> >> > >> Yes, that's

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

2016-08-22 Thread osstest service owner
flight 100580 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/100580/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 8866d337ea9eba258e942585b172d57d39376e70 baseline version: ovmf

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Xuquan (Euler)
On August 22, 2016 5:38 PM, Yang Zhang wrote: >On 2016/8/19 20:58, Xuquan (Euler) wrote: >> From 9b2df963c13ad27e2cffbeddfa3267782ac3da2a Mon Sep 17 00:00:00 2001 >> From: Quan Xu >> Date: Fri, 19 Aug 2016 20:40:31 +0800 >> Subject: [RFC PATCH] x86/apicv: fix RTC periodic

Re: [Xen-devel] [PATCH 1/2] xen/Kconfig: Drop redundant comments from Kconfig files

2016-08-22 Thread Julien Grall
Hi, On 19/08/16 16:54, Andrew Cooper wrote: Most of the comments are duplicated from the help text, and those without help provide no useful additional input. Signed-off-by: Andrew Cooper --- CC: Doug Goldstein CC: Jan Beulich

Re: [Xen-devel] unable start xen in hikey

2016-08-22 Thread Julien Grall
On 19/08/16 23:44, Kamenee Arumugam wrote: Hi Julien/Chen, Hello, Thanks, it is working for me after setting CONFIG_ARCH_HISI = y in config file. So, when i tried to boot linux thru startup.sh, it seems to hangs and below are the traces of log: Press ESC in 1 seconds to skip startup.nsh

Re: [Xen-devel] [PATCH 1/2] libs/gnttab: do not use alloca(3)

2016-08-22 Thread Wei Liu
On Mon, Aug 22, 2016 at 11:24:56AM +0100, David Vrabel wrote: > On 22/08/16 11:10, Wei Liu wrote: > > On Mon, Aug 22, 2016 at 10:46:50AM +0100, David Vrabel wrote: > >> On 17/08/16 15:33, Wei Liu wrote: > >>> The semantics of alloca(3) is not very nice. If the stack overflows, > >>> program

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Jan Beulich
>>> On 19.08.16 at 14:58, wrote: > From 9b2df963c13ad27e2cffbeddfa3267782ac3da2a Mon Sep 17 00:00:00 2001 > From: Quan Xu > Date: Fri, 19 Aug 2016 20:40:31 +0800 > Subject: [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue > > When Xen apicv

Re: [Xen-devel] [PATCH] tools/firmware: Rename bios.bin to seabios.bin

2016-08-22 Thread Wei Liu
On Fri, Aug 19, 2016 at 03:26:23PM +0100, Andrew Cooper wrote: > bios.bin as a name is far too generic. Rename it to seabios.bin. > > Signed-off-by: Andrew Cooper Hmm... I remember the first few versions of that series had it named seabios.bin and I acked that.

Re: [Xen-devel] Regression in kernel 4.7 xenstore

2016-08-22 Thread Jan Beulich
>>> On 21.08.16 at 00:33, wrote: > Hi, > > I'm running Xen 4.4.1 as packaged in Debian 8.5 (jessie) with home-baked > kernels (that is, from kernel.org) (hardware is x86-64, in case that > matters). > > After upgrading to kernel 4.7, xenstore-write is not able to set > the

Re: [Xen-devel] [PATCH 1/2] libs/gnttab: do not use alloca(3)

2016-08-22 Thread Wei Liu
On Mon, Aug 22, 2016 at 10:46:50AM +0100, David Vrabel wrote: > On 17/08/16 15:33, Wei Liu wrote: > > The semantics of alloca(3) is not very nice. If the stack overflows, > > program behaviour is undefined. > > > > Remove the use of alloca(3) and always use mmap. > > This is only using alloca()

[Xen-devel] [distros-debian-sid test] 67575: regressions - FAIL

2016-08-22 Thread Platform Team regression test user
flight 67575 distros-debian-sid real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/67575/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-armhf-sid-netboot-pygrub 9 debian-di-install fail REGR. vs. 66937

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

2016-08-22 Thread Jan Beulich
>>> 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: > - smaller Xen code base, > - one code base for

Re: [Xen-devel] [PATCH 1/2] libs/gnttab: do not use alloca(3)

2016-08-22 Thread David Vrabel
On 17/08/16 15:33, Wei Liu wrote: > The semantics of alloca(3) is not very nice. If the stack overflows, > program behaviour is undefined. > > Remove the use of alloca(3) and always use mmap. This is only using alloca() if the allocation is < PAGE_SIZE. I think assuming there's this much extra

Re: [Xen-devel] [MINIOS PATCH 0/3] x86: use ELF notes and unified linker script

2016-08-22 Thread Wei Liu
On Thu, Aug 18, 2016 at 11:15:23AM +0100, Wei Liu wrote: > Wei Liu (3): > Introduce asm_macros.h > x86: switch to use elfnote > x86: use unified linker script > Series pushed. ___ Xen-devel mailing list Xen-devel@lists.xen.org

Re: [Xen-devel] [PATCH] mini-os: fix coverity issues in printf.c

2016-08-22 Thread Wei Liu
On Wed, Aug 17, 2016 at 03:39:59PM +0200, Juergen Gross wrote: > Fix two issues discovered by coverity. > Pushed with fixed up commit message. ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel

Re: [Xen-devel] [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue

2016-08-22 Thread Yang Zhang
On 2016/8/19 20:58, Xuquan (Euler) wrote: From 9b2df963c13ad27e2cffbeddfa3267782ac3da2a Mon Sep 17 00:00:00 2001 From: Quan Xu Date: Fri, 19 Aug 2016 20:40:31 +0800 Subject: [RFC PATCH] x86/apicv: fix RTC periodic timer and apicv issue When Xen apicv is enabled, wall clock

Re: [Xen-devel] [PATCH 14/24] libxl: allow to set the ratelimit value online for Credit2

2016-08-22 Thread Ian Jackson
Dario Faggioli writes ("[PATCH 14/24] libxl: allow to set the ratelimit value online for Credit2"): ... > -rc = xc_sched_credit_params_set(ctx->xch, poolid, ); > -if ( rc < 0 ) { > -LOGE(ERROR, "setting sched credit param"); > -GC_FREE; > -return ERROR_FAIL; > +

Re: [Xen-devel] [PATCH 14/24] libxl: allow to set the ratelimit value online for Credit2

2016-08-22 Thread Ian Jackson
Dario Faggioli writes ("[PATCH 14/24] libxl: allow to set the ratelimit value online for Credit2"): > This is the remaining part of the plumbing (the libxl > one) necessary to be able to change the value of the > ratelimit_us parameter online, for Credit2 (like it is > already for Credit1). I

Re: [Xen-devel] Xen Wiki Write permissions

2016-08-22 Thread Wei Liu
Hi Neil On Sun, Aug 21, 2016 at 01:22:15PM -0400, Neil Sikka wrote: > Hello, how can i get write permission to update documentation on the Xen > Wiki? > Please send me your account name for Xen wiki and I will grant you write permission. Wei. > -- > My Blog:

[Xen-devel] [xen-unstable test] 100578: tolerable FAIL

2016-08-22 Thread osstest service owner
flight 100578 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/100578/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-xl-rtds 15 guest-start/debian.repeatfail like 100569 build-amd64-rumpuserxen

  1   2   >