FreeBSD source tree https://cgit-dev.freebsd.org/src/atom?h=releng%2F13.5 2023-10-26T20:38:41Z 2023-10-26T20:38:41Z Brooks Davis brooks@FreeBSD.org 2023-10-26T20:38:41Z urn:sha1:302ff7e6aff0da1ea3b4de79940aac8103e4bb9c Include support for translating 32-bit auxv vectors on non-64-bit platforms that aren't riscv (which has no 32-bit ABI support and probably never will). Reviewed by: markj Sponsored by: DARPA Differential Revision: https://reviews.freebsd.org/D42201 (cherry picked from commit 248fe3d3483cb3ec2c78dd31dc02a467060a6577) 2023-10-26T20:38:41Z Brooks Davis brooks@FreeBSD.org 2023-10-26T20:38:41Z urn:sha1:ea7f064a780b6f6bb023757a5eb8b997bd272687 Use source struct size not the destination struct size so we copy all the auxv entries, not just the first half of them. Fix a style issue on an adjacent line. Reviewed by: markj Sponsored by: DARPA Differential Revision: https://reviews.freebsd.org/D42200 (cherry picked from commit 8f06fabe39ac3ebca4ab448a456945008305a23f) 2023-10-26T20:38:41Z Brooks Davis brooks@FreeBSD.org 2023-10-26T20:38:41Z urn:sha1:efeba8da1d2da9d2c471936415cfad5f63daae76 Making this variable static makes is_elf32_sysctl() and callers thread unsafe. Use a less absurd length for sv_name. The longest name in the system is "FreeBSD ELF64 V2" which tips the scales at 16+1 bytes. We'll almost certainly have other problems if we exceed 32 characters. Reviewed by: markj Sponsored by: DARPA Differential Revision: https://reviews.freebsd.org/D42199 (cherry picked from commit 72a4ee26a7c665ae1c31abe1c6feeaa7ccaba140) 2023-10-26T20:38:41Z Brooks Davis brooks@FreeBSD.org 2023-10-26T20:38:41Z urn:sha1:9fcbd4c0d19cdbfa1fb4711c00c9c9cf84dbefb0 Avoid a weird dance through the union and treat all 32-bit values as unsigned integers. This avoids sign extension of flags and userspace pointers. Reviewed by: markj Sponsored by: DARPA Differential Revision: https://reviews.freebsd.org/D42198 (cherry picked from commit 9735cc0e41825bb9e95d16433d381ffe4c190f38) 2023-10-26T20:38:40Z Brooks Davis brooks@FreeBSD.org 2023-10-26T20:38:40Z urn:sha1:8888b14f1b771e65013601801a82b27ca539a5e0 Style demands a space after the switch keyword. Noticed reviewing code in CheriBSD that propagated the style bug. Reported by: markj Sponsored by: DARPA Reviewed by: kib, markj Differential Revision: https://reviews.freebsd.org/D42041 (cherry picked from commit ccac440f7cbb013de41aa3933f3f7be77225c44f) 2023-08-23T17:43:26Z Warner Losh imp@FreeBSD.org 2023-08-22T01:31:41Z urn:sha1:3d497e17ebd33fe0f58d773e35ab994d750258d6 Remove /^[\s*]*__FBSDID\("\$FreeBSD\$"\);?\s*\n/ Similar commit in main: (cherry picked from commit 1d386b48a555) 2022-04-16T02:18:18Z Konstantin Belousov kib@FreeBSD.org 2022-04-03T14:42:03Z urn:sha1:c649ead622cd59eff72ed8db80a6846565df8e8c (cherry picked from commit 039d1496b0790a27cbec18e06b9494bf41254019) 2022-03-02T21:56:30Z Eric van Gyzen vangyzen@FreeBSD.org 2021-05-27T16:33:22Z urn:sha1:d89523644308afd24d47ca3dcf4098d0a719e919 When collecting kernel stacks for a target process, if the process adds a thread between the two calls to sysctl, ignore the additional threads. Previously, procstat would print only a useless error message. Now, it prints a consistent snapshot of the stacks. We know that snapshot is already stale, but it could still be stale even with a more complex fix to reallocate and retry, so such a fix is hardly worth the effort. Reported by: Daniel.Mitchell@emc.com MFC after: 1 week Sponsored by: Dell EMC Isilon (cherry picked from commit 427f12f150e875c40acb84f292a80bfa0b90a1a2) 2021-12-09T00:23:29Z Konstantin Belousov kib@FreeBSD.org 2021-12-02T18:03:01Z urn:sha1:ed43c7ac0a8c29510a42b56f79a7db32855e9821 PR: 260174 (cherry picked from commit 7a9423d6f360e3758ca67fbb25d309140ea93670) 2021-12-09T00:23:28Z Konstantin Belousov kib@FreeBSD.org 2021-12-02T18:02:26Z urn:sha1:0ab5452ca3a10f1b285af2c98e2b5ea055cf2e53 (cherry picked from commit 0ea3e4a27bc0d1b0fdf876c3ef89d790e29ad5b1)