From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ozlabs.org (ozlabs.org [IPv6:2401:3900:2:1::2]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 93BC81A0BE4 for ; Thu, 14 Jan 2016 00:09:25 +1100 (AEDT) In-Reply-To: <1452600863-6294-1-git-send-email-mpe@ellerman.id.au> To: Michael Ellerman , From: Michael Ellerman Cc: ulrich.weigand@de.ibm.com, Anton Blanchard , rostedt@goodmis.org, amodra@gmail.com Subject: Re: [1/2] scripts/recordmcount.pl: support data in text section on powerpc Message-Id: <20160113130925.4CCC814076E@ozlabs.org> Date: Thu, 14 Jan 2016 00:09:25 +1100 (AEDT) List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, 2016-12-01 at 12:14:22 UTC, Michael Ellerman wrote: > From: Ulrich Weigand > > If a text section starts out with a data blob before the first > function start label, disassembly parsing doing in recordmcount.pl > gets confused on powerpc, leading to creation of corrupted module > objects. > > This was not a problem so far since the compiler would never create > such text sections. However, this has changed with a recent change > in GCC 6 to support distances of > 2GB between a function and its > assoicated TOC in the ELFv2 ABI, exposing this problem. > > There is already code in recordmcount.pl to handle such data blobs > on the sparc64 platform. This patch uses the same method to handle > those on powerpc as well. > > Cc: stable@vger.kernel.org > Signed-off-by: Ulrich Weigand > Signed-off-by: Michael Ellerman > Acked-by: Steven Rostedt Applied to powerpc next. https://git.kernel.org/powerpc/c/2e50c4bef77511b42cc226865d cheers