On Sun, Apr 28, 2019 at 12:23 AM Daniel Dadap <ddadap@xxxxxxxxxx> wrote: > > recordmcount.pl uses a set of regular expressions to parse the output of > objdump(1). However, if objdump(1) output is localized, it may not match > the regular expressions, thereby preventing recordmcount.pl from parsing > object files correctly. > > In order to allow recordmcount.pl to function correctly regardless of the > current locale settings, set LANG=C when running objdump(1). LC_ALL is > already unset in the top-level Makefile, so it is not necessary to also > override that environment variable. > > Signed-off-by: Daniel Dadap <ddadap@xxxxxxxxxx> > Reviewed-by: Robert Morell <rmorell@xxxxxxxxxx> Applied to linux-kbuild. Thanks. > --- > scripts/recordmcount.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl > index 68841d01162c..ffea46287f83 100755 > --- a/scripts/recordmcount.pl > +++ b/scripts/recordmcount.pl > @@ -493,7 +493,7 @@ sub update_funcs > # > # Step 2: find the sections and mcount call sites > # > -open(IN, "$objdump -hdr $inputfile|") || die "error running $objdump"; > +open(IN, "LANG=C $objdump -hdr $inputfile|") || die "error running $objdump"; > > my $text; > > -- > 2.18.1 > -- Best Regards Masahiro Yamada
