All of lore.kernel.org
 help / color / mirror / Atom feed
From: Namhyung Kim <namhyung@gmail.com>
To: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Cc: linux-kernel@vger.kernel.org,
	Arnaldo Carvalho de Melo <acme@kernel.org>,
	Tom Zanussi <tzanussi@gmail.com>
Subject: Re: [PATCH 2/2] perf script: handle the num array type in python properly
Date: Thu, 29 May 2014 13:43:20 +0900	[thread overview]
Message-ID: <87d2exgpp3.fsf@sejong.aot.lge.com> (raw)
In-Reply-To: <1401207274-8170-2-git-send-email-bigeasy@linutronix.de> (Sebastian Andrzej Siewior's message of "Tue, 27 May 2014 18:14:34 +0200")

On Tue, 27 May 2014 18:14:34 +0200, Sebastian Andrzej Siewior wrote:
> The raw_syscalls:sys_enter tracer for instance passes has one argument
> named 'arg' which is an array of 6 integers. Right the python scripts
> gets only 0 passed as an argument. The reason is that
> pevent_read_number() can not handle data types of 48 and returns always
> 0.
> This patch changes this by passing num array as list of nums which fit
> the description. As a result python will now see a list named arg which
> contains 6 (integer) items.

Acked-by: Namhyung Kim <namhyung@kernel.org>

But this requires a change in generation time as I faced a type error
when try to print an array argument as number.  I'll post a separate
patch for this.

Thanks,
Namhyung

  reply	other threads:[~2014-05-29  4:43 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-27 16:14 [PATCH 1/2] perf script: move the number processing into its own function Sebastian Andrzej Siewior
2014-05-27 16:14 ` [PATCH 2/2] perf script: handle the num array type in python properly Sebastian Andrzej Siewior
2014-05-29  4:43   ` Namhyung Kim [this message]
2014-05-30  9:39     ` Sebastian Andrzej Siewior
2014-05-29  4:44   ` [PATCH] perf script/python: Print array argument as string Namhyung Kim
2014-06-03 18:55     ` Jiri Olsa
2014-06-12 12:01     ` [tip:perf/core] " tip-bot for Namhyung Kim
2014-06-26 15:37       ` Sebastian Andrzej Siewior
2014-06-26 19:22         ` Jiri Olsa
2014-05-29  4:39 ` [PATCH 1/2] perf script: move the number processing into its own function Namhyung Kim

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87d2exgpp3.fsf@sejong.aot.lge.com \
    --to=namhyung@gmail.com \
    --cc=acme@kernel.org \
    --cc=bigeasy@linutronix.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tzanussi@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.