From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.org/gmane.linux.lib.musl.general/14833 Path: news.gmane.org!.POSTED.blaine.gmane.org!not-for-mail From: Rich Felker Newsgroups: gmane.linux.ports.arm.kernel,gmane.linux.lib.musl.general,gmane.linux.kernel,gmane.linux.kernel.stable,gmane.linux.kernel.api Subject: Re: [musl] [PATCH] arm64: uapi: Fix user space compile with musl libc Date: Sat, 19 Oct 2019 16:29:50 -0400 Message-ID: <20191019202950.GC16318@brightrain.aerifal.cx> References: <20191019201717.15358-1-hauke@hauke-m.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Injection-Info: blaine.gmane.org; posting-host="blaine.gmane.org:195.159.176.226"; logging-data="149263"; mail-complaints-to="usenet@blaine.gmane.org" User-Agent: Mutt/1.5.21 (2010-09-15) Cc: catalin.marinas@arm.com, musl@lists.openwall.com, linux-kernel@vger.kernel.org, stable@vger.kernel.org, linux-api@vger.kernel.org, will@kernel.org, linux-arm-kernel@lists.infradead.org To: Hauke Mehrtens Original-X-From: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org Sat Oct 19 22:30:10 2019 Return-path: Envelope-to: linux-arm-kernel@m.gmane.org Original-Received: from bombadil.infradead.org ([198.137.202.133]) by blaine.gmane.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1iLvMX-000ciB-Pn for linux-arm-kernel@m.gmane.org; Sat, 19 Oct 2019 22:30:10 +0200 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ShiiR40K0dgVDNqxeKLfBxhERQgwLFO5vXT+eSQDmSs=; b=WKpGlkn/4xB8NH BwJg8XOxyDGdbv7e9TyLOp/hDqOkuDf1LXdnwoY8EwG90X7wwEP6PlNRyCru5g6qUlq9FDQiOJrwX 4wi75nEumDihO9Q4fS2faJryvPZ+B+X2mP0iWIrXOnqlLErV3H6yIkECxJGqmCd+y90Ij/qj+Bx3x wrt05QIEtYI/YDvkF2m8rSojAIZ9bmZLrWDR1pYxAhO0U03aQnq+ckBQSCsjzGn6BOD3/Uyvmw6U0 f3KTjWG+OqZrzIzT5JkC0jfPP7mMkEq8kuDD8UzyV/zLxKxhOGD7GNfyZ44NcfUz5FvtYP22WH8Ma 4zqkGuu1OBIMKynTOnlA==; Original-Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1iLvMS-00060A-Vi; Sat, 19 Oct 2019 20:30:04 +0000 Original-Received: from 216-12-86-13.cv.mvl.ntelos.net ([216.12.86.13] helo=brightrain.aerifal.cx) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1iLvMP-0005U5-CU for linux-arm-kernel@lists.infradead.org; Sat, 19 Oct 2019 20:30:03 +0000 Original-Received: from dalias by brightrain.aerifal.cx with local (Exim 3.15 #2) id 1iLvME-0002wa-00; Sat, 19 Oct 2019 20:29:50 +0000 Content-Disposition: inline In-Reply-To: <20191019201717.15358-1-hauke@hauke-m.de> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20191019_133001_588743_FE324174 X-CRM114-Status: GOOD ( 22.73 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org Xref: news.gmane.org gmane.linux.ports.arm.kernel:761913 gmane.linux.lib.musl.general:14833 gmane.linux.kernel:3258348 gmane.linux.kernel.stable:381712 gmane.linux.kernel.api:38076 Archived-At: On Sat, Oct 19, 2019 at 10:17:17PM +0200, Hauke Mehrtens wrote: > musl libc also defines the structures in their arch/aarch64/bits/signal.h > header file. Some applications like strace and gdb include both of them > and then the structure definitions are clashing and the build of these > user space applications fails. > > This patch allows a libc to define a constant which tells the kernel > header file that the libc already defined these structures and that they > should not be defined by the kernel uapi header files any more to > prevent clashes. This is done in a similar way as it is already done for > other header files. > > When this patch was accepted into the kernel I will also update musl > libc to define these constants. I don't entirely object to this outright, but I'd really like to avoid adding further __UAPI_DEF_* suppressions. AIUI asm/sigcontext.h is not intended to be used with userspace headers. Is it still being indirectly included via some other uapi headers? (I thought that was fixed..) If so, that should really be fixed first, and then we can see if there's still motivation for the patch here. Rich > Signed-off-by: Hauke Mehrtens > Cc: stable@vger.kernel.org > --- > arch/arm64/include/uapi/asm/sigcontext.h | 13 +++++++++++++ > include/uapi/linux/libc-compat.h | 20 ++++++++++++++++++++ > 2 files changed, 33 insertions(+) > > diff --git a/arch/arm64/include/uapi/asm/sigcontext.h b/arch/arm64/include/uapi/asm/sigcontext.h > index 8b0ebce92427..92d911146137 100644 > --- a/arch/arm64/include/uapi/asm/sigcontext.h > +++ b/arch/arm64/include/uapi/asm/sigcontext.h > @@ -20,7 +20,9 @@ > #ifndef __ASSEMBLY__ > > #include > +#include > > +#if __UAPI_DEF_SIGCONTEXT > /* > * Signal context structure - contains all info to do with the state > * before the signal handler was invoked. > @@ -35,6 +37,7 @@ struct sigcontext { > /* 4K reserved for FP/SIMD state and future expansion */ > __u8 __reserved[4096] __attribute__((__aligned__(16))); > }; > +#endif > > /* > * Allocation of __reserved[]: > @@ -57,6 +60,7 @@ struct sigcontext { > * generated when userspace does not opt in for any such extension. > */ > > +#if __UAPI_DEF_AARCH64_CTX > /* > * Header to be used at the beginning of structures extending the user > * context. Such structures must be placed after the rt_sigframe on the stack > @@ -67,7 +71,9 @@ struct _aarch64_ctx { > __u32 magic; > __u32 size; > }; > +#endif > > +#if __UAPI_DEF_FPSIMD_CONTEXT > #define FPSIMD_MAGIC 0x46508001 > > struct fpsimd_context { > @@ -76,7 +82,9 @@ struct fpsimd_context { > __u32 fpcr; > __uint128_t vregs[32]; > }; > +#endif > > +#if __UAPI_DEF_ESR_CONTEXT > /* > * Note: similarly to all other integer fields, each V-register is stored in an > * endianness-dependent format, with the byte at offset i from the start of the > @@ -93,7 +101,9 @@ struct esr_context { > struct _aarch64_ctx head; > __u64 esr; > }; > +#endif > > +#if __UAPI_DEF_EXTRA_CONTEXT > /* > * extra_context: describes extra space in the signal frame for > * additional structures that don't fit in sigcontext.__reserved[]. > @@ -128,7 +138,9 @@ struct extra_context { > __u32 size; /* size in bytes of the extra space */ > __u32 __reserved[3]; > }; > +#endif > > +#if __UAPI_DEF_SVE_CONTEXT > #define SVE_MAGIC 0x53564501 > > struct sve_context { > @@ -136,6 +148,7 @@ struct sve_context { > __u16 vl; > __u16 __reserved[3]; > }; > +#endif > > #endif /* !__ASSEMBLY__ */ > > diff --git a/include/uapi/linux/libc-compat.h b/include/uapi/linux/libc-compat.h > index 8254c937c9f4..a863130f4638 100644 > --- a/include/uapi/linux/libc-compat.h > +++ b/include/uapi/linux/libc-compat.h > @@ -264,4 +264,24 @@ > > #endif /* __GLIBC__ */ > > +/* Definitions for arch/arm64/include/uapi/asm/sigcontext.h */ > +#ifndef __UAPI_DEF_SIGCONTEXT > +#define __UAPI_DEF_SIGCONTEXT 1 > +#endif > +#ifndef __UAPI_DEF_AARCH64_CTX > +#define __UAPI_DEF_AARCH64_CTX 1 > +#endif > +#ifndef __UAPI_DEF_FPSIMD_CONTEXT > +#define __UAPI_DEF_FPSIMD_CONTEXT 1 > +#endif > +#ifndef __UAPI_DEF_ESR_CONTEXT > +#define __UAPI_DEF_ESR_CONTEXT 1 > +#endif > +#ifndef __UAPI_DEF_EXTRA_CONTEXT > +#define __UAPI_DEF_EXTRA_CONTEXT 1 > +#endif > +#ifndef __UAPI_DEF_SVE_CONTEXT > +#define __UAPI_DEF_SVE_CONTEXT 1 > +#endif > + > #endif /* _UAPI_LIBC_COMPAT_H */ > -- > 2.20.1