470121 - Can't run callgrind_control with valgrind 3.21.0 because of perl errors

diff --git a/NEWS b/NEWS
index 4c5635dde11f8d3cf38a0b355cec5491f0f867f1..52ee38ab8b55e11989b69c731c0ec72ec6047e19 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -37,6 +37,7 @@ are not entered into bugzilla tend to get forgotten about or ignored.
 469049  link failure on ppc64 (big endian) valgrind 3.20
 469146  massif --ignore-fn does not ignore inlined functions
 469768  Make it possible to install gdb scripts in a different location
+470121  Can't run callgrind_control with valgrind 3.21.0 because of perl errors
 470520  Multiple realloc zero errors crash in MC_(eq_Error)
 470713  Failure on the Yosys project: valgrind: m_libcfile.c:1802
         (Bool vgPlain_realpath(const HChar *, HChar *)): Assertion 'resolved' failed

diff --git a/callgrind/callgrind_control.in b/callgrind/callgrind_control.in
index 083ffa29fcc6db0a176159b31c78a57e0c89a1a2..bee6661efb219dbf611193807f9ef0abab94fd37 100644 (file)
--- a/callgrind/callgrind_control.in
+++ b/callgrind/callgrind_control.in
@@ -29,6 +29,12 @@ use File::Basename;
 
 # vgdb_exe will be set to a vgdb found 'near' the callgrind_control file
 my $vgdb_exe = "";
+my $vgdbPrefixOption = "";
+my $cmd = "";
+my %cmd;
+my %cmdline;
+my $pid = -1;
+my @pids = ();
 
 sub getCallgrindPids {
 
@@ -50,6 +56,8 @@ sub getCallgrindPids {
   close LIST;
 }
 
+my $headerPrinted = 0;
+
 sub printHeader {
   if ($headerPrinted) { return; }
   $headerPrinted = 1;
@@ -95,11 +103,17 @@ sub printHelp {
 # Parts more or less copied from cg_annotate (author: Nicholas Nethercote)
 #
 
+my $event = "";
+my $events = "";
+my %events = ();
+my @events = ();
+my @show_events = ();
+my @show_order = ();
+
 sub prepareEvents {
 
   @events = split(/\s+/, $events);
-  %events = ();
-  $n = 0;
+  my $n = 0;
   foreach $event (@events) {
     $events{$event} = $n;
     $n++;
@@ -178,7 +192,7 @@ sub print_events ($)
 {
     my ($CC_col_widths) = @_;
 
-    foreach my $i (@show_order) { 
+    foreach my $i (@show_order) {
         my $event       = $events[$i];
         my $event_width = length($event);
         my $col_width   = $CC_col_widths->[$i];
@@ -209,7 +223,7 @@ if (-x $controldir . "/vgdb") {
 
 # To find the list of active pids, we need to have
 # the --vgdb-prefix option if given.
-$vgdbPrefixOption = "";
+my $arg = "";
 foreach $arg (@ARGV) {
     if ($arg =~ /^--vgdb-prefix=.*$/) {
         $vgdbPrefixOption=$arg;
@@ -219,15 +233,19 @@ foreach $arg (@ARGV) {
 
 getCallgrindPids;
 
-$requestEvents = 0;
-$requestDump = 0;
-$switchInstr = 0;
-$headerPrinted = 0;
-$dumpHint = "";
+my $requestEvents = 0;
+my $requestDump = 0;
+my $switchInstr = 0;
+my $dumpHint = "";
+my $printBacktrace = 0;
+my $printStatus = 0;
+my $switchInstrMode = "";
+my $requestKill = "";
+my $requestZero = "";
 
-$verbose = 0;
+my $verbose = 0;
 
-%spids = ();
+my %spids = ();
 foreach $arg (@ARGV) {
   if ($arg =~ /^-/) {
     if ($requestDump == 1) { $requestDump = 2; }
@@ -329,8 +347,8 @@ foreach $arg (@ARGV) {
   }
 
   if (defined $cmd{$arg}) { $spids{$arg} = 1; next; }
-  $nameFound = 0;
-  foreach $p (@pids) {
+  my $nameFound = 0;
+  foreach my $p (@pids) {
     if ($cmd{$p} =~ /$arg$/) {
       $nameFound = 1;
       $spids{$p} = 1;
@@ -353,11 +371,11 @@ if (scalar @pids == 0) {
   exit;
 }
 
-@spids = keys %spids;
+my @spids = keys %spids;
 if (scalar @spids >0) { @pids = @spids; }
 
-$vgdbCommand = "";
-$waitForAnswer = 0;
+my $vgdbCommand = "";
+my $waitForAnswer = 0;
 if ($requestDump) {
   $vgdbCommand = "dump";
   if ($dumpHint ne "") { $vgdbCommand .= " ".$dumpHint; }
@@ -371,7 +389,7 @@ if ($printStatus || $printBacktrace || $requestEvents) {
 }
 
 foreach $pid (@pids) {
-  $pidstr = "PID $pid: ";
+  my $pidstr = "PID $pid: ";
   if ($pid >0) { print $pidstr.$cmdline{$pid}; }
 
   if ($vgdbCommand eq "") {
@@ -385,24 +403,24 @@ foreach $pid (@pids) {
   }
   open RESULT, $vgdb_exe . " $vgdbPrefixOption --pid=$pid $vgdbCommand|";
 
-  @tids = ();
-  $ctid = 0;
-  %fcount = ();
-  %func = ();
-  %calls = ();
-  %events = ();
-  @events = ();
-  @threads = ();
-  %totals = ();
-
-  $exec_bbs = 0;
-  $dist_bbs = 0;
-  $exec_calls = 0;
-  $dist_calls = 0;
-  $dist_ctxs = 0;
-  $dist_funcs = 0;
-  $threads = "";
-  $events = "";
+  my @tids = ();
+  my $tid;
+  my $ctid = 0;
+  my %fcount = ();
+  my %func = ();
+  my %calls = ();
+  my @threads = ();
+  my %totals = ();
+  my $totals_width = [];
+
+  my $exec_bbs = 0;
+  my $dist_bbs = 0;
+  my $exec_calls = 0;
+  my $dist_calls = 0;
+  my $dist_ctxs = 0;
+  my $dist_funcs = 0;
+  my $threads = "";
+  my $instrumentation = "";
 
   while(<RESULT>) {
     if (/function-(\d+)-(\d+): (.+)$/) {
@@ -485,10 +503,10 @@ foreach $pid (@pids) {
       }
       print "Backtrace for Thread $tid\n";
 
-      $i = $fcount{$tid};
-      $c = 0;
+      my $i = $fcount{$tid};
+      my $c = 0;
       while($i>0 && $c<100) {
-       $fc = substr(" $c",-2);
+       my $fc = substr(" $c",-2);
        print "   [$fc]  ";
        if ($requestEvents >0) {
          print_CC($events{$tid,$i-1}, $totals_width);