[Xen-devel] [qemu-mainline test] 134560: trouble: blocked/broken/fail/pass

2019-04-10 Thread osstest service owner
flight 134560 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/134560/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64 broken build-arm64-xsm

Re: [Xen-devel] [PATCH RFC 00/49] xen: add core scheduling support

2019-04-10 Thread Dario Faggioli
On Fri, 2019-03-29 at 19:16 +0100, Dario Faggioli wrote: > On Fri, 2019-03-29 at 16:08 +0100, Juergen Gross wrote: > > I have done some very basic performance testing: on a 4 cpu system > > (2 cores with 2 threads each) I did a "make -j 4" for building the > > Xen > > hypervisor. With This test

[Xen-devel] [xen-unstable-smoke test] 134620: regressions - trouble: blocked/broken/fail

2019-04-10 Thread osstest service owner
flight 134620 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/134620/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64-xsm broken build-arm64-xsm 4

[Xen-devel] [linux-linus test] 134517: regressions - trouble: blocked/broken/fail/pass

2019-04-10 Thread osstest service owner
flight 134517 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/134517/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64-pvopsbroken build-arm64

Re: [Xen-devel] [PATCH 1/2] libxl: Add virtio vga interface support for qemu

2019-04-10 Thread Chris Patterson
For anyone looking at this... while I have tested and verified that both virtio-gpu and VirGL work, it's not without some hiccup. I have been running Ubuntu 19.04 with this config for a few days and I have had a couple VM freezes. 'xl dmesg' is spamming the following repeatedly: (XEN) irq.c:2479:

[Xen-devel] [xen-unstable-smoke test] 134612: regressions - trouble: blocked/broken/fail/pass

2019-04-10 Thread osstest service owner
flight 134612 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/134612/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64-xsm broken build-arm64-xsm 4

Re: [Xen-devel] [PATCH RFC 00/39] x86/KVM: Xen HVM guest support

2019-04-10 Thread Juergen Gross
On 10/04/2019 08:55, Ankur Arora wrote: > On 2019-04-08 10:04 p.m., Juergen Gross wrote: >> On 08/04/2019 19:31, Joao Martins wrote: >>> On 4/8/19 11:42 AM, Juergen Gross wrote: On 08/04/2019 12:36, Joao Martins wrote: > On 4/8/19 7:44 AM, Juergen Gross wrote: >> On 12/03/2019 18:14,

Re: [Xen-devel] [PATCH RFC 00/39] x86/KVM: Xen HVM guest support

2019-04-10 Thread Ankur Arora
On 2019-04-08 10:04 p.m., Juergen Gross wrote: On 08/04/2019 19:31, Joao Martins wrote: On 4/8/19 11:42 AM, Juergen Gross wrote: On 08/04/2019 12:36, Joao Martins wrote: On 4/8/19 7:44 AM, Juergen Gross wrote: On 12/03/2019 18:14, Joao Martins wrote: On 2/22/19 4:59 PM, Paolo Bonzini wrote:

Re: [edk2-devel] [PATCH v2 01/31] OvmfPkg/ResetSystemLib: Add missing dependency on PciLib

2019-04-10 Thread Laszlo Ersek
On 04/09/19 13:08, Anthony PERARD wrote: > and remove extra includes of OvmfPlatforms.h. (1) In commit messages, please don't rely on the subject line, and the first paragraph of the body, being read as a single sentence. (Similarly, please avoid empy commit message bodies.) If there's no better

Re: [edk2-devel] [PATCH v2 02/31] OvmfPkg: Create platform XenOvmf

2019-04-10 Thread Laszlo Ersek
On 04/09/19 13:08, Anthony PERARD wrote: > This is a copy of OvmfX64, removing VirtIO and some SMM. > > This new platform will be changed to make it works on two types of Xen > guest: HVM and PVH. > > Compare to OvmfX64, this patch: > > - changed: PLATFORM_GUID, OUTPUT_DIRECTORY,

Re: [Xen-devel] [PATCH] x86/msr: Fix fallout from mostly c/s 832c180

2019-04-10 Thread Andrew Cooper
On 10/04/2019 09:23, Wei Liu wrote: > On Tue, Apr 09, 2019 at 06:53:47PM +0100, Andrew Cooper wrote: >> The series 832c1803^..f61685a6 was committed without adequate review. >> >> * Fix the shim build by providing a !CONFIG_HVM declaration for >>hvm_get_guest_bndcfgs() >> * Revert the bogus

Re: [edk2-devel] [PATCH v2 02/31] OvmfPkg: Create platform XenOvmf

2019-04-10 Thread Jordan Justen
On 2019-04-09 04:08:15, Anthony PERARD wrote: > This is a copy of OvmfX64, removing VirtIO and some SMM. > > This new platform will be changed to make it works on two types of Xen > guest: HVM and PVH. > > Compare to OvmfX64, this patch: > > - changed: PLATFORM_GUID, OUTPUT_DIRECTORY,

[Xen-devel] [xen-unstable-coverity test] 134603: regressions - ALL FAIL

2019-04-10 Thread osstest service owner
flight 134603 xen-unstable-coverity real [real] http://logs.test-lab.xenproject.org/osstest/logs/134603/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: coverity-amd647 coverity-upload fail REGR. vs. 133615 version

Re: [Xen-devel] [PATCH v5] x86/mm: Clean up p2m_finish_type_change return value

2019-04-10 Thread George Dunlap
On 3/29/19 1:01 PM, Jan Beulich wrote: On 29.03.19 at 13:50, wrote: >> @@ -1174,9 +1174,9 @@ static int finish_type_change(struct p2m_domain *p2m, >> /* >> * ept->recalc could return 0/1/-ENOMEM. pt->recalc could return >> * 0/-ENOMEM/-ENOENT, -ENOENT isn't an

Re: [Xen-devel] [PATCH] x86/msr: Fix fallout from mostly c/s 832c180

2019-04-10 Thread Paul Durrant
> -Original Message- > From: Andrew Cooper [mailto:andrew.coop...@citrix.com] > Sent: 09 April 2019 18:54 > To: Xen-devel > Cc: Andrew Cooper ; Jan Beulich > ; Wei Liu > ; Roger Pau Monne ; Paul Durrant > ; > Jun Nakajima ; Kevin Tian > Subject: [PATCH] x86/msr: Fix fallout from mostly

Re: [Xen-devel] [PATCH] x86/msr: Fix fallout from mostly c/s 832c180

2019-04-10 Thread Paul Durrant
> -Original Message- > From: Paul Durrant > Sent: 10 April 2019 09:28 > To: Andrew Cooper ; Xen-devel > > Cc: Andrew Cooper ; Jan Beulich > ; Wei Liu > ; Roger Pau Monne ; Jun Nakajima > ; > Kevin Tian > Subject: RE: [PATCH] x86/msr: Fix fallout from mostly c/s 832c180 > > >

Re: [Xen-devel] [PATCH 1/9] libxl_internal: Remove lost comment

2019-04-10 Thread Wei Liu
On Tue, Apr 09, 2019 at 05:45:34PM +0100, Anthony PERARD wrote: > That comment as been separated from the function it defines by > 4197d3abbb3055d3798254eb7ba239bfb5824360, but then was not useful > anymore when the libxl__device_disk_add() prototype was removed by >

[Xen-devel] [PATCH] AMD/IOMMU: adjust IOMMU list head initialization

2019-04-10 Thread Jan Beulich
Do this statically, which will allow accessing the (empty) list even without having come through acpi_ivrs_init(). Signed-off-by: Jan Beulich --- a/xen/drivers/passthrough/amd/iommu_init.c +++ b/xen/drivers/passthrough/amd/iommu_init.c @@ -36,7 +36,7 @@ static struct tasklet amd_iommu_irq_task

Re: [Xen-devel] [PATCH] AMD/IOMMU: adjust IOMMU list head initialization

2019-04-10 Thread Andrew Cooper
On 10/04/2019 10:37, Jan Beulich wrote: > Do this statically, which will allow accessing the (empty) list even > without having come through acpi_ivrs_init(). > > Signed-off-by: Jan Beulich Reviewed-by: Andrew Cooper ___ Xen-devel mailing list

[Xen-devel] [PATCH v1] libxl: fix migration of PV and PVH domUs with and without qemu

2019-04-10 Thread Olaf Hering
If a domU has a qemu-xen instance attached, it is required to call qemus "xen-save-devices-state" method. Without it, the receiving side of a PV or PVH migration may be unable to lock the image: xen be: qdisk-51712: xen be: qdisk-51712: error: Failed to get "write" lock error: Failed to get

Re: [Xen-devel] [PATCH v1] libxl: update prototype of libxl__device_vkb_dm_needed

2019-04-10 Thread Wei Liu
On Wed, Apr 10, 2019 at 12:26:34PM +0200, Olaf Hering wrote: > Align code to match other usage of device_dm_needed_fn_t: > recieve a void pointer, assign to expected type and use it accordingly. > > No functional change expected. > > Signed-off-by: Olaf Hering Acked-by: Wei Liu

Re: [Xen-devel] [PATCH] x86/mem-sharing: statically initialize audit list head and lock

2019-04-10 Thread George Dunlap
On 4/10/19 12:13 PM, Andrew Cooper wrote: > On 10/04/2019 11:58, Jan Beulich wrote: >> There's no need to execute any instructions for doing so. >> >> Signed-off-by: Jan Beulich >> --- >> I wonder whether mem_sharing_init() shouldn't go away altogether then. > > I vote for removing it

Re: [Xen-devel] [PATCH v3 2/3] xen: remove tmem from hypervisor

2019-04-10 Thread Wei Liu
On Wed, Apr 10, 2019 at 04:55:50AM -0600, Jan Beulich wrote: > >>> On 09.04.19 at 18:52, wrote: > > --- a/xen/arch/x86/guest/hypercall_page.S > > +++ b/xen/arch/x86/guest/hypercall_page.S > > @@ -58,8 +58,8 @@ DECLARE_HYPERCALL(hvm_op) > > DECLARE_HYPERCALL(sysctl) > > DECLARE_HYPERCALL(domctl)

Re: [Xen-devel] [PATCH] x86/msr: Fix fallout from mostly c/s 832c180

2019-04-10 Thread Wei Liu
On Tue, Apr 09, 2019 at 06:53:47PM +0100, Andrew Cooper wrote: > The series 832c1803^..f61685a6 was committed without adequate review. > > * Fix the shim build by providing a !CONFIG_HVM declaration for >hvm_get_guest_bndcfgs() > * Revert the bogus de-const'ing of the vcpu pointer in >

[Xen-devel] [PATCH] x86/mem-sharing: statically initialize audit list head and lock

2019-04-10 Thread Jan Beulich
There's no need to execute any instructions for doing so. Signed-off-by: Jan Beulich --- I wonder whether mem_sharing_init() shouldn't go away altogether then. --- a/xen/arch/x86/mm/mem_sharing.c +++ b/xen/arch/x86/mm/mem_sharing.c @@ -65,8 +65,8 @@ static DEFINE_PER_CPU(pg_lock_data_t, __

Re: [Xen-devel] [PATCH v3 2/3] xen: remove tmem from hypervisor

2019-04-10 Thread Jan Beulich
>>> On 09.04.19 at 18:52, wrote: > --- a/xen/arch/x86/guest/hypercall_page.S > +++ b/xen/arch/x86/guest/hypercall_page.S > @@ -58,8 +58,8 @@ DECLARE_HYPERCALL(hvm_op) > DECLARE_HYPERCALL(sysctl) > DECLARE_HYPERCALL(domctl) > DECLARE_HYPERCALL(kexec_op) > -DECLARE_HYPERCALL(tmem_op) >

Re: [edk2-devel] [PATCH v2 01/31] OvmfPkg/ResetSystemLib: Add missing dependency on PciLib

2019-04-10 Thread Jordan Justen
On 2019-04-10 01:48:39, Laszlo Ersek wrote: > On 04/09/19 13:08, Anthony PERARD wrote: > > and remove extra includes of OvmfPlatforms.h. > > (1) In commit messages, please don't rely on the subject line, and the > first paragraph of the body, being read as a single sentence. I think

Re: [Xen-devel] [PATCH] xmalloc: statically initialize pool list head and lock

2019-04-10 Thread George Dunlap
On 4/10/19 11:58 AM, Jan Beulich wrote: > There's no need to execute any instructions for doing so. > > Signed-off-by: Jan Beulich Reviewed-by: George Dunlap ___ Xen-devel mailing list Xen-devel@lists.xenproject.org

Re: [Xen-devel] [PATCH v5] x86/mm: Clean up p2m_finish_type_change return value

2019-04-10 Thread Alexandru Stefan ISAILA
Roger/Paul are you ok with the latest changes? Can this go in? Regards, Alex On 29.03.2019 14:50, Alexandru Stefan ISAILA wrote: > In the case of any errors, finish_type_change() passes values returned > from p2m->recalc() up the stack (with some exceptions in the case where > an error is

Re: [edk2-devel] [PATCH v2 04/31] OvmfPkg: Introduce XenPlatformPei

2019-04-10 Thread Laszlo Ersek
On 04/09/19 13:08, Anthony PERARD wrote: > A copy of OvmfPkg/PlatformPei without some of QEMU specific > initialization, Xen does not support QemuFwCfg. > > This new module will be adjusted to accommodate Xen PVH. > > fw_cfg dependents that have been removed, which are dynamically skipped > when

Re: [Xen-devel] [PATCH 1/3] xen-bus: use a separate fd for each event channel

2019-04-10 Thread Anthony PERARD
On Mon, Apr 08, 2019 at 04:16:15PM +0100, Paul Durrant wrote: > To better support use of IOThread-s it will be necessary to be able to set > the AioContext for each XenEventChannel and hence it is necessary to open a > separate handle to libxenevtchan for each channel. > > This patch stops using

Re: [Xen-devel] [PATCH v5] x86/mm: Clean up p2m_finish_type_change return value

2019-04-10 Thread Paul Durrant
> -Original Message- > From: Xen-devel [mailto:xen-devel-boun...@lists.xenproject.org] On Behalf Of > Alexandru Stefan ISAILA > Sent: 10 April 2019 09:19 > To: xen-devel@lists.xenproject.org; George Dunlap ; > Paul Durrant > > Cc: Andrew Cooper ; Wei Liu ; > jbeul...@suse.com; Roger >

Re: [edk2-devel] [PATCH v2 03/31] OvmfPkg: Introduce XenResetVector

2019-04-10 Thread Laszlo Ersek
On 04/09/19 13:08, Anthony PERARD wrote: > Copy of OvmfPkg/ResetVector, with one changes: > - SEC_DEFAULT_CR0: enable cache (bit 30 or CD set to 0) > > Xen copies the OVMF code to RAM, there is no need to disable cache. > > This new module will later be modified to add a new entry point, more

[Xen-devel] [PATCH v1] libxl: update prototype of libxl__device_vkb_dm_needed

2019-04-10 Thread Olaf Hering
Align code to match other usage of device_dm_needed_fn_t: recieve a void pointer, assign to expected type and use it accordingly. No functional change expected. Signed-off-by: Olaf Hering --- tools/libxl/libxl_vkb.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git

Re: [Xen-devel] [PATCH] x86/msr: Fix fallout from mostly c/s 832c180

2019-04-10 Thread Wei Liu
On Wed, Apr 10, 2019 at 10:41:24AM +0100, Andrew Cooper wrote: > On 10/04/2019 09:23, Wei Liu wrote: > > On Tue, Apr 09, 2019 at 06:53:47PM +0100, Andrew Cooper wrote: > >> The series 832c1803^..f61685a6 was committed without adequate review. > >> > >> * Fix the shim build by providing a

Re: [Xen-devel] [PATCH] x86/msr: Fix fallout from mostly c/s 832c180

2019-04-10 Thread Jan Beulich
>>> On 09.04.19 at 19:53, wrote: > The series 832c1803^..f61685a6 was committed without adequate review. > > * Fix the shim build by providing a !CONFIG_HVM declaration for >hvm_get_guest_bndcfgs() > * Revert the bogus de-const'ing of the vcpu pointer in >vmx_get_guest_bndcfgs().

[Xen-devel] [distros-debian-squeeze test] 83918: trouble: blocked/broken

2019-04-10 Thread Platform Team regression test user
flight 83918 distros-debian-squeeze real [real] http://osstest.xensource.com/osstest/logs/83918/ Failures and problems with tests :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-pvopsbroken build-i386

Re: [Xen-devel] [PATCH] xmalloc: statically initialize pool list head and lock

2019-04-10 Thread Wei Liu
On Wed, Apr 10, 2019 at 04:58:30AM -0600, Jan Beulich wrote: > There's no need to execute any instructions for doing so. > > Signed-off-by: Jan Beulich Reviewed-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xenproject.org

Re: [Xen-devel] [PATCH v5] x86/mm: Clean up p2m_finish_type_change return value

2019-04-10 Thread Jan Beulich
>>> On 10.04.19 at 10:19, wrote: > Roger/Paul are you ok with the latest changes? Can this go in? The thing keeping it from going in is the lack of George's ack. Judging from the To: list of your mail you may even have meant him instead of Roger. Jan

Re: [Xen-devel] [PATCH v1] libxl: fix migration of PV and PVH domUs with and without qemu

2019-04-10 Thread Olaf Hering
Am Wed, 10 Apr 2019 12:33:38 +0200 schrieb Olaf Hering : > +/* will be set later */ That should read 'may be set later' because not all callers of libxl__domain_build_info_setdefault() have a need to update this field. Olaf pgpr4dfH_eiXz.pgp Description: Digitale Signatur von

[Xen-devel] [PATCH] xmalloc: statically initialize pool list head and lock

2019-04-10 Thread Jan Beulich
There's no need to execute any instructions for doing so. Signed-off-by: Jan Beulich --- a/xen/common/xmalloc_tlsf.c +++ b/xen/common/xmalloc_tlsf.c @@ -65,8 +65,8 @@ #define PREV_FREE (0x2) #define PREV_USED (0x0) -static spinlock_t pool_list_lock; -static struct list_head

Re: [Xen-devel] [PATCH] x86/mem-sharing: statically initialize audit list head and lock

2019-04-10 Thread Andrew Cooper
On 10/04/2019 11:58, Jan Beulich wrote: > There's no need to execute any instructions for doing so. > > Signed-off-by: Jan Beulich > --- > I wonder whether mem_sharing_init() shouldn't go away altogether then. I vote for removing it completely.  The printk is a out-of-character compared to other

Re: [Xen-devel] Xen ARM dom0less use PV drivers

2019-04-10 Thread Julien Grall
Hi, CCing Stefano who is looking after dom0less. On 08/04/2019 03:17, jinchen wrote: Hello experts:     The xen 4.12 support the Dom0less VM that start domu from xen not xl, but the PV drivers hasn't been supported. Do you have some guidance for how to enable PV drivers when using dom0less?

[Xen-devel] [linux-4.19 test] 134556: regressions - trouble: blocked/broken/fail/pass

2019-04-10 Thread osstest service owner
flight 134556 linux-4.19 real [real] http://logs.test-lab.xenproject.org/osstest/logs/134556/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64-pvopsbroken build-arm64

Re: [Xen-devel] [PATCH v3 2/3] xen: remove tmem from hypervisor

2019-04-10 Thread Wei Liu
On Wed, Apr 10, 2019 at 12:57:57PM +0100, Wei Liu wrote: > On Wed, Apr 10, 2019 at 04:55:50AM -0600, Jan Beulich wrote: > > >>> On 09.04.19 at 18:52, wrote: > > > --- a/xen/arch/x86/guest/hypercall_page.S > > > +++ b/xen/arch/x86/guest/hypercall_page.S > > > @@ -58,8 +58,8 @@

Re: [Xen-devel] [PATCH v3 2/3] xen: remove tmem from hypervisor

2019-04-10 Thread Wei Liu
On Wed, Apr 10, 2019 at 06:25:30AM -0600, Jan Beulich wrote: > >>> On 10.04.19 at 14:21, wrote: > > On Wed, Apr 10, 2019 at 06:12:35AM -0600, Jan Beulich wrote: > >> >>> On 10.04.19 at 13:57, wrote: > >> > On Wed, Apr 10, 2019 at 04:55:50AM -0600, Jan Beulich wrote: > >> >> >>> On 09.04.19 at

Re: [Xen-devel] [PATCH] x86/mem-sharing: statically initialize audit list head and lock

2019-04-10 Thread Tamas K Lengyel
On Wed, Apr 10, 2019 at 6:17 AM Jan Beulich wrote: > > >>> On 10.04.19 at 13:20, wrote: > > On 4/10/19 12:13 PM, Andrew Cooper wrote: > >> On 10/04/2019 11:58, Jan Beulich wrote: > >>> There's no need to execute any instructions for doing so. > >>> > >>> Signed-off-by: Jan Beulich > >>> --- >

[Xen-devel] [OSSTEST PATCH 55/62] make-flight: shadow test: Disable kpti in guests

2019-04-10 Thread Ian Jackson
Since Spectre/Meltdown, shadow has been a lot slower, especially with KPTI in the guest. Empirically, too slow (with the kernel from Debian stretch). CC: Andrew Cooper Signed-off-by: Ian Jackson --- make-flight | 1 + 1 file changed, 1 insertion(+) diff --git a/make-flight b/make-flight

[Xen-devel] [OSSTEST PATCH 58/62] dm restrict, fishdescriptor: Update to a fixed chiark-scripts

2019-04-10 Thread Ian Jackson
I have only just fixed a bug which stops our test from working and the fix is not upstream yet. Signed-off-by: Ian Jackson --- production-config | 1 + 1 file changed, 1 insertion(+) diff --git a/production-config b/production-config index 59c74cca..fadfe8b9 100644 --- a/production-config +++

[Xen-devel] [OSSTEST PATCH 57/62] platforms: Honour suite in get_arch_platforms

2019-04-10 Thread Ian Jackson
The available platforms may depend on the suite to be used. Actually implement that for HostDB::Executive. For Static, we leave it to the user. Signed-off-by: Ian Jackson --- Osstest/HostDB/Executive.pm | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git

[Xen-devel] [OSSTEST PATCH 36/62] TestSupport: target_somefile_leaf rename and change a variable

2019-04-10 Thread Ian Jackson
Rename this function. `getleaf' contains `get' which makes it sound like the function copies something, or returns answers suitable for getting, or something. Also rename `$rdest' to `$rfile' since it might be a source too. (Although we are not about to make it a source...) Signed-off-by: Ian

[Xen-devel] [OSSTEST PATCH 21/62] ts-kernel-build: disable host1x, which doesn't build

2019-04-10 Thread Ian Jackson
From: Wei Liu Empirically, on stretch armhf: drivers/gpu/host1x/cdma.c: In function `host1x_pushbuffer_init': drivers/gpu/host1x/cdma.c:94:48: error: passing argument 3 of `dma_alloc_wc' from incompatible pointer type [-Werror=incompatible-pointer-types] pb->mapped =

[Xen-devel] [OSSTEST PATCH 39/62] preseed_base: chmod ssh host private keys to placate sshd

2019-04-10 Thread Ian Jackson
Otherwise: Could not load host key: /etc/ssh/ssh_host_ecdsa_key @@@ @ WARNING: UNPROTECTED PRIVATE KEY FILE! @ @@@ Permissions 0640 for

[Xen-devel] [OSSTEST PATCH 59/62] Debian: Fix /lib/udev/ifupdown-hotplug to not run if / is ro

2019-04-10 Thread Ian Jackson
Empirically, without this, on Debian stretch: 1. udev starts before / is remounted rw 2. udev spots eth0 and runs /lib/udev/ifupdown-hotplug 3. ifupdown-hotplug runs ifup which runs isc-dhcp-client 4. isc-dhcp-client talks to the dhcp server to get an address 5. isc-dhcp-client tries to

Re: [edk2-devel] [PATCH v2 02/31] OvmfPkg: Create platform XenOvmf

2019-04-10 Thread Laszlo Ersek
On 04/10/19 11:48, Jordan Justen wrote: > On 2019-04-09 04:08:15, Anthony PERARD wrote: >> This is a copy of OvmfX64, removing VirtIO and some SMM. >> >> This new platform will be changed to make it works on two types of Xen >> guest: HVM and PVH. >> >> Compare to OvmfX64, this patch: >> >> -

[Xen-devel] [OSSTEST PATCH 51/62] dm restrict audit: always install (some) chiark-scripts

2019-04-10 Thread Ian Jackson
In dm restrict audit: install newer chiark-scripts for fishdescriptor arrangements were made to install suitable chiark-scripts for for jessie and stretch. For buster and later, the mainline Debian version of chiark-scripts is indeed sufficient, but nothing installed it. Do that.

[Xen-devel] [OSSTEST PATCH 46/62] Debian: set partman-lvm/device_remove_lvm_span

2019-04-10 Thread Ian Jackson
Web searching[1] suggests that this suppresses this error: !! ERROR: Unable to automatically remove LVM data Because the volume group(s) on the selected device also consist of physical volumes on other devices, it is not considered safe to remove its LVM data automatically. If you wish to

[Xen-devel] [OSSTEST PATCH 07/62] ts-leak-check: suppress systemd-shim, which leaks in stretch

2019-04-10 Thread Ian Jackson
From: Wei Liu Signed-off-by: Wei Liu Acked-by: Ian Jackson --- ts-leak-check | 1 + 1 file changed, 1 insertion(+) diff --git a/ts-leak-check b/ts-leak-check index 678d0696..41e6245d 100755 --- a/ts-leak-check +++ b/ts-leak-check @@ -202,6 +202,7 @@ xenstore /vm xenstore /libxl process

[Xen-devel] [OSSTEST PATCH 45/62] Debian: partman scripts: Run right away too

2019-04-10 Thread Ian Jackson
We are switching the installation of these to partman/early_command which runs as a result of a /lib/partman/init.d hook. That means that things we install don't get picked up, so run them right away (too). Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 4 ++-- 1 file changed, 2

[Xen-devel] [OSSTEST PATCH 49/62] Debian: Move preseed_backports_packages earlier

2019-04-10 Thread Ian Jackson
No functional change. Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 88 +++ 1 file changed, 44 insertions(+), 44 deletions(-) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index 9aa88822..25bf8e88 100644 --- a/Osstest/Debian.pm +++

[Xen-devel] [OSSTEST PATCH 53/62] ts-xen-install: Install libpciaccess0

2019-04-10 Thread Ian Jackson
In "ts-xen-build-prep: install packages for suites >jessie", libpciaccess-dev was added for the benefit of qemu. libvirt needs it too. We also need the runtime library. Without it, libvirt does not start: 2019-04-04 22:35:36.760+: 3623: error : virModuleLoadFile:53 : internal error:

Re: [Xen-devel] [PATCH 3/3] xen-bus / xen-block: add support for event channel polling

2019-04-10 Thread Anthony PERARD
On Mon, Apr 08, 2019 at 04:16:17PM +0100, Paul Durrant wrote: > This patch introduces a poll callback for event channel fd-s and uses > this to invoke the channel callback function. > > To properly support polling, it is necessary for the event channel callback > function to return a boolean

Re: [Xen-devel] [PATCH] xen-block: support feature-large-sector-size

2019-04-10 Thread Anthony PERARD
On Tue, Apr 09, 2019 at 05:40:38PM +0100, Paul Durrant wrote: > A recent Xen commit [1] clarified the semantics of sector based quantities > used in the blkif protocol such that it is now safe to create a xen-block > device with a logical_block_size != 512, as long as the device only > connects to

Re: [edk2-devel] [PATCH 0/4] OvmfPkg: replace MIT license blocks with SPDX IDs

2019-04-10 Thread Ard Biesheuvel
On Wed, 10 Apr 2019 at 05:58, Laszlo Ersek wrote: > > Repo: https://github.com/lersek/edk2.git > Branch: ovmf_spdx_mit > > For , we replaced > open-coded license text blocks with "SPDX-License-Identifier"s, almost > all over the edk2 tree. >

Re: [Xen-devel] [PATCH 2/3] xen-bus: allow AioContext to be specified for each event channel

2019-04-10 Thread Anthony PERARD
On Wed, Apr 10, 2019 at 04:20:05PM +0100, Paul Durrant wrote: > > -Original Message- > > From: Anthony PERARD [mailto:anthony.per...@citrix.com] > > Sent: 10 April 2019 13:57 > > To: Paul Durrant > > Cc: qemu-de...@nongnu.org; qemu-bl...@nongnu.org; > > xen-devel@lists.xenproject.org;

Re: [Xen-devel] [OSSTEST PATCH 21/62] ts-kernel-build: disable host1x, which doesn't build

2019-04-10 Thread Thierry Reding
On Wed, Apr 10, 2019 at 03:23:27PM +0100, Ian Jackson wrote: > From: Wei Liu > > Empirically, on stretch armhf: > > drivers/gpu/host1x/cdma.c: In function `host1x_pushbuffer_init': > drivers/gpu/host1x/cdma.c:94:48: error: passing argument 3 of > `dma_alloc_wc' from incompatible pointer

Re: [Xen-devel] [PATCH 2/3] xen-bus: allow AioContext to be specified for each event channel

2019-04-10 Thread Paul Durrant
> -Original Message- > From: Anthony PERARD [mailto:anthony.per...@citrix.com] > Sent: 10 April 2019 13:57 > To: Paul Durrant > Cc: qemu-de...@nongnu.org; qemu-bl...@nongnu.org; > xen-devel@lists.xenproject.org; Stefano Stabellini > ; Stefan Hajnoczi ; Kevin Wolf > ; Max > Reitz >

Re: [Xen-devel] [OSSTEST PATCH 21/62] ts-kernel-build: disable host1x, which doesn't build

2019-04-10 Thread Julien Grall
Hi, On 10/04/2019 16:27, Thierry Reding wrote: On Wed, Apr 10, 2019 at 03:23:27PM +0100, Ian Jackson wrote: From: Wei Liu Empirically, on stretch armhf: drivers/gpu/host1x/cdma.c: In function `host1x_pushbuffer_init': drivers/gpu/host1x/cdma.c:94:48: error: passing argument 3 of

Re: [Xen-devel] [OSSTEST PATCH 13/62] Extend workaround `clk_ignore_unused' to stretch

2019-04-10 Thread Julien Grall
Hi, On 10/04/2019 15:23, Ian Jackson wrote: From: Wei Liu This is https://bugs.xenproject.org/xen/bug/45 Still no resolution for this one :/. Without that parameter we lose uart output. Signed-off-by: Wei Liu CC: Stefano Stabellini CC: Julien Grall Acked-by: Julien Grall Cheers,

[Xen-devel] Ping SVM: [PATCH for-4.12] x86/svm: Fix handling of ICEBP intercepts

2019-04-10 Thread Andrew Cooper
The discovery of further cascade errors doesn't change the need for this patch. Its obviously missed the boat for 4.12, but still needed. I haven't reposed with the typo which Tamas fixed, but will fold that in on commit.  (I didn't think it worthy of sending out a v2, given the obviousness of

Re: [Xen-devel] [PATCH v3 1/3] x86/mm: Introduce altp2m_get_gfn_type_access

2019-04-10 Thread George Dunlap
On 4/9/19 1:03 PM, Alexandru Stefan ISAILA wrote: > This patch moves common code from p2m_set_altp2m_mem_access() and > p2m_change_altp2m_gfn() into one function > > Signed-off-by: Alexandru Isaila This patch contains a lot of behavioral changes which aren't mentioned or explained. For

Re: [Xen-devel] status of non-live migration of HVM with libvirt

2019-04-10 Thread Anthony PERARD
On Mon, Jan 07, 2019 at 11:07:43AM +0100, Olaf Hering wrote: > Am Fri, 4 Jan 2019 17:48:31 +0100 > schrieb Olaf Hering : > > > Am Fri, 4 Jan 2019 16:57:55 +0100 > > schrieb Olaf Hering : > > > > > worth keeping (and fixing) the concept of an "offline migration" > > > > And regarding the fix,

Re: [edk2-devel] [PATCH v2 02/31] OvmfPkg: Create platform XenOvmf

2019-04-10 Thread Ard Biesheuvel
On Wed, 10 Apr 2019 at 07:27, Laszlo Ersek wrote: > > On 04/10/19 11:48, Jordan Justen wrote: > > On 2019-04-09 04:08:15, Anthony PERARD wrote: > >> This is a copy of OvmfX64, removing VirtIO and some SMM. > >> > >> This new platform will be changed to make it works on two types of Xen > >>

Re: [Xen-devel] [OSSTEST PATCH 55/62] make-flight: shadow test: Disable kpti in guests

2019-04-10 Thread Andrew Cooper
On 10/04/2019 15:24, Ian Jackson wrote: > Since Spectre/Meltdown, shadow has been a lot slower, especially with > KPTI in the guest. Empirically, too slow (with the kernel from Debian > stretch). The speed of shadow pagetables hasn't changed - I don't think we even touched the shadow code at all

Re: [Xen-devel] [PATCH v1] libxl: fix migration of PV and PVH domUs with and without qemu

2019-04-10 Thread Olaf Hering
Am Wed, 10 Apr 2019 12:33:38 +0200 schrieb Olaf Hering : > goto error_out; This one should have been removed, now libxl fails right away. Will do more runtime testing. Waiting for feedback before sending v2. Olaf pgpVcn1cnjcYW.pgp Description: Digitale Signatur von OpenPGP

[Xen-devel] [xen-unstable-smoke test] 134597: regressions - trouble: blocked/broken/fail/pass

2019-04-10 Thread osstest service owner
flight 134597 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/134597/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-arm64-xsm broken build-arm64-xsm 4

Re: [Xen-devel] [PATCH 2/3] xen-bus: allow AioContext to be specified for each event channel

2019-04-10 Thread Paul Durrant
> -Original Message- > From: Anthony PERARD [mailto:anthony.per...@citrix.com] > Sent: 10 April 2019 16:23 > To: Paul Durrant > Cc: qemu-de...@nongnu.org; qemu-bl...@nongnu.org; > xen-devel@lists.xenproject.org; Stefano Stabellini > ; Stefan Hajnoczi ; Kevin Wolf > ; Max > Reitz >

Re: [Xen-devel] [PATCH for-4.12] x86/svm: Fix handling of ICEBP intercepts

2019-04-10 Thread Woods, Brian
On 2/1/19 8:49 AM, Andrew Cooper wrote: > c/s 9338a37d "x86/svm: implement debug events" added support for introspecting > ICEBP debug exceptions, but didn't account for the fact that > svm_get_insn_len() (previously __get_instruction_length) can fail and may > already raise #GP for the guest. >

Re: [Xen-devel] [PATCH] xen-block: support feature-large-sector-size

2019-04-10 Thread Paul Durrant
> -Original Message- > From: Anthony PERARD [mailto:anthony.per...@citrix.com] > Sent: 10 April 2019 16:52 > To: Paul Durrant > Cc: qemu-de...@nongnu.org; xen-devel@lists.xenproject.org; > qemu-bl...@nongnu.org; Stefano Stabellini > ; Stefan Hajnoczi ; Kevin Wolf > ; Max > Reitz >

Re: [Xen-devel] [OSSTEST PATCH 55/62] make-flight: shadow test: Disable kpti in guests

2019-04-10 Thread Ian Jackson
Andrew Cooper writes ("Re: [OSSTEST PATCH 55/62] make-flight: shadow test: Disable kpti in guests"): > On 10/04/2019 15:24, Ian Jackson wrote: > > Since Spectre/Meltdown, shadow has been a lot slower, especially with > > KPTI in the guest. Empirically, too slow (with the kernel from Debian > >

Re: [Xen-devel] [PATCH] x86/mem-sharing: statically initialize audit list head and lock

2019-04-10 Thread Jan Beulich
>>> On 10.04.19 at 13:20, wrote: > On 4/10/19 12:13 PM, Andrew Cooper wrote: >> On 10/04/2019 11:58, Jan Beulich wrote: >>> There's no need to execute any instructions for doing so. >>> >>> Signed-off-by: Jan Beulich >>> --- >>> I wonder whether mem_sharing_init() shouldn't go away altogether

[Xen-devel] [OSSTEST PATCH 62/62] Switch to Debian Stretch

2019-04-10 Thread Ian Jackson
From: Wei Liu Signed-off-by: Wei Liu Signed-off-by: Ian Jackson --- Osstest.pm| 2 +- production-config | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/Osstest.pm b/Osstest.pm index 92b1a0ea..7ce53fcb 100644 --- a/Osstest.pm +++ b/Osstest.pm @@ -87,7 +87,7 @@ our

[Xen-devel] [OSSTEST PATCH 28/62] ts-host-install: Unconditionally mkdir -p /etc/udev/rules.d

2019-04-10 Thread Ian Jackson
We are going to want this directory to exist so that we can put a canary in 70-persistent-net.rules. In the cases where the behaviour of osstest changes, the empty directory does not result in any overall change. Signed-off-by: Ian Jackson --- ts-host-install | 3 ++- 1 file changed, 2

[Xen-devel] [OSSTEST PATCH 14/62] adjust how to skip bootloader installation for arm32, in Stretch

2019-04-10 Thread Ian Jackson
From: Wei Liu Signed-off-by: Wei Liu Acked-by: Ian Jackson --- v4: Change case of suite names in comment. --- Osstest/Debian.pm | 4 1 file changed, 4 insertions(+) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index 91bffdff..85b1890d 100644 --- a/Osstest/Debian.pm +++

Re: [Xen-devel] [PATCH] x86: fix build race when generating temporary object files

2019-04-10 Thread Andrew Cooper
On 09/04/2019 15:57, Jan Beulich wrote: > The rules to generate xen-syms and xen.efi may run in parallel, but both > recursively invoke $(MAKE) to build symbol/relocation table temporary > object files. These recursive builds would both re-generate the .*.d2 > files (where needed). Both would in

[edk2-devel] [PATCH 3/4] OvmfPkg/IndustryStandard/Xen: replace MIT license text with SPDX ID

2019-04-10 Thread Laszlo Ersek
The header files under "OvmfPkg/Include/IndustryStandard/Xen" include the MIT License text. Replace those license blocks with "SPDX-License-Identifier: MIT" (which refers to ) for easier parsing by machines (and humans). This follows the tree-wide adoption of

[edk2-devel] [PATCH 1/4] OvmfPkg/License.txt: remove XenPvBlkDxe from the MIT licensed dir list

2019-04-10 Thread Laszlo Ersek
All files under "OvmfPkg/XenPvBlkDxe" carry SPDX-License-Identifier: BSD-2-Clause-Patent None of them are MIT-licensed. Update "OvmfPkg/License.txt" accordingly. Cc: Anthony Perard Cc: Ard Biesheuvel Cc: Jordan Justen Cc: Julien Grall Cc: Lars Kurth Cc: xen-devel@lists.xenproject.org

[edk2-devel] [PATCH 2/4] OvmfPkg/License.txt: refresh the MIT license text and include the SPDX ID

2019-04-10 Thread Laszlo Ersek
Refresh the MIT license text from , and add "SPDX-License-Identifier: MIT" for easier parsing by machines. This follows the tree-wide adoption of "SPDX-License-Identifier"s, made for ; in particular: -

[edk2-devel] [PATCH 0/4] OvmfPkg: replace MIT license blocks with SPDX IDs

2019-04-10 Thread Laszlo Ersek
Repo: https://github.com/lersek/edk2.git Branch: ovmf_spdx_mit For , we replaced open-coded license text blocks with "SPDX-License-Identifier"s, almost all over the edk2 tree. That change however was tied to a license update / CLA update too

[edk2-devel] [PATCH 4/4] OvmfPkg/XenBusDxe: replace MIT license text with SPDX ID

2019-04-10 Thread Laszlo Ersek
The "XenBus.c", "XenStore.c", "XenStore.h" files under "OvmfPkg/XenBusDxe" include the MIT License text. Replace those license blocks with "SPDX-License-Identifier: MIT" (which refers to ) for easier parsing by machines (and humans). This follows the tree-wide

Re: [Xen-devel] [PATCH v3 2/3] x86/mm: Introduce altp2m_set_entry_by_page_order

2019-04-10 Thread George Dunlap
On 4/9/19 1:03 PM, Alexandru Stefan ISAILA wrote: > This patch moves common code from p2m_set_altp2m_mem_access() and > p2m_change_altp2m_gfn() into one function > > Signed-off-by: Alexandru Isaila > --- > xen/arch/x86/mm/mem_access.c | 2 +- > xen/include/asm-x86/p2m.h| 11 +++ >

[Xen-devel] [OSSTEST PATCH 47/62] Debian: Add reference to bug numbers for erase-other-disks

2019-04-10 Thread Ian Jackson
Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index 073b776c..79b7960d 100644 --- a/Osstest/Debian.pm +++ b/Osstest/Debian.pm @@ -1171,6 +1171,7 @@ END2 chmod +x parted_devices END +# Work around

Re: [Xen-devel] [PATCH 2/3] xen-bus: allow AioContext to be specified for each event channel

2019-04-10 Thread Anthony PERARD
On Mon, Apr 08, 2019 at 04:16:16PM +0100, Paul Durrant wrote: > This patch adds an AioContext parameter to xen_device_bind_event_channel() > and then uses aio_set_fd_handler() to set the callback rather than > qemu_set_fd_handler(). > > Signed-off-by: Paul Durrant > --- > @@ -943,6 +944,7 @@

[Xen-devel] [OSSTEST PATCH 23/62] contents_make_cpio: Make files owned by root

2019-04-10 Thread Ian Jackson
This function is called to generate overlays for use, mainly, by the initramfs. We are going to use it to ship udev rules. Annoyingly, udev hates files which aren't owned by root - it simply ignores them. Signed-off-by: Ian Jackson --- Osstest/TestSupport.pm | 2 +- 1 file changed, 1

[Xen-devel] [OSSTEST PATCH 60/62] Debian: Fix /lib/udev/ifupdown-hotplug in guests from debootstrap

2019-04-10 Thread Ian Jackson
The ifupdown-hotplug problem just described affects guests too. Empirically, this occurs only with the guests from xen-tools. In my osstest stretch series development tests this resulted in occasional failures of ts-guest-start. The problem is not deterministic; and it may be that it is a race

[Xen-devel] [OSSTEST PATCH 24/62] overlay-persistent-net: Copy from jessie

2019-04-10 Thread Ian Jackson
These were copied from a system running Debian jessie. The nontrivial files are: # Copyright (C) 2006 Marco d'Itri # Copyright (C) 2007 Kay Sievers and licenced GPLv2+. That is compatible with osstest's AGPLv3+. Right now we do nothing with these. Signed-off-by: Ian Jackson ---

[Xen-devel] [OSSTEST PATCH 16/62] make-flight: guest should use jessie to test pvgrub

2019-04-10 Thread Ian Jackson
From: Wei Liu stretch has 64bit feature enabled for ext4, which pvgrub can't cope. We want to continue to test pvgrub, so specify jessie in the guest suite field. A consequence is that this test will test jessie forever. Eventually jessie will rot so badly that this test fails and then we will

[Xen-devel] [OSSTEST PATCH 50/62] dm restrict audit: actually install fishdescriptor in host

2019-04-10 Thread Ian Jackson
In dm restrict audit: install newer chiark-scripts for fishdescriptor arrangements were made to install a backport of chiark-scripts but the code was mistakenly placed in preseed_create_guest but of course it's needed in the host. Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 11

[Xen-devel] [OSSTEST PATCH 43/62] preseed_hook_installscript: Use partman/early_command, not preseed/

2019-04-10 Thread Ian Jackson
On iso-based installs, with stretch, preseed/early_command runs before the network is up. This causes the install to fail. Our existing call sites add things to /usr/lib/base-installer.d/ /lib/partman/init.d/ for which this is still early enough. Signed-off-by: Ian Jackson ---

[Xen-devel] [OSSTEST PATCH 19/62] Debian: Fix http:// url for bugs.xenproject.org

2019-04-10 Thread Ian Jackson
Signed-off-by: Ian Jackson --- Osstest/Debian.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Osstest/Debian.pm b/Osstest/Debian.pm index 80b4cf37..414cd897 100644 --- a/Osstest/Debian.pm +++ b/Osstest/Debian.pm @@ -239,7 +239,7 @@ END # Dom0 specific kernel

  1   2   >