On Thu, Mar 13, 2014 at 02:57:58PM +0000, AKASHI Takahiro wrote: > On 03/13/2014 09:38 PM, Will Deacon wrote: > > On Thu, Mar 13, 2014 at 10:13:46AM +0000, AKASHI Takahiro wrote: > >> diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl > >> index 91280b8..397b6b8 100755 > >> --- a/scripts/recordmcount.pl > >> +++ b/scripts/recordmcount.pl > >> @@ -279,6 +279,11 @@ if ($arch eq "x86_64") { > >> $mcount_regex = "^\\s*([0-9a-fA-F]+):\\s*R_ARM_(CALL|PC24|THM_CALL)" > >> . > >> "\\s+(__gnu_mcount_nc|mcount)\$"; > >> > >> +} elsif ($arch eq "arm64") { > >> + $alignment = 3; > >> + $section_type = '%progbits'; > >> + $mcount_regex = > >> "^\\s*([0-9a-fA-F]+):\\s*R_AARCH64_CALL26\\s+_mcount\$"; > > > > What about "mcount" (i.e. no underscore)? > > I don't think it works since such regex doesn't match to "CALL26 _mcount".
What I meant was, do you need to match "mcount" as well? However, after speaking to our GCC guys, they say they only support "_mcount" (preferred) and "__mcount" (which GCC itself doesn't emit) so you can leave this regex as-is. Will -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/