Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
Message-ID: <fe537ea1-01a0-d963-ec8b-f73788e049b3@xen0n.name>
Date:   Wed, 1 Jun 2022 19:30:10 +0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:103.0) Gecko/20100101
 Thunderbird/103.0a1
Subject: Re: [PATCH V12 00/24] arch: Add basic LoongArch support
Content-Language: en-US
To:     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>
Cc:     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>,
        Huacai Chen <chenhuacai@gmail.com>,
        Guo Ren <guoren@kernel.org>, Xuerui Wang <kernel@xen0n.name>,
        Jiaxun Yang <jiaxun.yang@flygoat.com>,
        Stephen Rothwell <sfr@canb.auug.org.au>,
        Eric Biederman <ebiederm@xmission.com>,
        Ard Biesheuvel <ardb@kernel.org>
References: <20220601100005.2989022-1-chenhuacai@loongson.cn>
From:   WANG Xuerui <kernel@xen0n.name>
In-Reply-To: <20220601100005.2989022-1-chenhuacai@loongson.cn>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 6/1/22 17:59, Huacai Chen wrote:
> <snip>
>
> Huacai Chen(24):
>   irqchip: Adjust Kconfig for Loongson.
>   irqchip/loongson-liointc: Fix build error for LoongArch.
>   Documentation: LoongArch: Add basic documentations.
>   Documentation/zh_CN: Add basic LoongArch documentations.
>   LoongArch: Add elf-related definitions.
>   LoongArch: Add writecombine support for drm.
>   LoongArch: Add build infrastructure.
>   LoongArch: Add CPU definition headers.
>   LoongArch: Add atomic/locking headers.
>   LoongArch: Add other common headers.
>   LoongArch: Add boot and setup routines.
>   LoongArch: Add exception/interrupt handling.
>   LoongArch: Add process management.
>   LoongArch: Add memory management.
>   LoongArch: Add system call support.
>   LoongArch: Add signal handling support.
>   LoongArch: Add elf and module support.
>   LoongArch: Add misc common routines.
>   LoongArch: Add some library functions.
>   LoongArch: Add PCI controller support.
>   LoongArch: Add VDSO and VSYSCALL support.
>   LoongArch: Add multi-processor (SMP) support.
>   LoongArch: Add Non-Uniform Memory Access (NUMA) support.
>   LoongArch: Add Loongson-3 default config file.
>
> Signed-off-by: Huacai Chen <chenhuacai@loongson.cn>

I have gone through every of these patches, and they now all look good 
to me. Thanks for all the revisions and cooperation!

I remember several more people have left comments on the v11 code; Eric 
commented on the signal handling bits and Ard may have something to say 
on the UEFI part. The first two cleanup patches to irqchip are 
previously sent in a separate series, and they are already trivial 
enough, but Marc may want to give the ack here too, for the patches to 
go in via the asm-generic tree.

At this time, although the port currently cannot work on any real 
hardware, the value is mostly for unblocking downstream development 
starting from the libc's, as the userspace ABI has been stable for 
several months already and there's probably no point in delaying even 
further. I've been working on the Gentoo/LoongArch port since August 
2021 and experienced every ABI break since then, so I probably know 
enough to make this statement.

I think, with the userspace ABI effectively stable already, and other 
implementation details available for continued improvement, we could 
make it into this merge window after all. People, please take a final 
quick look at this; your opinions and/or review and acks are appreciated.