Re: [PATCH v7 3/4] atomic: Add alignment check to instrumented atomic operations

[Geert Uytterhoeven]

Hi Finn,

On Tue, 13 Jan 2026 at 06:39, Finn Thain <fthain@xxxxxxxxxxxxxx> wrote:
> From: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
>
> Add a Kconfig option for debug builds which logs a warning when an
> instrumented atomic operation takes place that's misaligned.
> Some platforms don't trap for this.
>
> [ fthain: added __DISABLE_EXPORTS conditional and refactored as helper
> function. ]
>
> Cc: Sasha Levin <sashal@xxxxxxxxxx>
> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: Ingo Molnar <mingo@xxxxxxxxxx>
> Cc: Borislav Petkov <bp@xxxxxxxxx>
> Cc: Dave Hansen <dave.hansen@xxxxxxxxxxxxxxx>
> Cc: Ard Biesheuvel <ardb@xxxxxxxxxx>
> Cc: "H. Peter Anvin" <hpa@xxxxxxxxx>
> Link: https://lore.kernel.org/lkml/20250901093600.GF4067720@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/
> Link: https://lore.kernel.org/linux-next/df9fbd22-a648-ada4-fee0-68fe4325ff82@xxxxxxxxxxxxxx/
> Signed-off-by: Finn Thain <fthain@xxxxxxxxxxxxxx>
>
> ---
> Checkpatch.pl says...
> ERROR: Missing Signed-off-by: line by nominal patch author 'Peter Ziljstra <peterz@xxxxxxxxxxxxx>'

Alternatively, you can credit Peter using

    Suggested-by: Peter Zijlstra <peterz@xxxxxxxxxxxxx>

just before the Link-header pointing to his suggestion.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds