Page MenuHomeFreeBSD

vfs: reimplement vlrureclaim to actually use LRU
ClosedPublic

Authored by mjg on Jan 7 2020, 5:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 21, 9:07 PM
Unknown Object (File)
Mon, Nov 18, 5:13 AM
Unknown Object (File)
Sat, Nov 16, 1:48 PM
Unknown Object (File)
Fri, Nov 15, 4:24 PM
Unknown Object (File)
Thu, Nov 7, 6:22 PM
Unknown Object (File)
Oct 23 2024, 2:36 PM
Unknown Object (File)
Oct 5 2024, 4:53 PM
Unknown Object (File)
Oct 5 2024, 3:46 AM
Subscribers

Details

Summary

Global ordering introduced in D22997 makes it possible. Since the current version of the routine discussed issues with it, I did the same with the new routine. I consider it to be enough of an improvement to go in as it is.

Tested by setting kern.maxvnodes=5000 and doing buildkernel. Results were similar to the stock kernel since vnode both vnode pressure is too much AND vlrureclaim only has one mode to go over everything (instead of only part of the list). This will be fixed later.

Test Plan

stress2 and whatnot

Diff Detail

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

Event Timeline

This revision is now accepted and ready to land.Jan 11 2020, 3:18 PM
mjg edited the summary of this revision. (Show Details)
  • fix an accounting bug (returned value got ignored)
  • since this code does not blindly requeue everything just walk the list till the end
  • improvement was a side effect of a bug where vnlru would just not reclaim
This revision now requires review to proceed.Jan 11 2020, 4:30 PM

This change seems like an improvement over what we have now.

This revision is now accepted and ready to land.Jan 13 2020, 5:04 AM
This revision was automatically updated to reflect the committed changes.