Re: [PATCH v2] gnttab: defer allocation of status frame tracking array

2020-12-24 Thread Julien Grall
Hi Jan, On 23/12/2020 15:13, Jan Beulich wrote: This array can be large when many grant frames are permitted; avoid allocating it when it's not going to be used anyway, by doing this only in gnttab_populate_status_frames(). Signed-off-by: Jan Beulich --- v2: Defer allocation to when a domain a

[linux-linus test] 157864: regressions - FAIL

2020-12-24 Thread osstest service owner
flight 157864 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/157864/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-xsm7 xen-install fail REGR. vs. 152332 test-amd64-i386-xl-

[qemu-mainline test] 157862: regressions - FAIL

2020-12-24 Thread osstest service owner
flight 157862 qemu-mainline real [real] flight 157870 qemu-mainline real-retest [real] http://logs.test-lab.xenproject.org/osstest/logs/157862/ http://logs.test-lab.xenproject.org/osstest/logs/157870/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be

[PATCH 3/5] x86/xen: Add a no_vector_callback option to test PCI INTX delivery

2020-12-24 Thread David Woodhouse
From: David Woodhouse It's useful to be able to test non-vector event channel delivery, to make sure Linux will work properly on older Xen which doesn't have it. It's also useful for those working on Xen and Xen-compatible hypervisors, because there are guest kernels still in active use which us

[PATCH 5/5] x86/xen: Don't register PV spinlock IPI when it isn't going to be used

2020-12-24 Thread David Woodhouse
From: David Woodhouse When xen_have_vector_callback is false, we still register the PV spinlock kicker IPI on the secondary CPUs. Stop doing that. Signed-off-by: David Woodhouse --- arch/x86/xen/spinlock.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/x86/xen/s

[PATCH 2/5] xen: Set platform PCI device INTX affinity to CPU0

2020-12-24 Thread David Woodhouse
From: David Woodhouse With INTX or GSI delivery, Xen uses the event channel structures of CPU0. If the interrupt gets handled by Linux on a different CPU, then no events are seen as pending. Rather than introducing locking to allow other CPUs to process CPU0's events, just ensure that the PCI in

[PATCH 4/5] x86/xen: Don't register Xen IPIs when they aren't going to be used

2020-12-24 Thread David Woodhouse
From: David Woodhouse In the case where xen_have_vector_callback is false, we still register the IPI vectors in xen_smp_intr_init() for the secondary CPUs even though they aren't going to be used. Stop doing that. Signed-off-by: David Woodhouse --- arch/x86/xen/enlighten_hvm.c | 4 ++-- 1 file

[PATCH 1/5] xen: Fix event channel callback via INTX/GSI

2020-12-24 Thread David Woodhouse
From: David Woodhouse For a while, event channel notification via the PCI platform device has been broken, because we attempt to communicate with xenstore before we even have notifications working, with the xs_reset_watches() call in xs_init(). We tend to get away with this on Xen versions below

[PATCH 0/5] Xen INTX/GSI event channel delivery fixes

2020-12-24 Thread David Woodhouse
Fix various breakages with INTX/GSI event channel interrupt delivery, and add a command line option to test it so that it hopefully doesn't get so broken again. Karim attempted to rip this out in 2017 but put it back because it's still necessary with older versions of Xen. This fixes it properly,

[xen-unstable test] 157865: tolerable FAIL

2020-12-24 Thread osstest service owner
flight 157865 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/157865/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-xl-rtds 18 guest-start/debian.repeatfail like 157837 test-amd64-amd64-xl-qemuu-ws16-amd64

[PATCH] xen/iommu: smmu: Use 1UL << 31 rather than 1 << 31

2020-12-24 Thread Julien Grall
From: Julien Grall Replace all the use of 1 << 31 with 1UL << 31 to prevent undefined behavior in the SMMU driver. Signed-off-by: Julien Grall --- xen/drivers/passthrough/arm/smmu.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/xen/drivers/passthrough/arm/smmu.c

[linux-linus test] 157871: regressions - FAIL

2020-12-24 Thread osstest service owner
flight 157871 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/157871/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-xsm7 xen-install fail REGR. vs. 152332 test-amd64-i386-xl-

[PATCH] xen/iommu: smmu: Rework how the SMMU version is detected

2020-12-24 Thread Julien Grall
From: Julien Grall Clang 11 will throw the following error: smmu.c:2284:18: error: cast to smaller integer type 'enum arm_smmu_arch_version' from 'const void *' [-Werror,-Wvoid-pointer-to-enum-cast] smmu->version = (enum arm_smmu_arch_version)of_id->data; ^~~

[PATCH] xen/serial: scif: Rework how the parameters are found

2020-12-24 Thread Julien Grall
From: Julien Grall clang 11 will throw the following error while build Xen: scif-uart.c:333:33: error: cast to smaller integer type 'enum port_types' from 'const void *' [-Werror,-Wvoid-pointer-to-enum-cast] uart->params = &port_params[(enum port_types)match->data];

Re: [PATCH] xen/iommu: smmu: Rework how the SMMU version is detected

2020-12-24 Thread Andrew Cooper
On 24/12/2020 16:49, Julien Grall wrote: > From: Julien Grall > > Clang 11 will throw the following error: > > smmu.c:2284:18: error: cast to smaller integer type 'enum > arm_smmu_arch_version' from 'const void *' > [-Werror,-Wvoid-pointer-to-enum-cast] > smmu->version = (enum arm_smmu_a

Re: [PATCH] xen/iommu: smmu: Rework how the SMMU version is detected

2020-12-24 Thread Julien Grall
On 24/12/2020 17:00, Andrew Cooper wrote: On 24/12/2020 16:49, Julien Grall wrote: From: Julien Grall Clang 11 will throw the following error: smmu.c:2284:18: error: cast to smaller integer type 'enum arm_smmu_arch_version' from 'const void *' [-Werror,-Wvoid-pointer-to-enum-cast]

Re: [PATCH] xen/iommu: smmu: Rework how the SMMU version is detected

2020-12-24 Thread Andrew Cooper
On 24/12/2020 17:10, Julien Grall wrote: > > > On 24/12/2020 17:00, Andrew Cooper wrote: >> On 24/12/2020 16:49, Julien Grall wrote: >>> From: Julien Grall >>> >>> Clang 11 will throw the following error: >>> >>> smmu.c:2284:18: error: cast to smaller integer type 'enum >>> arm_smmu_arch_version'

[ovmf test] 157875: all pass - PUSHED

2020-12-24 Thread osstest service owner
flight 157875 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/157875/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 140674a4601f804302e79d08cb06f91c882ddf28 baseline version: ovmf e2747dbb5a44f4a463ecc

Re: [PATCH] xen/iommu: smmu: Rework how the SMMU version is detected

2020-12-24 Thread Julien Grall
On Thu, 24 Dec 2020 at 17:39, Andrew Cooper wrote: > > On 24/12/2020 17:10, Julien Grall wrote: > > > > > > On 24/12/2020 17:00, Andrew Cooper wrote: > >> On 24/12/2020 16:49, Julien Grall wrote: > >>> From: Julien Grall > >>> > >>> Clang 11 will throw the following error: > >>> > >>> smmu.c:2284

Re: [XEN PATCH v2] docs: set date to SOURCE_DATE_EPOCH if available

2020-12-24 Thread Hans van Kranenburg
Hi, On 12/23/20 5:56 PM, Maximilian Engelhardt wrote: > check if a GNU date that supports the '-u -d @...' options and syntax or > a BSD date are available. If so, use the appropriate options for the > date command to produce a custom date if SOURCE_DATE_EPOCH is defined. > If SOURCE_DATE_EPOCH is

Re: [PATCH] xen/iommu: smmu: Use 1UL << 31 rather than 1 << 31

2020-12-24 Thread Hans van Kranenburg
On 12/24/20 4:24 PM, Julien Grall wrote: > From: Julien Grall > > Replace all the use of 1 << 31 with 1UL << 31 to prevent undefined > behavior in the SMMU driver. You're replacing it by 1U, not 1UL, in the patch below. Hans > Signed-off-by: Julien Grall > --- > xen/drivers/passthrough/arm/s

Re: [PATCH] xen/serial: scif: Rework how the parameters are found

2020-12-24 Thread Oleksandr
On 24.12.20 18:50, Julien Grall wrote: Hi Julien From: Julien Grall clang 11 will throw the following error while build Xen: scif-uart.c:333:33: error: cast to smaller integer type 'enum port_types' from 'const void *' [-Werror,-Wvoid-pointer-to-enum-cast] uart->params = &port_params

[qemu-mainline test] 157872: regressions - FAIL

2020-12-24 Thread osstest service owner
flight 157872 qemu-mainline real [real] flight 157879 qemu-mainline real-retest [real] http://logs.test-lab.xenproject.org/osstest/logs/157872/ http://logs.test-lab.xenproject.org/osstest/logs/157879/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be

[linux-linus test] 157877: regressions - FAIL

2020-12-24 Thread osstest service owner
flight 157877 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/157877/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-xl-xsm7 xen-install fail REGR. vs. 152332 test-amd64-i386-xl-