Re: Re: Re: [PATCH 0/2] [BUGFIX] printk: Fix message continuation breakage involved with structured printk

2013-12-23 Thread Yoshihiro YUNOMAE
(2013/12/24 12:00), Kay Sievers wrote: On Tue, Dec 24, 2013 at 3:50 AM, Yoshihiro YUNOMAE wrote: (2013/12/20 20:29), Kay Sievers wrote: On Fri, Dec 20, 2013 at 10:41 AM, Yoshihiro YUNOMAE wrote: This patch set fixes message continuation breakage involved with structured printk. A SCSI

Re: Re: [PATCH 0/2] [BUGFIX] printk: Fix message continuation breakage involved with structured printk

2013-12-23 Thread Yoshihiro YUNOMAE
Hi Kay, Thank you for replying me. (2013/12/20 20:29), Kay Sievers wrote: On Fri, Dec 20, 2013 at 10:41 AM, Yoshihiro YUNOMAE wrote: This patch set fixes message continuation breakage involved with structured printk. A SCSI driver may output two continuation error messages like

Re: Re: [PATCH 0/2] [BUGFIX] printk: Fix message continuation breakage involved with structured printk

2013-12-23 Thread Yoshihiro YUNOMAE
Hi Kay, Thank you for replying me. (2013/12/20 20:29), Kay Sievers wrote: On Fri, Dec 20, 2013 at 10:41 AM, Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: This patch set fixes message continuation breakage involved with structured printk. A SCSI driver may output two continuation

Re: Re: Re: [PATCH 0/2] [BUGFIX] printk: Fix message continuation breakage involved with structured printk

2013-12-23 Thread Yoshihiro YUNOMAE
(2013/12/24 12:00), Kay Sievers wrote: On Tue, Dec 24, 2013 at 3:50 AM, Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: (2013/12/20 20:29), Kay Sievers wrote: On Fri, Dec 20, 2013 at 10:41 AM, Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: This patch set fixes message

[PATCH 0/2] [BUGFIX] printk: Fix message continuation breakage involved with structured printk

2013-12-20 Thread Yoshihiro YUNOMAE
e continued, this patch outputs those multiple dictionary information when we read /dev/kmsg. Thanks! --- Yoshihiro YUNOMAE (2): printk: Add dictionary information in structure cont printk: Delete LOG_NEWLINE flag for structured printk kernel

[PATCH 2/2] printk: Delete LOG_NEWLINE flag for structured printk

2013-12-20 Thread Yoshihiro YUNOMAE
4,229318,1819044105,-;( 4,229319,1819044108,-;(C 4,229320,1819044112,-;AAAAAAAAA) 4,229321,1819044115,-;) 4,229322,1819044118,-;(B) bb

[PATCH 1/2] printk: Add dictionary information in structure cont

2013-12-20 Thread Yoshihiro YUNOMAE
, then the information in cont is stored to log_buf. Signed-off-by: Yoshihiro YUNOMAE Cc: Kay Sievers Cc: Andrew Morton Cc: Joe Perches Cc: Tejun Heo Cc: Frederic Weisbecker Cc: linux-kernel@vger.kernel.org --- kernel/printk/printk.c | 70 1 file changed, 47

[PATCH 1/2] printk: Add dictionary information in structure cont

2013-12-20 Thread Yoshihiro YUNOMAE
, then the information in cont is stored to log_buf. Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com Cc: Kay Sievers k...@vrfy.org Cc: Andrew Morton a...@linux-foundation.org Cc: Joe Perches j...@perches.com Cc: Tejun Heo t...@kernel.org Cc: Frederic Weisbecker fweis...@gmail.com Cc: linux-kernel

[PATCH 2/2] printk: Delete LOG_NEWLINE flag for structured printk

2013-12-20 Thread Yoshihiro YUNOMAE
4,229325,1819046118,-;B) Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com Cc: Kay Sievers k...@vrfy.org Cc: Andrew Morton a...@linux-foundation.org Cc: Joe Perches j...@perches.com Cc: Tejun Heo t...@kernel.org Cc: Frederic

[PATCH 0/2] [BUGFIX] printk: Fix message continuation breakage involved with structured printk

2013-12-20 Thread Yoshihiro YUNOMAE
those multiple dictionary information when we read /dev/kmsg. Thanks! --- Yoshihiro YUNOMAE (2): printk: Add dictionary information in structure cont printk: Delete LOG_NEWLINE flag for structured printk kernel/printk/printk.c | 70

[PATCH V3 3/6] trace-cmd: Use poll(2) to wait for a message

2013-12-16 Thread Yoshihiro YUNOMAE
Use poll(2) to wait for a message. If a client/server cannot send a message for any reasons, the current server/client will wait in a blocking read operation. So, we use poll(2) for avoiding remaining in a blocking state. Signed-off-by: Yoshihiro YUNOMAE --- trace-msg.c | 42

[PATCH V3 0/6] trace-cmd: Support the feature recording trace data of guests on the host

2013-12-16 Thread Yoshihiro YUNOMAE
eature of trace data of guests and host to "report" Changes in V2: [1/5] Add a comment in open_udp() [2/5] Regacy protocol support in order to keep backward compatibility Changes in V3: [2/6] Change the license of trace-msg.c to LGPL v2.1 [4/6] Change _nw/_NW to _net/_NET [5/6] Change

[PATCH V3 1/6] [CLEANUP] trace-cmd: Split out binding a port and fork reader from open_udp()

2013-12-16 Thread Yoshihiro YUNOMAE
Split out binding a port and fork reader from open_udp() for avoiding duplicate codes between listen mode and virt-server mode. Changes in V2: Add a comment in open_udp() Signed-off-by: Yoshihiro YUNOMAE --- trace-listen.c | 38 ++ 1 file changed, 30

[PATCH V3 4/6] trace-cmd: Add virt-server mode for a virtualization environment

2013-12-16 Thread Yoshihiro YUNOMAE
/) - Use named pipe I/Fs of virtio-serial for trace data paths - Use UNIX domain socket for connecting clients on guests - Use splice(2) for collecting trace data of guests - Use libvirt when we boot guests Changes in V3: Change _nw/_NW to _net/_NET Signed-off-by: Yoshihiro YUNOM

[PATCH V3 6/6] trace-cmd: Add --dom option which makes a domain directory to virt-server

2013-12-16 Thread Yoshihiro YUNOMAE
-server --dom guest1 -c 2 - trace-cmd creates guest2 and guest3 directories # trace-cmd virt-server --dom guest2 -c 3 --dom guest3 -c 1 Changes in V3: Add this patch Signed-off-by: Yoshihiro YUNOMAE --- Documentation/trace-cmd-virt-server.1.txt | 57 --- trace-listen.c

[PATCH V3 5/6] trace-cmd: Add --virt option for record mode

2013-12-16 Thread Yoshihiro YUNOMAE
Change _nw/_NW to _net/_NET Signed-off-by: Yoshihiro YUNOMAE --- Documentation/trace-cmd-record.1.txt | 11 - trace-cmd.h |3 + trace-msg.c | 80 -- trace-msg.h |4 ++ trac

[PATCH V3 2/6] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-12-16 Thread Yoshihiro YUNOMAE
ers. Then, the message which the client receives is not "V2" but "\0", so the client will reconnect to the old server as the v1 protocol. Changes in V2: Regacy porotocol support in order to keep backward compatibility Changes in V3: Change the license of trace-msg.c to LGP

[PATCH V3 5/6] trace-cmd: Add --virt option for record mode

2013-12-16 Thread Yoshihiro YUNOMAE
: Change _nw/_NW to _net/_NET Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com --- Documentation/trace-cmd-record.1.txt | 11 - trace-cmd.h |3 + trace-msg.c | 80 -- trace-msg.h

[PATCH V3 2/6] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-12-16 Thread Yoshihiro YUNOMAE
v2.1 Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com --- Makefile |2 trace-cmd.h| 11 + trace-listen.c | 133 +++ trace-msg.c| 683 trace-msg.h| 27 ++ trace-output.c |4 trace

[PATCH V3 3/6] trace-cmd: Use poll(2) to wait for a message

2013-12-16 Thread Yoshihiro YUNOMAE
Use poll(2) to wait for a message. If a client/server cannot send a message for any reasons, the current server/client will wait in a blocking read operation. So, we use poll(2) for avoiding remaining in a blocking state. Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com

[PATCH V3 0/6] trace-cmd: Support the feature recording trace data of guests on the host

2013-12-16 Thread Yoshihiro YUNOMAE
this patch based on Steven's review (https://lkml.org/lkml/2013/10/14/618) Thank you, --- Yoshihiro YUNOMAE (6): [CLEANUP] trace-cmd: Split out binding a port and fork reader from open_udp() trace-cmd: Apply the trace-msg protocol for communication between a server and clients

[PATCH V3 1/6] [CLEANUP] trace-cmd: Split out binding a port and fork reader from open_udp()

2013-12-16 Thread Yoshihiro YUNOMAE
Split out binding a port and fork reader from open_udp() for avoiding duplicate codes between listen mode and virt-server mode. Changes in V2: Add a comment in open_udp() Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com --- trace-listen.c | 38

[PATCH V3 4/6] trace-cmd: Add virt-server mode for a virtualization environment

2013-12-16 Thread Yoshihiro YUNOMAE
of virtio-serial for trace data paths - Use UNIX domain socket for connecting clients on guests - Use splice(2) for collecting trace data of guests Restrictions - Use libvirt when we boot guests Changes in V3: Change _nw/_NW to _net/_NET Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae

[PATCH V3 6/6] trace-cmd: Add --dom option which makes a domain directory to virt-server

2013-12-16 Thread Yoshihiro YUNOMAE
. # trace-cmd virt-server --dom guest1 -c 2 - trace-cmd creates guest2 and guest3 directories # trace-cmd virt-server --dom guest2 -c 3 --dom guest3 -c 1 Changes in V3: Add this patch Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com --- Documentation/trace-cmd-virt-server.1

Re: [PATCH v2] trace-cmd: Add record --clock option to specify a trace clock

2013-12-15 Thread Yoshihiro YUNOMAE
re created, I think. Example: trace-cmd record -e sched --clock local -B foo -e block --clock x86-tsc Thanks, Yoshihiro YUNOMAE -- Yoshihiro YUNOMAE Software Platform Research Dept. Linux Technology Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: yoshihiro.yunomae...@hitachi.com -- To

Re: [PATCH v2] trace-cmd: Add record --clock option to specify a trace clock

2013-12-15 Thread Yoshihiro YUNOMAE
foo -e block --clock x86-tsc Thanks, Yoshihiro YUNOMAE -- Yoshihiro YUNOMAE Software Platform Research Dept. Linux Technology Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: yoshihiro.yunomae...@hitachi.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel

Re: [PATCH] timer: Add __sched to msleep() and msleep_interruptible()

2013-11-25 Thread Yoshihiro YUNOMAE
Hi, Would you review this patch? Thanks, Yoshihiro YUNOMAE (2013/11/11 17:18), Yoshihiro YUNOMAE wrote: Add __sched to msleep() and msleep_interruptible() for avoiding to show these functions in WCHAN. If a driver calls msleep() or msleep_interruptible() in a lot of places, users cannot

Re: [PATCH] timer: Add __sched to msleep() and msleep_interruptible()

2013-11-25 Thread Yoshihiro YUNOMAE
Hi, Would you review this patch? Thanks, Yoshihiro YUNOMAE (2013/11/11 17:18), Yoshihiro YUNOMAE wrote: Add __sched to msleep() and msleep_interruptible() for avoiding to show these functions in WCHAN. If a driver calls msleep() or msleep_interruptible() in a lot of places, users cannot

Re: [PATCH -tip 0/2] ftrace: Introduce the new I/F "nr_saved_cmdlines"

2013-11-19 Thread Yoshihiro YUNOMAE
Hi, Would you review this patch set? Thanks, Yoshihiro YUNOMAE (2013/11/08 13:26), Yoshihiro YUNOMAE wrote: Hi, This patch set introduces the new I/F "nr_saved_cmdlines" for increasing the number of saved cmdlines. Current saved_cmdlines can store just 128 command names and PIDs, b

Re: [PATCH -tip 0/2] ftrace: Introduce the new I/F nr_saved_cmdlines

2013-11-19 Thread Yoshihiro YUNOMAE
Hi, Would you review this patch set? Thanks, Yoshihiro YUNOMAE (2013/11/08 13:26), Yoshihiro YUNOMAE wrote: Hi, This patch set introduces the new I/F nr_saved_cmdlines for increasing the number of saved cmdlines. Current saved_cmdlines can store just 128 command names and PIDs, but process

[PATCH] timer: Add __sched to msleep() and msleep_interruptible()

2013-11-11 Thread Yoshihiro YUNOMAE
this patch, users can confirm the function calling msleep() or msleep_interruptible() from WCHAN. Signed-off-by: Yoshihiro YUNOMAE Cc: Thomas Gleixner Cc: linux-kernel@vger.kernel.org --- kernel/timer.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/timer.c b

[PATCH] timer: Add __sched to msleep() and msleep_interruptible()

2013-11-11 Thread Yoshihiro YUNOMAE
this patch, users can confirm the function calling msleep() or msleep_interruptible() from WCHAN. Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com Cc: Thomas Gleixner t...@linutronix.de Cc: linux-kernel@vger.kernel.org --- kernel/timer.c |4 ++-- 1 file changed, 2 insertions

[PATCH -tip 2/2] ftrace: Introduce nr_saved_cmdlines I/F

2013-11-07 Thread Yoshihiro YUNOMAE
off-by: Yoshihiro YUNOMAE Cc: Steven Rostedt Cc: Frederic Weisbecker Cc: Ingo Molnar Cc: linux-kernel@vger.kernel.org --- kernel/trace/trace.c | 211 +- 1 file changed, 189 insertions(+), 22 deletions(-) diff --git a/kernel/trace/trace.c b/ker

[PATCH -tip 1/2] ftrace: Make saved_cmdlines use seq_read

2013-11-07 Thread Yoshihiro YUNOMAE
usage, and a corruption of the output data. To address these issues, make saved_cmdlines use seq_read. Signed-off-by: Hidehiro Kawai Signed-off-by: Yoshihiro YUNOMAE Cc: Steven Rostedt Cc: Frederic Weisbecker Cc: Ingo Molnar Cc: linux-kernel@vger.kernel.org --- kernel/trace/trace.c | 89

[PATCH -tip 0/2] ftrace: Introduce the new I/F "nr_saved_cmdlines"

2013-11-07 Thread Yoshihiro YUNOMAE
the number of command name will be stored. And, if we read the I/F, we can get current maximum number of command name. The default number is 128 which is current default number, so this patch does not change the usage of memory for saved_cmdlines when we boot kernel. Thanks! --- Yoshihiro YUNOM

[PATCH -tip 1/2] ftrace: Make saved_cmdlines use seq_read

2013-11-07 Thread Yoshihiro YUNOMAE
usage, and a corruption of the output data. To address these issues, make saved_cmdlines use seq_read. Signed-off-by: Hidehiro Kawai hidehiro.kawai...@hitachi.com Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com Cc: Steven Rostedt rost...@goodmis.org Cc: Frederic Weisbecker fweis

[PATCH -tip 0/2] ftrace: Introduce the new I/F nr_saved_cmdlines

2013-11-07 Thread Yoshihiro YUNOMAE
will be stored. And, if we read the I/F, we can get current maximum number of command name. The default number is 128 which is current default number, so this patch does not change the usage of memory for saved_cmdlines when we boot kernel. Thanks! --- Yoshihiro YUNOMAE (2): ftrace: Make

[PATCH -tip 2/2] ftrace: Introduce nr_saved_cmdlines I/F

2013-11-07 Thread Yoshihiro YUNOMAE
-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com Cc: Steven Rostedt rost...@goodmis.org Cc: Frederic Weisbecker fweis...@gmail.com Cc: Ingo Molnar mi...@redhat.com Cc: linux-kernel@vger.kernel.org --- kernel/trace/trace.c | 211 +- 1 file changed

[tip:perf/core] tools lib traceevent: Add support for extracting trace_clock in report

2013-11-04 Thread tip-bot for Yoshihiro YUNOMAE
Commit-ID: 1b372ca52a02cc97520c13d79bdfb0a7ff81b772 Gitweb: http://git.kernel.org/tip/1b372ca52a02cc97520c13d79bdfb0a7ff81b772 Author: Yoshihiro YUNOMAE AuthorDate: Fri, 1 Nov 2013 17:53:53 -0400 Committer: Arnaldo Carvalho de Melo CommitDate: Mon, 4 Nov 2013 14:33:12 -0300 tools lib

[tip:perf/core] tools lib traceevent: Add support for extracting trace_clock in report

2013-11-04 Thread tip-bot for Yoshihiro YUNOMAE
Commit-ID: 1b372ca52a02cc97520c13d79bdfb0a7ff81b772 Gitweb: http://git.kernel.org/tip/1b372ca52a02cc97520c13d79bdfb0a7ff81b772 Author: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com AuthorDate: Fri, 1 Nov 2013 17:53:53 -0400 Committer: Arnaldo Carvalho de Melo a...@redhat.com

Re: [PATCH V2 4/5] trace-cmd: Add virt-server mode for a virtualization environment

2013-10-22 Thread Yoshihiro YUNOMAE
(2013/10/18 11:32), Steven Rostedt wrote: On Fri, 13 Sep 2013 11:06:37 +0900 Yoshihiro YUNOMAE wrote: static int *create_all_readers(int cpus, const char *node, const char *port, - int pagesize, int fd) + const char *domain, int

Re: Re: [PATCH V2 2/5] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-10-22 Thread Yoshihiro YUNOMAE
(2013/10/18 11:19), Steven Rostedt wrote: On Fri, 13 Sep 2013 11:06:32 +0900 Yoshihiro YUNOMAE wrote: diff --git a/trace-msg.c b/trace-msg.c new file mode 100644 index 000..cf82ff6 --- /dev/null +++ b/trace-msg.c @@ -0,0 +1,683 @@ +/* + * trace-msg.c : define message protocol

Re: [PATCH V2 0/5] trace-cmd: Support the feature recording trace data of guests on the host

2013-10-22 Thread Yoshihiro YUNOMAE
Hi Steven, (2013/10/19 0:06), Steven Rostedt wrote: On Fri, 13 Sep 2013 11:06:27 +0900 Yoshihiro YUNOMAE wrote: Hi Steven, This is a v2 patch set for realizing a part of "Integrated trace" feature which is a trace merging system for a virtualization environment. Currently, trac

Re: Re: [PATCH V2 2/5] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-10-22 Thread Yoshihiro YUNOMAE
(2013/10/18 11:19), Steven Rostedt wrote: On Fri, 13 Sep 2013 11:06:32 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: diff --git a/trace-msg.c b/trace-msg.c new file mode 100644 index 000..cf82ff6 --- /dev/null +++ b/trace-msg.c @@ -0,0 +1,683 @@ +/* + * trace-msg.c

Re: [PATCH V2 0/5] trace-cmd: Support the feature recording trace data of guests on the host

2013-10-22 Thread Yoshihiro YUNOMAE
Hi Steven, (2013/10/19 0:06), Steven Rostedt wrote: On Fri, 13 Sep 2013 11:06:27 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: Hi Steven, This is a v2 patch set for realizing a part of Integrated trace feature which is a trace merging system for a virtualization environment

Re: [PATCH V2 4/5] trace-cmd: Add virt-server mode for a virtualization environment

2013-10-22 Thread Yoshihiro YUNOMAE
(2013/10/18 11:32), Steven Rostedt wrote: On Fri, 13 Sep 2013 11:06:37 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: static int *create_all_readers(int cpus, const char *node, const char *port, - int pagesize, int fd

Re: Re: [PATCH V2 2/5] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-10-17 Thread Yoshihiro YUNOMAE
(2013/10/15 11:21), Steven Rostedt wrote: On Fri, 13 Sep 2013 11:06:32 +0900 Yoshihiro YUNOMAE wrote: Apply trace-msg protocol for communication between a server and clients. Currently, trace-listen(server) and trace-record -N(client) operate as follows

Re: Re: [PATCH V2 0/5] trace-cmd: Support the feature recording trace data of guests on the host

2013-10-17 Thread Yoshihiro YUNOMAE
Hi Steven, (2013/10/15 6:26), Steven Rostedt wrote: On Fri, 13 Sep 2013 11:06:27 +0900 Yoshihiro YUNOMAE wrote: 1. Run virt-server on a host # trace-cmd virt-server 2. Make guest domain directory # mkdir -p /tmp/trace-cmd/virt/ # chmod 710 /tmp/trace-cmd/virt/ # chgrp qemu

Re: Re: [PATCH V2 0/5] trace-cmd: Support the feature recording trace data of guests on the host

2013-10-17 Thread Yoshihiro YUNOMAE
Hi Steven, (2013/10/15 6:26), Steven Rostedt wrote: On Fri, 13 Sep 2013 11:06:27 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: How to use 1. Run virt-server on a host # trace-cmd virt-server 2. Make guest domain directory # mkdir -p /tmp/trace-cmd/virt/domain

Re: Re: [PATCH V2 2/5] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-10-17 Thread Yoshihiro YUNOMAE
(2013/10/15 11:21), Steven Rostedt wrote: On Fri, 13 Sep 2013 11:06:32 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: Apply trace-msg protocol for communication between a server and clients. Currently, trace-listen(server) and trace-record -N(client) operate as follows

Re: [PATCH V2 0/5] trace-cmd: Support the feature recording trace data of guests on the host

2013-10-10 Thread Yoshihiro YUNOMAE
Hi Steven, Would you review this patch set? Thanks, Yoshihiro YUNOMAE (2013/09/13 11:06), Yoshihiro YUNOMAE wrote: Hi Steven, This is a v2 patch set for realizing a part of "Integrated trace" feature which is a trace merging system for a virtualization environment. Currently, trac

Re: [PATCH V2 0/5] trace-cmd: Support the feature recording trace data of guests on the host

2013-10-10 Thread Yoshihiro YUNOMAE
Hi Steven, Would you review this patch set? Thanks, Yoshihiro YUNOMAE (2013/09/13 11:06), Yoshihiro YUNOMAE wrote: Hi Steven, This is a v2 patch set for realizing a part of Integrated trace feature which is a trace merging system for a virtualization environment. Currently, trace-cmd does

[PATCH V2 4/5] trace-cmd: Add virt-server mode for a virtualization environment

2013-09-12 Thread Yoshihiro YUNOMAE
/) - Use named pipe I/Fs of virtio-serial for trace data paths - Use UNIX domain socket for connecting clients on guests - Use splice(2) for collecting trace data of guests - Use libvirt when we boot guests Signed-off-by: Yoshihiro YUNOMAE --- Documentation/trace-cmd-virt-server.1.txt | 89

[PATCH V2 3/5] trace-cmd: Use poll(2) to wait for a message

2013-09-12 Thread Yoshihiro YUNOMAE
Use poll(2) to wait for a message. If a client/server cannot send a message for any reasons, the current server/client will wait in a blocking read operation. So, we use poll(2) for avoiding remaining in a blocking state. Signed-off-by: Yoshihiro YUNOMAE --- trace-msg.c | 42

[PATCH V2 2/5] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-09-12 Thread Yoshihiro YUNOMAE
the old server, so the client send "V2" as cpu numbers, "" as page size, and "0" as no options. On the other hands, the old server will understand the messages as cpus=0, pagesize=, and options=0, and then the server will send the message "\0" as port number

[PATCH V2 5/5] trace-cmd: Add --virt option for record mode

2013-09-12 Thread Yoshihiro YUNOMAE
* The client using virtio-serial does not wait for the connection message "tracecmd" from the server. The client sends the connection message MSG_TCONNECT first. This feature can use from kernel-3.6 which supports splice_read for ftrace and splice_write for virtio-serial. Signed-off-by:

[PATCH V2 0/5] trace-cmd: Support the feature recording trace data of guests on the host

2013-09-12 Thread Yoshihiro YUNOMAE
~2.9[%] ~1.7[%] - Add an agent mode based on "record --virt" - Add a merging feature of trace data of guests and host to "report" Changes in V2: [1/5] Add a comment in open_udp() [2/5] Regacy protocol support in order to keep backward compatibility Thank you,

[PATCH V2 1/5] [CLEANUP] trace-cmd: Split out binding a port and fork reader from open_udp()

2013-09-12 Thread Yoshihiro YUNOMAE
Split out binding a port and fork reader from open_udp() for avoiding duplicate codes between listen mode and virt-server mode. Changes in V2: Add a comment in open_udp() Signed-off-by: Yoshihiro YUNOMAE --- trace-listen.c | 38 ++ 1 file changed, 30

[PATCH V2 1/5] [CLEANUP] trace-cmd: Split out binding a port and fork reader from open_udp()

2013-09-12 Thread Yoshihiro YUNOMAE
Split out binding a port and fork reader from open_udp() for avoiding duplicate codes between listen mode and virt-server mode. Changes in V2: Add a comment in open_udp() Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com --- trace-listen.c | 38

[PATCH V2 0/5] trace-cmd: Support the feature recording trace data of guests on the host

2013-09-12 Thread Yoshihiro YUNOMAE
Changes in V2: [1/5] Add a comment in open_udp() [2/5] Regacy protocol support in order to keep backward compatibility Thank you, --- Yoshihiro YUNOMAE (5): [CLEANUP] trace-cmd: Split out binding a port and fork reader from open_udp() trace-cmd: Apply the trace-msg protocol

[PATCH V2 5/5] trace-cmd: Add --virt option for record mode

2013-09-12 Thread Yoshihiro YUNOMAE
-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com --- Documentation/trace-cmd-record.1.txt | 11 - trace-cmd.h |3 + trace-msg.c | 80 -- trace-msg.h |4 ++ trace-record.c

[PATCH V2 4/5] trace-cmd: Add virt-server mode for a virtualization environment

2013-09-12 Thread Yoshihiro YUNOMAE
of virtio-serial for trace data paths - Use UNIX domain socket for connecting clients on guests - Use splice(2) for collecting trace data of guests Restrictions - Use libvirt when we boot guests Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com --- Documentation/trace-cmd-virt

[PATCH V2 2/5] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-09-12 Thread Yoshihiro YUNOMAE
the message \0 as port numbers. Then, the message which the client receives is not V2 but \0, so the client will reconnect to the old server as the v1 protocol. Changes in V2: Regacy porotocol support in order to keep backward compatibility Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae

[PATCH V2 3/5] trace-cmd: Use poll(2) to wait for a message

2013-09-12 Thread Yoshihiro YUNOMAE
Use poll(2) to wait for a message. If a client/server cannot send a message for any reasons, the current server/client will wait in a blocking read operation. So, we use poll(2) for avoiding remaining in a blocking state. Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com

Re: Re: [PATCH] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking of ioapic_lock

2013-09-04 Thread Yoshihiro YUNOMAE
(2013/09/03 21:44), Eric W. Biederman wrote: > Yoshihiro YUNOMAE writes: > >> (2013/09/03 9:12), Eric W. Biederman wrote: >>>>>> Then again looking at the output of the latest dmesg, it seems the IO >>>>>> APIC >>>>>> is in

Re: Re: [PATCH] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking of ioapic_lock

2013-09-04 Thread Yoshihiro YUNOMAE
(2013/09/03 21:44), Eric W. Biederman wrote: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com writes: (2013/09/03 9:12), Eric W. Biederman wrote: Then again looking at the output of the latest dmesg, it seems the IO APIC is initialized way before the tsc is calibrated. So I am not sure

Re: Re: [PATCH] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking of ioapic_lock

2013-09-03 Thread Yoshihiro YUNOMAE
ardware devices unless it can't possibily do it. And > apics for the longest time were very very hard to reset in the target > kernel, but now that they are not. It makes sense for time permitting > to remove the now unnecessary code in the crashing kernel. Because > ultimately the les

Re: Re: [PATCH] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking of ioapic_lock

2013-09-03 Thread Yoshihiro YUNOMAE
code we have the fewer possible ways we can fail in a known broken kernel. Yes, I agree with you. Thanks, Yoshihiro YUNOMAE -- Yoshihiro YUNOMAE Software Platform Research Dept. Linux Technology Center Hitachi, Ltd., Yokohama Research Laboratory E-mail: yoshihiro.yunomae...@hitachi.com

Re: Re: [PATCH] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking of ioapic_lock

2013-09-01 Thread Yoshihiro YUNOMAE
Hi Eric and Don, Sorry for the late reply. (2013/08/31 9:58), Eric W. Biederman wrote: > Don Zickus writes: > >> On Tue, Aug 27, 2013 at 12:41:51PM +0900, Yoshihiro YUNOMAE wrote: >>> Hi Don, >>> >>> Sorry for the late reply. >>> >>>

Re: Re: [PATCH] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking of ioapic_lock

2013-09-01 Thread Yoshihiro YUNOMAE
Hi Eric and Don, Sorry for the late reply. (2013/08/31 9:58), Eric W. Biederman wrote: Don Zickus dzic...@redhat.com writes: On Tue, Aug 27, 2013 at 12:41:51PM +0900, Yoshihiro YUNOMAE wrote: Hi Don, Sorry for the late reply. (2013/08/22 22:11), Don Zickus wrote: On Thu, Aug 22, 2013

Re: Re: [RFC PATCH 08/11] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-08-28 Thread Yoshihiro YUNOMAE
(2013/08/28 23:42), Steven Rostedt wrote: On Wed, 28 Aug 2013 20:30:17 +0900 Yoshihiro YUNOMAE wrote: (2013/08/27 22:05), Steven Rostedt wrote: On Tue, 27 Aug 2013 19:23:24 +0900 Yoshihiro YUNOMAE wrote: OK, let me check that. Even if the old server will receive "V2", the server

Re: Re: [RFC PATCH 00/11] trace-cmd: Support the feature recording trace data of guests on the host

2013-08-28 Thread Yoshihiro YUNOMAE
(2013/08/27 22:00), Steven Rostedt wrote: On Tue, 27 Aug 2013 17:07:34 +0900 Yoshihiro YUNOMAE wrote: et, but I'd like to add such feature to fix host/guest clock difference in the next series. TSC offset values can be gotten as write_tsc_offset trace event from kernel-3.11. (see https

Re: Re: [RFC PATCH 08/11] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-08-28 Thread Yoshihiro YUNOMAE
(2013/08/27 22:05), Steven Rostedt wrote: On Tue, 27 Aug 2013 19:23:24 +0900 Yoshihiro YUNOMAE wrote: OK, let me check that. Even if the old server will receive "V2", the server will send port numbers instead of "V2" due to the old protocol. In that time, the new client

Re: Re: [RFC PATCH 08/11] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-08-28 Thread Yoshihiro YUNOMAE
(2013/08/27 22:05), Steven Rostedt wrote: On Tue, 27 Aug 2013 19:23:24 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: OK, let me check that. Even if the old server will receive V2, the server will send port numbers instead of V2 due to the old protocol. In that time, the new

Re: Re: [RFC PATCH 00/11] trace-cmd: Support the feature recording trace data of guests on the host

2013-08-28 Thread Yoshihiro YUNOMAE
(2013/08/27 22:00), Steven Rostedt wrote: On Tue, 27 Aug 2013 17:07:34 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: et, but I'd like to add such feature to fix host/guest clock difference in the next series. TSC offset values can be gotten as write_tsc_offset trace event from

Re: Re: [RFC PATCH 08/11] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-08-28 Thread Yoshihiro YUNOMAE
(2013/08/28 23:42), Steven Rostedt wrote: On Wed, 28 Aug 2013 20:30:17 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: (2013/08/27 22:05), Steven Rostedt wrote: On Tue, 27 Aug 2013 19:23:24 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: OK, let me check

Re: Re: [RFC PATCH 08/11] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-08-27 Thread Yoshihiro YUNOMAE
(2013/08/27 0:11), Steven Rostedt wrote: On Mon, 26 Aug 2013 10:50:33 +0900 Yoshihiro YUNOMAE wrote: 0. old server and old client Old servers send "tracecmd" as the first message. Old clients compare the first 8byte of the first message with "tracecmd". 1. new server

Re: Re: [RFC PATCH 07/11] [CLEANUP] trace-cmd: Split out binding a port and fork reader from open_udp()

2013-08-27 Thread Yoshihiro YUNOMAE
(2013/08/26 23:37), Steven Rostedt wrote: On Mon, 26 Aug 2013 10:48:15 +0900 Yoshihiro YUNOMAE wrote: +static int open_udp(const char *node, const char *port, int *pid, + int cpu, int pagesize, int start_port) +{ + int sfd; + int num_port; + + num_port

Re: Re: [RFC PATCH 00/11] trace-cmd: Support the feature recording trace data of guests on the host

2013-08-27 Thread Yoshihiro YUNOMAE
Hi Steven, (2013/08/26 23:22), Steven Rostedt wrote: On Mon, 26 Aug 2013 10:46:38 +0900 Yoshihiro YUNOMAE wrote: The --date option is used because the two machines are not in sync with the trace time stamp. What the date option does, is to sync the timestamp up with the gettimeofday

Re: Re: [RFC PATCH 00/11] trace-cmd: Support the feature recording trace data of guests on the host

2013-08-27 Thread Yoshihiro YUNOMAE
Hi Steven, (2013/08/26 23:22), Steven Rostedt wrote: On Mon, 26 Aug 2013 10:46:38 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: The --date option is used because the two machines are not in sync with the trace time stamp. What the date option does, is to sync the timestamp

Re: Re: [RFC PATCH 07/11] [CLEANUP] trace-cmd: Split out binding a port and fork reader from open_udp()

2013-08-27 Thread Yoshihiro YUNOMAE
(2013/08/26 23:37), Steven Rostedt wrote: On Mon, 26 Aug 2013 10:48:15 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: +static int open_udp(const char *node, const char *port, int *pid, + int cpu, int pagesize, int start_port) +{ + int sfd; + int

Re: Re: [RFC PATCH 08/11] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-08-27 Thread Yoshihiro YUNOMAE
(2013/08/27 0:11), Steven Rostedt wrote: On Mon, 26 Aug 2013 10:50:33 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: Network 0. old server and old client Old servers send tracecmd as the first message. Old clients compare the first 8byte of the first message with tracecmd. 1

Re: Re: Re: [PATCH] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking of ioapic_lock

2013-08-26 Thread Yoshihiro YUNOMAE
Hi Don, Sorry for the late reply. (2013/08/22 22:11), Don Zickus wrote: On Thu, Aug 22, 2013 at 05:38:07PM +0900, Yoshihiro YUNOMAE wrote: So, I agree with Eric, let's remove the disable_IO_APIC() stuff and keep the code simpler. Thank you for commenting about my patch. I didn't know you

Re: Re: Re: [PATCH] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking of ioapic_lock

2013-08-26 Thread Yoshihiro YUNOMAE
Hi Don, Sorry for the late reply. (2013/08/22 22:11), Don Zickus wrote: On Thu, Aug 22, 2013 at 05:38:07PM +0900, Yoshihiro YUNOMAE wrote: So, I agree with Eric, let's remove the disable_IO_APIC() stuff and keep the code simpler. Thank you for commenting about my patch. I didn't know you

Re: Re: [RFC PATCH 08/11] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-08-25 Thread Yoshihiro YUNOMAE
(2013/08/21 2:56), Steven Rostedt wrote: On Mon, 19 Aug 2013 18:46:39 +0900 Yoshihiro YUNOMAE wrote: This message protocol is incompatible with the previous unstructured message protocol. So, if an old(new)-version client tries to connect to an new(old)-version server, the operation should

Re: Re: [RFC PATCH 07/11] [CLEANUP] trace-cmd: Split out binding a port and fork reader from open_udp()

2013-08-25 Thread Yoshihiro YUNOMAE
(2013/08/21 2:49), Steven Rostedt wrote: On Mon, 19 Aug 2013 18:46:37 +0900 Yoshihiro YUNOMAE wrote: Split out binding a port and fork reader from open_udp() for avoiding duplicate codes between listen mode and virt-server mode. Signed-off-by: Yoshihiro YUNOMAE --- trace-listen.c | 34

Re: Re: [RFC PATCH 00/11] trace-cmd: Support the feature recording trace data of guests on the host

2013-08-25 Thread Yoshihiro YUNOMAE
Hi Steven, Thank you for reviewing my patches. Sorry for the late reply. (2013/08/21 1:00), Steven Rostedt wrote: On Mon, 19 Aug 2013 18:46:20 +0900 Yoshihiro YUNOMAE wrote: d) merge feature of trace data of multiple guests and a host in chronological order Current trace-cmd cannot

Re: Re: [RFC PATCH 00/11] trace-cmd: Support the feature recording trace data of guests on the host

2013-08-25 Thread Yoshihiro YUNOMAE
Hi Steven, Thank you for reviewing my patches. Sorry for the late reply. (2013/08/21 1:00), Steven Rostedt wrote: On Mon, 19 Aug 2013 18:46:20 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: d) merge feature of trace data of multiple guests and a host in chronological

Re: Re: [RFC PATCH 07/11] [CLEANUP] trace-cmd: Split out binding a port and fork reader from open_udp()

2013-08-25 Thread Yoshihiro YUNOMAE
(2013/08/21 2:49), Steven Rostedt wrote: On Mon, 19 Aug 2013 18:46:37 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: Split out binding a port and fork reader from open_udp() for avoiding duplicate codes between listen mode and virt-server mode. Signed-off-by: Yoshihiro

Re: Re: [RFC PATCH 08/11] trace-cmd: Apply the trace-msg protocol for communication between a server and clients

2013-08-25 Thread Yoshihiro YUNOMAE
(2013/08/21 2:56), Steven Rostedt wrote: On Mon, 19 Aug 2013 18:46:39 +0900 Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com wrote: This message protocol is incompatible with the previous unstructured message protocol. So, if an old(new)-version client tries to connect to an new(old

Re: Re: [PATCH] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking of ioapic_lock

2013-08-22 Thread Yoshihiro YUNOMAE
(2013/08/20 23:27), Don Zickus wrote: On Tue, Aug 20, 2013 at 03:12:32AM -0700, Eric W. Biederman wrote: Yoshihiro YUNOMAE writes: Hi Ingo, Thank you for fixing typos! OK, I'll fix them and rename to ioapic_zap_locks(). Thank you again! The better fix for this would be to remove

Re: Re: [PATCH] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking of ioapic_lock

2013-08-22 Thread Yoshihiro YUNOMAE
(2013/08/20 23:27), Don Zickus wrote: On Tue, Aug 20, 2013 at 03:12:32AM -0700, Eric W. Biederman wrote: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com writes: Hi Ingo, Thank you for fixing typos! OK, I'll fix them and rename to ioapic_zap_locks(). Thank you again! The better fix

[tip:x86/apic] x86/ioapic/kcrash: Prevent crash_kexec() from deadlocking on ioapic_lock

2013-08-20 Thread tip-bot for Yoshihiro YUNOMAE
Commit-ID: 17405453f4ad0220721a29978692081be6392b8f Gitweb: http://git.kernel.org/tip/17405453f4ad0220721a29978692081be6392b8f Author: Yoshihiro YUNOMAE AuthorDate: Tue, 20 Aug 2013 16:01:07 +0900 Committer: Ingo Molnar CommitDate: Tue, 20 Aug 2013 09:26:33 +0200 x86/ioapic/kcrash

[PATCH V2] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking on ioapic_lock

2013-08-20 Thread Yoshihiro YUNOMAE
t; clear_IO_APIC_pin()->ioapic_read_entry(). Changes in V2: Fix typos, change ioapic_lock_init() to ioapic_zap_locks(), and change "#if defined(CONFIG_X86_IO_APIC)" to "#ifdef CONFIG_X86_IO_APIC". Signed-off-by: Yoshihiro YUNOMAE Cc: Thomas G

[PATCH V2] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking on ioapic_lock

2013-08-20 Thread Yoshihiro YUNOMAE
()- clear_IO_APIC_pin()-ioapic_read_entry(). Changes in V2: Fix typos, change ioapic_lock_init() to ioapic_zap_locks(), and change #if defined(CONFIG_X86_IO_APIC) to #ifdef CONFIG_X86_IO_APIC. Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com Cc: Thomas Gleixner t

[tip:x86/apic] x86/ioapic/kcrash: Prevent crash_kexec() from deadlocking on ioapic_lock

2013-08-20 Thread tip-bot for Yoshihiro YUNOMAE
Commit-ID: 17405453f4ad0220721a29978692081be6392b8f Gitweb: http://git.kernel.org/tip/17405453f4ad0220721a29978692081be6392b8f Author: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com AuthorDate: Tue, 20 Aug 2013 16:01:07 +0900 Committer: Ingo Molnar mi...@kernel.org CommitDate: Tue

Re: Re: [PATCH] [BUGFIX] crash/ioapic: Prevent crash_kexec() from deadlocking of ioapic_lock

2013-08-19 Thread Yoshihiro YUNOMAE
Hi Ingo, Thank you for fixing typos! OK, I'll fix them and rename to ioapic_zap_locks(). Thank you again! Yoshihiro YUNOMAE (2013/08/19 18:46), Ingo Molnar wrote: * Yoshihiro YUNOMAE wrote: Prevent crash_kexec() from deadlocking of ioapic_lock. s/of/on When crash_kexec() is executed

[RFC PATCH 00/11] trace-cmd: Support the feature recording trace data of guests on the host

2013-08-19 Thread Yoshihiro YUNOMAE
Client means "record -N"(NW) or "record --virt"(virtio-serial). NW virtio-serial(splice) client(fedora19)~2.9[%] ~1.7[%] - Add an agent mode based on "record --virt" - Add a merging feature of trace data of guests and host to &quo

[RFC PATCH 02/11] [BUGFIX] trace-cmd: Add waitpid() when recorders are destoried

2013-08-19 Thread Yoshihiro YUNOMAE
if the parent process cannot die at once due to any reasons. Signed-off-by: Yoshihiro YUNOMAE --- trace-listen.c |1 + 1 file changed, 1 insertion(+) diff --git a/trace-listen.c b/trace-listen.c index dec1c00..6c1bcac 100644 --- a/trace-listen.c +++ b/trace-listen.c @@ -446,6 +446,7 @@ static void

[RFC PATCH 07/11] [CLEANUP] trace-cmd: Split out binding a port and fork reader from open_udp()

2013-08-19 Thread Yoshihiro YUNOMAE
Split out binding a port and fork reader from open_udp() for avoiding duplicate codes between listen mode and virt-server mode. Signed-off-by: Yoshihiro YUNOMAE --- trace-listen.c | 34 ++ 1 file changed, 26 insertions(+), 8 deletions(-) diff --git a/trace

<    1   2   3   4   5   >