public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
* [Bug tapsets/10975] New: cpuid() context tapset function unused and duplicate
@ 2009-11-17 22:22 mjw at redhat dot com
  2009-11-18 20:59 ` [Bug tapsets/10975] " fche at redhat dot com
  2010-08-16 22:25 ` fche at redhat dot com
  0 siblings, 2 replies; 3+ messages in thread
From: mjw at redhat dot com @ 2009-11-17 22:22 UTC (permalink / raw)
  To: systemtap

tapset/context.stp has:

    // cpuid() is not documented
    function cpuid:long () %{ /* pure */
            THIS->__retvalue = smp_processor_id();
    %}

Which is identical (except for not being unprivileged) to:

    /**
     * sfunction cpu - Returns the current cpu number.
     */
    function cpu:long () %{ /* pure */ /* unprivileged */
            THIS->__retvalue = smp_processor_id();
    %}

cpuid() is not used anywhere in the sources, documentation or examples.

Should we just nuke it? Or do we want to keep it for compatibility reasons?

-- 
           Summary: cpuid() context tapset function unused and duplicate
           Product: systemtap
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: tapsets
        AssignedTo: systemtap at sources dot redhat dot com
        ReportedBy: mjw at redhat dot com


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

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

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

* [Bug tapsets/10975] cpuid() context tapset function unused and duplicate
  2009-11-17 22:22 [Bug tapsets/10975] New: cpuid() context tapset function unused and duplicate mjw at redhat dot com
@ 2009-11-18 20:59 ` fche at redhat dot com
  2010-08-16 22:25 ` fche at redhat dot com
  1 sibling, 0 replies; 3+ messages in thread
From: fche at redhat dot com @ 2009-11-18 20:59 UTC (permalink / raw)
  To: systemtap


------- Additional Comments From fche at redhat dot com  2009-11-18 20:59 -------
We don't have a formal deprecation process, but
it shouldn't be "delete with no warning".


-- 


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

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

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

* [Bug tapsets/10975] cpuid() context tapset function unused and duplicate
  2009-11-17 22:22 [Bug tapsets/10975] New: cpuid() context tapset function unused and duplicate mjw at redhat dot com
  2009-11-18 20:59 ` [Bug tapsets/10975] " fche at redhat dot com
@ 2010-08-16 22:25 ` fche at redhat dot com
  1 sibling, 0 replies; 3+ messages in thread
From: fche at redhat dot com @ 2010-08-16 22:25 UTC (permalink / raw)
  To: systemtap


------- Additional Comments From fche at redhat dot com  2010-08-16 22:25 -------
deprecated commit 8dc503b

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


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

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

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

end of thread, other threads:[~2010-08-16 22:25 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-11-17 22:22 [Bug tapsets/10975] New: cpuid() context tapset function unused and duplicate mjw at redhat dot com
2009-11-18 20:59 ` [Bug tapsets/10975] " fche at redhat dot com
2010-08-16 22:25 ` fche 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).