On Mon, Apr 10, 2023 at 10:44 AM Florian Lehner <dev@xxxxxxxxxxx> wrote:
>
>
> -#ifndef arch_perf_out_copy_user
> -#define arch_perf_out_copy_user arch_perf_out_copy_user
> -
> -static inline unsigned long
> -arch_perf_out_copy_user(void *dst, const void *src, unsigned long n)
> -{
> - unsigned long ret;
> -
> - pagefault_disable();
> - ret = __copy_from_user_inatomic(dst, src, n);
> - pagefault_enable();
> -
> - return ret;
> -}
> -#endif
> +#define arch_perf_out_copy_user copy_from_user_nmi
This fails to build on aarch64. See BPF CI:
https://github.com/kernel-patches/bpf/actions/runs/4683969153/jobs/8299612449
Easy to fix, but I'd like to get an Ack from x86 folks for the 2nd patch.
I've applied the first patch to bpf-next, since it fixes the real
issue for kernels with hardened_usercopy=y
