changeset 2686:3d7ff04a7ed7

PR3828: Architectures unsupported by SystemTap tapsets throw a parse error 2017-04-03 Andrew John Hughes <gnu_andrew@member.fsf.org> PR3828: Architectures unsupported by SystemTap tapsets throw a parse error Patch by David Smith <dsmith@redhat.com> * AUTHORS: Add David Smith. * NEWS: Updated. * tapset/jstack.stp.in: Call the error function if no architecture is matched.
author Andrew John Hughes <gnu_andrew@member.fsf.org>
date Mon, 22 Feb 2021 00:28:53 +0000
parents 526d974bb37a
children dad45f85e3a3
files AUTHORS ChangeLog NEWS tapset/jstack.stp.in
diffstat 4 files changed, 19 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/AUTHORS	Mon Feb 22 00:22:47 2021 +0000
+++ b/AUTHORS	Mon Feb 22 00:28:53 2021 +0000
@@ -37,6 +37,7 @@
 Keith Seitz <keiths@redhat.com>
 Ralph Sennhauser <sera@gentoo.org>
 Yasumasa Suenaga  <yasuenag@gmail.com>
+David Smith <dsmith@redhat.com>
 Joshua Sumali <jsumali@redhat.com>
 Pavel Tisnovsky <ptisnovs@redhat.com>
 Christian Thalinger <twisti@complang.tuwien.ac.at>
--- a/ChangeLog	Mon Feb 22 00:22:47 2021 +0000
+++ b/ChangeLog	Mon Feb 22 00:28:53 2021 +0000
@@ -1,3 +1,13 @@
+2017-04-03  Andrew John Hughes  <gnu_andrew@member.fsf.org>
+
+	PR3828: Architectures unsupported by SystemTap
+	tapsets throw a parse error
+	Patch by David Smith <dsmith@redhat.com>
+	* AUTHORS: Add David Smith.
+	* NEWS: Updated.
+	* tapset/jstack.stp.in:
+	Call the error function if no architecture is matched.
+
 2017-03-15  Andrew John Hughes  <gnu_andrew@member.fsf.org>
 
 	PR3827: jstack.stp should support AArch64
--- a/NEWS	Mon Feb 22 00:22:47 2021 +0000
+++ b/NEWS	Mon Feb 22 00:28:53 2021 +0000
@@ -16,6 +16,7 @@
   - PR3827: jstack.stp should support AArch64
 * Bug fixes
   - PR3825: SystemTap is heavily confused by multiple JDKs
+  - PR3828: Architectures unsupported by SystemTap tapsets throw a parse error
 
 New in release 5.0.0 (2019-XX-XX):
 
--- a/tapset/jstack.stp.in	Mon Feb 22 00:22:47 2021 +0000
+++ b/tapset/jstack.stp.in	Mon Feb 22 00:28:53 2021 +0000
@@ -107,7 +107,13 @@
      pc_register = "pc";
      ptr_size = 8; // XXX - might be probing 32-on-64 jvm.
      ptr_mask = 0xFFFFFFFFFFFFFFFF;
-  %: **ERROR** unknown architecture
+  %:
+     sp_register = "";
+     fp_register = "";
+     pc_register = "";
+     ptr_size = 8;
+     ptr_mask = 0xFFFFFFFFFFFFFFFF;
+     error("unknown architecture")
   %) %) %)
 
   // Pretend we have an array at address zero and take address of second