Page MenuHomeFreeBSD

vfs: refactor vnode allocation
ClosedPublic

Authored by mjg on Jan 13 2020, 8:44 PM.

Details

Summary

Semantics are preserved. One benefit is that there are fewer accesses to both numvnodes and freevnodes.

Diff Detail

Repository
rS FreeBSD src repository
Lint
Lint Skipped
Unit
Unit Tests Skipped
Build Status
Buildable 28644

Event Timeline

mjg created this revision.Jan 13 2020, 8:44 PM
jeff accepted this revision.Jan 15 2020, 11:45 PM

I do not object to this patch but I would be more comfortable with kib reviewing.

This revision is now accepted and ready to land.Jan 15 2020, 11:45 PM
kib accepted this revision.Jan 16 2020, 2:01 PM
kib added inline comments.
sys/kern/vfs_subr.c
1508

Small change is that cyclecount was zeroed before when numvnodes < desiredvnodes.

1537

Style recommends putting binary op on the previous line.

1545

You can unlock vnode_list_mtx before wakeup.

This revision was automatically updated to reflect the committed changes.