Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <689e103a-07f0-a55c-9b34-073cf81f84c0@xen0n.name>
Date:   Sun, 15 May 2022 12:13:38 +0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.0a1
Subject: Re: [PATCH V3 03/22] LoongArch: Add elf-related definitions
Content-Language: en-US
To:     Huacai Chen <chenhuacai@gmail.com>, WANG Xuerui <kernel@xen0n.name>
Cc:     Huacai Chen <chenhuacai@loongson.cn>,
        Arnd Bergmann <arnd@arndb.de>,
        Andy Lutomirski <luto@kernel.org>,
        Thomas Gleixner <tglx@linutronix.de>,
        Peter Zijlstra <peterz@infradead.org>,
        Andrew Morton <akpm@linux-foundation.org>,
        David Airlie <airlied@linux.ie>,
        Jonathan Corbet <corbet@lwn.net>,
        Linus Torvalds <torvalds@linux-foundation.org>,
        linux-arch@vger.kernel.org, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org, Xuefeng Li <lixuefeng@loongson.cn>,
        Yanteng Si <siyanteng@loongson.cn>,
        Guo Ren <guoren@kernel.org>,
        Jiaxun Yang <jiaxun.yang@flygoat.com>,
        Stephen Rothwell <sfr@canb.auug.org.au>
References: <20220514080402.2650181-1-chenhuacai@loongson.cn>
 <20220514080402.2650181-4-chenhuacai@loongson.cn>
 <25efb0c1-f2e7-0052-c925-08dd778d7ad7@xen0n.name>
 <CAAhV-H6rBtmTQTpxdZtk26sP9SJui5dpv2e-2ZUyWTtpxpf9FQ@mail.gmail.com>
From:   WANG Xuerui <kernel@xen0n.name>
In-Reply-To: <CAAhV-H6rBtmTQTpxdZtk26sP9SJui5dpv2e-2ZUyWTtpxpf9FQ@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 5/14/22 22:11, Huacai Chen wrote:

> Hi, Xuerui,
>
> On Sat, May 14, 2022 at 9:29 PM WANG Xuerui <kernel@xen0n.name> 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@loongson.cn>
>>> ---
>>>    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".

With that:

Reviewed-by: WANG Xuerui <git@xen0n.name>