Re: [Xen-devel] [PATCH] xen/blkback: use rb_entry()

2016-12-20 Thread Roger Pau Monné
On Tue, Dec 20, 2016 at 11:47:03AM -0500, Konrad Rzeszutek Wilk wrote: > On Tue, Dec 20, 2016 at 10:02:19PM +0800, Geliang Tang wrote: > > To make the code clearer, use rb_entry() instead of container_of() to > > deal with rbtree. > > That is OK but I think 'container_of' is more clear. > >

Re: [Xen-devel] [PATCH 05/10] x86/HVM: use unambiguous register names

2016-12-20 Thread Andrew Cooper
On 20/12/2016 10:39, Jan Beulich wrote: > @@ -3032,16 +3032,16 @@ void hvm_task_switch( > if ( hvm_set_cr3(tss.cr3, 1) ) > goto out; > > -regs->eip= tss.eip; > -regs->eflags = tss.eflags | 2; > -regs->eax= tss.eax; > -regs->ecx= tss.ecx; > -regs->edx

Re: [Xen-devel] [PATCH] xen/blkback: use rb_entry()

2016-12-20 Thread Konrad Rzeszutek Wilk
On Tue, Dec 20, 2016 at 05:44:06PM +, Roger Pau Monné wrote: > On Tue, Dec 20, 2016 at 11:47:03AM -0500, Konrad Rzeszutek Wilk wrote: > > On Tue, Dec 20, 2016 at 10:02:19PM +0800, Geliang Tang wrote: > > > To make the code clearer, use rb_entry() instead of container_of() to > > > deal with

[Xen-devel] [xen-4.6-testing test] 103765: regressions - trouble: broken/fail/pass

2016-12-20 Thread osstest service owner
flight 103765 xen-4.6-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/103765/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-arndale 9 debian-install fail REGR. vs. 103407

Re: [Xen-devel] [PATCH v2 2/5] config/StdGNU.mk: Allows users to pass in EXTRA_CFLAGS

2016-12-20 Thread Doug Goldstein
On 12/20/16 1:46 PM, Alistair Francis wrote: > Signed-off-by: Alistair Francis > --- > config/StdGNU.mk | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/config/StdGNU.mk b/config/StdGNU.mk > index 6be8233..a6cdd82 100644 > --- a/config/StdGNU.mk > +++

[Xen-devel] [RFC 2/7] xen: Move xen files to accel/

2016-12-20 Thread Eduardo Habkost
Cc: Stefano Stabellini Cc: Anthony Perard Cc: xen-de...@lists.xensource.com Signed-off-by: Eduardo Habkost --- Makefile.target| 4 +--- xen-common.c => accel/xen-common.c | 0 xen-hvm.c =>

[Xen-devel] [RFC 0/7] Move accel, KVM, Xen, qtest files to accel/ subdir

2016-12-20 Thread Eduardo Habkost
This moves the KVM and Xen files to the an accel/ subdir. Instead of moving the *-stubs.c file to accel/ as-is, I tried to move most of the stub code to libqemustub.a. This way the obj-y logic for accel/ is simpler: obj-y includes accel/ only if CONFIG_SOFTMMU is set. The Xen stubs could be

[Xen-devel] [OSSTEST PATCH 6/9] db retry: Document $@ as an implicit parameter to need_retry

2016-12-20 Thread Ian Jackson
There are only two call sites and neither trashes $@ right now. We are going to use a more exception-friendly style. Signed-off-by: Ian Jackson --- Osstest/JobDB/Executive.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Xen-devel] [OSSTEST PATCH 4/9] db retry, bisection: Reset %jobs_created on db retry

2016-12-20 Thread Ian Jackson
%jobs_created is used for memoisation while populating the destination flight. We need to reset it when we restart flight construction, because those jobs were created in the discarded transaction. Otherwise we could create a flight with missing jobs. Signed-off-by: Ian Jackson

[Xen-devel] [OSSTEST PATCH 1/9] mg-schema-test-database: Revamp sequence handling

2016-12-20 Thread Ian Jackson
The initial value (at creation time) of a sequence appears in the schema, but is not of any consequence. To avoid the schema diff check failing in databases created in a slightly different way, it is necessary to copy the actual original initial sequence value for each sequence. Replace the

[Xen-devel] [OSSTEST PATCH 8/9] db retry: Use HandleError and exceptions to detect when to retry

2016-12-20 Thread Ian Jackson
It appears that sometimes, $dbh->state could be overwritten before $mjobdb->need_retry got to run. $dbh->err and $@ would still be right. I have not been able to explain this; I suspect that there is something exciting going on in the eval exception trapping. To try to isolate the problem, I

[Xen-devel] [OSSTEST PATCH 2/9] mg-schema-test-database: Wrap some withtest psql_do in subshells

2016-12-20 Thread Ian Jackson
Otherwise it takes effect for the rest of the script, which is not what is wanted ! As it happens, there are no accesses to the real db after this point, so this bug is latent. Signed-off-by: Ian Jackson --- mg-schema-test-database | 7 --- 1 file changed, 4

[Xen-devel] [PATCH v2] libacpi: Don't build x86-only AML for ARM64 mk_dsdt

2016-12-20 Thread Boris Ostrovsky
Commit d6ac8e22c7c5 ("acpi/x86: define ACPI IO registers for PVH guests") broke ARM64 build of mk_dsdt.c due to introduction of XEN_ACPI_CPU_MAP[_LEN] macros that are needed only for x86 guests. We could fix the build by dealing specifically with those macros but since post-MADT code is not

Re: [Xen-devel] Xen ARM - Exposing a PL011 to the guest

2016-12-20 Thread Stefano Stabellini
On Tue, 20 Dec 2016, Julien Grall wrote: > Hi Stefano, > > On 19/12/2016 21:24, Stefano Stabellini wrote: > > On Mon, 19 Dec 2016, Christoffer Dall wrote: > > > On Fri, Dec 16, 2016 at 05:03:13PM +, Julien Grall wrote: > > > > (CC rest maintainers for event channel questions) > > > > > > > >

Re: [Xen-devel] [PATCH v2 4/5] tools/blktap2: Fix missing header file

2016-12-20 Thread Konrad Rzeszutek Wilk
On Tue, Dec 20, 2016 at 11:46:59AM -0800, Alistair Francis wrote: > To avoid build errors relating to missing delcarations of ssize_t add > the appripriote header file to atomic.h. appropiate. > > Signed-off-by: Alistair Francis > --- >

Re: [Xen-devel] [PATCH] xen/arm: fix rank/vgic locks inversion bug

2016-12-20 Thread Stefano Stabellini
On Tue, 20 Dec 2016, Julien Grall wrote: > Hi Stefano, > > On 20/12/2016 00:22, Stefano Stabellini wrote: > > On Mon, 19 Dec 2016, Julien Grall wrote: > > > Hi Stefano, > > > > > > On 19/12/2016 23:30, Stefano Stabellini wrote: > > > > On Mon, 19 Dec 2016, Julien Grall wrote: > > > > > > > 2) We

Re: [Xen-devel] [PATCH v2 4/5] tools/blktap2: Fix missing header file

2016-12-20 Thread Doug Goldstein
On 12/20/16 1:46 PM, Alistair Francis wrote: > To avoid build errors relating to missing delcarations of ssize_t add declarations > the appripriote header file to atomic.h. appropriate > > Signed-off-by: Alistair Francis > --- > tools/blktap2/include/atomicio.h

Re: [Xen-devel] [PATCH v2 0/5] General Build Fixes

2016-12-20 Thread Konrad Rzeszutek Wilk
On Tue, Dec 20, 2016 at 11:46:55AM -0800, Alistair Francis wrote: > This patch series is a list of build issues that appeared when > buildling Xen 4.8.0 in buildroot. Hopefully some of them can be Is there an corresponding patch in buildroot for using Xen? Thank you for reposting! > accepted

Re: [Xen-devel] [PATCH 4/7] tools: Update sys/poll.h to poll.h

2016-12-20 Thread Alistair Francis
On Mon, Dec 19, 2016 at 8:00 PM, Doug Goldstein wrote: > On 12/19/16 12:01 PM, Alistair Francis wrote: >> On Sat, Dec 17, 2016 at 7:55 AM, Konrad Rzeszutek Wilk >> wrote: >>> On Fri, Dec 16, 2016 at 02:56:04PM -0800, Alistair Francis wrote: To

[Xen-devel] [PATCH] libacpi: Don't build x86-only AML for ARM64 mk_dsdt

2016-12-20 Thread Boris Ostrovsky
Commit d6ac8e22c7c5 ("acpi/x86: define ACPI IO registers for PVH guests") broke ARM64 build of mk_dsdt.c due to introduction of XEN_ACPI_CPU_MAP[_LEN] macros that are needed only for x86 guests. We could fix the build by dealing specifically with those macros but since post-MADT code is not

Re: [Xen-devel] [Qemu-devel] [RFC 0/7] Move accel, KVM, Xen, qtest files to accel/ subdir

2016-12-20 Thread Stefan Weil
On 12/20/16 18:43, Eduardo Habkost wrote: This moves the KVM and Xen files to the an accel/ subdir. I hope that it won't last long until we also get acceleration for Windows and macOS. Those HAXM files will have to be moved as well. Regards, Stefan

Re: [Xen-devel] Xen ARM community call - meeting minutes and date for the next one

2016-12-20 Thread Andrii Anisov
Sorry for the mess, I mean the xen-swiotlb issue on renesas board: > Bosch: problem with xen-swiotlb. It does not work properly on renesas board. > Stefano: please report the error on the ML > > ACTION: Bosch to send a bug report regarding xen-swiotlb -- *Andrii Anisov* *Lead Systems

[Xen-devel] [libvirt test] 103766: regressions - trouble: broken/fail/pass

2016-12-20 Thread osstest service owner
flight 103766 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/103766/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt-xsm 3 host-install(3)broken REGR. vs. 103479

Re: [Xen-devel] [PATCH v5 02/13] acpi/x86: Define ACPI IO registers for PVH guests

2016-12-20 Thread Julien Grall
Hello Boris, This patch is breaking compilation of mk_dsdt on ARM64 (see below). Boris can you please send a patch to fix this? In the future, please make sure that mk_dsdt at least build for all the targeted architectures. mk_dsdt.c: In function 'main': mk_dsdt.c:249:9: error:

Re: [Xen-devel] Xen ARM community call - meeting minutes and date for the next one

2016-12-20 Thread Andrii Anisov
Julien, Stefano, Are there any updates about: ACTION: Bosch to send a bug report regarding xen-swiotlb Edgar: IOMMU could not be used by the guest (Stage-1). This would be useful to implement driver in userspace. Julien: When will it be required? Edgar: It is a trend Any mailing

[Xen-devel] [RFC 1/7] xen: Move xen-*-stub.c to stubs/

2016-12-20 Thread Eduardo Habkost
Move xen stubs to stubs/ so they are handled automatically by libqemustub.a. Cc: Stefano Stabellini Cc: Anthony Perard Cc: xen-de...@lists.xensource.com Cc: Paolo Bonzini Signed-off-by: Eduardo Habkost

Re: [Xen-devel] [PATCH 1/7] Remove hardcoded strict -Werror checking

2016-12-20 Thread Alistair Francis
On Mon, Dec 19, 2016 at 7:53 PM, Doug Goldstein wrote: > On 12/17/16 9:51 AM, Konrad Rzeszutek Wilk wrote: >> On Fri, Dec 16, 2016 at 02:56:01PM -0800, Alistair Francis wrote: >>> Signed-off-by: Alistair Francis >> >> >> Why? > > *adjusts his

Re: [Xen-devel] [PATCH 1/7] Remove hardcoded strict -Werror checking

2016-12-20 Thread Alistair Francis
On Tue, Dec 20, 2016 at 9:21 AM, Konrad Rzeszutek Wilk wrote: > On Tue, Dec 20, 2016 at 11:02:15AM -0600, Doug Goldstein wrote: >> On 12/20/16 10:05 AM, Konrad Rzeszutek Wilk wrote: >> > On Mon, Dec 19, 2016 at 09:53:02PM -0600, Doug Goldstein wrote: >> >> On 12/17/16 9:51

[Xen-devel] [OSSTEST PATCH 3/9] cs-bisection-step: Do not acquire the repo lock

2016-12-20 Thread Ian Jackson
This is only required to avoid a lock inversion between the repo lock and database table locks, but we have no explicit database table locks any more. We do not want to hold the repo lock for an extended period, particularly when we are running a database retry loop. In practice, currently,

Re: [Xen-devel] [PATCH v2 3/5] tools/blktap2/vhd: Remove unused struct stat stats

2016-12-20 Thread Doug Goldstein
On 12/20/16 1:46 PM, Alistair Francis wrote: > The unsued variable 'struct stat stats' causes build errors in some > situations. As it isn't used just remove it. > > Signed-off-by: Alistair Francis > --- > tools/blktap2/vhd/lib/libvhd-journal.c | 1 - > 1 file

Re: [Xen-devel] Xen ARM community call - meeting minutes and date for the next one

2016-12-20 Thread Julien Grall
Hi Andrii, On 20/12/2016 19:00, Andrii Anisov wrote: Sorry for the mess, I mean the xen-swiotlb issue on renesas board: Bosch: problem with xen-swiotlb. It does not work properly on renesas board. Stefano: please report the error on the ML ACTION: Bosch to send a bug report regarding

Re: [Xen-devel] Xen ARM community call - meeting minutes and date for the next one

2016-12-20 Thread Konrad Rzeszutek Wilk
On Tue, Dec 20, 2016 at 08:00:27PM +0200, Andrii Anisov wrote: > Sorry for the mess, > > I mean the xen-swiotlb issue on renesas board: Can you make sure to CC me on it (since I am the maintainer of that code). > > > Bosch: problem with xen-swiotlb. It does not work properly on renesas > board.

[Xen-devel] [OSSTEST PATCH 0/9] db retry: fixes and workarounds

2016-12-20 Thread Ian Jackson
We have here: * Two patches to improve mg-schema-test-database a bit. * Fixes to cs-bisection-step, to survive and give right answers on db retry. * An attempt at a workarounds for a strange DBD::Pg behaviour. The workaround seems to WFM in my ad-hoc tests and I propose to deploy it in the

[Xen-devel] [OSSTEST PATCH 7/9] db retry: Break out Osstest::Executive::JobDB::_need_retry

2016-12-20 Thread Ian Jackson
We are going to want to reorganise this. As prep work, break the $dbh state checking (and the corresponding comment) into a separate sub. No functional changel. (There is still an anomaly: need_retry passes it $dbh_tests, not the $dbh it got from the caller. This will go away shortly.)

[Xen-devel] [OSSTEST PATCH 9/9] db retry: Retry on $dbh->state eq ''

2016-12-20 Thread Ian Jackson
This is supposed to represent success. But now that _need_retry is only called within a HandleError hook, we know there has been a failure. Retry such failures, in the hope that they are stochastic. If they aren't, we will fail eventually when we run out of retries. Signed-off-by: Ian Jackson

[Xen-devel] [OSSTEST PATCH 5/9] db retry, bisect: Cache build reuse investigations

2016-12-20 Thread Ian Jackson
If we previously searched for builds to reuse, trust our previous answers. We will only have seen data from committed transactions and we will only have looked at jobs in completed flights, which won't have changed. So any previously reuseable build is still reuseable. (Unless its stash check

[Xen-devel] [PATCH v2 1/5] Remove hardcoded strict -Werror checking

2016-12-20 Thread Alistair Francis
Signed-off-by: Alistair Francis --- Config.mk | 2 +- tools/blktap2/drivers/Makefile | 1 - tools/libxl/Makefile | 2 +- tools/xentrace/Makefile| 2 -- 4 files changed, 2 insertions(+), 5 deletions(-) diff --git a/Config.mk

[Xen-devel] [PATCH v2 4/5] tools/blktap2: Fix missing header file

2016-12-20 Thread Alistair Francis
To avoid build errors relating to missing delcarations of ssize_t add the appripriote header file to atomic.h. Signed-off-by: Alistair Francis --- tools/blktap2/include/atomicio.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/blktap2/include/atomicio.h

[Xen-devel] [PATCH v2 2/5] config/StdGNU.mk: Allows users to pass in EXTRA_CFLAGS

2016-12-20 Thread Alistair Francis
Signed-off-by: Alistair Francis --- config/StdGNU.mk | 3 +++ 1 file changed, 3 insertions(+) diff --git a/config/StdGNU.mk b/config/StdGNU.mk index 6be8233..a6cdd82 100644 --- a/config/StdGNU.mk +++ b/config/StdGNU.mk @@ -35,6 +35,9 @@ UTIL_LIBS = -lutil

[Xen-devel] [PATCH v2 5/5] tools/blktap2/drivers: Remove non-existent sys/sysctl.h include

2016-12-20 Thread Alistair Francis
To avoid build errors related to missing file 'sys/sysctl.h' by removing the #include statement. Signed-off-by: Alistair Francis --- tools/blktap2/drivers/block-remus.c | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/blktap2/drivers/block-remus.c

[Xen-devel] [PATCH v2 0/5] General Build Fixes

2016-12-20 Thread Alistair Francis
This patch series is a list of build issues that appeared when buildling Xen 4.8.0 in buildroot. Hopefully some of them can be accepted upstream to help others who are trying to build Xen in the future. V2: - Remove the #include path changes - It turns out this only applies to musl (although

[Xen-devel] [PATCH v2 3/5] tools/blktap2/vhd: Remove unused struct stat stats

2016-12-20 Thread Alistair Francis
The unsued variable 'struct stat stats' causes build errors in some situations. As it isn't used just remove it. Signed-off-by: Alistair Francis --- tools/blktap2/vhd/lib/libvhd-journal.c | 1 - 1 file changed, 1 deletion(-) diff --git

Re: [Xen-devel] [PATCH] xen/blkback: use rb_entry()

2016-12-20 Thread Konrad Rzeszutek Wilk
On Tue, Dec 20, 2016 at 01:53:21PM -0800, Eric Dumazet wrote: > On Tue, 2016-12-20 at 12:51 -0500, Konrad Rzeszutek Wilk wrote: > > On Tue, Dec 20, 2016 at 05:44:06PM +, Roger Pau Monné wrote: > > > On Tue, Dec 20, 2016 at 11:47:03AM -0500, Konrad Rzeszutek Wilk wrote: > > > > On Tue, Dec 20,

Re: [Xen-devel] Xen: ARM: Support to map mmio region specified in static ACPI tables

2016-12-20 Thread Stefano Stabellini
On Tue, 20 Dec 2016, Jan Beulich wrote: > >>> On 20.12.16 at 00:39, wrote: > > On Wed, 14 Dec 2016, Jiandi An wrote: > >> Xen currently does not handle mapping mmio regions specified in standard > > static ACPI tables such as BERT, TPM2, GT block, IORT, HEST, etc. There

Re: [Xen-devel] Xen ARM - Exposing a PL011 to the guest

2016-12-20 Thread Stefano Stabellini
On Tue, 20 Dec 2016, Stefano Stabellini wrote: > On Tue, 20 Dec 2016, Julien Grall wrote: > > Hi Stefano, > > > > On 19/12/2016 21:24, Stefano Stabellini wrote: > > > On Mon, 19 Dec 2016, Christoffer Dall wrote: > > > > On Fri, Dec 16, 2016 at 05:03:13PM +, Julien Grall wrote: > > > > > (CC

Re: [Xen-devel] [PATCH] xen/blkback: use rb_entry()

2016-12-20 Thread Eric Dumazet
On Tue, 2016-12-20 at 12:51 -0500, Konrad Rzeszutek Wilk wrote: > On Tue, Dec 20, 2016 at 05:44:06PM +, Roger Pau Monné wrote: > > On Tue, Dec 20, 2016 at 11:47:03AM -0500, Konrad Rzeszutek Wilk wrote: > > > On Tue, Dec 20, 2016 at 10:02:19PM +0800, Geliang Tang wrote: > > > > To make the code

Re: [Xen-devel] Xen: ARM: Support for mapping ECAM PCIe Config Space Specified In Static ACPI Table

2016-12-20 Thread Stefano Stabellini
On Tue, 20 Dec 2016, Julien Grall wrote: > Hi Stefano, > > On 20/12/2016 00:54, Stefano Stabellini wrote: > > On Mon, 19 Dec 2016, Julien Grall wrote: > > > On 16/12/2016 15:49, Julien Grall wrote: > > > > On 14/12/16 08:00, Jiandi An wrote: > > > > > Xen currently doesn't map ECAM space

Re: [Xen-devel] Xen: Support for mapping OperationRegion in ACPI ASL

2016-12-20 Thread Stefano Stabellini
On Tue, 20 Dec 2016, Jan Beulich wrote: > >>> On 20.12.16 at 00:01, wrote: > > This is actually not an ARM specific question, so changing the subject > > and CC'ing more people. > > > > On Wed, 14 Dec 2016, Konrad Rzeszutek Wilk wrote: > >> On Tue, Dec 13, 2016 at

Re: [Xen-devel] [PATCH v2 0/5] General Build Fixes

2016-12-20 Thread Alistair Francis
On Tue, Dec 20, 2016 at 11:53 AM, Konrad Rzeszutek Wilk wrote: > On Tue, Dec 20, 2016 at 11:46:55AM -0800, Alistair Francis wrote: >> This patch series is a list of build issues that appeared when >> buildling Xen 4.8.0 in buildroot. Hopefully some of them can be > > Is

Re: [Xen-devel] [PATCH v2 1/5] Remove hardcoded strict -Werror checking

2016-12-20 Thread Andrew Cooper
On 20/12/2016 20:06, Doug Goldstein wrote: > On 12/20/16 1:46 PM, Alistair Francis wrote: >> Signed-off-by: Alistair Francis >> --- >> Config.mk | 2 +- >> tools/blktap2/drivers/Makefile | 1 - >> tools/libxl/Makefile | 2 +- >>

[Xen-devel] [xen-4.8-testing test] 103767: tolerable trouble: broken/fail/pass - PUSHED

2016-12-20 Thread osstest service owner
flight 103767 xen-4.8-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/103767/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-amd64-i386-freebsd10-i386 3 host-install(3) broken in 103752 pass in 103767

Re: [Xen-devel] Xen ARM - Exposing a PL011 to the guest

2016-12-20 Thread Stefano Stabellini
On Tue, 20 Dec 2016, Bhupinder Thakur wrote: > Hi Stefano, > > Thanks for a detailed explanation. I have some queries. > > > Let me explain how the PV console protocol and drivers work, because > > they are a bit unusual. The first PV console is advertised via > > hvm_params. The guest calls: >

[Xen-devel] [linux-4.1 test] 103764: regressions - trouble: broken/fail/pass

2016-12-20 Thread osstest service owner
flight 103764 linux-4.1 real [real] http://logs.test-lab.xenproject.org/osstest/logs/103764/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-xl 6 xen-boot fail REGR. vs. 101737

Re: [Xen-devel] Xen ARM - Exposing a PL011 to the guest

2016-12-20 Thread Stefano Stabellini
On Tue, 20 Dec 2016, Christoffer Dall wrote: > Hi Stefano, > > On Mon, Dec 19, 2016 at 12:24:18PM -0800, Stefano Stabellini wrote: > > On Mon, 19 Dec 2016, Christoffer Dall wrote: > > > On Fri, Dec 16, 2016 at 05:03:13PM +, Julien Grall wrote: > > > > (CC rest maintainers for event channel

Re: [Xen-devel] [PATCH v2 1/5] Remove hardcoded strict -Werror checking

2016-12-20 Thread Doug Goldstein
On 12/20/16 1:46 PM, Alistair Francis wrote: > Signed-off-by: Alistair Francis > --- > Config.mk | 2 +- > tools/blktap2/drivers/Makefile | 1 - > tools/libxl/Makefile | 2 +- > tools/xentrace/Makefile| 2 -- > 4 files changed,

[Xen-devel] [distros-debian-snapshot test] 68247: tolerable FAIL

2016-12-20 Thread Platform Team regression test user
flight 68247 distros-debian-snapshot real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/68247/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-armhf-armhf-armhf-daily-netboot-pygrub 9 debian-di-install fail blocked in 68215

Re: [Xen-devel] Xen: ARM: Support for mapping ECAM PCIe Config Space Specified In Static ACPI Table

2016-12-20 Thread Stefano Stabellini
On Tue, 20 Dec 2016, Julien Grall wrote: > Hi Jiandi, > > On 20/12/2016 07:31, Jiandi An wrote: > > On 12/19/16 07:11, Julien Grall wrote: > > > > > > > > > On 19/12/2016 13:20, Jaggi, Manish wrote: > > > > > On 16/12/2016 15:49, Julien Grall wrote: > > > > > > On 14/12/16 08:00, Jiandi An

Re: [Xen-devel] [RFC] Xen PV Drivers Lifecycle

2016-12-20 Thread Stefano Stabellini
On Tue, 20 Dec 2016, Jan Beulich wrote: > >>> On 20.12.16 at 01:47, wrote: > > ## Design Phase > > > > The first step toward acceptance of a new PV protocol is to write a > > design document and send it to xen-devel. It should cover the xenstore > > handshake mechanism,

[Xen-devel] [ovmf baseline-only test] 68248: tolerable FAIL

2016-12-20 Thread Platform Team regression test user
This run is configured for baseline tests only. flight 68248 ovmf real [real] http://osstest.xs.citrite.net/~osstest/testlogs/logs/68248/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-i386-xl-qemuu-ovmf-amd64 9 debian-hvm-install

Re: [Xen-devel] [PATCH v2 1/5] Remove hardcoded strict -Werror checking

2016-12-20 Thread Alistair Francis
On Tue, Dec 20, 2016 at 12:16 PM, Andrew Cooper wrote: > On 20/12/2016 20:06, Doug Goldstein wrote: >> On 12/20/16 1:46 PM, Alistair Francis wrote: >>> Signed-off-by: Alistair Francis >>> --- >>> Config.mk | 2 +- >>>

Re: [Xen-devel] [PATCH v2 1/5] Remove hardcoded strict -Werror checking

2016-12-20 Thread Doug Goldstein
On 12/20/16 2:16 PM, Andrew Cooper wrote: > On 20/12/2016 20:06, Doug Goldstein wrote: >> On 12/20/16 1:46 PM, Alistair Francis wrote: >>> Signed-off-by: Alistair Francis >>> --- >>> Config.mk | 2 +- >>> tools/blktap2/drivers/Makefile | 1 - >>>

Re: [Xen-devel] [RFC 0/7] Move accel, KVM, Xen, qtest files to accel/ subdir

2016-12-20 Thread Stefano Stabellini
On Tue, 20 Dec 2016, Eduardo Habkost wrote: > This moves the KVM and Xen files to the an accel/ subdir. > > Instead of moving the *-stubs.c file to accel/ as-is, I tried to > move most of the stub code to libqemustub.a. This way the obj-y > logic for accel/ is simpler: obj-y includes accel/ only

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

2016-12-20 Thread Tian, Kevin
> From: Xuquan (Quan Xu) [mailto:xuqu...@huawei.com] > Sent: Tuesday, December 20, 2016 5:39 PM > > On December 20, 2016 4:32 PM, Jan Beulich wrote: > On 20.12.16 at 06:54, wrote: > >> On December 20, 2016 1:37 PM, Tian, Kevin wrote: > From: Xuquan (Quan Xu)

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

2016-12-20 Thread Tian, Kevin
> From: Xuquan (Quan Xu) [mailto:xuqu...@huawei.com] > Sent: Tuesday, December 20, 2016 9:12 PM > > On December 20, 2016 1:37 PM, Tian, Kevin wrote: > >> From: Xuquan (Quan Xu) [mailto:xuqu...@huawei.com] > >> Sent: Friday, December 16, 2016 5:40 PM > >I suppose you've verified this new version,

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

2016-12-20 Thread Xuquan (Quan Xu)
When Xen apicv is enabled, wall clock time is faster on Windows7-32 guest with high payload (with 2vCPU, captured from xentrace, in high payload, the count of IPI interrupt increases rapidly between these vCPUs). If IPI intrrupt (vector 0xe1) and periodic timer interrupt (vector 0xd1) are both

[Xen-devel] EPT initialization for domain U

2016-12-20 Thread 唐伟文
Hi, I really want to know when the ept (extended page table) for domain U is initialized and comes into use in xen. I already know the do_domctl() function can create domain U by handling the hypercall XEN_DOMCTL_createdomain. It actually initialize some data structures of ept such as default

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

2016-12-20 Thread Xuquan (Quan Xu)
On December 21, 2016 10:30 AM, Tian, Kevin wrote: >> From: Xuquan (Quan Xu) [mailto:xuqu...@huawei.com] >> Sent: Tuesday, December 20, 2016 9:12 PM >> >> On December 20, 2016 1:37 PM, Tian, Kevin wrote: >> >> From: Xuquan (Quan Xu) [mailto:xuqu...@huawei.com] >> >> Sent: Friday, December 16, 2016

Re: [Xen-devel] [Qemu-devel] [RFC 0/7] Move accel, KVM, Xen, qtest files to accel/ subdir

2016-12-20 Thread Thomas Huth
On 20.12.2016 18:43, Eduardo Habkost wrote: > This moves the KVM and Xen files to the an accel/ subdir. > > Instead of moving the *-stubs.c file to accel/ as-is, I tried to > move most of the stub code to libqemustub.a. This way the obj-y > logic for accel/ is simpler: obj-y includes accel/ only

[Xen-devel] [xen-4.5-testing test] 103755: trouble: broken/fail/pass

2016-12-20 Thread osstest service owner
flight 103755 xen-4.5-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/103755/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-qemut-debianhvm-amd64 3 host-install(3) broken REGR.

Re: [Xen-devel] [PATCH 2/2] x86/emul: Bugfixes to SYSCALL emulation

2016-12-20 Thread Jan Beulich
>>> On 19.12.16 at 17:37, wrote: > Introduce vendor_is() to allow emulation to have vendor-specific > behaviour. Adjust the SYSCALL behaviour on Intel to raise #UD when > executed outside of 64bit mode. I'd rather not see us go this route. I've been carrying a patch

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

2016-12-20 Thread Jan Beulich
>>> On 20.12.16 at 06:37, wrote: >> From: Xuquan (Quan Xu) [mailto:xuqu...@huawei.com] >> Sent: Friday, December 16, 2016 5:40 PM >> -if (pt_vector != -1) >> -vmx_set_eoi_exit_bitmap(v, pt_vector); >> +if ( pt_vector != -1 ) { >> +if

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

2016-12-20 Thread Jan Beulich
>>> On 20.12.16 at 09:53, wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: Tuesday, December 20, 2016 4:35 PM >> >> >>> On 20.12.16 at 06:37, wrote: >> >> From: Xuquan (Quan Xu) [mailto:xuqu...@huawei.com] >> >> Sent: Friday, December

Re: [Xen-devel] [PATCH v3] xen: sched: removal of redundant check in Credit

2016-12-20 Thread Praveen Kumar
Hi Dario, I tried with 'git am' to apply the patch after downloading the mbox file, that worked fine. Do let me know if that is ok. Regards, ~Praveen. On Sat, Dec 17, 2016 at 1:44 PM, Praveen Kumar wrote: > Hi, > > On Sat, Dec 17, 2016 at 7:16 AM, Dario Faggioli

[Xen-devel] Ping: [PATCH 0/3] x86emul: misc adjustments

2016-12-20 Thread Jan Beulich
>>> On 12.12.16 at 10:38, wrote: > These patches are grouped together merely because of contextual > dependencies. > > 1: correct EFLAGS.TF handling > 2: conditionally clear BNDn for branches > 3: some REX related polishing > > Signed-off-by: Jan Beulich

Re: [Xen-devel] [PATCH] Xen: ARM: Zero reserved fields of xatp before making hypervisor call

2016-12-20 Thread Julien Grall
Hi Jiandi, Please respect the netiquette and wrap line to 70-75 characters. On 20/12/2016 06:02, Jiandi An wrote: On 12/19/16 12:49, Stefano Stabellini wrote: On Mon, 19 Dec 2016, Juergen Gross wrote: On 19/12/16 03:56, Jiandi An wrote: Thanks for you comments. xatp is passed to XEN via

[Xen-devel] [PATCH 04/10] x86/oprofile: use unambiguous register names

2016-12-20 Thread Jan Beulich
This is in preparation of eliminating the mis-naming of 64-bit fields with 32-bit register names (eflags instead of rflags etc). Signed-off-by: Jan Beulich --- a/xen/arch/x86/oprofile/backtrace.c +++ b/xen/arch/x86/oprofile/backtrace.c @@ -150,7 +150,7 @@ static int

[Xen-devel] [PATCH 03/10] x86/shadow: use unambiguous register names

2016-12-20 Thread Jan Beulich
This is in preparation of eliminating the mis-naming of 64-bit fields with 32-bit register names (eflags instead of rflags etc). Signed-off-by: Jan Beulich --- a/xen/arch/x86/mm/shadow/common.c +++ b/xen/arch/x86/mm/shadow/common.c @@ -348,10 +348,10 @@ const struct

Re: [Xen-devel] [PATCH 03/10] x86/shadow: use unambiguous register names

2016-12-20 Thread Tim Deegan
At 03:38 -0700 on 20 Dec (1482205097), Jan Beulich wrote: > This is in preparation of eliminating the mis-naming of 64-bit fields > with 32-bit register names (eflags instead of rflags etc). > > Signed-off-by: Jan Beulich Acked-by: Tim Deegan

Re: [Xen-devel] [PATCH 1/7] Remove hardcoded strict -Werror checking

2016-12-20 Thread Konrad Rzeszutek Wilk
On Tue, Dec 20, 2016 at 11:02:15AM -0600, Doug Goldstein wrote: > On 12/20/16 10:05 AM, Konrad Rzeszutek Wilk wrote: > > On Mon, Dec 19, 2016 at 09:53:02PM -0600, Doug Goldstein wrote: > >> On 12/17/16 9:51 AM, Konrad Rzeszutek Wilk wrote: > >>> On Fri, Dec 16, 2016 at 02:56:01PM -0800, Alistair

Re: [Xen-devel] [PATCH 08/10] x86/vm-event: use unambiguous register names

2016-12-20 Thread Tamas K Lengyel
2016-12-20 3:42 GMT-07:00 Jan Beulich : > This is in preparation of eliminating the mis-naming of 64-bit fields > with 32-bit register names (eflags instead of rflags etc). > > Signed-off-by: Jan Beulich > Acked-by: Tamas K Lengyel

Re: [Xen-devel] [PATCH v4 11/14] xen/x86: parse Dom0 kernel for PVHv2

2016-12-20 Thread Roger Pau Monne
On Fri, Dec 09, 2016 at 10:05:18AM -0700, Jan Beulich wrote: > >>> On 30.11.16 at 17:49, wrote: > > @@ -1930,12 +1931,148 @@ static int __init hvm_setup_p2m(struct domain *d) > > #undef MB1_PAGES > > } > > > > +static int __init hvm_copy_to_phys(struct domain *d, paddr_t

Re: [Xen-devel] [PATCH 00/10] x86: register renaming (part I)

2016-12-20 Thread Andrew Cooper
On 20/12/2016 09:55, Jan Beulich wrote: > This is a first (of three, as far as current plans go) steps to do away > with misleading register names (eax instead of rax). > > 01: x86/MSR: introduce MSR access split/fold helpers > 02: x86/guest-walk: use unambiguous register names > 03: x86/shadow:

[Xen-devel] [xen-4.4-testing test] 103756: trouble: blocked/broken/fail/pass

2016-12-20 Thread osstest service owner
flight 103756 xen-4.4-testing real [real] http://logs.test-lab.xenproject.org/osstest/logs/103756/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-i386-xend 3 host-install(3)broken REGR. vs.

Re: [Xen-devel] Xen ARM - Exposing a PL011 to the guest

2016-12-20 Thread Bhupinder Thakur
Hi Stefano, Thanks for a detailed explanation. I have some queries. > > Let me explain how the PV console protocol and drivers work, because > they are a bit unusual. The first PV console is advertised via > hvm_params. The guest calls: > > hvm_get_parameter(HVM_PARAM_CONSOLE_EVTCHN, ); >

[Xen-devel] [PATCH 00/10] x86: register renaming (part I)

2016-12-20 Thread Jan Beulich
This is a first (of three, as far as current plans go) steps to do away with misleading register names (eax instead of rax). 01: x86/MSR: introduce MSR access split/fold helpers 02: x86/guest-walk: use unambiguous register names 03: x86/shadow: use unambiguous register names 04: x86/oprofile: use

[Xen-devel] [PATCH 01/10] x86/MSR: introduce MSR access split/fold helpers

2016-12-20 Thread Jan Beulich
This is in preparation of eliminating the mis-naming of 64-bit fields with 32-bit register names (eflags instead of rflags etc). Use the guaranteed 32-bit underscore prefixed names for now where appropriate. Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/hvm.c +++

[Xen-devel] [PATCH 09/10] x86/traps: use unambiguous register names

2016-12-20 Thread Jan Beulich
This is in preparation of eliminating the mis-naming of 64-bit fields with 32-bit register names (eflags instead of rflags etc). Use the guaranteed 32-bit underscore prefixed names for now where appropriate. Signed-off-by: Jan Beulich --- a/xen/arch/x86/traps.c +++

[Xen-devel] [PATCH 10/10] x86/misc: use unambiguous register names

2016-12-20 Thread Jan Beulich
This is in preparation of eliminating the mis-naming of 64-bit fields with 32-bit register names (eflags instead of rflags etc). Use the guaranteed 32-bit underscore prefixed names for now where appropriate. Signed-off-by: Jan Beulich --- a/xen/arch/x86/cpu/vpmu.c +++

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

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

Re: [Xen-devel] Xen: ARM: Support for mapping ECAM PCIe Config Space Specified In Static ACPI Table

2016-12-20 Thread Julien Grall
Hi Jiandi, On 20/12/2016 07:31, Jiandi An wrote: On 12/19/16 07:11, Julien Grall wrote: On 19/12/2016 13:20, Jaggi, Manish wrote: On 16/12/2016 15:49, Julien Grall wrote: On 14/12/16 08:00, Jiandi An wrote: Xen currently doesn't map ECAM space specified in static ACPI table. Seeking

[Xen-devel] [PATCH] x86emul: don't unconditionally clear segment bases upon null selector loads

2016-12-20 Thread Jan Beulich
AMD explicitly documents that namely FS and GS don't have their bases cleared in that case, and I see no reason why guests may not rely on that behavior. To facilitate this a new input field (the CPU vendor) is being added. Signed-off-by: Jan Beulich ---

Re: [Xen-devel] [PATCH] xsm: allow relevant permission during migrate and gpu-passthrough.

2016-12-20 Thread Anshul Makkar
On 20/12/2016 04:03, Doug Goldstein wrote: On 12/19/16 10:02 AM, Doug Goldstein wrote: On 12/14/16 3:09 PM, Daniel De Graaf wrote: On 12/12/2016 09:00 AM, Anshul Makkar wrote: During guest migrate allow permission to prevent spurious page faults. Prevents these errors: d73: Non-privileged

Re: [Xen-devel] Ping: [PATCH 0/3] x86emul: misc adjustments

2016-12-20 Thread Andrew Cooper
On 20/12/2016 09:04, Jan Beulich wrote: On 12.12.16 at 10:38, wrote: >> These patches are grouped together merely because of contextual >> dependencies. >> >> 1: correct EFLAGS.TF handling >> 2: conditionally clear BNDn for branches >> 3: some REX related polishing >> >>

Re: [Xen-devel] [PATCH 3/3] x86emul: some REX related polishing

2016-12-20 Thread Andrew Cooper
On 12/12/2016 10:00, Jan Beulich wrote: > While there are a few cases where it seems better to open-code REX_* > values, there's one where this clearly is a bad idea. And the SYSEXIT > emulation has no need to look at REX at all, it can simply use op_bytes > instead. > > Signed-off-by: Jan Beulich

Re: [Xen-devel] [PATCH] xen/arm: fix rank/vgic locks inversion bug

2016-12-20 Thread Julien Grall
Hi Stefano, On 20/12/2016 00:22, Stefano Stabellini wrote: On Mon, 19 Dec 2016, Julien Grall wrote: Hi Stefano, On 19/12/2016 23:30, Stefano Stabellini wrote: On Mon, 19 Dec 2016, Julien Grall wrote: 2) We run gic_update_one_lr and vgic_store_itargetsr in parallel safely and locklessly.

Re: [Xen-devel] [PATCH 1/2] x86/hvm: Don't emulate all instructions hitting the #UD intercept

2016-12-20 Thread Jan Beulich
>>> On 19.12.16 at 17:37, wrote: > Having the instruction emulator fill in all #UDs when using FEP is unhelpful > when trying to test emulation behaviour against hardware. > > Restrict emulation from the #UD intercept to the cross-vendor case, and when a > postive

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

2016-12-20 Thread Jan Beulich
>>> On 20.12.16 at 06:54, wrote: > On December 20, 2016 1:37 PM, Tian, Kevin wrote: >>> From: Xuquan (Quan Xu) [mailto:xuqu...@huawei.com] >>> Sent: Friday, December 16, 2016 5:40 PM >>> >>> From 89fffdd6b563b2723e24d17231715bb8c9f24f90 Mon Sep 17 00:00:00 >>2001 >>> From:

[Xen-devel] [PATCH 02/10] x86/guest-walk: use unambiguous register names

2016-12-20 Thread Jan Beulich
This is in preparation of eliminating the mis-naming of 64-bit fields with 32-bit register names (eflags instead of rflags etc). Use the guaranteed 32-bit underscore prefixed names for now where appropriate. Signed-off-by: Jan Beulich --- a/xen/arch/x86/mm/guest_walk.c +++

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

2016-12-20 Thread osstest service owner
flight 103763 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/103763/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 83c6c3bfe293cd0a5983375dfbf46d4f364b38aa baseline version: ovmf

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

2016-12-20 Thread Xuquan (Quan Xu)
On December 20, 2016 4:54 PM, Tian, Kevin wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: Tuesday, December 20, 2016 4:35 PM >> >> >>> On 20.12.16 at 06:37, wrote: >> >> From: Xuquan (Quan Xu) [mailto:xuqu...@huawei.com] >> >> Sent: Friday, December 16, 2016

  1   2   >