<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src/sys/ddb, branch releng/15.0</title>
<subtitle>FreeBSD source tree</subtitle>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/'/>
<entry>
<title>ddb ps: Print again the effective GID, separately</title>
<updated>2025-09-18T08:50:22+00:00</updated>
<author>
<name>Olivier Certner</name>
<email>olce@FreeBSD.org</email>
</author>
<published>2025-08-26T08:56:54+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=960dbe036d853dee08f3d5220c970bb6964baf2f'/>
<id>960dbe036d853dee08f3d5220c970bb6964baf2f</id>
<content type='text'>
Following commit be1f7435ef218b1d ("kern: start tracking cr_gid outside
of cr_groups[]"), cr_groups[] doesn't contain the effective GID anymore.
Fix the 'show proc' DDB command to show it again, and make it stand out
with respect to the supplementary ones.

Fixes:          be1f7435ef218b1d ("kern: start tracking cr_gid outside of cr_groups[]")
MFC after:      9 days
Sponsored by:   The FreeBSD Foundation
Differential Revision:  https://reviews.freebsd.org/D52251

(cherry picked from commit de974a0f1b73e79466c25f3c85fe727004576fea)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Following commit be1f7435ef218b1d ("kern: start tracking cr_gid outside
of cr_groups[]"), cr_groups[] doesn't contain the effective GID anymore.
Fix the 'show proc' DDB command to show it again, and make it stand out
with respect to the supplementary ones.

Fixes:          be1f7435ef218b1d ("kern: start tracking cr_gid outside of cr_groups[]")
MFC after:      9 days
Sponsored by:   The FreeBSD Foundation
Differential Revision:  https://reviews.freebsd.org/D52251

(cherry picked from commit de974a0f1b73e79466c25f3c85fe727004576fea)
</pre>
</div>
</content>
</entry>
<entry>
<title>machine/stdarg.h -&gt; sys/stdarg.h</title>
<updated>2025-06-11T16:39:02+00:00</updated>
<author>
<name>Brooks Davis</name>
<email>brooks@FreeBSD.org</email>
</author>
<published>2025-06-11T16:39:02+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=e453e498cbb88570a3ff7b3679de65c88707da95'/>
<id>e453e498cbb88570a3ff7b3679de65c88707da95</id>
<content type='text'>
Switch to using sys/stdarg.h for va_list type and va_* builtins.

Make an attempt to insert the include in a sensible place.  Where
style(9) was followed this is easy, where it was ignored, aim for the
first block of sys/*.h headers and don't get too fussy or try to fix
other style bugs.

Reviewed by:	imp
Exp-run by:	antoine (PR 286274)
Pull Request:	https://github.com/freebsd/freebsd-src/pull/1595
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Switch to using sys/stdarg.h for va_list type and va_* builtins.

Make an attempt to insert the include in a sensible place.  Where
style(9) was followed this is easy, where it was ignored, aim for the
first block of sys/*.h headers and don't get too fussy or try to fix
other style bugs.

Reviewed by:	imp
Exp-run by:	antoine (PR 286274)
Pull Request:	https://github.com/freebsd/freebsd-src/pull/1595
</pre>
</div>
</content>
</entry>
<entry>
<title>proc: Remove kernel stack swapping support, part 6</title>
<updated>2024-07-29T01:43:18+00:00</updated>
<author>
<name>Mark Johnston</name>
<email>markj@FreeBSD.org</email>
</author>
<published>2024-07-29T01:40:48+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=472888018ce141227f8b019b6663739c36bc608c'/>
<id>472888018ce141227f8b019b6663739c36bc608c</id>
<content type='text'>
- Remove most checks of the P_INMEM flag.
- Some uses remain since a few userspace tools, e.g., ps(1) and top(1)
  expect the flag to be set.  These can be cleaned up but the code has
  most likely been copy-pasted elsewhere and while linger for a long
  time.

Tested by:	pho
Reviewed by:	alc, imp, kib
Differential Revision:	https://reviews.freebsd.org/D46117
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
- Remove most checks of the P_INMEM flag.
- Some uses remain since a few userspace tools, e.g., ps(1) and top(1)
  expect the flag to be set.  These can be cleaned up but the code has
  most likely been copy-pasted elsewhere and while linger for a long
  time.

Tested by:	pho
Reviewed by:	alc, imp, kib
Differential Revision:	https://reviews.freebsd.org/D46117
</pre>
</div>
</content>
</entry>
<entry>
<title>thread: Remove kernel stack swapping support, part 4</title>
<updated>2024-07-29T01:40:22+00:00</updated>
<author>
<name>Mark Johnston</name>
<email>markj@FreeBSD.org</email>
</author>
<published>2024-07-29T01:40:22+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=e24a65528388f4debfb12e936f314f85ba6ac407'/>
<id>e24a65528388f4debfb12e936f314f85ba6ac407</id>
<content type='text'>
- Remove the IS_SWAPPED thread inhibitor state.
- Remove all uses of TD_IS_SWAPPED() in the kernel.
- Remove the TDF_CANSWAP flag.
- Remove the P_SWAPPINGOUT and P_SWAPPINGIN flags.

Tested by:	pho
Reviewed by:	alc, imp, kib
Differential Revision:	https://reviews.freebsd.org/D46115
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
- Remove the IS_SWAPPED thread inhibitor state.
- Remove all uses of TD_IS_SWAPPED() in the kernel.
- Remove the TDF_CANSWAP flag.
- Remove the P_SWAPPINGOUT and P_SWAPPINGIN flags.

Tested by:	pho
Reviewed by:	alc, imp, kib
Differential Revision:	https://reviews.freebsd.org/D46115
</pre>
</div>
</content>
</entry>
<entry>
<title>db_pprint: Properly handle complex pointer types</title>
<updated>2024-07-21T17:31:59+00:00</updated>
<author>
<name>Bojan Novković</name>
<email>bnovkov@FreeBSD.org</email>
</author>
<published>2024-07-21T16:51:22+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=1cbd613f3343c873ace8a56df2e515626a18ef22'/>
<id>1cbd613f3343c873ace8a56df2e515626a18ef22</id>
<content type='text'>
The existing pretty-printing code fails to properly print complex
pointer types. This commit fixes this behaviour by traversing the
chain of CTF types until a base type is encountered.

Approved by: markj (mentor)
Fixes: c21bc6f3c242
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The existing pretty-printing code fails to properly print complex
pointer types. This commit fixes this behaviour by traversing the
chain of CTF types until a base type is encountered.

Approved by: markj (mentor)
Fixes: c21bc6f3c242
</pre>
</div>
</content>
</entry>
<entry>
<title>db_pprint: Fix offset calculation for struct members</title>
<updated>2024-07-21T17:31:48+00:00</updated>
<author>
<name>Bojan Novković</name>
<email>bnovkov@FreeBSD.org</email>
</author>
<published>2024-07-21T16:45:33+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=82f5dfc121391604b079ea96aa14ea71e6b618c9'/>
<id>82f5dfc121391604b079ea96aa14ea71e6b618c9</id>
<content type='text'>
The struct pretty-printing code uses the ctm_offset field in
struct ctf_member_v3 to calculate the address of a struct member.
However, the code treats this as a byte offset rather than the
offset in bits, leading to wrong values being printed.
Fix this by diving with ctm_offset by NBBY.

Approved by: markj (mentor)
Fixes: c21bc6f3c242
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The struct pretty-printing code uses the ctm_offset field in
struct ctf_member_v3 to calculate the address of a struct member.
However, the code treats this as a byte offset rather than the
offset in bits, leading to wrong values being printed.
Fix this by diving with ctm_offset by NBBY.

Approved by: markj (mentor)
Fixes: c21bc6f3c242
</pre>
</div>
</content>
</entry>
<entry>
<title>ddb: make db_error reliably no-return</title>
<updated>2024-07-09T17:59:27+00:00</updated>
<author>
<name>Ryan Libby</name>
<email>rlibby@FreeBSD.org</email>
</author>
<published>2024-07-09T17:59:27+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=f0a7df4a6cc9a795680d580fcabd993379e22ae5'/>
<id>f0a7df4a6cc9a795680d580fcabd993379e22ae5</id>
<content type='text'>
Most code assumes db_error does not return, but according to
kdb_reenter_silent, there may be cases where it could.  Instead, panic
if kdb_reenter_silent returns and mark the routine as __dead2.  This
addresses gcc warnings.

Reported by:	GCC -Wmaybe-uninitialized
Reviewed by:	kib
Differential Revision:	https://reviews.freebsd.org/D45918
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Most code assumes db_error does not return, but according to
kdb_reenter_silent, there may be cases where it could.  Instead, panic
if kdb_reenter_silent returns and mark the routine as __dead2.  This
addresses gcc warnings.

Reported by:	GCC -Wmaybe-uninitialized
Reviewed by:	kib
Differential Revision:	https://reviews.freebsd.org/D45918
</pre>
</div>
</content>
</entry>
<entry>
<title>sys/ddb: Add hardware breakpoint support to ddb</title>
<updated>2024-04-12T09:32:41+00:00</updated>
<author>
<name>Andrew Turner</name>
<email>andrew@FreeBSD.org</email>
</author>
<published>2024-03-12T17:01:26+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=076b64e83ac9e0cfd361a58d5967dfa939fe8139'/>
<id>076b64e83ac9e0cfd361a58d5967dfa939fe8139</id>
<content type='text'>
As with hardware watchpoints add support for hardware breakpoints. The
command is only enabled on architectures that report support for them.
Currently no architectures do, however arm64 will add support in a
future change.

Reviewed by:	jhb (earlier version)
Sponsored by:	Arm Ltd
Sponsored by:	Innovate UK
Differential Revision:	https://reviews.freebsd.org/D22191
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
As with hardware watchpoints add support for hardware breakpoints. The
command is only enabled on architectures that report support for them.
Currently no architectures do, however arm64 will add support in a
future change.

Reviewed by:	jhb (earlier version)
Sponsored by:	Arm Ltd
Sponsored by:	Innovate UK
Differential Revision:	https://reviews.freebsd.org/D22191
</pre>
</div>
</content>
</entry>
<entry>
<title>ddb: Start to generalise breakpoints</title>
<updated>2024-04-12T09:29:13+00:00</updated>
<author>
<name>Andrew Turner</name>
<email>andrew@FreeBSD.org</email>
</author>
<published>2024-03-21T14:11:17+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=f4a6036a3a0d6749a435a4c8decfdb59c64ec619'/>
<id>f4a6036a3a0d6749a435a4c8decfdb59c64ec619</id>
<content type='text'>
To allow for hardware breakpoints it is useful to reuse the same
management code. Start to generalise the code by moving common data
into a new struct and pas this to internal functions to work with.

Sponsored by:	Arm Ltd
Differential Revision:	https://reviews.freebsd.org/D44461
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
To allow for hardware breakpoints it is useful to reuse the same
management code. Start to generalise the code by moving common data
into a new struct and pas this to internal functions to work with.

Sponsored by:	Arm Ltd
Differential Revision:	https://reviews.freebsd.org/D44461
</pre>
</div>
</content>
</entry>
<entry>
<title>ddb: Fix format string errors in db_pprint.c</title>
<updated>2024-04-03T18:14:13+00:00</updated>
<author>
<name>Mark Johnston</name>
<email>markj@FreeBSD.org</email>
</author>
<published>2024-04-03T18:12:45+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=d627293f03c1b5908adc440f2008c02662b94cd5'/>
<id>d627293f03c1b5908adc440f2008c02662b94cd5</id>
<content type='text'>
Reported by:	bapt
Fixes:	aada453dcbaa ("ddb: Properly pretty-print non-labeled enum values")
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Reported by:	bapt
Fixes:	aada453dcbaa ("ddb: Properly pretty-print non-labeled enum values")
</pre>
</div>
</content>
</entry>
</feed>
