PR6736: changelogs for post-merge compiled unwind/symbols data reorganization

8 files changed, 51 insertions, 7 deletions

diff --git a/ChangeLog b/ChangeLog
index 960f8232..5cafdd7c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+2008-07-10  Frank Ch. Eigler  <fche@elastic.org>
+
+	PR 6736.
+	* main.cxx (main,usage): Support new "-d MODULE" option to
+	populate comp-unwindsyms module list.
+	* stap.1.in: Document it.
+	* tapsets.cxx (dwarf_query::add_probe_point): Add probed module
+	to comp-unwindsyms set.
+	(utrace_builder::build): Ditto for probed user-space program.
+	* translate.cxx (emit_symbol_data): Emit _stp_module[] entry
+	for each comp-unwindsyms member, including relocation bases
+	and symbols, but not yet unwind data.
+	* NEWS: Remove mention of temporarily disabled unwinder.
+	* session.h: Corresponding changes.
+
 2008-07-09  Mark Wielaard  <mwielaard@redhat.com>
 
 	PR 6732
diff --git a/NEWS b/NEWS
index 434c9c71..103bd3c4 100644
--- a/NEWS
+++ b/NEWS
@@ -20,10 +20,6 @@
   number.  This allows using the same systemtap script if the rest
   of the file.c source only changes slightly.
 
-- Stack backtraces for x86 and x86-64 are generated by a dwarf
-  debuginfo-based unwinder based on the code from <jbeulich@novell.com>.
-  This should give more accurate backtraces.
-
 - A probe listing mode is available.
   % stap -l vm.*
   vm.brk
diff --git a/runtime/ChangeLog b/runtime/ChangeLog
index 25faffe2..de988561 100644
--- a/runtime/ChangeLog
+++ b/runtime/ChangeLog
@@ -1,3 +1,16 @@
+2008-07-10  Frank Ch. Eigler  <fche@elastic.org>
+
+	PR 6736.
+	* sym.c (_stp_module_relocate): Generalized to search all
+	_stp_module[]s.
+	(_stp_kallsyms_lookup_name): Gone.
+	(_stp_find_module_by_addr): Rewritten.
+	(_stp_get_unwind_info): Temporarily gone.
+	* sym.h (_stp_module): Simplify structure.
+	* unwind.c: Adapt just enough to compile with eliminated elements.
+
+	* probes.stp: Removed unused file.
+
 2008-07-09  Mark Wielaard  <mwielaard@redhat.com>
 
 	PR 6732
diff --git a/runtime/staprun/ChangeLog b/runtime/staprun/ChangeLog
index 27f4d8c8..a4f47880 100644
--- a/runtime/staprun/ChangeLog
+++ b/runtime/staprun/ChangeLog
@@ -1,3 +1,12 @@
+2008-07-10  Frank Ch. Eigler  <fche@elastic.org>
+
+	PR 6736.
+	* common.c (send_request): Move here ...
+	* mainloop.c: from here.
+	* staprun.c (send_relocations): New function to send kernel
+	and module relocation bases to probe module.
+	* staprun.h: Corresponding changes.
+
 2008-06-03  Frank Ch. Eigler  <fche@elastic.org>
 
 	PR 6429.
diff --git a/runtime/transport/ChangeLog b/runtime/transport/ChangeLog
index 4bb0868b..58431e54 100644
--- a/runtime/transport/ChangeLog
+++ b/runtime/transport/ChangeLog
@@ -1,3 +1,12 @@
+2008-07-10  Frank Ch. Eigler  <fche@elastic.org>
+
+	PR 6736.
+	* control.c (_stp_ctl_write_cmd): Accept STP_RELOCATION message.
+	* symbols.c: Gutted file.
+	(_stp_do_relocation): New function.
+	* transport.c (_stp_transport_init): Corresponding changes.
+	* transport_msgs.h: Ditto.
+
 2008-07-09  Frank Ch. Eigler  <fche@elastic.org>
 
 	PR5963
diff --git a/runtime/transport/control.c b/runtime/transport/control.c
index 09506bb1..7d78cca4 100644
--- a/runtime/transport/control.c
+++ b/runtime/transport/control.c
@@ -32,7 +32,6 @@ static ssize_t _stp_ctl_write_cmd(struct file *file, const char __user *buf, siz
 
 
 #ifdef DEBUG_TRANS
-        printk (KERN_INFO " control write_cmd: Got %s. len=%d\n", _stp_command_name[type], (int)count);
 	if (type < STP_MAX_CMD)
 		_dbug("Got %s. len=%d\n", _stp_command_name[type], (int)count);
 #endif
diff --git a/runtime/transport/procfs.c b/runtime/transport/procfs.c
index 64b48e4d..750e1994 100644
--- a/runtime/transport/procfs.c
+++ b/runtime/transport/procfs.c
@@ -122,8 +122,6 @@ static ssize_t _stp_ctl_write_cmd(struct file *file, const char __user *buf, siz
 	if (get_user(type, (int __user *)buf))
 		return -EFAULT;
 
-        printk (KERN_INFO " procfs write_cmd: Got %s. len=%d\n", _stp_command_name[type], (int)count);
-
 #if DEBUG_TRANSPORT > 0
 	if (type < STP_MAX_CMD)
 		_dbug("Got %s. len=%d\n", _stp_command_name[type], (int)count);
diff --git a/stap.1.in b/stap.1.in
index 7a3686d1..466c021b 100644
--- a/stap.1.in
+++ b/stap.1.in
@@ -152,6 +152,11 @@ Use the given name for the generated kernel object module, instead
 of a unique randomized name.  The generated kernel object module is
 copied to the current directory.
 .TP
+.BI \-d " MODULE"
+Add symbol/unwind information for the given module into the kernel object
+module.  This may enable symbolic tracebacks from those modules/programs,
+even if they do not have an explicit probe placed into them.
+.TP
 .BI \-o " FILE"
 Send standard output to named file. In bulk mode, percpu files will
 start with FILE_ followed by the cpu number.