Page MenuHomeFreeBSD

Relax vflush FORCECLOSE special vnode to only protect devfs vnodes
ClosedPublic

Authored by cse_cem_gmail_com on Feb 27 2015, 4:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 23, 11:54 PM
Unknown Object (File)
Sep 22 2024, 12:58 PM
Unknown Object (File)
Sep 21 2024, 2:50 PM
Unknown Object (File)
Sep 9 2024, 1:35 AM
Unknown Object (File)
Sep 4 2024, 9:58 PM
Unknown Object (File)
Aug 27 2024, 4:24 AM
Unknown Object (File)
Aug 25 2024, 2:05 AM
Unknown Object (File)
Aug 18 2024, 4:50 PM
Subscribers
None

Details

Summary

The VNASSERT in vflush() FORCECLOSE case is trying to panic early to prevent errors from yanking devices out from under filesystems. After discussion with kib, we probably only care about special vnodes on devfs.

Sponsored by: EMC / Isilon Storage Division

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

cse_cem_gmail_com retitled this revision from to Relax vflush FORCECLOSE special vnode to only protect devfs vnodes.
cse_cem_gmail_com updated this object.
cse_cem_gmail_com edited the test plan for this revision. (Show Details)
cse_cem_gmail_com added a reviewer: kib.
cse_cem_gmail_com updated this object.

Per discussion on IRC. Move extern declaration into sys/vnode.h so we don't need to import devfs.h in vfs_subr.c.

Drop static declaration of vfs_specops entirely; drop static on actual definition.

This revision is now accepted and ready to land.Feb 27 2015, 4:45 PM