diff mbox

tracetool dtrace disabled-events fix

Message ID 4F688831.5050709@nowonline.co.uk
State New
Headers show

Commit Message

Lee Essen March 20, 2012, 1:37 p.m. UTC
If there are "disabled" entries in the trace-events file then
linetod_nop() is called if the backend is dtrace, it's currently
not present.

Signed-off-by: Lee Essen <lee.essen@nowonline.co.uk>

--

Comments

Stefan Hajnoczi March 20, 2012, 4:42 p.m. UTC | #1
On Tue, Mar 20, 2012 at 1:37 PM, Lee Essen <lee.essen@nowonline.co.uk> wrote:
> If there are "disabled" entries in the trace-events file then
> linetod_nop() is called if the backend is dtrace, it's currently
> not present.
>
> Signed-off-by: Lee Essen <lee.essen@nowonline.co.uk>
>
> --
> diff --git a/scripts/tracetool b/scripts/tracetool
> index 65bd0a1..d5e5591 100755
> --- a/scripts/tracetool
> +++ b/scripts/tracetool
> @@ -161,6 +161,12 @@ linetoc_nop()
>     return
>  }
>
> +linetod_nop()
> +{
> +    # Used when "disabled" events are processed
> +    return
> +}

Please also add linetostap_nop() for SystemTap builds - the nop
function was missing there too.

Otherwise looks good.

Stefan
diff mbox

Patch

diff --git a/scripts/tracetool b/scripts/tracetool
index 65bd0a1..d5e5591 100755
--- a/scripts/tracetool
+++ b/scripts/tracetool
@@ -161,6 +161,12 @@  linetoc_nop()
      return
  }

+linetod_nop()
+{
+    # Used when "disabled" events are processed
+    return
+}
+
  linetoc_end_nop()
  {
      return