From mboxrd@z Thu Jan 1 00:00:00 1970 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on inbox.vuxu.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED, RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.4 Received: (qmail 2900 invoked from network); 30 May 2022 15:56:17 -0000 Received: from mother.openwall.net (195.42.179.200) by inbox.vuxu.org with ESMTPUTF8; 30 May 2022 15:56:17 -0000 Received: (qmail 28507 invoked by uid 550); 30 May 2022 15:56:15 -0000 Mailing-List: contact musl-help@lists.openwall.com; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-ID: Reply-To: musl@lists.openwall.com Received: (qmail 28469 invoked from network); 30 May 2022 15:56:14 -0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1653926162; bh=vZNOahuquc61LYGJE+zDOh1Tfc1eF8K3ftYrqLR3Jlg=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=dgYj/Yzu6xQgvbDr+aTMPj3JC11vuXrprxRvhYSLFKVR2F8er+ja/ylKcZNMSUjkT WGMwvpDTgm8oxIMmajdhHZfNdq8Wm9Bi7f9UP9icxcb6p6BvkjZZIwu+kLGCiKbXZd 39HUgv2IbaKQ1xEbQG9jYq0t18S9YjJKZGMtS7q80RHTpBZ1ZIm5oGDK7FiufTCUr/ U3+enbtWDaCLOR8mx60jcCGOXkqTnSOfG0Po2CrZysqy1vXk7emgiIsopVywnfNwnl fDEDzpuI4HVb4MMGALQW4bQRnPTk1+1hSmuGQ5xhW1ulVyCcy9FnGjXTY7D+7fXsPc IuUCHkGo1VgWw== X-Gm-Message-State: AOAM533+tt2xCOcJFT/WyZ4Y/SZ+FKHNVYUy/Oia+d2zAMQ2f9wwIdOF YdDLv0GprqAf4MdVBCGbqTIc57C7OEdNls3eK7w= X-Google-Smtp-Source: ABdhPJw8YN2D7OioGQU3VBmoUMbh0gioLp3yRY1XdaTroJXM/gLDD3ji8AXoRaouQ2fK2WQ4/43SP4bwYsUoMcacvmc= X-Received: by 2002:a25:4f0a:0:b0:64f:6a76:3d8f with SMTP id d10-20020a254f0a000000b0064f6a763d8fmr43939287ybb.134.1653926161185; Mon, 30 May 2022 08:56:01 -0700 (PDT) MIME-Version: 1.0 References: <358025d1-28e6-708b-d23d-3f22ae12a800@xen0n.name> In-Reply-To: <358025d1-28e6-708b-d23d-3f22ae12a800@xen0n.name> From: Arnd Bergmann Date: Mon, 30 May 2022 17:55:43 +0200 X-Gmail-Original-Message-ID: Message-ID: To: musl@lists.openwall.com Cc: Huacai Chen , WANG Xuerui , Linus Torvalds , linux-arch , GNU C Library , Yoshinori Sato , Peter Zijlstra , Marc Zyngier , Masahiro Yamada , Linux Kernel Mailing List , Jiaxun Yang , ACPI Devel Maling List , Jianmin Lv , linux-pci , Ard Biesheuvel , Huacai Chen Content-Type: text/plain; charset="UTF-8" Subject: Re: [musl] Re: [GIT PULL] asm-generic changes for 5.19 On Mon, May 30, 2022 at 5:00 PM WANG Xuerui wrote: > On 5/30/22 21:01, Huacai Chen wrote: > > Thanks for the recognition from both of you; it is my honor and pleasure > to contribute to the LoongArch port and to Linux in general. > > As I'm still not entirely satisfied with my kernel development skills, > plus my day job is not kernel-related nor Loongson/LoongArch-related at > all, listing me as reviewer should be enough for now. I will take care > of the arch as long as I have the hardware. Thanks, sounds good to me. > BTW, there were already several breakages when rebasing the previous > revision (I believe it's commit 215da6d2dac0 ("MAINTAINERS: Add > maintainer information for LoongArch")) on top of linus' tree. Right, at least most of these should be fairly easy to address by disabling the corresponding features. For the allmodconfig build, I see some warnings that are introduced in gcc-12.1 across all architectures, and those can be ignored for now. Some of the errors already have fixes on top of the 215da6d2dac0 commit, but some of the other commits look like we should leave them out here. I also see some conflicts between local symbol definitions and device drivers such as arch/loongarch/include/asm/loongarch.h:240:29: note: previous definition of 'csr_writel' with type 'void(u32, u32)' {aka 'void(unsigned int, unsigned int)'} 240 | static __always_inline void csr_writel(u32 val, u32 reg) | ^~~~~~~~~~ drivers/media/platform/amphion/vpu_core.h:10:5: error: conflicting types for 'csr_readl'; have 'u32(struct vpu_core *, u32)' {aka 'unsigned int(struct vpu_core *, unsigned int)'} and drivers/usb/cdns3/cdns3-imx.c:85: error: "PS_MASK" redefined [-Werror] I would suggest renaming the loongarch specific symbols here, though we may want to also change those drivers to use less generic identifiers. > Now I see > the loongarch-next HEAD is already rebased on top of what I believe to > be the current main branch, however I vaguely remember that it's not > good to base one's patches on top of "some random commit", so I wonder > whether the current branch state is appropriate for a PR? You are correct, a pull request should always be based on an -rc, orat least have the minimum set of dependencies. The branch was previously based on top of the spinlock implementation, which is still the best place to start here. Arnd