On 6/18/25 5:46 PM, Jan Beulich wrote:
On 10.06.2025 15:05, Oleksii Kurochko wrote:
Implementation is based on Arm code with some minor changes:
- Re-define INVALID_VMID.
- Re-define MAX_VMID.
- Add TLB flushing when VMID is re-used.
Also, as a part of this path structure p2m_domain is i
On 20.06.2025 13:11, Roger Pau Monne wrote:
> When not using CONFIG_BIGMEM there are some restrictions in the address
> width for allocations of the domain structure, as it's PDX truncated to 32
> bits it's stashed into page_info structure for domain allocated pages.
>
> The current logic to calcu
On Tue, Jun 24, 2025 at 4:21 PM Christian Lindig
wrote:
>
> I believe what you observe is a major source of inefficiency for the reason
> you describe: changes are acted upon too early because there is no way to
> observe that they are part of a transaction. So now heuristics come in like
> wai
On Tue, Jun 24, 2025 at 4:51 PM Christian Lindig
wrote:
>
>
>
> > On 24 Jun 2025, at 16:43, Andriy Sultanov wrote:
> >
> > But I think this only covers part of the problem. Looking at the concrete
> > case of
> > xenopsd and Windows VMs, xenopsd cares about most of the nodes it receives
> > watc
Hello,
Le 24/06/2025 à 16:53, Andriy Sultanov a écrit :
> Currently, as far as I am aware, the ability of xenstore clients to
> properly
> handle and detect batch updates is somewhat lacking. Transactions are not
> directly visible to the clients watching a particular directory - they will
> recei
On 2025-06-16 09:01, Jan Beulich wrote:
Stop the compiler from inlining non-trivial memset() and memcpy() (for
memset() see e.g. map_vcpu_info() or kimage_load_segments() for
examples). This way we even keep the compiler from using REP STOSQ /
REP MOVSQ when we'd prefer REP STOSB / REP MOVSB (whe
On Tue, Jun 24, 2025 at 09:50:54AM +0200, Orzel, Michal wrote:
>
>
> On 24/06/2025 07:46, Jan Beulich wrote:
> > On 24.06.2025 05:55, dm...@proton.me wrote:
> >> From: Denis Mukhin
> >>
> >> Replace VPL011_{LOCK,UNLOCK} macros with raw spinlock calls to improve
> >> readability.
> >
> > I'm not
On Tue, Jun 24, 2025 at 04:12:46PM +0100, Frediano Ziglio wrote:
> On Tue, Jun 24, 2025 at 3:35 PM Marek Marczykowski-Górecki
> wrote:
> >
> > On Tue, Jun 24, 2025 at 03:05:25PM +0100, Frediano Ziglio wrote:
> > > OT: the flow of read_file (specifically "what" handling) looks
> > > weird... can I
On 24/06/2025 05:55, dm...@proton.me wrote:
> From: Denis Mukhin
>
> Rename HWDOM_VUART to HAS_VUART_MMIO.
>
> This emulator emulates only one register and the use of the emulator is
> limited to early boot console in the guest OS.
>
> No functional change.
>
> Signed-off-by: Denis Mukhin
On Tue, Jun 24, 2025 at 03:27:08PM +0100, Frediano Ziglio wrote:
> A boot loader can load files from outside ESP.
> In these cases device could be not provided or path could
> be something not supported.
> In these cases allows to boot anyway, all information
> could be provided using UKI or using
On 24.06.2025 01:45, Stefano Stabellini wrote:
> Rule 21.16 is about the types of arguments allowed for memcpy.
Seeing the subject - is it memcmp(), memcpy(), or both? (Writing from
home, where I don't have the Misra spec to hand, and hence can't check
what coverage the rule has.)
Jan
The combination of GRUB2, EFI and UKI allows potentially more flexibility.
For instance is possible to load xen.efi from a no ESP partition leaving
a boot loader like GRUB2 taking care of the file loading.
This however requires some changes in Xen to be less restrictive.
Specifically for GRUB2 thes
On 6/24/25 12:44 PM, Jan Beulich wrote:
On 24.06.2025 11:46, Oleksii Kurochko wrote:
On 6/18/25 5:46 PM, Jan Beulich wrote:
On 10.06.2025 15:05, Oleksii Kurochko wrote:
--- /dev/null
+++ b/xen/arch/riscv/p2m.c
@@ -0,0 +1,115 @@
+#include
+#include
+#include
+#include
+#include
+
+#includ
On 20.06.2025 13:11, Roger Pau Monne wrote:
> Introduce a command line option to allow disabling PDX compression. The
> disabling is done by turning pfn_pdx_add_region() into a no-op, so when
> attempting to initialize the selected compression algorithm the array of
> ranges to compress is empty.
On 20.06.2025 13:11, Roger Pau Monne wrote:
> --- /dev/null
> +++ b/xen/arch/x86/include/asm/pdx.h
> @@ -0,0 +1,75 @@
> +/* SPDX-License-Identifier: GPL-2.0-only */
> +
> +#ifndef X86_PDX_H
> +#define X86_PDX_H
> +
> +#ifndef CONFIG_PDX_NONE
> +
> +#include
> +
> +/*
> + * Introduce a macro to avo
On 20.06.2025 13:11, Roger Pau Monne wrote:
> Move fill_mask(), pdx_region_mask() and pdx_init_mask() to the
> !CONFIG_PDX_NONE section in preparation of them also being used by a newly
> added PDX compression.
>
> No functional change intended.
>
> Signed-off-by: Roger Pau Monné
Acked-by: Jan
On 20.06.2025 13:11, Roger Pau Monne wrote:
> With the appearance of Intel Sierra Forest and Granite Rapids it's now
> possible to get a production x86 host with the following memory map:
>
> SRAT: Node 0 PXM 0 [, 7fff]
> SRAT: Node 0 PXM 0 [0001, 00807f
On 24.06.2025 16:35, Marek Marczykowski-Górecki wrote:
> On Tue, Jun 24, 2025 at 03:05:25PM +0100, Frediano Ziglio wrote:
>> OT: the flow of read_file (specifically "what" handling) looks
>> weird... can I change it?
>
> It tries to avoid duplication of PrintErr calls, while including what
> faile
On Tue, Jun 24, 2025 at 03:05:25PM +0100, Frediano Ziglio wrote:
> OT: the flow of read_file (specifically "what" handling) looks
> weird... can I change it?
It tries to avoid duplication of PrintErr calls, while including what
failed in the error message. Looks like somebody really wanted to avoi
On 2025-06-16 09:01, Jan Beulich wrote:
The present clear_page_sse2() is useful in case a page isn't going to
get touched again soon, or if we want to limit churn on the caches.
Amend it by alternatively using CLZERO, which has been found to be quite
a bit faster on Zen2 hardware at least. Note t
The "buffer" macro is not defined.
Fixes: 59e087bf6a9c ("xen/keyhandler: Drop keyhandler_scratch")
Signed-off-by: Frediano Ziglio
Acked-by: Marek Marczykowski-Górecki
---
Changes since v1:
- fixed commit message;
- added Acked-by.
---
xen/common/efi/boot.c | 1 -
1 file changed, 1 deletion(-)
On Tue, Jun 24, 2025 at 12:11:10PM +0200, Orzel, Michal wrote:
>
>
> On 24/06/2025 05:55, dm...@proton.me wrote:
> > From: Denis Mukhin
> >
> > Use generic names prefixed with 'vuart_' in public PL011 emulator data
> > structures and functions.
> >
> > No functional change intended.
> >
> > Sign
On 6/18/25 5:15 PM, Jan Beulich wrote:
On 10.06.2025 15:05, Oleksii Kurochko wrote:
Instruct the remote harts to execute one or more HFENCE.GVMA instructions,
covering the range of guest physical addresses between start_addr and
start_addr + size for all the guests.
Here and in the code commen
On Tue, Jun 24, 2025 at 09:31:55AM +0100, Frediano Ziglio wrote:
> Allows to load Xen using "linux" and "initrd" GRUB2 commands.
> This can be used with UKI to separate initrd in a different module
> instead of bundling all together.
> Bundling all together can be a problem with Secure Boot where
>
On 24.06.25 16:51, Andriy Sultanov wrote:
Currently, as far as I am aware, the ability of xenstore clients to properly
handle and detect batch updates is somewhat lacking. Transactions are not
directly visible to the clients watching a particular directory - they will
receive a lot of individual
On 24.06.2025 12:33, Oleksii Kurochko wrote:
> On 6/18/25 5:15 PM, Jan Beulich wrote:
>> On 10.06.2025 15:05, Oleksii Kurochko wrote:
>>> Instruct the remote harts to execute one or more HFENCE.GVMA instructions,
>>> covering the range of guest physical addresses between start_addr and
>>> start_ad
From: Denis Mukhin
Use generic names prefixed with 'vuart_' in public PL011 emulator data
structures and functions.
No functional change intended.
Signed-off-by: Denis Mukhin
---
xen/arch/arm/dom0less-build.c | 4 ++--
xen/arch/arm/domain.c | 3 ++-
xen/arch/arm/domctl.c
On 6/24/25 3:51 PM, Andriy Sultanov wrote:
Suggestion:
WATCH_EVENT's req_id and tx_id are currently 0. Could it be possible, for
example, to modify this such that watch events coming as a result of a
transaction commit (a "batch") have tx_id of the corresponding
transaction
and req_id of, say,
On Tue, Jun 24, 2025 at 3:35 PM Marek Marczykowski-Górecki
wrote:
>
> On Tue, Jun 24, 2025 at 03:05:25PM +0100, Frediano Ziglio wrote:
> > OT: the flow of read_file (specifically "what" handling) looks
> > weird... can I change it?
>
> It tries to avoid duplication of PrintErr calls, while includi
On Tue, Jun 24, 2025 at 4:01 PM Andriy Sultanov
wrote:
>
> On 6/24/25 3:51 PM, Andriy Sultanov wrote:
>
> > Suggestion:
> > WATCH_EVENT's req_id and tx_id are currently 0. Could it be possible, for
> > example, to modify this such that watch events coming as a result of a
> > transaction commit (a
On 6/24/25 4:01 PM, Jan Beulich wrote:
+sbi_remote_hfence_gvma_vmid(d->dirty_cpumask, 0, 0, p2m->vmid);
You're creating d; it cannot possibly have run on any CPU yet. IOW
d->dirty_cpumask will be reliably empty here. I think it would be hard to
avoid issuing the flush to all CPUs here i
On Tue, Jun 24, 2025 at 09:31:54AM +0100, Frediano Ziglio wrote:
> A boot loader can load files from outside ESP.
> In these cases device could be not provided or path could
> be something not supported.
> In these cases allows to boot anyway, all information
> could be provided using UKI or using
On 24.06.2025 15:47, Oleksii Kurochko wrote:
> On 6/24/25 12:44 PM, Jan Beulich wrote:
>> On 24.06.2025 11:46, Oleksii Kurochko wrote:
>>> On 6/18/25 5:46 PM, Jan Beulich wrote:
On 10.06.2025 15:05, Oleksii Kurochko wrote:
> --- /dev/null
> +++ b/xen/arch/riscv/p2m.c
> @@ -0,0 +1,1
Commit 48d32458bcd4 ("x86, idle: add barriers to CLFLUSH workaround") was
inherited from Linux and added MFENCEs around the AAI65 errata fix.
The SDM now states:
Executions of the CLFLUSH instruction are ordered with respect to each
other and with respect to writes, locked read-modify-write i
Assorted fixes for mwait/monitor that I've had sitting around too long.
https://gitlab.com/xen-project/hardware/xen-staging/-/pipelines/1887059030
Andrew Cooper (4):
x86/idle: Move monitor()/mwait() wrappers into cpu-idle.c
x86/idle: Remove MFENCEs for CLFLUSH_MONITOR
Revert part of "x86/mw
Sort includes, and drop trailing whitespace.
No functional change.
Signed-off-by: Andrew Cooper
---
CC: Jan Beulich
CC: Roger Pau Monné
---
xen/arch/x86/acpi/cpu_idle.c | 51 ++--
1 file changed, 25 insertions(+), 26 deletions(-)
diff --git a/xen/arch/x86/acpi
They're not used by any other translation unit, so shouldn't live in
asm/processor.h, which is included almost everywhere.
Our new toolchain baseline knows the MONITOR/MWAIT instructions, so use them
directly rather than using raw hex.
Change the hint/extention parameters from long to int. They'
I believe what you observe is a major source of inefficiency for the reason you
describe: changes are acted upon too early because there is no way to observe
that they are part of a transaction. So now heuristics come in like waiting for
more changes before acting on the ones observed. I wonder
On Tue, Jun 24, 2025 at 09:49:39AM +0200, Orzel, Michal wrote:
>
>
> On 24/06/2025 05:55, dm...@proton.me wrote:
> > From: Denis Mukhin
> >
> > Move vpl011 DT node parsing from common Arm code to PL011 emulator code.
> It's not parsing, it's DT node generation.
Oh, that's right, overlooked.
Tha
On Tue, Jun 24, 2025 at 12:57 PM Andrew Cooper
wrote:
>
> On 24/06/2025 12:45 pm, Frediano Ziglio wrote:
> > The "buffer" macro is not defined.
> >
> > Fixes: 59e087bf6a9c ("xen/keyhandler: Drop keyhandler_scratch")
> > Signed-off-by: Frediano Ziglio
> > Acked-by: Marek Marczykowski-Górecki
>
>
On Tue, Jun 24, 2025 at 1:38 PM Marek Marczykowski-Górecki
wrote:
>
> On Tue, Jun 24, 2025 at 09:31:54AM +0100, Frediano Ziglio wrote:
> > A boot loader can load files from outside ESP.
> > In these cases device could be not provided or path could
> > be something not supported.
> > In these cases
Currently, as far as I am aware, the ability of xenstore clients to properly
handle and detect batch updates is somewhat lacking. Transactions are not
directly visible to the clients watching a particular directory - they will
receive a lot of individual watch_event's once the transaction is commi
On 6/24/25 4:06 PM, Jürgen Groß wrote:
The main reason for the large number of watch events after a
transaction is
the fact that the watch for e.g. detecting the addition of a new block
device
will be set on a node being common for all potential block devices
handled
by the watcher. This resul
> On 24 Jun 2025, at 16:43, Andriy Sultanov wrote:
>
> But I think this only covers part of the problem. Looking at the concrete
> case of
> xenopsd and Windows VMs, xenopsd cares about most of the nodes it receives
> watch events from, the issue is that it doesn't know when these are grouped
On Thu, 22 May 2025 16:52:10 -0700, Sean Christopherson wrote:
> Non-KVM folks,
>
> I am hoping to route this through the KVM tree (6.17 or later), as the non-KVM
> changes should be glorified nops. Please holler if you object to that idea.
>
> Hyper-V folks in particular, let me know if you wan
On 24.06.2025 10:12, Chen, Jiqian wrote:
> On 2025/6/20 14:34, Jan Beulich wrote:
>> On 19.06.2025 08:14, Chen, Jiqian wrote:
>>> On 2025/6/18 22:33, Jan Beulich wrote:
On 12.06.2025 11:29, Jiqian Chen wrote:
> --- a/xen/include/xen/vpci.h
> +++ b/xen/include/xen/vpci.h
> @@ -13,11
On 24/06/2025 05:55, dm...@proton.me wrote:
> From: Denis Mukhin
>
> Move vpl011 DT node parsing from common Arm code to PL011 emulator code.
It's not parsing, it's DT node generation.
We usually keep all the DT node generation functions in one place. I'm not sure
if we want to move them to
On 24.06.2025 10:32, Chen, Jiqian wrote:
> On 2025/6/24 16:28, Jan Beulich wrote:
>> On 24.06.2025 10:26, Chen, Jiqian wrote:
>>> On 2025/6/24 16:17, Jan Beulich wrote:
On 24.06.2025 10:12, Chen, Jiqian wrote:
> On 2025/6/20 14:34, Jan Beulich wrote:
>> On 19.06.2025 08:14, Chen, Jiqia
On 2025/6/24 18:08, Jan Beulich wrote:
> On 24.06.2025 11:29, Chen, Jiqian wrote:
>> On 2025/6/24 16:05, Jan Beulich wrote:
>>> On 24.06.2025 10:02, Chen, Jiqian wrote:
On 2025/6/20 14:38, Jan Beulich wrote:
> On 19.06.2025 08:39, Chen, Jiqian wrote:
>> On 2025/6/18 22:05, Jan Beulich
On 24/06/2025 23:46, dm...@proton.me wrote:
> On Tue, Jun 24, 2025 at 09:50:54AM +0200, Orzel, Michal wrote:
>>
>>
>> On 24/06/2025 07:46, Jan Beulich wrote:
>>> On 24.06.2025 05:55, dm...@proton.me wrote:
From: Denis Mukhin
Replace VPL011_{LOCK,UNLOCK} macros with raw spinlock c
From: Denis Mukhin
Introduce domain_console for grouping data structures used for integrating
domain's diagnostic console with Xen's console driver.
Group all pbuf-related data structures under domain_console. Rename the moved
fields to plain .buf, .idx and .lock names, since all uses of the fi
On Tue, Jun 24, 2025 at 09:40:02AM +0200, Jan Beulich wrote:
> On 24.06.2025 09:36, dm...@proton.me wrote:
> > On Tue, Jun 24, 2025 at 07:53:04AM +0200, Jan Beulich wrote:
> >> On 24.06.2025 05:57, dm...@proton.me wrote:
> >>> --- a/xen/drivers/vuart/Kconfig
> >>> +++ b/xen/drivers/vuart/Kconfig
>
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
Reviewed-by: Andre Przywara
Acke
On 24.06.2025 11:46, Oleksii Kurochko wrote:
> On 6/18/25 5:46 PM, Jan Beulich wrote:
>> On 10.06.2025 15:05, Oleksii Kurochko wrote:
>>> --- /dev/null
>>> +++ b/xen/arch/riscv/p2m.c
>>> @@ -0,0 +1,115 @@
>>> +#include
>>> +#include
>>> +#include
>>> +#include
>>> +#include
>>> +
>>> +#include
On 24.06.2025 11:49, Chen, Jiqian wrote:
> On 2025/6/18 22:45, Jan Beulich wrote:
>> On 12.06.2025 11:29, Jiqian Chen wrote:
>>> --- a/xen/drivers/vpci/msi.c
>>> +++ b/xen/drivers/vpci/msi.c
>>> @@ -193,6 +193,33 @@ static void cf_check mask_write(
>>> msi->mask = val;
>>> }
>>>
>>> +static
Hi,
On 20/06/2025 10:49, Hari Limaye 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.
From: Luca Fancellu
Introduce flags_has_rwx() function that will check if
On 24.06.2025 10:42, Oleksii Moisieiev wrote:
> Adding Roger and Jan to the conversation.
>
> Please see below.
Why is this? I did answer that question at the bottom already.
Jan
> On 23/06/2025 00:30, Stefano Stabellini wrote:
>> On Thu, 19 Jun 2025, Oleksii Moisieiev wrote:
>>> On 18/06/2025
On 24.06.2025 00:51, Stefano Stabellini wrote:
> On Mon, 23 Jun 2025, Demi Marie Obenour wrote:
>> On 6/23/25 11:44, Jan Beulich wrote:
>>> On 21.06.2025 02:41, Stefano Stabellini wrote:
>>> Also a more fundamental question I was wondering about: If Control had
>>> full privilege, nothing else in t
On 20.06.2025 13:11, Roger Pau Monne wrote:
> --- a/xen/arch/arm/setup.c
> +++ b/xen/arch/arm/setup.c
> @@ -255,6 +255,10 @@ void __init init_pdx(void)
> {
> const struct membanks *mem = bootinfo_get_mem();
> paddr_t bank_start, bank_size, bank_end;
> +unsigned int bank;
> +
> +f
On 24.06.2025 02:20, victorm.l...@amd.com wrote:
> From: Nicola Vetrini
>
> Use {get,put}_unaligned_t to ensure that reads and writes are
> safe to perform even on potentially misaligned pointers.
Also applicable to the Arm patch: Please can such patches mention the
main subject of the rule, not
On 25.06.2025 00:54, dm...@proton.me wrote:
> On Tue, Jun 24, 2025 at 09:40:02AM +0200, Jan Beulich wrote:
>> On 24.06.2025 09:36, dm...@proton.me wrote:
>>> On Tue, Jun 24, 2025 at 07:53:04AM +0200, Jan Beulich wrote:
On 24.06.2025 05:57, dm...@proton.me wrote:
> --- a/xen/drivers/vuart/K
On 24.06.2025 22:14, Jason Andryuk wrote:
> On 2025-06-24 01:25, Jan Beulich wrote:
>> On 24.06.2025 00:51, Stefano Stabellini wrote:
>>> On Mon, 23 Jun 2025, Demi Marie Obenour wrote:
On 6/23/25 11:44, Jan Beulich wrote:
> On 21.06.2025 02:41, Stefano Stabellini wrote:
> Also a more f
On Mon, Jun 23, 2025 at 10:09:54AM +0200, Jan Beulich wrote:
> On 23.06.2025 03:16, dm...@proton.me wrote:
> > From: Denis Mukhin
> >
> > Fix potential memory leak in domain_create() in late hardware domain case.
> >
> > Fixes: b959f3b820f5 ("xen: introduce hardware domain create flag")
> > Signed
On 24.06.2025 05:55, dm...@proton.me wrote:
> From: Denis Mukhin
>
> Replace VPL011_{LOCK,UNLOCK} macros with raw spinlock calls to improve
> readability.
I'm not an Arm maintainer, so I have limited say here, but: How is this
improving readability? It better utilizes available local variables,
On Tue, Jun 24, 2025 at 08:13:08AM +0200, Orzel, Michal wrote:
>
>
> On 24/06/2025 05:55, dm...@proton.me wrote:
> > From: Denis Mukhin
> >
> > Rename CONFIG_SBSA_VUART_CONSOLE to CONFIG_HAS_VUART_PL011.
> Why? We emulate SBSA UART and not PL011. Despite the similarities (the former
> is
> a su
From: Volodymyr Babchuk
Implement suspend and resume callbacks for the SCIF UART driver,
enabled when CONFIG_SYSTEM_SUSPEND is set. This allows proper
handling of UART state across system suspend/resume cycles.
Tested on Renesas R-Car H3 Starter Kit.
Signed-off-by: Volodymyr Babchuk
Signed-off
On Tue, Jun 24, 2025 at 07:50:33AM +0200, Jan Beulich wrote:
> On 24.06.2025 05:56, dm...@proton.me wrote:
> > From: Denis Mukhin
> >
> > Move PL011 emulator to the new location for UART emulators.
> >
> > No functional change intended.
> >
> > Signed-off-by: Denis Mukhin
> > ---
> > xen/arch/ar
The "buffer" macro is not defined.
It was defined till commit 59e087bf6a9c8ad805294c32dfd7c77068d81eff
(cfr "xen/keyhandler: Drop keyhandler_scratch").
Signed-off-by: Frediano Ziglio
---
xen/common/efi/boot.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/xen/common/efi/boot.c b/xen/common/e
On 2025/6/20 14:29, Jan Beulich wrote:
> On 19.06.2025 04:29, Chen, Jiqian wrote:
>> On 2025/6/18 21:52, Jan Beulich wrote:
>>> On 12.06.2025 11:29, Jiqian Chen wrote:
--- a/xen/drivers/vpci/header.c
+++ b/xen/drivers/vpci/header.c
@@ -836,6 +836,42 @@ static int vpci_init_capability
On Tue, Jun 24, 2025 at 08:37:22AM +0200, Orzel, Michal wrote:
>
>
> On 24/06/2025 05:55, dm...@proton.me wrote:
> > From: Denis Mukhin
> >
> > Rename HWDOM_VUART to HAS_VUART_MMIO.
> >
> > This emulator emulates only one register and the use of the emulator is
> > limited to early boot console
On 24.06.2025 05:57, dm...@proton.me wrote:
> --- a/xen/drivers/vuart/Kconfig
> +++ b/xen/drivers/vuart/Kconfig
> @@ -3,6 +3,15 @@ config HAS_VUART
>
> if (ARM_32 || ARM_64)
>
> +config HAS_VUART_MMIO
> + bool "Simple MMIO-based emulated UART support"
Perhaps in a separate change this sho
From: Denis Mukhin
Simplify code a bit since there's only one user of vuart_print_char().
No functional change.
Signed-off-by: Denis Mukhin
---
xen/arch/arm/vuart.c | 45 ++--
1 file changed, 18 insertions(+), 27 deletions(-)
diff --git a/xen/arch/arm
On 24/06/2025 07:46, Jan Beulich wrote:
> On 24.06.2025 05:55, dm...@proton.me wrote:
>> From: Denis Mukhin
>>
>> Replace VPL011_{LOCK,UNLOCK} macros with raw spinlock calls to improve
>> readability.
>
> I'm not an Arm maintainer, so I have limited say here, but: How is this
> improving read
On 23.06.2025 22:20, dm...@proton.me wrote:
> From: Denis Mukhin
>
> Introduce domain_console for grouping data structures used for integrating
> domain's diagnostic console with Xen's console driver.
>
> Group all pbuf-related data structures under domain_console. Rename the moved
> fields to p
On 23.06.2025 22:18, Andrew Cooper wrote:
> On 16/06/2025 7:27 am, Jan Beulich wrote:
>> To expand on my earlier suggestion (ab)using the "efi" global: With
>> the linker script having this
>>
>> #ifdef EFI
>> .reloc ALIGN(4) : {
>> __base_relocs_start = .;
>> *(.reloc)
>> __base_relo
On 2025/6/20 14:38, Jan Beulich wrote:
> On 19.06.2025 08:39, Chen, Jiqian wrote:
>> On 2025/6/18 22:05, Jan Beulich wrote:
>>> On 12.06.2025 11:29, Jiqian Chen wrote:
--- a/xen/drivers/vpci/msix.c
+++ b/xen/drivers/vpci/msix.c
@@ -703,9 +703,13 @@ static int cf_check init_msix(struc
On 24.06.2025 10:02, Chen, Jiqian wrote:
> On 2025/6/20 14:38, Jan Beulich wrote:
>> On 19.06.2025 08:39, Chen, Jiqian wrote:
>>> On 2025/6/18 22:05, Jan Beulich wrote:
On 12.06.2025 11:29, Jiqian Chen wrote:
> --- a/xen/drivers/vpci/msix.c
> +++ b/xen/drivers/vpci/msix.c
> @@ -703
On 2025/6/20 14:34, Jan Beulich wrote:
> On 19.06.2025 08:14, Chen, Jiqian wrote:
>> On 2025/6/18 22:33, Jan Beulich wrote:
>>> On 12.06.2025 11:29, Jiqian Chen wrote:
--- a/xen/include/xen/vpci.h
+++ b/xen/include/xen/vpci.h
@@ -13,11 +13,12 @@ typedef uint32_t vpci_read_t(const str
On 2025/6/24 16:44, Jan Beulich wrote:
> On 24.06.2025 10:32, Chen, Jiqian wrote:
>> On 2025/6/24 16:28, Jan Beulich wrote:
>>> On 24.06.2025 10:26, Chen, Jiqian wrote:
On 2025/6/24 16:17, Jan Beulich wrote:
> On 24.06.2025 10:12, Chen, Jiqian wrote:
>> On 2025/6/20 14:34, Jan Beulich
On 2025/6/24 16:17, Jan Beulich wrote:
> On 24.06.2025 10:12, Chen, Jiqian wrote:
>> On 2025/6/20 14:34, Jan Beulich wrote:
>>> On 19.06.2025 08:14, Chen, Jiqian wrote:
On 2025/6/18 22:33, Jan Beulich wrote:
> On 12.06.2025 11:29, Jiqian Chen wrote:
>> --- a/xen/include/xen/vpci.h
On 24.06.2025 10:29, Mykola Kvach wrote:
> On Tue, Jun 24, 2025 at 10:53 AM Jan Beulich wrote:
>> On 24.06.2025 09:18, Mykola Kvach wrote:
>>> @@ -281,6 +313,10 @@ static struct uart_driver __read_mostly
>>> scif_uart_driver = {
>>> .start_tx = scif_uart_start_tx,
>>> .stop_tx
On 2025/6/24 16:28, Jan Beulich wrote:
> On 24.06.2025 10:26, Chen, Jiqian wrote:
>> On 2025/6/24 16:17, Jan Beulich wrote:
>>> On 24.06.2025 10:12, Chen, Jiqian wrote:
On 2025/6/20 14:34, Jan Beulich wrote:
> On 19.06.2025 08:14, Chen, Jiqian wrote:
>> On 2025/6/18 22:33, Jan Beulich
From: Mykola Kvach
This patch adds CONFIG_SYSTEM_SUSPEND guards around freeze_domains
and thaw_domains functions.
This ensures they are only compiled into the hypervisor when the system
suspend functionality is enabled, aligning their inclusion with their
specific use case.
This addresses two M
On 24.06.2025 10:26, Chen, Jiqian wrote:
> On 2025/6/24 16:17, Jan Beulich wrote:
>> On 24.06.2025 10:12, Chen, Jiqian wrote:
>>> On 2025/6/20 14:34, Jan Beulich wrote:
On 19.06.2025 08:14, Chen, Jiqian wrote:
> On 2025/6/18 22:33, Jan Beulich wrote:
>> On 12.06.2025 11:29, Jiqian Chen
Hi Jan,
On Tue, Jun 24, 2025 at 10:53 AM Jan Beulich wrote:
>
> On 24.06.2025 09:18, Mykola Kvach wrote:
> > @@ -281,6 +313,10 @@ static struct uart_driver __read_mostly
> > scif_uart_driver = {
> > .start_tx = scif_uart_start_tx,
> > .stop_tx = scif_uart_stop_tx,
> > .vu
On Tue, Jun 24, 2025 at 7:36 AM Frediano Ziglio
wrote:
>
Typo on title "Do not undefined not defined macro" -> "Do not undefine
not defined macro"
> The "buffer" macro is not defined.
> It was defined till commit 59e087bf6a9c8ad805294c32dfd7c77068d81eff
> (cfr "xen/keyhandler: Drop keyhandler_sc
Adding Roger and Jan to the conversation.
Please see below.
On 23/06/2025 00:30, Stefano Stabellini wrote:
> On Thu, 19 Jun 2025, Oleksii Moisieiev wrote:
>> On 18/06/2025 03:04, Stefano Stabellini wrote:
>>> On Thu, 12 Jun 2025, Oleksii Moisieiev wrote:
Hi Stefano,
I'm very sorry
On 24/06/2025 11:47, Jan Beulich wrote:
> On 24.06.2025 10:42, Oleksii Moisieiev wrote:
>> Adding Roger and Jan to the conversation.
>>
>> Please see below.
> Why is this? I did answer that question at the bottom already.
>
> Jan
Oh sorry, missed this email..
>> On 23/06/2025 00:30, Stefano Stabel
On 2025/6/18 22:45, Jan Beulich wrote:
> On 12.06.2025 11:29, Jiqian Chen wrote:
>> --- a/xen/drivers/vpci/msi.c
>> +++ b/xen/drivers/vpci/msi.c
>> @@ -193,6 +193,33 @@ static void cf_check mask_write(
>> msi->mask = val;
>> }
>>
>> +static int cf_check cleanup_msi(struct pci_dev *pdev)
>>
On Tue, Jun 24, 2025 at 11:32 AM Jan Beulich wrote:
>
> On 24.06.2025 10:29, Mykola Kvach wrote:
> > On Tue, Jun 24, 2025 at 10:53 AM Jan Beulich wrote:
> >> On 24.06.2025 09:18, Mykola Kvach wrote:
> >>> @@ -281,6 +313,10 @@ static struct uart_driver __read_mostly
> >>> scif_uart_driver = {
> >
On 2025/6/24 16:05, Jan Beulich wrote:
> On 24.06.2025 10:02, Chen, Jiqian wrote:
>> On 2025/6/20 14:38, Jan Beulich wrote:
>>> On 19.06.2025 08:39, Chen, Jiqian wrote:
On 2025/6/18 22:05, Jan Beulich wrote:
> On 12.06.2025 11:29, Jiqian Chen wrote:
>> --- a/xen/drivers/vpci/msix.c
>>>
On 24.06.2025 11:29, Chen, Jiqian wrote:
> On 2025/6/24 16:05, Jan Beulich wrote:
>> On 24.06.2025 10:02, Chen, Jiqian wrote:
>>> On 2025/6/20 14:38, Jan Beulich wrote:
On 19.06.2025 08:39, Chen, Jiqian wrote:
> On 2025/6/18 22:05, Jan Beulich wrote:
>> On 12.06.2025 11:29, Jiqian Chen
From: Mykola Kvach
This patch wraps the suspend/resume console callbacks and related code within
CONFIG_SYSTEM_SUSPEND blocks. This ensures that these functions and their
calls are only included in the build when CONFIG_SYSTEM_SUSPEND is enabled.
This addresses Misra Rule 2.1 violations.
Sugges
Allows to load Xen using "linux" and "initrd" GRUB2 commands.
This can be used with UKI to separate initrd in a different module
instead of bundling all together.
Bundling all together can be a problem with Secure Boot where
we need to sign the bundle making harder to change it.
As initrd content d
On 24/06/2025 05:55, dm...@proton.me wrote:
> From: Denis Mukhin
>
> Use generic names prefixed with 'vuart_' in public PL011 emulator data
> structures and functions.
>
> No functional change intended.
>
> Signed-off-by: Denis Mukhin
> ---
> xen/arch/arm/dom0less-build.c | 4 ++--
>
On 24.06.2025 09:18, Mykola Kvach wrote:
> @@ -281,6 +313,10 @@ static struct uart_driver __read_mostly scif_uart_driver
> = {
> .start_tx = scif_uart_start_tx,
> .stop_tx = scif_uart_stop_tx,
> .vuart_info = scif_vuart_info,
> +#ifdef CONFIG_SYSTEM_SUSPEND
> +.suspen
A boot loader can load files from outside ESP.
In these cases device could be not provided or path could
be something not supported.
In these cases allows to boot anyway, all information
could be provided using UKI or using other boot loader
features.
Signed-off-by: Frediano Ziglio
---
xen/commo
97 matches
Mail list logo