[v5,0/2] powerpc module arch checks

Message ID	20221031120733.3956781-1-npiggin@gmail.com (mailing list archive)
Headers	show Return-Path: <linux-modules-owner@kernel.org> From: Nicholas Piggin <npiggin@gmail.com> To: linux-modules@vger.kernel.org, linuxppc-dev@lists.ozlabs.org Cc: Nicholas Piggin <npiggin@gmail.com>, linux-kernel@vger.kernel.org, Michael Ellerman <mpe@ellerman.id.au>, Jessica Yu <jeyu@kernel.org>, Luis Chamberlain <mcgrof@kernel.org> Subject: [PATCH v5 0/2] powerpc module arch checks Date: Mon, 31 Oct 2022 22:07:31 +1000 Message-Id: <20221031120733.3956781-1-npiggin@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	powerpc module arch checks \| expand [v5,0/2] powerpc module arch checks [v5,1/2] module: add module_elf_check_arch for module-specific checks [v5,2/2] powerpc/64: Add module check for ELF ABI version

Message ID

20221031120733.3956781-1-npiggin@gmail.com (mailing list archive)

Headers

From: Nicholas Piggin <npiggin@gmail.com>
To: linux-modules@vger.kernel.org, linuxppc-dev@lists.ozlabs.org
Cc: Nicholas Piggin <npiggin@gmail.com>, linux-kernel@vger.kernel.org,
        Michael Ellerman <mpe@ellerman.id.au>,
        Jessica Yu <jeyu@kernel.org>,
        Luis Chamberlain <mcgrof@kernel.org>
Subject: [PATCH v5 0/2] powerpc module arch checks
Date: Mon, 31 Oct 2022 22:07:31 +1000
Message-Id: <20221031120733.3956781-1-npiggin@gmail.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: bulk

Series

powerpc module arch checks | expand

Message

Nicholas Piggin Oct. 31, 2022, 12:07 p.m. UTC

These slipped through the cracks. Picking them up again...

The story so far is that Jessica gave us the new scheme in patch 1,
now slightly rebased. Patch 2 implements the additional check
that powerpc wants which originally came from Michael, and has been
updated to the new approach.

This was previously attached to the ELFv2 build option for big-endian
kernels, but it can go ahead of that option.

Just checking everybody is still okay with the code and their SOBs,
and Luis if you would be okay for patch 1 to be merged via powerpc or
prefer to take it in the module tree (or maybe you object to the
code in the first place).

Thanks,
Nick

Nicholas Piggin (2):
  module: add module_elf_check_arch for module-specific checks
  powerpc/64: Add module check for ELF ABI version

 arch/powerpc/kernel/module.c | 17 +++++++++++++++++
 include/linux/moduleloader.h |  3 +++
 kernel/module/main.c         | 10 ++++++++++
 3 files changed, 30 insertions(+)

Comments

Luis Chamberlain Nov. 3, 2022, 12:01 a.m. UTC | #1

On Mon, Oct 31, 2022 at 10:07:31PM +1000, Nicholas Piggin wrote:
> Luis if you would be okay for patch 1 to be merged via powerpc or
> prefer to take it in the module tree (or maybe you object to the
> code in the first place).

Looks good to me, and nothing on my radar which would cause a conflict
so happy for you to take it via powerpc or I can take it and apply it
right away to tricke / get tested on linux-next by tomorrow.

Let me know.

  Luis

Michael Ellerman Nov. 3, 2022, 9:35 a.m. UTC | #2

Luis Chamberlain <mcgrof@kernel.org> writes:
> On Mon, Oct 31, 2022 at 10:07:31PM +1000, Nicholas Piggin wrote:
>> Luis if you would be okay for patch 1 to be merged via powerpc or
>> prefer to take it in the module tree (or maybe you object to the
>> code in the first place).
>
> Looks good to me, and nothing on my radar which would cause a conflict
> so happy for you to take it via powerpc or I can take it and apply it
> right away to tricke / get tested on linux-next by tomorrow.
>
> Let me know.

Thanks. I guess it's mostly a nop on other architectures, so probably
makes sense for me to take it.

cheers