Received: by 2002:a19:771d:0:0:0:0:0 with SMTP id s29csp1237240lfc; Wed, 1 Jun 2022 12:40:02 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz6Vv4NkNbIudn4wIM2Ez/afkYz1bxTBM/MvnjOkaa43RpayJhrN0K+akQ4Ccb6bNkerVYJ X-Received: by 2002:a17:90a:c202:b0:1e2:e772:5f0a with SMTP id e2-20020a17090ac20200b001e2e7725f0amr998314pjt.109.1654112402247; Wed, 01 Jun 2022 12:40:02 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1654112402; cv=none; d=google.com; s=arc-20160816; b=MBPZn/yURUI3COPQna3BHj399YL+rPypa9QUZ1eGe+qj8yfG6hoO52PIBXK6nOO6EC oes+ZD+MPwltqSyTT2Q36nelvsfrE4md8K73mB3/wk1Z94hdxeR2PHEJ8qTDR4UnIsGP yejqGs35ipVfnQzZB0nCE70kNiCqVwq6jK5S5lS0DBwB/HlYVZluUZv9fkTPfxvQ15Ic gD0BQJ0edtHcjMb15H9rec5+eGlpHJJknGaPQSgEVsp1UNGU5Z+2Db0xe1OSYENE6b0o r8VDZH00hIIBzBcm/GPcblOzzT2dPdTDl6990/FlEC71GbruA4Nxf8WZhXcN3g9Zv5a4 XBDg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=45FoDcmgt2AQbjbnjqLI2GikHs8r1Fd1zyPAMDYDywI=; b=JBUSh9Xfruh1Hsdmqux1nnkkPX1C0h27j1HXHJ5iwPpaVEtS0HGAxjuEbW13ps9dhK S9M/SfcnuzEY+6Ekfyzdw1/qvfdAhOtCuOMH/AGMcSVtmGMX47A/3uhCsck2Ekal8pu3 Dab8xrj1AMxhM+14nnedhbucPh17SyrD/kHGwLQylrnkHPln//op+KzoG/CJQXdaWUNV v+Rpr5jb0WtGpYEMdu02ZALDgvhsy+TUANxoNSBzGrdcdoorDv2Y0DcUtbEs1RMbH3rW ENc0eX7tVyGKuaq1GjJmbgb/WS/ng1C6QvAex6HiC/VseFhgGL1kuoKI4kJhD3tRLltM SWKA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@xen0n.name header.s=mail header.b=rCSys+WS; spf=softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net. [23.128.96.19]) by mx.google.com with ESMTPS id d7-20020a630e07000000b0039905cd1790si3405001pgl.579.2022.06.01.12.40.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 01 Jun 2022 12:40:02 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@xen0n.name header.s=mail header.b=rCSys+WS; spf=softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id D9DF45F6E; Wed, 1 Jun 2022 12:06:45 -0700 (PDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1356183AbiFAQok (ORCPT + 99 others); Wed, 1 Jun 2022 12:44:40 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38684 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1356180AbiFAQog (ORCPT ); Wed, 1 Jun 2022 12:44:36 -0400 Received: from mailbox.box.xen0n.name (mail.xen0n.name [115.28.160.31]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id F17D89E9DA; Wed, 1 Jun 2022 09:44:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=xen0n.name; s=mail; t=1654101870; bh=D86uvLtUnid4ac7RPV6LXcQ6KwoBx2mBLbHHxLqwrus=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=rCSys+WSoaWiFQDm31CP75+HYXB6/F1uJSD8F2HZfW31dKUPSaK3hTAq2ub1Skgxq 4F+g0b2xTlO92waMrhrZ/d/Zv2R5bQVlDwHuk33MfL+SrYXR0/2yCTLOVQJhNTjTcE UB+cQ/tIApWLxneHte7BXvPhqhkz3K4LQXZagY88= Received: from [192.168.9.172] (unknown [101.88.28.48]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mailbox.box.xen0n.name (Postfix) with ESMTPSA id 5D2C460104; Thu, 2 Jun 2022 00:44:30 +0800 (CST) Message-ID: <47b559c0-b1e8-e800-0491-2431e2083dad@xen0n.name> Date: Thu, 2 Jun 2022 00:44:30 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:103.0) Gecko/20100101 Thunderbird/103.0a1 Subject: Re: [musl] Re: [GIT PULL] asm-generic changes for 5.19 Content-Language: en-US To: Ard Biesheuvel , Arnd Bergmann Cc: WANG Xuerui , Huacai Chen , linux-arch , GNU C Library , Yoshinori Sato , Peter Zijlstra , Marc Zyngier , Masahiro Yamada , musl@lists.openwall.com, Linux Kernel Mailing List , Jiaxun Yang , ACPI Devel Maling List , Jianmin Lv , linux-pci , Linus Torvalds , Huacai Chen References: <358025d1-28e6-708b-d23d-3f22ae12a800@xen0n.name> <832c3ae8-6c68-db2c-2c7f-0a5cd3071543@xen0n.name> From: WANG Xuerui In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-3.2 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,NICE_REPLY_A,RDNS_NONE,SPF_HELO_NONE, T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Ard, On 6/2/22 00:01, Ard Biesheuvel wrote: > On Wed, 1 Jun 2022 at 09:41, Arnd Bergmann wrote: >> On Wed, Jun 1, 2022 at 7:52 AM WANG Xuerui wrote: >>> On 6/1/22 00:01, Huacai Chen wrote: >>>> https://git.kernel.org/pub/scm/linux/kernel/git/chenhuacai/linux-loongson.git/log/?h=loongarch-next >>>> has been updated. Now this branch droped irqchip drivers and pci >>>> drivers. But the existing irqchip drivers need some small adjustment >>>> to avoid build errors [1], and I hope Marc can give an Acked-by. >>>> Thanks. >>>> >>>> This branch can be built with defconfig and allmodconfig (except >>>> drivers/platform/surface/aggregator/controller.c, because it requires >>>> 8bit/16bit cmpxchg, which I was told to remove their support). >>>> >>>> [1] https://lore.kernel.org/lkml/e7cf33a170d0b4e98e53744f60dbf922@kernel.org/T/#t >>> I see the loongarch-next HEAD has been updated and it's now purely arch >>> changes aside from the two trivial irqchip cleanups. Some other changes >>> to the v11 patchset [1] are included, but arguably minor enough to not >>> invalidate previous Reviewed-by tags. >> Very nice! I don't see exactly how the previous build bugs were addressed, >> but I can confirm that this version builds. Regarding the two irqchip patches, >> 621e7015b529 ("irqchip/loongson-liointc: Fix build error for LoongArch") is >> a good way to work around the mips oddity, and I have no problem taking >> that through the asm-generic tree. The other one, f54b4a166023 ("irqchip: >> Adjust Kconfig for Loongson"), looks mostly unnecessary, and I think only >> the LOONGSON_HTPIC change should be included here, while I would >> leave out the COMPILE_TEST changes and instead have the driver >> changes take care of making it possible to keep building it on x86, possibly >> doing >> >> depends on MACH_LOONGSON64 || (COMPILE_TEST && ACPI) >> >> in the future, after the loongarch64 ACPI support is merged. >> >>> After some small tweaks: >>> >>> - adding "#include " to arch/loongarch/include/asm/ptrace.h, >>> - adding an arch/loongarch/include/uapi/asm/bpf_perf_event.h with the >>> same content as arch/arm64's, and >>> - adding "depends on ARM64 || X86" to >>> drivers/platform/surface/aggregator/Kconfig, >>> >>> the current loongarch-next HEAD (commit >>> 36552a24f70d21b7d63d9ef490561dbdc13798d7) now passes allmodconfig build >>> (with CONFIG_WERROR disabled; my Gentoo-flavored gcc-12 seems to emit >>> warnings on a few drivers). >> The only one of these issues that I see is the surface aggregator one. >> I think we can address all three as follow-up fixes after -rc1 if the port >> gets merged and these are still required. >> >>> The majority of userspace ABI has been stable for a few months already, >>> after the addition of orig_a0 and removal of newfstatat; the necessary >>> changes to switch to statx are already reviewed [2] / merged [3], and >>> have been integrated into the LoongArch port of Gentoo for a while. Eric >>> looked at the v11 and gave comments, and changes were made according to >>> the suggestions, but it'd probably better to get a proper Reviewed-by. >> Right. >> >>> Among the rest of patches, I think maybe the EFI/boot protocol part >>> still need approval/ack from the EFI maintainer. However because the >>> current port isn't going to be able to run on any real hardware, maybe >>> that part could be done later; I'm not sure if the unacknowledged EFI >>> bits should be removed as well. >> Ard, do you have any last comments on this? >> > It would be nice if the questions I raised against the previous > revision (v11) were addressed (or at least answered) first. In > general, I think this is feeling a bit rushed and IMHO we should > probably defer this to the next cycle. Actually I think Huacai did reply to your review on v11: https://lore.kernel.org/all/CAAhV-H7KAg8RxN7M=WiOOh0fDhEKTyqrwp6V-SC0cyR0iMrdeg@mail.gmail.com/. It's a bit unfortunate that he probably didn't justify some of the approaches enough, and it's especially unfortunate that some of the points (like maybe the kernel version string in the EFI stub header) are result of their internal discussion, which I presume to be especially hard to change due to their particularly worrying corporate dynamics... But again, my point is that the userspace ABI in particular is *not* rushed -- it has been brewing since v1 of the port which is already several months ago, and multiple distro-building efforts are already underway. We (LoongArch distro packagers) want to freeze the userspace ABI so that many downstream efforts wouldn't be blocked by the merging of kernel port. As the boot protocol is technically not part of the userspace ABI that toolchains care about, and we already know it'll be a rather standards-compliant UEFI implementation even if this part gets dropped for brewing one more cycle, would taking this part out work for you?