[PATCH] Don't spam trace report about missing fields

[Avi Kivity]

Instead of spamming the output with complaints about missing fields, simply
return an error to the caller, which can print something out or do something
more intelligent about them.

Fixes kvm plugin tracing older kernels.

Signed-off-by: Avi Kivity 
---
 parse-events.c |   15 +++
 1 files changed, 3 insertions(+), 12 deletions(-)

diff --git a/parse-events.c b/parse-events.c
index f0b0324..7a4c3a6 100644
--- a/parse-events.c
+++ b/parse-events.c
@@ -4446,17 +4446,11 @@ int get_field_val(struct trace_seq *s, struct 
format_field *field,
  const char *name, struct record *record,
  unsigned long long *val, int err)
 {
-   if (!field) {
-   if (err)
-   trace_seq_printf(s, "", name);
+   if (!field)
return -1;
-   }
 
-   if (pevent_read_number_field(field, record->data, val)) {
-   if (err)
-   trace_seq_printf(s, " %s=INVALID", name);
+   if (pevent_read_number_field(field, record->data, val))
return -1;
-   }
 
return 0;
 }
@@ -4489,11 +4483,8 @@ void *pevent_get_field_raw(struct trace_seq *s, struct 
event_format *event,
 
field = pevent_find_field(event, name);
 
-   if (!field) {
-   if (err)
-   trace_seq_printf(s, "", name);
+   if (!field)
return NULL;
-   }
 
/* Allow @len to be NULL */
if (!len)
-- 
1.7.1

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Don't spam trace report about missing fields

[Steven Rostedt]

On Mon, 2010-11-29 at 18:20 +0200, Avi Kivity wrote:
> Instead of spamming the output with complaints about missing fields, simply
> return an error to the caller, which can print something out or do something
> more intelligent about them.
> 
> Fixes kvm plugin tracing older kernels.

Hmm, what about just passing in err=0?

-- Steve

> 
> Signed-off-by: Avi Kivity 
> ---
>  parse-events.c |   15 +++
>  1 files changed, 3 insertions(+), 12 deletions(-)
> 
> diff --git a/parse-events.c b/parse-events.c
> index f0b0324..7a4c3a6 100644
> --- a/parse-events.c
> +++ b/parse-events.c
> @@ -4446,17 +4446,11 @@ int get_field_val(struct trace_seq *s, struct 
> format_field *field,
> const char *name, struct record *record,
> unsigned long long *val, int err)
>  {
> - if (!field) {
> - if (err)
> - trace_seq_printf(s, "", name);
> + if (!field)
>   return -1;
> - }
>  
> - if (pevent_read_number_field(field, record->data, val)) {
> - if (err)
> - trace_seq_printf(s, " %s=INVALID", name);
> + if (pevent_read_number_field(field, record->data, val))
>   return -1;
> - }
>  
>   return 0;
>  }
> @@ -4489,11 +4483,8 @@ void *pevent_get_field_raw(struct trace_seq *s, struct 
> event_format *event,
>  
>   field = pevent_find_field(event, name);
>  
> - if (!field) {
> - if (err)
> - trace_seq_printf(s, "", name);
> + if (!field)
>   return NULL;
> - }
>  
>   /* Allow @len to be NULL */
>   if (!len)


--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] Don't spam trace report about missing fields

[Avi Kivity]

On 11/29/2010 06:26 PM, Steven Rostedt wrote:

On Mon, 2010-11-29 at 18:20 +0200, Avi Kivity wrote:
>  Instead of spamming the output with complaints about missing fields, simply
>  return an error to the caller, which can print something out or do something
>  more intelligent about them.
>
>  Fixes kvm plugin tracing older kernels.

Hmm, what about just passing in err=0?


I was just patching the printk out it mindlessly.  Will send a new patch.

--
error compiling committee.c: too many arguments to function

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html