Page MenuHomeFreeBSD
Differential D56863

vm_map_growstack(): give a hint to user that stack was blown out
ClosedPublic
Actions

Authored by kib on May 6 2026, 10:56 PM.
Tags
None
Referenced Files
F163040290: D56863.id177371.diff
Sun, Jul 19, 12:44 PM
Unknown Object (File)
Sat, Jul 18, 8:31 PM
Unknown Object (File)
Sat, Jul 18, 7:00 AM
Unknown Object (File)
Sat, Jul 18, 5:23 AM
Unknown Object (File)
Thu, Jul 16, 2:38 AM
Unknown Object (File)
Wed, Jul 15, 7:21 AM
Unknown Object (File)
Wed, Jul 15, 7:21 AM
Unknown Object (File)
Thu, Jul 9, 10:36 AM
View All 52 Files
Subscribers
imp

Details

Reviewers
alc
markj
Commits
rG18c5a26f8a74: vm_map_growstack(): give a hint to user that stack was blown out
rG642dd17ee943: vm_map_growstack(): consistently use local vars instead of curthread/proc

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib created this revision.May 6 2026, 10:56 PM
Herald added a subscriber: imp. · View Herald TranscriptMay 6 2026, 10:56 PM
kib requested review of this revision.May 6 2026, 10:56 PM
kib added reviewers: alc, markj.May 6 2026, 11:57 PM

RACCT cases are excluded for the reason, they are self-inflicted.

markj added inline comments.May 7 2026, 3:52 PM
sys/vm/vm_map.c
4801

Is it worth including the thread ID too?

kib marked an inline comment as done.May 7 2026, 4:02 PM
kib added inline comments.
sys/vm/vm_map.c
4801

I thought about that, and eventually decided not to, because usually the program is killed. But since you asked, lets add it.

kib updated this revision to Diff 177368.May 7 2026, 4:03 PM
kib marked an inline comment as done.

Print tid.
Consistently use a local variable instead of curproc.

markj accepted this revision.May 7 2026, 4:13 PM
This revision is now accepted and ready to land.May 7 2026, 4:13 PM
Closed by commit rG642dd17ee943: vm_map_growstack(): consistently use local vars instead of curthread/proc (authored by kib). · Explain WhyMay 7 2026, 5:01 PM
This revision was automatically updated to reflect the committed changes.
kib added a commit: rG642dd17ee943: vm_map_growstack(): consistently use local vars instead of curthread/proc.
kib added a commit: rG18c5a26f8a74: vm_map_growstack(): give a hint to user that stack was blown out.
alc added inline comments.May 7 2026, 5:07 PM
sys/vm/vm_map.c
4759

In the same vein as your second commit, this could use vm instead of spelling it out as p->p_vmspace.

Revision Contents
Changeset List

PathSize
sys/
vm/
14 lines

Diff 177371

View Options

sys/vm/vm_map.c

Loading...