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

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] [PATCH v4] x86/apicv: fix RTC periodic timer and apicv issue

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

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

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] [PATCH v2 1/5] Remove hardcoded strict -Werror checking

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] [PATCH v3] x86/apicv: fix RTC periodic timer and apicv issue

> 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

> 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] EPT initialization for domain U

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] [RFC 0/7] Move accel, KVM, Xen, qtest files to accel/ subdir

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 v2 1/5] Remove hardcoded strict -Werror checking

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] [RFC] Xen PV Drivers Lifecycle

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,

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

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: Support for mapping OperationRegion in ACPI ASL

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] Xen: ARM: Support for mapping ECAM PCIe Config Space Specified In Static ACPI Table

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] Xen: ARM: Support for mapping ECAM PCIe Config Space Specified In Static ACPI Table

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 ARM - Exposing a PL011 to the guest

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] [ovmf baseline-only test] 68248: tolerable FAIL

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] xen/blkback: use rb_entry()

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] [PATCH] xen/blkback: use rb_entry()

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

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

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

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

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

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

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

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

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 v2 1/5] Remove hardcoded strict -Werror checking

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 +- >>

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

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,

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

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 0/5] General Build Fixes

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 v2 4/5] tools/blktap2: Fix missing header file

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] Xen ARM - Exposing a PL011 to the guest

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

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 v2 3/5] tools/blktap2/vhd: Remove unused struct stat stats

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] [PATCH v2 2/5] config/StdGNU.mk: Allows users to pass in EXTRA_CFLAGS

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] [PATCH v2 0/5] General Build Fixes

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 2/5] config/StdGNU.mk: Allows users to pass in EXTRA_CFLAGS

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

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 3/5] tools/blktap2/vhd: Remove unused struct stat stats

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

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

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 1/5] Remove hardcoded strict -Werror checking

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

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

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

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

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

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

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

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

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

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,

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

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 0/9] db retry: fixes and workarounds

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

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 5/9] db retry, bisect: Cache build reuse investigations

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] [OSSTEST PATCH 6/9] db retry: Document $@ as an implicit parameter to need_retry

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 1/9] mg-schema-test-database: Revamp sequence handling

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

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 4/9] db retry, bisection: Reset %jobs_created on db retry

%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 2/9] mg-schema-test-database: Wrap some withtest psql_do in subshells

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] [xen-4.6-testing test] 103765: regressions - trouble: broken/fail/pass

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] Xen ARM community call - meeting minutes and date for the next one

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] [libvirt test] 103766: regressions - trouble: broken/fail/pass

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

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

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

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

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

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

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

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

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

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

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] [RFC 1/7] xen: Move xen-*-stub.c to stubs/

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] Xen ARM community call - meeting minutes and date for the next one

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

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

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. > >

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

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

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

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

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 00/10] x86: register renaming (part I)

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:

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

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 08/10] x86/vm-event: use unambiguous register names

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 1/7] Remove hardcoded strict -Werror checking

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

[Xen-devel] [xen-unstable test] 103762: regressions - trouble: broken/fail/pass

flight 103762 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/103762/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-xtf-amd64-amd64-33 host-install(3)broken REGR. vs. 103466

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

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 Francis wrote: Signed-off-by: Alistair Francis

[Xen-devel] merlot0 and merlot1 (was Re: [xen-4.4-testing test] 103756: trouble: blocked/broken/fail/pass)

Jan Beulich writes ("[Xen-devel] merlot0 and merlot1 (was Re: [xen-4.4-testing test] 103756: trouble: blocked/broken/fail/pass)"): > I consider this difficult to imagine, but it's also not entirely > impossible. Since this appears to recur, are there perhaps logs > (ideally more than one

Re: [Xen-devel] [PATCH v5 05/13] pvh/acpi: Handle ACPI accesses for PVH guests

On 12/20/2016 11:46 AM, Andrew Cooper wrote: > On 20/12/2016 15:41, Jan Beulich wrote: > On 20.12.16 at 16:29, wrote: >>> On 12/20/2016 09:47 AM, Jan Beulich wrote: >>> On 20.12.16 at 15:35, wrote: > On 12/20/2016 06:50 AM, Jan

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

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. Roger, thoughts? > > Signed-off-by: Geliang Tang > --- >

Re: [Xen-devel] [PATCH v5 05/13] pvh/acpi: Handle ACPI accesses for PVH guests

On 20/12/2016 15:41, Jan Beulich wrote: On 20.12.16 at 16:29, wrote: >> On 12/20/2016 09:47 AM, Jan Beulich wrote: >> On 20.12.16 at 15:35, wrote: On 12/20/2016 06:50 AM, Jan Beulich wrote: >> +else >> +{

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

On 20/12/16 15:02, Geliang Tang wrote: > To make the code clearer, use rb_entry() instead of container_of() to > deal with rbtree. > > Signed-off-by: Geliang Tang Reviewed-by: Juergen Gross > --- > drivers/xen/evtchn.c | 4 ++-- > 1 file changed, 2

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

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 Francis wrote: > >> Signed-off-by: Alistair Francis > > > > > > Why? > > *adjusts his distro

[Xen-devel] [xen-unstable-smoke test] 103774: regressions - FAIL

flight 103774 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/103774/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-amd64-libvirt17 guest-start/debian.repeat fail REGR. vs. 103771 Tests which

Re: [Xen-devel] [PATCH v5 05/13] pvh/acpi: Handle ACPI accesses for PVH guests

>>> On 20.12.16 at 16:29, wrote: > On 12/20/2016 09:47 AM, Jan Beulich wrote: > On 20.12.16 at 15:35, wrote: >>> On 12/20/2016 06:50 AM, Jan Beulich wrote: > +else > +{ > +uint32_t v = *val; > + > +

Re: [Xen-devel] [PATCH v5 12/13] pvh/acpi: Save ACPI registers for PVH guests

>>> On 20.12.16 at 16:09, wrote: > On 12/20/2016 08:57 AM, Jan Beulich wrote: > On 17.12.16 at 00:18, wrote: >>> --- a/xen/arch/x86/hvm/pmtimer.c >>> +++ b/xen/arch/x86/hvm/pmtimer.c >>> @@ -257,7 +257,11 @@ static int acpi_save(struct

Re: [Xen-devel] [PATCH v5 04/13] pvh/acpi: Install handlers for ACPI-related PVH IO accesses

On 12/20/2016 09:55 AM, Andrew Cooper wrote: >>> Is this file is not supposed to be used by anyone outside of the Xen tree? >> I don't think so, no. In any event - prior additions did not do >> any precautions to guard possible foreign consumers. Maybe >> Andrew has an opinion here ... > Our

Re: [Xen-devel] [PATCH v5 05/13] pvh/acpi: Handle ACPI accesses for PVH guests

On 12/20/2016 09:47 AM, Jan Beulich wrote: On 20.12.16 at 15:35, wrote: >> On 12/20/2016 06:50 AM, Jan Beulich wrote: +else +{ +uint32_t v = *val; + +/* Status register is write-1-to-clear by guests */ +

Re: [Xen-devel] [PATCH v5 12/13] pvh/acpi: Save ACPI registers for PVH guests

On 12/20/2016 08:57 AM, Jan Beulich wrote: On 17.12.16 at 00:18, wrote: >> --- a/xen/arch/x86/hvm/pmtimer.c >> +++ b/xen/arch/x86/hvm/pmtimer.c >> @@ -257,7 +257,11 @@ static int acpi_save(struct domain *d, >> hvm_domain_context_t *h) >> int rc; >> >>

Re: [Xen-devel] [PATCH v5 04/13] pvh/acpi: Install handlers for ACPI-related PVH IO accesses

On 20/12/2016 14:45, Jan Beulich wrote: On 20.12.16 at 15:16, wrote: >> On 12/20/2016 09:10 AM, Jan Beulich wrote: >> On 20.12.16 at 15:03, wrote: On 12/20/2016 06:24 AM, Jan Beulich wrote: On 17.12.16 at 00:18,

Re: [Xen-devel] [PATCH v5 08/13] pvh: Send an SCI on VCPU hotplug event

On 12/20/2016 08:37 AM, Jan Beulich wrote: On 17.12.16 at 00:18, wrote: >> @@ -128,6 +130,13 @@ static int acpi_access_common(struct domain *d, bool >> is_guest_access, >> *en = (((v & 0xff) << 8) | (*en & 0xff)) & *mask_en; >> break; >>

Re: [Xen-devel] [PATCH v5 06/13] x86/domctl: Handle ACPI access from domctl

>>> On 20.12.16 at 15:45, wrote: > On 12/20/2016 08:24 AM, Jan Beulich wrote: >> >>> -static int acpi_access_common(struct domain *d, >>> +static int acpi_access_common(struct domain *d, bool is_guest_access, >> Why? I thought the domctl is needed only for updating the

Re: [Xen-devel] [PATCH v5 05/13] pvh/acpi: Handle ACPI accesses for PVH guests

>>> On 20.12.16 at 15:35, wrote: > On 12/20/2016 06:50 AM, Jan Beulich wrote: >>> +else >>> +{ >>> +uint32_t v = *val; >>> + >>> +/* Status register is write-1-to-clear by guests */ >>> +switch ( port & 3 ) >>> +{ >>> +

Re: [Xen-devel] [PATCH v5 04/13] pvh/acpi: Install handlers for ACPI-related PVH IO accesses

>>> On 20.12.16 at 15:16, wrote: > On 12/20/2016 09:10 AM, Jan Beulich wrote: > On 20.12.16 at 15:03, wrote: >>> On 12/20/2016 06:24 AM, Jan Beulich wrote: >>> On 17.12.16 at 00:18, wrote: > ---

Re: [Xen-devel] [PATCH v5 06/13] x86/domctl: Handle ACPI access from domctl

On 12/20/2016 08:24 AM, Jan Beulich wrote: > >> -static int acpi_access_common(struct domain *d, >> +static int acpi_access_common(struct domain *d, bool is_guest_access, > Why? I thought the domctl is needed only for updating the CPU > map? Or maybe it would help if the patch had a non-empty >

Re: [Xen-devel] [PATCH v5 05/13] pvh/acpi: Handle ACPI accesses for PVH guests

On 12/20/2016 06:50 AM, Jan Beulich wrote: > >> + >> +if ( dir == XEN_DOMCTL_ACPI_READ ) >> +{ >> +uint32_t mask = (bytes < 4) ? ~0U << (bytes * 8) : 0; >> +uint32_t data = (((uint32_t)*en) << 16) | *sts; > There's one pair of pointless parentheses around the cast >

Re: [Xen-devel] [PATCH v5 04/13] pvh/acpi: Install handlers for ACPI-related PVH IO accesses

On 12/20/2016 09:10 AM, Jan Beulich wrote: On 20.12.16 at 15:03, wrote: >> On 12/20/2016 06:24 AM, Jan Beulich wrote: >> On 17.12.16 at 00:18, wrote: --- a/xen/include/public/arch-x86/hvm/save.h +++

Re: [Xen-devel] [PATCH v5 04/13] pvh/acpi: Install handlers for ACPI-related PVH IO accesses

>>> On 20.12.16 at 15:03, wrote: > On 12/20/2016 06:24 AM, Jan Beulich wrote: > On 17.12.16 at 00:18, wrote: >>> --- a/xen/include/public/arch-x86/hvm/save.h >>> +++ b/xen/include/public/arch-x86/hvm/save.h >>> @@ -527,7 +527,37 @@

Re: [Xen-devel] [PATCH v5 04/13] pvh/acpi: Install handlers for ACPI-related PVH IO accesses

On 12/20/2016 06:24 AM, Jan Beulich wrote: On 17.12.16 at 00:18, wrote: >> --- a/xen/include/public/arch-x86/hvm/save.h >> +++ b/xen/include/public/arch-x86/hvm/save.h >> @@ -527,7 +527,37 @@ DECLARE_HVM_SAVE_TYPE(HPET, 12, struct hvm_hw_hpet); >> /* >> * PM

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

To make the code clearer, use rb_entry() instead of container_of() to deal with rbtree. Signed-off-by: Geliang Tang --- drivers/xen/evtchn.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/xen/evtchn.c b/drivers/xen/evtchn.c index

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

To make the code clearer, use rb_entry() instead of container_of() to deal with rbtree. Signed-off-by: Geliang Tang --- drivers/block/xen-blkback/blkback.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/block/xen-blkback/blkback.c

Re: [Xen-devel] [PATCH v5 12/13] pvh/acpi: Save ACPI registers for PVH guests

>>> On 17.12.16 at 00:18, wrote: > --- a/xen/arch/x86/hvm/pmtimer.c > +++ b/xen/arch/x86/hvm/pmtimer.c > @@ -257,7 +257,11 @@ static int acpi_save(struct domain *d, > hvm_domain_context_t *h) > int rc; > > if ( !has_vpm(d) ) > +{ > +if (

Re: [Xen-devel] [PATCH v5 08/13] pvh: Send an SCI on VCPU hotplug event

>>> On 17.12.16 at 00:18, wrote: > @@ -128,6 +130,13 @@ static int acpi_access_common(struct domain *d, bool > is_guest_access, > *en = (((v & 0xff) << 8) | (*en & 0xff)) & *mask_en; > break; > } > + > +/* > + * If a

  1   2   >