[PATCH v6 10/11] virtio-gpu: Initialize Venus

2023-12-19 Thread Huang Rui
From: Antonio Caggiano Request Venus when initializing VirGL. Signed-off-by: Antonio Caggiano Signed-off-by: Huang Rui --- Changes in v6: - Remove the unstable API flags check because virglrenderer is already 1.0. - Squash the render server flag support into "Initialize Venus".

[PATCH v6 11/11] virtio-gpu: make blob scanout use dmabuf fd

2023-12-19 Thread Huang Rui
From: Robert Beckett This relies on a virglrenderer change to include the dmabuf fd when returning resource info. Signed-off-by: Robert Beckett Signed-off-by: Huang Rui --- Changes in v6: - Add scanout_blob function for virtio-gpu-virgl. - Update for new virgl_gpu_resource.

Re: [XEN PATCH v2 7/7] automation/eclair_analysis: avoid violation of MISRA Rule 2.1

2023-12-19 Thread Nicola Vetrini
On 2023-12-19 02:43, Stefano Stabellini wrote: On Mon, 18 Dec 2023, Stefano Stabellini wrote: On Mon, 18 Dec 2023, Nicola Vetrini wrote: > The presence of an unlinked object file triggers a violation > of MISRA C Rule 2.1, which is deviated, as it's not part of > the final Xen binary. > > No

Re: [XEN PATCH] xen: add acmacros.h to exclude-list.json

2023-12-19 Thread Jan Beulich
On 19.12.2023 10:02, Nicola Vetrini wrote: > --- a/docs/misra/exclude-list.json > +++ b/docs/misra/exclude-list.json > @@ -209,6 +209,10 @@ >"rel_path": "include/acpi/acglobal.h", >"comment": "Imported from Linux, ignore for now" > }, > +{ > +

Re: [XEN PATCH] xen: add acmacros.h to exclude-list.json

2023-12-19 Thread Nicola Vetrini
On 2023-12-19 11:37, Jan Beulich wrote: On 19.12.2023 10:02, Nicola Vetrini wrote: --- a/docs/misra/exclude-list.json +++ b/docs/misra/exclude-list.json @@ -209,6 +209,10 @@ "rel_path": "include/acpi/acglobal.h", "comment": "Imported from Linux, ignore for now" },

Re: [XEN PATCH v2 2/7] x86/mm: address MISRA C:2012 Rule 2.1

2023-12-19 Thread Jan Beulich
On 18.12.2023 11:17, Nicola Vetrini wrote: > --- a/xen/arch/x86/mm.c > +++ b/xen/arch/x86/mm.c > @@ -4887,8 +4887,6 @@ long arch_memory_op(unsigned long cmd, > XEN_GUEST_HANDLE_PARAM(void) arg) > default: > return subarch_memory_op(cmd, arg); > } > - > -return 0; > } When

Re: [XEN PATCH] xen: add acmacros.h to exclude-list.json

2023-12-19 Thread Luca Fancellu
Hi! > On 19 Dec 2023, at 11:05, Nicola Vetrini wrote: > > On 2023-12-19 11:51, Nicola Vetrini wrote: >> On 2023-12-19 11:37, Jan Beulich wrote: >>> On 19.12.2023 10:02, Nicola Vetrini wrote: --- a/docs/misra/exclude-list.json +++ b/docs/misra/exclude-list.json @@ -209,6 +209,10

[PATCH] x86/platform: tidy do_platform_op() a little

2023-12-19 Thread Jan Beulich
The function has a few stray scopes and inconsistent use (indentation) of break statements. Drop the stray braces and bring all the break-s in line with one another. This in particular means dropping a redundant break from XENPF_cpu_offline handling, pleasing Misra C:2012 rule 2.1. No functional

Re: [PATCH v6 09/11] virtio-gpu: Support Venus capset

2023-12-19 Thread Pierre-Eric Pelloux-Prayer
Hi Ray, Antonio, Le 19/12/2023 à 08:53, Huang Rui a écrit : From: Antonio Caggiano Add support for the Venus capset, which enables Vulkan support through the Venus Vulkan driver for virtio-gpu. Signed-off-by: Antonio Caggiano Signed-off-by: Huang Rui --- No change in v6.

Re: [PATCH v4 4/4] x86/PV: issue branch prediction barrier when switching 64-bit guest to kernel mode

2023-12-19 Thread Jan Beulich
On 18.12.2023 18:24, Roger Pau Monné wrote: > On Tue, Feb 14, 2023 at 05:12:08PM +0100, Jan Beulich wrote: >> Since both kernel and user mode run in ring 3, they run in the same >> "predictor mode". > > That only true when IBRS is enabled, otherwise all CPU modes share the > same predictor mode?

Re: [linus:master] [x86/entry] be5341eb0d: WARNING:CPU:#PID:#at_int80_emulation

2023-12-19 Thread Borislav Petkov
On Tue, Dec 19, 2023 at 04:49:14PM +0800, kernel test robot wrote: > [ 13.481107][ T48] WARNING: CPU: 0 PID: 48 at int80_emulation > (arch/x86/entry/common.c:164) > [ 13.481454][ T48] Modules linked in: > [ 13.481655][ T48] CPU: 0 PID: 48 Comm: init Tainted: G N >

Re: [XEN PATCH 5/5] xen/wait: address violations of MISRA C Rule 11.9

2023-12-19 Thread Jan Beulich
On 14.12.2023 12:44, Nicola Vetrini wrote: > No functional change. > > Signed-off-by: Nicola Vetrini Acked-by: Jan Beulich Nevertheless, imo ... > --- a/xen/common/wait.c > +++ b/xen/common/wait.c > @@ -125,7 +125,7 @@ static void __prepare_to_wait(struct waitqueue_vcpu *wqv) > struct

[PATCH v2 3/6] xen: add deviations for Rule 11.8

2023-12-19 Thread Simone Ballarin
From: Maria Celeste Cesario The xen sources contain violations of MISRA C:2012 Rule 11.8 whose headline states: "A conversion shall not remove any const, volatile or _Atomic qualification from the type pointed to by a pointer". The macro 'container_of' violates the Rule because it casts away

[PATCH v2 5/6] xen: remove unused function ERR_CAST

2023-12-19 Thread Simone Ballarin
From: Maria Celeste Cesario Function ERR_CAST contains a violation of MISRA C:2012 Rule 11.8, whose headline states: "A conversion shall not remove any const, volatile or _Atomic qualification from the type pointed to by a pointer". Since the function has no users, it is appropriate to remove

[PATCH v2 6/6] xen/common: address violations of MISRA C:2012 Rule 11.8

2023-12-19 Thread Simone Ballarin
From: Maria Celeste Cesario The xen sources contain violations of MISRA C:2012 Rule 11.8 whose headline states: "A conversion shall not remove any const, volatile or _Atomic qualification from the type pointed to by a pointer". Change cast type from void* to uintptr_t. void* type cast resulted

[PATCH v2 0/6] xen: address violations of MISRA C:2012 Rule 11.8

2023-12-19 Thread Simone Ballarin
From: Maria Celeste Cesario The xen sources contain violations of MISRA C:2012 Rule 11.8 whose headline states: "A conversion shall not remove any const, volatile or _Atomic qualification from the type pointed to by a pointer". This patch amends or drops casts that unnecessarily drop const

[PATCH v2 2/6] xen/ppc: address violations of MISRA C:2012 Rule 11.8.

2023-12-19 Thread Simone Ballarin
From: Maria Celeste Cesario The xen sources contain violations of MISRA C:2012 Rule 11.8 whose headline states: "A conversion shall not remove any const, volatile or _Atomic qualification from the type pointed to by a pointer". Fix violation by adding missing const qualifier in cast.

[PATCH v2 4/6] xen: add SAF deviation for safe cast removal

2023-12-19 Thread Simone Ballarin
From: Maria Celeste Cesario The xen sources contain violations of MISRA C:2012 Rule 11.8 whose headline states: "A conversion shall not remove any const, volatile or _Atomic qualification from the type pointed to by a pointer". Add SAF-3-safe deviation: removal of const qualifier to comply with

[PATCH v2 1/6] xen/arm: address violations of MISRA C:2012 Rule 11.8

2023-12-19 Thread Simone Ballarin
From: Maria Celeste Cesario The xen sources contain violations of MISRA C:2012 Rule 11.8 whose headline states: "A conversion shall not remove any const, volatile or _Atomic qualification from the type pointed to by a pointer". Add missing const qualifiers in casts. The variables are originally

[XEN PATCH] xen: add acmacros.h to exclude-list.json

2023-12-19 Thread Nicola Vetrini
The file was inherited from Linux and ACPI CA, therefore it's not subject to MISRA compliance at the moment. No functional change. Signed-off-by: Nicola Vetrini --- docs/misra/exclude-list.json | 4 1 file changed, 4 insertions(+) diff --git a/docs/misra/exclude-list.json

Re: [XEN PATCH 4/5] x86/hvm: dom0: use helper to get sizeof struct field

2023-12-19 Thread Jan Beulich
On 14.12.2023 22:32, Stefano Stabellini wrote: > On Thu, 14 Dec 2023, Nicola Vetrini wrote: >> Use of the proper helper macro also resolves a violation >> of MISRA C Rule 11.9. >> No functional change. >> >> Signed-off-by: Nicola Vetrini > > Reviewed-by: Stefano Stabellini Acked-by: Jan

Re: [XEN PATCH] xen: add acmacros.h to exclude-list.json

2023-12-19 Thread Nicola Vetrini
On 2023-12-19 11:51, Nicola Vetrini wrote: On 2023-12-19 11:37, Jan Beulich wrote: On 19.12.2023 10:02, Nicola Vetrini wrote: --- a/docs/misra/exclude-list.json +++ b/docs/misra/exclude-list.json @@ -209,6 +209,10 @@ "rel_path": "include/acpi/acglobal.h", "comment":

Re: [PATCH v2 5/6] xen: remove unused function ERR_CAST

2023-12-19 Thread Jan Beulich
On 19.12.2023 12:05, Simone Ballarin wrote: > From: Maria Celeste Cesario > > Function ERR_CAST contains a violation of MISRA C:2012 Rule 11.8, > whose headline states: > "A conversion shall not remove any const, volatile or _Atomic qualification > from the type pointed to by a pointer". > >

[linus:master] [x86/entry] be5341eb0d: WARNING:CPU:#PID:#at_int80_emulation

2023-12-19 Thread kernel test robot
mer syscall while CONFIG_POSIX_TIMERS is not set [ 13.517868][ T48] CFI failure at int80_emulation+0x67/0xb0 (target: sys_ni_posix_timers+0x0/0x70; expected type: 0xb02b34d9 The kernel config and materials to reproduce are available at: https://download.01.org/0day-ci/archive/20231219/20231

Re: [PATCH v6 02/11] virtio-gpu: Configure new feature flag context_create_with_flags for virglrenderer

2023-12-19 Thread Antonio Caggiano
Hi Huang Rui, Thank you for this new version. All patches which I did not sign off are reviewed by me :) Cheers, Antonio Caggiano On 19/12/2023 08:53, Huang Rui wrote: Configure a new feature flag (context_create_with_flags) for virglrenderer. Originally-by: Antonio Caggiano Signed-off-by:

Re: [BUG]i2c_hid_acpi broken with 4.17.2 on Framework Laptop 13 AMD

2023-12-19 Thread Jan Beulich
On 18.12.2023 18:04, Sébastien Chaumat wrote: > Le lun. 18 déc. 2023, 17:44, Jan Beulich a écrit : > >> On 18.12.2023 17:21, Sébastien Chaumat wrote: >>> On 05.12.2023 21:31, Sébastien Chaumat wrote: > This issue seems that IRQ 7 (the GPIO controller) is natively fasteoi > (so level

[PATCH] x86/mm: use "break" in arch_memory_op()

2023-12-19 Thread Jan Beulich
The final return statement is unreachable and hence disliked by Misra C:2012 (rule 2.1). Convert those case-specific (main) return statements which already use "rc", or in one case when it can be used without further adding of code, to break. No functional change intended. Signed-off-by: Jan

Re: [PATCH v3 6/7] x86/iommu: switch hwdom IOMMU to use a rangeset

2023-12-19 Thread Jan Beulich
On 15.12.2023 15:18, Roger Pau Monne wrote: > The current loop that iterates from 0 to the maximum RAM address in order to > setup the IOMMU mappings is highly inefficient, and it will get worse as the > amount of RAM increases. It's also not accounting for any reserved regions > past the last

Re: Possible bug in Xen

2023-12-19 Thread Andrew Cooper
On 19/12/2023 4:28 pm, Joe Tretter wrote: > On 12/19/23 10:05, Andrew Cooper wrote: >> Is it always the same test which fails, or is it random? > Which test fails seems to be random (see attached screenshot). >> Looking at https://github.com/Tarsnap/scrypt it's only a trivial piece >> of userspace

Re: [PATCH v2 3/7] xen/common: Move Arm's bootfdt to common

2023-12-19 Thread Julien Grall
Hi Jan, On 19/12/2023 17:03, Jan Beulich wrote: On 15.12.2023 03:43, Shawn Anastasio wrote: Move Arm's bootfdt.c to xen/common so that it can be used by other device tree architectures like PPC and RISCV. Only a minor change to conditionalize a call to a function only available on

Re: [PATCH v6 9/9] xen/arm: Map ITS doorbell register to IOMMU page tables

2023-12-19 Thread Julien Grall
Hi, On 09/11/2023 18:27, Stewart Hildebrand wrote: From: Rahul Singh When ITS is enabled and PCI devices that are behind an SMMU generate an MSI interrupt, SMMU fault will be observed as there is currently no mapping in p2m table for the ITS translation register (GITS_TRANSLATER). A mapping

Re: [PATCH v4 1/4] x86/livepatch: align functions to ensure minimal distance between entry points

2023-12-19 Thread Andrew Cooper
On 15/12/2023 11:18 am, Roger Pau Monne wrote: > The minimal function size requirements for livepatch are either 5 bytes (for "for an x86 livepatch", seeing as we're touching multiple architectures worth of files. I know it's at the end of the sentence, but it wants to be earlier to be clearer.

[ovmf test] 184177: all pass - PUSHED

2023-12-19 Thread osstest service owner
flight 184177 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/184177/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf 8c1e9f9c6fa7b5137003b0cfa6d54a6bada16d8e baseline version: ovmf

Re: [PATCH v4 2/4] xen/x86: introduce self modifying code test

2023-12-19 Thread Andrew Cooper
On 15/12/2023 11:18 am, Roger Pau Monne wrote: > Introduce a helper to perform checks related to self modifying code, and start > by creating a simple test to check that alternatives have been applied. > > Such test is hooked into the boot process and called just after alternatives > have been

Re: How to boot FreeBSD for arm 32 bit as DomU with u-boot on my ARM Chromebook

2023-12-19 Thread Mario Marietto
I've asked some help on the channel #arm on Reddit and someone replied : https://www.reddit.com/r/arm/comments/18mcir8/i_cant_boot_freebsd_for_arm32_bit_as_domu_with/ Maybe his answer can be useful to understand why it does not work. On Tue, Dec 19, 2023 at 8:33 PM Stefano Stabellini wrote: >

Re: [BUG]i2c_hid_acpi broken with 4.17.2 on Framework Laptop 13 AMD

2023-12-19 Thread Sébastien Chaumat
Le mar. 19 déc. 2023 à 20:03, Sébastien Chaumat a écrit : > Le mar. 19 déc. 2023 à 16:15, Sébastien Chaumat a > écrit : > > > > I did add an extra printk in PHYSDEVOP_setup_gsi > > so the "first one" is my printk (available in xl dmesg) > > the second message is from xen_register_gsi (from

Re: [PATCH v11 02/17] pci: introduce per-domain PCI rwlock

2023-12-19 Thread Stefano Stabellini
On Sat, 2 Dec 2023, Volodymyr Babchuk wrote: > Add per-domain d->pci_lock that protects access to > d->pdev_list. Purpose of this lock is to give guarantees to VPCI code > that underlying pdev will not disappear under feet. This is a rw-lock, > but this patch adds only write_lock()s. There will be

Re: How to boot FreeBSD for arm 32 bit as DomU with u-boot on my ARM Chromebook

2023-12-19 Thread Warner Losh
I'd think you'd need the right virtualization loader. I'm not entirely sure the u-boot.bin you've been creating is for a dom-u.. If I misunderstood, then the below isn't good advice. Chain booting the u-boot, the first u-boot initializes things so you want to start with stage after the SPL. But

[ovmf test] 184185: all pass - PUSHED

2023-12-19 Thread osstest service owner
flight 184185 ovmf real [real] http://logs.test-lab.xenproject.org/osstest/logs/184185/ Perfect :-) All tests in this flight passed as required version targeted for testing: ovmf a83d953dc2741f650cc273f39fe803ae406b0fad baseline version: ovmf

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

2023-12-19 Thread osstest service owner
flight 184184 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/184184/ 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: [linus:master] [x86/entry] be5341eb0d: WARNING:CPU:#PID:#at_int80_emulation

2023-12-19 Thread Linus Torvalds
On Tue, 19 Dec 2023 at 01:58, Borislav Petkov wrote: > > Looking at the dmesg, I think you missed the most important part - the > preceding line: > > [ 13.480504][ T48] CFI failure at int80_emulation+0x67/0xb0 (target: > sys_ni_posix_timers+0x0/0x70; expected type: 0xb02b34d9) >

[xen-unstable test] 184176: regressions - FAIL

2023-12-19 Thread osstest service owner
flight 184176 xen-unstable real [real] http://logs.test-lab.xenproject.org/osstest/logs/184176/ Regressions :-( Tests which did not succeed and are blocking, including tests which could not be run: test-amd64-i386-qemut-rhel6hvm-intel 12 redhat-install fail REGR. vs. 184172

Re: [BUG]i2c_hid_acpi broken with 4.17.2 on Framework Laptop 13 AMD

2023-12-19 Thread Sébastien Chaumat
Le mer. 20 déc. 2023 à 00:50, Sébastien Chaumat a écrit : > > > Le mer. 20 déc. 2023 à 00:11, Sébastien Chaumat a > écrit : > >> >> >> Le mer. 20 déc. 2023 à 00:06, Sébastien Chaumat a >> écrit : >> >>> >>> >>> Le mar. 19 déc. 2023 à 20:03, Sébastien Chaumat a >>> écrit : >>> Le mar. 19

Re: [PATCH] x86/mm: use "break" in arch_memory_op()

2023-12-19 Thread Stefano Stabellini
On Tue, 19 Dec 2023, Jan Beulich wrote: > The final return statement is unreachable and hence disliked by Misra > C:2012 (rule 2.1). Convert those case-specific (main) return statements > which already use "rc", or in one case when it can be used without > further adding of code, to break. > > No

Re: [PATCH v2 1/6] xen/arm: address violations of MISRA C:2012 Rule 11.8

2023-12-19 Thread Stefano Stabellini
On Tue, 19 Dec 2023, Simone Ballarin wrote: > From: Maria Celeste Cesario > > The xen sources contain violations of MISRA C:2012 Rule 11.8 whose > headline states: > "A conversion shall not remove any const, volatile or _Atomic qualification > from the type pointed to by a pointer". > > Add

xen | Successful pipeline for staging | 725acf8e

2023-12-19 Thread GitLab
Pipeline #1113609566 has passed! Project: xen ( https://gitlab.com/xen-project/xen ) Branch: staging ( https://gitlab.com/xen-project/xen/-/commits/staging ) Commit: 725acf8e ( https://gitlab.com/xen-project/xen/-/commit/725acf8e4d406bf0a24441ac1738eb6f4f6ef0c3 ) Commit Message: xen/arm:

Re: [BUG]i2c_hid_acpi broken with 4.17.2 on Framework Laptop 13 AMD

2023-12-19 Thread Sébastien Chaumat
Le mer. 20 déc. 2023 à 00:06, Sébastien Chaumat a écrit : > > > Le mar. 19 déc. 2023 à 20:03, Sébastien Chaumat a > écrit : > >> Le mar. 19 déc. 2023 à 16:15, Sébastien Chaumat a >> écrit : >> > >> > I did add an extra printk in PHYSDEVOP_setup_gsi >> > so the "first one" is my printk

Re: [PATCH v2] docs/misra/rules.rst: add Rule 17.1

2023-12-19 Thread Stefano Stabellini
On Tue, 19 Dec 2023, Julien Grall wrote: > Hi Stefano, > > On 15/12/2023 21:02, Stefano Stabellini wrote: > > On Fri, 15 Dec 2023, Julien Grall wrote: > > > Hi Stefano, > > > > > > On 09/12/2023 01:39, Stefano Stabellini wrote: > > > > > > > > Signed-off-by: Stefano Stabellini > > > > --- > >

Re: [PATCH v2 3/6] xen: add deviations for Rule 11.8

2023-12-19 Thread Stefano Stabellini
On Tue, 19 Dec 2023, Simone Ballarin wrote: > From: Maria Celeste Cesario > > The xen sources contain violations of MISRA C:2012 Rule 11.8 whose > headline states: > "A conversion shall not remove any const, volatile or _Atomic qualification > from the type pointed to by a pointer". > > The

Re: [linus:master] [x86/entry] be5341eb0d: WARNING:CPU:#PID:#at_int80_emulation

2023-12-19 Thread Linus Torvalds
On Tue, 19 Dec 2023 at 12:17, Linus Torvalds wrote: > > That said, I still think that just getting rid of this horrid special > case for posix timers is the right thing, and we should just remove > that SYS_NI() alias thing entirely. IOW, something like the attached patch. It's not extensively

Re: [XEN PATCH] xen: add acmacros.h to exclude-list.json

2023-12-19 Thread Stefano Stabellini
On Tue, 19 Dec 2023, Luca Fancellu wrote: > Hi! > > > On 19 Dec 2023, at 11:05, Nicola Vetrini wrote: > > > > On 2023-12-19 11:51, Nicola Vetrini wrote: > >> On 2023-12-19 11:37, Jan Beulich wrote: > >>> On 19.12.2023 10:02, Nicola Vetrini wrote: > --- a/docs/misra/exclude-list.json >

Re: [PATCH] x86/platform: tidy do_platform_op() a little

2023-12-19 Thread Stefano Stabellini
On Tue, 19 Dec 2023, Jan Beulich wrote: > The function has a few stray scopes and inconsistent use (indentation) > of break statements. Drop the stray braces and bring all the break-s in > line with one another. This in particular means dropping a redundant > break from XENPF_cpu_offline handling,

Re: [RFC PATCH] xen/dm: arm: Introudce arm_inject_msi DM op

2023-12-19 Thread Stefano Stabellini
On Tue, 19 Dec 2023, Julien Grall wrote: > Hi, > > On 19/12/2023 14:18, Mykyta Poturai wrote: > > Following up with relevant QEMU patch link. > > > > https://patchwork.kernel.org/project/qemu-devel/patch/c7a180a5874f036c246fc39f921eefafecbc8c76.1702994649.git.mykyta_potu...@epam.com/ > > I

Re: [PATCH v2 5/6] xen: remove unused function ERR_CAST

2023-12-19 Thread Stefano Stabellini
On Tue, 19 Dec 2023, Simone Ballarin wrote: > From: Maria Celeste Cesario > > Function ERR_CAST contains a violation of MISRA C:2012 Rule 11.8, > whose headline states: > "A conversion shall not remove any const, volatile or _Atomic qualification > from the type pointed to by a pointer". > >

Re: [PATCH v11 01/17] pci: msi: pass pdev to pci_enable_msi() function

2023-12-19 Thread Stefano Stabellini
On Sat, 2 Dec 2023, Volodymyr Babchuk wrote: > Previously pci_enable_msi() function obtained pdev pointer by itself, > but taking into account upcoming changes to PCI locking, it is better > when caller passes already acquired pdev pointer to the function, > because caller knows better how to

Re: [BUG]i2c_hid_acpi broken with 4.17.2 on Framework Laptop 13 AMD

2023-12-19 Thread Sébastien Chaumat
Le mer. 20 déc. 2023 à 00:11, Sébastien Chaumat a écrit : > > > Le mer. 20 déc. 2023 à 00:06, Sébastien Chaumat a > écrit : > >> >> >> Le mar. 19 déc. 2023 à 20:03, Sébastien Chaumat a >> écrit : >> >>> Le mar. 19 déc. 2023 à 16:15, Sébastien Chaumat a >>> écrit : >>> > >>> > I did add an

Re: [PATCH v2 2/7] xen/asm-generic: Introduce generic setup.h

2023-12-19 Thread Jan Beulich
On 15.12.2023 03:43, Shawn Anastasio wrote: > Introduce setup.h to asm-generic based off of ARM's to define all > stubs necessary to compile bootfdt.c And - I'm sorry to ask - how's that "generic"? I'm also not convinced at all that a generic header can validly use BUG() in the way you propose.

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

2023-12-19 Thread osstest service owner
flight 184179 linux-linus real [real] flight 184188 linux-linus real-retest [real] http://logs.test-lab.xenproject.org/osstest/logs/184179/ http://logs.test-lab.xenproject.org/osstest/logs/184188/ Failures :-/ but no regressions. Tests which are failing intermittently (not blocking):

Re: [linus:master] [x86/entry] be5341eb0d: WARNING:CPU:#PID:#at_int80_emulation

2023-12-19 Thread Sami Tolvanen
On Tue, Dec 19, 2023 at 10:21 AM Linus Torvalds wrote: > > On Tue, 19 Dec 2023 at 01:58, Borislav Petkov wrote: > > > > Looking at the dmesg, I think you missed the most important part - the > > preceding line: > > > > [ 13.480504][ T48] CFI failure at int80_emulation+0x67/0xb0 (target: > >

Re: [PATCH v6 7/9] xen/arm: Fix mapping for PCI bridge mmio region

2023-12-19 Thread Julien Grall
Hi Stewart, On 09/11/2023 18:27, Stewart Hildebrand wrote: From: Rahul Singh Current code skip the mapping for PCI bridge MMIO region to dom0 when pci_passthrough_enabled flag is set. Mapping should be skip when has_vpci(d) is enabled for the domain, as we need to skip the mapping only when

Re: [XEN PATCH] xen/arm: ffa: return fpi size from FFA_PARTITION_INFO_GET

2023-12-19 Thread Julien Grall
Hi, On 18/12/2023 14:33, Bertrand Marquis wrote: Hi Jens, On 13 Dec 2023, at 11:31, Jens Wiklander wrote: Until now has FFA_PARTITION_INFO_GET always returned zero in w3, but FF-A v1.1 requires FFA_PARTITION_INFO_GET to return the size of each partition information descriptor returned if

Re: [linus:master] [x86/entry] be5341eb0d: WARNING:CPU:#PID:#at_int80_emulation

2023-12-19 Thread Andrew Cooper
On 19/12/2023 6:20 pm, Linus Torvalds wrote: > On Tue, 19 Dec 2023 at 01:58, Borislav Petkov wrote: >> Looking at the dmesg, I think you missed the most important part - the >> preceding line: >> >> [ 13.480504][ T48] CFI failure at int80_emulation+0x67/0xb0 (target: >>

Re: [PATCH v6 5/9] xen/arm: smmuv2: Add PCI devices support for SMMUv2

2023-12-19 Thread Julien Grall
Hi Stewart, On 09/11/2023 18:27, Stewart Hildebrand wrote: From: Oleksandr Andrushchenko In general, we should avoid blank commit. In this case, I think some details would be useful about the implementation. Some of the details I am interested in is how the logic works and why we don't

xen | Successful pipeline for staging | 1b42304f

2023-12-19 Thread GitLab
Pipeline #1113302155 has passed! Project: xen ( https://gitlab.com/xen-project/xen ) Branch: staging ( https://gitlab.com/xen-project/xen/-/commits/staging ) Commit: 1b42304f ( https://gitlab.com/xen-project/xen/-/commit/1b42304fd1d84adcf7caf8c12c33ec66347b6dcc ) Commit Message: xen/arm:

Re: [PATCH v2 04/14] aio: make aio_context_acquire()/aio_context_release() a no-op

2023-12-19 Thread Stefan Hajnoczi
The following hack makes the test pass but there are larger safety issues that I'll need to look at on Wednesday: diff --git a/nbd/server.c b/nbd/server.c index 895cf0a752..cf4b7d5c6d 100644 --- a/nbd/server.c +++ b/nbd/server.c @@ -1617,7 +1617,7 @@ static void nbd_drained_begin(void *opaque)

Re: [PATCH v4 4/4] x86/PV: issue branch prediction barrier when switching 64-bit guest to kernel mode

2023-12-19 Thread Roger Pau Monné
On Tue, Dec 19, 2023 at 04:11:09PM +0100, Roger Pau Monné wrote: > On Tue, Dec 19, 2023 at 03:06:50PM +0100, Jan Beulich wrote: > > On 19.12.2023 12:48, Roger Pau Monné wrote: > > > On Tue, Dec 19, 2023 at 10:56:16AM +0100, Jan Beulich wrote: > > >> On 18.12.2023 18:24, Roger Pau Monné wrote: > >

Re: [PATCH v2 1/7] xen/asm-generic: Introduce generic static-shmem.h

2023-12-19 Thread Jan Beulich
On 15.12.2023 03:43, Shawn Anastasio wrote: > --- /dev/null > +++ b/xen/include/asm-generic/static-shmem.h > @@ -0,0 +1,12 @@ > +/* SPDX-License-Identifier: GPL-2.0-only */ > + > +#ifndef __ASM_GENERIC_STATIC_SHMEM_H__ > +#define __ASM_GENERIC_STATIC_SHMEM_H__ > + > +static inline int

Re: [PATCH v2 04/14] aio: make aio_context_acquire()/aio_context_release() a no-op

2023-12-19 Thread Kevin Wolf
Am 19.12.2023 um 16:28 hat Kevin Wolf geschrieben: > Am 05.12.2023 um 19:20 hat Stefan Hajnoczi geschrieben: > > aio_context_acquire()/aio_context_release() has been replaced by > > fine-grained locking to protect state shared by multiple threads. The > > AioContext lock still plays the role of

Re: [PATCH v2] docs/misra/rules.rst: add Rule 17.1

2023-12-19 Thread Julien Grall
Hi Stefano, On 15/12/2023 21:02, Stefano Stabellini wrote: On Fri, 15 Dec 2023, Julien Grall wrote: Hi Stefano, On 09/12/2023 01:39, Stefano Stabellini wrote: Signed-off-by: Stefano Stabellini --- Changes in v2: - separated 17.1 in its own patch - add a comment diff --git

Re: [PATCH v6 6/9] xen/arm: smmuv3: Add PCI devices support for SMMUv3

2023-12-19 Thread Julien Grall
Hi, On 09/11/2023 18:27, Stewart Hildebrand wrote: From: Rahul Singh Looking at the code, most of my comments on SMMUv2 apply here too. So I will not repeat them. Cheers, -- Julien Grall

Re: How to boot FreeBSD for arm 32 bit as DomU with u-boot on my ARM Chromebook

2023-12-19 Thread Stefano Stabellini
+Michal Hi Mario, I am not sure about booting FreeBSD, but I am certain that u-boot works fine as DomU kernel on ARMv8 (not sure about ARMv7). With this config file: name="test" kernel="u-boot.bin" extra = "console=hvc0" memory=256 vcpus=1 disk = [ 'FreeBSD-13.2-RELEASE-armv7.img,raw,xvda' ]

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

2023-12-19 Thread osstest service owner
flight 184178 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/184178/ 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 v3 7/7] x86/iommu: cleanup unused functions

2023-12-19 Thread Jan Beulich
On 15.12.2023 15:18, Roger Pau Monne wrote: > Remove xen_in_range() and vpci_is_mmcfg_address() now that hey are unused. > > Adjust comments to point to the new functions that replace the existing ones. And because of this affecting what was done earlier in the series, perhaps best if patches

Re: [PATCH v6 2/5] xen/vpci: move xen_domctl_createdomain vPCI flag to common

2023-12-19 Thread Stewart Hildebrand
On 12/15/23 04:36, Rahul Singh wrote: > Hi Stewart, > >> On 29 Nov 2023, at 9:25 pm, Stewart Hildebrand >> wrote: >> >> On 11/14/23 04:11, Jan Beulich wrote: >>> On 13.11.2023 23:21, Stewart Hildebrand wrote: @@ -709,10 +710,17 @@ int arch_sanitise_domain_config(struct

Re: [PATCH v3 5/7] x86/iommu: remove regions not to be mapped

2023-12-19 Thread Jan Beulich
On 15.12.2023 15:18, Roger Pau Monne wrote: > Introduce the code to remove regions not to be mapped from the rangeset > that will be used to setup the IOMMU page tables for the hardware domain. > > This change also introduces two new functions: remove_xen_ranges() and > vpci_subtract_mmcfg() that

Re: [PATCH v4 2/4] xen/x86: introduce self modifying code test

2023-12-19 Thread Jan Beulich
On 15.12.2023 12:18, Roger Pau Monne wrote: > Introduce a helper to perform checks related to self modifying code, and start > by creating a simple test to check that alternatives have been applied. > > Such test is hooked into the boot process and called just after alternatives > have been

Re: hvmloader - allow_memory_relocate overlaps

2023-12-19 Thread Jan Beulich
On 16.12.2023 08:01, Neowutran wrote: > I am wondering if the variable "allow_memory_relocate" is still > relevant today and if its default value is still relevant. > Should it be defined to 0 by default instead of 1 (it seems to be a > workaround for qemu-traditional, so maybe an outdated

Re: [BUG]i2c_hid_acpi broken with 4.17.2 on Framework Laptop 13 AMD

2023-12-19 Thread Sébastien Chaumat
Le mar. 19 déc. 2023 à 16:15, Sébastien Chaumat a écrit : > > I did add an extra printk in PHYSDEVOP_setup_gsi > so the "first one" is my printk (available in xl dmesg) > the second message is from xen_register_gsi (from linux kernel) > > Le mar. 19 déc. 2023 à 14:15, Jan Beulich a écrit : > > >

Re: [PATCH v6 8/9] xen/arm: enable dom0 to use PCI devices with pci-passthrough=no

2023-12-19 Thread Julien Grall
Hi Stewart, On 09/11/2023 18:27, Stewart Hildebrand wrote: Enable the use of IOMMU + PCI in dom0 without having to specify "pci-passthrough=yes". We rely on dom0 to initialize the PCI controller and perform a PHYSDEVOP_pci_device_add call to add each device to SMMU. Enable pci_init() for

Re: [PATCH v4 1/4] x86/livepatch: align functions to ensure minimal distance between entry points

2023-12-19 Thread Jan Beulich
On 15.12.2023 12:18, Roger Pau Monne wrote: > The minimal function size requirements for livepatch are either 5 bytes (for > jmp) or 9 bytes (for endbr + jmp) on x86, and always 4 bytes on Arm. Ensure > that distance between functions entry points is always at least of the minimal > required size

Re: [PATCH v2 06/14] block: remove AioContext locking

2023-12-19 Thread Stefan Hajnoczi
On Tue, 19 Dec 2023 at 10:59, Kevin Wolf wrote: > > Am 05.12.2023 um 19:20 hat Stefan Hajnoczi geschrieben: > > This is the big patch that removes > > aio_context_acquire()/aio_context_release() from the block layer and > > affected block layer users. > > > > There isn't a clean way to split this

Re: [linus:master] [x86/entry] be5341eb0d: WARNING:CPU:#PID:#at_int80_emulation

2023-12-19 Thread Linus Torvalds
On Tue, 19 Dec 2023 at 11:15, Andrew Cooper wrote: > > -asmlinkage long sys_ni_posix_timers(void); > +asmlinkage long sys_ni_posix_timers(const struct pt_regs *regs); I don't think it should be asmlinkage. That means "use legacy asm calling conventions", and for x86-32 that means pass on stack.

Re: [PATCH v2 6/6] xen/common: address violations of MISRA C:2012 Rule 11.8

2023-12-19 Thread Jan Beulich
On 19.12.2023 12:05, Simone Ballarin wrote: > From: Maria Celeste Cesario > > The xen sources contain violations of MISRA C:2012 Rule 11.8 whose > headline states: > "A conversion shall not remove any const, volatile or _Atomic qualification > from the type pointed to by a pointer". > > Change

Re: [PATCH v2 3/7] xen/common: Move Arm's bootfdt to common

2023-12-19 Thread Jan Beulich
On 15.12.2023 03:43, Shawn Anastasio wrote: > Move Arm's bootfdt.c to xen/common so that it can be used by other > device tree architectures like PPC and RISCV. Only a minor change to > conditionalize a call to a function only available on EFI-supporting > targets was made to the code itself. > >

Re: [BUG]i2c_hid_acpi broken with 4.17.2 on Framework Laptop 13 AMD

2023-12-19 Thread Sébastien Chaumat
I did add an extra printk in PHYSDEVOP_setup_gsi so the "first one" is my printk (available in xl dmesg) the second message is from xen_register_gsi (from linux kernel) Le mar. 19 déc. 2023 à 14:15, Jan Beulich a écrit : > > On 18.12.2023 17:21, Sébastien Chaumat wrote: > > On 05.12.2023

Re: [PATCH v4 1/5] x86/HVM: split restore state checking from state loading

2023-12-19 Thread Jan Beulich
On 19.12.2023 15:36, Roger Pau Monné wrote: > On Mon, Dec 18, 2023 at 03:39:55PM +0100, Jan Beulich wrote: >> ..., at least as reasonably feasible without making a check hook >> mandatory (in particular strict vs relaxed/zero-extend length checking >> can't be done early this way). >> >> Note that

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

2023-12-19 Thread osstest service owner
flight 184174 xen-unstable-smoke real [real] http://logs.test-lab.xenproject.org/osstest/logs/184174/ 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 v2 04/14] aio: make aio_context_acquire()/aio_context_release() a no-op

2023-12-19 Thread Kevin Wolf
Am 05.12.2023 um 19:20 hat Stefan Hajnoczi geschrieben: > aio_context_acquire()/aio_context_release() has been replaced by > fine-grained locking to protect state shared by multiple threads. The > AioContext lock still plays the role of balancing locking in > AIO_WAIT_WHILE() and many functions in

Re: [PATCH v3 2/7] x86/p2m: move and rename paging_max_paddr_bits()

2023-12-19 Thread Jan Beulich
On 15.12.2023 15:18, Roger Pau Monne wrote: > The function also supports non-paging domains, and hence it being placed in > p2m.h and named with the paging_ prefix is misleading. > > Move to x86 domain.c and rename to domain_max_paddr_bits(). Moving to a > different header is non trivial, as the

Re: [PATCH v3 3/7] amd-vi: set IOMMU page table levels based on guest reported paddr width

2023-12-19 Thread Jan Beulich
On 15.12.2023 15:18, Roger Pau Monne wrote: > However take into account the minimum number of levels required by unity maps > when setting the page table levels. > > The previous setting of the page table levels for PV guests based on the > highest RAM address was bogus, as there can be other

Re: [PATCH v2 06/14] block: remove AioContext locking

2023-12-19 Thread Kevin Wolf
Am 05.12.2023 um 19:20 hat Stefan Hajnoczi geschrieben: > This is the big patch that removes > aio_context_acquire()/aio_context_release() from the block layer and > affected block layer users. > > There isn't a clean way to split this patch and the reviewers are likely > the same group of

Re: [PATCH v3 4/7] x86/iommu: introduce a rangeset to perform hwdom IOMMU setup

2023-12-19 Thread Jan Beulich
On 15.12.2023 15:18, Roger Pau Monne wrote: > --- a/xen/drivers/passthrough/x86/iommu.c > +++ b/xen/drivers/passthrough/x86/iommu.c > @@ -370,10 +370,88 @@ static unsigned int __hwdom_init hwdom_iommu_map(const > struct domain *d, > return perms; > } > > +struct map_data { > +struct

Re: Possible bug in Xen

2023-12-19 Thread Andrew Cooper
On 19/12/2023 12:32 am, Joe Tretter wrote: > Hi Andrew. > > Yes, running the scrypt unit test twice simultaneously made the error > occur faster. > This time, this wasn't required because it failed straight away - the > boot parameter is present: Is it always the same test which fails, or is it

Re: [RFC PATCH] xen/dm: arm: Introudce arm_inject_msi DM op

2023-12-19 Thread Julien Grall
Hi Jan, On 19/12/2023 14:17, Jan Beulich wrote: On 19.12.2023 15:12, Julien Grall wrote: On 19/12/2023 13:57, Jan Beulich wrote: On 19.12.2023 14:48, Mykyta Poturai wrote: This patch adds the ability for the device emulator to inject MSI interrupts into guests. This is done by adding a new

Re: [RFC PATCH] xen/dm: arm: Introudce arm_inject_msi DM op

2023-12-19 Thread Jan Beulich
On 19.12.2023 15:19, Andrew Cooper wrote: > On 19/12/2023 1:48 pm, Mykyta Poturai wrote: >> This patch adds the ability for the device emulator to inject MSI >> interrupts into guests. This is done by adding a new DM op to the device >> model library. >> >> It is not possible to reuse already

xen | Successful pipeline for staging | 05b6368b

2023-12-19 Thread GitLab
Pipeline #1112742505 has passed! Project: xen ( https://gitlab.com/xen-project/xen ) Branch: staging ( https://gitlab.com/xen-project/xen/-/commits/staging ) Commit: 05b6368b ( https://gitlab.com/xen-project/xen/-/commit/05b6368ba01e07aa3b2b1f1856d77332971a7bdf ) Commit Message: x86: don't

Re: [RFC PATCH] xen/dm: arm: Introudce arm_inject_msi DM op

2023-12-19 Thread Julien Grall
Hi, On 19/12/2023 14:18, Mykyta Poturai wrote: Following up with relevant QEMU patch link. https://patchwork.kernel.org/project/qemu-devel/patch/c7a180a5874f036c246fc39f921eefafecbc8c76.1702994649.git.mykyta_potu...@epam.com/ I don't seem to have the patch in my inbox. I guess you didn't CC

Re: [RFC PATCH] xen/dm: arm: Introudce arm_inject_msi DM op

2023-12-19 Thread Andrew Cooper
On 19/12/2023 2:25 pm, Jan Beulich wrote: > On 19.12.2023 15:19, Andrew Cooper wrote: >> On 19/12/2023 1:48 pm, Mykyta Poturai wrote: >>> This patch adds the ability for the device emulator to inject MSI >>> interrupts into guests. This is done by adding a new DM op to the device >>> model

Re: [PATCH v4 1/5] x86/HVM: split restore state checking from state loading

2023-12-19 Thread Roger Pau Monné
On Mon, Dec 18, 2023 at 03:39:55PM +0100, Jan Beulich wrote: > ..., at least as reasonably feasible without making a check hook > mandatory (in particular strict vs relaxed/zero-extend length checking > can't be done early this way). > > Note that only one of the two uses of "real" hvm_load() is

Re: [PATCH v4 5/5] x86/vIRQ: split PCI link load state checking from actual loading

2023-12-19 Thread Roger Pau Monné
On Mon, Dec 18, 2023 at 03:41:14PM +0100, Jan Beulich wrote: > Move the checking into a check hook, and add checking of the padding > fields as well. > > Signed-off-by: Jan Beulich Reviewed-by: Roger Pau Monné Thanks.

  1   2   >