From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 56987 invoked by alias); 26 Jun 2017 13:33:03 -0000 Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org Received: (qmail 56951 invoked by uid 89); 26 Jun 2017 13:33:02 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 spammy=king X-Spam-User: qpsmtpd, 2 recipients X-HELO: foss.arm.com Received: from foss.arm.com (HELO foss.arm.com) (217.140.101.70) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 26 Jun 2017 13:33:00 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id A621D80D; Mon, 26 Jun 2017 06:32:59 -0700 (PDT) Received: from e103592.cambridge.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.72.51.249]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id C1EB93F41F; Mon, 26 Jun 2017 06:32:58 -0700 (PDT) Date: Mon, 26 Jun 2017 13:33:00 -0000 From: Dave Martin To: Russell King - ARM Linux Cc: gdb@sourceware.org, Edmund Grimley-Evans , libc-alpha@sourceware.org, linux-arm-kernel@lists.infradead.org Subject: Re: [RFC PATCH 0/2] ARM: Fix unparseable signal frame with CONFIG_IWMMXT Message-ID: <20170626133255.GH8543@e103592.cambridge.arm.com> References: <1498059983-13438-1-git-send-email-Dave.Martin@arm.com> <20170626101304.GG4902@n2100.armlinux.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170626101304.GG4902@n2100.armlinux.org.uk> User-Agent: Mutt/1.5.23 (2014-03-12) X-SW-Source: 2017-06/txt/msg00025.txt.bz2 On Mon, Jun 26, 2017 at 11:13:04AM +0100, Russell King - ARM Linux wrote: > On Wed, Jun 21, 2017 at 04:46:01PM +0100, Dave Martin wrote: > > In kernels with CONFIG_IWMMXT=y running on non-iWMMXt hardware, the > > signal frame can be left partially uninitialised in such a way > > that userspace cannot parse uc_regspace[] safely. In particular, > > this means that the VFP registers cannot be located reliably in the > > signal frame when a multi_v7_defconfig kernel is run on the > > majority of platforms. > > > > I don't know whether any userspace has implemented any sort of > > workaround for this, but the ABI by itself is insufficient anyway. > > > > This series attempts to omit the spurious iWMMXt record when > > appropriate. > > > > Not extensively tested, and the ABI impact is unknown for now. > > Hmm, I would actually suggest that we poke in a correct size for the > missing iWMMXt record, and an invalid magic number as the "simple" > solution for this - that doesn't make any layout changes to the > data structures, and is probably the safest solution for backporting. This avoids altering the sigframe layout at all in this case, which feels less dirsuptive, but overall I'm not sure it's lower-risk. I'm concerned that there are a some userspace sigframe parsers out there that work only by accident, especially given that the kernel sigreturn implementation is the primary example and that doesn't need to be fully robust (since the kernel lays out the sigframe itself during signal delivery). > Going forward, I think something along the lines of your proposal is > okay. I'm happy to do either, or propose one approach for stable and the other for mainline, but it's hard to know which is least likely to break userspace, or exactly what the ABI is. Cheers ---Dave