Re: [PATCH V3 03/22] LoongArch: Add elf-related definitions

From: Huacai Chen
Date: Sun May 15 2022 - 07:47:38 EST


Hi, Xuerui,

On Sun, May 15, 2022 at 12:13 PM WANG Xuerui <kernel@xxxxxxxxxx> wrote:
>
> On 5/14/22 22:11, Huacai Chen wrote:
>
> > Hi, Xuerui,
> >
> > On Sat, May 14, 2022 at 9:29 PM WANG Xuerui <kernel@xxxxxxxxxx> wrote:
> >> Hi,
> >>
> >> Why this patch is from V3? I guess it's by mistake so would you re-send
> >> a proper v10?
> > This is just a typo, it is really V10.
> Okay, got it.
> >
> > Huacai
> >> On 5/14/22 16:03, Huacai Chen wrote:
> >>> Add elf-related definitions for LoongArch, including: EM_LOONGARCH,
> >>> KEXEC_ARCH_LOONGARCH, AUDIT_ARCH_LOONGARCH32, AUDIT_ARCH_LOONGARCH64
> >>> and NT_LOONGARCH_*.
> >>>
> >>> Signed-off-by: Huacai Chen <chenhuacai@xxxxxxxxxxx>
> >>> ---
> >>> include/uapi/linux/audit.h | 2 ++
> >>> include/uapi/linux/elf-em.h | 1 +
> >>> include/uapi/linux/elf.h | 5 +++++
> >>> include/uapi/linux/kexec.h | 1 +
> >>> scripts/sorttable.c | 5 +++++
> >>> 5 files changed, 14 insertions(+)
>
> The code changes all look good to me, only minor typographic nit: please
> change "elf" to "ELF".
OK, thanks,

Huacai
>
> With that:
>
> Reviewed-by: WANG Xuerui <git@xxxxxxxxxx>
>