Skip to content

os::get_process_uid and os::rss for BSD #11

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
snake66 wants to merge 7 commits into
base: bsd-port
Choose a base branch
from
Open

os::get_process_uid and os::rss for BSD #11

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
56e6475
Only macOS needs to include mach.h
snake66 May 21, 2026
9e7292f
BSD support for RSS
battleblow Aug 11, 2024
d217060
Fix os::Bsd::get_process_uid() for OpenBSD and NetBSD
bsdkurt Jan 23, 2026
a87298d
Fix NULL -> nullptr in os_bsd.cpp
snake66 Jun 18, 2026
376cedf
Update os::rss() impl for BSD from feedback
snake66 Jul 3, 2026
bd8bdcd
Refactor common code to get proc_info on BSD
snake66 Jul 3, 2026
b3e450f
Fix silly mistakes
snake66 Jul 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
58 changes: 51 additions & 7 deletions src/hotspot/os/bsd/os_bsd.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,6 @@
# include <fcntl.h>
# include <fenv.h>
# include <inttypes.h>
# include <mach/mach.h>
# include <poll.h>
# include <pthread.h>
# include <pwd.h>
Expand Down Expand Up @@ -112,6 +111,7 @@

#ifdef __APPLE__
#include <libproc.h>
#include <mach/mach.h>
#include <mach/task_info.h>
#include <mach-o/dyld.h>

Expand All @@ -124,6 +124,32 @@
#include <pthread_np.h>
#endif

#if defined(__APPLE__)
#define KERN_PROC_MIB KERN_PROC
#define KINFO_PROC_T kinfo_proc
#define KI_UID kp_eproc.e_ucred.cr_uid
#define KI_PID kp_proc.p_pid
#elif defined(__OpenBSD__)
#define KERN_PROC_MIB KERN_PROC
#define KINFO_PROC_T kinfo_proc
#define KI_RSS p_vm_rssize
#define KI_UID p_uid
#define KI_PID p_pid
#elif defined(__FreeBSD__)
#include <sys/user.h>
#define KERN_PROC_MIB KERN_PROC
#define KINFO_PROC_T kinfo_proc
#define KI_RSS ki_rssize
#define KI_UID ki_uid
#define KI_PID ki_pid
#elif defined(__NetBSD__)
#define KERN_PROC_MIB KERN_PROC2
#define KINFO_PROC_T kinfo_proc2
#define KI_RSS p_vm_rssize
#define KI_UID p_uid
#define KI_PID p_pid
#endif

@tstuefe tstuefe Jul 1, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an eyesore. Can this be shortened, and/or moved into a helper include?


#ifndef MAP_ANONYMOUS
#define MAP_ANONYMOUS MAP_ANON
#endif
Expand Down Expand Up @@ -250,6 +276,17 @@ physical_memory_size_type os::Machine::physical_memory() {
return Bsd::physical_memory();
}

static int bsd_get_procinfo(pid_t pid, struct KINFO_PROC_T *kp, size_t *bufSize) {
int mib[] = {CTL_KERN, KERN_PROC_MIB, KERN_PROC_PID, pid,
# if !defined(__FreeBSD__) && !defined(__APPLE__)
static_cast<int>(*bufSize), 1
# endif
};
const u_int namelen = sizeof(mib)/sizeof(mib[0]);

return sysctl(mib, namelen, kp, bufSize, nullptr, 0);
}

size_t os::rss() {
size_t rss = 0;
#ifdef __APPLE__
Expand All @@ -261,6 +298,13 @@ size_t os::rss() {
if (ret == KERN_SUCCESS) {
rss = info.resident_size;
}
#else
struct KINFO_PROC_T kp;
size_t bufSize = sizeof(kp);

if (bsd_get_procinfo(getpid(), &kp, &bufSize) != -1) {
rss = kp.KI_RSS * getpagesize();
}
#endif // __APPLE__

return rss;
Expand Down Expand Up @@ -899,12 +943,12 @@ pid_t os::Bsd::gettid() {

// Returns the uid of a process or -1 on error.
uid_t os::Bsd::get_process_uid(pid_t pid) {
struct kinfo_proc kp;
size_t size = sizeof kp;
int mib_kern[4] = {CTL_KERN, KERN_PROC, KERN_PROC_PID, pid};
if (sysctl(mib_kern, 4, &kp, &size, nullptr, 0) == 0) {
if (size > 0 && kp.kp_proc.p_pid == pid) {
return kp.kp_eproc.e_ucred.cr_uid;
struct KINFO_PROC_T kp;
size_t size = sizeof(kp);

if (bsd_get_procinfo(pid, &kp, &size) == 0) {
if (size > 0 && kp.KI_PID == pid) {
return kp.KI_UID;
}
}
return (uid_t)-1;
Expand Down