vm: Remove kernel stack swapping support, part 6
ClosedPublic
Actions

Authored by markj on Jul 24 2024, 9:39 PM.

Details

Reviewers

jhb
alc
kib
imp

Commits

rG472888018ce1: proc: Remove kernel stack swapping support, part 6

Summary

Remove most checks of the P_INMEM flag.
Some uses remain since a few userspace tools expect the flag to be set.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Jul 24 2024, 9:39 PM

Herald added subscribers: olce, emaste, imp. · View Herald TranscriptJul 24 2024, 9:39 PM

markj requested review of this revision.Jul 24 2024, 9:39 PM

Harbormaster completed remote builds in B58815: Diff 141341.Jul 24 2024, 9:39 PM

markj added a parent revision: D46116: vm: Remove kernel stack swapping support, part 5.Jul 24 2024, 9:39 PM

markj added a child revision: D46118: vm: Remove kernel stack swapping support, part 7.

Can we document which processes/tools do that?

This revision is now accepted and ready to land.Jul 24 2024, 9:51 PM

In D46117#1051135, @imp wrote:

Can we document which processes/tools do that?

In the commit log message you mean? Sure. It's ps(1) and top(1) mainly, but I'm sure that code's been copy-pasted around.

kib added inline comments.Jul 25 2024, 10:49 AM

sys/fs/procfs/procfs_status.c
124–125	Could this be properly de-indented, please?
sys/kern/sys_process.c
98–99	IMO this macro should go away.

Handle kib's comments.

This revision now requires review to proceed.Jul 25 2024, 1:59 PM

Harbormaster completed remote builds in B58826: Diff 141373.Jul 25 2024, 2:00 PM

kib accepted this revision.Jul 25 2024, 2:06 PM

This revision is now accepted and ready to land.Jul 25 2024, 2:06 PM

alc added inline comments.Jul 26 2024, 7:17 AM

sys/kern/sys_process.c
74	drop ()

markj marked an inline comment as done.Jul 26 2024, 12:59 PM

alc accepted this revision.Jul 27 2024, 12:17 AM

Closed by commit rG472888018ce1: proc: Remove kernel stack swapping support, part 6 (authored by markj). · Explain WhyJul 29 2024, 1:50 AM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG472888018ce1: proc: Remove kernel stack swapping support, part 6.

Revision Contents
Changeset List

Path

Size

sys/

ddb/

db_command.c

10 lines

db_ps.c

2 lines

db_thread.c

7 lines

fs/

procfs/

procfs_status.c

26 lines

i386/

linux/

linux_ptrace_machdep.c

4 lines

kern/

kern_proc.c

5 lines

kern_racct.c

6 lines

sys_process.c

65 lines

sys/

proc.h

2 lines

user.h

2 lines

vm/

vm_swapout.c

7 lines

Diff 141494

View Options

sys/ddb/db_command.c

View Options

sys/ddb/db_ps.c

View Options

sys/ddb/db_thread.c

View Options

sys/fs/procfs/procfs_status.c

View Options

sys/i386/linux/linux_ptrace_machdep.c

View Options

sys/kern/kern_proc.c

View Options

sys/kern/kern_racct.c

View Options

sys/kern/sys_process.c

View Options

sys/sys/proc.h

View Options

sys/sys/user.h

View Options

vm: Remove kernel stack swapping support, part 6ClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 141494

sys/ddb/db_command.c

sys/ddb/db_ps.c

sys/ddb/db_thread.c

sys/fs/procfs/procfs_status.c

sys/i386/linux/linux_ptrace_machdep.c

sys/kern/kern_proc.c

sys/kern/kern_racct.c

sys/kern/sys_process.c

sys/sys/proc.h

sys/sys/user.h

sys/vm/vm_swapout.c

vm: Remove kernel stack swapping support, part 6
ClosedPublic
Actions

Revision Contents
Changeset List