public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
* [Bug translator/2643] New: Translator fails pass 2 if probe alias does not have the matching .return alias
@ 2006-05-05 23:11 hien at us dot ibm dot com
  2006-05-05 23:29 ` [Bug translator/2643] probe point wildcard expansion excessively optimistic fche at redhat dot com
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: hien at us dot ibm dot com @ 2006-05-05 23:11 UTC (permalink / raw)
  To: systemtap

Recently, we removed the syscall.exit.return and and syscall.exit_group.return
aliases in the syscalls.stp. That cause the translator fails pass 2 when do try
to compile this script


probe syscall.*, syscall.*.return {

/* do something */

}

Looks like the translator is looking for the match pair of function entry and
return.

-- 
           Summary: Translator fails pass 2 if probe alias does not have the
                    matching .return alias
           Product: systemtap
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: translator
        AssignedTo: systemtap at sources dot redhat dot com
        ReportedBy: hien at us dot ibm dot com


http://sourceware.org/bugzilla/show_bug.cgi?id=2643

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug translator/2643] probe point wildcard expansion excessively optimistic
  2006-05-05 23:11 [Bug translator/2643] New: Translator fails pass 2 if probe alias does not have the matching .return alias hien at us dot ibm dot com
@ 2006-05-05 23:29 ` fche at redhat dot com
  2006-05-05 23:35 ` fche at redhat dot com
  2006-05-18 14:46 ` dsmith at redhat dot com
  2 siblings, 0 replies; 4+ messages in thread
From: fche at redhat dot com @ 2006-05-05 23:29 UTC (permalink / raw)
  To: systemtap


------- Additional Comments From fche at redhat dot com  2006-05-05 23:29 -------
The patch for bug #2433 awakened a latent bug in the translator that expands
probe point wildcards.  The same problem appears with this case:

probe a.a.a = begin { }
probe a.b = begin { }
probe a.*.a  { }

The wildcard should expand to "a.a.a" only, but since it proceeds naively one
component at a time and lacks lookahead, it also expands to "a.b.a".

One quick hack as a workaround until the real fix is to add back some dummy
probe aliases for the two recently deceased.

probe syscall.exit.return = end {}
probe syscall.exit_group.return = end {}


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |eteo at redhat dot com
            Summary|Translator fails pass 2 if  |probe point wildcard
                   |probe alias does not have   |expansion excessively
                   |the matching .return alias  |optimistic


http://sourceware.org/bugzilla/show_bug.cgi?id=2643

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug translator/2643] probe point wildcard expansion excessively optimistic
  2006-05-05 23:11 [Bug translator/2643] New: Translator fails pass 2 if probe alias does not have the matching .return alias hien at us dot ibm dot com
  2006-05-05 23:29 ` [Bug translator/2643] probe point wildcard expansion excessively optimistic fche at redhat dot com
@ 2006-05-05 23:35 ` fche at redhat dot com
  2006-05-18 14:46 ` dsmith at redhat dot com
  2 siblings, 0 replies; 4+ messages in thread
From: fche at redhat dot com @ 2006-05-05 23:35 UTC (permalink / raw)
  To: systemtap


------- Additional Comments From fche at redhat dot com  2006-05-05 23:35 -------
By the way, bug #2111 comment #4 hints at the reason why this regression wasn't
caught by the testsuite: buildok/syscall.stp has been a de facto "ko"
(expect-fail) test case, waiting for a new syscalls embedded-C buildok test case.

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=2643

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Bug translator/2643] probe point wildcard expansion excessively optimistic
  2006-05-05 23:11 [Bug translator/2643] New: Translator fails pass 2 if probe alias does not have the matching .return alias hien at us dot ibm dot com
  2006-05-05 23:29 ` [Bug translator/2643] probe point wildcard expansion excessively optimistic fche at redhat dot com
  2006-05-05 23:35 ` fche at redhat dot com
@ 2006-05-18 14:46 ` dsmith at redhat dot com
  2 siblings, 0 replies; 4+ messages in thread
From: dsmith at redhat dot com @ 2006-05-18 14:46 UTC (permalink / raw)
  To: systemtap



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|systemtap at sources dot    |dsmith at redhat dot com
                   |redhat dot com              |
             Status|NEW                         |ASSIGNED


http://sourceware.org/bugzilla/show_bug.cgi?id=2643

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2006-05-18 14:46 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-05-05 23:11 [Bug translator/2643] New: Translator fails pass 2 if probe alias does not have the matching .return alias hien at us dot ibm dot com
2006-05-05 23:29 ` [Bug translator/2643] probe point wildcard expansion excessively optimistic fche at redhat dot com
2006-05-05 23:35 ` fche at redhat dot com
2006-05-18 14:46 ` dsmith at redhat dot com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).