From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5701 invoked by alias); 14 Mar 2011 17:14:05 -0000 Received: (qmail 5688 invoked by uid 22791); 14 Mar 2011 17:14:04 -0000 X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE X-Spam-Check-By: sourceware.org Received: from hrndva-omtalb.mail.rr.com (HELO hrndva-omtalb.mail.rr.com) (71.74.56.124) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 14 Mar 2011 17:13:59 +0000 X-Authority-Analysis: v=1.1 cv=UQuFHoD2CPQ248x8AXEbKhr4z9AaDqApxmEl3BhfZ64= c=1 sm=0 a=OjXHl2S905kA:10 a=Q9fys5e9bTEA:10 a=OPBmh+XkhLl+Enan7BmTLg==:17 a=meVymXHHAAAA:8 a=AKMhOgkj_QJxKB-hJr4A:9 a=Abqye561LVGYEMGOfsYA:7 a=MURfYrteXVQ5wjnhus-LIz2QBwwA:4 a=PUjeQqilurYA:10 a=jeBq3FmKZ4MA:10 a=OPBmh+XkhLl+Enan7BmTLg==:117 Received: from [67.242.120.143] ([67.242.120.143:51467] helo=[192.168.23.10]) by hrndva-oedge02.mail.rr.com (envelope-from ) (ecelerity 2.2.3.46 r()) with ESMTP id E3/EC-00963-2DC4E7D4; Mon, 14 Mar 2011 17:13:57 +0000 Subject: Re: [PATCH v2 2.6.38-rc8-tip 1/20] 1: mm: Move replace_page() to mm/memory.c From: Steven Rostedt To: Srikar Dronamraju Cc: Peter Zijlstra , Ingo Molnar , Linux-mm , Arnaldo Carvalho de Melo , Linus Torvalds , Christoph Hellwig , Masami Hiramatsu , Ananth N Mavinakayanahalli , Oleg Nesterov , LKML , SystemTap , Jim Keniston , Roland McGrath , Andi Kleen , Andrew Morton , "Paul E. McKenney" In-Reply-To: <20110314170227.GN24254@linux.vnet.ibm.com> References: <20110314133403.27435.7901.sendpatchset@localhost6.localdomain6> <20110314133413.27435.67467.sendpatchset@localhost6.localdomain6> <1300112195.9910.92.camel@gandalf.stny.rr.com> <20110314170227.GN24254@linux.vnet.ibm.com> Content-Type: text/plain; charset="ISO-8859-15" Date: Mon, 14 Mar 2011 17:14:00 -0000 Message-ID: <1300122834.9910.126.camel@gandalf.stny.rr.com> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Mailing-List: contact systemtap-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: systemtap-owner@sourceware.org X-SW-Source: 2011-q1/txt/msg00378.txt.bz2 On Mon, 2011-03-14 at 22:32 +0530, Srikar Dronamraju wrote: > * Steven Rostedt [2011-03-14 10:16:35]: > > > On Mon, 2011-03-14 at 19:04 +0530, Srikar Dronamraju wrote: > > > User bkpt will use background page replacement approach to insert/delete > > > breakpoints. Background page replacement approach is based on > > > replace_page. Hence replace_page() loses its static attribute. > > > > > > > Just a nitpick, but since replace_page() is being moved, could you > > specify that in the change log. Something like: > > > > "Hence, replace_page() is moved from ksm.c into memory.c and its static > > attribute is removed." > > Okay, Will take care to add the moved from ksm.c into memory.c in the > next version of patchset. Thanks! > > I like to see in the change log "move x to y" when that is actually > > done, because it is hard to see if anything actually changed when code > > is moved. Ideally it is best to move code in one patch and make the > > As discussed in IRC, moving and removing the static attribute had to > be one patch so that mm/ksm.c compiles correctly. The other option we > have is to remove the static attribute first and then moving the > function. Hmm, maybe that may be a good idea. Since it is really two changes. One is to make it global for other usages. I'm not even sure why you moved it. The change log for the move can explain that. -- Steve > > > change in another. If you do cut another version of this patch set, > > could you do that. This alone is not enough to require a new release. > > >