Re: [Xen-devel] [PATCH v2 1/8] xenalyze: add to tools/xentrace/xenalyze

[George Dunlap <george.dunlap@xxxxxxxxxxxxx>]

On 05/11/2015 03:23 PM, Ian Campbell wrote:
> On Mon, 2015-05-11 at 16:15 +0200, Olaf Hering wrote:
>> On Thu, May 07, George Dunlap wrote:
>>
>>> On Thu, May 7, 2015 at 11:35 AM, Olaf Hering <olaf@xxxxxxxxx> wrote:
>>
>>>>  tools/xentrace/xenalyze/Makefile   |    21 +
>>
>>> I personally think it would make more sense to just put the xenalyze
>>> directory in tools/.
> 
> What (if anything) is the relationship between xentrace and xenalyze?
> 
> I think the latter post processes the result of the former, is that
> right?

Yes.  So someone might argue that, both having to do with the xen
tracing system, they should live in the same directory; that's where
xentrace_format lives, after all.

However, all xentrace does is basically map the hypervisor buffers and
dump them to disk.  Other than that, there is no functional overlap
between them.

It looks like olaf is only checking in the core xenalyze binary, not the
other side bits (like the html documentation), so keeping them in the
same directory might make sense.

 -George

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel