Re: [Xen-devel] [PATCH LIVEPATCH-BUILD-TOOLS] EFI build with -ffunction-sections fails.

2016-07-14 Thread Ross Lagerwall
On 07/13/2016 09:14 PM, Konrad Rzeszutek Wilk wrote: When we build Xen with the Rules.mk modified we end up: ... snip A bit of grepping showed that the issue is with: --section-alignment=0x20 which is used on the linker command line and this fix replaces the --section-alignment to be 4

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

2016-07-14 Thread osstest service owner
flight 97279 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/97279/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemut-debianhvm-amd64 9 debian-hvm-install fail REGR. vs. 96211 test-amd64-i386-xl

[Xen-devel] [PATCH LIVEPATCH-BUILD-TOOLS v2 2/3] Remove --xen-debug

2016-07-14 Thread Ross Lagerwall
With Xen commit bacbf0cb7349 ("build: convert debug to Kconfig"), the debug build is controlled via Kconfig, so drop the separate --xen-debug option to livepatch-build. Signed-off-by: Ross Lagerwall --- livepatch-build | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git

[Xen-devel] [PATCH LIVEPATCH-BUILD-TOOLS v2 1/3] Update to use a .config file

2016-07-14 Thread Ross Lagerwall
Require the user to pass a .config file matching the original build's .config to ensure that the build configuration is identical. Signed-off-by: Ross Lagerwall --- livepatch-build | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/livepatch-build b/livepatch-bui

[Xen-devel] [PATCH LIVEPATCH-BUILD-TOOLS v2 3/3] Update README.md

2016-07-14 Thread Ross Lagerwall
Update the example and project status. Add Contributing and Maintainers sections. Signed-off-by: Ross Lagerwall --- README.md | 76 --- 1 file changed, 53 insertions(+), 23 deletions(-) diff --git a/README.md b/README.md index 9fb709f.

Re: [Xen-devel] [PATCH 2/3] Update to use a .config file

2016-07-14 Thread Ross Lagerwall
On 06/15/2016 03:00 PM, Konrad Rzeszutek Wilk wrote: On Wed, Jun 15, 2016 at 09:08:46AM +0100, Ross Lagerwall wrote: On 06/14/2016 04:35 PM, Konrad Rzeszutek Wilk wrote: On Fri, Jun 10, 2016 at 12:02:44PM +0100, Ross Lagerwall wrote: Remove the old --xen-debug option, and instead, require the

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

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

[Xen-devel] [PATCH LIVEPATCH-BUILD-TOOLS] Support "make install"

2016-07-14 Thread Ross Lagerwall
Add rules to support using "make install". Use "make install DESTDIR=... PREFIX=..." to customize the installation path. Signed-off-by: Ross Lagerwall --- Makefile| 10 ++ livepatch-build | 20 +--- 2 files changed, 27 insertions(+), 3 deletions(-) diff --git a/

[Xen-devel] [PATCH v3 0/8] adjustments

2016-07-14 Thread Corneliu ZUZU
Corneliu ZUZU (8): 1/8: asm-arm/atomic.h: fix arm32|arm64 macros duplication Reviewed-by: Stefano Stabellini Changed: * also moved mistakenly omitted atomic_xchg() * arm empty line fixes moved here 2/8: asm-x86/atomic.h: minor: proper atomic_inc_and_test() placement

[Xen-devel] [PATCH v3 1/8] asm-arm/atomic.h: fix arm32|arm64 macros duplication

2016-07-14 Thread Corneliu ZUZU
Move duplicate macros between asm-arm/arm32/atomic.h and asm-arm/arm64/atomic.h to asm-arm/atomic.h. Also empty line fixes. Signed-off-by: Corneliu ZUZU Reviewed-by: Stefano Stabellini --- Changed since v2: * also moved mistakenly omitted atomic_xchg() * arm empty line fixes moved here ---

[Xen-devel] [PATCH v3 2/8] asm-x86/atomic.h: minor: proper atomic_inc_and_test() placement

2016-07-14 Thread Corneliu ZUZU
Place atomic_inc_and_test() implementation after atomic_inc(). Also empty line fix. Signed-off-by: Corneliu ZUZU --- Changed since v2: arm empty line fixes moved from here --- xen/include/asm-x86/atomic.h | 39 +++ 1 file changed, 19 insertions(+), 20 deletion

[Xen-devel] [PATCH v3 3/8] asm-arm/atomic.h: reorder macros to match x86-side

2016-07-14 Thread Corneliu ZUZU
Reorder macro definitions to match x86-side. Signed-off-by: Corneliu ZUZU Reviewed-by: Stefano Stabellini --- Changed since v2: --- xen/include/asm-arm/atomic.h | 19 +-- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/xen/include/asm-arm/atomic.h b/xen/include/a

[Xen-devel] [PATCH v3 4/8] asm/atomic.h: common prototyping (add xen/atomic.h)

2016-07-14 Thread Corneliu ZUZU
Create a common-side to establish, among others, prototypes of atomic functions called from common-code. Done to avoid introducing inconsistencies between arch-side headers when we make subtle changes to one of them. Some arm-side macros had to be turned into inline functions in the process. Rem

[Xen-devel] [PATCH v3 5/8] fix: make atomic_read() param const

2016-07-14 Thread Corneliu ZUZU
This wouldn't let me make a param of a function that used atomic_read() const. Signed-off-by: Corneliu ZUZU Reviewed-by: Andrew Cooper --- Changed since v2: --- xen/include/asm-arm/atomic.h | 2 +- xen/include/asm-x86/atomic.h | 2 +- xen/include/xen/atomic.h | 2 +- 3 files changed, 3 in

[Xen-devel] [PATCH v3 6/8] asm-arm/atomic.h: atomic_{inc, dec}_return: macros to inline functions

2016-07-14 Thread Corneliu ZUZU
Turn atomic_inc_return and atomic_dec_return atomic.h macros to inline functions. Signed-off-by: Corneliu ZUZU --- xen/include/asm-arm/atomic.h | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/xen/include/asm-arm/atomic.h b/xen/include/asm-arm/atomic.h index 78d

[Xen-devel] [PATCH v3 7/8] arm/atomic.h: remove '__' prefix for __atomic_add_unless()

2016-07-14 Thread Corneliu ZUZU
The built-in leading underscores ('__') don't serve any purpose, so rename __atomic_add_unless() -> atomic_add_unless(). Signed-off-by: Corneliu ZUZU --- xen/include/asm-arm/arm32/atomic.h | 2 +- xen/include/asm-arm/arm64/atomic.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff -

[Xen-devel] [PATCH v3 8/8] asm-x86/atomic.h: implement missing, add common prototypes

2016-07-14 Thread Corneliu ZUZU
- implement missing functions atomic_{sub,inc,dec}_return(), atomic_add_unless() on X86 and also add prototypes for them in common - add missing macro atomic_xchg for X86 Signed-off-by: Corneliu ZUZU --- xen/include/asm-x86/atomic.h | 27 +++ xen/include/xen/atomic.h

Re: [Xen-devel] [PATCH v3 5/8] fix: make atomic_read() param const

2016-07-14 Thread Andrew Cooper
On 14/07/16 10:13, Corneliu ZUZU wrote: > This wouldn't let me make a param of a function that used atomic_read() const. > > Signed-off-by: Corneliu ZUZU > Reviewed-by: Andrew Cooper You have dropped a Reviewed-by: Stefano Stabellini from v2. ~Andrew __

Re: [Xen-devel] [PATCH v2 1/5] asm-arm/atomic.h: fix arm32|arm64 macros duplication

2016-07-14 Thread Julien Grall
On 14/07/16 06:11, Corneliu ZUZU wrote: On 7/13/2016 10:12 PM, Julien Grall wrote: Hi Corneliu, On 13/07/2016 15:18, Corneliu ZUZU wrote: Move duplicate macros between asm-arm/arm32/atomic.h and asm-arm/arm64/atomic.h to asm-arm/atomic.h. asm-arm/arm*/atomic.h were a copy from Linux. I don

Re: [Xen-devel] [PATCH v3 2/8] asm-x86/atomic.h: minor: proper atomic_inc_and_test() placement

2016-07-14 Thread Andrew Cooper
On 14/07/16 10:10, Corneliu ZUZU wrote: > Place atomic_inc_and_test() implementation after atomic_inc(). > Also empty line fix. > > Signed-off-by: Corneliu ZUZU Reviewed-by: Andrew Cooper ___ Xen-devel mailing list Xen-devel@lists.xen.org https://lists

Re: [Xen-devel] [PATCH v3 8/8] asm-x86/atomic.h: implement missing, add common prototypes

2016-07-14 Thread Andrew Cooper
On 14/07/16 10:14, Corneliu ZUZU wrote: > - implement missing functions atomic_{sub,inc,dec}_return(), > atomic_add_unless() > on X86 and also add prototypes for them in common > > - add missing macro atomic_xchg for X86 > > Signed-off-by: Corneliu ZUZU Reviewed-by: Andrew Cooper ___

Re: [Xen-devel] [PATCH v3 5/8] fix: make atomic_read() param const

2016-07-14 Thread Corneliu ZUZU
On 7/14/2016 12:26 PM, Andrew Cooper wrote: On 14/07/16 10:13, Corneliu ZUZU wrote: This wouldn't let me make a param of a function that used atomic_read() const. Signed-off-by: Corneliu ZUZU Reviewed-by: Andrew Cooper You have dropped a Reviewed-by: Stefano Stabellini from v2. ~Andrew

Re: [Xen-devel] [PATCH v3 5/8] fix: make atomic_read() param const

2016-07-14 Thread Corneliu ZUZU
On 7/14/2016 12:26 PM, Andrew Cooper wrote: On 14/07/16 10:13, Corneliu ZUZU wrote: This wouldn't let me make a param of a function that used atomic_read() const. Signed-off-by: Corneliu ZUZU Reviewed-by: Andrew Cooper You have dropped a Reviewed-by: Stefano Stabellini from v2. ~Andrew O

Re: [Xen-devel] [PATCH v2] libxl: trigger attach events for devices attached before xl devd startup

2016-07-14 Thread Wei Liu
On Mon, Jul 11, 2016 at 12:44:42PM +0200, Marek Marczykowski-Górecki wrote: > When this daemon is started after creating backend device, that device > will not be configured. > > Racy situation: > 1. driver domain is started > 2. frontend domain is started (just after kicking driver domain off) >

Re: [Xen-devel] [PATCH 1/2] xen: fix a (latent) cpupool-related race during domain destroy

2016-07-14 Thread Andrew Cooper
On 14/07/16 07:41, Dario Faggioli wrote: > So, during domain destruction, we do: > cpupool_rm_domain()[ in domain_destroy() ] > sched_destroy_domain() [ in complete_domain_destroy() ] > > Therefore, there's a window during which, from the > scheduler's point of view, a domain is still there,

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

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

Re: [Xen-devel] [PATCH v3 8/8] asm-x86/atomic.h: implement missing, add common prototypes

2016-07-14 Thread Corneliu ZUZU
On 7/14/2016 12:29 PM, Andrew Cooper wrote: On 14/07/16 10:14, Corneliu ZUZU wrote: - implement missing functions atomic_{sub,inc,dec}_return(), atomic_add_unless() on X86 and also add prototypes for them in common - add missing macro atomic_xchg for X86 Signed-off-by: Corneliu ZUZU Revi

Re: [Xen-devel] [PATCH v3 8/8] asm-x86/atomic.h: implement missing, add common prototypes

2016-07-14 Thread Andrew Cooper
On 14/07/16 10:39, Corneliu ZUZU wrote: > On 7/14/2016 12:29 PM, Andrew Cooper wrote: >> On 14/07/16 10:14, Corneliu ZUZU wrote: >>> - implement missing functions atomic_{sub,inc,dec}_return(), >>> atomic_add_unless() >>> on X86 and also add prototypes for them in common >>> >>> - add missing m

Re: [Xen-devel] [PATCH v3 8/8] asm-x86/atomic.h: implement missing, add common prototypes

2016-07-14 Thread Corneliu ZUZU
On 7/14/2016 12:44 PM, Andrew Cooper wrote: On 14/07/16 10:39, Corneliu ZUZU wrote: On 7/14/2016 12:29 PM, Andrew Cooper wrote: On 14/07/16 10:14, Corneliu ZUZU wrote: - implement missing functions atomic_{sub,inc,dec}_return(), atomic_add_unless() on X86 and also add prototypes for them in

[Xen-devel] [PATCH] x86/emulate: add support for movq xmm,xmm/m64

2016-07-14 Thread Mihai Donțu
Signed-off-by: Mihai Donțu --- tools/tests/x86_emulator/test_x86_emulator.c | 23 +++ xen/arch/x86/x86_emulate/x86_emulate.c | 7 --- 2 files changed, 27 insertions(+), 3 deletions(-) diff --git a/tools/tests/x86_emulator/test_x86_emulator.c b/tools/tests/x86_emul

Re: [Xen-devel] [PATCH] xen: remove xenstore watches of backends when terminating qemu

2016-07-14 Thread Juergen Gross
On 13/07/16 14:31, Juergen Gross wrote: > Xenstore watches of the /local/domain//backend/ directories > are never removed. This can lead to a memory leak in xenstored, > especially when xenstored is running in another domain (this will be > the case either for a system with xenstore-stubdom, or wit

Re: [Xen-devel] [PATCH v3 7/8] arm/atomic.h: remove '__' prefix for __atomic_add_unless()

2016-07-14 Thread Julien Grall
Hi Corneliu, On 14/07/16 10:14, Corneliu ZUZU wrote: The built-in leading underscores ('__') don't serve any purpose, so rename __atomic_add_unless() -> atomic_add_unless(). The leading underscores are from the Linux code. We decided to keep those files unmodified to help syncing atomic code.

Re: [Xen-devel] [PATCH v3 7/8] arm/atomic.h: remove '__' prefix for __atomic_add_unless()

2016-07-14 Thread Corneliu ZUZU
On 7/14/2016 1:00 PM, Julien Grall wrote: Hi Corneliu, On 14/07/16 10:14, Corneliu ZUZU wrote: The built-in leading underscores ('__') don't serve any purpose, so rename __atomic_add_unless() -> atomic_add_unless(). The leading underscores are from the Linux code. We decided to keep those f

Re: [Xen-devel] [PATCH v2 1/5] asm-arm/atomic.h: fix arm32|arm64 macros duplication

2016-07-14 Thread Corneliu ZUZU
On 7/14/2016 12:26 PM, Julien Grall wrote: On 14/07/16 06:11, Corneliu ZUZU wrote: On 7/13/2016 10:12 PM, Julien Grall wrote: Hi Corneliu, On 13/07/2016 15:18, Corneliu ZUZU wrote: Move duplicate macros between asm-arm/arm32/atomic.h and asm-arm/arm64/atomic.h to asm-arm/atomic.h. asm-arm

Re: [Xen-devel] [PATCH v2 1/5] asm-arm/atomic.h: fix arm32|arm64 macros duplication

2016-07-14 Thread Julien Grall
On 14/07/16 11:11, Corneliu ZUZU wrote: On 7/14/2016 12:26 PM, Julien Grall wrote: On 14/07/16 06:11, Corneliu ZUZU wrote: On 7/13/2016 10:12 PM, Julien Grall wrote: Hi Corneliu, On 13/07/2016 15:18, Corneliu ZUZU wrote: Move duplicate macros between asm-arm/arm32/atomic.h and asm-arm/ar

Re: [Xen-devel] [PATCH v4] xen/arm: Add a clock property

2016-07-14 Thread Julien Grall
Hi Dirk, On 14/07/16 07:31, Dirk Behme wrote: On 13.07.2016 23:03, Michael Turquette wrote: Quoting Dirk Behme (2016-07-13 11:56:30) On 13.07.2016 20:43, Stefano Stabellini wrote: On Wed, 13 Jul 2016, Dirk Behme wrote: On 13.07.2016 00:26, Michael Turquette wrote: Quoting Dirk Behme (2016-0

Re: [Xen-devel] Converting heap page_infos to contiguous virtual

2016-07-14 Thread George Dunlap
On 13/07/16 21:57, Boris Ostrovsky wrote: > On 07/13/2016 04:34 PM, Andrew Cooper wrote: >> On 13/07/2016 21:17, Boris Ostrovsky wrote: >>> On 07/13/2016 04:02 PM, Andrew Cooper wrote: On 13/07/16 20:44, Boris Ostrovsky wrote: > I would like to clear a bunch of Xen heap pages at once (i.e.

Re: [Xen-devel] [PATCH v4] xen/arm: Add a clock property

2016-07-14 Thread Stefano Stabellini
On Wed, 13 Jul 2016, Michael Turquette wrote: > Quoting Dirk Behme (2016-07-13 11:56:30) > > On 13.07.2016 20:43, Stefano Stabellini wrote: > > > On Wed, 13 Jul 2016, Dirk Behme wrote: > > >> On 13.07.2016 00:26, Michael Turquette wrote: > > >>> Quoting Dirk Behme (2016-07-12 00:46:45) > > Clo

Re: [Xen-devel] [PATCH v4] xen/arm: Add a clock property

2016-07-14 Thread Stefano Stabellini
On Thu, 14 Jul 2016, Dirk Behme wrote: > On 13.07.2016 21:07, Stefano Stabellini wrote: > > On Wed, 13 Jul 2016, Dirk Behme wrote: > > > On 13.07.2016 20:35, Stefano Stabellini wrote: > > > > On Tue, 12 Jul 2016, Dirk Behme wrote: > > > > > Clocks described by this property are reserved for use by

Re: [Xen-devel] [PATCH v4] xen/arm: Add a clock property

2016-07-14 Thread Dirk Behme
On 14.07.2016 12:14, Julien Grall wrote: Hi Dirk, On 14/07/16 07:31, Dirk Behme wrote: On 13.07.2016 23:03, Michael Turquette wrote: Quoting Dirk Behme (2016-07-13 11:56:30) On 13.07.2016 20:43, Stefano Stabellini wrote: On Wed, 13 Jul 2016, Dirk Behme wrote: On 13.07.2016 00:26, Michael Tu

Re: [Xen-devel] Converting heap page_infos to contiguous virtual

2016-07-14 Thread Andrew Cooper
On 14/07/16 11:25, George Dunlap wrote: > On 13/07/16 21:57, Boris Ostrovsky wrote: >> On 07/13/2016 04:34 PM, Andrew Cooper wrote: >>> On 13/07/2016 21:17, Boris Ostrovsky wrote: On 07/13/2016 04:02 PM, Andrew Cooper wrote: > On 13/07/16 20:44, Boris Ostrovsky wrote: >> I would like t

Re: [Xen-devel] [PATCH v4] xen/arm: Add a clock property

2016-07-14 Thread Stefano Stabellini
On Thu, 14 Jul 2016, Dirk Behme wrote: > On 13.07.2016 23:03, Michael Turquette wrote: > > Quoting Dirk Behme (2016-07-13 11:56:30) > > > On 13.07.2016 20:43, Stefano Stabellini wrote: > > > > On Wed, 13 Jul 2016, Dirk Behme wrote: > > > > > On 13.07.2016 00:26, Michael Turquette wrote: > > > > > >

Re: [Xen-devel] [PATCH v3 2/2] qdisk - hw/block/xen_disk: grant copy implementation

2016-07-14 Thread Wei Liu
On Wed, Jun 22, 2016 at 10:38:53AM +0200, Paulina Szubarczyk wrote: > Copy data operated on during request from/to local buffers to/from > the grant references. > > Before grant copy operation local buffers must be allocated what is > done by calling ioreq_init_copy_buffers. For the 'read' operati

Re: [Xen-devel] [PATCH v4] xen/arm: Add a clock property

2016-07-14 Thread Dirk Behme
On 14.07.2016 12:38, Stefano Stabellini wrote: On Thu, 14 Jul 2016, Dirk Behme wrote: On 13.07.2016 23:03, Michael Turquette wrote: Quoting Dirk Behme (2016-07-13 11:56:30) On 13.07.2016 20:43, Stefano Stabellini wrote: On Wed, 13 Jul 2016, Dirk Behme wrote: On 13.07.2016 00:26, Michael Turq

Re: [Xen-devel] [PATCH v3 7/8] arm/atomic.h: remove '__' prefix for __atomic_add_unless()

2016-07-14 Thread Stefano Stabellini
On Thu, 14 Jul 2016, Corneliu ZUZU wrote: > On 7/14/2016 1:00 PM, Julien Grall wrote: > > Hi Corneliu, > > > > On 14/07/16 10:14, Corneliu ZUZU wrote: > > > The built-in leading underscores ('__') don't serve any purpose, so rename > > > __atomic_add_unless() -> atomic_add_unless(). > > > > The l

Re: [Xen-devel] [PATCH v3 6/8] asm-arm/atomic.h: atomic_{inc, dec}_return: macros to inline functions

2016-07-14 Thread Stefano Stabellini
On Thu, 14 Jul 2016, Corneliu ZUZU wrote: > Turn atomic_inc_return and atomic_dec_return atomic.h macros to inline > functions. > > Signed-off-by: Corneliu ZUZU Reviewed-by: Stefano Stabellini > xen/include/asm-arm/atomic.h | 13 ++--- > 1 file changed, 10 insertions(+), 3 deletions(

Re: [Xen-devel] [PATCH v3 4/8] asm/atomic.h: common prototyping (add xen/atomic.h)

2016-07-14 Thread Stefano Stabellini
On Thu, 14 Jul 2016, Corneliu ZUZU wrote: > Create a common-side to establish, among others, prototypes of > atomic functions called from common-code. Done to avoid introducing > inconsistencies between arch-side headers when we make subtle > changes to one of them. > > Some arm-side macros had

Re: [Xen-devel] [PATCH 1/9] xen/arm: Simply the definition of PAGE_SIZE by using the macro _AC

2016-07-14 Thread Stefano Stabellini
On Wed, 22 Jun 2016, Julien Grall wrote: > The macro _AC is used to define constant for both assembly and C. > > Signed-off-by: Julien Grall Reviewed-by: Stefano Stabellini > xen/include/asm-arm/config.h | 7 +-- > 1 file changed, 1 insertion(+), 6 deletions(-) > > diff --git a/xen/incl

Re: [Xen-devel] [PATCH 2/9] xen/arm: traps: Second attempt to correctly use the content of HPFAR_EL2

2016-07-14 Thread Stefano Stabellini
On Wed, 22 Jun 2016, Julien Grall wrote: > Commit c051618 "xen/arm: traps: Correctly interpret the content of the > register HPFAR_EL2" attempted to fix the interpretation of HPFAR_EL2. > > However, the register contains a 4KB-aligned address. This means that > the reported address is not directly

Re: [Xen-devel] [PATCH 3/9] xen/arm: traps: Data Abort are always unconditional

2016-07-14 Thread Stefano Stabellini
On Wed, 22 Jun 2016, Julien Grall wrote: > The HSR encoding for an exception from a data abort does not contain a > conditional code (see G6-4264 in ARM DDI 0487A.i) because they are > always conditional. > > So drop the pointless condition check. > > Signed-off-by: Julien Grall Reviewed-by: St

Re: [Xen-devel] [PATCH 4/9] xen/arm: traps: Simplify the switch in do_trap_*_abort_guest

2016-07-14 Thread Stefano Stabellini
On Wed, 22 Jun 2016, Julien Grall wrote: > The fault status we care are all the form xx where xx is the lookup ^ in the form of > level that gave the fault. We can simply the code by masking the 2 least ^ simplify > sig

Re: [Xen-devel] [PATCH v2 16/17] libxc/xc_dom_arm: Copy ACPI tables to guest space

2016-07-14 Thread Wei Liu
On Wed, Jul 13, 2016 at 01:08:57PM -0400, Boris Ostrovsky wrote: > On 07/13/2016 11:22 AM, Julien Grall wrote: > > Hello, > > > > On 12/07/2016 17:58, Boris Ostrovsky wrote: > >> On 07/12/2016 12:10 PM, Julien Grall wrote: > >>> On 12/07/2016 16:08, Boris Ostrovsky wrote: > On 07/12/2016 10:57

[Xen-devel] [qemu-mainline test] 97282: regressions - trouble: broken/fail/pass

2016-07-14 Thread osstest service owner
flight 97282 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/97282/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt-qemuu-debianhvm-amd64-xsm 3 host-install(3) broken REGR. vs. 96791 test-

Re: [Xen-devel] [PATCH 4/9] xen/arm: traps: Simplify the switch in do_trap_*_abort_guest

2016-07-14 Thread Julien Grall
Hi Stefano, On 14/07/16 12:12, Stefano Stabellini wrote: On Wed, 22 Jun 2016, Julien Grall wrote: The fault status we care are all the form xx where xx is the lookup ^ in the form of level that gave the fault. We can simply the code by masking the 2 lea

Re: [Xen-devel] [PATCH v2 16/17] libxc/xc_dom_arm: Copy ACPI tables to guest space

2016-07-14 Thread Julien Grall
Hello, On 13/07/16 18:08, Boris Ostrovsky wrote: On 07/13/2016 11:22 AM, Julien Grall wrote: On 12/07/2016 17:58, Boris Ostrovsky wrote: The config file specifies resources provided by the host. How the guest views those resources is not important, I think. This would need to be clarified. F

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

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

Re: [Xen-devel] [BUG] kernel BUG at drivers/block/xen-blkfront.c:1711

2016-07-14 Thread Evgenii Shatokhin
On 11.07.2016 15:04, Bob Liu wrote: On 07/11/2016 04:50 PM, Evgenii Shatokhin wrote: On 06.06.2016 11:42, Dario Faggioli wrote: Just Cc-ing some Linux, block, and Xen on CentOS people... Ping. Any suggestions how to debug this or what might cause the problem? Obviously, we cannot control

Re: [Xen-devel] [BUG] kernel BUG at drivers/block/xen-blkfront.c:1711

2016-07-14 Thread Bob Liu
On 07/14/2016 07:49 PM, Evgenii Shatokhin wrote: > On 11.07.2016 15:04, Bob Liu wrote: >> >> >> On 07/11/2016 04:50 PM, Evgenii Shatokhin wrote: >>> On 06.06.2016 11:42, Dario Faggioli wrote: Just Cc-ing some Linux, block, and Xen on CentOS people... >>> >>> Ping. >>> >>> Any suggestions

Re: [Xen-devel] [Qemu-block] [PATCH 05/17] block: Convert BB interface to byte-based discards

2016-07-14 Thread Stefan Hajnoczi
On Wed, Jun 22, 2016 at 09:51:02AM -0600, Eric Blake wrote: > Change sector-based blk_discard(), blk_co_discard(), and > blk_aio_discard() to instead be byte-based *_pdiscard() > functions. NBD gets a lot simpler now that ignoring the > unaligned portion of a byte-based discard request is handled

Re: [Xen-devel] Converting heap page_infos to contiguous virtual

2016-07-14 Thread Julien Grall
Hi, On 14/07/16 11:34, Andrew Cooper wrote: On 14/07/16 11:25, George Dunlap wrote: On 13/07/16 21:57, Boris Ostrovsky wrote: On 07/13/2016 04:34 PM, Andrew Cooper wrote: On 13/07/2016 21:17, Boris Ostrovsky wrote: On 07/13/2016 04:02 PM, Andrew Cooper wrote: On 13/07/16 20:44, Boris Ostrov

Re: [Xen-devel] [BUG] kernel BUG at drivers/block/xen-blkfront.c:1711

2016-07-14 Thread Evgenii Shatokhin
On 14.07.2016 15:04, Bob Liu wrote: On 07/14/2016 07:49 PM, Evgenii Shatokhin wrote: On 11.07.2016 15:04, Bob Liu wrote: On 07/11/2016 04:50 PM, Evgenii Shatokhin wrote: On 06.06.2016 11:42, Dario Faggioli wrote: Just Cc-ing some Linux, block, and Xen on CentOS people... Ping. Any sugg

[Xen-devel] Linux 3.18 and Linux 4.1 reproducible OOM crashes

2016-07-14 Thread Ian Jackson
osstest service owner writes ("[linux-3.18 test] 97278: regressions - FAIL"): > flight 97278 linux-3.18 real [real] > http://logs.test-lab.xenproject.org/osstest/logs/97278/ > > Regressions :-( > > Tests which did not succeed and are blocking, > including tests which could not be run: ... > test

[Xen-devel] [distros-debian-wheezy test] 66566: tolerable trouble: blocked/broken

2016-07-14 Thread Platform Team regression test user
flight 66566 distros-debian-wheezy real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/66566/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): build-armhf-pvops 3 host-install(3) broken like 44432 build-armhf

Re: [Xen-devel] [PATCH 0/8] x86: audit and remove needless module.h includes

2016-07-14 Thread Ingo Molnar
* Paul Gortmaker wrote: > To that end, I have done allmodconfig, allyesconfig and allnoconfig > for both 32 bit and 64 bit x86 with these changes on the linux-next > from today, which presumably has an up to date copy of tip in it. It does, still I get this on allnoconfig with your patches appl

Re: [Xen-devel] Converting heap page_infos to contiguous virtual

2016-07-14 Thread Andrew Cooper
On 14/07/16 13:42, Julien Grall wrote: > Hi, > > On 14/07/16 11:34, Andrew Cooper wrote: >> On 14/07/16 11:25, George Dunlap wrote: >>> On 13/07/16 21:57, Boris Ostrovsky wrote: On 07/13/2016 04:34 PM, Andrew Cooper wrote: > On 13/07/2016 21:17, Boris Ostrovsky wrote: >> On 07/13/2016

Re: [Xen-devel] [PATCH] Add optional ACPI device for Windows Continuum

2016-07-14 Thread Andrew Cooper
On 13/07/16 10:16, Owen Smith wrote: > Windows 10 supports a specific ACPI device for handling the > switch between tablet mode and desktop mode. The meer existance > of this device is the mimimum to allow tablet/desktop mode to > be switched. > Tablet mode referes to the "undocked" state where all

Re: [Xen-devel] Converting heap page_infos to contiguous virtual

2016-07-14 Thread Andrew Cooper
On 13/07/16 22:43, Boris Ostrovsky wrote: > On 07/13/2016 05:06 PM, Andrew Cooper wrote: >> On 13/07/2016 21:57, Boris Ostrovsky wrote: >>> On 07/13/2016 04:34 PM, Andrew Cooper wrote: On 13/07/2016 21:17, Boris Ostrovsky wrote: > On 07/13/2016 04:02 PM, Andrew Cooper wrote: >> On 13/0

Re: [Xen-devel] [PATCH 4/9] xen/arm: traps: Simplify the switch in do_trap_*_abort_guest

2016-07-14 Thread Stefano Stabellini
On Thu, 14 Jul 2016, Julien Grall wrote: > Hi Stefano, > > On 14/07/16 12:12, Stefano Stabellini wrote: > > On Wed, 22 Jun 2016, Julien Grall wrote: > > > The fault status we care are all the form xx where xx is the lookup > > > > ^ in the form of > > > > >

Re: [Xen-devel] [PATCH v2 16/17] libxc/xc_dom_arm: Copy ACPI tables to guest space

2016-07-14 Thread Stefano Stabellini
On Wed, 13 Jul 2016, Julien Grall wrote: > Hello, > > On 12/07/2016 17:58, Boris Ostrovsky wrote: > > On 07/12/2016 12:10 PM, Julien Grall wrote: > > > On 12/07/2016 16:08, Boris Ostrovsky wrote: > > > > On 07/12/2016 10:57 AM, Shannon Zhao wrote: > > > > > On 2016年07月12日 22:50, Wei Liu wrote: > >

[Xen-devel] [xen-unstable baseline-only test] 66565: tolerable FAIL

2016-07-14 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 66565 xen-unstable real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/66565/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): build-i386-rumpuserxen6 xen-build

Re: [Xen-devel] [PATCH 5/9] xen/arm: Provide macros to help creating workaround helpers

2016-07-14 Thread Stefano Stabellini
On Wed, 22 Jun 2016, Julien Grall wrote: > Workarounds may require to execute a different path when the platform > is affected by the associated erratum. Furthermore, this may need to > be called in the common code. > > To avoid too much intrusion/overhead, the workaround helpers need to > be a no

Re: [Xen-devel] [PATCH V3 04/10] arm/gic-v3: Parse per-cpu redistributor entry in GICC subtable

2016-07-14 Thread Julien Grall
Hi Shanker, On 27/06/16 21:33, Shanker Donthineni wrote: The redistributor address can be specified either as part of GICC or GICR subtable depending on the power domain. The current driver doesn't support parsing redistributor entry that is defined in GICC subtable. The GIC CPU subtable entry h

Re: [Xen-devel] [PATCH V3 01/10] arm/gic-v3: Use acpi_table_parse_madt() to parse MADT subtables

2016-07-14 Thread Stefano Stabellini
On Mon, 27 Jun 2016, Shanker Donthineni wrote: > The function acpi_table_parse_madt() does the same functionality as > function acpi_parse_entries() expect it takes a few arguments. > > Signed-off-by: Shanker Donthineni I committed patches 1 to 7 > xen/arch/arm/gic-v3.c | 27 ++---

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

2016-07-14 Thread osstest service owner
flight 97286 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/97286/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-libvirt5 libvirt-build fail REGR. vs. 97003 build-armhf-libvirt

[Xen-devel] [PATCH v6 1/2] xsm: rework policy_buffer globals

2016-07-14 Thread Daniel De Graaf
This makes the buffers function parameters instead of globals, in preparation for adding alternate locations for the policy. Signed-off-by: Daniel De Graaf Reviewed-by: Jan Beulich --- Changes since v5: - Adjusted __init annotation placement - Removed unneeded cast to char* xen/include/xsm/

[Xen-devel] [PATCH v6 2/2] xsm: add a default policy to .init.data

2016-07-14 Thread Daniel De Graaf
This adds a Kconfig option and support for including the XSM policy from tools/flask/policy in the hypervisor so that the bootloader does not need to provide a policy to get sane behavior from an XSM-enabled hypervisor. The policy provided by the bootloader, if present, will override the built-in

Re: [Xen-devel] [PATCH v6 1/2] xsm: rework policy_buffer globals

2016-07-14 Thread Andrew Cooper
On 14/07/16 15:18, Daniel De Graaf wrote: > This makes the buffers function parameters instead of globals, in > preparation for adding alternate locations for the policy. > > Signed-off-by: Daniel De Graaf > Reviewed-by: Jan Beulich Reviewed-by: Andrew Cooper __

Re: [Xen-devel] [PATCH v6 2/2] xsm: add a default policy to .init.data

2016-07-14 Thread Andrew Cooper
On 14/07/16 15:18, Daniel De Graaf wrote: > This adds a Kconfig option and support for including the XSM policy from > tools/flask/policy in the hypervisor so that the bootloader does not > need to provide a policy to get sane behavior from an XSM-enabled > hypervisor. The policy provided by the b

Re: [Xen-devel] [PATCH 6/9] xen/arm: Use check_workaround to handle the erratum 766422

2016-07-14 Thread Stefano Stabellini
On Wed, 22 Jun 2016, Julien Grall wrote: > Currently, Xen is reading the MIDR everytime it has to check whether > the processor is affected by the erratum 766422. > > This could take advantage of the new capability bitfields to detect > whether the processor is affected at boot time. > > With thi

Re: [Xen-devel] [PATCH 6/9] xen/arm: Use check_workaround to handle the erratum 766422

2016-07-14 Thread Julien Grall
Hi Stefano, On 14/07/16 15:34, Stefano Stabellini wrote: On Wed, 22 Jun 2016, Julien Grall wrote: Currently, Xen is reading the MIDR everytime it has to check whether the processor is affected by the erratum 766422. This could take advantage of the new capability bitfields to detect whether th

Re: [Xen-devel] [PATCH] Add optional ACPI device for Windows Continuum

2016-07-14 Thread Owen Smith
> -Original Message- > From: Andrew Cooper [mailto:andrew.coop...@citrix.com] > Sent: 14 July 2016 14:21 > To: Owen Smith; xen-de...@lists.xenproject.org > Subject: Re: [Xen-devel] [PATCH] Add optional ACPI device for Windows > Continuum > > On 13/07/16 10:16, Owen Smith wrote: > > Window

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

2016-07-14 Thread osstest service owner
flight 97302 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/97302/ 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] [PATCH v2] XSM-Policy: allow source domain access to setpodtarget and getpodtarget for ballooning.

2016-07-14 Thread Anshul Makkar
Access to setpodtarget and getpodtarget is required by dom0 to set the balloon targets for domU. The patch gives source domain (dom0) access to set this target for domU and resolve the following permission denied erro message during ballooning : avc: denied { setpodtarget } for domid=0 target=9 s

Re: [Xen-devel] [PATCH 1/2] xen: fix a (latent) cpupool-related race during domain destroy

2016-07-14 Thread Dario Faggioli
On Thu, 2016-07-14 at 10:37 +0100, Andrew Cooper wrote: > On 14/07/16 07:41, Dario Faggioli wrote: > > > > So, during domain destruction, we do: > >  cpupool_rm_domain()[ in domain_destroy() ] > >  sched_destroy_domain() [ in complete_domain_destroy() ] > > > > Therefore, there's a window dur

Re: [Xen-devel] [PATCH 1/2] xen: fix a (latent) cpupool-related race during domain destroy

2016-07-14 Thread Andrew Cooper
On 14/07/16 15:54, Dario Faggioli wrote: > On Thu, 2016-07-14 at 10:37 +0100, Andrew Cooper wrote: >> On 14/07/16 07:41, Dario Faggioli wrote: >>> So, during domain destruction, we do: >>> cpupool_rm_domain()[ in domain_destroy() ] >>> sched_destroy_domain() [ in complete_domain_destroy() ] >

Re: [Xen-devel] [PATCH 7/9] xen/arm: traps: MMIO should only be emulated for fault translation

2016-07-14 Thread Stefano Stabellini
On Wed, 22 Jun 2016, Julien Grall wrote: > The function do_trap_data_abort_guest assumes that a stage-2 data abort > can only be taken for a translation fault or permission fault today. > > Whilst this is true today, it might not be in the future. Rather than > emulating the MMIO for any fault oth

Re: [Xen-devel] [PATCH v6 1/2] xsm: rework policy_buffer globals

2016-07-14 Thread Andrew Cooper
On 14/07/16 15:18, Daniel De Graaf wrote: > This makes the buffers function parameters instead of globals, in > preparation for adding alternate locations for the policy. > > Signed-off-by: Daniel De Graaf > Reviewed-by: Jan Beulich > --- Reviewed and committed both patches. ~Andrew __

Re: [Xen-devel] [PATCH v2] XSM-Policy: allow source domain access to setpodtarget and getpodtarget for ballooning.

2016-07-14 Thread Andrew Cooper
On 14/07/16 15:46, Anshul Makkar wrote: > Access to setpodtarget and getpodtarget is required by dom0 to set the balloon > targets for domU. The patch gives source domain (dom0) access to set > this target for domU and resolve the following permission denied erro > message during ballooning : > avc

Re: [Xen-devel] [PATCH 0/8] x86: audit and remove needless module.h includes

2016-07-14 Thread Paul Gortmaker
[Re: [PATCH 0/8] x86: audit and remove needless module.h includes] On 14/07/2016 (Thu 15:04) Ingo Molnar wrote: > > * Paul Gortmaker wrote: > > > To that end, I have done allmodconfig, allyesconfig and allnoconfig > > for both 32 bit and 64 bit x86 with these changes on the linux-next > > from

Re: [Xen-devel] [PATCH 8/9] xen/arm: traps: Avoid unnecessary VA -> IPA translation in abort handlers

2016-07-14 Thread Stefano Stabellini
On Wed, 22 Jun 2016, Julien Grall wrote: > Translating a VA to a IPA is expensive. Currently, Xen is assuming that > HPFAR_EL2 is only valid when the stage-2 data/instruction abort happened > during a translation table walk of a first stage translation (i.e S1PTW > is set). > > However, based on t

Re: [Xen-devel] [PATCH 7/9] xen/arm: traps: MMIO should only be emulated for fault translation

2016-07-14 Thread Stefano Stabellini
On Thu, 14 Jul 2016, Julien Grall wrote: > Hi Stefano, > > On 14/07/16 16:06, Stefano Stabellini wrote: > > On Wed, 22 Jun 2016, Julien Grall wrote: > > > -if (handle_mmio(&info)) > > > -{ > > > -advance_pc(regs, hsr); > > > -return; > > > +if ( handle_mmio(&info) )

Re: [Xen-devel] [PATCH 7/9] xen/arm: traps: MMIO should only be emulated for fault translation

2016-07-14 Thread Julien Grall
On 14/07/16 16:28, Stefano Stabellini wrote: On Thu, 14 Jul 2016, Julien Grall wrote: Hi Stefano, On 14/07/16 16:06, Stefano Stabellini wrote: On Wed, 22 Jun 2016, Julien Grall wrote: -if (handle_mmio(&info)) -{ -advance_pc(regs, hsr); -return; +if ( handle_m

Re: [Xen-devel] [PATCH 7/9] xen/arm: traps: MMIO should only be emulated for fault translation

2016-07-14 Thread Julien Grall
Hi Stefano, On 14/07/16 16:06, Stefano Stabellini wrote: On Wed, 22 Jun 2016, Julien Grall wrote: -if (handle_mmio(&info)) -{ -advance_pc(regs, hsr); -return; +if ( handle_mmio(&info) ) +{ +advance_pc(regs, hsr); +return; +

Re: [Xen-devel] [PATCH V3 01/10] arm/gic-v3: Use acpi_table_parse_madt() to parse MADT subtables

2016-07-14 Thread Shanker Donthineni
Hi Stefano/Juilen On 07/14/2016 09:18 AM, Stefano Stabellini wrote: On Mon, 27 Jun 2016, Shanker Donthineni wrote: The function acpi_table_parse_madt() does the same functionality as function acpi_parse_entries() expect it takes a few arguments. Signed-off-by: Shanker Donthineni I committed

Re: [Xen-devel] [PATCH 8/9] xen/arm: traps: Avoid unnecessary VA -> IPA translation in abort handlers

2016-07-14 Thread Julien Grall
On 14/07/16 16:27, Stefano Stabellini wrote: On Wed, 22 Jun 2016, Julien Grall wrote: diff --git a/xen/arch/arm/traps.c b/xen/arch/arm/traps.c index 591de3c..0edc2cc 100644 --- a/xen/arch/arm/traps.c +++ b/xen/arch/arm/traps.c @@ -2383,13 +2383,28 @@ static inline paddr_t get_faulting_ipa(vadd

Re: [Xen-devel] [v9 00/19] QEMU:Xen stubdom vTPM for HVM virtual machine(QEMU Part)

2016-07-14 Thread Stefano Stabellini
Hi Quan, thanks for CC'ing me. sstabell...@kernel.org is the right address to reach me now. I am also CC'ing Anthony Perard who is Xen co-maintainer in QEMU. Cheers, Stefano On Wed, 13 Jul 2016, Xu, Quan wrote: > Emil, Thanks for your effort ( today I just come back to return my laptop). > >

Re: [Xen-devel] [PATCH 8/9] xen/arm: traps: Avoid unnecessary VA -> IPA translation in abort handlers

2016-07-14 Thread Stefano Stabellini
On Thu, 14 Jul 2016, Julien Grall wrote: > On 14/07/16 16:27, Stefano Stabellini wrote: > > On Wed, 22 Jun 2016, Julien Grall wrote: > > > diff --git a/xen/arch/arm/traps.c b/xen/arch/arm/traps.c > > > index 591de3c..0edc2cc 100644 > > > --- a/xen/arch/arm/traps.c > > > +++ b/xen/arch/arm/traps.c >

Re: [Xen-devel] [PATCH 9/9] xen/arm: arm64: Add Cortex-A57 erratum 834220 workaround

2016-07-14 Thread Stefano Stabellini
On Wed, 22 Jun 2016, Julien Grall wrote: > The ARM erratum applies to certain revisions of Cortex-A57. The > processor may report a Stage 2 translation fault as the result of > Stage 1 fault for load crossing a page boundary when there is a > permission fault or device memory fault at stage 1 and a

[Xen-devel] OVMF very slow on AMD

2016-07-14 Thread Anthony PERARD
Hi, I've been investigating why OVMF is very slow in a Xen guest on an AMD host. This, I think, is the current failure that osstest is having. I've only look at a specific part of OVMF where the slowdown is very obvious on AMD vs Intel, the decompression. This is what I get on AMD, via the Xen

Re: [Xen-devel] [PATCH v4] xen/arm: Add a clock property

2016-07-14 Thread Stefano Stabellini
On Thu, 14 Jul 2016, Dirk Behme wrote: > On 14.07.2016 12:38, Stefano Stabellini wrote: > > On Thu, 14 Jul 2016, Dirk Behme wrote: > > > On 13.07.2016 23:03, Michael Turquette wrote: > > > > Quoting Dirk Behme (2016-07-13 11:56:30) > > > > > On 13.07.2016 20:43, Stefano Stabellini wrote: > > > > >

  1   2   >