[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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



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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.