Page MenuHomeFreeBSD

Move OBJT_VNODE specific code from vm_object_terminate() to vnode_destroy_vobject().
ClosedPublic

Authored by kib on Aug 21 2019, 9:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 15, 2:11 PM
Unknown Object (File)
Sat, Mar 15, 5:33 AM
Unknown Object (File)
Thu, Feb 20, 4:17 PM
Unknown Object (File)
Feb 8 2025, 1:02 PM
Unknown Object (File)
Jan 24 2025, 12:26 AM
Unknown Object (File)
Dec 12 2024, 7:28 PM
Unknown Object (File)
Dec 9 2024, 8:29 AM
Unknown Object (File)
Nov 26 2024, 5:23 AM
Subscribers

Details

Summary

Peter, could you, please, check this change ?

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 25997

Event Timeline

In general I like this but the double setting of OBJ_DEAD does make it harder to detect things like double frees.

Untangle multiple OBJ_DEAD sets.

This revision is now accepted and ready to land.Aug 21 2019, 9:49 PM

Redo the patch back to the initial variant. Note that there is only one place that calls vm_object_terminate_dead(), so it costs nothing to set OBJ_DEAD flag there and make vm_object_terminate() to expect OBJ_DEAD is already set.

This is cleaner IMO, since both terminate_dead and terminate really not needed.

This revision now requires review to proceed.Aug 21 2019, 9:51 PM
This revision is now accepted and ready to land.Aug 21 2019, 10:14 PM
kib added a subscriber: pho.

I ran all of the stress2 tests with D21357.61095.diff on amd64. LGTM.