From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 2A8B8DDFBD for ; Wed, 20 Aug 2008 11:19:54 +1000 (EST) Subject: Re: ftrace introduces instability into kernel 2.6.27(-rc2,-rc3) From: Benjamin Herrenschmidt To: Jeremy Fitzhardinge In-Reply-To: <48AB5E36.8030400@goop.org> References: <48A9901B.1080900@redhat.com> <20080818154746.GA26835@Krystal> <48A9AFA7.8080508@freescale.com> <1219110814.8062.2.camel@pasglop> <1219113549.8062.13.camel@pasglop> <20080819024707.GA22659@Krystal> <20080819173453.GA28239@Krystal> <1219182471.7826.16.camel@pasglop> <48AB5E36.8030400@goop.org> Content-Type: text/plain Date: Wed, 20 Aug 2008 11:17:28 +1000 Message-Id: <1219195048.7826.23.camel@pasglop> Mime-Version: 1.0 Cc: Nick Piggin , "Paul E. McKenney" , Mathieu Desnoyers , linux-kernel@vger.kernel.org, Steven Rostedt , linuxppc-dev@ozlabs.org, Steven Rostedt , Scott Wood , Eran Liberty Reply-To: benh@kernel.crashing.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, 2008-08-19 at 16:58 -0700, Jeremy Fitzhardinge wrote: > Benjamin Herrenschmidt wrote: > >> Ok, there are two cases where it's ok : > >> > >> 1 - in stop_machine, considering we are not touching code executed in > >> NMI handlers. > >> 2 - when using my replace_instruction_safe() which uses a temporary > >> breakpoint when doing the instruction replacement. > >> > >> In those cases you could use text_poke_early(). > >> > > > > Note that vmap/vunmap will be very slow. > > > > Don't we have Nick's speedups now? Not sure what speedups this is, but it's stlil not something you want to do a lot ... setting up and tearing down MMU mappings isn't something I'd like to see happening on a per-mcount basis. Ben