<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src-test/sys/cddl/dev/dtrace/arm, branch main</title>
<subtitle>FreeBSD source tree</subtitle>
<id>https://cgit-dev.freebsd.org/src-test/atom?h=main</id>
<link rel='self' href='https://cgit-dev.freebsd.org/src-test/atom?h=main'/>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/'/>
<updated>2020-07-21T07:41:36Z</updated>
<entry>
<title>dtrace/fbt: fix return probe arguments on arm</title>
<updated>2020-07-21T07:41:36Z</updated>
<author>
<name>Andriy Gapon</name>
<email>avg@FreeBSD.org</email>
</author>
<published>2020-07-21T07:41:36Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=2032c532aad7def4246282ff51cc36ac6f2c5db0'/>
<id>urn:sha1:2032c532aad7def4246282ff51cc36ac6f2c5db0</id>
<content type='text'>
arg0 should be an offset of the return point within the function, arg1
should be the return value.  Previously the return probe had arguments as
if for the entry probe.

Tested on armv7.

andrew noted that the same problem seems to be present on arm64, mips,
and riscv.
I am not sure if I will get around to fixing those.  So, platform users
or anyone looking to make a contribution please be aware of this
opportunity.

Reviewed by:	markj
MFC after:	1 week
Differential Revision: https://reviews.freebsd.org/D25685
</content>
</entry>
<entry>
<title>Remove scary-looking printf output that happens when you kldload dtrace on</title>
<updated>2020-01-09T22:51:37Z</updated>
<author>
<name>Ian Lepore</name>
<email>ian@FreeBSD.org</email>
</author>
<published>2020-01-09T22:51:37Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=8bfc473c0e62b34f3c9a5887f054e1e10e550e96'/>
<id>urn:sha1:8bfc473c0e62b34f3c9a5887f054e1e10e550e96</id>
<content type='text'>
arm.  Replace it with a comment block explaining why the function is empty
on 32-bit arm.
</content>
</entry>
<entry>
<title>Rename assym.s to assym.inc</title>
<updated>2018-03-20T17:58:51Z</updated>
<author>
<name>Ed Maste</name>
<email>emaste@FreeBSD.org</email>
</author>
<published>2018-03-20T17:58:51Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=fc2a8776a2f9945f22301cd92ec6d3ac1f817028'/>
<id>urn:sha1:fc2a8776a2f9945f22301cd92ec6d3ac1f817028</id>
<content type='text'>
assym is only to be included by other .s files, and should never
actually be assembled by itself.

Reviewed by:	imp, bdrewery (earlier)
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/D14180
</content>
</entry>
<entry>
<title>Corrected misspelled versions of rendezvous.</title>
<updated>2017-04-09T02:00:03Z</updated>
<author>
<name>Patrick Kelsey</name>
<email>pkelsey@FreeBSD.org</email>
</author>
<published>2017-04-09T02:00:03Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=67d955aab4020aa26e0f5812439e85af319da144'/>
<id>urn:sha1:67d955aab4020aa26e0f5812439e85af319da144</id>
<content type='text'>
The MFC will include a compat definition of smp_no_rendevous_barrier()
that calls smp_no_rendezvous_barrier().

Reviewed by:	gnn, kib
MFC after:	1 week
Differential Revision:	https://reviews.freebsd.org/D10313
</content>
</entry>
<entry>
<title>Fix improper use of "its".</title>
<updated>2016-11-08T23:59:41Z</updated>
<author>
<name>Bryan Drewery</name>
<email>bdrewery@FreeBSD.org</email>
</author>
<published>2016-11-08T23:59:41Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=28323add0916d61a7bcea74f30e9b40b837809ce'/>
<id>urn:sha1:28323add0916d61a7bcea74f30e9b40b837809ce</id>
<content type='text'>
Sponsored by:	Dell EMC Isilon
</content>
</entry>
<entry>
<title>Fix include order as required post r308415</title>
<updated>2016-11-07T20:02:18Z</updated>
<author>
<name>Oleksandr Tymoshenko</name>
<email>gonzo@FreeBSD.org</email>
</author>
<published>2016-11-07T20:02:18Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=d30e308465d973014a18687829542e3b40c3e8db'/>
<id>urn:sha1:d30e308465d973014a18687829542e3b40c3e8db</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Make the second argument of dtrace_invop() a trapframe pointer.</title>
<updated>2016-04-17T23:08:47Z</updated>
<author>
<name>Mark Johnston</name>
<email>markj@FreeBSD.org</email>
</author>
<published>2016-04-17T23:08:47Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=6c2806594b351ec39cca31bf386bdee79571fa36'/>
<id>urn:sha1:6c2806594b351ec39cca31bf386bdee79571fa36</id>
<content type='text'>
Currently this argument is a pointer into the stack which is used by FBT
to fetch the first five probe arguments. On all non-x86 architectures it's
simply the trapframe address, so this change has no functional impact. On
amd64 it's a pointer into the trapframe such that stack[1 .. 5] gives the
first five argument registers, which are deliberately grouped together in
the amd64 trapframe definition.

A trapframe argument simplifies the invop handlers on !x86 and makes the
x86 FBT invop handler easier to understand. Moreover, it allows for invop
handlers that may want to modify the register set of the interrupted thread.
</content>
</entry>
<entry>
<title>Remove unused variables dtrace_in_probe and dtrace_in_probe_addr.</title>
<updated>2016-03-17T18:55:54Z</updated>
<author>
<name>Mark Johnston</name>
<email>markj@FreeBSD.org</email>
</author>
<published>2016-03-17T18:55:54Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=48cc2d5e22611c835382b77ce02b1f63e75cc6b9'/>
<id>urn:sha1:48cc2d5e22611c835382b77ce02b1f63e75cc6b9</id>
<content type='text'>
</content>
</entry>
<entry>
<title>As &lt;machine/vmparam.h&gt; is included from &lt;vm/vm_param.h&gt;, there is no</title>
<updated>2016-02-22T09:08:04Z</updated>
<author>
<name>Svatopluk Kraus</name>
<email>skra@FreeBSD.org</email>
</author>
<published>2016-02-22T09:08:04Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=35a0bc126056419b34b7bad9a8488ac20dcdbc8e'/>
<id>urn:sha1:35a0bc126056419b34b7bad9a8488ac20dcdbc8e</id>
<content type='text'>
need to include it explicitly when &lt;vm/vm_param.h&gt; is already included.

Suggested by:	alc
Reviewed by:	alc
Differential Revision:	https://reviews.freebsd.org/D5379
</content>
</entry>
<entry>
<title>Don't re-define LOCORE when dtrace is built-in to the kernel.</title>
<updated>2015-06-10T09:59:26Z</updated>
<author>
<name>Ruslan Bukin</name>
<email>br@FreeBSD.org</email>
</author>
<published>2015-06-10T09:59:26Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=8bd0e175953787f1209a257c622415b6e93cf29c'/>
<id>urn:sha1:8bd0e175953787f1209a257c622415b6e93cf29c</id>
<content type='text'>
</content>
</entry>
</feed>
