On Tue, 2024-10-15 at 14:32 +0200, Jan Beulich wrote:
> On 15.10.2024 11:11, oleksii.kuroc...@gmail.com wrote:
> > On Tue, 2024-10-15 at 08:33 +0200, Jan Beulich wrote:
> > > On 10.10.2024 17:30, Oleksii Kurochko wrote:
> > > > --- a/xen/arch/riscv/setup.c
> > > > +++ b/xen/arch/riscv/setup.c
> > >
On 16.10.2024 12:34, Alejandro Vallejo wrote:
> On Wed Oct 16, 2024 at 8:46 AM BST, Jan Beulich wrote:
>> --- a/tools/tests/x86_emulator/evex-disp8.c
>> +++ b/tools/tests/x86_emulator/evex-disp8.c
>> @@ -911,10 +911,8 @@ static void test_one(const struct test *
>> n = test->scale == SC_vl ? vs
flight 188114 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/188114/
Perfect :-)
All tests in this flight passed as required
version targeted for testing:
ovmf d44b82270b6b1589cc2850fde24a3686cb5ef779
baseline version:
ovmf a232e0cd2fe2bb882a5dd
For some reason I entirely consistently screwed these up.
Signed-off-by: Jan Beulich
--- a/tools/tests/x86_emulator/evex-disp8.c
+++ b/tools/tests/x86_emulator/evex-disp8.c
@@ -905,8 +905,8 @@ static void test_one(const struct test *
goto fail;
for ( i = 0; i < (test->scale == SC
Another leftover from the dropping of Xeon Phi support.
Signed-off-by: Jan Beulich
---
Note: I'm deliberately not switching to use of the conditional operator,
as the form as is resulting now is what we'll want for APX (which is
where I noticed this small piece of dead logic).
--- a/tools/tests/
On Wed Oct 16, 2024 at 8:46 AM BST, Jan Beulich wrote:
> Another leftover from the dropping of Xeon Phi support.
>
> Signed-off-by: Jan Beulich
> ---
> Note: I'm deliberately not switching to use of the conditional operator,
> as the form as is resulting now is what we'll want for APX (which is
>
Rename the helper to better reflect its function.
Signed-off-by: Kirill A. Shutemov
Suggested-by: Dave Hansen
---
arch/x86/hyperv/ivm.c | 2 +-
arch/x86/include/asm/mtrr.h| 10 +-
arch/x86/kernel/cpu/mtrr/generic.c | 6 +++---
arch/x86/kernel/cpu/mtrr/mtrr.c|
On Wed Oct 16, 2024 at 10:15 AM BST, Oleksii Kurochko wrote:
> Implement the `maddr_to_virt()` function to convert a machine address
> to a virtual address. This function is specifically designed to be used
> only for the DIRECTMAP region, so a check has been added to ensure that
> the address does
On 16.10.2024 08:57, Philipp Stanner wrote:
> On Tue, 2024-10-15 at 13:53 -0600, Alex Williamson wrote:
>> On Tue, 15 Oct 2024 20:51:23 +0200
>> Philipp Stanner wrote:
>>
>>> pci_intx() and its managed counterpart pcim_intx() only exist for
>>> older
>>> drivers which have not been ported yet for
On Wed, Oct 16, 2024 at 09:33:32AM +0100, Frediano Ziglio wrote:
> On Tue, Oct 15, 2024 at 2:51 PM Anthony PERARD
> wrote:
> > On Mon, Oct 14, 2024 at 05:32:26PM +0100, Frediano Ziglio wrote:
> > > On Mon, Oct 14, 2024 at 4:31 PM Anthony PERARD
> > > wrote:
> > > > On Mon, Oct 14, 2024 at 09:53
This serie contains various patches to rework how firmware discovery and
feature detection is done and allow to have a more fine granular
filtering of the calls we do or not to the firmware.
There is also a patch introducing the use of the "bit 15" convention
from the FF-A specification to disting
Move the direct message handling function in its own source file and
rename it to have a ffa_ prefix.
This is a preparation to add support for indirect messages which will
go into this newly created source file.
Signed-off-by: Bertrand Marquis
---
Changes in v2:
- rebase
---
xen/arch/arm/tee/Mak
Rework the partition info get implementation to use the correct size of
structure depending on the version of the protocol and simplifies the
structure copy to use only memcpy and prevent recreating the structure
each time.
The goal here is to have an implementation that will be easier to
maintain
Rework firmware discovery during probe:
- move prints into the probe
- rename ffa_version to ffa_fw_version as the variable identifies the
version of the firmware and not the one we support
- add error prints when allocation fail during probe
No functional changes.
Signed-off-by: Bertrand Marqu
Fix FFA version negotiation with the firmware to follow the
specification guidance more closely (see FF-A Specification Version 1.1
in chapter 13.2.1).
When the firmware returns OK we can have several cases:
- the version requested is accepted but the firmware supports a greater
one in the same m
Make use and required to have bit 15 convention respected by secure
world (having bit 15 of IDs set for secure endpoints and non-set for
non-secure ones).
If any secure partition has an ID with bit 15 not set, it will not be
possible to contact or detect them.
Print an error log during probe for ea
Add support for FFA_MSG_SEND2 to send indirect messages from a VM to a
secure partition.
Signed-off-by: Bertrand Marquis
---
Changes in v2:
- rebase
---
xen/arch/arm/tee/ffa.c | 5
xen/arch/arm/tee/ffa_msg.c | 49 ++
xen/arch/arm/tee/ffa_private.
Store the list of ABI we need in a list and go through the list instead
of having a list of conditions inside the code.
No functional change.
Signed-off-by: Bertrand Marquis
---
Changes in v2:
- Store a string version of ABI needed from firmware and print the name
of the ABI not supported inst
Create a bitmap to store which feature is supported or not by the
firmware and use it to filter which calls are done to the firmware.
While there reoder ABI definition by numbers to easily find the min and
max ones.
Signed-off-by: Bertrand Marquis
---
Changes in v2:
- rename fw_feat to abi and m
Remove the per VM flag to store if notifications are enabled or not as
the only case where they are not, if notifications are enabled globally,
will make the VM creation fail.
Also use the opportunity to always give the notifications interrupts IDs
to VM. If the firmware does not support notificati
When an RXTX buffer is mapped by a VM transmit it to the SPMC when it
supports RX_ACQUIRE.
As a consequence of that, we must acquire the RX buffer of a VM from the
SPMC when we want to use it:
- create a generic acquire and release function to get the rx buffer of
a VM which gets it from the SPMC
On Tue, Oct 15, 2024 at 2:51 PM Anthony PERARD
wrote:
>
> On Mon, Oct 14, 2024 at 05:32:26PM +0100, Frediano Ziglio wrote:
> > On Mon, Oct 14, 2024 at 4:31 PM Anthony PERARD
> > wrote:
> > >
> > > On Mon, Oct 14, 2024 at 09:53:28AM +0100, Frediano Ziglio wrote:
> > > > diff --git a/xen/arch/x86/b
flight 188098 linux-linus real [real]
flight 188113 linux-linus real-retest [real]
http://logs.test-lab.xenproject.org/osstest/logs/188098/
http://logs.test-lab.xenproject.org/osstest/logs/188113/
Failures :-/ but no regressions.
Tests which are failing intermittently (not blocking):
test-amd64-
On 11.10.2024 17:27, Stewart Hildebrand wrote:
> Add links between a VF's struct pci_dev and its associated PF struct
> pci_dev. Move the calls to pci_get_pdev()/pci_add_device() down to avoid
> dropping and re-acquiring the pcidevs_lock().
>
> During PF removal, unlink VF from PF and mark the VF
On 11.10.2024 17:27, Stewart Hildebrand wrote:
> In commit 4f78438b45e2 ("vpci: use per-domain PCI lock to protect vpci
> structure") a lock was moved from allocate_and_map_msi_pirq() to the
> caller and changed from pcidevs_lock() to read_lock(&d->pci_lock).
> However, one call path wasn't updated
On Wed Oct 16, 2024 at 8:45 AM BST, Jan Beulich wrote:
> For some reason I entirely consistently screwed these up.
>
> Signed-off-by: Jan Beulich
Reviewed-by: Alejandro Vallejo
We should really give another push to the clang-format effort. This whole class
of mistakes would be a thing of the
On 10.06.2024 19:10, Petr Beneš wrote:
> Petr Beneš (9):
> tools/ocaml: Fix mixed tabs/spaces
> tools/ocaml: Add missing ocaml bindings for altp2m_opts
> xen: Refactor altp2m options into a structured format
> tools/xl: Add altp2m_count parameter
> docs/man: Add altp2m_count parameter to
On 16/10/2024 8:50 am, oleksii.kuroc...@gmail.com wrote:
> On Tue, 2024-10-15 at 14:32 +0200, Jan Beulich wrote:
>> On 15.10.2024 11:11, oleksii.kuroc...@gmail.com wrote:
>>> On Tue, 2024-10-15 at 08:33 +0200, Jan Beulich wrote:
On 10.10.2024 17:30, Oleksii Kurochko wrote:
> --- a/xen/arch
On 16/10/2024 8:46 am, Jan Beulich wrote:
> Another leftover from the dropping of Xeon Phi support.
>
> Signed-off-by: Jan Beulich
Acked-by: Andrew Cooper
On 16.10.2024 12:06, Alejandro Vallejo wrote:
> On Wed Oct 16, 2024 at 8:45 AM BST, Jan Beulich wrote:
>> For some reason I entirely consistently screwed these up.
>>
>> Signed-off-by: Jan Beulich
>
> Reviewed-by: Alejandro Vallejo
Thanks.
> We should really give another push to the clang-fo
On 16/10/2024 10:21 am, Frediano Ziglio wrote:
> diff --git a/xen/arch/x86/boot/Makefile b/xen/arch/x86/boot/Makefile
> index c665a9b420..c57c9f83d3 100644
> --- a/xen/arch/x86/boot/Makefile
> +++ b/xen/arch/x86/boot/Makefile
> @@ -1,12 +1,16 @@
> obj-bin-y += head.o
> obj-bin-y += built-in-32.o
Add support for indirect messages between VMs.
This is only enabled if CONFIG_FFA_VM_TO_VM is selected.
Signed-off-by: Bertrand Marquis
---
xen/arch/arm/tee/ffa_msg.c | 96 ++
1 file changed, 88 insertions(+), 8 deletions(-)
diff --git a/xen/arch/arm/tee/ffa_
On 09.10.2024 15:34, Jan Beulich wrote:
> On 09.10.2024 15:20, Andrew Cooper wrote:
>> On 09/10/2024 10:21 am, Jan Beulich wrote:
>>> In {big,little}_endian.h the changes are entirely mechanical, except for
>>> dealing with casting away of const from pointers-to-const on lines
>>> touched anyway.
>
Philipp Stanner writes:
> pci_intx() is a hybrid function which can sometimes be managed through
> devres. To remove this hybrid nature from pci_intx(), it is necessary to
> port users to either an always-managed or a never-managed version.
>
> qtnfmac enables its PCI-Device with pcim_enable_devi
On Wed, 2024-10-16 at 10:43 +0200, Heiner Kallweit wrote:
> On 16.10.2024 08:57, Philipp Stanner wrote:
> > On Tue, 2024-10-15 at 13:53 -0600, Alex Williamson wrote:
> > > On Tue, 15 Oct 2024 20:51:23 +0200
> > > Philipp Stanner wrote:
> > >
> > > > pci_intx() and its managed counterpart pcim_int
On 16.10.2024 09:50, oleksii.kuroc...@gmail.com wrote:
> On Tue, 2024-10-15 at 14:32 +0200, Jan Beulich wrote:
>> On 15.10.2024 11:11, oleksii.kuroc...@gmail.com wrote:
>>> On Tue, 2024-10-15 at 08:33 +0200, Jan Beulich wrote:
On 10.10.2024 17:30, Oleksii Kurochko wrote:
> --- a/xen/arch/r
Add a call to end_boot_allocator() in start_xen() to finalize the
boot memory allocator, moving free pages to the domain sub-allocator.
After initializing the memory subsystem, update `system_state` from
`SYS_STATE_early_boot` to `SYS_STATE_boot`, signifying the end of the
early boot phase.
Signe
To avoid the following linkage fail the stub for share_xen_page_with_guest()
is introduced:
riscv64-linux-gnu-ld: prelink.o: in function `tasklet_kill':
/build/xen/common/tasklet.c:176: undefined reference to
`share_xen_page_with_guest'
riscv64-linux-gnu-ld: ./.xen-syms.0: hidden symbol
`sh
Finish initializing the memory subsystem by mapping the direct map and
frame table.
In the case of RISC-V 64, which has a large virtual address space
(the minimum supported MMU mode is Sv39, providing TB of VA space),
the direct map and frame table are mapped starting from physical address
0. This
Call vm_init() to initialize the VMAP_DEFAULT virtual range.
To support this, introduce the populate_pt_range() and
arch_vmap_virt_end() functions, which are used by
vm_init()->vm_init_type().
Signed-off-by: Oleksii Kurochko
---
xen/arch/riscv/mm.c| 11 +--
xen/arch/riscv/pt.c|
Implement the `maddr_to_virt()` function to convert a machine address
to a virtual address. This function is specifically designed to be used
only for the DIRECTMAP region, so a check has been added to ensure that
the address does not exceed `DIRECTMAP_SIZE`.
Signed-off-by: Oleksii Kurochko
---
Introduce the implementation of setup_mm(), which includes:
1. Adding all free regions to the boot allocator, as memory is needed
to allocate page tables used for frame table mapping.
2. Calculating RAM size and the RAM end address.
3. Setting up direct map mappings from the PFN of physical addr
Move code from efi-boot.h to a separate, new, reloc-trampoline.c file.
Reuse this new code, compiling it for 32bit as well, to replace assembly
code in head.S doing the same thing.
Signed-off-by: Frediano Ziglio
Reviewed-by: Andrew Cooper
---
Changes since v3:
- fixed a typo in comment;
- added
Add support to raise a Rx buffer full notification to a VM.
This function will be used for indirect message support between VM and
is only activated if CONFIG_FFA_VM_TO_VM is selected.
Even if there are 32 framework notifications possible, right now only
one is defined so the implementation is sim
The current method to include 32 bit C boot code is:
- compile each function we want to use into a separate object file;
- each function is compiled with -fpic option;
- convert these object files to binary files. This operation removes GOP
which we don't want in the executable;
- a small assembl
No more need to pass from assembly code.
Signed-off-by: Frediano Ziglio
Reviewed-by: Andrew Cooper
---
Changes since v1:
- split the 2 variable changes into 2 commits.
Changes since v2:
- revert commit order.
---
xen/arch/x86/boot/build32.lds.S | 1 +
xen/arch/x86/boot/head.S| 10 +---
Signed-off-by: Frediano Ziglio
---
xen/arch/x86/boot/reloc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/xen/arch/x86/boot/reloc.c b/xen/arch/x86/boot/reloc.c
index e50e161b27..e725cfb6eb 100644
--- a/xen/arch/x86/boot/reloc.c
+++ b/xen/arch/x86/boot/reloc.c
@@ -65,7 +65,7
This series attempt to:
- use more C code, that is replace some assembly code with C;
- avoid some code duplication between C and assembly;
- prevent some issues having relocations in C code.
The idea is extending the current C to binary code conversion
done for 32 bit C code called from head.S ma
No more need to pass from assembly code.
Signed-off-by: Frediano Ziglio
Reviewed-by: Andrew Cooper
---
Changes since v1:
- split the 2 variable changes into 2 commits.
Changes since v2:
- revert commit order;
- avoid useless casts.
---
xen/arch/x86/boot/head.S | 6 +-
xen/arch/x86/boot/re
This RFC serie is a first proposal to enable VM to VM communications
through FF-A on Arm.
It is sent as an RFC because some aspects of this serie will need to be
discussed:
- How to control/configure which VM can communicate with which VM
- How to enable this without conflicting with Optee support
Create a CONFIG_FFA_VM_TO_VM parameter to activate FFA communication
between VMs.
When activated list VMs in the system with FF-A support in part_info_get.
WARNING: There is no filtering for now and all VMs are listed !!
Signed-off-by: Bertrand Marquis
---
xen/arch/arm/tee/Kconfig| 11
When VM to VM support is activated and there is no suitable FF-A support
in the firmware, enable FF-A support for VMs to allow using it for VM to
VM communications.
If there is Optee running in the secure world and using the non FF-A
communication system, having CONFIG_FFA_VM_TO_VM could be non fun
On Wed, Oct 16, 2024 at 2:30 PM Andrew Cooper wrote:
>
> Hello,
>
> Preempting some future work I'm expecting to arrive, I had a go at using
> __builtin_*() in obj32.
>
> This is formed of 2 patches on top of this series:
> https://xenbits.xen.org/gitweb/?p=people/andrewcoop/xen.git;a=shortlog;h=r
On Wed, Oct 16, 2024 at 12:25 PM Anthony PERARD
wrote:
>
> On Wed, Oct 16, 2024 at 09:33:32AM +0100, Frediano Ziglio wrote:
> > On Tue, Oct 15, 2024 at 2:51 PM Anthony PERARD
> > wrote:
> > > On Mon, Oct 14, 2024 at 05:32:26PM +0100, Frediano Ziglio wrote:
> > > > On Mon, Oct 14, 2024 at 4:31 PM
Hello,
Preempting some future work I'm expecting to arrive, I had a go at using
__builtin_*() in obj32.
This is formed of 2 patches on top of this series:
https://xenbits.xen.org/gitweb/?p=people/andrewcoop/xen.git;a=shortlog;h=refs/heads/xen-lib32
Patch 1 introduces lib32 beside obj32, with str
On 16.10.2024 14:41, Marek Marczykowski-Górecki wrote:
> --- a/docs/misc/xen-command-line.pandoc
> +++ b/docs/misc/xen-command-line.pandoc
> @@ -2153,14 +2153,13 @@ Specify the host reboot method.
>
> `kbd` instructs Xen to reboot the host via the keyboard controller.
>
> -`acpi` instructs Xen
Reflect changed default mode, and fix formatting of `efi` value.
Fixes: d81dd3130351 ("x86/shutdown: change default reboot method preference")
Signed-off-by: Marek Marczykowski-Górecki
---
Changes in v2:
- wrap
- update Default: line too
---
docs/misc/xen-command-line.pandoc | 8
1 file
flight 188108 libvirt real [real]
http://logs.test-lab.xenproject.org/osstest/logs/188108/
Failures :-/ but no regressions.
Tests which did not succeed, but are not blocking:
test-armhf-armhf-libvirt 16 saverestore-support-checkfail like 188089
test-amd64-amd64-libvirt 15 migrate-s
On Wed Oct 16, 2024 at 11:54 AM BST, Jan Beulich wrote:
> On 16.10.2024 12:34, Alejandro Vallejo wrote:
> > On Wed Oct 16, 2024 at 8:46 AM BST, Jan Beulich wrote:
> >> --- a/tools/tests/x86_emulator/evex-disp8.c
> >> +++ b/tools/tests/x86_emulator/evex-disp8.c
> >> @@ -911,10 +911,8 @@ static void
On 10/16/24 05:52, Jan Beulich wrote:
> On 11.10.2024 17:27, Stewart Hildebrand wrote:
>> Add links between a VF's struct pci_dev and its associated PF struct
>> pci_dev. Move the calls to pci_get_pdev()/pci_add_device() down to avoid
>> dropping and re-acquiring the pcidevs_lock().
>>
>> During PF
On Wed Oct 16, 2024 at 11:15 AM BST, Jan Beulich wrote:
> On 16.10.2024 12:06, Alejandro Vallejo wrote:
> > On Wed Oct 16, 2024 at 8:45 AM BST, Jan Beulich wrote:
> >> For some reason I entirely consistently screwed these up.
> >>
> >> Signed-off-by: Jan Beulich
> >
> > Reviewed-by: Alejandro V
+ Juergen Gross, Stefano Stabellini
This was a rather tricky series to get the recipients correct for and my script
did not realize that "supporter" was a pseudonym for "maintainer" so you were
missed off the original post. Appologies!
More context in cover letter:
https://lore.kernel.org/all/202
Reflect changed default mode, and fix formatting of `efi` value.
Fixes: d81dd3130351 ("x86/shutdown: change default reboot method preference")
Signed-off-by: Marek Marczykowski-Górecki
---
docs/misc/xen-command-line.pandoc | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/
On 16.10.2024 14:41, Marek Marczykowski-Górecki wrote:
> Reflect changed default mode, and fix formatting of `efi` value.
>
> Fixes: d81dd3130351 ("x86/shutdown: change default reboot method preference")
> Signed-off-by: Marek Marczykowski-Górecki
Acked-by: Jan Beulich
with ...
> --- a/docs/mi
On 16.10.2024 15:17, Marek Marczykowski-Górecki wrote:
> Reflect changed default mode, and fix formatting of `efi` value.
>
> Fixes: d81dd3130351 ("x86/shutdown: change default reboot method preference")
> Signed-off-by: Marek Marczykowski-Górecki
Acked-by: Jan Beulich
flight 188110 xen-unstable real [real]
http://logs.test-lab.xenproject.org/osstest/logs/188110/
Regressions :-(
Tests which did not succeed and are blocking,
including tests which could not be run:
test-amd64-amd64-xl-qemut-debianhvm-i386-xsm 12 debian-hvm-install fail REGR.
vs. 188095
build-a
flight 188125 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/188125/
Perfect :-)
All tests in this flight passed as required
version targeted for testing:
ovmf 1b19ccfdfa0a6eeecfcb8fcccf5437150ca41c3e
baseline version:
ovmf d44b82270b6b1589cc285
flight 188131 ovmf real [real]
http://logs.test-lab.xenproject.org/osstest/logs/188131/
Perfect :-)
All tests in this flight passed as required
version targeted for testing:
ovmf b7342074a649599078553e33acd6d7a79e950683
baseline version:
ovmf 1b19ccfdfa0a6eeecfcb8
On 15/10/2024 17:51, Julien Grall wrote:
Hi Ayan,
Hi Julien,
On 15/10/2024 16:56, Ayan Kumar Halder wrote:
On 14/10/2024 20:03, Luca Fancellu wrote:
Hi Ayan,
Hi Luca,
diff --git a/xen/arch/arm/arm64/mpu/head.S
b/xen/arch/arm/arm64/mpu/ head.S
new file mode 100644
index 00..4
On 10/15/24 9:51 PM, Philipp Stanner wrote:
> pci_intx() is a hybrid function which can sometimes be managed through
> devres. To remove this hybrid nature from pci_intx(), it is necessary to
> port users to either an always-managed or a never-managed version.
>
> All users in ata enable their PC
Hi,
We've encountered a problem booting an 'ovmf' hvm instance from a .iso
image when dm_restrict=1. The deprivileged qemu process can't connect
the .iso and in the qemu-dm .log:
qemu-system-i386: failed to create 'qdisk' device '5632': realization of device
xen-cdrom failed: failed xs_open: No
71 matches
Mail list logo
