Page MenuHomeFreeBSD

vfs: add accessors to ->v_irflag
ClosedPublic

Authored by mjg on Dec 28 2020, 3:28 AM.
Tags
None
Referenced Files
F102952347: D27793.diff
Tue, Nov 19, 3:25 AM
Unknown Object (File)
Sat, Nov 16, 7:22 PM
Unknown Object (File)
Wed, Oct 23, 2:06 AM
Unknown Object (File)
Oct 6 2024, 12:42 AM
Unknown Object (File)
Oct 5 2024, 5:54 AM
Unknown Object (File)
Oct 2 2024, 9:26 PM
Unknown Object (File)
Oct 2 2024, 2:29 PM
Unknown Object (File)
Oct 1 2024, 3:17 AM
Subscribers
None

Details

Summary

The v_irflag field is mostly accessed in an unlocked manner, but no measures were taken to prevent compiler mischief. While here this adds assertions as to state of the field.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

mjg requested review of this revision.Dec 28 2020, 3:28 AM
mjg created this revision.
sys/kern/vfs_subr.c
1824

This is an artifact from coccinelle. Will patch later.

sys/fs/nullfs/null_subr.c
272–273

May be add vn_irflag_set_unlocked() that does VI_LOCK/set/VI_UNLOCK.

sys/sys/vnode.h
990

!= 0

sys/ufs/ufs/ufs_vnops.c
288

This should be _set_cond.

  • better assert
  • more accessors
  • rebase
This revision is now accepted and ready to land.Jan 1 2021, 1:39 PM
This revision was automatically updated to reflect the committed changes.