Re: [PATCH] xen/privcmd: Use memdup_array_user() in alloc_ioreq()

2024-02-12 Thread Juergen Gross
On 28.01.24 18:09, Markus Elfring wrote: From: Markus Elfring Date: Sun, 28 Jan 2024 17:50:43 +0100 * The function “memdup_array_user” was added with the commit 313ebe47d75558511aa1237b6e35c663b5c0ec6f ("string.h: add array-wrappers for (v)memdup_user()"). Thus use it accordingly.

Re: [PATCH v3 1/8] serial: fake IRQ-regs context in poll handlers

2024-02-12 Thread Jan Beulich
On 13.02.2024 04:43, Marek Marczykowski wrote: > On Mon, Feb 12, 2024 at 10:04:38AM +0100, Jan Beulich wrote: >> On 08.02.2024 23:00, Julien Grall wrote: >>> On 05/02/2024 13:27, Jan Beulich wrote: In preparation of dropping the register parameters from serial_[rt]x_interrupt() and in

Re: [PATCH v1] docs/misra/rules.rst: catch up with accepted rules

2024-02-12 Thread Jan Beulich
On 13.02.2024 00:18, Stefano Stabellini wrote: > On Mon, 12 Feb 2024, Jan Beulich wrote: >> On 10.02.2024 02:00, Stefano Stabellini wrote: >>> Update docs/misra/rules.rst to reflect the MISRA C rules accepted in the >>> last couple of months. >>> >>> Signed-off-by: Stefano Stabellini >>> --- >>>

Re: Return type of clean_and_invalidate_dcache_va_range

2024-02-12 Thread Jan Beulich
On 12.02.2024 19:38, Julien Grall wrote: > An alternative would be to introduced arch_grant_cache_flush() and move > the if/else logic there. Something like: > > diff --git a/xen/arch/arm/include/asm/page.h > b/xen/arch/arm/include/asm/page.h > index 69f817d1e68a..4a3de49762a1 100644 > ---

Re: Bug#1063270: The "64bits time_t transition" in Debian/Xen

2024-02-12 Thread Hans van Kranenburg
Hi, On 2/12/24 18:43, Andrew Cooper wrote: > On 12/02/2024 5:27 pm, zithro wrote: >> Hey all, >> >> the Debian project is focused on the "2038 time_t" switch. >> So the maintainers of the Debian Xen package must ensure that all >> imported Xen code conforms to the new Debian standards. >> >> I

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

2024-02-12 Thread osstest service owner
flight 184653 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/184653/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt 16 saverestore-support-checkfail like 184648

Re: [PATCH v3 1/8] serial: fake IRQ-regs context in poll handlers

2024-02-12 Thread Marek Marczykowski
On Mon, Feb 12, 2024 at 10:04:38AM +0100, Jan Beulich wrote: > On 08.02.2024 23:00, Julien Grall wrote: > > On 05/02/2024 13:27, Jan Beulich wrote: > >> In preparation of dropping the register parameters from > >> serial_[rt]x_interrupt() and in turn from IRQ handler functions, > >> register state

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

2024-02-12 Thread osstest service owner
flight 184652 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/184652/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-xl-qemut-win7-amd64 19 guest-stopfail like 184649 test-amd64-amd64-xl-qemuu-win7-amd64

Re: [PATCH] swiotlb-xen: remove redundant NULL check

2024-02-12 Thread Stefano Stabellini
Hi Daniil, Against which Linux branch was this patch generated? Cheers, Stefano On Sun, 11 Feb 2024, Daniil Dulov wrote: > In this case hwdev cannot be NULL, so remove redundant NULL check. > > Found by Linux Verification Center (linuxtesting.org) with SVACE. > > Fixes: b097186fd29d

Re: Serious AMD-Vi issue

2024-02-12 Thread Elliott Mitchell
On Thu, Jan 25, 2024 at 12:24:53PM -0800, Elliott Mitchell wrote: > Apparently this was first noticed with 4.14, but more recently I've been > able to reproduce the issue: > > https://bugs.debian.org/988477 > > The original observation features MD-RAID1 using a pair of Samsung > SATA-attached

Re: [PATCH v1] docs/misra/rules.rst: catch up with accepted rules

2024-02-12 Thread Stefano Stabellini
On Mon, 12 Feb 2024, Jan Beulich wrote: > On 10.02.2024 02:00, Stefano Stabellini wrote: > > Update docs/misra/rules.rst to reflect the MISRA C rules accepted in the > > last couple of months. > > > > Signed-off-by: Stefano Stabellini > > --- > > > > In the notes section I added some info about

OpenFastTrace presentation for the Xen Community

2024-02-12 Thread Stefano Stabellini
Hi all, As we continue our journey toward Xen Safety Certifiability, one of the next challenges is safety requirements and their traceability. We need to be able to link high level requirements documents to mid and low level requirements documents, to test specifications and to gitlab tests.

[xen-unstable test] 184651: regressions - trouble: broken/fail/pass

2024-02-12 Thread osstest service owner
flight 184651 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/184651/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-xtf-amd64-amd64-3 broken test-xtf-amd64-amd64-35

Re: [RFC KERNEL PATCH v4 3/3] PCI/sysfs: Add gsi sysfs for pci_dev

2024-02-12 Thread Bjorn Helgaas
On Mon, Feb 12, 2024 at 10:13:28AM +0100, Roger Pau Monné wrote: > On Fri, Feb 09, 2024 at 03:05:49PM -0600, Bjorn Helgaas wrote: > > On Thu, Feb 01, 2024 at 09:39:49AM +0100, Roger Pau Monné wrote: > > > On Wed, Jan 31, 2024 at 01:00:14PM -0600, Bjorn Helgaas wrote: > > > > On Wed, Jan 31, 2024

Re: Return type of clean_and_invalidate_dcache_va_range

2024-02-12 Thread Julien Grall
Hi Nicola, On 12/02/2024 14:56, Nicola Vetrini wrote: On 2024-02-12 09:26, Jan Beulich wrote: On 10.02.2024 11:17, Julien Grall wrote: Hi, On 09/02/2024 22:02, Stefano Stabellini wrote: On Fri, 9 Feb 2024, Nicola Vetrini wrote: Hi all, In the context of violations of MISRA C:2012 Rule

Re: The "64bits time_t transition" in Debian/Xen

2024-02-12 Thread Andrew Cooper
On 12/02/2024 5:27 pm, zithro wrote: > Hey all, > > the Debian project is focused on the "2038 time_t" switch. > So the maintainers of the Debian Xen package must ensure that all > imported Xen code conforms to the new Debian standards. > > I was asked by Andrew Cooper to post here about this,

The "64bits time_t transition" in Debian/Xen

2024-02-12 Thread zithro
Hey all, the Debian project is focused on the "2038 time_t" switch. So the maintainers of the Debian Xen package must ensure that all imported Xen code conforms to the new Debian standards. I was asked by Andrew Cooper to post here about this, I'll quote him : "So I had been idly wondering

Re: [PATCH v3] x86/xen: Add some null pointer checking to smp.c

2024-02-12 Thread Juergen Gross
On 19.01.24 10:49, Kunwu Chan wrote: kasprintf() returns a pointer to dynamically allocated memory which can be NULL upon failure. Ensure the allocation was successful by checking the pointer validity. Signed-off-by: Kunwu Chan Reported-by: kernel test robot Closes:

Re: [PATCH 2/7] IOMMU: rename and re-type ats_enabled

2024-02-12 Thread Jan Beulich
On 12.02.2024 16:38, Roger Pau Monné wrote: > On Mon, Feb 12, 2024 at 11:45:33AM +0100, Jan Beulich wrote: >> On 12.02.2024 10:39, Roger Pau Monné wrote: >>> On Thu, Feb 08, 2024 at 04:49:46PM +0100, Jan Beulich wrote: On 08.02.2024 12:49, Roger Pau Monné wrote: > On Mon, Feb 05, 2024 at

Re: [PATCH v4 09/30] xen/riscv: introduce bitops.h

2024-02-12 Thread Jan Beulich
On 05.02.2024 16:32, Oleksii Kurochko wrote: > --- /dev/null > +++ b/xen/arch/riscv/include/asm/bitops.h > @@ -0,0 +1,164 @@ > +/* SPDX-License-Identifier: GPL-2.0 */ > +/* Copyright (C) 2012 Regents of the University of California */ > + > +#ifndef _ASM_RISCV_BITOPS_H > +#define

Re: [PATCH 2/7] IOMMU: rename and re-type ats_enabled

2024-02-12 Thread Roger Pau Monné
On Mon, Feb 12, 2024 at 11:45:33AM +0100, Jan Beulich wrote: > On 12.02.2024 10:39, Roger Pau Monné wrote: > > On Thu, Feb 08, 2024 at 04:49:46PM +0100, Jan Beulich wrote: > >> On 08.02.2024 12:49, Roger Pau Monné wrote: > >>> On Mon, Feb 05, 2024 at 02:55:43PM +0100, Jan Beulich wrote: >

Re: [PATCH] xen/xenbus: document will_handle argument for xenbus_watch_path()

2024-02-12 Thread Juergen Gross
On 12.01.24 19:59, SeongJae Park wrote: Commit 2e85d32b1c86 ("xen/xenbus: Add 'will_handle' callback support in xenbus_watch_path()") added will_handle argument to xenbus_watch_path() and its wrapper, xenbus_watch_pathfmt(), but didn't document it on the kerneldoc comments of the function. This

Re: [PATCH v4 28/30] xen/rirscv: add minimal amount of stubs to build full Xen

2024-02-12 Thread Jan Beulich
On 05.02.2024 16:32, Oleksii Kurochko wrote: > --- a/xen/arch/riscv/early_printk.c > +++ b/xen/arch/riscv/early_printk.c > @@ -207,4 +207,3 @@ void printk(const char *format, ...) > } > > #endif > - Unrelated change? > --- a/xen/arch/riscv/traps.c > +++ b/xen/arch/riscv/traps.c > @@ -4,6

Re: [PATCH v4 19/30] xen/riscv: introduce event.h

2024-02-12 Thread Jan Beulich
On 05.02.2024 16:32, Oleksii Kurochko wrote: > Signed-off-by: Oleksii Kurochko Acked-by: Jan Beulich again with a nit, though: > --- /dev/null > +++ b/xen/arch/riscv/include/asm/event.h > @@ -0,0 +1,40 @@ > +/* SPDX-License-Identifier: GPL-2.0-only */ > +#ifndef __ASM_RISCV_EVENT_H__ >

Re: [PATCH v4 18/30] xen/riscv: introduce time.h

2024-02-12 Thread Jan Beulich
On 05.02.2024 16:32, Oleksii Kurochko wrote: > Signed-off-by: Oleksii Kurochko > Acked-by: Jan Beulich Nevertheless ... > --- /dev/null > +++ b/xen/arch/riscv/include/asm/time.h > @@ -0,0 +1,29 @@ > +/* SPDX-License-Identifier: GPL-2.0-only */ > +#ifndef __ASM_RISCV_TIME_H__ > +#define

Re: [PATCH v4 16/30] xen/riscv: introduce p2m.h

2024-02-12 Thread Jan Beulich
On 05.02.2024 16:32, Oleksii Kurochko wrote: > Signed-off-by: Oleksii Kurochko Acked-by: Jan Beulich with two more nits: > --- /dev/null > +++ b/xen/arch/riscv/include/asm/p2m.h > @@ -0,0 +1,102 @@ > +/* SPDX-License-Identifier: GPL-2.0-only */ > +#ifndef __ASM_RISCV_P2M_H__ > +#define

Re: [PATCH v4 11/30] xen/riscv: introduce smp.h

2024-02-12 Thread Jan Beulich
On 05.02.2024 16:32, Oleksii Kurochko wrote: > Signed-off-by: Oleksii Kurochko Acked-by: Jan Beulich

Re: [PATCH v4 06/30] xen: avoid generation of empty asm/iommu.h

2024-02-12 Thread Jan Beulich
On 05.02.2024 16:32, Oleksii Kurochko wrote: > asm/iommu.h shouldn't ... need to ... > be included when CONFIG_HAS_PASSTHROUGH > isn't enabled. > As is ifdef-ed by CONFIG_HAS_PASSTHROUGH it should > be also ifdef-ed field "struct arch_iommu arch" in struct domain_iommu > as definition of

Re: [PATCH v4 05/30] xen/riscv: introduce guest_atomics.h

2024-02-12 Thread Jan Beulich
On 05.02.2024 16:32, Oleksii Kurochko wrote: > Signed-off-by: Oleksii Kurochko Acked-by: Jan Beulich

Re: [PATCH v4 03/30] xen: add support in public/hvm/save.h for PPC and RISC-V

2024-02-12 Thread Jan Beulich
On 05.02.2024 16:32, Oleksii Kurochko wrote: > No specific header is needed to include in public/hvm/save.h for > PPC and RISC-V for now. > > Code related to PPC was changed based on the comment: > https://lore.kernel.org/xen-devel/c2f3280e-2208-496b-a0b5-fda1a2076...@raptorengineering.com/ > >

Re: [PATCH v4 02/30] xen/riscv: use some asm-generic headers

2024-02-12 Thread Jan Beulich
On 05.02.2024 16:32, Oleksii Kurochko wrote: > Some headers are the same as asm-generic verions of them > so use them instead of arch-specific headers. Just to mention it (I'll commit this as is, unless asked to do otherwise): With this description I'd expect those "some headers" to be removed by

Re: Return type of clean_and_invalidate_dcache_va_range

2024-02-12 Thread Nicola Vetrini
On 2024-02-12 09:26, Jan Beulich wrote: On 10.02.2024 11:17, Julien Grall wrote: Hi, On 09/02/2024 22:02, Stefano Stabellini wrote: On Fri, 9 Feb 2024, Nicola Vetrini wrote: Hi all, In the context of violations of MISRA C:2012 Rule 17.7: "The value returned by a function having non-void

Re: [PATCH v2 2/2] iommu/vt-d: switch to common RMRR checker

2024-02-12 Thread Jan Beulich
On 07.02.2024 16:34, Roger Pau Monne wrote: > Use the newly introduced generic unity map checker. > > Also drop the message recommending the usage of iommu_inclusive_mapping: the > ranges would end up being mapped anyway even if some of the checks above > failed, regardless of whether

Re: [PATCH v2 1/2] iommu/x86: introduce a generic IVMD/RMRR range validity helper

2024-02-12 Thread Jan Beulich
On 07.02.2024 16:34, Roger Pau Monne wrote: > IVMD and RMRR ranges are functionally equivalent, and as so could use the same > validity checker. > > Move the IVMD to x86 common IOMMU code and adjust the function to take a pair > of [start, end] mfn parameters. > > So far only the AMD-Vi side is

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

2024-02-12 Thread osstest service owner
flight 184649 linux-linus real [real] http://logs.test-lab.xenproject.org/osstest/logs/184649/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-amd64-amd64-xl-qemut-win7-amd64 19 guest-stopfail like 184647 test-amd64-amd64-xl-qemuu-win7-amd64

Re: [PATCH v2 2/2] x86/p2m: aid the compiler in folding p2m_is_...()

2024-02-12 Thread Jan Beulich
On 07.02.2024 04:07, George Dunlap wrote: > On Thu, Feb 1, 2024 at 10:15 PM Jan Beulich wrote: > --- a/xen/arch/x86/mm/p2m.c > +++ b/xen/arch/x86/mm/p2m.c > @@ -379,7 +379,7 @@ struct page_info *p2m_get_page_from_gfn( > return page; > =20 > /* Error path: not a suitable GFN

Re: [PATCH v8 5/7] xen/asm-generic: introduce generic device.h

2024-02-12 Thread Jan Beulich
On 09.02.2024 19:00, Oleksii Kurochko wrote: > --- /dev/null > +++ b/xen/include/asm-generic/device.h > @@ -0,0 +1,149 @@ > +/* SPDX-License-Identifier: GPL-2.0-only */ > +#ifndef __ASM_GENERIC_DEVICE_H__ > +#define __ASM_GENERIC_DEVICE_H__ > + > +#include > + > +enum device_type > +{ > +#ifdef

Re: [PATCH v2] xen/lib: introduce generic find next bit operations

2024-02-12 Thread Jan Beulich
On 09.02.2024 18:58, Oleksii Kurochko wrote: > find-next-bit.c is common for Arm64, PPC and RISCV64, > so it is moved to xen/lib. > > PPC has been transitioned to generic functions from find-next-bit.c > since it now shares the same implementation as the PPC-specific code. > > The MISRA exclude

Re: [PATCH v2] x86/vmx: add support for virtualize SPEC_CTRL

2024-02-12 Thread Jan Beulich
On 09.02.2024 12:40, Roger Pau Monne wrote: > @@ -1378,6 +1379,10 @@ static int construct_vmcs(struct vcpu *v) > rc = vmx_add_msr(v, MSR_PRED_CMD, PRED_CMD_IBPB, > VMX_MSR_HOST); > > +/* Set any bits we don't allow toggling in the mask field. */ > +if (

[PATCH 2/2] x86/MCE: adjust x86_mc_get_cpu_info()

2024-02-12 Thread Jan Beulich
Move logic independent of c->apicid's initialization status out of the if/else, leveraging that cpu_data[] now doesn't start out zero- initialized. Constify c and have it have an initializer. Signed-off-by: Jan Beulich --- a/xen/arch/x86/cpu/mcheck/mce.c +++ b/xen/arch/x86/cpu/mcheck/mce.c @@

[PATCH 1/2] x86/CPU: re-work populating of cpu_data[]

2024-02-12 Thread Jan Beulich
Forever since its introduction x86_mc_get_cpu_info() has been checking a structure field against BAD_APICID which would never have been set to that particular value. Arrange for this to actually be true ahead of CPU detection having run, which then also includes pruning after CPU removal. For

[PATCH 0/2] x86: cpu_data[] populating / consuming

2024-02-12 Thread Jan Beulich
Kind of fallout from looking through the AP bringup acceleration series. 1: CPU: re-work populating of cpu_data[] 2: MCE: adjust x86_mc_get_cpu_info() Jan

Re: [XEN PATCH v4] xen/arm: ffa: reclaim shared memory on guest destroy

2024-02-12 Thread Bertrand Marquis
Hi Jens, > On 5 Feb 2024, at 16:49, Jens Wiklander wrote: > > When an FF-A enabled guest is destroyed it may leave behind memory > shared with SPs. This memory must be reclaimed before it's reused or an > SP may make changes to memory used by a new unrelated guest. So when the > domain is

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

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

Re: [PATCH 3/7] VT-d: respect ACPI SATC's ATC_REQUIRED flag

2024-02-12 Thread Jan Beulich
On 08.02.2024 13:42, Roger Pau Monné wrote: > On Mon, Feb 05, 2024 at 02:56:14PM +0100, Jan Beulich wrote: >> When the flag is set, permit Dom0 to control the device (no worse than >> what we had before and in line with other "best effort" behavior we use >> when it comes to Dom0), but suppress

Re: [PATCH 0/4] libxl: blktap/tapback support

2024-02-12 Thread Anthony PERARD
On Thu, Feb 01, 2024 at 01:30:20PM -0500, Jason Andryuk wrote: > This patch re-introduces blktap support to libxl. Unlike earlier > versions, it does not link against any blktap library. libxl changes > are needed to write to the vbd3 backend XenStore nodes. > > blktap has three components.

Re: [PATCH 2/7] IOMMU: rename and re-type ats_enabled

2024-02-12 Thread Jan Beulich
On 12.02.2024 10:39, Roger Pau Monné wrote: > On Thu, Feb 08, 2024 at 04:49:46PM +0100, Jan Beulich wrote: >> On 08.02.2024 12:49, Roger Pau Monné wrote: >>> On Mon, Feb 05, 2024 at 02:55:43PM +0100, Jan Beulich wrote: Make the variable a tristate, with (as done elsewhere) a negative value

Re: [PATCH 4/4] libxl: Support blktap with HVM device model

2024-02-12 Thread Anthony PERARD
On Thu, Feb 01, 2024 at 01:30:24PM -0500, Jason Andryuk wrote: > diff --git a/tools/libs/light/libxl_disk.c b/tools/libs/light/libxl_disk.c > @@ -1337,10 +1338,18 @@ char *libxl__device_disk_find_local_path(libxl__gc > *gc, > LOGD(DEBUG, guest_domid, "Attempting to read node %s",

Re: [PATCH 3/4] hotplug: Update block-tap

2024-02-12 Thread Anthony PERARD
On Thu, Feb 01, 2024 at 01:30:23PM -0500, Jason Andryuk wrote: > Implement a sharing check like the regular block script. > > Checking tapback inside block-tap is too late since it needs to be > running to transition the backend to InitWait before block-tap is run. > > tap-ctl check will be

Re: [PATCH 2/4] libxl: Add support for blktap vbd3

2024-02-12 Thread Anthony PERARD
On Thu, Feb 01, 2024 at 01:30:22PM -0500, Jason Andryuk wrote: > This patch re-introduces blktap support to libxl. Unlike earlier > versions, it does not link against any blktap library. libxl changes > are needed to write to the vbd3 backend XenStore nodes. > > blktap has three components.

Re: [PATCH 7/7] VT-d: move {,un}map_vtd_domain_page()

2024-02-12 Thread Roger Pau Monné
On Mon, Feb 12, 2024 at 10:46:55AM +0100, Jan Beulich wrote: > On 09.02.2024 09:39, Roger Pau Monné wrote: > > On Mon, Feb 05, 2024 at 02:57:30PM +0100, Jan Beulich wrote: > >> ..., thus allowing them to become static. There's nothing x86-specific > >> about these functions anyway. > >> > >> Since

Re: [PATCH 1/7] VT-d: parse ACPI "SoC Integrated Address Translation Cache Reporting Structure"s

2024-02-12 Thread Roger Pau Monné
On Mon, Feb 12, 2024 at 10:32:00AM +0100, Jan Beulich wrote: > On 09.02.2024 10:00, Roger Pau Monné wrote: > > On Thu, Feb 08, 2024 at 04:29:34PM +0100, Jan Beulich wrote: > >> On 08.02.2024 10:17, Roger Pau Monné wrote: > >>> On Mon, Feb 05, 2024 at 02:55:17PM +0100, Jan Beulich wrote: > +

Re: [PATCH v8 2/7] automation: introduce fixed randconfig for RISC-V

2024-02-12 Thread Oleksii
On Mon, 2024-02-12 at 09:39 +0100, Michal Orzel wrote: > Hi Oleksii, Hi Michal, > > On 09/02/2024 19:00, Oleksii Kurochko wrote: > > > > > > This patch introduces the file riscv-fixed-randconfig.yaml, > > which includes all configurations that should be disabled for > > randconfig builds. > >

Re: [PATCH 7/7] VT-d: move {,un}map_vtd_domain_page()

2024-02-12 Thread Jan Beulich
On 09.02.2024 09:39, Roger Pau Monné wrote: > On Mon, Feb 05, 2024 at 02:57:30PM +0100, Jan Beulich wrote: >> ..., thus allowing them to become static. There's nothing x86-specific >> about these functions anyway. >> >> Since only the "iommu_inclusive_mapping" parameter declaration would be >>

Re: [PATCH v8 1/7] automation: ensure values in EXTRA_FIXED_RANDCONFIG are separated by new line

2024-02-12 Thread Oleksii
On Mon, 2024-02-12 at 09:12 +0100, Michal Orzel wrote: > Hi Oleksii, Hi Michal, > > On 09/02/2024 19:00, Oleksii Kurochko wrote: > > > > > > Kconfig tool expects each configuration to be on a new line. > > > > The current version of the build script puts all of > > ${EXTRA_FIXED_RANDCONFIG} >

Re: [PATCH 2/7] IOMMU: rename and re-type ats_enabled

2024-02-12 Thread Roger Pau Monné
On Thu, Feb 08, 2024 at 04:49:46PM +0100, Jan Beulich wrote: > On 08.02.2024 12:49, Roger Pau Monné wrote: > > On Mon, Feb 05, 2024 at 02:55:43PM +0100, Jan Beulich wrote: > >> Make the variable a tristate, with (as done elsewhere) a negative value > >> meaning "default". Since all use sites need

Re: [PATCH 1/7] VT-d: parse ACPI "SoC Integrated Address Translation Cache Reporting Structure"s

2024-02-12 Thread Jan Beulich
On 09.02.2024 10:00, Roger Pau Monné wrote: > On Thu, Feb 08, 2024 at 04:29:34PM +0100, Jan Beulich wrote: >> On 08.02.2024 10:17, Roger Pau Monné wrote: >>> On Mon, Feb 05, 2024 at 02:55:17PM +0100, Jan Beulich wrote: +{ +dprintk(XENLOG_WARNING VTDPREFIX, +

[xen-unstable test] 184648: tolerable FAIL

2024-02-12 Thread osstest service owner
flight 184648 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/184648/ Failures :-/ but no regressions. Tests which did not succeed, but are not blocking: test-armhf-armhf-libvirt 16 saverestore-support-checkfail like 184645

Errors when trying to create VM with "channel" device(s)

2024-02-12 Thread Jürgen Groß
A customer is complaining to be unable to create multiple "channel" devices in a HVM domain. I've verified the same happens with upstream Xen. The "channel" configuration is: channel = [ "connection=socket, name=test.ch1, path=/run/testsocket1", "connection=socket, name=test.ch2,

Re: [PATCH v3 2/8] keyhandler: drop regs parameter from handle_keyregs()

2024-02-12 Thread Jan Beulich
On 08.02.2024 23:09, Julien Grall wrote: > On 05/02/2024 13:28, Jan Beulich wrote: >> In preparation for further removal of regs parameters, drop it here. In >> the two places where it's actually needed, retrieve IRQ context if >> available, or else guest context. >> >> Suggested-by: Andrew Cooper

Re: [RFC KERNEL PATCH v4 3/3] PCI/sysfs: Add gsi sysfs for pci_dev

2024-02-12 Thread Roger Pau Monné
On Fri, Feb 09, 2024 at 03:05:49PM -0600, Bjorn Helgaas wrote: > On Thu, Feb 01, 2024 at 09:39:49AM +0100, Roger Pau Monné wrote: > > On Wed, Jan 31, 2024 at 01:00:14PM -0600, Bjorn Helgaas wrote: > > > On Wed, Jan 31, 2024 at 09:58:19AM +0100, Roger Pau Monné wrote: > > > > On Tue, Jan 30, 2024

Re: [PATCH v3 1/8] serial: fake IRQ-regs context in poll handlers

2024-02-12 Thread Jan Beulich
On 08.02.2024 23:00, Julien Grall wrote: > On 05/02/2024 13:27, Jan Beulich wrote: >> In preparation of dropping the register parameters from >> serial_[rt]x_interrupt() and in turn from IRQ handler functions, >> register state needs making available another way for the few key >> handlers which

Re: [XEN PATCH v2 2/3] x86/uaccess: replace __{get,put}_user_bad() with STATIC_ASSERT_UNREACHABLE()

2024-02-12 Thread Jan Beulich
On 09.02.2024 10:50, Federico Serafini wrote: > On 08/02/24 12:14, Jan Beulich wrote: >> On 08.02.2024 11:45, Federico Serafini wrote: >>> On 07/02/24 17:19, Jan Beulich wrote: On 07.02.2024 16:58, Federico Serafini wrote: > On 07/02/24 16:24, Jan Beulich wrote: >> On 07.02.2024

Re: [PATCH v8 2/7] automation: introduce fixed randconfig for RISC-V

2024-02-12 Thread Michal Orzel
Hi Oleksii, On 09/02/2024 19:00, Oleksii Kurochko wrote: > > > This patch introduces the file riscv-fixed-randconfig.yaml, > which includes all configurations that should be disabled for > randconfig builds. > > Suggested-by: Stefano Stabellini > Signed-off-by: Oleksii Kurochko > --- > The

Re: Return type of clean_and_invalidate_dcache_va_range

2024-02-12 Thread Jan Beulich
On 10.02.2024 11:17, Julien Grall wrote: > Hi, > > On 09/02/2024 22:02, Stefano Stabellini wrote: >> On Fri, 9 Feb 2024, Nicola Vetrini wrote: >>> Hi all, >>> >>> In the context of violations of MISRA C:2012 Rule 17.7: "The value returned >>> by >>> a function having non-void return type shall

Re: [PATCH v1] docs/misra/rules.rst: catch up with accepted rules

2024-02-12 Thread Jan Beulich
On 10.02.2024 02:00, Stefano Stabellini wrote: > Update docs/misra/rules.rst to reflect the MISRA C rules accepted in the > last couple of months. > > Signed-off-by: Stefano Stabellini > --- > > In the notes section I added some info about the deviations, but in any > case the appropriate info

Re: [PATCH v8 1/7] automation: ensure values in EXTRA_FIXED_RANDCONFIG are separated by new line

2024-02-12 Thread Michal Orzel
Hi Oleksii, On 09/02/2024 19:00, Oleksii Kurochko wrote: > > > Kconfig tool expects each configuration to be on a new line. > > The current version of the build script puts all of ${EXTRA_FIXED_RANDCONFIG} > in a single line and configs are seperated by spaces. > > As a result, only the first