<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src/sys/ddb, branch release/13.2.0</title>
<subtitle>FreeBSD source tree</subtitle>
<id>https://cgit-dev.freebsd.org/src/atom?h=release%2F13.2.0</id>
<link rel='self' href='https://cgit-dev.freebsd.org/src/atom?h=release%2F13.2.0'/>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/'/>
<updated>2023-02-06T19:34:37Z</updated>
<entry>
<title>ddb: have 'reset' command use normal reboot path</title>
<updated>2023-02-06T19:34:37Z</updated>
<author>
<name>Mitchell Horne</name>
<email>mhorne@FreeBSD.org</email>
</author>
<published>2023-01-07T18:09:28Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=f0e49ab720d3bc71f82dc1e73db4047f3a03defb'/>
<id>urn:sha1:f0e49ab720d3bc71f82dc1e73db4047f3a03defb</id>
<content type='text'>
This conditionally gives all registered shutdown handlers a chance to
perform the reboot, with cpu_reset() being the fallback. The '\s'
modifier can be used with the command to get the previous behaviour.

The motivation is that some platforms may not be able do anything
meaningful via cpu_reset(), due to a lack of standardized reset
mechanism and/or firmware shortcomings. However, they may have a
separate device driver attached that normally performs the reboot. Such
is the case for some versions of the Raspberry Pi, where reset via PSCI
fails, but the BCM2835 watchdog driver has a shutdown hook.

Reported by:	bz
Reviewed by:	markj (slightly earlier version)
MFC after:	2 weeks
Differential Revision:	https://reviews.freebsd.org/D37981

(cherry picked from commit 5644850620aead7c257a4e3040e20201b510f499)
</content>
</entry>
<entry>
<title>ddb: print the actual syscall name</title>
<updated>2022-11-06T14:54:46Z</updated>
<author>
<name>Mitchell Horne</name>
<email>mhorne@FreeBSD.org</email>
</author>
<published>2022-10-28T21:20:05Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=3e00f4ab107d68d5f7782c8bdeeead41cb06b38b'/>
<id>urn:sha1:3e00f4ab107d68d5f7782c8bdeeead41cb06b38b</id>
<content type='text'>
Some architectures will pretty-print a system call trap in the
backtrace. Rather than printing the symbol, use the syscallname()
function to pull the string from the sv_syscallnames array corresponding
to the process. This simplifies the function somewhat.

Mostly, this will result in dropping the "sys" prefix, e.g. "sys_exit"
will now be printed simply as "exit".

Make two minor tweaks to the function signature: use a u_int for the
syscall number since this is a more correct type (see the 'code' member
of struct syscall_args), and make the thread pointer the first argument.
The latter is more natural and conventional.

Suggested by:   jrtc27
Reviewed by:	jrtc27, markj, jhb
MFC after:	1 week
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/D37200

(cherry picked from commit aba921bd9e1869dae9ae4cc6e0c048f997401034)
</content>
</entry>
<entry>
<title>ddb: de-duplicate decode_syscall()</title>
<updated>2022-11-06T14:54:46Z</updated>
<author>
<name>Mitchell Horne</name>
<email>mhorne@FreeBSD.org</email>
</author>
<published>2022-10-02T22:46:00Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=a4dd26ead25db6d5aa07938ccb83413172c9cf2b'/>
<id>urn:sha1:a4dd26ead25db6d5aa07938ccb83413172c9cf2b</id>
<content type='text'>
Only i386 and amd64 print the decoded syscall name in the backtrace.
This de-duplication facilitates further changes and adoption by other
platforms.

Reviewed by:	jrtc27, markj, jhb
MFC after:	1 week
Differential Revision:	https://reviews.freebsd.org/D36565

(cherry picked from commit 754cb545b68ba0a1643792763d000018ffe2afec)
(cherry picked from commit 6f8a182b15c4c14cfb9462b806a928ddaf47c87e)
(cherry picked from commit 89e5ef8917af900558c2d275f5d5c4c703520ead)
(cherry picked from commit 85b715baae22ca12f2d24e1b2980f3d5b32937b8)
</content>
</entry>
<entry>
<title>ddb: add _FLAGS command variants</title>
<updated>2022-08-11T14:22:57Z</updated>
<author>
<name>Mitchell Horne</name>
<email>mhorne@FreeBSD.org</email>
</author>
<published>2022-06-30T15:58:22Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=53021bbf8255f57a422f9874a108d4a679b180a2'/>
<id>urn:sha1:53021bbf8255f57a422f9874a108d4a679b180a2</id>
<content type='text'>
Provide _FLAGS variants of the various command definition macros, in
anticipation of adding a new flag. This can also be used for some
existing commands which require special flag values.

Reviewed by:	markj
MFC after:	3 days
Sponsored by:	Juniper Networks, Inc.
Sponsored by:	Klara, Inc.
Differential Revision:	https://reviews.freebsd.org/D35581

(cherry picked from commit 7ce58d4e8c0cfcb42ec404aed6185b5eb7eef8a8)
</content>
</entry>
<entry>
<title>db_command.c: use designated initializers</title>
<updated>2022-08-11T14:22:26Z</updated>
<author>
<name>Mitchell Horne</name>
<email>mhorne@FreeBSD.org</email>
</author>
<published>2022-07-05T14:45:47Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=742622502ec0f29e0bea4ee36cefd18fae144a58'/>
<id>urn:sha1:742622502ec0f29e0bea4ee36cefd18fae144a58</id>
<content type='text'>
Provide separate helper macros for regular commands and next-level table
commands as they are mutually exclusive. This ensures proper
initialization of each element and allows us to exclude some redundant
fields, such as specifying .more = NULL for every regular command.

Reviewed by:	markj, jhb
MFC after:	3 days
Sponsored by:	Juniper Networks, Inc.
Sponsored by:	Klara, Inc.
Differential Revision:	https://reviews.freebsd.org/D35580

(cherry picked from commit 8a0994823365740f4d92842c2d6f7f524310ba64)
</content>
</entry>
<entry>
<title>db_command.c: style</title>
<updated>2022-08-11T14:22:26Z</updated>
<author>
<name>Mitchell Horne</name>
<email>mhorne@FreeBSD.org</email>
</author>
<published>2022-07-05T14:44:36Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=e4f615b3e0f564df1dee5b8bc6cd40259bd4c156'/>
<id>urn:sha1:e4f615b3e0f564df1dee5b8bc6cd40259bd4c156</id>
<content type='text'>
Reviewed by:	jhb
MFC after:	3 days
Sponsored by:	Juniper Networks, Inc.
Sponsored by:	Klara, Inc.
Differential Revision:	https://reviews.freebsd.org/D35368

(cherry picked from commit 4f2ad6243f15ef325aa8daa680949b0ee1076c6e)
</content>
</entry>
<entry>
<title>Adjust db_flush_line() definition to avoid clang 15 warning</title>
<updated>2022-07-23T08:56:44Z</updated>
<author>
<name>Dimitry Andric</name>
<email>dim@FreeBSD.org</email>
</author>
<published>2022-07-19T18:31:08Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=08c55de237ed0067781981ef4704480938186104'/>
<id>urn:sha1:08c55de237ed0067781981ef4704480938186104</id>
<content type='text'>
With clang 15, the following -Werror warnings is produced:

    sys/ddb/db_lex.c:94:14: error: a function declaration without a prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes]
    db_flush_line()
                 ^
                  void

This is because db_flush_line() is declared with a (void) argument
list, but defined with an empty argument list. Make the definition match
the declaration.

MFC after:	3 days

(cherry picked from commit 939cb349b2ca4d3fcdc72ab1258eadee5e54881c)
</content>
</entry>
<entry>
<title>kerneldump: remove physical from dump routines</title>
<updated>2022-06-27T19:32:06Z</updated>
<author>
<name>Mitchell Horne</name>
<email>mhorne@FreeBSD.org</email>
</author>
<published>2022-05-13T13:40:05Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=5a96b88f05d1b16637dc6bd7a387f601981ff593'/>
<id>urn:sha1:5a96b88f05d1b16637dc6bd7a387f601981ff593</id>
<content type='text'>
It is unused, especially now that the underlying d_dumper methods do not
accept the argument.

Reviewed by:	markj
MFC after:	2 weeks
Differential Revision:	https://reviews.freebsd.org/D35174

(cherry picked from commit db71383b8836a586051b25c20992066d153c6941)
</content>
</entry>
<entry>
<title>ddb: namespacing of struct command</title>
<updated>2022-06-23T22:19:48Z</updated>
<author>
<name>Mitchell Horne</name>
<email>mhorne@FreeBSD.org</email>
</author>
<published>2022-06-14T16:09:11Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=70bb14b902ffef656a9497acb5f5073b47c25bed'/>
<id>urn:sha1:70bb14b902ffef656a9497acb5f5073b47c25bed</id>
<content type='text'>
'command' is too generic for something specific to the kernel debugger;
change this so it is less likely to collide with local variable names.
Also rename struct command_table to struct db_command_table.

Reviewed by:	markj
MFC after:	1 week
Sponsored by:	Juniper Networks, Inc.
Sponsored by:	Klara, Inc.
Differential Revision:	https://reviews.freebsd.org/D35367

(cherry picked from commit 4ef7db5a7e5c06cd8788e6f653ee955ace44b6c9)
</content>
</entry>
<entry>
<title>Create sys/reg.h for the common code previously in machine/reg.h</title>
<updated>2022-05-12T22:12:59Z</updated>
<author>
<name>Andrew Turner</name>
<email>andrew@FreeBSD.org</email>
</author>
<published>2021-08-27T09:38:40Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=e5e2c7ffa0f7d9ba0874b95924f4d977b116f45b'/>
<id>urn:sha1:e5e2c7ffa0f7d9ba0874b95924f4d977b116f45b</id>
<content type='text'>
Move the common kernel function signatures from machine/reg.h to a new
sys/reg.h. This is in preperation for adding PT_GETREGSET to ptrace(2).

Reviewed by:	imp, markj
Sponsored by:	DARPA, AFRL (original work)
Sponsored by:	The FreeBSD Foundation
Differential Revision:	https://reviews.freebsd.org/D19830

(cherry picked from commit b792434150d66b9b2356fb9a7548f4c7f0a0f16c)
</content>
</entry>
</feed>
