From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964976Ab3CZQSV (ORCPT ); Tue, 26 Mar 2013 12:18:21 -0400 Received: from mail-pa0-f41.google.com ([209.85.220.41]:53313 "EHLO mail-pa0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934922Ab3CZQSS (ORCPT ); Tue, 26 Mar 2013 12:18:18 -0400 Date: Tue, 26 Mar 2013 09:18:10 -0700 From: Greg KH To: Kees Cook Cc: Jiri Slaby , Sebastian Gottschall , LKML , "# 3.4.x" , Andrew Morton Subject: Re: Build breakage in 3.8.4 [was: Linux 3.8.4] Message-ID: <20130326161810.GB18581@kroah.com> References: <20130320201554.GA18281@kroah.com> <514F5480.6040501@suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Mar 26, 2013 at 08:56:20AM -0700, Kees Cook wrote: > On Sun, Mar 24, 2013 at 12:31 PM, Jiri Slaby wrote: > > On 03/20/2013 11:25 PM, Sebastian Gottschall wrote: > > > >> Am 20.03.2013 21:15, schrieb Greg KH: > >>> I'm announcing the release of the 3.8.4 kernel. > > ... > >>> signal: always clear sa_restorer on execve > >> consider that this kernel is not compileable on mips targets due the > >> patch in kernel/signal.c > >> > >> SA_RESTORER is defined in mips, but sa_restorer does not exist on that > >> platform > > > > The same for ia64... > > > > kernel/signal.c: In function 'flush_signal_handlers': > > kernel/signal.c:441: error: 'struct sigaction' has no member named > > 'sa_restorer' > > > > Do we need this in 3.0, 3.4, 3.8 too? > > commit 522cff142d7d2f9230839c9e1f21a4d8bcc22a4a > > Author: Andrew Morton > > Date: Wed Mar 13 14:59:34 2013 -0700 > > > > kernel/signal.c: use __ARCH_HAS_SA_RESTORER instead of SA_RESTORER > > > > __ARCH_HAS_SA_RESTORER is the preferred conditional for use in 3.9 and > > later kernels, per Kees. > > Hrm, if so, we'll need to also backport the changes that aded > __ARCH_HAS_SA_RESTORER... Ok, well, can someone provide me the needed patches? Otherwise I will have to revert the offending change from the stable releases. thanks, greg k-h