[PATCH v15 13/16] arm64/delay: enable testing smp_cond_load_relaxed_timeout()

[bot+bpf-ci at kernel.org]

> arm64/delay: enable testing smp_cond_load_relaxed_timeout()
>
> This enables the barrier tests to be built as a module.
>
> Cc: Catalin Marinas <catalin.marinas at arm.com>
> Cc: Will Deacon <will at kernel.org>
> Acked-by: Will Deacon <will at kernel.org>
> Signed-off-by: Ankur Arora <ankur.a.arora at oracle.com>

This isn't a bug, but half of the diff lands in
drivers/clocksource/arm_arch_timer.c, which has its own MAINTAINERS entry
(ARM ARCHITECTED TIMER, Marc Zyngier) and its own changelog prefix
convention (clocksource/drivers/arm_arch_timer: ...).

The subject uses only the arm64/delay prefix and the Cc/Ack lines name
only the arm64 maintainers, so the clocksource hunk is easy to miss when
the patch is routed or reviewed.

Would it help to either split the arm_arch_timer.c export into its own
clocksource-prefixed patch, or widen the subject/Cc so the clocksource
change is visible to its maintainer - and mention in the changelog that
the two symbols get pulled in via cpu_poll_relax()?

> diff --git a/arch/arm64/lib/delay.c b/arch/arm64/lib/delay.c
> index f08eacef2f0a..20bc2894b65b 100644
> --- a/arch/arm64/lib/delay.c
> +++ b/arch/arm64/lib/delay.c
> @@ -12,6 +12,7 @@
>  #include <linux/kernel.h>
>  #include <linux/module.h>
>  #include <linux/timex.h>
> +#include <kunit/visibility.h>
>  #include <asm/delay-const.h>
>
>  #include <clocksource/arm_arch_timer.h>
> @@ -30,6 +31,7 @@ u64 notrace __delay_cycles(void)
>  	guard(preempt_notrace)();
>  	return __arch_counter_get_cntvct_stable();
>  }
> +EXPORT_SYMBOL_IF_KUNIT(__delay_cycles);
>
>  void __delay(unsigned long cycles)
>  {
> diff --git a/drivers/clocksource/arm_arch_timer.c b/drivers/clocksource/arm_arch_timer.c
> index cc9a8b399004..227447b19b14 100644
> --- a/drivers/clocksource/arm_arch_timer.c
> +++ b/drivers/clocksource/arm_arch_timer.c
> @@ -28,6 +28,7 @@
>  #include <linux/acpi.h>
>  #include <linux/arm-smccc.h>
>  #include <linux/ptp_kvm.h>
> +#include <kunit/visibility.h>
>
>  #include <asm/arch_timer.h>
>  #include <asm/virt.h>
> @@ -893,6 +894,7 @@ bool arch_timer_evtstrm_available(void)
>  	 */
>  	return cpumask_test_cpu(raw_smp_processor_id(), &evtstrm_available);
>  }
> +EXPORT_SYMBOL_IF_KUNIT(arch_timer_evtstrm_available);
>
>  static struct arch_timer_kvm_info arch_timer_kvm_info;


---
AI reviewed your patch. Please fix the bug or email reply why it's not a bug.
See: https://github.com/kernel-patches/vmtest/blob/master/ci/claude/README.md

CI run summary: https://github.com/kernel-patches/bpf/actions/runs/33438155296