[xen-unstable test] 165480: tolerable FAIL - PUSHED

2021-10-12 Thread osstest service owner
flight 165480 xen-unstable real [real] flight 165490 xen-unstable real-retest [real] http://logs.test-lab.xenproject.org/osstest/logs/165480/ http://logs.test-lab.xenproject.org/osstest/logs/165490/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-armh

Re: [PATCH v6 2/2] arm/efi: load dom0 modules from DT using UEFI

2021-10-12 Thread Jan Beulich
On 13.10.2021 02:49, Stefano Stabellini wrote: > On Tue, 12 Oct 2021, Luca Fancellu wrote: >>> On 12 Oct 2021, at 02:31, Stefano Stabellini wrote: >>> >>> On Mon, 11 Oct 2021, Julien Grall wrote: Hi Stefano, On 11/10/2021 22:24, Stefano Stabellini wrote: >> diff --git a/xen/arch

Re: [PATCH v5 07/11] xen/domctl: Introduce XEN_DOMCTL_CDF_vpci flag

2021-10-12 Thread Jan Beulich
On 12.10.2021 23:48, Stefano Stabellini wrote: > On Mon, 11 Oct 2021, Roger Pau Monné wrote: >> On Wed, Oct 06, 2021 at 06:40:33PM +0100, Rahul Singh wrote: >>> Introduce XEN_DOMCTL_CDF_vpci flag to enable VPCI support in XEN. >>> Reject the use of this new flag for x86 as VPCI is not supported for

Re: [PATCH v5 08/11] xen/arm: Enable the existing x86 virtual PCI support for ARM.

2021-10-12 Thread Jan Beulich
On 12.10.2021 23:37, Stefano Stabellini wrote: > On Tue, 12 Oct 2021, Jan Beulich wrote: >> On 11.10.2021 20:18, Stefano Stabellini wrote: >>> On Mon, 11 Oct 2021, Jan Beulich wrote: On 11.10.2021 15:34, Bertrand Marquis wrote: >> On 11 Oct 2021, at 14:09, Jan Beulich wrote: >> On 11.

Re: [RFC PATCH v2 00/16] Initial support for machine creation via QMP

2021-10-12 Thread Mark Burton
Fixed Cheers Mark. > On 13 Oct 2021, at 00:16, Alistair Francis wrote: > > On Thu, Sep 23, 2021 at 2:22 AM Damien Hedde > wrote: >> >> Hi, >> >> The goal of this work is to bring dynamic machine creation to QEMU: >> we want to setup a machine without compiling a specific machine C >> code.

Re: [PATCH xen 2/2] xen: update system time immediately when VCPUOP_register_vcpu_info

2021-10-12 Thread Jan Beulich
On 12.10.2021 18:15, Dongli Zhang wrote: > Hi Jan, > > On 10/12/21 8:49 AM, Jan Beulich wrote: >> On 12.10.2021 17:43, Dongli Zhang wrote: >>> Hi Jan, >>> >>> On 10/12/21 1:40 AM, Jan Beulich wrote: On 12.10.2021 09:24, Dongli Zhang wrote: > The guest may access the pv vcpu_time_info imme

[xen-unstable-smoke test] 165485: tolerable all pass - PUSHED

2021-10-12 Thread osstest service owner
flight 165485 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/165485/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 15 migrate-support-checkfail never pass test-arm64-arm64-xl-xsm 1

[qemu-mainline test] 165477: tolerable FAIL - PUSHED

2021-10-12 Thread osstest service owner
flight 165477 qemu-mainline real [real] http://logs.test-lab.xenproject.org/osstest/logs/165477/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-xl-qemuu-win7-amd64 19 guest-stopfail like 165468 test-amd64-amd64-qemuu-nested-amd 2

[linux-5.4 test] 165476: regressions - FAIL

2021-10-12 Thread osstest service owner
flight 165476 linux-5.4 real [real] flight 165484 linux-5.4 real-retest [real] http://logs.test-lab.xenproject.org/osstest/logs/165476/ http://logs.test-lab.xenproject.org/osstest/logs/165484/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: t

[PATCH AUTOSEL 5.10 05/11] xen/x86: prevent PVH type from getting clobbered

2021-10-12 Thread Sasha Levin
From: Jan Beulich [ Upstream commit 9172b5c4a778da1f855b2e3780b1afabb3cfd523 ] Like xen_start_flags, xen_domain_type gets set before .bss gets cleared. Hence this variable also needs to be prevented from getting put in .bss, which is possible because XEN_NATIVE is an enumerator evaluating to zer

[PATCH AUTOSEL 5.14 06/17] xen/x86: prevent PVH type from getting clobbered

2021-10-12 Thread Sasha Levin
From: Jan Beulich [ Upstream commit 9172b5c4a778da1f855b2e3780b1afabb3cfd523 ] Like xen_start_flags, xen_domain_type gets set before .bss gets cleared. Hence this variable also needs to be prevented from getting put in .bss, which is possible because XEN_NATIVE is an enumerator evaluating to zer

Re: [PATCH v5 2/3] xen/arm: Check for PMU platform support

2021-10-12 Thread Stefano Stabellini
On Tue, 12 Oct 2021, Julien Grall wrote: > Hi Michal, > > On 12/10/2021 09:13, Michal Orzel wrote: > > ID_AA64DFR0_EL1/ID_DFR0_EL1 registers provide > > information about PMU support. Replace structure > > dbg64/dbg32 with a union and fill in all the > > register fields according to document: > >

Re: [PATCH v6 1/2] arm/efi: Use dom0less configuration when using EFI boot

2021-10-12 Thread Stefano Stabellini
On Tue, 12 Oct 2021, Jan Beulich wrote: > On 12.10.2021 11:05, Luca Fancellu wrote: > >> On 12 Oct 2021, at 09:45, Jan Beulich wrote: > >> On 11.10.2021 20:15, Luca Fancellu wrote: > >>> --- a/xen/common/efi/boot.c > >>> +++ b/xen/common/efi/boot.c > >>> @@ -166,6 +166,13 @@ static void __init Pri

Re: [PATCH v6 2/2] arm/efi: load dom0 modules from DT using UEFI

2021-10-12 Thread Stefano Stabellini
On Tue, 12 Oct 2021, Luca Fancellu wrote: > > On 12 Oct 2021, at 02:31, Stefano Stabellini wrote: > > > > On Mon, 11 Oct 2021, Julien Grall wrote: > >> Hi Stefano, > >> > >> On 11/10/2021 22:24, Stefano Stabellini wrote: > diff --git a/xen/arch/arm/efi/efi-boot.h b/xen/arch/arm/efi/efi-boot

Re: [PATCH v2] xen/arm: vgic to ignore GICD ICPENRn registers access

2021-10-12 Thread Stefano Stabellini
On Tue, 12 Oct 2021, Julien Grall wrote: > On 12/10/2021 07:24, Hongda Deng wrote: > > Currently, Xen will return IO unhandled when guests access GICD ICPENRn > > registers. This will raise a data abort inside guest. For Linux Guest, > > these virtual registers will not be accessed. But for Zephyr,

Re: [PATCH v6 00/11] PCI: Drop duplicated tracking of a pci_dev's bound driver

2021-10-12 Thread Bjorn Helgaas
On Mon, Oct 04, 2021 at 02:59:24PM +0200, Uwe Kleine-König wrote: > Hello, > > this is v6 of the quest to drop the "driver" member from struct pci_dev > which tracks the same data (apart from a constant offset) as dev.driver. I like this a lot and applied it to pci/driver for v5.16, thanks! I sp

Re: [RFC PATCH v2 09/16] hw/core/machine: Remove the dynamic sysbus devices type check

2021-10-12 Thread Alistair Francis
On Thu, Sep 23, 2021 at 2:23 AM Damien Hedde wrote: > > Now that we check sysbus device types during device creation, we > can remove the check done in the machine init done notifier. > This was the only thing done by this notifier, so we remove the > whole sysbus_notifier structure of the Machine

[xen-unstable-smoke test] 165482: tolerable all pass - PUSHED

2021-10-12 Thread osstest service owner
flight 165482 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/165482/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 15 migrate-support-checkfail never pass test-arm64-arm64-xl-xsm 1

Re: [RFC PATCH v2 08/16] qdev-monitor: Check sysbus device type before creating it

2021-10-12 Thread Alistair Francis
On Thu, Sep 23, 2021 at 2:53 AM Damien Hedde wrote: > > Add an early check to test if the requested sysbus device type > is allowed by the current machine before creating the device. This > impacts both -device cli option and device_add qmp command. > > Before this patch, the check was done well a

Re: [RFC PATCH v2 06/16] qapi: Allow device_add to execute in machine initialized phase

2021-10-12 Thread Alistair Francis
On Thu, Sep 23, 2021 at 2:26 AM Damien Hedde wrote: > > From: Mirela Grujic > > To configure a machine using QMP we need the device_add command to > execute at machine initialized phase. > > Note: for device_add command in qdev.json adding the 'allow-init-config' > option has no effect because th

Re: [RFC PATCH v2 03/16] qapi: Implement x-machine-init QMP command

2021-10-12 Thread Alistair Francis
On Thu, Sep 23, 2021 at 2:17 AM Damien Hedde wrote: > > From: Mirela Grujic > > The x-machine-init QMP command is available only if the -preconfig option > is used and the current machine initialization phase is accel-created. > > The command triggers QEMU to enter machine initialized phase and w

Re: [RFC PATCH v2 00/16] Initial support for machine creation via QMP

2021-10-12 Thread Alistair Francis
On Thu, Sep 23, 2021 at 2:22 AM Damien Hedde wrote: > > Hi, > > The goal of this work is to bring dynamic machine creation to QEMU: > we want to setup a machine without compiling a specific machine C > code. It would ease supporting highly configurable platforms (for > example resulting from an au

Re: [RFC PATCH v2 02/16] qapi: Implement query-machine-phase QMP command

2021-10-12 Thread Alistair Francis
On Thu, Sep 23, 2021 at 2:20 AM Damien Hedde wrote: > > From: Mirela Grujic > > The command returns current machine initialization phase. > From now on, the MachineInitPhase enum is generated from the > QAPI schema. > > Signed-off-by: Mirela Grujic Acked-by: Alistair Francis Alistair > --- >

Re: [PATCH v5 2/3] xen/arm: Check for PMU platform support

2021-10-12 Thread Julien Grall
Hi Michal, On 12/10/2021 09:13, Michal Orzel wrote: ID_AA64DFR0_EL1/ID_DFR0_EL1 registers provide information about PMU support. Replace structure dbg64/dbg32 with a union and fill in all the register fields according to document: ARM Architecture Registers(DDI 0595, 2021-06). Add macros boot_d

Re: [PATCH v2] xen/arm: vgic to ignore GICD ICPENRn registers access

2021-10-12 Thread Julien Grall
Hi, On 12/10/2021 07:24, Hongda Deng wrote: Currently, Xen will return IO unhandled when guests access GICD ICPENRn registers. This will raise a data abort inside guest. For Linux Guest, these virtual registers will not be accessed. But for Zephyr, in its GIC initialization code, these virtual r

Re: [PATCH v5 07/11] xen/domctl: Introduce XEN_DOMCTL_CDF_vpci flag

2021-10-12 Thread Stefano Stabellini
On Mon, 11 Oct 2021, Roger Pau Monné wrote: > On Wed, Oct 06, 2021 at 06:40:33PM +0100, Rahul Singh wrote: > > Introduce XEN_DOMCTL_CDF_vpci flag to enable VPCI support in XEN. > > Reject the use of this new flag for x86 as VPCI is not supported for > > DOMU guests for x86. > > I don't like this a

Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions for DomU

2021-10-12 Thread Oleksandr
On 13.10.21 00:22, Julien Grall wrote: Hi Oleksandr, Hi Julien, Ian. Julien, thank you for the detailed answer, I will analyze it tomorrow. Ian, I think, there is no reason in providing git branch with the acks folded in + my minor fix for the debug message as it was discussed before, it

Re: [PATCH v5 08/11] xen/arm: Enable the existing x86 virtual PCI support for ARM.

2021-10-12 Thread Stefano Stabellini
On Tue, 12 Oct 2021, Jan Beulich wrote: > On 11.10.2021 20:18, Stefano Stabellini wrote: > > On Mon, 11 Oct 2021, Jan Beulich wrote: > >> On 11.10.2021 15:34, Bertrand Marquis wrote: > On 11 Oct 2021, at 14:09, Jan Beulich wrote: > On 11.10.2021 14:41, Bertrand Marquis wrote: > > But

Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions for DomU

2021-10-12 Thread Julien Grall
Hi Oleksandr, On 12/10/2021 18:42, Oleksandr wrote: On 12.10.21 19:05, Julien Grall wrote: On 11/10/2021 18:48, Oleksandr Tyshchenko wrote: ---   tools/libs/light/libxl_arm.c  | 76 ---   xen/include/public/arch-arm.h |  2 ++   2 files changed, 73 inser

[PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo

2021-10-12 Thread Oleksandr Tyshchenko
From: Oleksandr Tyshchenko We need to pass info about maximum supported guest physical address space size to the toolstack on Arm in order to properly calculate the base and size of the extended region (safe range) for the guest. The extended region is unused address space which could be safely u

Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.

2021-10-12 Thread Stefano Stabellini
On Tue, 12 Oct 2021, Ian Jackson wrote: > Bertrand Marquis writes ("Re: [PATCH v5 01/11] xen/arm: > xc_domain_ioport_permission(..) not supported on ARM."): > > I can add something in the commit message about the fact that we improve > > performance and prevent to do a call that is and will not be

Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo

2021-10-12 Thread Oleksandr
On 12.10.21 21:07, Ian Jackson wrote: Hi Ian Oleksandr writes ("Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo"): Please note, it is going to be a new version of this patch based on today's discussion with Jan. OK. Please retain my R-b if you do

Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo

2021-10-12 Thread Ian Jackson
Oleksandr writes ("Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo"): > Please note, it is going to be a new version of this patch based on > today's discussion with Jan. OK. Please retain my R-b if you don't change any of the tools parts. Ian.

[xen-unstable-smoke test] 165479: tolerable all pass - PUSHED

2021-10-12 Thread osstest service owner
flight 165479 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/165479/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 15 migrate-support-checkfail never pass test-arm64-arm64-xl-xsm 1

Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo

2021-10-12 Thread Oleksandr
On 12.10.21 19:18, Ian Jackson wrote: Hi Ian Oleksandr writes ("Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo"): Oleksandr Tyshchenko writes ("[PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo"): You say "to

Re: [PATCH v5 08/11] xen/arm: Enable the existing x86 virtual PCI support for ARM.

2021-10-12 Thread Bertrand Marquis
Hi Julien, > On 12 Oct 2021, at 17:20, Julien Grall wrote: > > > > On 12/10/2021 17:12, Bertrand Marquis wrote: >> Hi Julien, >>> On 12 Oct 2021, at 16:04, Julien Grall wrote: >>> >>> On 11/10/2021 13:41, Bertrand Marquis wrote: Hi Jan, >>> >>> Hi Bertrand, >>> As Rahul is on lea

Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions for DomU

2021-10-12 Thread Oleksandr
On 12.10.21 19:05, Julien Grall wrote: Hi Oleksandr, Hi Julien On 11/10/2021 18:48, Oleksandr Tyshchenko wrote: ---   tools/libs/light/libxl_arm.c  | 76 ---   xen/include/public/arch-arm.h |  2 ++   2 files changed, 73 insertions(+), 5 deletions

Re: [PATCH linux 1/2] xen: delay xen_hvm_init_time_ops() if kdump is boot on vcpu>=32

2021-10-12 Thread Boris Ostrovsky
On 10/12/21 3:24 AM, Dongli Zhang wrote: The sched_clock() can be used very early since upstream commit 857baa87b642 ("sched/clock: Enable sched clock early"). In addition, with upstream commit 38669ba205d1 ("x86/xen/time: Output xen sched_clock time from 0"), kdump kernel in Xen HVM guest may

Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.

2021-10-12 Thread Ian Jackson
Bertrand Marquis writes ("Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM."): > I can add something in the commit message about the fact that we improve > performance and prevent to do a call that is and will not be supported in Xen. Thanks but I'm afraid I don'

Re: [PATCH v5 08/11] xen/arm: Enable the existing x86 virtual PCI support for ARM.

2021-10-12 Thread Julien Grall
On 12/10/2021 17:12, Bertrand Marquis wrote: Hi Julien, On 12 Oct 2021, at 16:04, Julien Grall wrote: On 11/10/2021 13:41, Bertrand Marquis wrote: Hi Jan, Hi Bertrand, As Rahul is on leave, I will answer you and make the changes needed. On 7 Oct 2021, at 14:43, Jan Beulich wrote: In

Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo

2021-10-12 Thread Ian Jackson
Oleksandr writes ("Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo"): > > Oleksandr Tyshchenko writes ("[PATCH V6 1/2] xen/arm: Introduce gpaddr_bits > > field to struct xen_domctl_getdomaininfo"): > > You say "to the toolstack", but you are exposing thi

Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.

2021-10-12 Thread Bertrand Marquis
Hi Ian, > On 12 Oct 2021, at 15:53, Ian Jackson wrote: > > Bertrand Marquis writes ("Re: [PATCH v5 01/11] xen/arm: > xc_domain_ioport_permission(..) not supported on ARM."): >> So there is no comment remaining on this patch ? > > I have been following this thread. I think I have a notion of w

Re: [PATCH xen 2/2] xen: update system time immediately when VCPUOP_register_vcpu_info

2021-10-12 Thread Dongli Zhang
Hi Jan, On 10/12/21 8:49 AM, Jan Beulich wrote: > On 12.10.2021 17:43, Dongli Zhang wrote: >> Hi Jan, >> >> On 10/12/21 1:40 AM, Jan Beulich wrote: >>> On 12.10.2021 09:24, Dongli Zhang wrote: The guest may access the pv vcpu_time_info immediately after VCPUOP_register_vcpu_info. This is

Re: [PATCH v5 08/11] xen/arm: Enable the existing x86 virtual PCI support for ARM.

2021-10-12 Thread Bertrand Marquis
Hi Julien, > On 12 Oct 2021, at 16:04, Julien Grall wrote: > > On 11/10/2021 13:41, Bertrand Marquis wrote: >> Hi Jan, > > Hi Bertrand, > >> As Rahul is on leave, I will answer you and make the changes needed. >>> On 7 Oct 2021, at 14:43, Jan Beulich wrote: >>> Independent of this - is bare m

Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions for DomU

2021-10-12 Thread Julien Grall
Hi Oleksandr, On 11/10/2021 18:48, Oleksandr Tyshchenko wrote: --- tools/libs/light/libxl_arm.c | 76 --- xen/include/public/arch-arm.h | 2 ++ 2 files changed, 73 insertions(+), 5 deletions(-) diff --git a/tools/libs/light/libxl_arm.c b/tools/libs/

Re: [PATCH 0/2] Fix the Xen HVM kdump/kexec boot panic issue

2021-10-12 Thread Dongli Zhang
Hi Juergen, On 10/12/21 1:47 AM, Juergen Gross wrote: > On 12.10.21 09:24, Dongli Zhang wrote: >> When the kdump/kexec is enabled at HVM VM side, to panic kernel will trap >> to xen side with reason=soft_reset. As a result, the xen will reboot the VM >> with the kdump kernel. >> >> Unfortunately,

Re: [PATCH xen 2/2] xen: update system time immediately when VCPUOP_register_vcpu_info

2021-10-12 Thread Jan Beulich
On 12.10.2021 17:43, Dongli Zhang wrote: > Hi Jan, > > On 10/12/21 1:40 AM, Jan Beulich wrote: >> On 12.10.2021 09:24, Dongli Zhang wrote: >>> The guest may access the pv vcpu_time_info immediately after >>> VCPUOP_register_vcpu_info. This is to borrow the idea of >>> VCPUOP_register_vcpu_time_mem

Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo

2021-10-12 Thread Oleksandr
On 12.10.21 18:15, Ian Jackson wrote: Hi Ian Oleksandr Tyshchenko writes ("[PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo"): From: Oleksandr Tyshchenko We need to pass info about maximum supported guest physical address space size to the toolstack o

Re: [PATCH xen 2/2] xen: update system time immediately when VCPUOP_register_vcpu_info

2021-10-12 Thread Dongli Zhang
Hi Jan, On 10/12/21 1:40 AM, Jan Beulich wrote: > On 12.10.2021 09:24, Dongli Zhang wrote: >> The guest may access the pv vcpu_time_info immediately after >> VCPUOP_register_vcpu_info. This is to borrow the idea of >> VCPUOP_register_vcpu_time_memory_area, where the >> force_update_vcpu_system_tim

[xen-unstable test] 165471: tolerable FAIL - PUSHED

2021-10-12 Thread osstest service owner
flight 165471 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/165471/ Failures :-/ but no regressions. Regressions which are regarded as allowable (not blocking): test-amd64-amd64-xl-rtds 20 guest-localmigrate/x10 fail REGR. vs. 165467 test-armhf-armhf-xl-rtds

Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions for DomU

2021-10-12 Thread Oleksandr
On 12.10.21 18:32, Ian Jackson wrote: Hi Ian Oleksandr writes ("Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions for DomU"): On 11.10.21 20:48, Oleksandr Tyshchenko wrote: Hi Ian, Stefano +LOG(DEBUG, "Extended region %u: %#"PRIx64"->%#"PRIx64"\n", If this appears to

Re: [PATCH v5 3/3] xen: Expose the PMU to the guests

2021-10-12 Thread Julien Grall
Hi Michal, On 12/10/2021 09:13, Michal Orzel wrote: diff --git a/SUPPORT.md b/SUPPORT.md index 317392d8f3..2f1f60c49a 100644 --- a/SUPPORT.md +++ b/SUPPORT.md @@ -256,6 +256,15 @@ from dom0. Status, x86: Tech Preview +### vPMU support There is already a section vPMU that is meant t

Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions for DomU

2021-10-12 Thread Ian Jackson
Oleksandr writes ("Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions for DomU"): > On 11.10.21 20:48, Oleksandr Tyshchenko wrote: > Hi Ian, Stefano > > +LOG(DEBUG, "Extended region %u: %#"PRIx64"->%#"PRIx64"\n", > > If this appears to be the last version, may I please ask to

Re: [PATCH v4 3/3] xen: Expose the PMU to the guests

2021-10-12 Thread Julien Grall
Hi Bertrand, On 11/10/2021 15:11, Bertrand Marquis wrote: On 11/10/2021 10:00, Michal Orzel wrote: Add parameter vpmu to xl domain configuration syntax to enable the access to PMU registers by disabling the PMU traps(currently only for ARM). The current status is that the PMU registers are not

Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions for DomU

2021-10-12 Thread Oleksandr
On 11.10.21 20:48, Oleksandr Tyshchenko wrote: Hi Ian, Stefano From: Oleksandr Tyshchenko The extended region (safe range) is a region of guest physical address space which is unused and could be safely used to create grant/foreign mappings instead of wasting real RAM pages from the domain

Re: ARM series with tools patches for 4.16

2021-10-12 Thread Ian Jackson
Stefano Stabellini writes ("ARM series with tools patches for 4.16"): > I am writing this email as a summary of the outstanding ARM series > targeting 4.16 that we are aiming to complete by Oct 15. Thanks. This is really helpful. > There has been a lot of traffic on the mailing list and in a few

Re: [REGRESSION][BISECTED] 5.15-rc1: Broken AHCI on NVIDIA ION (MCP79)

2021-10-12 Thread Josef Johansson
On 10/12/21 15:33, Josef Johansson wrote: > On 10/12/21 15:07, Jason Andryuk wrote: >> On Tue, Oct 12, 2021 at 2:09 AM Josef Johansson wrote: >>> On 10/11/21 21:34, Josef Johansson wrote: On 10/11/21 20:47, Josef Johansson wrote: > More can be read over at freedesktop: > https://gitla

[linux-linus test] 165470: tolerable FAIL - PUSHED

2021-10-12 Thread osstest service owner
flight 165470 linux-linus real [real] flight 165478 linux-linus real-retest [real] http://logs.test-lab.xenproject.org/osstest/logs/165470/ http://logs.test-lab.xenproject.org/osstest/logs/165478/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking): test-armhf-

Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo

2021-10-12 Thread Ian Jackson
Oleksandr Tyshchenko writes ("[PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo"): > From: Oleksandr Tyshchenko > > We need to pass info about maximum supported guest physical > address space size to the toolstack on Arm in order to properly > calculate the b

Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions for DomU

2021-10-12 Thread Ian Jackson
Stefano Stabellini writes ("Re: [PATCH V6 2/2] libxl/arm: Add handling of extended regions for DomU"): > On Mon, 11 Oct 2021, Oleksandr Tyshchenko wrote: > > From: Oleksandr Tyshchenko > > > > The extended region (safe range) is a region of guest physical > > address space which is unused and co

Re: [PATCH] libxl: CODING_STYLE: Explicitly deprecate #ifdef

2021-10-12 Thread Juergen Gross
On 12.10.21 16:52, Ian Jackson wrote: We don't use ifdefs in the main code. Actually document this. Signed-off-by: Ian Jackson Reviewed-by: Juergen Gross Juergen OpenPGP_0xB0DE9DD628BF132F.asc Description: OpenPGP public key OpenPGP_signature Description: OpenPGP digital signature

Re: [PATCH] libxl: CODING_STYLE: Explicitly deprecate #ifdef

2021-10-12 Thread Anthony PERARD
On Tue, Oct 12, 2021 at 03:52:26PM +0100, Ian Jackson wrote: > We don't use ifdefs in the main code. Actually document this. > > Signed-off-by: Ian Jackson > --- > tools/libs/light/CODING_STYLE | 10 ++ > 1 file changed, 10 insertions(+) > > diff --git a/tools/libs/light/CODING_STYLE b

Re: [PATCH v5 08/11] xen/arm: Enable the existing x86 virtual PCI support for ARM.

2021-10-12 Thread Julien Grall
On 11/10/2021 13:41, Bertrand Marquis wrote: Hi Jan, Hi Bertrand, As Rahul is on leave, I will answer you and make the changes needed. On 7 Oct 2021, at 14:43, Jan Beulich wrote: Independent of this - is bare metal Arm enforcing this same alignment restriction (unconditionally)? Iirc on x8

Re: [PATCH v5 10/11] arm/libxl: Emulated PCI device tree node in libxl

2021-10-12 Thread Ian Jackson
Rahul Singh writes ("[PATCH v5 10/11] arm/libxl: Emulated PCI device tree node in libxl"): > libxl will create an emulated PCI device tree node in the device tree to > enable the guest OS to discover the virtual PCI during guest boot. > Emulated PCI device tree node will only be created when there

Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.

2021-10-12 Thread Ian Jackson
Bertrand Marquis writes ("Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM."): > So there is no comment remaining on this patch ? I have been following this thread. I think I have a notion of why this is needed but I'm not 100% clear on it. The commit message s

[PATCH] libxl: CODING_STYLE: Explicitly deprecate #ifdef

2021-10-12 Thread Ian Jackson
We don't use ifdefs in the main code. Actually document this. Signed-off-by: Ian Jackson --- tools/libs/light/CODING_STYLE | 10 ++ 1 file changed, 10 insertions(+) diff --git a/tools/libs/light/CODING_STYLE b/tools/libs/light/CODING_STYLE index 3d572f6925..c2342c9d3b 100644 --- a/tool

[ovmf test] 165474: all pass - PUSHED

2021-10-12 Thread osstest service owner
flight 165474 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/165474/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf ba4ae92234b1985a89b3abed221d825b8d9ef9e2 baseline version: ovmf 19ee56c4b33faa3307889

Re: [PATCH v5 08/11] xen/arm: Enable the existing x86 virtual PCI support for ARM.

2021-10-12 Thread Bertrand Marquis
Hi Julien, > On 12 Oct 2021, at 15:32, Julien Grall wrote: > > Hi Rahul, > > On 06/10/2021 18:40, Rahul Singh wrote: >> diff --git a/xen/include/asm-arm/domain.h b/xen/include/asm-arm/domain.h >> index c9277b5c6d..91d614b37e 100644 >> --- a/xen/include/asm-arm/domain.h >> +++ b/xen/include/asm-

Re: [PATCH v5 08/11] xen/arm: Enable the existing x86 virtual PCI support for ARM.

2021-10-12 Thread Julien Grall
Hi Rahul, On 06/10/2021 18:40, Rahul Singh wrote: diff --git a/xen/include/asm-arm/domain.h b/xen/include/asm-arm/domain.h index c9277b5c6d..91d614b37e 100644 --- a/xen/include/asm-arm/domain.h +++ b/xen/include/asm-arm/domain.h @@ -2,6 +2,7 @@ #define __ASM_DOMAIN_H__ #include +#includ

Re: [PATCH v5 3/3] xen: Expose the PMU to the guests

2021-10-12 Thread Anthony PERARD
On Tue, Oct 12, 2021 at 10:13:23AM +0200, Michal Orzel wrote: > diff --git a/tools/libs/light/libxl_create.c b/tools/libs/light/libxl_create.c > index e356b2106d..2a0234ec16 100644 > --- a/tools/libs/light/libxl_create.c > +++ b/tools/libs/light/libxl_create.c > @@ -622,6 +623,9 @@ int libxl__domai

Re: [PATCH] x86/xen: Remove redundant irq_enter/exit() invocations

2021-10-12 Thread Juergen Gross
On 12.10.21 15:33, Thomas Gleixner wrote: All these handlers are regular device interrupt handlers, so they already went through the proper entry code which handles this correctly. Signed-off-by: Thomas Gleixner Cc: Juergen Gross Cc: Stefano Stabellini Cc: Boris Ostrovsky Cc: x...@kernel.org

Re: [RFC 0/1] tag IOMMU-protected devices in Dom0 fdt

2021-10-12 Thread Roman Skakun
Understood. Thank you! вт, 12 окт. 2021 г. в 17:20, Julien Grall : > > > > On 12/10/2021 15:11, Roman Skakun wrote: > > Hi, > > Hi Roman, > > > Would you be so kind to look at my patches, it would help me to > > know exactly whether I'm moving correctly and whether I've chosen > > the right path?

Re: [RFC 0/1] tag IOMMU-protected devices in Dom0 fdt

2021-10-12 Thread Julien Grall
On 12/10/2021 15:11, Roman Skakun wrote: Hi, Hi Roman, Would you be so kind to look at my patches, it would help me to know exactly whether I'm moving correctly and whether I've chosen the right path? This is in my queue to review. At the moment, I am prioritizing work targeting 4.16. U

Re: [RFC 0/1] tag IOMMU-protected devices in Dom0 fdt

2021-10-12 Thread Roman Skakun
Hi, Would you be so kind to look at my patches, it would help me to know exactly whether I'm moving correctly and whether I've chosen the right path? Thanks! Best regards, Roman пн, 4 окт. 2021 г. в 12:54, Roman Skakun : > > From: Roman Skakun > > At the moment, Dom0 can't distinguish which dev

[PATCH v6 2/2] tools/xenstore: set open file descriptor limit for xenstored

2021-10-12 Thread Ian Jackson
Juergen Gross writes ("[PATCH v6 2/2] tools/xenstore: set open file descriptor limit for xenstored"): > Add a configuration item for the maximum number of open file > descriptors xenstored should be allowed to have. > > The default should be "unlimited" in order not to restrict xenstored > in the

[PATCH v6 2/2] tools/xenstore: set open file descriptor limit for xenstored

2021-10-12 Thread Juergen Gross
Add a configuration item for the maximum number of open file descriptors xenstored should be allowed to have. The default should be "unlimited" in order not to restrict xenstored in the number of domains it can support, but unfortunately the kernel is normally limiting the maximum value via /proc/

[PATCH v6 1/2] tools/xenstore: set oom score for xenstore daemon on Linux

2021-10-12 Thread Juergen Gross
Xenstored is absolutely mandatory for a Xen host and it can't be restarted, so being killed by OOM-killer in case of memory shortage is to be avoided. Set /proc/$pid/oom_score_adj (if available) per default to -500 (this translates to 50% of dom0 memory size) in order to allow xenstored to use lar

[PATCH v6 0/2] tools/xenstore: set resource limits of xenstored

2021-10-12 Thread Juergen Gross
Set some limits for xenstored in order to avoid it being killed by OOM killer, or to run out of file descriptors. Changes in V6: - start daemon directly via prlimit Changes in V5: - respect /proc/sys/fs/nr_open (Ian Jackson) Changes in V4: - add comments - switch to configure open file descripto

[xen-unstable-smoke test] 165475: tolerable all pass - PUSHED

2021-10-12 Thread osstest service owner
flight 165475 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/165475/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-libvirt 15 migrate-support-checkfail never pass test-arm64-arm64-xl-xsm 1

Re: [REGRESSION][BISECTED] 5.15-rc1: Broken AHCI on NVIDIA ION (MCP79)

2021-10-12 Thread Josef Johansson
On 10/12/21 15:07, Jason Andryuk wrote: > On Tue, Oct 12, 2021 at 2:09 AM Josef Johansson wrote: >> On 10/11/21 21:34, Josef Johansson wrote: >>> On 10/11/21 20:47, Josef Johansson wrote: More can be read over at freedesktop: https://gitlab.freedesktop.org/drm/amd/-/issues/1715 > Hi, Jos

[PATCH] x86/xen: Remove redundant irq_enter/exit() invocations

2021-10-12 Thread Thomas Gleixner
All these handlers are regular device interrupt handlers, so they already went through the proper entry code which handles this correctly. Signed-off-by: Thomas Gleixner Cc: Juergen Gross Cc: Stefano Stabellini Cc: Boris Ostrovsky Cc: x...@kernel.org Cc: xen-devel@lists.xenproject.org --- arc

Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo

2021-10-12 Thread Jan Beulich
On 12.10.2021 15:18, Oleksandr wrote: > On 12.10.21 14:49, Jan Beulich wrote: >> On 12.10.2021 13:28, Oleksandr wrote: >>> On 12.10.21 12:24, Jan Beulich wrote: On 11.10.2021 19:48, Oleksandr Tyshchenko wrote: > @@ -150,6 +150,7 @@ struct xen_domctl_getdomaininfo { >uint32_t ss

Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo

2021-10-12 Thread Oleksandr
On 12.10.21 14:49, Jan Beulich wrote: Hi Jan On 12.10.2021 13:28, Oleksandr wrote: On 12.10.21 12:24, Jan Beulich wrote: On 11.10.2021 19:48, Oleksandr Tyshchenko wrote: @@ -150,6 +150,7 @@ struct xen_domctl_getdomaininfo { uint32_t ssidref; xen_domain_handle_t handle;

Re: [REGRESSION][BISECTED] 5.15-rc1: Broken AHCI on NVIDIA ION (MCP79)

2021-10-12 Thread Jason Andryuk
On Tue, Oct 12, 2021 at 2:09 AM Josef Johansson wrote: > > On 10/11/21 21:34, Josef Johansson wrote: > > On 10/11/21 20:47, Josef Johansson wrote: > >> More can be read over at freedesktop: > >> https://gitlab.freedesktop.org/drm/amd/-/issues/1715 Hi, Josef, If you compare commit fcacdfbef5a1633

Re: [PATCH v2 11/12] xen/arch: Drop asm-*/trace.h

2021-10-12 Thread Julien Grall
Hi Andrew, On 20/09/2021 18:25, Andrew Cooper wrote: The feature is x86 and pv-dom0 specific, and each architecture's main trace.h files are empty. Don't force all new architectures to create an empty file. While moving the declaration of tb_init_done, change from int to bool. Signed-off-by:

[libvirt test] 165473: regressions - FAIL

2021-10-12 Thread osstest service owner
flight 165473 libvirt real [real] http://logs.test-lab.xenproject.org/osstest/logs/165473/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: build-armhf-libvirt 6 libvirt-buildfail REGR. vs. 151777 build-amd64-libvirt

Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo

2021-10-12 Thread Jan Beulich
On 12.10.2021 13:28, Oleksandr wrote: > On 12.10.21 12:24, Jan Beulich wrote: >> On 11.10.2021 19:48, Oleksandr Tyshchenko wrote: >>> @@ -150,6 +150,7 @@ struct xen_domctl_getdomaininfo { >>> uint32_t ssidref; >>> xen_domain_handle_t handle; >>> uint32_t cpupool; >>> +uint8_t

Re: [PATCH V6 1/2] xen/arm: Introduce gpaddr_bits field to struct xen_domctl_getdomaininfo

2021-10-12 Thread Oleksandr
On 12.10.21 12:24, Jan Beulich wrote: Hi Jan Thank you for thorough review. On 11.10.2021 19:48, Oleksandr Tyshchenko wrote: --- a/xen/include/public/domctl.h +++ b/xen/include/public/domctl.h @@ -38,7 +38,7 @@ #include "hvm/save.h" #include "memory.h" -#define XEN_DOMCTL_INTERFACE_V

Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.

2021-10-12 Thread Jan Beulich
On 12.10.2021 12:41, Bertrand Marquis wrote: > So there is no comment remaining on this patch ? No, but the comments weren't on the patch anyway, but on a question raised in the course of the discussion. > Would it possible to get an ack on it ? No, as I can't ack this patch: I'm not a maintaine

Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.

2021-10-12 Thread Bertrand Marquis
Hi Jan, > On 12 Oct 2021, at 11:20, Jan Beulich wrote: > > On 12.10.2021 12:06, Oleksandr Andrushchenko wrote: >> On 12.10.21 13:01, Jan Beulich wrote: >>> On 12.10.2021 11:38, Oleksandr Andrushchenko wrote: On 12.10.21 12:32, Jan Beulich wrote: > The minimal thing I'd suggest (or maybe

Re: [PATCH v5 07/11] xen/domctl: Introduce XEN_DOMCTL_CDF_vpci flag

2021-10-12 Thread Michal Orzel
Hi Roger, On 11.10.2021 11:27, Roger Pau Monné wrote: > On Wed, Oct 06, 2021 at 06:40:33PM +0100, Rahul Singh wrote: >> Introduce XEN_DOMCTL_CDF_vpci flag to enable VPCI support in XEN. >> Reject the use of this new flag for x86 as VPCI is not supported for >> DOMU guests for x86. > > I don't lik

Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.

2021-10-12 Thread Jan Beulich
On 12.10.2021 12:06, Oleksandr Andrushchenko wrote: > On 12.10.21 13:01, Jan Beulich wrote: >> On 12.10.2021 11:38, Oleksandr Andrushchenko wrote: >>> On 12.10.21 12:32, Jan Beulich wrote: The minimal thing I'd suggest (or maybe you're doing this already) would be to expose such BARs to t

[linux-5.4 test] 165469: regressions - FAIL

2021-10-12 Thread osstest service owner
flight 165469 linux-5.4 real [real] http://logs.test-lab.xenproject.org/osstest/logs/165469/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-armhf-armhf-xl-credit1 14 guest-start fail REGR. vs. 165206 Tests which are faili

Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.

2021-10-12 Thread Oleksandr Andrushchenko
On 12.10.21 13:01, Jan Beulich wrote: > On 12.10.2021 11:38, Oleksandr Andrushchenko wrote: >> On 12.10.21 12:32, Jan Beulich wrote: >>> On 12.10.2021 10:41, Bertrand Marquis wrote: > On 12 Oct 2021, at 09:29, Jan Beulich wrote: > On 11.10.2021 19:11, Bertrand Marquis wrote: >>> On 1

Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.

2021-10-12 Thread Jan Beulich
On 12.10.2021 11:40, Bertrand Marquis wrote: > Hi Jan, > >> On 12 Oct 2021, at 10:32, Jan Beulich wrote: >> >> On 12.10.2021 10:41, Bertrand Marquis wrote: >>> Hi Jan, >>> On 12 Oct 2021, at 09:29, Jan Beulich wrote: On 11.10.2021 19:11, Bertrand Marquis wrote: >> On 11 Oct 20

Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.

2021-10-12 Thread Jan Beulich
On 12.10.2021 11:38, Oleksandr Andrushchenko wrote: > On 12.10.21 12:32, Jan Beulich wrote: >> On 12.10.2021 10:41, Bertrand Marquis wrote: On 12 Oct 2021, at 09:29, Jan Beulich wrote: On 11.10.2021 19:11, Bertrand Marquis wrote: >> On 11 Oct 2021, at 17:32, Roger Pau Monné wrote: >

Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.

2021-10-12 Thread Bertrand Marquis
Hi Jan, > On 12 Oct 2021, at 10:32, Jan Beulich wrote: > > On 12.10.2021 10:41, Bertrand Marquis wrote: >> Hi Jan, >> >>> On 12 Oct 2021, at 09:29, Jan Beulich wrote: >>> >>> On 11.10.2021 19:11, Bertrand Marquis wrote: > On 11 Oct 2021, at 17:32, Roger Pau Monné wrote: > On Mon, Oct

Re: [PATCH v5 01/11] xen/arm: xc_domain_ioport_permission(..) not supported on ARM.

2021-10-12 Thread Oleksandr Andrushchenko
On 12.10.21 12:32, Jan Beulich wrote: > On 12.10.2021 10:41, Bertrand Marquis wrote: >> Hi Jan, >> >>> On 12 Oct 2021, at 09:29, Jan Beulich wrote: >>> >>> On 11.10.2021 19:11, Bertrand Marquis wrote: > On 11 Oct 2021, at 17:32, Roger Pau Monné wrote: > On Mon, Oct 11, 2021 at 02:16:19P

Re: [PATCH v6 1/2] arm/efi: Use dom0less configuration when using EFI boot

2021-10-12 Thread Jan Beulich
On 12.10.2021 11:05, Luca Fancellu wrote: >> On 12 Oct 2021, at 09:45, Jan Beulich wrote: >> On 11.10.2021 20:15, Luca Fancellu wrote: >>> --- a/xen/common/efi/boot.c >>> +++ b/xen/common/efi/boot.c >>> @@ -166,6 +166,13 @@ static void __init PrintErr(const CHAR16 *s) >>> StdErr->OutputString(

Re: [PATCH v6 2/2] arm/efi: load dom0 modules from DT using UEFI

2021-10-12 Thread Julien Grall
Hi Stefano, On 12/10/2021 02:31, Stefano Stabellini wrote: On Mon, 11 Oct 2021, Julien Grall wrote: Hi Stefano, On 11/10/2021 22:24, Stefano Stabellini wrote: diff --git a/xen/arch/arm/efi/efi-boot.h b/xen/arch/arm/efi/efi-boot.h index 840728d6c0..076b827bdd 100644 --- a/xen/arch/arm/efi/efi-

  1   2   >