From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dario Faggioli Subject: Re: Xentrace on Xilinx ARM Date: Tue, 8 Mar 2016 19:15:02 +0100 Message-ID: <1457460902.3102.282.camel@citrix.com> References: <1457192589.2959.603.camel@citrix.com> <1457440918.3102.193.camel@citrix.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============7762494740459835875==" Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Ben Sanda , "xen-devel@lists.xen.org" Cc: Paul Sujkov , George Dunlap , Julien Grall , "stefano.stabellini" List-Id: xen-devel@lists.xenproject.org --===============7762494740459835875== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="=-b8PE5NqtZZZkrceQ8lPP" --=-b8PE5NqtZZZkrceQ8lPP Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, 2016-03-08 at 18:04 +0000, Ben Sanda wrote: > All, >=20 > To update to the current situation. I have been able to get > xentrace() and > xenalyze working completely (at least as far as I can tell) on ARM. >=20 Great! :-) > For xentrace there were changes to the memory allocation routines to > allow > mapping of the Xen Heap by dom0, correcting the MFN->PFN > translations, adding > the trace buffer initialization to setup.c (init_trace_bufs), and > correcting the > get_cycles() call to provide the system TSC. For the get_cycles() > call I > gathered that was supposed to return the raw tick count, not a > translated > ticks->real time timestamp. I then had to call xenalyze with the core > frequency > defined so the timestamps made sence. >=20 > Paul: Was there anything else you did I missed? >=20 Sorry, I can't really comment, as I know few about this part of the tracing infra, and nothing about ARM. I do encourage Paul to chime in, but, actually, that would probably be even easier when we'll have the code posted. So, you know as they say in Open Source, don't you: release early, release often! :-) > > It's not part of any Xen image. It's a command line tool to be > > used, usually > > but not necessarily, in dom0, build and installed together with the > > other > > tools... At least in my case, for x86 builds and installs.=C2=A0 > For xenalyze I had to modify the makefile to build xenalyze on the > ARM platform > (it was specifically removed from the ARM build). Once that was > corrected I > could find and call it from dom0.=20 > Great again! > It built only locally to Xen though (could > only run from dom0), I could not use it from the native Linux > development > environment (I don't know if you're supposed to be able to? Or since > I'm running > ARM it built for ARM not x86 and thus could not be used natively). >=20 Yeah, well, if both dev and test environments are x86, I've done it a couple of times, but I don't think its critical that we support that. If architectures don't match, I actually think it is _the_right_thing_ that it does not work (as you say yourself)! :-) > I plan to push they changes in as a patch to the mainline if that > seems > reasonable to everyone. >=20 Looking forward to it. :-) Thanks and regards, Dario --=20 <> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) --=-b8PE5NqtZZZkrceQ8lPP Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iEYEABECAAYFAlbfFqYACgkQk4XaBE3IOsQomgCfQ0BKWMJH6gURF2bVJXWbk3sO PrwAn3UVn0eNB7GFoFCJVaFKDA+17Qar =1/mv -----END PGP SIGNATURE----- --=-b8PE5NqtZZZkrceQ8lPP-- --===============7762494740459835875== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVs IG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVuLm9yZwpodHRwOi8vbGlzdHMueGVuLm9y Zy94ZW4tZGV2ZWwK --===============7762494740459835875==--