From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932816AbZJ3SSc (ORCPT ); Fri, 30 Oct 2009 14:18:32 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932701AbZJ3SSb (ORCPT ); Fri, 30 Oct 2009 14:18:31 -0400 Received: from mga11.intel.com ([192.55.52.93]:41410 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932692AbZJ3SSb (ORCPT ); Fri, 30 Oct 2009 14:18:31 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.44,655,1249282800"; d="scan'208";a="509314731" Subject: Re: [patch 3/3] x86_64, cpa: use only text section in set_kernel_text_rw/ro From: Suresh Siddha Reply-To: Suresh Siddha To: "rostedt@goodmis.org" Cc: "mingo@elte.hu" , "hpa@zytor.com" , "linux-kernel@vger.kernel.org" In-Reply-To: <1256781977.26028.2139.camel@gandalf.stny.rr.com> References: <20091029024820.996634347@sbs-t61.sc.intel.com> <20091029024821.164525222@sbs-t61.sc.intel.com> <1256781977.26028.2139.camel@gandalf.stny.rr.com> Content-Type: text/plain Organization: Intel Corp Date: Fri, 30 Oct 2009 11:17:08 -0800 Message-Id: <1256930228.2679.5.camel@sbs-t61.sc.intel.com> Mime-Version: 1.0 X-Mailer: Evolution 2.26.3 (2.26.3-1.fc11) Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2009-10-28 at 19:06 -0700, Steven Rostedt wrote: > On Wed, 2009-10-28 at 18:46 -0800, Suresh Siddha wrote: > > plain text document attachment (fix_kernel_text_ro_rw.patch) > > set_kernel_text_rw()/set_kernel_text_ro() are marking pages starting > > from _text to __start_rodata as RW or RO. > > > > With CONFIG_DEBUG_RODATA, there might be free pages (associated with > > padding the sections to 2MB large page boundary) between text > > and rodata sections that are given back to page allocator. So we should > > use only use the start (__text) and end (__stop___ex_table) of the > > text section in set_kernel_text_rw()/set_kernel_text_ro(). > > > > Cc: Steven Rostedt > > Signed-off-by: Suresh Siddha > > --- > > > > I'll go ahead and test this to make sure it does not bust ftrace with an > allyesconfig. Steve, Good to go? If so, Ingo can you please consider this patchset? thanks, suresh