Re: [PATCH v2 1/2] cfi: Flip headers

Sami Tolvanen
2023-12-04T19:18:50Z

On Thu, Nov 30, 2023 at 5:43 AM Peter Zijlstra <peterz@infradead.org> wrote:
>
> Normal include order is that linux/foo.h should include asm/foo.h, CFI has it
> the wrong way around.
>
> Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
> ---
> arch/riscv/include/asm/cfi.h | 3 ++-
> arch/riscv/kernel/cfi.c | 2 +-
> arch/x86/include/asm/cfi.h | 3 ++-
> arch/x86/kernel/cfi.c | 4 ++--
> include/asm-generic/Kbuild | 1 +
> include/asm-generic/cfi.h | 5 +++++
> include/linux/cfi.h | 1 +
> 7 files changed, 14 insertions(+), 5 deletions(-)

Looks good to me, thanks!

Reviewed-by: Sami Tolvanen <samitolvanen@google.com>

Sami