On Fri, Jan 09, 2009 at 02:57:38PM -0500, Steven Rostedt wrote: > > On Fri, 9 Jan 2009, Sam Ravnborg wrote: > > So we duplicate the cases where we identify "this .o file > > is used in a module". > > This is not good. > > > > Also there is no reason to do this for the .i, .s and .lst > > targets as they do not invode recordmcount.pl anyway. > > > > If your build your patch on top of the following we avoid > > the duplicated list. > > > > Sam > > Thanks Sam, > > Can I add this on top of Shaohua's and put your Signed-off-by, as well > as an Acked-by on Shaohua's? My patch was made on top of -linus so you need to use the flag in the call to recordmount.pl. Please feel free to put my Acked-by: on the patch, but as this does not originate from me it would be the wrong documentation trail to add my Sigend-off-by as I just gave an improvement proposal. Sam -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html