Re: [PATCH v2 02/11] x86/mkreloc: fix obtaining PE image base address

2025-04-10 Thread Roger Pau Monné
On Tue, Apr 08, 2025 at 02:34:48PM +0200, Jan Beulich wrote: > On 08.04.2025 13:21, Roger Pau Monné wrote: > > On Wed, Apr 02, 2025 at 09:46:53AM +0200, Jan Beulich wrote: > >> x86/EFI: correct mkreloc header (field) reading > >> > >> With us now reading the full combined optional and NT headers, t

Re: [PATCH] arm/irq: Reduce size of irq_desc array to exclude local IRQs

2025-04-10 Thread Orzel, Michal
On 08/04/2025 05:42, Mykola Kvach wrote: > > > From: Mykola Kvach > > SGI and PPI descriptors are banked and stored in the per-CPU local_irq_desc > array, so not all elements of the global irq_desc array are used. This is > already accounted for in the descriptor lookup logic inside __irq_to

Re: [PATCH v2 2/8] vpci/header: Emulate legacy capability list for host

2025-04-10 Thread Jan Beulich
On 09.04.2025 08:45, Jiqian Chen wrote: > Current logic of init_header() only emulates legacy capability list > for guest, expand it to emulate for host too. So that it will be > easy to hide a capability whose initialization fails and no need > to distinguish host or guest. Before I (maybe) look

Re: [PATCH v3 04/16] kconfig: introduce option to independently enable libfdt

2025-04-10 Thread Alejandro Vallejo
On Thu Apr 10, 2025 at 10:04 AM BST, Jan Beulich wrote: > On 08.04.2025 18:07, Alejandro Vallejo wrote: >> From: "Daniel P. Smith" >> >> Currently, the inclusion of libfdt is controlled by the >> CONFIG_HAS_DEVICE_TREE >> kconfig flag. This flag also changes behavior in a few places, such as boo

Re: [PATCH TEST-ARTEFACTS 0/8] Cleanup and Linux ARM64 support

2025-04-10 Thread Andrew Cooper
On 10/04/2025 1:09 am, Jason Andryuk wrote: > On 2025-04-09 13:01, Andrew Cooper wrote: >> On 09/04/2025 5:36 pm, Andrew Cooper wrote: >>> Various bits of cleanup, and support for arm64 Linux builds. >>> >>> Run using the new Linux 6.6.86 on (most) x86, and ARM64: >>>    >>> https://gitlab.com/xen-

Re: [PATCH v3 11/16] x86/hyperlaunch: locate dom0 initrd with hyperlaunch

2025-04-10 Thread Jan Beulich
On 08.04.2025 18:07, Alejandro Vallejo wrote: > --- a/xen/arch/x86/domain-builder/fdt.c > +++ b/xen/arch/x86/domain-builder/fdt.c > @@ -195,6 +195,35 @@ static int __init process_domain_node( > !((char *)__va(bd->kernel->cmdline_pa))[0] ) > bd->kernel->fdt_cmdline

Re: [PATCH v3 01/16] x86/boot: introduce boot domain

2025-04-10 Thread Daniel P. Smith
On 4/9/25 02:24, Jan Beulich wrote: On 08.04.2025 18:07, Alejandro Vallejo wrote: From: "Daniel P. Smith" To begin moving toward allowing the hypervisor to construct more than one domain at boot, a container is needed for a domain's build information. Introduce a new header, , that contains th

Re: [PATCH v3 01/16] x86/boot: introduce boot domain

2025-04-10 Thread Daniel P. Smith
On 4/9/25 06:28, Alejandro Vallejo wrote: On Wed Apr 9, 2025 at 7:24 AM BST, Jan Beulich wrote: On 08.04.2025 18:07, Alejandro Vallejo wrote: From: "Daniel P. Smith" To begin moving toward allowing the hypervisor to construct more than one domain at boot, a container is needed for a domain's

Re: [PATCH] x86/EFI: sanitize DLL characteristics in binary

2025-04-10 Thread Daniel P. Smith
On 4/2/25 04:51, Jan Beulich wrote: In GNU ld --disable-reloc-section implies --disable-dynamicbase (and also --disable-high-entropy-va, just fyi). Therefore to yield functionally identical binaries independent of whether mkreloc needs to come into play, add --dynamicbase as well. GNU ld further

Re: [PATCH v3 05/16] kconfig: introduce domain builder config option

2025-04-10 Thread Alejandro Vallejo
On Thu Apr 10, 2025 at 10:08 AM BST, Jan Beulich wrote: > On 08.04.2025 18:07, Alejandro Vallejo wrote: >> --- /dev/null >> +++ b/xen/arch/x86/domain-builder/Kconfig > > This really looks to be the first patch where it needs settling on whether > all of this is to live under arch/x86/. If it is to,

Re: [GIT PULL] xen: branch for v6.15-rc2

2025-04-10 Thread pr-tracker-bot
The pull request you sent on Thu, 10 Apr 2025 11:07:13 +0200: > git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git > for-linus-6.15a-rc2-tag has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/2eb959eeecc64fa56e9f89a5fc496da297585cbe Thank you! -- Deet-doot-dot,

Re: [PATCH 5/6] Infrastructure for arm64 linux builds

2025-04-10 Thread Andrew Cooper
On 10/04/2025 2:51 pm, Anthony PERARD wrote: > On Thu, Apr 10, 2025 at 12:46:27PM +0100, Andrew Cooper wrote: >> diff --git a/images/alpine/3.18-arm64-build.dockerfile >> b/images/alpine/3.18-arm64-build.dockerfile >> new file mode 100644 >> index ..b6d9bd1af531 >> --- /dev/null >> +++

Re: [PATCH v2 2/7] arm/mpu: Provide access to the MPU region from the C code

2025-04-10 Thread Ayan Kumar Halder
Hi Luca, On 07/04/2025 10:14, Luca Fancellu wrote: CAUTION: This message has originated from an External Source. Please use proper judgment and caution when opening attachments, clicking links, or responding to this email. Implement some utility function in order to access the MPU regions fr

Re: [PATCH 1/3] CI: Update x86 tests from Linux 6.1.19 to 6.6.56

2025-04-10 Thread Marek Marczykowski-Górecki
On Thu, Apr 10, 2025 at 01:05:18PM +0100, Andrew Cooper wrote: > Linux 6.6.56 was already added to test-artifacts for the argo testing, and > this removes one moving part while cleaning things up. > > Drop the associated export job, and dockerfile. > > Signed-off-by: Andrew Cooper Reviewed-by:

Re: [PATCH 2/3] CI: Update ARM64 tests from Linux 6.6.74 to 6.6.86

2025-04-10 Thread Marek Marczykowski-Górecki
On Thu, Apr 10, 2025 at 01:05:19PM +0100, Andrew Cooper wrote: > Switch over to test-artifacts. > > Drop the associated export job, and dockerfile. > > Signed-off-by: Andrew Cooper Reviewed-by: Marek Marczykowski-Górecki -- Best Regards, Marek Marczykowski-Górecki Invisible Things Lab sign

Re: [PATCH 3/3] CI: Drop stale test-artifacts/alpine/3.18.dockerfile

2025-04-10 Thread Marek Marczykowski-Górecki
On Thu, Apr 10, 2025 at 01:05:20PM +0100, Andrew Cooper wrote: > This should have been dropped too, as the authoritative copy lives in another > repo now. > > Fixes: babe11b46c1a ("CI: Drop alpine-3.18-rootfs-export and use > test-artefacts") > Signed-off-by: Andrew Cooper Reviewed-by: Marek Ma

Re: [PATCH v1 03/14] xen/riscv: introduce ioremap()

2025-04-10 Thread Jan Beulich
On 08.04.2025 17:57, Oleksii Kurochko wrote: > Based on RISC-V unpriviliged spec ( Version 20240411 ): > ``` > For implementations that conform to the RISC-V Unix Platform Specification, > I/O devices and DMA operations are required to access memory coherently and > via strongly ordered I/O channel

Re: [PATCH v2 1/3] kasan: Avoid sleepable page allocation from atomic context

2025-04-10 Thread Alexander Gordeev
On Wed, Apr 09, 2025 at 04:56:29PM +0200, Andrey Ryabinin wrote: Hi Andrey, ... > >>> - page = __get_free_page(GFP_KERNEL); > >>> + page = __get_free_page(GFP_ATOMIC); > >>> if (!page) > >> I think a better way to fix this would be moving out allocation from > >> atomic context. Allocate page

Re: [PATCH v2 0/5] Fix lazy mmu mode

2025-04-10 Thread Alexander Gordeev
On Mon, Mar 03, 2025 at 02:15:34PM +, Ryan Roberts wrote: Hi Ryan, > I'm planning to implement lazy mmu mode for arm64 to optimize vmalloc. As part > of that, I will extend lazy mmu mode to cover kernel mappings in vmalloc table > walkers. While lazy mmu mode is already used for kernel mappin

Re: [PATCH TEST-ARTEFACTS] Fix adduser --shell path

2025-04-10 Thread Andrew Cooper
On 10/04/2025 4:56 pm, Anthony PERARD wrote: > We need the full path to a shell in `/etc/passwd`, otherwise `su user` > doesn't work. > > $ su user > su: can't execute 'bash': No such file or directory > > Signed-off-by: Anthony PERARD Acked-by: Andrew Cooper Would you like me to pick t

Re: [PATCH v1 07/14] xen/riscv: Introduce intc_hw_operations abstraction

2025-04-10 Thread Jan Beulich
On 08.04.2025 17:57, Oleksii Kurochko wrote: > --- a/xen/arch/riscv/include/asm/intc.h > +++ b/xen/arch/riscv/include/asm/intc.h > @@ -17,6 +17,29 @@ struct intc_info { > const struct dt_device_node *node; > }; > > +struct intc_hw_operations { > +/* Hold intc hw information */ > +co

[PATCH] xen: mapcache: Fix finding matching entry

2025-04-10 Thread Aleksandr Partanen
If we have request without lock and hit unlocked or invalid entry during the search, we remap it immediately, even if we have matching entry in next entries in bucket. This leads to duplication of mappings of the same size, and to possibility of selecting the wrong element during invalidation and u

Re: [PATCH v1 05/14] xen/riscv: introduce platform_get_irq()

2025-04-10 Thread Jan Beulich
On 08.04.2025 17:57, Oleksii Kurochko wrote: > @@ -21,6 +22,22 @@ static struct intc_info __ro_after_init aplic_info = { > .hw_version = INTC_APLIC, > }; > > +static int aplic_irq_xlate(const uint32_t *intspec, unsigned int intsize, As you start adding functions calling indirectly, please

[PATCH] Arm: cpu_*_map adjustments

2025-04-10 Thread Jan Beulich
First, they all start out zeroed. There's no point doing an initial cpumask_clear() on them. Next, only cpu_online_map may be altered post-boot, and even that only rarely. Add respective placement attributes. Finally, cpu_present_map really isn't anything more than an alias of cpu_possible_map. A

Re: [PATCH v3 06/16] x86/hyperlaunch: introduce the domain builder

2025-04-10 Thread Alejandro Vallejo
On Wed Apr 9, 2025 at 7:53 PM BST, Denis Mukhin wrote: > On Tuesday, April 8th, 2025 at 9:07 AM, Alejandro Vallejo > wrote: > >> >> >> From: "Daniel P. Smith" dpsm...@apertussolutions.com >> >> >> Introduce the domain builder which is capable of consuming a device tree as >> the >> first boo

Re: [PATCH 3/6] Adjust Linux build script to work with other major versions

2025-04-10 Thread Marek Marczykowski-Górecki
On Thu, Apr 10, 2025 at 12:46:25PM +0100, Andrew Cooper wrote: > Simply the tar expression, as 'tar xf' can figure out the compression > automatically. > > Signed-off-by: Andrew Cooper Reviewed-by: Marek Marczykowski-Górecki > --- > CC: Roger Pau Monné > CC: Stefano Stabellini > CC: Michal O

Re: [PATCH v3 02/16] x86/boot: introduce domid field to struct boot_domain

2025-04-10 Thread Jason Andryuk
On 2025-04-09 06:33, Alejandro Vallejo wrote: On Wed Apr 9, 2025 at 7:34 AM BST, Jan Beulich wrote: On 08.04.2025 18:07, Alejandro Vallejo wrote: @@ -1010,15 +1010,15 @@ static struct domain *__init create_dom0(struct boot_info *bi) dom0_cfg.flags |= XEN_DOMCTL_CDF_iommu; /*

Re: [PATCH v1 04/14] xen/riscv: introduce init_IRQ()

2025-04-10 Thread Jan Beulich
On 08.04.2025 17:57, Oleksii Kurochko wrote: > --- a/xen/arch/riscv/include/asm/irq.h > +++ b/xen/arch/riscv/include/asm/irq.h > @@ -3,6 +3,28 @@ > #define ASM__RISCV__IRQ_H > > #include > +#include > + > +#define NR_IRQS 1024 > + > +/* > + * TODO: Should IRQ_TYPE_* be moved to xen/irq.h and

Re: [PATCH v1 01/14] xen/riscv: implement get_s_time()

2025-04-10 Thread Jan Beulich
On 08.04.2025 17:57, Oleksii Kurochko wrote: > @@ -23,6 +24,11 @@ static inline cycles_t get_cycles(void) > return csr_read(CSR_TIME); > } > > +static inline s_time_t ticks_to_ns(uint64_t ticks) > +{ > +return muldiv64(ticks, SECONDS(1), 1000 * cpu_khz); > +} Why the extra multiplicati

Re: [PATCH v3 16/16] x86/hyperlaunch: add capabilities to boot domain

2025-04-10 Thread Jan Beulich
On 08.04.2025 18:07, Alejandro Vallejo wrote: > From: "Daniel P. Smith" > > Introduce the ability to assign capabilities to a domain via its definition in > device tree. The first capability enabled to select is the control domain > capability. The capability property is a bitfield in both the de

Re: [PATCH 2/6] Clean up Gitlab yaml

2025-04-10 Thread Anthony PERARD
On Thu, Apr 10, 2025 at 12:46:24PM +0100, Andrew Cooper wrote: > Factor out the registry into a common location. Drop the worflow name, as all > it does is hide the commit message of the change that triggered the pipeline. > List all the stages. Set a default expiry of 1 month. > > Note all the

Re: [PATCH 6/6] Linux builds for x86 and ARM64

2025-04-10 Thread Anthony PERARD
On Thu, Apr 10, 2025 at 12:46:28PM +0100, Andrew Cooper wrote: > For ARM64, the tests in Xen currently use 6.6.74, but 6.6.86 works so update > to the latest. > > For x86, stay at 6.6.56. There's a PVH regression upstream in need of fixing. > > Signed-off-by: Andrew Cooper Acked-by: Anthony PE

Re: [PATCH v2 3/3] mm: Protect kernel pgtables in apply_to_pte_range()

2025-04-10 Thread Alexander Gordeev
On Tue, Apr 08, 2025 at 06:07:32PM +0200, Alexander Gordeev wrote: Hi Andrew, > The lazy MMU mode can only be entered and left under the protection > of the page table locks for all page tables which may be modified. Heiko Carstens noticed that the above claim is not valid, since v6.15-rc1 commi

Re: [PATCH v3 07/16] x86/hyperlaunch: initial support for hyperlaunch device tree

2025-04-10 Thread Alejandro Vallejo
On Thu Apr 10, 2025 at 11:10 AM BST, Jan Beulich wrote: > On 08.04.2025 18:07, Alejandro Vallejo wrote: >> --- a/xen/arch/x86/domain-builder/core.c >> +++ b/xen/arch/x86/domain-builder/core.c >> @@ -44,6 +44,21 @@ void __init builder_init(struct boot_info *bi) >> break; >> } >

Re: [PATCH] x86/EFI: sanitize DLL characteristics in binary

2025-04-10 Thread Jan Beulich
On 10.04.2025 16:39, Daniel P. Smith wrote: > On 4/2/25 04:51, Jan Beulich wrote: >> --- a/xen/arch/x86/arch.mk >> +++ b/xen/arch/x86/arch.mk >> @@ -113,6 +113,7 @@ efi-nr-fixups := $(shell LC_ALL=C $(OBJD >> >> ifeq ($(efi-nr-fixups),2) >> MKRELOC := : >> +EFI_LDFLAGS += --disable-high-entr

Re: [PATCH v3 01/16] x86/boot: introduce boot domain

2025-04-10 Thread Jan Beulich
On 10.04.2025 15:09, Daniel P. Smith wrote: > On 4/9/25 02:24, Jan Beulich wrote: >> On 08.04.2025 18:07, Alejandro Vallejo wrote: >>> From: "Daniel P. Smith" >>> >>> To begin moving toward allowing the hypervisor to construct more than one >>> domain at boot, a container is needed for a domain's

Re: [PATCH 1/6] Port containerise

2025-04-10 Thread Anthony PERARD
On Thu, Apr 10, 2025 at 12:46:23PM +0100, Andrew Cooper wrote: > While it pains me to keep the wrong spelling, do so for consistency. > > Signed-off-by: Andrew Cooper Reviewed-by: Anthony PERARD Thanks, -- Anthony Perard | Vates XCP-ng Developer XCP-ng & Xen Orchestra - Vates solutions we

Re: [PATCH TEST-ARTEFACTS] Fix adduser --shell path

2025-04-10 Thread Andrew Cooper
On 10/04/2025 5:38 pm, Anthony PERARD wrote: > On Thu, Apr 10, 2025 at 04:58:59PM +0100, Andrew Cooper wrote: >> On 10/04/2025 4:56 pm, Anthony PERARD wrote: >>> We need the full path to a shell in `/etc/passwd`, otherwise `su user` >>> doesn't work. >>> >>> $ su user >>> su: can't execute

[GIT PULL] xen: branch for v6.15-rc2

2025-04-10 Thread Juergen Gross
Linus, Please git pull the following tag: git://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git for-linus-6.15a-rc2-tag xen: branch for v6.15-rc2 It contains the following patches: - A simple fix adding the module description of the Xenbus frontend module. - A fix correcting the xen-a

Re: [PATCH v3 05/16] kconfig: introduce domain builder config option

2025-04-10 Thread Jan Beulich
On 08.04.2025 18:07, Alejandro Vallejo wrote: > --- /dev/null > +++ b/xen/arch/x86/domain-builder/Kconfig This really looks to be the first patch where it needs settling on whether all of this is to live under arch/x86/. If it is to, the reasons need writing down somewhere - maybe not here, but at

Re: [PATCH v2 02/11] x86/mkreloc: fix obtaining PE image base address

2025-04-10 Thread Jan Beulich
On 10.04.2025 09:20, Roger Pau Monné wrote: > On Tue, Apr 08, 2025 at 02:34:48PM +0200, Jan Beulich wrote: >> On 08.04.2025 13:21, Roger Pau Monné wrote: >>> On Wed, Apr 02, 2025 at 09:46:53AM +0200, Jan Beulich wrote: @@ -54,31 +56,40 @@ static unsigned int load(const char *nam

Re: [PATCH 3/8] Fix container user setup

2025-04-10 Thread Anthony PERARD
On Wed, Apr 09, 2025 at 05:36:57PM +0100, Andrew Cooper wrote: > Use USER (and 'user' as normal), rather than specifying configuration from the > outside. This also fixes the fact that root-needing containers need to > override the default. This patch also removes one "docker:platform" which is r

Re: [PATCH 4/8] Clean up Gitlab yaml

2025-04-10 Thread Andrew Cooper
On 10/04/2025 11:07 am, Anthony PERARD wrote: > On Wed, Apr 09, 2025 at 05:36:58PM +0100, Andrew Cooper wrote: >> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml >> index 36ec6a7e1ee5..5a0a853e551d 100644 >> --- a/.gitlab-ci.yml >> +++ b/.gitlab-ci.yml >> @@ -1,12 +1,19 @@ >> +variables: >> + REGISTR

Re: [PATCH v3 04/16] kconfig: introduce option to independently enable libfdt

2025-04-10 Thread Jan Beulich
On 08.04.2025 18:07, Alejandro Vallejo wrote: > From: "Daniel P. Smith" > > Currently, the inclusion of libfdt is controlled by the CONFIG_HAS_DEVICE_TREE > kconfig flag. This flag also changes behavior in a few places, such as boot > module processing for XSM. To support the ability to include l

Re: [PATCH 2/8] Port containerise

2025-04-10 Thread Andrew Cooper
On 10/04/2025 10:33 am, Anthony PERARD wrote: > On Wed, Apr 09, 2025 at 05:36:56PM +0100, Andrew Cooper wrote: >> +BASE="registry.gitlab.com/xen-project/hardware/test-artifacts" >> +case "_${CONTAINER}" in >> +_alpine-x86_64-base) CONTAINER="${BASE}/alpine:x86_64-base" ;; > There's no "images/a

Re: [PATCH 4/8] Clean up Gitlab yaml

2025-04-10 Thread Andrew Cooper
On 09/04/2025 6:15 pm, Marek Marczykowski-Górecki wrote: > On Wed, Apr 09, 2025 at 05:36:58PM +0100, Andrew Cooper wrote: >> Factor out the registry into a common location. Fix the worflow name. List >> all the stages. Set a default expiry of 1 month. > This is okay, since last artifacts for a b

Re: [PATCH 7/8] Infrastructure for arm64 linux builds

2025-04-10 Thread Andrew Cooper
On 09/04/2025 6:15 pm, Andrew Cooper wrote: > On 09/04/2025 6:11 pm, Marek Marczykowski-Górecki wrote: >> On Wed, Apr 09, 2025 at 05:37:01PM +0100, Andrew Cooper wrote: >>> Signed-off-by: Andrew Cooper >>> --- >>> CC: Roger Pau Monné >>> CC: Stefano Stabellini >>> CC: Michal Orzel >>> CC: Marek

Re: [PATCH TEST-ARTEFACTS 0/8] Cleanup and Linux ARM64 support

2025-04-10 Thread Jürgen Groß
On 10.04.25 02:09, Jason Andryuk wrote: On 2025-04-09 13:01, Andrew Cooper wrote: On 09/04/2025 5:36 pm, Andrew Cooper wrote: Various bits of cleanup, and support for arm64 Linux builds. Run using the new Linux 6.6.86 on (most) x86, and ARM64:    https://gitlab.com/xen-project/hardware/xen-sta

Re: [PATCH v2 01/15] x86/boot: introduce boot domain

2025-04-10 Thread Jan Beulich
On 10.04.2025 01:55, Daniel P. Smith wrote: > On 4/7/25 03:10, Jan Beulich wrote: >> On 05.04.2025 02:04, Daniel P. Smith wrote: >>> On 1/30/25 08:45, Jan Beulich wrote: On 26.12.2024 17:57, Daniel P. Smith wrote: > @@ -596,9 +597,10 @@ int __init dom0_setup_permissions(struct domain *d) >

Re: [PATCH RESEND] x86/EFI: correct mkreloc header (field) reading

2025-04-10 Thread Roger Pau Monné
On Thu, Apr 10, 2025 at 09:45:15AM +0200, Jan Beulich wrote: > With us now reading the full combined optional and NT headers, the > subsequent reading of (and seeking to) NT header fields is wrong. Since > PE32 and PE32+ NT headers are different anyway (beyond the image base > oddity extending acro

Re: [PATCH 4/8] Clean up Gitlab yaml

2025-04-10 Thread Anthony PERARD
On Wed, Apr 09, 2025 at 05:36:58PM +0100, Andrew Cooper wrote: > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml > index 36ec6a7e1ee5..5a0a853e551d 100644 > --- a/.gitlab-ci.yml > +++ b/.gitlab-ci.yml > @@ -1,12 +1,19 @@ > +variables: > + REGISTRY: registry.gitlab.com/xen-project/hardware/test-artifa

[PATCH 4/6] Factor our x86-isms in the linux build script

2025-04-10 Thread Andrew Cooper
... in preparation to use it for arm64 too. Rename the script. Signed-off-by: Andrew Cooper Reviewed-by: Marek Marczykowski-Górecki --- CC: Roger Pau Monné CC: Stefano Stabellini CC: Michal Orzel CC: Marek Marczykowski-Górecki CC: Anthony PERARD --- .gitlab-ci.yml

[PATCH 3/6] Adjust Linux build script to work with other major versions

2025-04-10 Thread Andrew Cooper
Simply the tar expression, as 'tar xf' can figure out the compression automatically. Signed-off-by: Andrew Cooper --- CC: Roger Pau Monné CC: Stefano Stabellini CC: Michal Orzel CC: Marek Marczykowski-Górecki CC: Anthony PERARD v2: * Use 'tar xf' --- scripts/x86_64-kernel-linux.sh | 5 +++

[PATCH 1/6] Port containerise

2025-04-10 Thread Andrew Cooper
While it pains me to keep the wrong spelling, do so for consistency. Signed-off-by: Andrew Cooper --- CC: Roger Pau Monné CC: Stefano Stabellini CC: Michal Orzel CC: Marek Marczykowski-Górecki CC: Anthony PERARD v2: * Switch back to alpine:x86_64-rootfs as the rename isn't in the series ye

[PATCH TEST-ARTEFACTS v2 0/6] Cleanup and Linux ARM64 support

2025-04-10 Thread Andrew Cooper
Two patches merged. Only minor tweaks to the rest. https://gitlab.com/xen-project/hardware/test-artifacts/-/pipelines/1762204421 is a pipline showing clean kernel builds. https://gitlab.com/xen-project/hardware/xen-staging/-/pipelines/1762221755 is a Xen pipline using the artefacts from ^. Andr

[PATCH 2/6] Clean up Gitlab yaml

2025-04-10 Thread Andrew Cooper
Factor out the registry into a common location. Drop the worflow name, as all it does is hide the commit message of the change that triggered the pipeline. List all the stages. Set a default expiry of 1 month. Note all the current jobs as legacy. Their naming scheme needs changing, and we'll us

[PATCH 6/6] Linux builds for x86 and ARM64

2025-04-10 Thread Andrew Cooper
For ARM64, the tests in Xen currently use 6.6.74, but 6.6.86 works so update to the latest. For x86, stay at 6.6.56. There's a PVH regression upstream in need of fixing. Signed-off-by: Andrew Cooper --- CC: Roger Pau Monné CC: Stefano Stabellini CC: Michal Orzel CC: Marek Marczykowski-Góreck

[PATCH v3 05/16] kconfig: introduce domain builder config option

2025-04-10 Thread Alejandro Vallejo
From: "Daniel P. Smith" Hyperlaunch domain builder will be the consolidated boot time domain building logic framework. Introduces the config option to enable this domain builder to eventually turn on the ability to load the domain configuration via a flattened device tree. Signed-off-by: Daniel

Re: [PATCH v3 14/16] x86/hyperlaunch: add memory parsing to domain config

2025-04-10 Thread Jan Beulich
On 08.04.2025 18:07, Alejandro Vallejo wrote: > @@ -212,6 +213,39 @@ static int __init process_domain_node( > else > printk("PV\n"); > } > +else if ( strncmp(prop_name, "memory", name_len) == 0 ) > +{ > +uint64_t kb; > +

[PATCH 0/3] CI: Cleanup/Upgrade of Linux used for testing

2025-04-10 Thread Andrew Cooper
Switch fully on to test-artefacts for Linux. Patch 3 is a repost. Patch 1 supercedes a prior posting. https://gitlab.com/xen-project/hardware/xen-staging/-/pipelines/1762221755 Andrew Cooper (3): CI: Update x86 tests from Linux 6.1.19 to 6.6.56 CI: Update ARM64 tests from Linux 6.6.74 to 6.

[PATCH 1/3] CI: Update x86 tests from Linux 6.1.19 to 6.6.56

2025-04-10 Thread Andrew Cooper
Linux 6.6.56 was already added to test-artifacts for the argo testing, and this removes one moving part while cleaning things up. Drop the associated export job, and dockerfile. Signed-off-by: Andrew Cooper --- CC: Anthony PERARD CC: Stefano Stabellini CC: Michal Orzel CC: Doug Goldstein CC:

[PATCH 2/3] CI: Update ARM64 tests from Linux 6.6.74 to 6.6.86

2025-04-10 Thread Andrew Cooper
Switch over to test-artifacts. Drop the associated export job, and dockerfile. Signed-off-by: Andrew Cooper --- CC: Anthony PERARD CC: Stefano Stabellini CC: Michal Orzel CC: Doug Goldstein CC: Marek Marczykowski-Górecki --- automation/gitlab-ci/build.yaml | 11 -- automa

[PATCH 3/3] CI: Drop stale test-artifacts/alpine/3.18.dockerfile

2025-04-10 Thread Andrew Cooper
This should have been dropped too, as the authoritative copy lives in another repo now. Fixes: babe11b46c1a ("CI: Drop alpine-3.18-rootfs-export and use test-artefacts") Signed-off-by: Andrew Cooper --- CC: Anthony PERARD CC: Stefano Stabellini CC: Michal Orzel CC: Doug Goldstein CC: Marek M

Re: [PATCH v3 12/16] x86/hyperlaunch: add domain id parsing to domain config

2025-04-10 Thread Jan Beulich
On 08.04.2025 18:07, Alejandro Vallejo wrote: > From: "Daniel P. Smith" > > Introduce the ability to specify the desired domain id for the domain > definition. The domain id will be populated in the domid property of the > domain > node in the device tree configuration. Nit: Odd splitting of lin

Re: [PATCH v3 15/16] x86/hyperlaunch: add max vcpu parsing of hyperlaunch device tree

2025-04-10 Thread Jan Beulich
On 08.04.2025 18:07, Alejandro Vallejo wrote: > From: "Daniel P. Smith" > > Introduce the `cpus` property, named as such for dom0less compatibility, that > represents the maximum number of vpcus to allocate for a domain. In the device Nit: vcpus > --- a/xen/arch/x86/domain-builder/fdt.c > +++ b

Re: [PATCH 7/8] Infrastructure for arm64 linux builds

2025-04-10 Thread Andrew Cooper
On 10/04/2025 11:41 am, Andrew Cooper wrote: > On 09/04/2025 6:15 pm, Andrew Cooper wrote: >> On 09/04/2025 6:11 pm, Marek Marczykowski-Górecki wrote: >>> On Wed, Apr 09, 2025 at 05:37:01PM +0100, Andrew Cooper wrote: Signed-off-by: Andrew Cooper --- CC: Roger Pau Monné CC: St

Re: [PATCH v3 04/16] kconfig: introduce option to independently enable libfdt

2025-04-10 Thread Alejandro Vallejo
On Wed Apr 9, 2025 at 7:57 PM BST, Denis Mukhin wrote: > On Tuesday, April 8th, 2025 at 9:07 AM, Alejandro Vallejo > wrote: > >> >> >> From: "Daniel P. Smith" dpsm...@apertussolutions.com >> >> >> Currently, the inclusion of libfdt is controlled by the >> CONFIG_HAS_DEVICE_TREE >> kconfig fl

Re: [PATCH v3 16/16] x86/hyperlaunch: add capabilities to boot domain

2025-04-10 Thread Jan Beulich
On 08.04.2025 18:07, Alejandro Vallejo wrote: > From: "Daniel P. Smith" > > Introduce the ability to assign capabilities to a domain via its definition in > device tree. The first capability enabled to select is the control domain > capability. The capability property is a bitfield in both the de

Re: [PATCH v3 03/16] x86/boot: add cmdline to struct boot_domain

2025-04-10 Thread Alejandro Vallejo
On Wed Apr 9, 2025 at 10:05 PM BST, Denis Mukhin wrote: > On Tuesday, April 8th, 2025 at 9:07 AM, Alejandro Vallejo > wrote: > >> >> >> From: "Daniel P. Smith" dpsm...@apertussolutions.com >> >> >> Add a container for the "cooked" command line for a domain. This >> provides for the backing me

Ping: [PATCH] x86/EFI: sanitize DLL characteristics in binary

2025-04-10 Thread Jan Beulich
On 02.04.2025 10:51, Jan Beulich wrote: > In GNU ld --disable-reloc-section implies --disable-dynamicbase (and > also --disable-high-entropy-va, just fyi). Therefore to yield > functionally identical binaries independent of whether mkreloc needs to > come into play, add --dynamicbase as well. > >

Re: [PATCH 7/8] Infrastructure for arm64 linux builds

2025-04-10 Thread Marek Marczykowski-Górecki
On Thu, Apr 10, 2025 at 11:57:23AM +0100, Andrew Cooper wrote: > The awk complaint has gone away too, so I suppose that's a knock-on effect. It isn't? https://gitlab.com/xen-project/hardware/test-artifacts/-/jobs/9683454411#L106 Anyway, hopefully harmless. -- Best Regards, Marek Marczykowski-G

Re: [PATCH 2/6] Clean up Gitlab yaml

2025-04-10 Thread Marek Marczykowski-Górecki
On Thu, Apr 10, 2025 at 12:46:24PM +0100, Andrew Cooper wrote: > Factor out the registry into a common location. Drop the worflow name, as all > it does is hide the commit message of the change that triggered the pipeline. > List all the stages. Set a default expiry of 1 month. > > Note all the

Re: [PATCH v3 10/16] x86/hyperlaunch: obtain cmdline from device tree

2025-04-10 Thread Jan Beulich
On 08.04.2025 18:07, Alejandro Vallejo wrote: > --- a/xen/arch/x86/domain-builder/core.c > +++ b/xen/arch/x86/domain-builder/core.c > @@ -8,9 +8,37 @@ > #include > > #include > +#include > > #include "fdt.h" > > +size_t __init builder_get_cmdline_size(struct boot_info *bi, int offset) >

Re: [PATCH v2 1/8] driver/pci: Get next capability without passing caps

2025-04-10 Thread Jan Beulich
On 09.04.2025 08:45, Jiqian Chen wrote: > --- a/xen/drivers/pci/pci.c > +++ b/xen/drivers/pci/pci.c > @@ -40,7 +40,7 @@ unsigned int pci_find_cap_offset(pci_sbdf_t sbdf, unsigned > int cap) > } > > unsigned int pci_find_next_cap_ttl(pci_sbdf_t sbdf, unsigned int pos, > -

Re: [PATCH v3 06/16] x86/hyperlaunch: introduce the domain builder

2025-04-10 Thread Jan Beulich
On 09.04.2025 20:53, Denis Mukhin wrote: > On Tuesday, April 8th, 2025 at 9:07 AM, Alejandro Vallejo > wrote: > >> >> >> From: "Daniel P. Smith" dpsm...@apertussolutions.com >> >> >> Introduce the domain builder which is capable of consuming a device tree as >> the >> first boot module. If it f

Re: [PATCH 2/2] x86/hvm: fix write emulation of RO ranges

2025-04-10 Thread Roger Pau Monné
On Thu, Apr 10, 2025 at 08:27:49AM +0200, Jan Beulich wrote: > On 09.04.2025 17:33, Roger Pau Monné wrote: > > On Wed, Apr 09, 2025 at 04:08:47PM +0200, Jan Beulich wrote: > >> On 09.04.2025 16:01, Roger Pau Monné wrote: > >>> As I understand the checks done in > >>> mmio_ro_emulated_write() are t

[PATCH RESEND] x86/EFI: correct mkreloc header (field) reading

2025-04-10 Thread Jan Beulich
With us now reading the full combined optional and NT headers, the subsequent reading of (and seeking to) NT header fields is wrong. Since PE32 and PE32+ NT headers are different anyway (beyond the image base oddity extending across both headers), switch to using a union. This allows to fetch the i

Re: [PATCH v3] xen/riscv: Increase XEN_VIRT_SIZE

2025-04-10 Thread Jan Beulich
On 09.04.2025 21:01, Oleksii Kurochko wrote: > --- a/xen/arch/riscv/include/asm/mm.h > +++ b/xen/arch/riscv/include/asm/mm.h > @@ -9,6 +9,7 @@ > #include > #include > #include > +#include > #include > > #include > @@ -35,6 +36,11 @@ static inline void *maddr_to_virt(paddr_t ma) >

[PATCH] tools/libxl: Skip invalid IRQs

2025-04-10 Thread Jason Andryuk
A PCI device's irq field is an 8-bit number. A value of 0xff indicates that the device is not connected. Additionally, the Linux ACPI code can convert these 0xff values to IRQ_NOTCONNECTED(0x8000) because "0x8000 is guaranteed to be outside the available range of interrupts and easy to di

[PATCH v2 5/7] arm/mpu: Introduce MPU memory mapping flags

2025-04-10 Thread Luca Fancellu
Introduce the MPU memory mapping flags in asm/page.h. Signed-off-by: Luca Fancellu --- xen/arch/arm/include/asm/page.h | 25 + 1 file changed, 25 insertions(+) diff --git a/xen/arch/arm/include/asm/page.h b/xen/arch/arm/include/asm/page.h index 69f817d1e68a..22f7d2c6cb30

Re: [PATCH] x86/cpu: Validate CPUID leaf 0x2 EDX output

2025-04-10 Thread Andrew Cooper
On 19/03/2025 11:24 am, Jan Beulich wrote: > From: Ahmed S. Darwish > > CPUID leaf 0x2 emits one-byte descriptors in its four output registers > EAX, EBX, ECX, and EDX. For these descriptors to be valid, the most > significant bit (MSB) of each register must be clear. > > Leaf 0x2 parsing at inte

Re: [PATCH test-artifacts v2 14/12] scripts: build initrd cpio

2025-04-10 Thread Marek Marczykowski-Górecki
On Mon, Apr 07, 2025 at 05:22:05PM +0100, Andrew Cooper wrote: > On 07/04/2025 5:17 pm, Andrew Cooper wrote: > > On 07/04/2025 1:31 pm, Marek Marczykowski-Górecki wrote: > >> Build initrd format directly digestable by the kernel. Additionally, > >> include it itself inside as boot/initrd-domU file,

Re: [PATCH] tools/libxl: do not use `-c -E` compiler options together

2025-04-10 Thread Anthony PERARD
On Mon, Apr 07, 2025 at 01:09:38PM +0200, Roger Pau Monne wrote: > It makes no sense to request for preprocessor only output and also request > object file generation. Fix the _libxl.api-for-check target to only use > -E (preprocessor output). > > Also Clang 20.0 reports an error if both options a

[PATCH v1 06/14] xen/riscv: riscv_of_processor_hartid() implementation

2025-04-10 Thread Oleksii Kurochko
Implements riscv_of_processor_hartid() to get the hart ID of the given device tree node and do some checks if CPU is available and given device tree node has proper riscv,isa property. As a helper function of_get_cpu_hwid() is introduced to deal specifically with reg propery of a CPU device node.

[PATCH v5 3/5] ARM: sa1100: Switch to new sys-off handler API

2025-04-10 Thread Andrew Davis
Kernel now supports chained power-off handlers. Use register_platform_power_off() that registers a platform level power-off handler. Legacy pm_power_off() will be removed once all drivers and archs are converted to the new sys-off API. Signed-off-by: Andrew Davis --- arch/arm/mach-sa1100/generic

Re: [PATCH v3 07/16] x86/hyperlaunch: initial support for hyperlaunch device tree

2025-04-10 Thread Jan Beulich
On 08.04.2025 18:07, Alejandro Vallejo wrote: > --- a/xen/arch/x86/domain-builder/core.c > +++ b/xen/arch/x86/domain-builder/core.c > @@ -44,6 +44,21 @@ void __init builder_init(struct boot_info *bi) > break; > } > } > + > +if ( bi->hyperlaunch_enabled ) > +{ Not

Re: [PATCH v1 06/14] xen/riscv: riscv_of_processor_hartid() implementation

2025-04-10 Thread Jan Beulich
On 08.04.2025 17:57, Oleksii Kurochko wrote: > @@ -13,3 +16,68 @@ void __init smp_clear_cpu_maps(void) > cpumask_set_cpu(0, &cpu_online_map); > cpumask_copy(&cpu_present_map, &cpu_possible_map); > } > + > +/** > + * of_get_cpu_hwid - Get the hardware ID from a CPU device node > + * > + *

[PATCH v2 0/3] mm: Fix apply_to_pte_range() vs lazy MMU mode

2025-04-10 Thread Alexander Gordeev
Hi All, Chages since v1: - left fixes only, improvements will be posted separately; - Fixes: and -stable tags added to patch descriptions; This series is an attempt to fix the violation of lazy MMU mode context requirement as described for arch_enter_lazy_mmu_mode(): This mode can only be en

[PATCH v2 6/7] arm/mpu: Implement early_fdt_map support in MPU systems

2025-04-10 Thread Luca Fancellu
Implement early_fdt_map() function, that is responsible to map the device tree blob in the early stages of the boot process, since at this stage the MPU C data structure are not yet initialised, it is using low level APIs to write into the MPU registers at a fixed MPU region number. The MPU memory

Re: [PATCH TEST-ARTEFACTS] Fix adduser --shell path

2025-04-10 Thread Marek Marczykowski-Górecki
On Thu, Apr 10, 2025 at 05:57:23PM +0100, Andrew Cooper wrote: > On 10/04/2025 5:38 pm, Anthony PERARD wrote: > > And by the way, you might want to add `gawk` as > > well to replace busybox's awk, to make the linux build a bit more happy, > > even if it doesn't seems to be an issue right now. > >

Re: [PATCH v2 2/7] arm/mpu: Provide access to the MPU region from the C code

2025-04-10 Thread Ayan Kumar Halder
On 09/04/2025 11:21, Luca Fancellu wrote: Hi Ayan, Hi, On 9 Apr 2025, at 11:07, Ayan Kumar Halder wrote: On 09/04/2025 09:26, Luca Fancellu wrote: Hi Ayan, Hi Luca, The point of the code was to don’t issue an isb() every time we change the selector, of course the code would be easier

Re: [PATCH v1 1/3] xen/console: cleanup conring management

2025-04-10 Thread Jan Beulich
On 03.04.2025 02:06, dm...@proton.me wrote: > From: Denis Mukhin > > Move console_locks_busted handling inside conring_puts() to remove > tasklet code duplication. I for one wouldn't expect tasklet scheduling to happen in a function of this name. IOW I think the little bit of duplication is warr

Re: [PATCH] xen: fix multicall debug feature

2025-04-10 Thread Borislav Petkov
On Thu, Mar 27, 2025 at 08:06:02PM +0100, Juergen Gross wrote: > Initializing a percpu variable with the address of a struct tagged as > .initdata is breaking the build with CONFIG_SECTION_MISMATCH_WARN_ONLY > not set to "y". > > Fix that by using an access function instead returning the .initdata

[PATCH test-artifacts v2 13/12] scripts: add rc_verbose=yes to /etc/rc.conf

2025-04-10 Thread Marek Marczykowski-Górecki
This is done in every single test job, so do it at the initrd build time. Signed-off-by: Marek Marczykowski-Górecki --- scripts/x86_64-rootfs-alpine.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/scripts/x86_64-rootfs-alpine.sh b/scripts/x86_64-rootfs-alpine.sh index f8b04c5..b70b3a5 100

Re: [PATCH v3 08/16] x86/hyperlaunch: Add helpers to locate multiboot modules

2025-04-10 Thread Jan Beulich
On 08.04.2025 18:07, Alejandro Vallejo wrote: > --- a/xen/arch/x86/domain-builder/fdt.c > +++ b/xen/arch/x86/domain-builder/fdt.c > @@ -13,6 +13,148 @@ > > #include "fdt.h" > > +/* > + * Unpacks a "reg" property into its address and size constituents. > + * > + * @param prop Pointer t

Re: [PATCH TEST-ARTEFACTS] Fix adduser --shell path

2025-04-10 Thread Andrew Cooper
On 10/04/2025 6:18 pm, Marek Marczykowski-Górecki wrote: > On Thu, Apr 10, 2025 at 05:57:23PM +0100, Andrew Cooper wrote: >> On 10/04/2025 5:38 pm, Anthony PERARD wrote: >>> And by the way, you might want to add `gawk` as >>> well to replace busybox's awk, to make the linux build a bit more happy,

Re: [PATCH v2 2/7] arm/mpu: Provide access to the MPU region from the C code

2025-04-10 Thread Luca Fancellu
Hi Ayan, > On 8 Apr 2025, at 15:02, Ayan Kumar Halder wrote: > > Hi Luca, > >> >> +static void prepare_selector(uint8_t sel) >> +{ >> +/* >> + * {read,write}_protection_region works using the direct access to the >> 0..15 >> + * regions, so in order to save the isb() overhead, cha

Re: [PATCH v2 06/12] CI: consistently use DOCKER_CMD in makefiles

2025-04-10 Thread Andrew Cooper
On 07/04/2025 1:31 pm, Marek Marczykowski-Górecki wrote: > This allows rebuilding containers using podman too. > > Signed-off-by: Marek Marczykowski-Górecki > Reviewed-by: Stefano Stabellini > --- > automation/build/Makefile | 4 ++-- > automation/tests-artifacts/Makefile | 4 ++-- > 2

Re: [PATCH TEST-ARTEFACTS 0/8] Cleanup and Linux ARM64 support

2025-04-10 Thread Alejandro Vallejo
On Thu Apr 10, 2025 at 7:20 PM BST, Alejandro Vallejo wrote: > On Thu Apr 10, 2025 at 10:17 AM BST, Andrew Cooper wrote: >> On 10/04/2025 1:09 am, Jason Andryuk wrote: >>> On 2025-04-09 13:01, Andrew Cooper wrote: On 09/04/2025 5:36 pm, Andrew Cooper wrote: > Various bits of cleanup, and s

Re: [PATCH TEST-ARTEFACTS 0/8] Cleanup and Linux ARM64 support

2025-04-10 Thread Alejandro Vallejo
On Thu Apr 10, 2025 at 10:17 AM BST, Andrew Cooper wrote: > On 10/04/2025 1:09 am, Jason Andryuk wrote: >> On 2025-04-09 13:01, Andrew Cooper wrote: >>> On 09/04/2025 5:36 pm, Andrew Cooper wrote: Various bits of cleanup, and support for arm64 Linux builds. Run using the new Linux 6.

  1   2   >