Received: by 2002:a05:6a10:1287:0:0:0:0 with SMTP id d7csp3730791pxv; Mon, 19 Jul 2021 07:27:18 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzq4eCFKoQTCx14PPHyDIELLJikV8Rn+L9tJZrSYbFfn6+glkCZuyI+YflrSn4evNlMY4MR X-Received: by 2002:a50:ce45:: with SMTP id k5mr15675472edj.168.1626704837921; Mon, 19 Jul 2021 07:27:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1626704837; cv=none; d=google.com; s=arc-20160816; b=PWVrxD9YZn4duPROS5asdo1GqUUyVs+DwXS52KtR0fLhUHW3FPKEv2wueRYXJD2/JK mXBt5MgoUci9tFujWtllwxhvXN5XrPeTvhe4swPypmC3TvcsBHuww90saTfxFIAXqQZP Fhjq+hOfn1ZYkTkqsUVjEldu1bL4pq+ZclBMj2j1ufUxP+duxarcyij/MnSBfQJck3Eo MI6CwwD2O75Xg/6amVphr7FMiUVVxklQ+UxJJI9qf5WmdmnDEbhprEAlCyZMuwCFbj0y 2s4aVaP56B0vkeSHJQ47veFirciBr6dxXVBRdRawYXx8w5dXhPTCXVViobhtPVxQ8C54 JSnw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-transfer-encoding :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=ItRFoeoYK07XW/z7kw93QqTzs0UQ5GzRO74k6L3Wg68=; b=Zq0KeGmLPA9LnxTK5LmMIs+DDYIF2Pltm8AtJ2l0Qp2K9eO99QcvGh/EhwnPpyAkpc KCS5foV12ShRyaEwqv9C3x1vO0UVgFT9Ok9e1fH+lOh1gM+5bExXdb+K/45JnTteEA8W ThjFy5KJYEXk5xH8ayfn+wQvEnnXsc1NT1ZpjmJ/1CNDjd6nTLV3x9Wgazoq3Qt6Z/W2 +rot24hlLSgz8Xx3QO6vQg0lqegKYwwQKG9fm5nJLXvGUspt6fX/K65vybfWvNwhwhoH s8LMlMpkh0ry0K4vLmnASsXYhw3Drcd/tlWvHDKW3W4S21wcoUyYJwurfetDF3N3uXdm K/lw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@atlas.cz header.s=mail header.b=cuMQsYui; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=atlas.cz Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id jg33si21534078ejc.650.2021.07.19.07.26.54; Mon, 19 Jul 2021 07:27:17 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@atlas.cz header.s=mail header.b=cuMQsYui; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=atlas.cz Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241123AbhGSNoX (ORCPT + 99 others); Mon, 19 Jul 2021 09:44:23 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51406 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S241087AbhGSNoW (ORCPT ); Mon, 19 Jul 2021 09:44:22 -0400 X-Greylist: delayed 113 seconds by postgrey-1.37 at lindbergh.monkeyblade.net; Mon, 19 Jul 2021 06:51:03 PDT Received: from gmmr1.centrum.cz (gmmr1.centrum.cz [IPv6:2a00:da80:0:502::2]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 34952C061574 for ; Mon, 19 Jul 2021 06:51:03 -0700 (PDT) Received: from gmmr-4.centrum.cz (unknown [10.255.254.15]) by gmmr1.centrum.cz (Postfix) with ESMTP id 7CCF48000095; Mon, 19 Jul 2021 16:23:16 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=atlas.cz; s=mail; t=1626704596; bh=ItRFoeoYK07XW/z7kw93QqTzs0UQ5GzRO74k6L3Wg68=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=cuMQsYuiZty67QPen/57wYoCdgIXYrJ/yy0L22XVTfMtQlqrGg5l58IKIzoCpt2Xz naM77TL9eXT4ROqtBBN+F2+r4L0R0XUJ+dVxd5s6duHplCgLCWO6X8vdh1S5K3xcz5 uro+CIs7YZAI75GIdiZrPcXQQmpfPspGNA19fTeI= Received: from vm1.excello.cz (vm1.excello.cz [IPv6:2001:67c:1591::3]) by gmmr-4.centrum.cz (Postfix) with QMQP id 786791000B3B9; Mon, 19 Jul 2021 16:23:16 +0200 (CEST) Received: from vm1.excello.cz by vm1.excello.cz (VF-Scanner: Clear:RC:0(2a00:da80:1:502::7):SC:0(-1.5/5.0):CC:0:; processed in 1.0 s); 19 Jul 2021 14:23:16 +0000 X-VF-Scanner-ID: 20210719142315.468404.8314.vm1.excello.cz.0 X-Spam-Status: No, hits=-1.5, required=5.0 Received: from gmmr-2.centrum.cz (2a00:da80:1:502::7) by out2.virusfree.cz with ESMTPS (TLSv1.3, TLS_AES_256_GCM_SHA384); 19 Jul 2021 16:23:15 +0200 Received: from gm-smtp10.centrum.cz (unknown [10.255.254.38]) by gmmr-2.centrum.cz (Postfix) with ESMTP id 5B87B20055F6C; Mon, 19 Jul 2021 16:23:15 +0200 (CEST) Received: from arkam (unknown [94.113.86.190]) by gm-smtp10.centrum.cz (Postfix) with ESMTPA id D6F03C063A2A; Mon, 19 Jul 2021 16:23:14 +0200 (CEST) Date: Mon, 19 Jul 2021 16:23:13 +0200 From: Petr =?utf-8?B?VmFuxJtr?= To: Khem Raj Cc: Denys Vlasenko , Ingo Molnar , linux-kernel@vger.kernel.org, Alexander Shishkin , Arnaldo Carvalho de Melo , Chanho Park , Greg Kroah-Hartman , Jiri Olsa , John Garry , Josh Poimboeuf , Kate Stewart , Leo Yan , Mark Rutland , Martin Schwidefsky , Masami Hiramatsu , Mathieu Poirier , Matt Redfearn , Mike Leach , Namhyung Kim , Peter Zijlstra , Philippe Ombredanne , Stephane Eranian , Suzuki K Poulose , Thomas Gleixner , Vasily Gorbik , Will Deacon Subject: Re: Unknown __always_inline in linux/swab.h Message-ID: References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jul 16, 2021 at 08:41:57AM -0700, Khem Raj wrote: > On Fri, Jul 16, 2021 at 7:27 AM Petr Vaněk wrote: > > > > Hi all, > > > > I cannot build the kernel on my box with musl libc since release 5.12, > > see the kernel 5.13.1 build log below. Bisect points to the commit > > 1d509f2a6ebc ("x86/insn: Support big endian cross-compiles") merged to > > 5.12-rc1, but it probably only triggers the issue from different place. > > > > I searched through the mailing list [1] and it seems that people face to > > a similar issue with perf tool as well [2-5]. Related issues were also > > reported in Gentoo [6] and alpinelinux [7]. Both distributions basically > > fixed it with the patch from [3] which works for me and it is the best > > solution, in my opinion. I believe it should be applied to the upstream > > and backported to 5.12 and 5.13. > > > > Systems with glibc are not affected because glibc defines its own > > __always_inline in sys/cdefs.h which is indirectly included with stdio.h > > and/or stdlib.h in top of tools/objtool/arch/x86/decode.c file. > > > > Note: I CCed you because your address was mentioned in [2-5] or in > > bisected commit. > > > > does https://lore.kernel.org/lkml/20181213180129.1373-1-raj.khem@gmail.com/ > work for you ? Yes, it does, however, I still think the https://lore.kernel.org/lkml/1514973450-12447-1-git-send-email-matt.redfearn@mips.com/ patch is better, because linux/stddef.h includes linux/compiler_types.h, which should be enough for linux/swab.h. However, your patch is definitely safer if linux/compiler.h is needed for some reason I don't see right now. > It was not applied but there were no comments on rejecting it either, > so I guess it either fell through cracks somehow. > > > Thanks, > > Petr > > > > References: > > > > [1] https://lore.kernel.org/lkml/?q=swab+__always_inline > > [2] https://lore.kernel.org/lkml/20210204005810.GZ854763@kernel.org/ > > [3] https://lore.kernel.org/lkml/1514973450-12447-1-git-send-email-matt.redfearn@mips.com/ > > [4] https://lore.kernel.org/lkml/20181213180129.1373-1-raj.khem@gmail.com/ > > [5] https://lore.kernel.org/lkml/20201210144844.72580-1-chanho61.park@samsung.com/ > > [6] https://bugs.gentoo.org/791364 > > [7] https://gitlab.alpinelinux.org/alpine/aports/-/issues/11119 > > > > Build log: > > > > make[1]: Entering directory '/var/tmp/portage/linux' > > GEN Makefile > > DESCEND objtool > > CALL /usr/src/linux-5.13.1-gentoo/scripts/atomic/check-atomics.sh > > CALL /usr/src/linux-5.13.1-gentoo/scripts/checksyscalls.sh > > CC /var/tmp/portage/linux/tools/objtool/arch/x86/decode.o > > CC /var/tmp/portage/linux/tools/objtool/str_error_r.o > > CC /var/tmp/portage/linux/tools/objtool/librbtree.o > > In file included from /usr/include/linux/byteorder/little_endian.h:13, > > from /usr/include/asm/byteorder.h:5, > > from /usr/src/linux-5.13.1-gentoo/tools/arch/x86/include/asm/insn.h:10, > > from arch/x86/decode.c:10: > > /usr/include/linux/swab.h:136:23: error: expected ';' before 'unsigned' > > 136 | static __always_inline unsigned long __swab(const unsigned long y) > > | ^~~~~~~~~ > > | ; > > /usr/include/linux/swab.h:171:8: error: unknown type name '__always_inline' > > 171 | static __always_inline __u16 __swab16p(const __u16 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/swab.h:171:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__swab16p' > > 171 | static __always_inline __u16 __swab16p(const __u16 *p) > > | ^~~~~~~~~ > > /usr/include/linux/swab.h:184:8: error: unknown type name '__always_inline' > > 184 | static __always_inline __u32 __swab32p(const __u32 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/swab.h:184:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__swab32p' > > 184 | static __always_inline __u32 __swab32p(const __u32 *p) > > | ^~~~~~~~~ > > /usr/include/linux/swab.h:197:8: error: unknown type name '__always_inline' > > 197 | static __always_inline __u64 __swab64p(const __u64 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/swab.h:197:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__swab64p' > > 197 | static __always_inline __u64 __swab64p(const __u64 *p) > > | ^~~~~~~~~ > > /usr/include/linux/swab.h:252:23: error: expected ';' before 'void' > > 252 | static __always_inline void __swab32s(__u32 *p) > > | ^~~~~ > > | ; > > /usr/include/linux/swab.h:265:23: error: expected ';' before 'void' > > 265 | static __always_inline void __swab64s(__u64 *p) > > | ^~~~~ > > | ; > > In file included from /usr/include/asm/byteorder.h:5, > > from /usr/src/linux-5.13.1-gentoo/tools/arch/x86/include/asm/insn.h:10, > > from arch/x86/decode.c:10: > > /usr/include/linux/byteorder/little_endian.h:44:8: error: unknown type name '__always_inline' > > 44 | static __always_inline __le64 __cpu_to_le64p(const __u64 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:44:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_le64p' > > 44 | static __always_inline __le64 __cpu_to_le64p(const __u64 *p) > > | ^~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:48:8: error: unknown type name '__always_inline' > > 48 | static __always_inline __u64 __le64_to_cpup(const __le64 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:48:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__le64_to_cpup' > > 48 | static __always_inline __u64 __le64_to_cpup(const __le64 *p) > > | ^~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:52:8: error: unknown type name '__always_inline' > > 52 | static __always_inline __le32 __cpu_to_le32p(const __u32 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:52:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_le32p' > > 52 | static __always_inline __le32 __cpu_to_le32p(const __u32 *p) > > | ^~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:56:8: error: unknown type name '__always_inline' > > 56 | static __always_inline __u32 __le32_to_cpup(const __le32 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:56:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__le32_to_cpup' > > 56 | static __always_inline __u32 __le32_to_cpup(const __le32 *p) > > | ^~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:60:8: error: unknown type name '__always_inline' > > 60 | static __always_inline __le16 __cpu_to_le16p(const __u16 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:60:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_le16p' > > 60 | static __always_inline __le16 __cpu_to_le16p(const __u16 *p) > > | ^~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:64:8: error: unknown type name '__always_inline' > > 64 | static __always_inline __u16 __le16_to_cpup(const __le16 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:64:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__le16_to_cpup' > > 64 | static __always_inline __u16 __le16_to_cpup(const __le16 *p) > > | ^~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:68:8: error: unknown type name '__always_inline' > > 68 | static __always_inline __be64 __cpu_to_be64p(const __u64 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:68:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_be64p' > > 68 | static __always_inline __be64 __cpu_to_be64p(const __u64 *p) > > | ^~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:72:8: error: unknown type name '__always_inline' > > 72 | static __always_inline __u64 __be64_to_cpup(const __be64 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:72:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__be64_to_cpup' > > 72 | static __always_inline __u64 __be64_to_cpup(const __be64 *p) > > | ^~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:76:8: error: unknown type name '__always_inline' > > 76 | static __always_inline __be32 __cpu_to_be32p(const __u32 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:76:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_be32p' > > 76 | static __always_inline __be32 __cpu_to_be32p(const __u32 *p) > > | ^~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:80:8: error: unknown type name '__always_inline' > > 80 | static __always_inline __u32 __be32_to_cpup(const __be32 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:80:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__be32_to_cpup' > > 80 | static __always_inline __u32 __be32_to_cpup(const __be32 *p) > > | ^~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:84:8: error: unknown type name '__always_inline' > > 84 | static __always_inline __be16 __cpu_to_be16p(const __u16 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:84:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_be16p' > > 84 | static __always_inline __be16 __cpu_to_be16p(const __u16 *p) > > | ^~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:88:8: error: unknown type name '__always_inline' > > 88 | static __always_inline __u16 __be16_to_cpup(const __be16 *p) > > | ^~~~~~~~~~~~~~~ > > /usr/include/linux/byteorder/little_endian.h:88:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__be16_to_cpup' > > 88 | static __always_inline __u16 __be16_to_cpup(const __be16 *p) > > | ^~~~~~~~~~~~~~ > > make[5]: *** [/usr/src/linux-5.13.1-gentoo/tools/build/Makefile.build:97: /var/tmp/portage/linux/tools/objtool/arch/x86/decode.o] Error 1 > > make[4]: *** [/usr/src/linux-5.13.1-gentoo/tools/build/Makefile.build:139: arch/x86] Error 2 > > make[4]: *** Waiting for unfinished jobs.... > > make[3]: *** [Makefile:56: /var/tmp/portage/linux/tools/objtool/objtool-in.o] Error 2 > > make[2]: *** [Makefile:69: objtool] Error 2 > > make[1]: *** [/usr/src/linux-5.13.1-gentoo/Makefile:1974: tools/objtool] Error 2 > > make[1]: Leaving directory '/var/tmp/portage/linux' > > make: *** [Makefile:215: __sub-make] Error 2