Page MenuHomeFreeBSD
Differential D45019

vm: Fix error handling in vm_thread_stack_back()
ClosedPublic
Actions

Authored by markj on Apr 29 2024, 11:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 15, 12:56 PM
Unknown Object (File)
Sun, Sep 6, 12:52 AM
Unknown Object (File)
Sat, Sep 5, 3:51 PM
Unknown Object (File)
Fri, Sep 4, 10:43 PM
Unknown Object (File)
Fri, Sep 4, 10:22 PM
Unknown Object (File)
Thu, Sep 3, 3:41 PM
Unknown Object (File)
Thu, Sep 3, 7:10 AM
Unknown Object (File)
Thu, Sep 3, 3:18 AM
View All Files
Subscribers
imp

Details

Reviewers
bnovkov
alc
kib
Commits
rG661a83f9bf9f: vm: Fix error handling in vm_thread_stack_back()
Summary

vm_object_page_remove() wants to busy the page, but that won't work
here. (Kernel stack pages are always busy.)

Make the error handling path look more like vm_thread_stack_dispose().

Reported by: pho
Fixes: 7a79d0669761 ("vm: improve kstack_object pindex calculation to avoid pindex holes")

Diff Detail

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

Event Timeline

markj created this revision.Apr 29 2024, 11:56 PM
Herald added a subscriber: imp. · View Herald TranscriptApr 29 2024, 11:56 PM
markj requested review of this revision.Apr 29 2024, 11:56 PM
Harbormaster completed remote builds in B57456: Diff 137860.Apr 29 2024, 11:56 PM
kib accepted this revision.Apr 30 2024, 12:19 AM
kib added inline comments.
sys/vm/vm_glue.c
635

This blank line is not needed.

This revision is now accepted and ready to land.Apr 30 2024, 12:19 AM
markj marked an inline comment as done.Apr 30 2024, 12:20 AM
bnovkov accepted this revision.Apr 30 2024, 7:29 AM
Closed by commit rG661a83f9bf9f: vm: Fix error handling in vm_thread_stack_back() (authored by markj). · Explain WhyApr 30 2024, 1:46 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG661a83f9bf9f: vm: Fix error handling in vm_thread_stack_back().

Revision Contents
Changeset List

PathSize
sys/
vm/
6 lines

Diff 137891

View Options

sys/vm/vm_glue.c

Loading...