Date: Sun, 14 Apr 2013 11:18:23 -0400 From: "J.R. Oldroyd" <fbsd@opal.com> To: Alexander Leidinger <Alexander@leidinger.net> Cc: freebsd-current@freebsd.org Subject: Re: DTrace of radeonkms on 9.1 Message-ID: <20130414111823.19e30e63@shibato> In-Reply-To: <20130401143652.00001870@unknown> References: <20130327180714.5beae7d6@shibato> <20130401143652.00001870@unknown>
next in thread | previous in thread | raw e-mail | index | archive | help
--Sig_/cfxMsEF0P/4pgVqjEC0kB6K Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Mon, 1 Apr 2013 14:36:52 +0200 Alexander Leidinger <Alexander@leidinger.= net> wrote: > > On Wed, 27 Mar 2013 18:07:14 -0400 > "J.R. Oldroyd" <fbsd@opal.com> wrote: >=20 > > Is there any known magic involved in getting DTrace to do its thing on > > 9.1-release? > >=20 > > I am trying to use it to debug a memory leak problem with the > > radeonkms driver under 9.x. >=20 > Can you check if you have the same dtrace-problem with -current? I > would expect that 9.1 already has some dtrace-fixes regarding new > probes in run-time loaded modules, this is just to verify this > assumption. >=20 > Assuming there is some kind of dead-lock in this module-load interaction > with dtrace, you could modify the radeonkms module to do it's > initialization magic once a sysctl is set to 1, instead of doing this > magic at module-load time. This way you could load the module, start > the dtrace script and then issue the magic sysctl. >=20 > Bye, > Alexander. >=20 Thanks for the suggestion. The problem I was hoping to use DTrace to debug turned out to be an incompletely back-ported vm_alloc_page_contig() function. This was discovered using careful code analysis. Having improved the back-port of this function, the radeonkms driver is now running fine on 9.1-release. The need for DTtrace has therefore gone away for the time being. That said, the back-port of the VM function is not 100%, yet. I've had to put this on hold due to other commitments, but when I get back to this, I probably need to talk to our VM experts to get a list of commits that will be needed to properly merge that function into 9-stable. -jr --Sig_/cfxMsEF0P/4pgVqjEC0kB6K Content-Type: application/pgp-signature; name=signature.asc Content-Disposition: attachment; filename=signature.asc -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (FreeBSD) iEYEARECAAYFAlFqyMcACgkQls33urr0k4lWgQCeJYAVlvXKHB32kaj02hIg/gKE HRgAnj0uLakPdTIq8oqcb7e9/lbK46QI =U+9J -----END PGP SIGNATURE----- --Sig_/cfxMsEF0P/4pgVqjEC0kB6K--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20130414111823.19e30e63>