Re: [PATCH Dovetail 6.16 v4 0/3] Dovetail 6.16: Fix some regressions introduced during last forward port

[Philippe Gerum <rpm@xenomai.org>]

Florian Bezdeka <florian.bezdeka@siemens.com> writes:

> Hi all,
>
> seems the last forward port to 6.16 introduced some regressions.
> Please review carefully, thanks!
>
> With that applied I have two open problems:
>   - Running dmesg kills the system, no idea yet
>     Requires at least 3 CPUs on my qemu system to reproduce.
>     Update: I know how to fix that, but not fully understood yet.
>     Update: Discussion ongoing, see ML.
>
>   - I'm able to reproduce [1] on 6.16 now
>     Update: This one is understood now. Xenomai 3 bug.
>     Tests already ongoing.
>     Update: Fix posted to ML and merged into Xenomai next.
>
> Cc: Philippe Gerum <rpm@xenomai.org>
> Cc: Jan Kiszka <jan.kiszka@siemens.com>
>
> [1] https://lore.kernel.org/xenomai/6fcb3f9830056d99161ea8f50a016b99d98a7c71.camel@siemens.com/T/
>
> Signed-off-by: Florian Bezdeka <florian.bezdeka@siemens.com>
> ---
> Changes in v4:
> - Patch 1: Define _TLF_KERNEL_FPU_PREEMPTED as an alias to 
>            TS_KERNEL_FPU_PREEMPTED
> - Link to v3: https://lore.kernel.org/r/20250820-wip-flo-fixes-for-dovetail-6-15-v3-0-ae4a674d8e10@siemens.com
>
> Changes in v3:
> - Rework patch 1: Move preemption marker to thread local flags
> - Added patch 2: Align arm64 and x86 fpu preemption code
> - Patch 3: was Patch 2 in v2, updated to match latest Linux code
> - Link to v2: https://lore.kernel.org/r/20250820-wip-flo-fixes-for-dovetail-6-15-v2-0-535e4fa355f7@siemens.com
>
> Changes in v2:
> - Patch 1: Reworked as discussed on the list
>            Following the same pattern as for arm64 
> - Link to v1: https://lore.kernel.org/r/20250820-wip-flo-fixes-for-dovetail-6-15-v1-0-d44f50cb4b0b@siemens.com
>
> ---
> Florian Bezdeka (3):
>       x86/fpu: dovetail: Fix FPU corruption
>       arm64: fpsimd: dovetail: Align with x86 implementation
>       irq_pipeline: Fix pipelining code for level triggered IRQs
>
>  arch/arm64/include/asm/thread_info.h | 10 +++++-----
>  arch/arm64/kernel/fpsimd.c           |  7 +++++--
>  arch/x86/include/asm/fpu/sched.h     | 17 ++++-------------
>  arch/x86/include/asm/fpu/types.h     | 12 ------------
>  arch/x86/include/asm/thread_info.h   | 10 ++++++----
>  arch/x86/kernel/fpu/core.c           | 13 +++++--------
>  kernel/irq/chip.c                    | 22 ++++++++++++++++++++--
>  7 files changed, 45 insertions(+), 46 deletions(-)
> ---
> base-commit: eec111a874f5899e1078ef785ccf29dff986e5a0
> change-id: 20250820-wip-flo-fixes-for-dovetail-6-15-1368f7f6e0b1
>
> Best regards,

Thanks Florian, merged.

-- 
Philippe.