Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S932162AbbEVExK (ORCPT <rfc822;w@1wt.eu>);
	Fri, 22 May 2015 00:53:10 -0400
Received: from mail9.hitachi.co.jp ([133.145.228.44]:40990 "EHLO
	mail9.hitachi.co.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1751046AbbEVExJ (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Fri, 22 May 2015 00:53:09 -0400
Message-ID: <555EB62D.4070008@hitachi.com>
Date: Fri, 22 May 2015 13:53:01 +0900
From: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
Organization: Hitachi, Ltd., Japan
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0
MIME-Version: 1.0
To: Steven Rostedt <rostedt@goodmis.org>
CC: Hidehiro Kawai <hidehiro.kawai.ez@hitachi.com>,
        yrl.pp-manager.tt@hitachi.com,
        Aaron Fabbri <aaronx.j.fabbri@intel.com>, linux-kernel@vger.kernel.org,
        Divya Vyas <edivya.vyas@gmail.com>
Subject: Re: [PATCH trace-cmd V5 2/6] trace-cmd/listen: Introduce trace-msg
 protocol (protocol v2)
References: <20141222174736.10068.90306.stgit@localhost.localdomain>	<20141222174750.10068.41354.stgit@localhost.localdomain> <20150519231202.35614532@grimm.local.home>
In-Reply-To: <20150519231202.35614532@grimm.local.home>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 2507
Lines: 90

Hi Steven,

On 2015/05/20 12:12, Steven Rostedt wrote:
> On Mon, 22 Dec 2014 12:47:50 -0500
> Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> wrote:
> 
>> Introduce new trace-msg protocol (protocol V2) for more
>> flexible messaging. V1 protocol which is currently used
>> by trace-cmd server and client, is based on a simple
>> text messages. It is impossible to extend the protocol
>> without breaking backward compatibility. The V2 protocol
>> introduced by this patch is a binary message-based protocol
>> and it is able to extend by just adding message tags.
>>
>> <How to test>
>> [1] Backward compatibility checks
>>  We need to test backward compatibility of this patch for old
>>  trace-cmds(client/server). So, this patch was tested for [2]
>>  command checks in following 3 types:
>>
>> 	<client>	<server>
>> 	  new		  old
>> 	  old		  new
>> 	  new		  new
>>
>> [2] Command checks
>> - server (common)
>> 	# trace-cmd listen -p 12345
>>
>> 1) record
>> - client
>> 	# trace-cmd record -e sched -N <server IP>:12345
>> 	^C
>>
>> 2) record + multiple buffers
>> - client
>> 	# trace-cmd record -B foo -e sched -N <server IP>:12345
>> 	^C
>>
>> 3) extract
>> - client
>> 	# ./trace-cmd start -e sched
>> 	# sleep 5
>> 	# ./trace-cmd stop
>> 	# ./trace-cmd extract -N <server IP>:12345
>>
>> 4) extract + snapshot
>> - client
>> 	# ./trace-cmd start -e sched
>> 	# sleep 5
>> 	# ./trace-cmd snapshot -s
>> 	# ./trace-cmd stop
>> 	# ./trace-cmd extract -N <server IP>:12345 -s
>>
>> Signed-off-by: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
>>
> 
> Hi Masami,
> 
> Is this a rewrite of Yoshihiro's work? I would like to keep the From:
> from him if he did the initial work, and add both of your
> signed-off-bys.

OK, but actually he left Hitachi, so his e-mail address is invalid...
Is that OK to use that old email address?

> Note, I did pull in some of his patches to a temp branch and extended
> on them. I'll look to see if I have to modify this code as well.

OK, thanks!

> 
> Thanks!
> 
> -- Steve
> 
> 


-- 
Masami HIRAMATSU
Linux Technology Research Center, System Productivity Research Dept.
Center for Technology Innovation - Systems Engineering
Hitachi, Ltd., Research & Development Group
E-mail: masami.hiramatsu.pt@hitachi.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/